Cleanup: Remove forward declarations in various directories.

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

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

12 files changed, 13 insertions, 40 deletions

diff --git a/chrome/common/service_process_util.h b/chrome/common/service_process_util.h
index b5f631e..f9defa5 100644
--- a/chrome/common/service_process_util.h
+++ b/chrome/common/service_process_util.h
@@ -14,7 +14,6 @@
 #include "base/shared_memory.h"
 #include "ipc/ipc_channel_handle.h"
 
-class Task;
 class CommandLine;
 
 namespace base {
@@ -89,7 +88,6 @@ class ServiceProcessState {
   IPC::ChannelHandle GetServiceProcessChannel();
 
  private:
-
 #if !defined(OS_MACOSX)
   // Create the shared memory data for the service process.
   bool CreateSharedData();
diff --git a/chrome_frame/cfproxy.h b/chrome_frame/cfproxy.h
index e5169dc..2d305e8 100644
--- a/chrome_frame/cfproxy.h
+++ b/chrome_frame/cfproxy.h
@@ -7,9 +7,11 @@
 #pragma once
 
 #include <windows.h>
+
 #include <map>  // for proxy factory
 #include <vector>
 #include <string>
+
 #include "base/synchronization/lock.h"
 #include "base/time.h"         // for base::TimeDelta
 #include "base/file_path.h"
@@ -20,18 +22,9 @@
 class ChromeProxyDelegate;
 class ChromeProxyFactory;
 class GURL;
-struct AttachExternalTabParams;
-struct AutomationURLRequest;
-struct ContextMenuModel;
 struct ExternalTabSettings;
-struct MiniContextMenuParams;
-struct NavigationInfo;
 struct ProxyParams;
 
-namespace net {
-class URLRequestStatus;
-}  // namespace net
-
 // Some callers of synchronous messages wants a context to be passed back
 // in order to identify the call they made. Presumably one can make
 // multiple sync calls of same type (as async) and want to identify what
diff --git a/chrome_frame/chrome_active_document.h b/chrome_frame/chrome_active_document.h
index 96232c6..b9dc092 100644
--- a/chrome_frame/chrome_active_document.h
+++ b/chrome_frame/chrome_active_document.h
@@ -30,7 +30,6 @@
 #include "third_party/active_doc/in_place_menu.h"
 #include "third_party/active_doc/ole_document_impl.h"
 
-class Thread;
 class TabProxy;
 class ChromeActiveDocument;
 
@@ -344,7 +343,7 @@ END_EXEC_COMMAND_MAP()
   bool PreProcessContextMenu(HMENU menu);
   bool HandleContextMenuCommand(UINT cmd, const MiniContextMenuParams& params);
 
- // ChromeFramePlugin overrides.
+  // ChromeFramePlugin overrides.
   virtual void OnAutomationServerReady();
 
   // IEnumPrivacyRecords
diff --git a/chrome_frame/external_tab.h b/chrome_frame/external_tab.h
index 2ac46a5..2932581 100644
--- a/chrome_frame/external_tab.h
+++ b/chrome_frame/external_tab.h
@@ -22,22 +22,24 @@
 #include "content/public/common/page_zoom.h"
 #include "googleurl/src/gurl.h"
 
+struct AttachExternalTabParams;
+struct AutomationURLRequest;
+struct ContextMenuModel;
+struct MiniContextMenuParams;
+struct NavigationInfo;
 class Task;
-class CancelableTask;
 
 namespace base {
-class TimeDelta;
 class WaitableEvent;
 }
 
-namespace IPC {
-struct NavigationInfo;
-}
-
 namespace gfx {
 class Rect;
 }
 
+namespace net {
+class URLRequestStatus;
+}
 
 // This is the delegate/callback interface that has to be implemented
 // by the customers of ExternalTabProxy class.
diff --git a/chrome_frame/urlmon_url_request_private.h b/chrome_frame/urlmon_url_request_private.h
index b4075b5..ad4a691 100644
--- a/chrome_frame/urlmon_url_request_private.h
+++ b/chrome_frame/urlmon_url_request_private.h
@@ -7,6 +7,7 @@
 
 #include <atlbase.h>
 #include <atlcom.h>
+
 #include <string>
 
 #include "base/callback_old.h"
@@ -16,8 +17,6 @@
 #include "net/url_request/url_request_status.h"
 #include "testing/gtest/include/gtest/gtest_prod.h"
 
-class RequestData;
-
 class UrlmonUrlRequest
   : public CComObjectRootEx<CComMultiThreadModel>,
     public PluginUrlRequest,
diff --git a/content/browser/host_zoom_map.h b/content/browser/host_zoom_map.h
index 4579b88..d0325ab 100644
--- a/content/browser/host_zoom_map.h
+++ b/content/browser/host_zoom_map.h
@@ -21,8 +21,6 @@
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 
-class GURL;
-
 // HostZoomMap needs to be deleted on the UI thread because it listens
 // to notifications on there (and holds a NotificationRegistrar).
 class CONTENT_EXPORT HostZoomMap
diff --git a/content/browser/renderer_host/render_view_host_delegate.h b/content/browser/renderer_host/render_view_host_delegate.h
index 824cb19..674f331 100644
--- a/content/browser/renderer_host/render_view_host_delegate.h
+++ b/content/browser/renderer_host/render_view_host_delegate.h
@@ -24,8 +24,6 @@
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebPopupType.h"
 #include "webkit/glue/window_open_disposition.h"
 
-class BackgroundContents;
-class BookmarkNode;
 class GURL;
 class RenderViewHost;
 class SkBitmap;
diff --git a/content/renderer/media/audio_message_filter.h b/content/renderer/media/audio_message_filter.h
index 5888a86..54cc3b6 100644
--- a/content/renderer/media/audio_message_filter.h
+++ b/content/renderer/media/audio_message_filter.h
@@ -20,10 +20,6 @@
 #include "ipc/ipc_channel_proxy.h"
 #include "media/audio/audio_buffers_state.h"
 
-namespace base {
-class Time;
-}
-
 class CONTENT_EXPORT AudioMessageFilter
     : public IPC::ChannelProxy::MessageFilter {
  public:
diff --git a/content/renderer/render_thread_impl.h b/content/renderer/render_thread_impl.h
index 615161a..b62878d 100644
--- a/content/renderer/render_thread_impl.h
+++ b/content/renderer/render_thread_impl.h
@@ -31,7 +31,6 @@ class DevToolsAgentFilter;
 struct DOMStorageMsg_Event_Params;
 class GpuChannelHost;
 class IndexedDBDispatcher;
-class RendererHistogram;
 class RendererWebKitPlatformSupportImpl;
 class SkBitmap;
 class VideoCaptureImplManager;
diff --git a/net/http/http_stream_factory_impl_job.h b/net/http/http_stream_factory_impl_job.h
index 66aee43..b14897c 100644
--- a/net/http/http_stream_factory_impl_job.h
+++ b/net/http/http_stream_factory_impl_job.h
@@ -24,7 +24,6 @@ class ClientSocketHandle;
 class HttpAuthController;
 class HttpNetworkSession;
 class HttpStream;
-class TransportSocketParams;
 
 // An HttpStreamRequestImpl exists for each stream which is in progress of being
 // created for the StreamFactory.
diff --git a/remoting/protocol/connection_to_client.h b/remoting/protocol/connection_to_client.h
index 418e071..a721a48 100644
--- a/remoting/protocol/connection_to_client.h
+++ b/remoting/protocol/connection_to_client.h
@@ -14,10 +14,6 @@
 #include "remoting/protocol/session.h"
 #include "remoting/protocol/video_writer.h"
 
-namespace base {
-class MessageLoopProxy;
-}  // namespace base
-
 namespace remoting {
 namespace protocol {
 
@@ -52,7 +48,7 @@ class ConnectionToClient : public base::NonThreadSafe {
 
   // Constructs a ConnectionToClient object for the |session|. Takes
   // ownership of |session|.
-  ConnectionToClient(Session* session);
+  explicit ConnectionToClient(Session* session);
   virtual ~ConnectionToClient();
 
   // Set |event_handler| for connection events. |event_handler| is
diff --git a/remoting/protocol/jingle_datagram_connector.h b/remoting/protocol/jingle_datagram_connector.h
index bbd7e944..621bbee 100644
--- a/remoting/protocol/jingle_datagram_connector.h
+++ b/remoting/protocol/jingle_datagram_connector.h
@@ -11,10 +11,6 @@
 #include "remoting/protocol/jingle_channel_connector.h"
 #include "remoting/protocol/session.h"
 
-namespace jingle_glue {
-class TransportChannelSocketAdapter;
-}  // namespace jingle_glue
-
 namespace remoting {
 namespace protocol {