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

Unified Diff: cc/trees/layer_tree_host_impl.cc

Issue 2810813004: Hide fullscreen rotation jank (Closed)
Patch Set: Show a black frame during fullscreen transitions to hide jank 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 62e587ce860fceb390dcb40e31992f43b843aa36..26c9c8d008b670ee90dd5c708e7aa4af06fe0694 100644
--- a/cc/trees/layer_tree_host_impl.cc
+++ b/cc/trees/layer_tree_host_impl.cc
@@ -1587,6 +1587,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 =
@@ -1639,6 +1640,8 @@ CompositorFrameMetadata LayerTreeHostImpl::MakeCompositorFrameMetadata() const {
metadata.root_scroll_offset =
gfx::ScrollOffsetToVector2dF(active_tree_->TotalScrollOffset());
+ metadata.is_fullscreen = is_fullscreen_;
+
return metadata;
}
@@ -2147,6 +2150,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
« no previous file with comments | « cc/trees/layer_tree_host_impl.h ('k') | cc/trees/proxy.h » ('j') | cc/trees/proxy.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698