Index: chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanel.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanel.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanel.java |
index d202dbffbe05f6b86e1c8dffb1548cdfefa197f2..fec65b8410af7fa647b541338b2db2941d8d7bea 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanel.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanel.java |
@@ -29,7 +29,7 @@ import org.chromium.chrome.browser.multiwindow.MultiWindowUtils; |
import org.chromium.chrome.browser.tab.Tab; |
import org.chromium.content.browser.ContentViewClient; |
import org.chromium.content.browser.ContentViewCore; |
-import org.chromium.content_public.common.TopControlsState; |
+import org.chromium.content_public.common.BrowserControlsState; |
import org.chromium.ui.base.LocalizationUtils; |
import org.chromium.ui.resources.ResourceManager; |
@@ -289,9 +289,9 @@ public class OverlayPanel extends OverlayPanelAnimation implements ActivityState |
} |
/** |
- * @return The absolute amount in DP that the top controls have shifted off screen. |
+ * @return The absolute amount in DP that the browser controls have shifted off screen. |
*/ |
- protected float getTopControlsOffsetDp() { |
+ protected float getBrowserControlsOffsetDp() { |
if (mActivity == null || mActivity.getFullscreenManager() == null) return 0.0f; |
return -mActivity.getFullscreenManager().getControlOffset() * mPxToDp; |
} |
@@ -430,25 +430,25 @@ public class OverlayPanel extends OverlayPanelAnimation implements ActivityState |
} |
/** |
- * Updates the top controls state for the base tab. As these values are set at the renderer |
+ * Updates the browser controls state for the base tab. As these values are set at the renderer |
* level, there is potential for this impacting other tabs that might share the same |
- * process. See {@link Tab#updateTopControlsState(int current, boolean animate)} |
+ * process. See {@link Tab#updateBrowserControlsState(int current, boolean animate)} |
* @param current The desired current state for the controls. Pass |
- * {@link TopControlsState#BOTH} to preserve the current position. |
+ * {@link BrowserControlsState#BOTH} to preserve the current position. |
* @param animate Whether the controls should animate to the specified ending condition or |
* should jump immediately. |
*/ |
- public void updateTopControlsState(int current, boolean animate) { |
+ public void updateBrowserControlsState(int current, boolean animate) { |
Tab currentTab = mActivity.getActivityTab(); |
if (currentTab != null) { |
- currentTab.updateTopControlsState(current, animate); |
+ currentTab.updateBrowserControlsState(current, animate); |
} |
} |
/** |
* Sets the top control state based on the internals of the panel. |
*/ |
- public void updateTopControlsState() { |
+ public void updateBrowserControlsState() { |
if (mContent == null) return; |
if (isFullWidthSizePanel()) { |
@@ -459,9 +459,9 @@ public class OverlayPanel extends OverlayPanelAnimation implements ActivityState |
// minus the Toolbar height. |
// |
// This is necessary to fix the bugs: crbug.com/510205 and crbug.com/510206 |
- mContent.updateTopControlsState(false, true, false); |
+ mContent.updateBrowserControlsState(false, true, false); |
} else { |
- mContent.updateTopControlsState(true, false, false); |
+ mContent.updateBrowserControlsState(true, false, false); |
} |
} |
@@ -823,17 +823,17 @@ public class OverlayPanel extends OverlayPanelAnimation implements ActivityState |
public boolean handlesTabCreating() { |
// If the panel is not opened, do not handle tab creating. |
if (!isPanelOpened()) return false; |
- // Updates TopControls' State so the Toolbar becomes visible. |
+ // Updates BrowserControls' State so the Toolbar becomes visible. |
// TODO(pedrosimonetti): The transition when promoting to a new tab is only smooth |
// if the SearchContentView's vertical scroll position is zero. Otherwise the |
// ContentView will appear to jump in the screen. Coordinate with @dtrainor to solve |
// this problem. |
- updateTopControlsState(TopControlsState.BOTH, false); |
+ updateBrowserControlsState(BrowserControlsState.BOTH, false); |
return true; |
} |
@Override |
- public boolean shouldHideAndroidTopControls() { |
+ public boolean shouldHideAndroidBrowserControls() { |
return isPanelOpened(); |
} |