Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3524)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/fullscreen/FullscreenManager.java

Issue 2484293003: Use only renderer driven offsets for fullscreen state. (Closed)
Patch Set: Partially disable testControlsShownOnUnresponsiveRenderer due to timing issues with renderer logic … Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/fullscreen/FullscreenManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/FullscreenManager.java b/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/FullscreenManager.java
index b86776b357220011325536913e260721fced93ed..d3044e10400ada546d91e8243d5e1ae4fd72f2d8 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/FullscreenManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/FullscreenManager.java
@@ -45,36 +45,6 @@ public abstract class FullscreenManager {
}
/**
- * Trigger a temporary showing of the browser controls.
- */
- public abstract void showControlsTransient();
-
- /**
- * Trigger a permanent showing of the browser controls until requested otherwise.
- *
- * @return The token that determines whether the requester still needs persistent controls to
- * be present on the screen.
- * @see #hideControlsPersistent(int)
- */
- public abstract int showControlsPersistent();
-
- /**
- * Same behavior as {@link #showControlsPersistent()} but also handles removing a previously
- * requested token if necessary.
- *
- * @param oldToken The old fullscreen token to be cleared.
- * @return The fullscreen token as defined in {@link #showControlsPersistent()}.
- */
- public abstract int showControlsPersistentAndClearOldToken(int oldToken);
-
- /**
- * Notify the manager that the browser controls are no longer required for the given token.
- *
- * @param token The fullscreen token returned from {@link #showControlsPersistent()}.
- */
- public abstract void hideControlsPersistent(int token);
-
- /**
* @return The height of the top controls in pixels.
*/
public abstract int getTopControlsHeight();

Powered by Google App Engine
This is Rietveld 408576698