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

Unified Diff: cc/trees/layer_tree_host.cc

Issue 14139013: Hide location bar on Javascript-initiated scroll. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge ShowTopControls into UpdateTopControlsState, use new TopControlsState enum in plumbing, gener… Created 7 years, 8 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
Index: cc/trees/layer_tree_host.cc
diff --git a/cc/trees/layer_tree_host.cc b/cc/trees/layer_tree_host.cc
index 708318756cfd4c9ee5694ae3748f6305ad1aaf53..4ce71b7bbd48d92a35942ffec7d79168387087b7 100644
--- a/cc/trees/layer_tree_host.cc
+++ b/cc/trees/layer_tree_host.cc
@@ -1051,8 +1051,8 @@ void LayerTreeHost::SetDeviceScaleFactor(float device_scale_factor) {
SetNeedsCommit();
}
-void LayerTreeHost::UpdateTopControlsState(bool enable_hiding,
- bool enable_showing,
+void LayerTreeHost::UpdateTopControlsState(TopControlsState constraints,
+ TopControlsState current,
bool animate) {
if (!settings_.calculate_top_controls_position)
return;
@@ -1061,8 +1061,8 @@ void LayerTreeHost::UpdateTopControlsState(bool enable_hiding,
proxy_->ImplThread()->PostTask(
base::Bind(&TopControlsManager::UpdateTopControlsState,
top_controls_manager_weak_ptr_,
- enable_hiding,
- enable_showing,
+ constraints,
+ current,
animate));
}

Powered by Google App Engine
This is Rietveld 408576698