[cr144][Android] New arg isWebApp at MenuButtonCoordinator.ctor
Chromium change: https://source.chromium.org/chromium/chromium/src/+/11cd4d96fe20bbb08611bf54cb7e5272e5741707 [MinimalUi_MenuButton] Add setBackgroundInsets for menuButton This is required for ensuring that the menu button on the WebAppHeader is inline with system controls in Samsung Dex https://hsv.googleplex.com/6483804729376768 Bug: 445700593 Change-Id: Id85bb648699f39eb8af8583918eea561530d36d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7128934
This commit is contained in:
+5
-3
@@ -42,10 +42,11 @@ public class BraveMenuButtonCoordinator extends MenuButtonCoordinator {
|
||||
Supplier<Boolean> isInOverviewModeSupplier,
|
||||
ThemeColorProvider themeColorProvider,
|
||||
IncognitoStateProvider incognitoStateProvider,
|
||||
Supplier<MenuButtonState> menuButtonStateSupplier,
|
||||
Supplier<@Nullable MenuButtonState> menuButtonStateSupplier,
|
||||
Runnable onMenuButtonClicked,
|
||||
@IdRes int menuButtonId,
|
||||
@Nullable VisibilityDelegate visibilityDelegate) {
|
||||
@Nullable VisibilityDelegate visibilityDelegate,
|
||||
boolean isWebApp) {
|
||||
super(
|
||||
activity,
|
||||
appMenuCoordinatorSupplier,
|
||||
@@ -60,7 +61,8 @@ public class BraveMenuButtonCoordinator extends MenuButtonCoordinator {
|
||||
menuButtonStateSupplier,
|
||||
onMenuButtonClicked,
|
||||
menuButtonId,
|
||||
visibilityDelegate);
|
||||
visibilityDelegate,
|
||||
isWebApp);
|
||||
|
||||
mActivity = windowAndroid.getActivity().get();
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user