Fixes errors like
../../chrome/android/java/res/layout/binance_deposit_layout.xml:39: Error: Must use app:tint instead of android:tint [UseAppTint]
android:tint="@android:color/white"
Chromium change:
https://chromium.googlesource.com/chromium/src/+/33026f0b97add89f468693c385db94d821e55b6e
Android: Enable custom lint checks from aars (reland)
Some aar files (e.g. androidx) that chrome depends on contain a lint.jar
file and/or a annotations.zip file. The lint.jar file includes custom
android lint checks. The annotations.zip file contains @IntDef and other
annotations needed for lint checks that are usually available by source
but are not available in aars.
By using these files, we can make our existing lint checks more
comprehensive as well as benefit from lint checks by the aar library
authors.
See newly ignored checks in lint-baseline.xml which is updated in this
CL as an example. Actually fixing these errors is left for future CLs
in order to enable them quicker and avoid unnecessary relands.
Original CL: https://crrev.com/c/2510793
Fixed:
- Avoid ../../ prefix in temporary extraction of aar files.
Tbr: Reland with trivial fix.
Bug: 1078229
Chromium change:
https://chromium.googlesource.com/chromium/src/+/88757c64b874d89657ba1e2a4c53120d17fc84ff
Toolbar: HomeButton without HomepageManager dependency
Made HomeButton work without dependency on chrome.browser.homepage
classes. All the deps are now deferred to ToolbarManager.
The dependencies are injected not through ctor but init method
introduced in this CL since the home button and the enclosing view
(ToolbarLayout) are instantiated from xml.
Menu handling logic was moved to HomepageManager to make it test-
friendly.
Also removed a bunch of dead code in HomeButton. Several public setters
(ActivityTabObserver, ThemeColorProvider, StartSurfaceSupplier) were
used by BrowsingModeBottomToolbarCoordinator only (see
https://crrev.com/c/1981099). There's no callsite left any more as
the class was already gone.
Bug: 1127732
Chromium change:
https://chromium.googlesource.com/chromium/src/+/88757c64b874d89657ba1e2a4c53120d17fc84ff
Toolbar: HomeButton without HomepageManager dependency
Made HomeButton work without dependency on chrome.browser.homepage
classes. All the deps are now deferred to ToolbarManager.
The dependencies are injected not through ctor but init method
introduced in this CL since the home button and the enclosing view
(ToolbarLayout) are instantiated from xml.
Menu handling logic was moved to HomepageManager to make it test-
friendly.
Also removed a bunch of dead code in HomeButton. Several public setters
(ActivityTabObserver, ThemeColorProvider, StartSurfaceSupplier) were
used by BrowsingModeBottomToolbarCoordinator only (see
https://crrev.com/c/1981099). There's no callsite left any more as
the class was already gone.
Bug: 1127732
Chromium change:
https://chromium.googlesource.com/chromium/src/+/1edd6aa74a4f0835f65483d27929717eaadff0b9
Video IPH : Added video iph card
This CL adds:
1 - VideoIPHCreator that takes a ViewStub and inflates into an IPH
2 - A manager class on NTP to manage showing various video tutorial IPHs
3 - Added IPH features for search and download
4 - Moved VideoTutorialServiceFactory package to upper level
TODO:
- The IPH layout isn't upto spec. Needs one more pass.
- The IPH configs aren't fully correct. Needs one more pass. I added
them here for the sake of review clarity and completeness.
Bug: 1117147
Deep