[cr149][Android] Plumb LocationBarFocusScrimHandler to LocationBarMediator
Chromium changes: https://chromium.googlesource.com/chromium/src/+/9968c93603137808ad94865feef269431b0c6db1 commit 9968c93603137808ad94865feef269431b0c6db1 Author: Tomasz Wiszkowski <ender@google.com> Date: Mon Apr 20 15:03:13 2026 -0700 Plumb LocationBarFocusScrimHandler to LocationBarMediator. This change should allow us to invoke Scrim when we need it. Bug: 485180312 Change-Id: I0bb1f203d0e15c4948a61cedb4bb4c5e328b7d1a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7773078 Commit-Queue: Tomasz Wiszkowski <ender@google.com> Reviewed-by: Sky Malice <skym@chromium.org> Auto-Submit: Tomasz Wiszkowski <ender@google.com> Reviewed-by: Patrick Noland <pnoland@chromium.org> Cr-Commit-Position: refs/heads/main@{#1617792}
This commit is contained in:
@@ -99,7 +99,8 @@ public class BraveLocationBarCoordinator extends LocationBarCoordinator {
|
||||
Function<Tab, @Nullable Bitmap> tabFaviconFunction,
|
||||
SnackbarManager snackbarManager,
|
||||
View bottomContainerView,
|
||||
@Nullable OmniboxChipManager omniboxChipManager) {
|
||||
@Nullable OmniboxChipManager omniboxChipManager,
|
||||
@Nullable LocationBarFocusScrimHandler scrimHandler) {
|
||||
super(
|
||||
locationBarLayout,
|
||||
autocompleteAnchorView,
|
||||
@@ -140,7 +141,8 @@ public class BraveLocationBarCoordinator extends LocationBarCoordinator {
|
||||
tabFaviconFunction,
|
||||
snackbarManager,
|
||||
bottomContainerView,
|
||||
omniboxChipManager);
|
||||
omniboxChipManager,
|
||||
scrimHandler);
|
||||
|
||||
if (mUrlBar != null) {
|
||||
((UrlBar) mUrlBar).setSelectAllOnFocus(true);
|
||||
|
||||
@@ -86,7 +86,8 @@ public class BraveLocationBarMediator extends LocationBarMediator {
|
||||
@Nullable PageZoomIndicatorCoordinator pageZoomIndicatorCoordinator,
|
||||
FuseboxCoordinator fuseboxCoordinator,
|
||||
LocationBarEmbedder locationBarEmbedder,
|
||||
@Nullable OmniboxChipManager omniboxChipManager) {
|
||||
@Nullable OmniboxChipManager omniboxChipManager,
|
||||
@Nullable LocationBarFocusScrimHandler scrimHandler) {
|
||||
super(
|
||||
context,
|
||||
locationBarLayout,
|
||||
@@ -109,7 +110,8 @@ public class BraveLocationBarMediator extends LocationBarMediator {
|
||||
pageZoomIndicatorCoordinator,
|
||||
fuseboxCoordinator,
|
||||
locationBarEmbedder,
|
||||
omniboxChipManager);
|
||||
omniboxChipManager,
|
||||
scrimHandler);
|
||||
}
|
||||
|
||||
public static Class<OmniboxUma> getOmniboxUmaClass() {
|
||||
|
||||
@@ -105,6 +105,7 @@ import org.chromium.chrome.browser.omnibox.DeferredIMEWindowInsetApplicationCall
|
||||
import org.chromium.chrome.browser.omnibox.LocationBarDataProvider;
|
||||
import org.chromium.chrome.browser.omnibox.LocationBarEmbedder;
|
||||
import org.chromium.chrome.browser.omnibox.LocationBarEmbedderUiOverrides;
|
||||
import org.chromium.chrome.browser.omnibox.LocationBarFocusScrimHandler;
|
||||
import org.chromium.chrome.browser.omnibox.LocationBarLayout;
|
||||
import org.chromium.chrome.browser.omnibox.OmniboxChipManager;
|
||||
import org.chromium.chrome.browser.omnibox.OverrideUrlLoadingDelegate;
|
||||
@@ -1815,7 +1816,8 @@ public class BytecodeTest {
|
||||
Function.class,
|
||||
SnackbarManager.class,
|
||||
View.class,
|
||||
OmniboxChipManager.class));
|
||||
OmniboxChipManager.class,
|
||||
LocationBarFocusScrimHandler.class));
|
||||
Assert.assertTrue(
|
||||
constructorsMatch(
|
||||
"org/chromium/chrome/browser/omnibox/LocationBarMediator",
|
||||
@@ -1841,7 +1843,8 @@ public class BytecodeTest {
|
||||
PageZoomIndicatorCoordinator.class,
|
||||
FuseboxCoordinator.class,
|
||||
LocationBarEmbedder.class,
|
||||
OmniboxChipManager.class));
|
||||
OmniboxChipManager.class,
|
||||
LocationBarFocusScrimHandler.class));
|
||||
Assert.assertTrue(
|
||||
constructorsMatch(
|
||||
"org/chromium/chrome/browser/AppHooks",
|
||||
|
||||
Reference in New Issue
Block a user