Refactor GuestView construction

chrome/browser/guest_view/app_view/app_view_guest.h

 // Copyright 2014 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.
 
 #ifndef CHROME_BROWSER_GUEST_VIEW_APP_VIEW_APP_VIEW_GUEST_H_
 #define CHROME_BROWSER_GUEST_VIEW_APP_VIEW_APP_VIEW_GUEST_H_
 
 #include "base/id_map.h"
 #include "chrome/browser/guest_view/guest_view.h"
 #include "extensions/browser/extension_function_dispatcher.h"
 
 namespace extensions {
 class Extension;
 class ExtensionHost;
 };
 
 // An AppViewGuest provides the browser-side implementation of <appview> API.
-// AppViewGuest is created on attachment. That is, when a guest WEbContnets is
+// AppViewGuest is created on attachment. That is, when a guest WebContents is
 // associated with a particular embedder WebContents. This happens on calls to
 // the connect API.
 class AppViewGuest : public GuestView<AppViewGuest>,
                      public extensions::ExtensionFunctionDispatcher::Delegate {
  public:
   static const char Type[];
 
   // Completes the creation of a WebContents associated with the provided
   // |guest_extensions_id|.
   static bool CompletePendingRequest(
       content::BrowserContext* browser_context,
       const GURL& url,
       int guest_instance_id,
       const std::string& guest_extension_id);
 
-  AppViewGuest(content::BrowserContext* browser_context,
-               int guest_instance_id);
+  static GuestViewBase* Create(content::BrowserContext* browser_context,
+                               int guest_instance_id);
 
   // ExtensionFunctionDispatcher::Delegate implementation.
   virtual extensions::WindowController* GetExtensionWindowController() const
       OVERRIDE;
   virtual content::WebContents* GetAssociatedWebContents() const OVERRIDE;
 
   // content::WebContentsObserver implementation.
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
 
   // content::WebContentsDelegate implementation.
   virtual bool HandleContextMenu(
       const content::ContextMenuParams& params) OVERRIDE;
 
   // GuestViewBase implementation.
   virtual bool CanEmbedderUseGuestView(
       const std::string& embedder_extension_id) OVERRIDE;
   virtual void CreateWebContents(
       const std::string& embedder_extension_id,
       int embedder_render_process_id,
       const base::DictionaryValue& create_params,
       const WebContentsCreatedCallback& callback) OVERRIDE;
   virtual void DidAttachToEmbedder() OVERRIDE;
   virtual void DidInitialize() OVERRIDE;
 
  private:
+  AppViewGuest(content::BrowserContext* browser_context,
+               int guest_instance_id);
+
   virtual ~AppViewGuest();
 
   void OnRequest(const ExtensionHostMsg_Request_Params& params);
 
   void CompleteCreateWebContents(const GURL& url,
                                  const extensions::Extension* guest_extension,
                                  const WebContentsCreatedCallback& callback);
 
   void LaunchAppAndFireEvent(const WebContentsCreatedCallback& callback,
                              extensions::ExtensionHost* extension_host);
 
   GURL url_;
   std::string guest_extension_id_;
   scoped_ptr<extensions::ExtensionFunctionDispatcher>
       extension_function_dispatcher_;
 
   // This is used to ensure pending tasks will not fire after this object is
   // destroyed.
   base::WeakPtrFactory<AppViewGuest> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(AppViewGuest);
 };
 
 #endif  // CHROME_BROWSER_GUEST_VIEW_APP_VIEW_APP_VIEW_GUEST_H_