Reland "Enforce constant size and device scale factor for surfaces" with fixes

ui/android/delegated_frame_host_android.cc

 // Copyright 2016 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 "ui/android/delegated_frame_host_android.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "cc/layers/solid_color_layer.h"
@@ skipping 67 matching lines @@
   if (local_surface_id != surface_info_.id().local_surface_id()) {
     DestroyDelegatedContent();
     DCHECK(!content_layer_);
 
     cc::RenderPass* root_pass = frame.render_pass_list.back().get();
     gfx::Size frame_size = root_pass->output_rect.size();
     surface_info_ = cc::SurfaceInfo(
         cc::SurfaceId(frame_sink_id_, local_surface_id), 1.f, frame_size);
     has_transparent_background_ = root_pass->has_transparent_background;
 
-    support_->SubmitCompositorFrame(local_surface_id, std::move(frame));
+    bool result =
+        support_->SubmitCompositorFrame(local_surface_id, std::move(frame));
+    DCHECK(result);
+
     content_layer_ = CreateSurfaceLayer(surface_manager_, surface_info_,
                                         !has_transparent_background_);
     view_->GetLayer()->AddChild(content_layer_);
   } else {
     support_->SubmitCompositorFrame(local_surface_id, std::move(frame));
   }
 }
 
 void DelegatedFrameHostAndroid::DidNotProduceFrame(
     const cc::BeginFrameAck& ack) {
@@ skipping 97 matching lines @@
   support_ = cc::CompositorFrameSinkSupport::Create(
       this, surface_manager_, frame_sink_id_, is_root,
       handles_frame_sink_id_invalidation, needs_sync_points);
 }
 
 cc::SurfaceId DelegatedFrameHostAndroid::SurfaceId() const {
   return surface_info_.id();
 }
 
 }  // namespace ui