Chromium change:
https://github.com/chromium/chromium/commit/8fe1cbc3480be673ef4fe7fd3f71a13b97a6ef0c
Incognito Reauth: Fix reauth screen not restored when activity is killed
Under memory pressure (emulated by the "Don't keep activities" setting), IncognitoReauthController gets killed after starting the reauthentication flow. When restored, savedInstanceState is checked for incognitoReauthPending when re-initializing the IncognitoReauthController, to determine whether to resume the reauthentication flow.
The problem lies when the SavedInstanceState is reset in
ChromeTabbedActivity::onStartWithNative which sets it to null before it
is checked for the incognitoReauthPending key in
RootUiCoordinator::onFinishNativeInitialization, making
mIsIncognitoReauthPendingOnRestore in IncognitoReauthController always
return false and the reauth screen not getting restored when the
activity is killed.
This CL saves the incognitoReauthPending key value from
SavedInstanceState in the the constructor of RootUiCoordinator before
it is reset.
(cherry picked from commit 318d0b3)
Demo: https://drive.google.com/file/d/16nBDAFA3bIk065EsGGmynMLTY_zU7Uop/view?usp=sharing
Bug: 1365633
Chromium change:
https://github.com/chromium/chromium/commit/debfcd6f15364600b8791da5fe285b20d07f64f5
[Fixit] Remove BakedBezierInterpolator
Interpolators.java fully replaces BakedBezierInterpolator which is
deprecated. This should save a small amount of memory/binary size
by removing the pre-defined arrays. All Interpolators are now shared
:D
This did require moving Interpolator.java for deps reasons, but that
should be fine.
Low-Coverage-Reason: Large scale change
Bug: 1454611
Otherwise there is an error:
Warning in ../../brave/android/java/proguard.flags at line 46, column 1:
Proguard configuration rule does not match anything: `-keep,allowrepackaging,allowobfuscation,allowshrinking class * extends com.google.gson.reflect.TypeToken {
<init>();
}`
Chromium change:
https://github.com/chromium/chromium/commit/008f790110002c3427867eea0acfbe23cef83309
Remove dependencies on SyncService in //chrome and move to //components.
Deletes the instance in //chrome.
Inlines the SyncServiceWrapper into FakeSyncServiceImpl.
Where possible, reuses the profile in the existing code to reduce the
usage of the deprecated method that does not take the Profile reference.
Low-Coverage-Reason: Mechanical refactoring
BUG=1380925
Change-Id: I43853ab63e39c53b8d1408f51c998d076d11a7ad
Chromium change:
https://github.com/chromium/chromium/commit/7acb8d6bc8401edde79c01e49f6764f821aded8f
Reland "JNI: Only using 1 registration per apk"
This reverts commit 6f773ca.
Reason for revert: Now I'm keeping around the extra jni_registration
until I can migrate all uses off of it.
Original change's description:
> Revert "JNI: Only using 1 registration per apk"
>
> This reverts commit a74d255.
>
> Reason for revert: broke internal bots
>
> Original change's description:
> > JNI: Only using 1 registration per apk
> >
> > We now associate a JNI registration with a native library, via our new
> > native_with_jni template. We no longer have a generate_jni_registration
> > happening with pure Java in the apk template.
> >
> > We now only output 1 GEN_JNI per srcjar, and instead of collecting all
> > JNI from modules and outputting many GEN_JNIs in one srcjar.
> >
> > Bug: 1406611
> > Change-Id: Iec40dc7c443e206f177fa8a7d5f9635cefa4c747
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4219332
> > Commit-Queue: Sam Maier <smaier@chromium.org>
> > Owners-Override: Sam Maier <smaier@chromium.org>
> > Reviewed-by: Andrew Grieve <agrieve@chromium.org>
> > Cr-Commit-Position: refs/heads/main@{#1142236}
>
> Bug: 1406611
> Change-Id: Ifedf95de229bd16039ae026dfe3ccc90b66f154c
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4518235
> Commit-Queue: Lijin Shen <lazzzis@google.com>
> Auto-Submit: Sam Maier <smaier@chromium.org>
> Reviewed-by: Lijin Shen <lazzzis@google.com>
> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Owners-Override: Lijin Shen <lazzzis@google.com>
> Cr-Commit-Position: refs/heads/main@{#1142294}
Bug: 1406611
* implement buy screen
* fix style
* Open buy link on android and clean up
* add test
* - Update id (being used by deposit PR) and comment
* fix button width (width is still more than spec)
* update file name to match desktop style
* clean up
* fix security warning with 'noopener'
* clean up and add docs per deposit PR review
* remove router
* update log
* Separate android buy directory to generate resources
* Reduce redundant test code and formatting
Expose deposit Web UI on Android.
The new deposit page can be accessed by navigating to `brave://wallet/deposit-funds` or tapping on the FAB in the Wallet section and selecting Deposit.
Implements a full screen activity to add custom assets or NFTs.
Fix some minor issues to make it consistant with iOS and Desktop implementation:
* For Solana tokens the label `Token address` becomes `Mint address`.
* The new UI got rid of the cancel button as the same action can be achieved by tapping on the back arrow icon or using the system navigation controls.
* Token ID for ERC-721 is not shown for Solana NFTs.
* Text watcher has been improved moving its implementation into a class interface.
* Minor improvements to `NetworkSpinnerAdapter` class.
Artem Samoilenko