Mandoline: Allow submitting CompositorFrames directly to mojo::Views

mojo/cc/output_surface_mojo.cc

 // Copyright 2014 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 "mojo/cc/output_surface_mojo.h"
 
 #include "base/bind.h"
 #include "cc/output/compositor_frame.h"
 #include "cc/output/output_surface_client.h"
 #include "mojo/converters/geometry/geometry_type_converters.h"
 #include "mojo/converters/surfaces/surfaces_type_converters.h"
 
 namespace mojo {
 
 OutputSurfaceMojo::OutputSurfaceMojo(
     OutputSurfaceMojoClient* client,
     const scoped_refptr<cc::ContextProvider>& context_provider,
-    ScopedMessagePipeHandle surface_handle)
+    ScopedMessagePipeHandle compositor_frame_receiver_handle)
     : cc::OutputSurface(context_provider),
       output_surface_mojo_client_(client),
-      surface_handle_(surface_handle.Pass()),
+      compositor_frame_receiver_handle_(
+          compositor_frame_receiver_handle.Pass()),
       id_namespace_(0u),
       local_id_(0u) {
   capabilities_.delegated_rendering = true;
   capabilities_.max_frames_pending = 1;
 }
 
 OutputSurfaceMojo::~OutputSurfaceMojo() {
 }
 
 void OutputSurfaceMojo::SetIdNamespace(uint32_t id_namespace) {
   id_namespace_ = id_namespace;
   if (local_id_) {
     cc::SurfaceId qualified_id(static_cast<uint64_t>(id_namespace_) << 32 |
                                local_id_);
     output_surface_mojo_client_->DidCreateSurface(qualified_id);
   }
 }
 
 bool OutputSurfaceMojo::BindToClient(cc::OutputSurfaceClient* client) {
-  surface_.Bind(InterfacePtrInfo<Surface>(surface_handle_.Pass(), 0u));
-  surface_->GetIdNamespace(
-      base::Bind(&OutputSurfaceMojo::SetIdNamespace, base::Unretained(this)));
+  receiver_.Bind(InterfacePtrInfo<CompositorFrameReceiver>(
+      compositor_frame_receiver_handle_.Pass(), 0u));
   return cc::OutputSurface::BindToClient(client);
 }
 
 void OutputSurfaceMojo::SwapBuffers(cc::CompositorFrame* frame) {
-  gfx::Size frame_size =
-      frame->delegated_frame_data->render_pass_list.back()->output_rect.size();
-  if (frame_size != surface_size_) {
-    if (local_id_ != 0u) {
-      surface_->DestroySurface(local_id_);
-    }
-    local_id_++;
-    surface_->CreateSurface(local_id_);
-    if (id_namespace_) {
-      cc::SurfaceId qualified_id(static_cast<uint64_t>(id_namespace_) << 32 |
-                                 local_id_);
-      output_surface_mojo_client_->DidCreateSurface(qualified_id);
-    }
-    surface_size_ = frame_size;
-  }
-
-  surface_->SubmitFrame(local_id_,
-                        CompositorFrame::From(*frame),
-                        mojo::Closure());
+  receiver_->SubmitFrame(CompositorFrame::From(*frame),

[rjkroege, 2015/08/13 18:04:01]

what thread and process does this code run on?

[Fady Samuel, 2015/08/17 21:11:11]

html_viewer on the compositor thread currently. However, this isn't a part of
the html_viewer component (it's common code) so I don't want to refer to
html_viewer in this file.

+                         mojo::Closure());
 
   client_->DidSwapBuffers();
   client_->DidSwapBuffersComplete();
 }
 
 }  // namespace mojo