Mandoline: Allow submitting CompositorFrames directly to mojo::Views

components/html_viewer/html_frame.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/html_viewer/html_frame.h"
 
 #include <algorithm>
 #include <limits>
 
 #include "base/bind.h"
@@ skipping 581 matching lines @@
 
 void HTMLFrame::didCancelCompositionOnSelectionChange() {
   // TODO(penghuang): Update text input state.
 }
 
 void HTMLFrame::didChangeContents() {
   // TODO(penghuang): Update text input state.
 }
 
 void HTMLFrame::initializeLayerTreeView() {
-  mojo::URLRequestPtr request(mojo::URLRequest::New());
-  request->url = mojo::String::From("mojo:view_manager");
-  mojo::SurfacePtr surface;
-  GetLocalRootApp()->ConnectToService(request.Pass(), &surface);
+  DCHECK(view());
+
+  mojo::CompositorFrameReceiverPtr receiver;
+  view()->RequestCompositorFrameReceiver(GetProxy(&receiver));

[rjkroege, 2015/08/19 23:45:44]

This mojo GetProxy name overlaps badly with the cc proxy stuff. Oh well

[Fady Samuel, 2015/08/20 16:22:44]

Acknowledged.

[rjkroege, 2015/08/21 18:54:45]

perhaps you could do mojo::GetProxy for clarity?

[Fady Samuel, 2015/08/21 21:31:10]

Done.

 
   mojo::URLRequestPtr request2(mojo::URLRequest::New());
   request2->url = mojo::String::From("mojo:view_manager");
   mojo::GpuPtr gpu_service;
   GetLocalRootApp()->ConnectToService(request2.Pass(), &gpu_service);
   web_layer_tree_view_impl_.reset(new WebLayerTreeViewImpl(
       global_state()->compositor_thread(),
       global_state()->gpu_memory_buffer_manager(),
       global_state()->raster_thread_helper()->task_graph_runner(),
-      surface.Pass(), gpu_service.Pass()));
+      receiver.Pass(), gpu_service.Pass()));
 }
 
 blink::WebLayerTreeView* HTMLFrame::layerTreeView() {
   return web_layer_tree_view_impl_.get();
 }
 
 void HTMLFrame::resetInputMethod() {
   // When this method gets called, WebWidgetClient implementation should
   // reset the input method by cancelling any ongoing composition.
   // TODO(penghuang): Reset IME.
@@ skipping 276 matching lines @@
 
 void HTMLFrame::reload(bool ignore_cache, bool is_client_redirect) {
   NOTIMPLEMENTED();
 }
 
 void HTMLFrame::forwardInputEvent(const blink::WebInputEvent* event) {
   NOTIMPLEMENTED();
 }
 
 }  // namespace mojo