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

Unified Diff: cc/trees/layer_tree_host_impl.cc

Issue 2810813004: Hide fullscreen rotation jank (Closed)
Patch Set: Build error fixes Created 3 years, 7 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_impl.cc
diff --git a/cc/trees/layer_tree_host_impl.cc b/cc/trees/layer_tree_host_impl.cc
index ce654574d98ece3e980b0914e75b7340db0eb76b..9901abc7d8de7e1c97ed00b5d24472a3ed8b380a 100644
--- a/cc/trees/layer_tree_host_impl.cc
+++ b/cc/trees/layer_tree_host_impl.cc
@@ -1607,6 +1607,7 @@ CompositorFrameMetadata LayerTreeHostImpl::MakeCompositorFrameMetadata() const {
metadata.page_scale_factor = active_tree_->current_page_scale_factor();
metadata.scrollable_viewport_size = active_tree_->ScrollableViewportSize();
metadata.root_layer_size = active_tree_->ScrollableSize();
+ metadata.device_viewport_size = active_tree_->device_viewport_size();
metadata.min_page_scale_factor = active_tree_->min_page_scale_factor();
metadata.max_page_scale_factor = active_tree_->max_page_scale_factor();
metadata.top_controls_height =
@@ -1659,6 +1660,8 @@ CompositorFrameMetadata LayerTreeHostImpl::MakeCompositorFrameMetadata() const {
metadata.root_scroll_offset =
gfx::ScrollOffsetToVector2dF(active_tree_->TotalScrollOffset());
+ metadata.is_fullscreen = is_fullscreen_;
+
return metadata;
}
@@ -2197,6 +2200,12 @@ void LayerTreeHostImpl::SetVisible(bool visible) {
}
}
+void LayerTreeHostImpl::SetIsFullscreen(bool is_fullscreen) {
+ DCHECK(task_runner_provider_->IsImplThread());
+
+ is_fullscreen_ = is_fullscreen;
+}
+
void LayerTreeHostImpl::SetNeedsOneBeginImplFrame() {
// TODO(miletus): This is just the compositor-thread-side call to the
// SwapPromiseMonitor to say something happened that may cause a swap in the

Powered by Google App Engine
This is Rietveld 408576698