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

remoting/client/plugin/pepper_view_proxy.h

Index: remoting/client/plugin/pepper_view_proxy.h
diff --git a/remoting/client/plugin/pepper_view_proxy.h b/remoting/client/plugin/pepper_view_proxy.h
index d9075442b8ad13a741991582bdb02441e8559794..a34a3c512aab7d41eaca565270e129b6a10e864d 100644
--- a/remoting/client/plugin/pepper_view_proxy.h
+++ b/remoting/client/plugin/pepper_view_proxy.h
@@ -41,9 +41,6 @@ class PepperViewProxy : public base::RefCountedThreadSafe<PepperViewProxy>,
   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,
@@ -60,6 +57,10 @@ class PepperViewProxy : public base::RefCountedThreadSafe<PepperViewProxy>,
 
   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.