Implement OverlayPanels as SceneOverlays

chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanelManager.java

Index: chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanelManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanelManager.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanelManager.java
index 6e8f2f48523c0361422b9b8d889732dbca01fc0d..dec1711f5190a8d5c5ea9f20fcea12d682bfb10b 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanelManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanelManager.java
@@ -6,6 +6,7 @@ package org.chromium.chrome.browser.compositor.bottombar;
 
 import android.view.ViewGroup;
 
+import org.chromium.base.VisibleForTesting;
 import org.chromium.chrome.browser.compositor.bottombar.OverlayPanel.StateChangeReason;
 import org.chromium.ui.resources.dynamics.DynamicResourceLoader;
 
@@ -47,9 +48,6 @@ public class OverlayPanelManager {
     /** When a panel is suppressed, this the reason the pending panel is to be shown. */
     private StateChangeReason mPendingReason;
 
-    /** This handles communication between each panel and it's host layout. */
-    private OverlayPanelHost mOverlayPanelHost;
-
     /** This handles resource loading for each panels. */
     private DynamicResourceLoader mDynamicResourceLoader;
 
@@ -133,6 +131,7 @@ public class OverlayPanelManager {
      * Get the panel that has been determined to be active.
      * @return The active OverlayPanel.
      */
+    @VisibleForTesting
     public OverlayPanel getActivePanel() {
         return mActivePanel;
     }
@@ -149,23 +148,11 @@ public class OverlayPanelManager {
         mSuppressedPanel = null;
 
         // Clear references to held resources.
-        mOverlayPanelHost = null;
         mDynamicResourceLoader = null;
         mContainerViewGroup = null;
     }
 
     /**
-     * Set the panel host for all OverlayPanels.
-     * @param host The OverlayPanel host.
-     */
-    public void setPanelHost(OverlayPanelHost host) {
-        mOverlayPanelHost = host;
-        for (OverlayPanel p : mPanelSet) {
-            p.setHost(host);
-        }
-    }
-
-    /**
      * Set the resource loader for all OverlayPanels.
      * @param host The OverlayPanel host.
      */
@@ -188,17 +175,6 @@ public class OverlayPanelManager {
     }
 
     /**
-     * Send size change event to all panels.
-     * @param width The new width.
-     * @param height The new height.
-     */
-    public void onSizeChanged(float width, float height) {
-        for (OverlayPanel p : mPanelSet) {
-            p.onSizeChanged(width, height);
-        }
-    }
-
-    /**
      * Add a panel to the collection that is managed by this class. If any of the setters for this
      * class were called before a panel was added, that panel will still get those resources.
      * @param panel An OverlayPanel to be managed.
@@ -206,9 +182,6 @@ public class OverlayPanelManager {
     public void registerPanel(OverlayPanel panel) {
         // If any of the setters for this manager were called before some panel registration,
         // make sure that panel gets the appropriate resources.
-        if (mOverlayPanelHost != null) {
-            panel.setHost(mOverlayPanelHost);
-        }
         if (mDynamicResourceLoader != null) {
             panel.setDynamicResourceLoader(mDynamicResourceLoader);
         }