Chromium change:
https://github.com/brave/chromium/commit/74c1833620fbc342306cf9e1df0c457c8a2fa253
Make IntentHandler stateless
IntentHandler is made more confusing by most of it being stateless
statics, and some of it being stateful for an Activity. There's no
reason for it to be stateful, so simplify by making it stateless.
Change-Id: Ic4d6e3c823c149f2a0ae027503dc4c7516cceaaa
* Tweak primary text color and action bar background color
* Tweak chip selector colors
* Optimize text color attributes
* Network selector icon size
* Tweak Assets text view
* Apply new Wallet background color
* Tweak Wallet background color and toolbar background color
* Add divider lines below toolbar and above navigation bar
* Tweak bottom navigation view style and minor improvements
* Apply code formatting
* Remove duplicated style
Chromium change:
https://github.com/brave/chromium/commit/795632a1e53a0b1cf760238b2d5c7cb392fc0f22
Strip GestureDetector.
The change removes GestureDetector used to differentiate between
Omnibox taps and long-presses.
Long-press gesture has lost all its meaning back when we introduced
Search Ready Omnibox and Clipboard support: the omnibox comes up empty
(meaning the user cannot use the gesture to select all its content),
and cannot use this gesture to directly paste into the omnibox (it's
never been directly supported; we have dedicated actions for that).
As such, the long-press on the Omnibox does nothing else than focus
the Omnibox.
The change simplifies metrics reporting and obsoletes the
OmniboxFocusReason.OmniboxLongClick action.
The change further simplifies the focus flow by removing "set and
reset omnibox text": upon Focus event, LocationBarMediator no longer
sets the Omnibox text content (which next had to be cleared by the
EditUrlSuggestionProcessor). Instead, the LocationBarMediator supplies
an empty input right from the start. This works the same way the
current logic does, skipping one extra content setting, and
simplifying some of the Omnibox interfaces.
Change-Id: I067d538c2c0f424603ed520f6d9fbd06c5f5da6f
Chromium change:
https://github.com/brave/chromium/commit/698cb6c47e3e1005941f5b4e0289ba594454a4c7
[APS] Trigger zero-prefix dropdown on tapping the omnibox after it is
auto-focused
Opening the NTP automatically focuses on the omnibox with a blinking
cursor and without triggering the zero-prefix dropdown when a physical
keyboard is connected to a tablet device.
This CL adds support to show the zero-prefix dropdown when the omnibox
is tapped at this point.
This CL also renames/repurposes AutocompleteMediator#onUrlFocusChange() to report metrics and activate/deactivate the autocomplete session as follows:
1. The autocomplete session will not be activated when the omnibox is initially focused. This will be done only if the user taps on / types into the omnibox after such focus.
2. If the user chooses not to interact with the omnibox after such focus, the autocomplete session deactivation logic in this method will not be executed.
Bug: 1445822
Chromium changes:
https://github.com/brave/chromium/commit/f300646db04f84970e8d67045ce05dda5b968a90
[Tab Switcher] Remove isGridTabSwitcherEnabled
Now that M117 is branched and TabGroupsContinuationAndroid is the
default everywhere the old flags for the initial development of the Grid
Tab Switcher can finally be cleaned up.
This is the fourth step which removes the isGridTabSwitcherEnabled
method everywhere. As a result there are also a number of cascading
side effects in distant files.
Bug: 1466158
https://github.com/brave/chromium/commit/9724e9d8e7a2c1b07cd20964de56ab95bbdda72f
[Tab Switcher] Remove isTabGroupsAndroidEnabled
Now that M117 is branched and TabGroupsContinuationAndroid is the
default everywhere the old flags for the initial development of the Grid
Tab Switcher can finally be cleaned up.
This is the fifth step which removes the isTabGroupsAndroidEnabled
method everywhere. As a result there are also a number of cascading
side effects outside the tab switcher proper.
Bug: 1466158
Artem Samoilenko