[cr147][Android] Rename FeedActionDelegate -> ActionDelegateFactory
Chromium changes: https://chromium.googlesource.com/chromium/src/+/b5a2265620263469792b5a322df732b40d2e8c83 commit b5a2265620263469792b5a322df732b40d2e8c83 Author: Anton Goncharenko <agonch@google.com> Date: Wed Feb 18 06:02:50 2026 -0800 [Signin][Android] Migrate Feed Surfaces to activity-less sign-in flow This is part of the effort to reduce reliance on SigninAndHistorySyncActivity by sign-in entry points. Integrated BottomSheetSigninAndHistorySyncCoordinator initialization and cleanup in FeedActionDelegateImpl. The new sign-in flow is hidden behind ENABLE_SEAMLESS_SIGNIN and ENABLE_ACTIVITYLESS_SIGNIN_ALL_ENTRY_POINT feature flags. Design Doc: go/activityless-signin Bug: 478812795 Change-Id: I58f670525db5c9669a9c0af25245df2e49dec5c9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7557299 Commit-Queue: Liza Bipin <mlbipin@google.com> Reviewed-by: Adam Arcaro <adamta@google.com> Reviewed-by: Liza Bipin <mlbipin@google.com> Auto-Submit: Anton Goncharenko <agonch@google.com> Cr-Commit-Position: refs/heads/main@{#1586376}
This commit is contained in:
@@ -167,19 +167,20 @@ public class BraveNewTabPage extends NewTabPage {
|
||||
mNewTabPageLayout = (NewTabPageLayout) inflater.inflate(R.layout.new_tab_page_layout, null);
|
||||
|
||||
// No-op stub to deal with non-null requirement
|
||||
FeedActionDelegate actionDelegate =
|
||||
new FeedActionDelegate() {
|
||||
@Override
|
||||
public void openSuggestionUrl(
|
||||
int disposition,
|
||||
LoadUrlParams params,
|
||||
boolean inGroup,
|
||||
int pageId,
|
||||
PageLoadObserver pageLoadObserver,
|
||||
Callback<VisitResult> onVisitComplete) {
|
||||
assert false : "Not supposed to be invoked";
|
||||
}
|
||||
};
|
||||
FeedSurfaceCoordinator.ActionDelegateFactory actionDelegate =
|
||||
() ->
|
||||
new FeedActionDelegate() {
|
||||
@Override
|
||||
public void openSuggestionUrl(
|
||||
int disposition,
|
||||
LoadUrlParams params,
|
||||
boolean inGroup,
|
||||
int pageId,
|
||||
PageLoadObserver pageLoadObserver,
|
||||
Callback<VisitResult> onVisitComplete) {
|
||||
assert false : "Not supposed to be invoked";
|
||||
}
|
||||
};
|
||||
|
||||
assertNonNull(mBrowserControlsStateProvider);
|
||||
assertNonNull(mToolbarSupplier);
|
||||
|
||||
Reference in New Issue
Block a user