Index: chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ChromeFullscreenManager.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ChromeFullscreenManager.java b/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ChromeFullscreenManager.java |
index 15b30a6e6fe6b73f609b6c3c27aa5d50ef2691cc..eb141802101afebf37623e4ec6f7e6ed0f6e645d 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ChromeFullscreenManager.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ChromeFullscreenManager.java |
@@ -31,7 +31,6 @@ import org.chromium.base.TraceEvent; |
import org.chromium.base.VisibleForTesting; |
import org.chromium.chrome.browser.fullscreen.FullscreenHtmlApiHandler.FullscreenHtmlApiDelegate; |
import org.chromium.chrome.browser.tab.Tab; |
-import org.chromium.chrome.browser.tabmodel.TabModelSelector; |
import org.chromium.chrome.browser.widget.ControlContainer; |
import org.chromium.content.browser.ContentVideoView; |
import org.chromium.content.browser.ContentViewCore; |
@@ -181,15 +180,13 @@ public class ChromeFullscreenManager |
* Creates an instance of the fullscreen mode manager. |
* @param activity The activity that supports fullscreen. |
* @param controlContainer Container holding the controls (Toolbar). |
- * @param modelSelector The model selector providing access to the current tab. |
* @param resControlContainerHeight The dimension resource ID for the control container height. |
* @param supportsBrowserOverride Whether we want to disable the token system used by the |
browser. |
*/ |
public ChromeFullscreenManager(Activity activity, ControlContainer controlContainer, |
- TabModelSelector modelSelector, int resControlContainerHeight, |
- boolean supportsBrowserOverride) { |
- super(activity.getWindow(), modelSelector); |
+ int resControlContainerHeight, boolean supportsBrowserOverride) { |
+ super(activity.getWindow()); |
mActivity = activity; |
ApplicationStatus.registerStateListenerForActivity(this, activity); |
@@ -240,7 +237,7 @@ public class ChromeFullscreenManager |
return new FullscreenHtmlApiDelegate() { |
@Override |
public void onEnterFullscreen() { |
- Tab tab = getActiveTab(); |
+ Tab tab = getTab(); |
if (getControlOffset() == -mControlContainerHeight) { |
// The top controls are currently hidden. |
getHtmlApiHandler().enterFullscreen(tab); |
@@ -510,7 +507,7 @@ public class ChromeFullscreenManager |
} |
} |
- final Tab tab = getActiveTab(); |
+ final Tab tab = getTab(); |
if (tab != null && offset == -mControlContainerHeight && mIsEnteringPersistentModeState) { |
getHtmlApiHandler().enterFullscreen(tab); |
mIsEnteringPersistentModeState = false; |
@@ -595,19 +592,14 @@ public class ChromeFullscreenManager |
} |
} |
- private Tab getActiveTab() { |
- Tab tab = getTabModelSelector().getCurrentTab(); |
- return tab; |
- } |
- |
private ContentViewCore getActiveContentViewCore() { |
- Tab tab = getActiveTab(); |
+ Tab tab = getTab(); |
return tab != null ? tab.getContentViewCore() : null; |
} |
@Override |
public void setPositionsForTabToNonFullscreen() { |
- Tab tab = getActiveTab(); |
+ Tab tab = getTab(); |
if (tab == null || tab.isShowingTopControlsEnabled()) { |
setPositionsForTab(0, mControlContainerHeight); |
} else { |