[cr142][Android] New arg topControlsStacker at TopToolbarCoordinator.ctor
Chromium change: https://source.chromium.org/chromium/chromium/src/+/7849fe4a5084ba98d8b3e38acac8ce86589d855b [Bookmarks Bar] Hide the Android widget hairline when not scrolled This CL completes the work of correcting the hairline of the toolbar to account for bookmark bar visibility. We add a new method for TopControlLayer's to check if they are the bottom layer in the stack by calling the TopControlsStacker. This can be used by the Toolbar to hide the hairline when it is not at the bottom. We call this whenever the top controls layer height changes. We update the Android widget of the hairline to hide when not scrolled, to match the behavior of the scene layer, which hides its drop shadow when the bookmark bar is visible. Bug: 394614019 Change-Id: Ib212bfb7732e4b619f4155b187d4445c05d5499b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6955916
This commit is contained in:
+5
-2
@@ -17,6 +17,7 @@ import org.chromium.base.supplier.OneshotSupplier;
|
||||
import org.chromium.build.annotations.Nullable;
|
||||
import org.chromium.chrome.R;
|
||||
import org.chromium.chrome.browser.browser_controls.BrowserStateBrowserControlsVisibilityDelegate;
|
||||
import org.chromium.chrome.browser.browser_controls.TopControlsStacker;
|
||||
import org.chromium.chrome.browser.fullscreen.FullscreenManager;
|
||||
import org.chromium.chrome.browser.layouts.LayoutStateProvider;
|
||||
import org.chromium.chrome.browser.tab.Tab;
|
||||
@@ -89,7 +90,8 @@ public class BraveTopToolbarCoordinator extends TopToolbarCoordinator {
|
||||
@Nullable BackButtonCoordinator backButtonCoordinator,
|
||||
@Nullable ForwardButtonCoordinator forwardButtonCoordinator,
|
||||
@Nullable HomeButtonDisplay homeButtonDisplay,
|
||||
@Nullable ExtensionToolbarCoordinator extensionToolbarCoordinator) {
|
||||
@Nullable ExtensionToolbarCoordinator extensionToolbarCoordinator,
|
||||
TopControlsStacker topControlsStacker) {
|
||||
super(
|
||||
controlContainer,
|
||||
toolbarLayout,
|
||||
@@ -123,7 +125,8 @@ public class BraveTopToolbarCoordinator extends TopToolbarCoordinator {
|
||||
backButtonCoordinator,
|
||||
forwardButtonCoordinator,
|
||||
homeButtonDisplay,
|
||||
extensionToolbarCoordinator);
|
||||
extensionToolbarCoordinator,
|
||||
topControlsStacker);
|
||||
|
||||
mBraveToolbarLayout = toolbarLayout;
|
||||
mBraveMenuButtonCoordinator = browsingModeMenuButtonCoordinator;
|
||||
|
||||
Reference in New Issue
Block a user