Refactor OverlayPanel height calculations

chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/readermode/ReaderModePanel.java

 // Copyright 2015 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.compositor.bottombar.readermode;
 
 import android.content.Context;
 
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.compositor.LayerTitleCache;
@@ skipping 293 matching lines @@
     public float getOffsetY() {
         // Do not attempt to auto-hide the reader mode bar if the toolbar is less than a certain
         // height.
         boolean shouldAutoHide = getToolbarHeight() >= getBarHeightPeeking();
         // This will cause the reader mode bar to behave like the top controls; sliding out of
         // view as the page scrolls.
         return super.getOffsetY() + (shouldAutoHide ? getTopControlsOffsetDp() : 0.0f);
     }
 
     @Override
-    public void onSizeChanged(float width, float height) {
-        super.onSizeChanged(width, height);
+    public void onSizeChanged(float width, float height, float visibleViewportOffsetY) {

[Changwan Ryu, 2016/07/07 03:29:32]

onLayoutChanged?

[mdjones, 2016/07/07 18:33:40]

Done.

+        super.onSizeChanged(width, height, visibleViewportOffsetY);
         if (mManagerDelegate != null) {
             mManagerDelegate.onSizeChanged();
         }
     }
 
     @Override
     protected float calculateBasePageDesiredOffset() {
         return -getToolbarHeight();
     }
 
@@ skipping 30 matching lines @@
     /**
      * Destroys the ReaderModeBarControl.
      */
     protected void destroyReaderModeBarControl() {
         if (mReaderModeBarControl != null) {
             mReaderModeBarControl.destroy();
             mReaderModeBarControl = null;
         }
     }
 }