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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ComposedBrowserControlsVisibilityDelegate.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/ComposedBrowserControlsVisibilityDelegate.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ComposedBrowserControlsVisibilityDelegate.java b/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ComposedBrowserControlsVisibilityDelegate.java
new file mode 100644
index 0000000000000000000000000000000000000000..ea2089af6859a46d13a8b5127ef7e984db3aef8f
--- /dev/null
+++ b/chrome/android/java/src/org/chromium/chrome/browser/fullscreen/ComposedBrowserControlsVisibilityDelegate.java
@@ -0,0 +1,46 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+package org.chromium.chrome.browser.fullscreen;
+
+import org.chromium.chrome.browser.tab.BrowserControlsVisibilityDelegate;
+
+import java.util.ArrayList;
+import java.util.List;
+
+/**
+ * Delegate for the visibility of browser controls that combines the results of other delegates.
+ */
+public class ComposedBrowserControlsVisibilityDelegate
+ implements BrowserControlsVisibilityDelegate {
+
+ private final List<BrowserControlsVisibilityDelegate> mDelegates;
+
+ /**
+ * Constructs a composed visibility delegate that will generate results based on the delegates
+ * passed in.
+ */
+ public ComposedBrowserControlsVisibilityDelegate(
+ BrowserControlsVisibilityDelegate... delegates) {
+ mDelegates = new ArrayList<>();
+ for (int i = 0; i < delegates.length; i++) mDelegates.add(delegates[i]);
+ }
+
+ @Override
+ public boolean isShowingBrowserControlsEnabled() {
+ for (int i = 0; i < mDelegates.size(); i++) {
+ if (!mDelegates.get(i).isShowingBrowserControlsEnabled()) return false;
+ }
+ return true;
+ }
+
+ @Override
+ public boolean isHidingBrowserControlsEnabled() {
+ for (int i = 0; i < mDelegates.size(); i++) {
+ if (!mDelegates.get(i).isHidingBrowserControlsEnabled()) return false;
+ }
+ return true;
+ }
+
+}

Powered by Google App Engine
This is Rietveld 408576698