Implement WebCompositorOutputSurface

content/renderer/gpu/compositor_thread.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/gpu/compositor_thread.h"
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
+#include "content/common/view_messages.h"
 #include "content/renderer/gpu/input_event_filter.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebActiveWheelFlingParameters.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebCompositorInputHandlerClient.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebCompositorInputHandler.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebView.h"
 
 using WebKit::WebCompositorInputHandler;
 using WebKit::WebInputEvent;
 
 //------------------------------------------------------------------------------
@@ skipping 28 matching lines @@
     return input_handler_;
   }
 
   // WebCompositorInputHandlerClient methods:
 
   virtual void willShutdown() {
     compositor_thread_->RemoveInputHandler(routing_id_);
   }
 
   virtual void didHandleInputEvent() {
-    compositor_thread_->filter_->DidHandleInputEvent();
+    compositor_thread_->input_filter_->DidHandleInputEvent();
   }
 
   virtual void didNotHandleInputEvent(bool send_to_widget) {
-    compositor_thread_->filter_->DidNotHandleInputEvent(send_to_widget);
+    compositor_thread_->input_filter_->DidNotHandleInputEvent(send_to_widget);
   }
 
  private:
   friend class base::RefCountedThreadSafe<InputHandlerWrapper>;
 
   virtual ~InputHandlerWrapper() {
     input_handler_->setClient(NULL);
   }
 
   CompositorThread* compositor_thread_;
   int routing_id_;
   WebKit::WebCompositorInputHandler* input_handler_;
   scoped_refptr<base::MessageLoopProxy> main_loop_;
 
   // Can only be accessed on the main thread.
   base::WeakPtr<RenderViewImpl> render_view_impl_;
 
   DISALLOW_COPY_AND_ASSIGN(InputHandlerWrapper);
 };
 
 //------------------------------------------------------------------------------
 
 CompositorThread::CompositorThread(IPC::Listener* main_listener)
     : thread_("Compositor") {
-  filter_ =
+  input_filter_ =
       new InputEventFilter(main_listener,
                            thread_.message_loop()->message_loop_proxy(),
                            base::Bind(&CompositorThread::HandleInputEvent,
                                       base::Unretained(this)));
+
+  uint32 messages[] = {ViewMsg_UpdateVSyncParameters::ID};

[darin (slow to review), 2012/07/24 23:56:43]

nit: const uint32 kMessages[] = { ViewMsg_...::ID };

+  compositor_filter_ =
+    new IPC::ForwardingMessageFilter(
+        messages, arraysize(messages),
+        thread_.message_loop()->message_loop_proxy(),
+        base::Bind(&CompositorThread::OnCompositorMessageReceived,
+                   base::Unretained(this)));
 }
 
 CompositorThread::~CompositorThread() {
 }
 
-IPC::ChannelProxy::MessageFilter* CompositorThread::GetMessageFilter() const {
-  return filter_;
+IPC::ChannelProxy::MessageFilter* CompositorThread::GetInputFilter() const {
+  return input_filter_;
 }
 
-void CompositorThread::AddInputHandler(
+IPC::ChannelProxy::MessageFilter*
+    CompositorThread::GetCompositorFilter() const {
+  return compositor_filter_;
+}
+

[darin (slow to review), 2012/07/24 23:56:43]

nit: extra new line

+
+void CompositorThread::AddHandlers(
     int routing_id, int input_handler_id,
     const base::WeakPtr<RenderViewImpl>& render_view_impl) {
   DCHECK_NE(thread_.message_loop(), MessageLoop::current());
 
   thread_.message_loop()->PostTask(
       FROM_HERE,
-      base::Bind(&CompositorThread::AddInputHandlerOnCompositorThread,
+      base::Bind(&CompositorThread::AddHandlersOnCompositorThread,
                  base::Unretained(this),
                  routing_id,
                  input_handler_id,
                  base::MessageLoopProxy::current(),
                  render_view_impl));
 }
 
-void CompositorThread::AddInputHandlerOnCompositorThread(
+void CompositorThread::AddHandlersOnCompositorThread(
     int routing_id, int input_handler_id,
     scoped_refptr<base::MessageLoopProxy> main_loop,
     const base::WeakPtr<RenderViewImpl>& render_view_impl) {
 
   DCHECK_EQ(thread_.message_loop(), MessageLoop::current());
 
   WebCompositorInputHandler* input_handler =
       WebCompositorInputHandler::fromIdentifier(input_handler_id);
   if (!input_handler)
     return;
 
   if (input_handlers_.count(routing_id) != 0) {
     // It's valid to call AddInputHandler() for the same routing id with the
     // same input_handler many times, but it's not valid to change the
     // input_handler for a route.
     DCHECK_EQ(input_handlers_[routing_id]->input_handler(), input_handler);
     return;
   }
 
-  TRACE_EVENT0("CompositorThread::AddInputHandler", "AddingRoute");
-  filter_->AddRoute(routing_id);
+  TRACE_EVENT0("renderer", "CompositorThread::AddHandlersOnCompositorThread");
+  input_filter_->AddRoute(routing_id);
+  compositor_filter_->AddRoute(routing_id);
   input_handlers_[routing_id] =
       make_scoped_refptr(new InputHandlerWrapper(this,
           routing_id, input_handler, main_loop, render_view_impl));
 }
 
 void CompositorThread::RemoveInputHandler(int routing_id) {
   DCHECK_EQ(MessageLoop::current(), thread_.message_loop());
 
-  TRACE_EVENT0("CompositorThread::RemoveInputHandler", "RemovingRoute");
+  TRACE_EVENT0("renderer", "CompositorThread::RemoveInputHandler");
 
-  filter_->RemoveRoute(routing_id);
+  input_filter_->RemoveRoute(routing_id);
+  compositor_filter_->RemoveRoute(routing_id);
   input_handlers_.erase(routing_id);
 }
 
 void CompositorThread::HandleInputEvent(
     int routing_id,
     const WebInputEvent* input_event) {
+  TRACE_EVENT0("renderer", "CompositorThread::HandleInputEvent");
   DCHECK_EQ(MessageLoop::current(), thread_.message_loop());
 
   InputHandlerMap::iterator it = input_handlers_.find(routing_id);
   if (it == input_handlers_.end()) {
-    TRACE_EVENT0("CompositorThread::HandleInputEvent", "NoInputHandlerFound");
+    TRACE_EVENT0("renderer", "NoInputHandlerFound");
     // Oops, we no longer have an interested input handler..
-    filter_->DidNotHandleInputEvent(true);
+    input_filter_->DidNotHandleInputEvent(true);
     return;
   }
 
   it->second->input_handler()->handleInputEvent(*input_event);
 }
+
+void CompositorThread::OnCompositorMessageReceived(
+    const IPC::Message& message) {
+  TRACE_EVENT0("renderer", "CompositorThread::OnCompositorMessageReceived");
+  DCHECK_EQ(message.type(), ViewMsg_UpdateVSyncParameters::ID);
+
+  InputHandlerMap::iterator it = input_handlers_.find(message.routing_id());

[darin (slow to review), 2012/07/24 23:56:43]

on the webkit side, perhaps we should be pushing these parameters to a different
interface than the input handler?  maybe that's overloading the concept of input
handler too much?

+  if (it == input_handlers_.end())
+    return;
+
+  // TODO(nduca): Push the updated vsync paramters to the compositor.
+}