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

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

Issue 2848223003: Enforce constant size and device scale factor for surfaces (Closed)
Patch Set: Address comments 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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_compositor_frame_sink.h" 5 #include "components/viz/frame_sinks/gpu_compositor_frame_sink.h"
6 6
7 namespace viz { 7 namespace viz {
8 8
9 GpuCompositorFrameSink::GpuCompositorFrameSink( 9 GpuCompositorFrameSink::GpuCompositorFrameSink(
10 GpuCompositorFrameSinkDelegate* delegate, 10 GpuCompositorFrameSinkDelegate* delegate,
(...skipping 29 matching lines...) Expand all
40 support_->EvictCurrentSurface(); 40 support_->EvictCurrentSurface();
41 } 41 }
42 42
43 void GpuCompositorFrameSink::SetNeedsBeginFrame(bool needs_begin_frame) { 43 void GpuCompositorFrameSink::SetNeedsBeginFrame(bool needs_begin_frame) {
44 support_->SetNeedsBeginFrame(needs_begin_frame); 44 support_->SetNeedsBeginFrame(needs_begin_frame);
45 } 45 }
46 46
47 void GpuCompositorFrameSink::SubmitCompositorFrame( 47 void GpuCompositorFrameSink::SubmitCompositorFrame(
48 const cc::LocalSurfaceId& local_surface_id, 48 const cc::LocalSurfaceId& local_surface_id,
49 cc::CompositorFrame frame) { 49 cc::CompositorFrame frame) {
50 support_->SubmitCompositorFrame(local_surface_id, std::move(frame)); 50 if (!support_->SubmitCompositorFrame(local_surface_id, std::move(frame))) {
51 compositor_frame_sink_binding_.Close();
52 OnClientConnectionLost();
53 }
51 } 54 }
52 55
53 void GpuCompositorFrameSink::DidNotProduceFrame( 56 void GpuCompositorFrameSink::DidNotProduceFrame(
54 const cc::BeginFrameAck& begin_frame_ack) { 57 const cc::BeginFrameAck& begin_frame_ack) {
55 support_->DidNotProduceFrame(begin_frame_ack); 58 support_->DidNotProduceFrame(begin_frame_ack);
56 } 59 }
57 60
58 void GpuCompositorFrameSink::DidReceiveCompositorFrameAck( 61 void GpuCompositorFrameSink::DidReceiveCompositorFrameAck(
59 const cc::ReturnedResourceArray& resources) { 62 const cc::ReturnedResourceArray& resources) {
60 if (client_) 63 if (client_)
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 } 97 }
95 98
96 void GpuCompositorFrameSink::OnPrivateConnectionLost() { 99 void GpuCompositorFrameSink::OnPrivateConnectionLost() {
97 private_connection_lost_ = true; 100 private_connection_lost_ = true;
98 // Request destruction of |this| only if both connections are lost. 101 // Request destruction of |this| only if both connections are lost.
99 delegate_->OnPrivateConnectionLost(support_->frame_sink_id(), 102 delegate_->OnPrivateConnectionLost(support_->frame_sink_id(),
100 client_connection_lost_); 103 client_connection_lost_);
101 } 104 }
102 105
103 } // namespace viz 106 } // namespace viz
OLDNEW
« no previous file with comments | « cc/test/test_compositor_frame_sink.cc ('k') | components/viz/frame_sinks/gpu_root_compositor_frame_sink.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698