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

Side by Side Diff: components/viz/frame_sinks/gpu_root_compositor_frame_sink.cc

Issue 2848223003: Enforce constant size and device scale factor for surfaces (Closed)
Patch Set: Fixed dchecks 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/viz/frame_sinks/gpu_root_compositor_frame_sink.h" 5 #include "components/viz/frame_sinks/gpu_root_compositor_frame_sink.h"
6 6
7 #include "cc/surfaces/compositor_frame_sink_support.h" 7 #include "cc/surfaces/compositor_frame_sink_support.h"
8 #include "cc/surfaces/display.h" 8 #include "cc/surfaces/display.h"
9 9
10 namespace viz { 10 namespace viz {
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 support_->EvictCurrentSurface(); 79 support_->EvictCurrentSurface();
80 } 80 }
81 81
82 void GpuRootCompositorFrameSink::SetNeedsBeginFrame(bool needs_begin_frame) { 82 void GpuRootCompositorFrameSink::SetNeedsBeginFrame(bool needs_begin_frame) {
83 support_->SetNeedsBeginFrame(needs_begin_frame); 83 support_->SetNeedsBeginFrame(needs_begin_frame);
84 } 84 }
85 85
86 void GpuRootCompositorFrameSink::SubmitCompositorFrame( 86 void GpuRootCompositorFrameSink::SubmitCompositorFrame(
87 const cc::LocalSurfaceId& local_surface_id, 87 const cc::LocalSurfaceId& local_surface_id,
88 cc::CompositorFrame frame) { 88 cc::CompositorFrame frame) {
89 support_->SubmitCompositorFrame(local_surface_id, std::move(frame)); 89 bool result =
danakj 2017/05/19 19:31:39 same here, the name result isnt adding anything
90 support_->SubmitCompositorFrame(local_surface_id, std::move(frame));
91 if (!result) {
92 compositor_frame_sink_binding_.Close();
93 OnClientConnectionLost();
94 }
90 } 95 }
91 96
92 void GpuRootCompositorFrameSink::BeginFrameDidNotSwap( 97 void GpuRootCompositorFrameSink::BeginFrameDidNotSwap(
93 const cc::BeginFrameAck& begin_frame_ack) { 98 const cc::BeginFrameAck& begin_frame_ack) {
94 support_->BeginFrameDidNotSwap(begin_frame_ack); 99 support_->BeginFrameDidNotSwap(begin_frame_ack);
95 } 100 }
96 101
97 void GpuRootCompositorFrameSink::ClaimTemporaryReference( 102 void GpuRootCompositorFrameSink::ClaimTemporaryReference(
98 const cc::SurfaceId& surface_id) { 103 const cc::SurfaceId& surface_id) {
99 support_->ClaimTemporaryReference(surface_id); 104 support_->ClaimTemporaryReference(surface_id);
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
144 } 149 }
145 150
146 void GpuRootCompositorFrameSink::OnPrivateConnectionLost() { 151 void GpuRootCompositorFrameSink::OnPrivateConnectionLost() {
147 private_connection_lost_ = true; 152 private_connection_lost_ = true;
148 // Request destruction of |this| only if both connections are lost. 153 // Request destruction of |this| only if both connections are lost.
149 delegate_->OnPrivateConnectionLost(support_->frame_sink_id(), 154 delegate_->OnPrivateConnectionLost(support_->frame_sink_id(),
150 client_connection_lost_); 155 client_connection_lost_);
151 } 156 }
152 157
153 } // namespace viz 158 } // namespace viz
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698