Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade from Chromium 102 to Chromium 103 (1.40.x) #13755

Merged
merged 4 commits into from
Jun 16, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
4 changes: 2 additions & 2 deletions android/brave_java_sources.gni
Original file line number Diff line number Diff line change
Expand Up @@ -287,6 +287,7 @@ brave_java_sources = [
"../../brave/android/java/org/chromium/chrome/browser/site_settings/BraveWalletEthereumConnectedSitesPreference.java",
"../../brave/android/java/org/chromium/chrome/browser/site_settings/DesktopModePreferences.java",
"../../brave/android/java/org/chromium/chrome/browser/site_settings/PlayYTVideoInBrowserPreferences.java",
"../../brave/android/java/org/chromium/chrome/browser/suggestions/tile/BraveMostVisitedTilesMediator.java",
"../../brave/android/java/org/chromium/chrome/browser/suggestions/tile/BraveTileView.java",
"../../brave/android/java/org/chromium/chrome/browser/sync/BraveSyncDevices.java",
"../../brave/android/java/org/chromium/chrome/browser/sync/settings/BraveManageSyncSettings.java",
Expand All @@ -295,7 +296,6 @@ brave_java_sources = [
"../../brave/android/java/org/chromium/chrome/browser/toolbar/BraveIncognitoToggleTabLayout.java",
"../../brave/android/java/org/chromium/chrome/browser/toolbar/BraveToolbarManager.java",
"../../brave/android/java/org/chromium/chrome/browser/toolbar/bottom/BookmarksButton.java",
"../../brave/android/java/org/chromium/chrome/browser/toolbar/bottom/BottomTabSwitcherActionMenuCoordinator.java",
"../../brave/android/java/org/chromium/chrome/browser/toolbar/bottom/BottomToolbarConfiguration.java",
"../../brave/android/java/org/chromium/chrome/browser/toolbar/bottom/BottomToolbarCoordinator.java",
"../../brave/android/java/org/chromium/chrome/browser/toolbar/bottom/BottomToolbarNewTabButton.java",
Expand All @@ -308,7 +308,6 @@ brave_java_sources = [
"../../brave/android/java/org/chromium/chrome/browser/toolbar/bottom/BrowsingModeBottomToolbarModel.java",
"../../brave/android/java/org/chromium/chrome/browser/toolbar/bottom/BrowsingModeBottomToolbarViewBinder.java",
"../../brave/android/java/org/chromium/chrome/browser/toolbar/bottom/SearchAccelerator.java",
"../../brave/android/java/org/chromium/chrome/browser/toolbar/bottom/ShareButton.java",
"../../brave/android/java/org/chromium/chrome/browser/toolbar/bottom/TabSwitcherBottomToolbarCoordinator.java",
"../../brave/android/java/org/chromium/chrome/browser/toolbar/bottom/TabSwitcherBottomToolbarMediator.java",
"../../brave/android/java/org/chromium/chrome/browser/toolbar/bottom/TabSwitcherBottomToolbarModel.java",
Expand Down Expand Up @@ -389,6 +388,7 @@ brave_browser_ui_android_appmenu_internal_sources = [

brave_browser_ui_android_toolbar_sources = [
"//brave/browser/ui/android/toolbar/java/src/org/chromium/chrome/browser/toolbar/bottom/BraveBottomControlsMediator.java",
"//brave/browser/ui/android/toolbar/java/src/org/chromium/chrome/browser/toolbar/top/BottomTabSwitcherActionMenuCoordinator.java",
"//brave/browser/ui/android/toolbar/java/src/org/chromium/chrome/browser/toolbar/top/BraveToolbarLayout.java",
]

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@
import org.chromium.base.supplier.ObservableSupplier;
import org.chromium.base.supplier.OneshotSupplier;
import org.chromium.base.supplier.Supplier;
import org.chromium.chrome.browser.compositor.layouts.OverviewModeBehavior;
import org.chromium.chrome.browser.compositor.layouts.content.TabContentManager;
import org.chromium.chrome.browser.layouts.LayoutStateProvider;
import org.chromium.chrome.browser.lifecycle.ActivityLifecycleDispatcher;
import org.chromium.chrome.browser.share.ShareDelegate;
import org.chromium.chrome.browser.tabmodel.IncognitoStateProvider;
Expand Down Expand Up @@ -54,13 +54,13 @@ public BraveTabGroupUiCoordinator(@NonNull Activity activity, @NonNull ViewGroup
@NonNull Supplier<DynamicResourceLoader> dynamicResourceLoaderSupplier,
@NonNull TabCreatorManager tabCreatorManager,
@NonNull Supplier<ShareDelegate> shareDelegateSupplier,
@NonNull OneshotSupplier<OverviewModeBehavior> overviewModeBehaviorSupplier,
@NonNull OneshotSupplier<LayoutStateProvider> layoutStateProviderSupplier,
@NonNull SnackbarManager snackbarManager) {
super(activity, parentView, incognitoStateProvider, scrimCoordinator,
omniboxFocusStateSupplier, bottomSheetController, activityLifecycleDispatcher,
isWarmOnResumeSupplier, tabModelSelector, tabContentManager, rootView,
dynamicResourceLoaderSupplier, tabCreatorManager, shareDelegateSupplier,
overviewModeBehaviorSupplier, snackbarManager);
layoutStateProviderSupplier, snackbarManager);

mIncognitoStateProvider = incognitoStateProvider;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@
import org.chromium.chrome.browser.tabmodel.TabModelSelector;
import org.chromium.chrome.browser.toolbar.top.Toolbar;
import org.chromium.chrome.browser.ui.messages.snackbar.SnackbarManager;
import org.chromium.chrome.browser.xsurface.FeedLaunchReliabilityLogger.SurfaceType;
import org.chromium.components.browser_ui.bottomsheet.BottomSheetController;
import org.chromium.ui.UiUtils;
import org.chromium.ui.base.WindowAndroid;
Expand All @@ -52,17 +53,17 @@ public BraveFeedSurfaceCoordinator(Activity activity, SnackbarManager snackbarMa
@Nullable ScrollableContainerDelegate externalScrollableContainerDelegate,
@NewTabPageLaunchOrigin int launchOrigin,
PrivacyPreferencesManagerImpl privacyPreferencesManager,
@NonNull Supplier<Toolbar> toolbarSupplier,
FeedLaunchReliabilityLoggingState launchReliabilityLoggingState,
@Nullable FeedSwipeRefreshLayout swipeRefreshLayout, boolean overScrollDisabled,
@Nullable ViewGroup viewportView, FeedActionDelegate actionDelegate,
HelpAndFeedbackLauncher helpAndFeedbackLauncher) {
@NonNull Supplier<Toolbar> toolbarSupplier, @SurfaceType int surfaceType,
long embeddingSurfaceCreatedTimeNs, @Nullable FeedSwipeRefreshLayout swipeRefreshLayout,
boolean overScrollDisabled, @Nullable ViewGroup viewportView,
FeedActionDelegate actionDelegate, HelpAndFeedbackLauncher helpAndFeedbackLauncher,
TabModelSelector tabModelSelector) {
super(activity, snackbarManager, windowAndroid, snapScrollHelper, ntpHeader, toolbarHeight,
showDarkBackground, delegate, profile, isPlaceholderShownInitially,
bottomSheetController, shareDelegateSupplier, externalScrollableContainerDelegate,
launchOrigin, privacyPreferencesManager, toolbarSupplier,
launchReliabilityLoggingState, swipeRefreshLayout, overScrollDisabled, viewportView,
actionDelegate, helpAndFeedbackLauncher);
launchOrigin, privacyPreferencesManager, toolbarSupplier, surfaceType,
embeddingSurfaceCreatedTimeNs, swipeRefreshLayout, overScrollDisabled, viewportView,
actionDelegate, helpAndFeedbackLauncher, tabModelSelector);
}

public void createScrollViewForPolicy() {
Expand Down Expand Up @@ -98,8 +99,4 @@ public void createScrollViewForPolicy() {
public NestedScrollView getScrollViewForPolicy() {
return mScrollViewForPolicy;
}

public boolean isReliabilityLoggingEnabled() {
return false;
}
}
24 changes: 17 additions & 7 deletions android/java/apk_for_test.flags
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,17 @@
}

-keep class org.chromium.chrome.browser.ntp.NewTabPageLayout {
*** mMvTilesContainerLayout;
*** insertSiteSectionView(...);
*** mSiteSectionView;
*** mTileGroup;
}

-keep class org.chromium.chrome.browser.suggestions.tile.MostVisitedTilesMediator {
public <init>(...);
*** updateTileGridPlaceholderVisibility(...);
}

-keep class org.chromium.chrome.browser.suggestions.tile.BraveMostVisitedTilesMediator {
public <init>(...);
*** updateTileGridPlaceholderVisibility(...);
}

Expand All @@ -48,6 +56,7 @@
*** mNewTabPageLayout;
*** mFeedSurfaceProvider;
*** mToolbarSupplier;
*** mTabModelSelector;
*** updateSearchProviderHasLogo(...);
}

Expand All @@ -69,6 +78,7 @@
*** mSyncEncryption;
*** mReviewSyncData;
*** mSyncPaymentsIntegration;
*** mSyncReadingList;
*** mTurnOffSync;
}

Expand Down Expand Up @@ -129,7 +139,6 @@
*** mIsWarmOnResumeSupplier;
*** mTabContentManager;
*** mTabCreatorManager;
*** mOverviewModeBehaviorSupplier;
*** mSnackbarManager;
*** onOrientationChange(...);
*** updateBookmarkButtonStatus(...);
Expand Down Expand Up @@ -195,8 +204,7 @@
}

-keep class org.chromium.chrome.browser.bookmarks.BookmarkUtils {
*** showBookmarkBottomSheet(...);
*** addBookmarkAndShowSnackbar(...);
*** addOrEditBookmark(...);
}

-keep class org.chromium.chrome.browser.toolbar.BraveToolbarManager {
Expand Down Expand Up @@ -326,14 +334,12 @@
*** mActivity;
*** mNtpHeader;
*** mRootView;
*** isReliabilityLoggingEnabled(...);
}

-keep class org.chromium.chrome.browser.feed.BraveFeedSurfaceCoordinator {
*** mScrollViewForPolicy;
*** mNtpHeader;
*** mRootView;
*** isReliabilityLoggingEnabled(...);
}

-keep class org.chromium.chrome.browser.feed.FeedSurfaceMediator {
Expand Down Expand Up @@ -471,3 +477,7 @@
public <init>(...);
*** createManageDevicesLink(...);
}

-keep class org.chromium.chrome.browser.ChromeApplicationImpl

-keep class org.chromium.chrome.browser.BraveApplicationImplBase
Original file line number Diff line number Diff line change
Expand Up @@ -1438,7 +1438,7 @@ public ObservableSupplier<BrowserControlsManager> getBrowserControlsManagerSuppl
}

public int getToolbarShadowHeight() {
View toolbarShadow = findViewById(R.id.toolbar_shadow);
View toolbarShadow = findViewById(R.id.toolbar_hairline);
assert toolbarShadow != null;
if (toolbarShadow != null) {
return toolbarShadow.getHeight();
Expand All @@ -1447,7 +1447,7 @@ public int getToolbarShadowHeight() {
}

public int getToolbarBottom() {
View toolbarShadow = findViewById(R.id.toolbar_shadow);
View toolbarShadow = findViewById(R.id.toolbar_hairline);
assert toolbarShadow != null;
if (toolbarShadow != null) {
return toolbarShadow.getBottom();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@
import org.chromium.chrome.R;
import org.chromium.chrome.browser.ActivityTabProvider;
import org.chromium.chrome.browser.bookmarks.BookmarkBridge;
import org.chromium.chrome.browser.compositor.layouts.OverviewModeBehavior;
import org.chromium.chrome.browser.feed.webfeed.WebFeedBridge;
import org.chromium.chrome.browser.layouts.LayoutStateProvider;
import org.chromium.chrome.browser.multiwindow.MultiWindowModeStateDispatcher;
import org.chromium.chrome.browser.tabmodel.TabModelSelector;
import org.chromium.chrome.browser.toolbar.ToolbarManager;
Expand All @@ -36,11 +36,11 @@ public BraveAppMenuPropertiesDelegateImpl(Context context,
ActivityTabProvider activityTabProvider,
MultiWindowModeStateDispatcher multiWindowModeStateDispatcher,
TabModelSelector tabModelSelector, ToolbarManager toolbarManager, View decorView,
@Nullable OneshotSupplier<OverviewModeBehavior> overviewModeBehaviorSupplier,
@Nullable OneshotSupplier<LayoutStateProvider> layoutStateProvidersSupplier,
@Nullable OneshotSupplier<StartSurface> startSurfaceSupplier,
ObservableSupplier<BookmarkBridge> bookmarkBridgeSupplier) {
super(context, activityTabProvider, multiWindowModeStateDispatcher, tabModelSelector,
toolbarManager, decorView, overviewModeBehaviorSupplier, startSurfaceSupplier,
toolbarManager, decorView, layoutStateProvidersSupplier, startSurfaceSupplier,
bookmarkBridgeSupplier);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,10 @@
import org.chromium.chrome.browser.BraveFeatureList;
import org.chromium.chrome.browser.app.appmenu.AppMenuIconRowFooter;
import org.chromium.chrome.browser.bookmarks.BookmarkBridge;
import org.chromium.chrome.browser.compositor.layouts.OverviewModeBehavior;
import org.chromium.chrome.browser.feed.webfeed.WebFeedBridge;
import org.chromium.chrome.browser.feed.webfeed.WebFeedSnackbarController;
import org.chromium.chrome.browser.flags.ChromeFeatureList;
import org.chromium.chrome.browser.layouts.LayoutStateProvider;
import org.chromium.chrome.browser.multiwindow.MultiWindowModeStateDispatcher;
import org.chromium.chrome.browser.preferences.BravePrefServiceBridge;
import org.chromium.chrome.browser.set_default_browser.BraveSetDefaultBrowserUtils;
Expand Down Expand Up @@ -55,13 +55,13 @@ public BraveTabbedAppMenuPropertiesDelegate(Context context,
MultiWindowModeStateDispatcher multiWindowModeStateDispatcher,
TabModelSelector tabModelSelector, ToolbarManager toolbarManager, View decorView,
AppMenuDelegate appMenuDelegate,
OneshotSupplier<OverviewModeBehavior> overviewModeBehaviorSupplier,
OneshotSupplier<LayoutStateProvider> layoutStateProvider,
OneshotSupplier<StartSurface> startSurfaceSupplier,
ObservableSupplier<BookmarkBridge> bookmarkBridgeSupplier,
WebFeedSnackbarController.FeedLauncher feedLauncher,
ModalDialogManager modalDialogManager, SnackbarManager snackbarManager) {
super(context, activityTabProvider, multiWindowModeStateDispatcher, tabModelSelector,
toolbarManager, decorView, appMenuDelegate, overviewModeBehaviorSupplier,
toolbarManager, decorView, appMenuDelegate, layoutStateProvider,
startSurfaceSupplier, bookmarkBridgeSupplier, feedLauncher, modalDialogManager,
snackbarManager);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,25 +31,11 @@
* A class holding static util functions for bookmark.
*/
public class BraveBookmarkUtils extends BookmarkUtils {
private static final String TAG = "BraveBookmarkUtils";
/**
* If the tab has already been bookmarked, start {@link BookmarkEditActivity} for the
* normal bookmark or show the reading list page for reading list bookmark.
* If not, add the bookmark to {@link BookmarkModel}, and show a snackbar notifying the user.
*
* @param existingBookmarkItem The {@link BookmarkItem} if the tab has already been bookmarked.
* @param bookmarkModel The bookmark model.
* @param tab The tab to add or edit a bookmark.
* @param snackbarManager The {@link SnackbarManager} used to show the snackbar.
* @param bottomSheetController The {@link BottomSheetController} used to show the bottom sheet.
* @param activity Current activity.
* @param fromCustomTab boolean indicates whether it is called by Custom Tab.
* @param callback Invoked with the resulting bookmark ID, which could be null if unsuccessful.
*/
public static void addOrEditBookmark(@Nullable BookmarkItem existingBookmarkItem,
BookmarkModel bookmarkModel, Tab tab, SnackbarManager snackbarManager,
BottomSheetController bottomSheetController, Activity activity, boolean fromCustomTab,
Callback<BookmarkId> callback) {
@BookmarkType int bookmarkType, Callback<BookmarkId> callback,
boolean fromExplicitTrackUi) {
assert bookmarkModel.isBookmarkModelLoaded();
if (existingBookmarkItem != null) {
if (snackbarManager.isShowing()) {
Expand All @@ -61,28 +47,8 @@ public static void addOrEditBookmark(@Nullable BookmarkItem existingBookmarkItem
return;
}

if (CachedFeatureFlags.isEnabled(ChromeFeatureList.READ_LATER)) {
// Show a bottom sheet to let the user select target bookmark folder.
showBookmarkBottomSheet(bookmarkModel, tab, snackbarManager, bottomSheetController,
activity, fromCustomTab, callback);
return;
}

BookmarkId newBookmarkId = addBookmarkAndShowSnackbar(
bookmarkModel, tab, snackbarManager, activity, fromCustomTab);
callback.onResult(newBookmarkId);
}

protected static void showBookmarkBottomSheet(BookmarkModel bookmarkModel, Tab tab,
SnackbarManager snackbarManager, BottomSheetController bottomSheetController,
Activity activity, boolean fromCustomTab, Callback<BookmarkId> callback) {
assert (false);
}

// The legacy code path to add or edit bookmark without triggering the bookmark bottom sheet.
protected static BookmarkId addBookmarkAndShowSnackbar(BookmarkModel bookmarkModel, Tab tab,
SnackbarManager snackbarManager, Activity activity, boolean fromCustomTab) {
assert (false);
return null;
BookmarkUtils.addOrEditBookmark(existingBookmarkItem, bookmarkModel, tab, snackbarManager,
bottomSheetController, activity, fromCustomTab, bookmarkType, callback,
fromExplicitTrackUi);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,15 @@
* You can obtain one at http://mozilla.org/MPL/2.0/. */

package org.chromium.chrome.browser.firstrun;

import android.app.Activity;

import org.chromium.base.supplier.OneshotSupplier;
import org.chromium.components.signin.AccountManagerFacadeProvider;

public abstract class BraveFirstRunFlowSequencer extends FirstRunFlowSequencer {
public BraveFirstRunFlowSequencer(Activity activity) {
super(activity);
super(activity,
new ChildAccountStatusSupplier(AccountManagerFacadeProvider.getInstance(), null));
}

@Override
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@
import org.chromium.chrome.browser.BraveRewardsHelper;
import org.chromium.chrome.browser.app.BraveActivity;
import org.chromium.chrome.browser.customtabs.CustomTabActivity;
import org.chromium.chrome.browser.firstrun.BraveFirstRunFlowSequencer;
import org.chromium.chrome.browser.night_mode.GlobalNightModeStateProviderHolder;
import org.chromium.chrome.browser.onboarding.OnboardingPrefManager;
import org.chromium.chrome.browser.preferences.BravePrefServiceBridge;
Expand All @@ -44,7 +45,7 @@ public class P3aOnboardingActivity extends FirstRunActivityBase {
private boolean mInitializeViewsDone;
private boolean mInvokePostWorkAtInitializeViews;
private boolean mIsP3aEnabled;
private FirstRunFlowSequencer mFirstRunFlowSequencer;
private BraveFirstRunFlowSequencer mFirstRunFlowSequencer;
private CheckBox mP3aOnboardingCheckbox;
private Button mBtnContinue;

Expand Down Expand Up @@ -168,8 +169,9 @@ public void finishNativeInitialization() {
public void onBackPressed() {}

@Override
protected void triggerLayoutInflation() {
mFirstRunFlowSequencer = new FirstRunFlowSequencer(this) {
public void triggerLayoutInflation() {
super.triggerLayoutInflation();
mFirstRunFlowSequencer = new BraveFirstRunFlowSequencer(this) {
@Override
public void onFlowIsKnown(Bundle freProperties) {
initializeViews();
Expand Down
Loading