Minor cleanup of resource creation. This eliminates a bunch of copy-and-paste

code for determining the instance for every request, and also removes a bunch
of unnecessary Create() functions for simple resources.

TEST=none
BUG=none
Review URL: http://codereview.chromium.org/7233005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90111 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 5 insertions, 4 deletions

diff --git a/webkit/plugins/ppapi/ppb_flash_menu_impl.h b/webkit/plugins/ppapi/ppb_flash_menu_impl.h
index 455b973..dcb56e4 100644
--- a/webkit/plugins/ppapi/ppb_flash_menu_impl.h
+++ b/webkit/plugins/ppapi/ppb_flash_menu_impl.h
@@ -24,14 +24,11 @@ namespace ppapi {
 class PPB_Flash_Menu_Impl : public Resource,
                             public ::ppapi::thunk::PPB_Flash_Menu_API {
  public:
-  explicit PPB_Flash_Menu_Impl(PluginInstance* instance);
   virtual ~PPB_Flash_Menu_Impl();
 
-  static PP_Resource Create(PP_Instance instance,
+  static PP_Resource Create(PluginInstance* instance,
                             const PP_Flash_Menu* menu_data);
 
-  bool Init(const PP_Flash_Menu* menu_data);
-
   // ResourceObjectBase.
   virtual ::ppapi::thunk::PPB_Flash_Menu_API* AsPPB_Flash_Menu_API() OVERRIDE;
 
@@ -47,6 +44,10 @@ class PPB_Flash_Menu_Impl : public Resource,
   const MenuData& menu_data() const { return menu_data_; }
 
  private:
+  explicit PPB_Flash_Menu_Impl(PluginInstance* instance);
+
+  bool Init(const PP_Flash_Menu* menu_data);
+
   MenuData menu_data_;
 
   // We send |WebMenuItem|s, which have an |unsigned| "action" field instead of