Allow changing top controls height

content/browser/renderer_host/render_widget_host_view_android.cc

 // Copyright (c) 2012 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.
 
 #include "content/browser/renderer_host/render_widget_host_view_android.h"
 
 #include <android/bitmap.h>
 
 #include "base/android/build_info.h"
 #include "base/basictypes.h"
@@ skipping 651 matching lines @@
   return gfx::Rect(content_view_core_->GetViewSize());
 }
 
 gfx::Size RenderWidgetHostViewAndroid::GetPhysicalBackingSize() const {
   if (!content_view_core_)
     return gfx::Size();
 
   return content_view_core_->GetPhysicalBackingSize();
 }
 
-float RenderWidgetHostViewAndroid::GetTopControlsLayoutHeight() const {
+bool RenderWidgetHostViewAndroid::GetTopControlsShrinkBlinkSize() const {

[palmer, 2014/12/01 19:57:54]

The name |GetTopControlsShrinkBlinkSize| is really confusing. "get...size"
sounds like it should return a number.

[David Trainor- moved to gerrit, 2014/12/02 05:53:25]

Yeah I can see that being a bit confusing.  I could call it
getTopControlsShrinkBlinkViewport, but that still kind of makes it sound like it
should return a Rect or a Size object.  Maybe getTopControlsShrinkBlinkLayout?

+  if (!content_view_core_)
+    return false;
+
+  // Whether or not Blink's viewport size should be shrunk by the height of the
+  // URL-bar.
+  return content_view_core_->GetTopControlsShrinkBlinkSize();
+}
+
+float RenderWidgetHostViewAndroid::GetTopControlsHeight() const {
   if (!content_view_core_)
     return 0.f;
 
-  // The amount that the viewport size given to Blink is shrunk by the URL-bar.
-  return content_view_core_->GetTopControlsLayoutHeightDip();
+  // The height of the top controls.
+  return content_view_core_->GetTopControlsHeightDip();
 }
 
 void RenderWidgetHostViewAndroid::UpdateCursor(const WebCursor& cursor) {
   // There are no cursors on Android.
 }
 
 void RenderWidgetHostViewAndroid::SetIsLoading(bool is_loading) {
   // Do nothing. The UI notification is handled through ContentViewClient which
   // is TabContentsDelegate.
 }
@@ skipping 1188 matching lines @@
   results->orientationAngle = display.RotationAsDegree();
   results->orientationType =
       RenderWidgetHostViewBase::GetOrientationTypeForMobile(display);
   gfx::DeviceDisplayInfo info;
   results->depth = info.GetBitsPerPixel();
   results->depthPerComponent = info.GetBitsPerComponent();
   results->isMonochrome = (results->depthPerComponent == 0);
 }
 
 } // namespace content