Enforce constant size and device scale factor for surfaces

components/viz/frame_sinks/gpu_compositor_frame_sink.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/viz/frame_sinks/gpu_compositor_frame_sink.h"
 
 namespace viz {
 
 GpuCompositorFrameSink::GpuCompositorFrameSink(
     GpuCompositorFrameSinkDelegate* delegate,
@@ skipping 29 matching lines @@
   support_->EvictCurrentSurface();
 }
 
 void GpuCompositorFrameSink::SetNeedsBeginFrame(bool needs_begin_frame) {
   support_->SetNeedsBeginFrame(needs_begin_frame);
 }
 
 void GpuCompositorFrameSink::SubmitCompositorFrame(
     const cc::LocalSurfaceId& local_surface_id,
     cc::CompositorFrame frame) {
-  support_->SubmitCompositorFrame(local_surface_id, std::move(frame));
+  bool result =

[danakj, 2017/05/19 19:31:39]

it's okay to keep this without the bool var, I think I'd prefer that

+      support_->SubmitCompositorFrame(local_surface_id, std::move(frame));
+  if (!result) {
+    compositor_frame_sink_binding_.Close();
+    OnClientConnectionLost();
+  }
 }
 
 void GpuCompositorFrameSink::BeginFrameDidNotSwap(
     const cc::BeginFrameAck& begin_frame_ack) {
   support_->BeginFrameDidNotSwap(begin_frame_ack);
 }
 
 void GpuCompositorFrameSink::DidReceiveCompositorFrameAck(
     const cc::ReturnedResourceArray& resources) {
   if (client_)
@@ skipping 33 matching lines @@
 }
 
 void GpuCompositorFrameSink::OnPrivateConnectionLost() {
   private_connection_lost_ = true;
   // Request destruction of |this| only if both connections are lost.
   delegate_->OnPrivateConnectionLost(support_->frame_sink_id(),
                                      client_connection_lost_);
 }
 
 }  // namespace viz