[cr146][Android] profileSupplier arg at TopToolbarCoordinator.ctor
Chromium change: https://source.chromium.org/chromium/chromium/src/+/cb00be75174e655ef5bafa4cbd9403923bea0d1b [Signin][Android] Create Signin Button skeleton & flag flow Introduces the SigninButton component for the Android NTP in the toolbar. Currently, this component does nothing and is behind the SigninLevelUpButton feature flag. This CL sets up the flag logic such that, when enabled, the IdentityDisc (which will eventually be replaced by the SigninButton) is removed from the Adaptive Toolbar and the SigninButton is added to the Top Toolbar. There is no View or XML for SigninButton currently as it is just a skeleton component. The implementation for SigninButton will be in follow-up CLs. Bug: 478828569 Change-Id: Ibbf7d61687e18f12fd9d6a6401fe4da8c414b599 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7511406
This commit is contained in:
+5
-2
@@ -24,6 +24,7 @@ import org.chromium.chrome.browser.browser_controls.BrowserStateBrowserControlsV
|
||||
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.profiles.Profile;
|
||||
import org.chromium.chrome.browser.tab.Tab;
|
||||
import org.chromium.chrome.browser.tab.TabObscuringHandler;
|
||||
import org.chromium.chrome.browser.tabmodel.IncognitoStateProvider;
|
||||
@@ -97,7 +98,8 @@ public class BraveTopToolbarCoordinator extends TopToolbarCoordinator {
|
||||
@Nullable HomeButtonDisplay homeButtonDisplay,
|
||||
TopControlsStacker topControlsStacker,
|
||||
BrowserControlsVisibilityManager browserControlsVisibilityManager,
|
||||
Supplier<Integer> incognitoWindowCountSupplier) {
|
||||
Supplier<Integer> incognitoWindowCountSupplier,
|
||||
MonotonicObservableSupplier<Profile> profileSupplier) {
|
||||
super(
|
||||
controlContainer,
|
||||
toolbarLayout,
|
||||
@@ -134,7 +136,8 @@ public class BraveTopToolbarCoordinator extends TopToolbarCoordinator {
|
||||
homeButtonDisplay,
|
||||
topControlsStacker,
|
||||
browserControlsVisibilityManager,
|
||||
incognitoWindowCountSupplier);
|
||||
incognitoWindowCountSupplier,
|
||||
profileSupplier);
|
||||
|
||||
mBraveToolbarLayout = toolbarLayout;
|
||||
mBraveMenuButtonCoordinator = browsingModeMenuButtonCoordinator;
|
||||
|
||||
@@ -1611,7 +1611,8 @@ public class BytecodeTest {
|
||||
HomeButtonDisplay.class,
|
||||
TopControlsStacker.class,
|
||||
BrowserControlsVisibilityManager.class,
|
||||
Supplier.class));
|
||||
Supplier.class,
|
||||
MonotonicObservableSupplier.class));
|
||||
Assert.assertTrue(
|
||||
constructorsMatch(
|
||||
"org/chromium/chrome/browser/toolbar/menu_button/MenuButtonCoordinator", // presubmit: ignore-long-line
|
||||
|
||||
Reference in New Issue
Block a user