Cleanup: Make RenderViewHostDelegateViewHelper::CreateNewWindow() private.

chrome/browser/tab_contents/render_view_host_delegate_helper.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.
 
 #ifndef CHROME_BROWSER_TAB_CONTENTS_RENDER_VIEW_HOST_DELEGATE_HELPER_H_
 #define CHROME_BROWSER_TAB_CONTENTS_RENDER_VIEW_HOST_DELEGATE_HELPER_H_
 #pragma once
 
 #include <map>
+#include <string>
 
 #include "base/basictypes.h"
 #include "content/browser/webui/web_ui.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "content/public/common/window_container_type.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebPopupType.h"
-#include "ui/gfx/rect.h"
 #include "webkit/glue/webpreferences.h"
 #include "webkit/glue/window_open_disposition.h"
 
 class BackgroundContents;
 class Profile;
 class RenderViewHost;
 class RenderViewHostDelegate;
 class RenderWidgetHost;
 class RenderWidgetHostView;
 class SiteInstance;
 class TabContents;
 struct ViewHostMsg_CreateWindow_Params;
 
 namespace content {
 class BrowserContext;
 class RenderProcessHost;
 }
 
+namespace gfx {
+class Rect;
+}
+
 // Provides helper methods that provide common implementations of some
 // RenderViewHostDelegate::View methods.
 class RenderViewHostDelegateViewHelper : public content::NotificationObserver {
  public:
   RenderViewHostDelegateViewHelper();
   virtual ~RenderViewHostDelegateViewHelper();
 
-  // Creates a new renderer for window.open. This will either be a
-  // BackgroundContents (if the window_container_type ==
-  // WINDOW_CONTAINER_TYPE_BACKGROUND and permissions allow) or a TabContents.
-  // If a TabContents is created, it is returned. Otherwise NULL is returned.
-  TabContents* CreateNewWindow(
-      int route_id,
-      Profile* profile,
-      SiteInstance* site,
-      WebUI::TypeID webui_type,
-      RenderViewHostDelegate* opener,
-      WindowContainerType window_container_type,
-      const string16& frame_name);
-
   // Creates a new RenderWidgetHost and saves it for later retrieval by
   // GetCreatedWidget.
   RenderWidgetHostView* CreateNewWidget(int route_id,
                                         WebKit::WebPopupType popup_type,
                                         content::RenderProcessHost* process);
 
   RenderWidgetHostView* CreateNewFullscreenWidget(
       int route_id, content::RenderProcessHost* process);
 
   // Finds the new RenderWidgetHost and returns it. Note that this can only be
@@ skipping 24 matching lines @@
                                           const gfx::Rect& initial_pos);
   RenderWidgetHostView* ShowCreatedFullscreenWidget(TabContents* tab_contents,
                                                     int route_id);
 
  private:
   // content::NotificationObserver implementation
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
+  // Creates a new renderer for window.open. This will either be a
+  // BackgroundContents (if the window_container_type ==
+  // WINDOW_CONTAINER_TYPE_BACKGROUND and permissions allow) or a TabContents.
+  // If a TabContents is created, it is returned. Otherwise NULL is returned.
+  TabContents* CreateNewWindow(
+      int route_id,
+      Profile* profile,
+      SiteInstance* site,
+      WebUI::TypeID webui_type,
+      RenderViewHostDelegate* opener,
+      WindowContainerType window_container_type,
+      const string16& frame_name);
+
   BackgroundContents* MaybeCreateBackgroundContents(
       int route_id,
       Profile* profile,
       SiteInstance* site,
       const GURL& opener_url,
       const string16& frame_name);
 
   // Tracks created RenderViewHost objects that have not been shown yet.
   // They are identified by the route ID passed to CreateNewWindow.
   typedef std::map<int, RenderViewHost*> PendingContents;
@@ skipping 22 matching lines @@
                                      const std::string& value);
   static void ClearInspectorSettings(content::BrowserContext* browser_context);
 
  private:
   RenderViewHostDelegateHelper();
 
   DISALLOW_COPY_AND_ASSIGN(RenderViewHostDelegateHelper);
 };
 
 #endif  // CHROME_BROWSER_TAB_CONTENTS_RENDER_VIEW_HOST_DELEGATE_HELPER_H_