Chromium change:
https://source.chromium.org/chromium/chromium/src/+/9ada9b92ec00dedb31c75b5e8e77d08a3290b19c
[E2E] Observe TopInsetProvider directly in dependent components
This CL refactors the edge-to-edge top inset observation pattern by
having ToolbarPositionController and AutocompleteCoordinator observe
TopInsetProvider directly, instead of receiving callbacks forwarded
through ToolbarManager.
Previously, ToolbarManager observed TopInsetProvider and forwarded
onToEdgeChange notifications to ToolbarPositionController and
LocationBarCoordinator (which then forwarded to
AutocompleteCoordinator). This created unnecessary coupling and
indirection.
This CL does not introduce any visible behavior changes.
Changes:
- ToolbarPositionController now directly observes TopInsetProvider
- AutocompleteCoordinator now directly observes TopInsetProvider
- Removed intermediate forwarding methods from ToolbarManager and
LocationBarCoordinator
- Added TopInsetProviderSupplier parameter to LocationBarCoordinator
and AutocompleteCoordinator constructors
Bug: 476788729
Change-Id: Ic7344e896ebd8c9e04a821d1cd6df6775255e69c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7487625
AlexeyBarabash
09a0066e4d