Cleanup: Remove some cruft from RenderViewHostDelegate.

content/browser/renderer_host/render_view_host_delegate.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 CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_DELEGATE_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_DELEGATE_H_
 #pragma once
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/i18n/rtl.h"
-#include "base/memory/ref_counted.h"
 #include "base/process_util.h"
 #include "base/string16.h"
-#include "base/values.h"
 #include "content/common/content_export.h"
 #include "content/public/common/view_type.h"
-#include "content/public/common/window_container_type.h"
 #include "ipc/ipc_channel.h"
 #include "net/base/load_states.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebDragOperation.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebPopupType.h"
 #include "webkit/glue/window_open_disposition.h"
 
 class GURL;
 class RenderViewHost;
 class SkBitmap;
 class TabContents;
 class WebKeyboardEvent;
 struct ContextMenuParams;
 struct NativeWebKeyboardEvent;
 struct ViewHostMsg_CreateWindow_Params;
 struct ViewHostMsg_FrameNavigate_Params;
 struct WebDropData;
 struct WebMenuItem;
 struct WebPreferences;
 
+namespace base {
+class ListValue;
+}
+
 namespace content {
 class BrowserContext;
 struct FileChooserParams;
 struct RendererPreferences;
 }
 
 namespace gfx {
 class Point;
 class Rect;
 class Size;
@@ skipping 22 matching lines @@
     // shown yet. That should happen in response to ShowCreatedWindow.
     // |params.window_container_type| describes the type of RenderViewHost
     // container that is requested -- in particular, the window.open call may
     // have specified 'background' and 'persistent' in the feature string.
     //
     // The passed |params.frame_name| parameter is the name parameter that was
     // passed to window.open(), and will be empty if none was passed.
     //
     // Note: this is not called "CreateWindow" because that will clash with
     // the Windows function which is actually a #define.
-    //
-    // NOTE: this takes ownership of @modal_dialog_event

[Lei Zhang, 2011/12/01 18:06:57]

Apparently this comment has been invalid for 2 years. :\

[Avi (use Gerrit), 2011/12/01 20:10:14]

haha

     virtual void CreateNewWindow(
         int route_id,
         const ViewHostMsg_CreateWindow_Params& params) = 0;
 
     // The page is trying to open a new widget (e.g. a select popup). The
     // widget should be created associated with the given route, but it should
     // not be shown yet. That should happen in response to ShowCreatedWidget.
     // |popup_type| indicates if the widget is a popup and what kind of popup it
     // is (select, autofill...).
     virtual void CreateNewWidget(int route_id,
@@ skipping 287 matching lines @@
   virtual void RequestToLockMouse() {}
 
   // Notification that the view has lost the mouse lock.
   virtual void LostMouseLock() {}
 
  protected:
   virtual ~RenderViewHostDelegate() {}
 };
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_DELEGATE_H_