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

Unified Diff: content/renderer/gpu/render_widget_compositor.cc

Issue 814083004: Notify main-thread of top controls state changes (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix compile error after rebase Created 5 years, 10 months 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
« no previous file with comments | « content/renderer/gpu/render_widget_compositor.h ('k') | content/renderer/render_view_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/gpu/render_widget_compositor.cc
diff --git a/content/renderer/gpu/render_widget_compositor.cc b/content/renderer/gpu/render_widget_compositor.cc
index 9b6c288bdde14289087aa1abb9348dd2f82a5da7..286929bcd4a5b7190c543c5ecd8e6fe28d7785d4 100644
--- a/content/renderer/gpu/render_widget_compositor.cc
+++ b/content/renderer/gpu/render_widget_compositor.cc
@@ -64,6 +64,7 @@ using blink::WebFloatPoint;
using blink::WebRect;
using blink::WebSelectionBound;
using blink::WebSize;
+using blink::WebTopControlsState;
namespace content {
namespace {
@@ -155,6 +156,20 @@ gfx::Size CalculateDefaultTileSize() {
return gfx::Size(default_tile_size, default_tile_size);
}
+// Check cc::TopControlsState, and blink::WebTopControlsState
+// are kept in sync.
+static_assert(int(blink::WebTopControlsBoth) == int(cc::BOTH),
+ "mismatching enums: BOTH");
+static_assert(int(blink::WebTopControlsHidden) == int(cc::HIDDEN),
+ "mismatching enums: HIDDEN");
+static_assert(int(blink::WebTopControlsShown) == int(cc::SHOWN),
+ "mismatching enums: SHOWN");
+
+static cc::TopControlsState ConvertTopControlsState(
+ WebTopControlsState state) {
+ return static_cast<cc::TopControlsState>(state);
+}
+
} // namespace
// static
@@ -476,23 +491,6 @@ void RenderWidgetCompositor::SetRasterizeOnlyVisibleContent() {
layer_tree_host_->SetDebugState(current);
}
-void RenderWidgetCompositor::UpdateTopControlsState(
- cc::TopControlsState constraints,
- cc::TopControlsState current,
- bool animate) {
- layer_tree_host_->UpdateTopControlsState(constraints,
- current,
- animate);
-}
-
-void RenderWidgetCompositor::SetTopControlsShrinkBlinkSize(bool shrink) {
- layer_tree_host_->SetTopControlsShrinkBlinkSize(shrink);
-}
-
-void RenderWidgetCompositor::SetTopControlsHeight(float height) {
- layer_tree_host_->SetTopControlsHeight(height);
-}
-
void RenderWidgetCompositor::SetNeedsRedrawRect(gfx::Rect damage_rect) {
layer_tree_host_->SetNeedsRedrawRect(damage_rect);
}
@@ -762,6 +760,19 @@ void RenderWidgetCompositor::setShowScrollBottleneckRects(bool show) {
layer_tree_host_->SetDebugState(debug_state);
}
+void RenderWidgetCompositor::updateTopControlsState(
+ WebTopControlsState constraints,
+ WebTopControlsState current,
+ bool animate) {
+ layer_tree_host_->UpdateTopControlsState(ConvertTopControlsState(constraints),
+ ConvertTopControlsState(current),
+ animate);
+}
+
+void RenderWidgetCompositor::setTopControlsHeight(float height, bool shrink) {
+ layer_tree_host_->SetTopControlsHeight(height, shrink);
+}
+
void RenderWidgetCompositor::setTopControlsContentOffset(float offset) {
setTopControlsShownRatio(offset);
}
« no previous file with comments | « content/renderer/gpu/render_widget_compositor.h ('k') | content/renderer/render_view_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698