3D Compositing in <browser>, first draft.

content/renderer/browser_plugin/browser_plugin.cc

 // Copyright (c) 2012 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 "content/renderer/browser_plugin/browser_plugin.h"
 
 #include "base/message_loop.h"
 #include "base/string_util.h"
 #include "content/common/browser_plugin_messages.h"
 #include "content/browser/renderer_host/backing_store.h"
@@ skipping 58 matching lines @@
   RenderThreadImpl::current()->browser_plugin_manager()->
       RemoveBrowserPlugin(id_);
 }
 
 void BrowserPlugin::SetSrcAttribute(const std::string& src) {
   // TODO(fsamuel): What if the BrowserPlugin element's parent frame
   // changes?
   if (src == src_ && !guest_crashed_)
     return;
   if (!src.empty()) {
-    render_view_->Send(new BrowserPluginHostMsg_NavigateFromEmbedder(
-        render_view_->GetRoutingID(),
-        id_,
-        parent_frame_,
-        src));
+    WebKit::WebView* web_view = render_view_->webview();
+    WebGraphicsContext3DCommandBufferImpl* context = static_cast<WebGraphicsContext3DCommandBufferImpl*>(web_view->sharedGraphicsContext3D());
+    DCHECK(context);
+    BrowserPluginHostMsg_Surface_Params params;
+    params.gpu_process_id = context->GetGPUProcessID();
+    params.client_id = context->GetChannelID();
+    params.context_id = context->GetContextID();
+    params.texture_id[0] = context->createTexture();
+    params.texture_id[1] = context->createTexture();
+    params.sync_point = context->insertSyncPoint();
+    printf("Giving guest textures %u and %u\n", params.texture_id[0], params.texture_id[1]);
+    if (web_view) {
+      render_view_->Send(new BrowserPluginHostMsg_NavigateFromEmbedder(
+          render_view_->GetRoutingID(),
+          id_,
+          parent_frame_,
+          src,
+          params));
+    }
   }
   src_ = src;
   guest_crashed_ = false;
 }
 
 int BrowserPlugin::GetBackingStoreWidth() const {
   return backing_store_.get() ? backing_store_->GetSize().width() : 0;
 }
 
 int BrowserPlugin::GetBackingStoreHeight() const {
@@ skipping 49 matching lines @@
 
 void BrowserPlugin::UpdateURL(const GURL& url) {
   src_ = url.spec();
   // TODO(fsamuel): Send a custom message?
 }
 
 void BrowserPlugin::AdvanceFocus(bool reverse) {
   render_view_->GetWebView()->advanceFocus(reverse);
 }
 
+void BrowserPlugin::BuffersSwapped(uint64 surface_handle, const BrowserPlugin_SwapInfo& info) {
+  DCHECK(surface_handle != 0);
+  fprintf(stderr, "Browser plugin swapping buffers on surface %lu\n", surface_handle);
+  container_->setBackingTextureId(surface_handle);
+  container_->commitBackingTexture();
+  compositing_callbacks_.push_back(base::Bind(&BrowserPlugin::PerformBuffersSwappedACK, base::Unretained(this), info));

[Fady Samuel, 2012/07/06 15:14:44]

So are you sure you're delaying ACK'ing enough here? WillInitiatePaint is called
BEFORE composite in RenderWidget::DoDeferredUpdate. It sounds like it's possible
that while we're compositing, the guest might be off drawing the next frame? I
guess it's using the other texture as the back buffer at this point? Maybe this
makes sense then? I'm not sure. We will have to discuss.

[scshunt, 2012/07/06 16:39:03]

I was not, and that was the cause of the flicker. I have changed the call
location in my build in RenderViewImpl from WillInitiatePaint to
DidInitiatePaint and the flicker went away.

+}
+
 void BrowserPlugin::PostMessage(const std::string& message,
                                 const std::string& target_origin) {
   fprintf(stderr, ">>>%s message: \"%s\"\n", __PRETTY_FUNCTION__,
           message.c_str());
 }
 
+void BrowserPlugin::WillInitiatePaint() {
+  for (std::vector< base::Callback<void(void)> >::const_iterator
+       it = compositing_callbacks_.begin();

[Fady Samuel, 2012/07/06 15:14:44]

Do we ever really expect to get multiple compositing callbacks here?

[scshunt, 2012/07/06 16:39:03]

Not sure. It occurs to me that it would probably be better to store a single
callback change on the RenderViewImpl in any case.

+       it != compositing_callbacks_.end(); ++it) {
+    it->Run();
+  }
+  compositing_callbacks_.clear();
+}
+
+void BrowserPlugin::PerformBuffersSwappedACK(const BrowserPlugin_SwapInfo& info) {
+  WebKit::WebView* webview = render_view_->webview();
+  DCHECK(webview);
+  WebGraphicsContext3DCommandBufferImpl* buffer = static_cast<WebGraphicsContext3DCommandBufferImpl*>(webview->sharedGraphicsContext3D());
+  DCHECK(buffer);
+  uint32 sync_point = buffer->insertSyncPoint();
+  render_view_->Send(new BrowserPluginHostMsg_BuffersSwappedACK(render_view_->GetRoutingID(), info, sync_point));
+  fprintf(stderr, "Sent browser plugin buffers swapped ACK\n");
+}
+
 WebKit::WebPluginContainer* BrowserPlugin::container() const {
   return container_;
 }
 
 bool BrowserPlugin::initialize(WebPluginContainer* container) {
   container_ = container;
   return true;
 }
 
 void BrowserPlugin::destroy() {
@@ skipping 116 matching lines @@
 }
 
 void BrowserPlugin::didFailLoadingFrameRequest(
     const WebKit::WebURL& url,
     void* notify_data,
     const WebKit::WebURLError& error) {
 }
 
 }  // namespace browser_plugin
 }  // namespace content