Index: chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java |
index c4051e13f4037e7c6a7b9644bd9722cd142f46d8..b8ba4640883ab4c849238fce71efaf464d6adc03 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java |
@@ -75,7 +75,6 @@ import org.chromium.chrome.browser.contextualsearch.ContextualSearchManager.Cont |
import org.chromium.chrome.browser.datausage.DataUseTabUIManager; |
import org.chromium.chrome.browser.device.DeviceClassManager; |
import org.chromium.chrome.browser.dom_distiller.DistilledPagePrefsView; |
-import org.chromium.chrome.browser.dom_distiller.ReaderModeActivityDelegate; |
import org.chromium.chrome.browser.dom_distiller.ReaderModeManager; |
import org.chromium.chrome.browser.enhancedbookmarks.EnhancedBookmarkUtils; |
import org.chromium.chrome.browser.enhancedbookmarks.EnhancedBookmarksModel; |
@@ -208,7 +207,7 @@ public abstract class ChromeActivity extends AsyncInitializationActivity |
private ChromeFullscreenManager mFullscreenManager; |
private CompositorViewHolder mCompositorViewHolder; |
private ContextualSearchManager mContextualSearchManager; |
- private ReaderModeActivityDelegate mReaderModeActivityDelegate; |
+ private ReaderModeManager mReaderModeManager; |
private SnackbarManager mSnackbarManager; |
private LoFiBarPopupController mLoFiBarPopupController; |
private DataUseSnackbarController mDataUseSnackbarController; |
@@ -447,7 +446,7 @@ public abstract class ChromeActivity extends AsyncInitializationActivity |
} |
if (ReaderModeManager.isEnabled(this)) { |
- mReaderModeActivityDelegate = new ReaderModeActivityDelegate(this); |
+ mReaderModeManager = new ReaderModeManager(getTabModelSelector(), this); |
} |
TraceEvent.end("ChromeActivity:CompositorInitialization"); |
@@ -753,9 +752,9 @@ public abstract class ChromeActivity extends AsyncInitializationActivity |
@SuppressLint("NewApi") |
@Override |
protected final void onDestroy() { |
- if (mReaderModeActivityDelegate != null) { |
- mReaderModeActivityDelegate.destroy(); |
- mReaderModeActivityDelegate = null; |
+ if (mReaderModeManager != null) { |
+ mReaderModeManager.destroy(); |
+ mReaderModeManager = null; |
} |
if (mContextualSearchManager != null) { |
@@ -1200,14 +1199,6 @@ public abstract class ChromeActivity extends AsyncInitializationActivity |
} |
/** |
- * @return A {@link ReaderModeActivityDelegate} instance or {@code null} if reader mode is |
- * not enabled. |
- */ |
- public ReaderModeActivityDelegate getReaderModeActivityDelegate() { |
- return mReaderModeActivityDelegate; |
- } |
- |
- /** |
* Create a full-screen manager to be used by this activity. |
* @param controlContainer The control container that will be controlled by the full-screen |
* manager. |
@@ -1259,12 +1250,6 @@ public abstract class ChromeActivity extends AsyncInitializationActivity |
mContextualSearchManager.setSearchContentViewDelegate(layoutManager); |
} |
- if (mReaderModeActivityDelegate != null) { |
- mReaderModeActivityDelegate.initialize(contentContainer); |
- mReaderModeActivityDelegate.setDynamicResourceLoader( |
- mCompositorViewHolder.getDynamicResourceLoader()); |
- } |
- |
layoutManager.addSceneChangeObserver(this); |
mCompositorViewHolder.setLayoutManager(layoutManager); |
mCompositorViewHolder.setFocusable(false); |
@@ -1272,7 +1257,8 @@ public abstract class ChromeActivity extends AsyncInitializationActivity |
mCompositorViewHolder.setFullscreenHandler(mFullscreenManager); |
mCompositorViewHolder.setUrlBar(urlBar); |
mCompositorViewHolder.onFinishNativeInitialization(getTabModelSelector(), this, |
- getTabContentManager(), contentContainer, mContextualSearchManager); |
+ getTabContentManager(), contentContainer, mContextualSearchManager, |
+ mReaderModeManager); |
if (controlContainer != null |
&& DeviceClassManager.enableToolbarSwipe(FeatureUtilities.isDocumentMode(this))) { |
@@ -1289,7 +1275,10 @@ public abstract class ChromeActivity extends AsyncInitializationActivity |
@Override |
public void onOrientationChange(int orientation) { |
+ // TODO(mdjones): Orientation change for panels should not be handled here. The event |
+ // should probably be passed to the OverlayPanelManager. |
if (mContextualSearchManager != null) mContextualSearchManager.onOrientationChange(); |
+ if (mReaderModeManager != null) mReaderModeManager.onOrientationChange(); |
if (mToolbarManager != null) mToolbarManager.onOrientationChange(); |
} |