Change Chromoting client to use Pepper's new Resource-base InputEvents.

remoting/client/plugin/pepper_view_proxy.h

 // Copyright (c) 2011 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.
 
 // PepperViewProxy is used to invoke PepperView object on pepper thread. It
 // has the same interface as PepperView. When a method calls is received on
 // any chromoting threads it delegates the method call to pepper thread.
 // It also provide a detach mechanism so that when PepperView object is
 // destroyed PepperViewProxy will not call it anymore. This is important in
 // providing a safe shutdown of ChromotingInstance and PepperView.
@@ skipping 23 matching lines @@
   // ChromotingView implementation.
   virtual bool Initialize() OVERRIDE;
   virtual void TearDown() OVERRIDE;
   virtual void Paint() OVERRIDE;
   virtual void SetSolidFill(uint32 color) OVERRIDE;
   virtual void UnsetSolidFill() OVERRIDE;
   virtual void SetConnectionState(ConnectionState state) OVERRIDE;
   virtual void UpdateLoginStatus(bool success, const std::string& info)
       OVERRIDE;
   virtual void SetViewport(int x, int y, int width, int height) OVERRIDE;
-  // This method returns a value, so must run synchronously, so must be
-  // called only on the pepper thread.
-  virtual gfx::Point ConvertScreenToHost(const gfx::Point& p) const OVERRIDE;
 
   // FrameConsumer implementation.
   virtual void AllocateFrame(media::VideoFrame::Format format,
                              size_t width,
                              size_t height,
                              base::TimeDelta timestamp,
                              base::TimeDelta duration,
                              scoped_refptr<media::VideoFrame>* frame_out,
                              Task* done);
   virtual void ReleaseFrame(media::VideoFrame* frame);
   virtual void OnPartialFrameOutput(media::VideoFrame* frame,
                                     UpdatedRects* rects,
                                     Task* done);
 
   void SetScaleToFit(bool scale_to_fit);
 
+  // This method returns a value, so must run synchronously, so must be
+  // called only on the pepper thread.

[Wez, 2011/07/20 00:58:35]

nit: How does this differ from the other methods on this class?

[garykac, 2011/07/20 18:55:28]

It returns a value, whereas all the other methods return void.

I think I'm misunderstanding your question here.

+  pp::Point ConvertScreenToHost(const pp::Point& p) const;
+
   // Remove the reference to |instance_| and |view_| by setting the value to
   // NULL.
   // This method should only be called on pepper thread.
   void Detach();
 
  private:
   // This variable is accessed on chromoting threads and pepper thread.
   // This is initialized when this object is constructed. Its value is reset
   // to NULL on pepper thread when Detach() is called and there will be no
   // other threads accessing this variable at the same time. Given the above
   // conditions locking this variable is not necessary.
   ChromotingInstance* instance_;
 
   // This variable is only accessed on the pepper thread. Locking is not
   // necessary.
   PepperView* view_;
 
   DISALLOW_COPY_AND_ASSIGN(PepperViewProxy);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_PLUGIN_PEPPER_VIEW_PROXY_H_