Get rid of a few more interfaces from RenderViewHostDelegate that aren't need...

chrome/test/automation/tab_proxy.h

 // Copyright (c) 2010 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_TEST_AUTOMATION_TAB_PROXY_H_
 #define CHROME_TEST_AUTOMATION_TAB_PROXY_H_
 #pragma once
 
 #include "build/build_config.h"  // NOLINT
 
 #if defined(OS_WIN)
 #include <wtypes.h>  // NOLINT
 #endif
 
 #include <string>
 #include <vector>
 
 #include "base/compiler_specific.h"
 #include "base/observer_list.h"
 #include "chrome/browser/download/save_package.h"
 #include "chrome/common/automation_constants.h"
 #include "chrome/common/page_type.h"
 #include "chrome/common/security_style.h"
 #include "chrome/test/automation/automation_handle_tracker.h"
 #include "chrome/test/automation/dom_element_proxy.h"
 #include "chrome/test/automation/javascript_execution_controller.h"
+#include "webkit/glue/window_open_disposition.h"

[Avi (use Gerrit), 2011/01/25 14:28:33]

This include is used in this file?

[jam, 2011/01/25 17:38:31]

yep, lower down.  Now that save_package.h doesn't include
chrome/browser/renderer_host/render_view_host_delegate.h, I had to add it.

 
 class GURL;
 class Value;
 namespace IPC {
 class Message;
 };
 
 enum FindInPageDirection { BACK = 0, FWD = 1 };
 enum FindInPageCase { IGNORE_CASE = 0, CASE_SENSITIVE = 1 };
 // Specifies the font size on a page which is requested by an automation
@@ skipping 394 matching lines @@
   // purposes.
   void LastObjectRemoved();
 
  private:
   base::Lock list_lock_;  // Protects the observers_list_.
   ObserverList<TabProxyDelegate> observers_list_;
   DISALLOW_COPY_AND_ASSIGN(TabProxy);
 };
 
 #endif  // CHROME_TEST_AUTOMATION_TAB_PROXY_H_