refactoring mouse lock to support pepper and WebKit

content/renderer/pepper_plugin_delegate_impl.h

Index: content/renderer/pepper_plugin_delegate_impl.h
diff --git a/content/renderer/pepper_plugin_delegate_impl.h b/content/renderer/pepper_plugin_delegate_impl.h
index fc2e3576de30e149bb9cc96d38b7e70e4ab6748a..6baced5cc3a475c07659e8d8d6e6d8110150fb4d 100644
--- a/content/renderer/pepper_plugin_delegate_impl.h
+++ b/content/renderer/pepper_plugin_delegate_impl.h
@@ -189,15 +189,8 @@ class PepperPluginDelegateImpl
       int selection_end);
   void OnImeConfirmComposition(const string16& text);
 
-  // Notification that the request to lock the mouse has completed.
-  void OnLockMouseACK(bool succeeded);
-  // Notification that the plugin instance has lost the mouse lock.
-  void OnMouseLockLost();
   // Notification that a mouse event has arrived at the render view.
-  // Returns true if no further handling is needed. For example, if the mouse is
-  // currently locked, this method directly dispatches the event to the owner of
-  // the mouse lock and returns true.
-  bool HandleMouseEvent(const WebKit::WebMouseEvent& event);
+  void WillHandleMouseEvent();
 
   // PluginDelegate implementation.
   virtual void PluginFocusChanged(webkit::ppapi::PluginInstance* instance,
@@ -375,7 +368,7 @@ class PepperPluginDelegateImpl
   virtual base::SharedMemory* CreateAnonymousSharedMemory(uint32_t size)
       OVERRIDE;
   virtual ::ppapi::Preferences GetPreferences() OVERRIDE;
-  virtual void LockMouse(webkit::ppapi::PluginInstance* instance) OVERRIDE;
+  virtual bool LockMouse(webkit::ppapi::PluginInstance* instance) OVERRIDE;
   virtual void UnlockMouse(webkit::ppapi::PluginInstance* instance) OVERRIDE;
   virtual void DidChangeCursor(webkit::ppapi::PluginInstance* instance,
                                const WebKit::WebCursorInfo& cursor) OVERRIDE;
@@ -423,10 +416,6 @@ class PepperPluginDelegateImpl
   scoped_refptr<PpapiBrokerImpl> CreatePpapiBroker(
       webkit::ppapi::PluginModule* plugin_module);
 
-  bool MouseLockedOrPending() const {
-    return mouse_locked_ || pending_lock_request_ || pending_unlock_request_;
-  }
-
   // Implementation of PepperParentContextProvider.
   virtual RendererGLContext* GetParentContextForPlatformContext3D() OVERRIDE;
 
@@ -471,21 +460,6 @@ class PepperPluginDelegateImpl
   // progress.
   string16 composition_text_;
 
-  // |mouse_lock_owner_| is not owned by this class. We can know about when it
-  // is destroyed via InstanceDeleted().
-  // |mouse_lock_owner_| being non-NULL doesn't indicate that currently the
-  // mouse has been locked. It is possible that a request to lock the mouse has
-  // been sent, but the response hasn't arrived yet.
-  webkit::ppapi::PluginInstance* mouse_lock_owner_;
-  bool mouse_locked_;
-  // If both |pending_lock_request_| and |pending_unlock_request_| are true,
-  // it means a lock request was sent before an unlock request and we haven't
-  // received responses for them.
-  // The logic in LockMouse() makes sure that a lock request won't be sent when
-  // there is a pending unlock request.
-  bool pending_lock_request_;
-  bool pending_unlock_request_;
-
   // The plugin instance that received the last mouse event. It is set to NULL
   // if the last mouse event went to elements other than Pepper plugins.
   // |last_mouse_event_target_| is not owned by this class. We can know about