Commit 02befe74 authored by Jian Li's avatar Jian Li Committed by Commit Bot

Remove the construction of FeedStreamSurface that is not needed

Bug: none
Change-Id: Ia397468378c7a94d04024fb3f444995c1003efea
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2234548Reviewed-by: default avatarDan H <harringtond@chromium.org>
Commit-Queue: Jian Li <jianli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#776101}
parent 862ef950
...@@ -39,8 +39,6 @@ import org.chromium.chrome.browser.feed.shared.stream.Header; ...@@ -39,8 +39,6 @@ import org.chromium.chrome.browser.feed.shared.stream.Header;
import org.chromium.chrome.browser.feed.shared.stream.NonDismissibleHeader; import org.chromium.chrome.browser.feed.shared.stream.NonDismissibleHeader;
import org.chromium.chrome.browser.feed.shared.stream.Stream; import org.chromium.chrome.browser.feed.shared.stream.Stream;
import org.chromium.chrome.browser.feed.tooltip.BasicTooltipApi; import org.chromium.chrome.browser.feed.tooltip.BasicTooltipApi;
import org.chromium.chrome.browser.feed.v2.FeedServiceBridge;
import org.chromium.chrome.browser.feed.v2.FeedStreamSurface;
import org.chromium.chrome.browser.flags.ChromeFeatureList; import org.chromium.chrome.browser.flags.ChromeFeatureList;
import org.chromium.chrome.browser.native_page.ContextMenuManager; import org.chromium.chrome.browser.native_page.ContextMenuManager;
import org.chromium.chrome.browser.native_page.NativePageNavigationDelegate; import org.chromium.chrome.browser.native_page.NativePageNavigationDelegate;
...@@ -81,7 +79,6 @@ public class FeedSurfaceCoordinator implements FeedSurfaceProvider { ...@@ -81,7 +79,6 @@ public class FeedSurfaceCoordinator implements FeedSurfaceProvider {
private final int mDefaultMargin; private final int mDefaultMargin;
private final int mWideMargin; private final int mWideMargin;
private final FeedSurfaceMediator mMediator; private final FeedSurfaceMediator mMediator;
private FeedStreamSurface mFeedStreamSurface;
private UiConfig mUiConfig; private UiConfig mUiConfig;
private FrameLayout mRootView; private FrameLayout mRootView;
...@@ -345,15 +342,6 @@ public class FeedSurfaceCoordinator implements FeedSurfaceProvider { ...@@ -345,15 +342,6 @@ public class FeedSurfaceCoordinator implements FeedSurfaceProvider {
mMediator.onHomepagePromoStateChange(); mMediator.onHomepagePromoStateChange();
} }
// Native should already have been loaded because of FeedSurfaceMediator.
if (FeedServiceBridge.isEnabled()) {
// TODO(jianli): Temporary: simulate opening the feed V2 surface. This should probably
// move to FeedSurfaceMediator.
mFeedStreamSurface = new FeedStreamSurface(
tabModelSelector, tabProvider, mActivity, mSnackbarManager);
mFeedStreamSurface.surfaceOpened();
}
mUserEducationHelper = new UserEducationHelper(mActivity); mUserEducationHelper = new UserEducationHelper(mActivity);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment