Cleanup: Remove unneeded forward declarations in webkit/.

BUG=none
TEST=none
TBR=OWNERS

Review URL: http://codereview.chromium.org/8539001

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

19 files changed, 26 insertions, 54 deletions

diff --git a/webkit/chromeos/fileapi/file_access_permissions.h b/webkit/chromeos/fileapi/file_access_permissions.h
index e946634..cc6ddbb 100644
--- a/webkit/chromeos/fileapi/file_access_permissions.h
+++ b/webkit/chromeos/fileapi/file_access_permissions.h
@@ -13,8 +13,6 @@
 #include "base/file_path.h"
 #include "base/synchronization/lock.h"
 
-class Extension;
-
 namespace chromeos {
 
 class FileAccessPermissions {
@@ -39,6 +37,6 @@ class FileAccessPermissions {
   PathAccessMap path_map_;
 };
 
-};
+}  // namespace chromeos
 
 #endif  // WEBKIT_CHROMEOS_FILEAPI_FILE_ACCESS_PERMISSIONS_H_
diff --git a/webkit/fileapi/file_system_context.h b/webkit/fileapi/file_system_context.h
index fa6b30c..de0aab0 100644
--- a/webkit/fileapi/file_system_context.h
+++ b/webkit/fileapi/file_system_context.h
@@ -18,7 +18,6 @@ class MessageLoopProxy;
 }
 
 namespace quota {
-class QuotaClient;
 class QuotaManagerProxy;
 }
 
@@ -27,7 +26,6 @@ namespace fileapi {
 class FileSystemContext;
 class FileSystemPathManager;
 class FileSystemQuotaUtil;
-class FileSystemUsageTracker;
 class SandboxMountPointProvider;
 
 struct DefaultContextDeleter;
diff --git a/webkit/fileapi/file_system_file_util_proxy.h b/webkit/fileapi/file_system_file_util_proxy.h
index d8e52ad..16eee74 100644
--- a/webkit/fileapi/file_system_file_util_proxy.h
+++ b/webkit/fileapi/file_system_file_util_proxy.h
@@ -17,13 +17,10 @@
 
 namespace base {
 class MessageLoopProxy;
-class Time;
 }
 
 namespace fileapi {
 
-class FileSystemOperationContext;
-
 using base::MessageLoopProxy;
 using base::PlatformFile;
 using base::PlatformFileError;
diff --git a/webkit/fileapi/file_system_operation.h b/webkit/fileapi/file_system_operation.h
index a85f3f1..ea38357 100644
--- a/webkit/fileapi/file_system_operation.h
+++ b/webkit/fileapi/file_system_operation.h
@@ -5,6 +5,7 @@
 #ifndef WEBKIT_FILEAPI_FILE_SYSTEM_OPERATION_H_
 #define WEBKIT_FILEAPI_FILE_SYSTEM_OPERATION_H_
 
+#include <string>
 #include <vector>
 
 #include "base/file_path.h"
@@ -38,7 +39,6 @@ class FileSystemCallbackDispatcher;
 class FileSystemContext;
 class FileWriterDelegate;
 class FileSystemOperationTest;
-class FileSystemQuotaUtil;
 
 // This class is designed to serve one-time file system operation per instance.
 // Only one method(CreateFile, CreateDirectory, Copy, Move, DirectoryExists,
diff --git a/webkit/fileapi/file_system_test_helper.h b/webkit/fileapi/file_system_test_helper.h
index 6562318..0db1a07 100644
--- a/webkit/fileapi/file_system_test_helper.h
+++ b/webkit/fileapi/file_system_test_helper.h
@@ -29,7 +29,6 @@ class FileSystemContext;
 class FileSystemFileUtil;
 class FileSystemOperation;
 class FileSystemOperationContext;
-class FileSystemPathManager;
 
 // Filesystem test helper class that encapsulates test environment for
 // a given {origin, type} pair.
diff --git a/webkit/fileapi/local_file_util.h b/webkit/fileapi/local_file_util.h
index cd564a4..5e36cda 100644
--- a/webkit/fileapi/local_file_util.h
+++ b/webkit/fileapi/local_file_util.h
@@ -19,7 +19,6 @@
 
 namespace base {
 struct PlatformFileInfo;
-class MessageLoopProxy;
 class Time;
 }
 
diff --git a/webkit/fileapi/webfilewriter_base.h b/webkit/fileapi/webfilewriter_base.h
index 3601256..ec02a1c 100644
--- a/webkit/fileapi/webfilewriter_base.h
+++ b/webkit/fileapi/webfilewriter_base.h
@@ -11,7 +11,6 @@
 
 namespace WebKit {
 class WebFileWriterClient;
-class WebString;
 class WebURL;
 }
 
diff --git a/webkit/glue/media/simple_data_source.h b/webkit/glue/media/simple_data_source.h
index fd3a282..cea3c35 100644
--- a/webkit/glue/media/simple_data_source.h
+++ b/webkit/glue/media/simple_data_source.h
@@ -25,7 +25,6 @@
 #include "webkit/glue/media/web_data_source.h"
 
 class MessageLoop;
-class WebMediaPlayerDelegateImpl;
 
 namespace media {
 class MediaLog;
diff --git a/webkit/glue/password_form_dom_manager.h b/webkit/glue/password_form_dom_manager.h
index 63a6f45..7f201ff 100644
--- a/webkit/glue/password_form_dom_manager.h
+++ b/webkit/glue/password_form_dom_manager.h
@@ -1,10 +1,12 @@
-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+// 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 WEBKIT_GLUE_PASSWORD_FORM_DOM_MANAGER_H_
 #define WEBKIT_GLUE_PASSWORD_FORM_DOM_MANAGER_H_
 
+#include <map>
+
 #include "webkit/glue/form_data.h"
 #include "webkit/glue/password_form.h"
 
@@ -12,8 +14,6 @@ namespace WebKit {
 class WebForm;
 }
 
-class GURL;
-
 namespace webkit_glue {
 
 // Structure used for autofilling password forms.
diff --git a/webkit/glue/scoped_clipboard_writer_glue.h b/webkit/glue/scoped_clipboard_writer_glue.h
index 13b96a3..9b331fc 100644
--- a/webkit/glue/scoped_clipboard_writer_glue.h
+++ b/webkit/glue/scoped_clipboard_writer_glue.h
@@ -1,21 +1,19 @@
-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// 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 SCOPED_CLIPBOARD_WRITER_GLUE_H_
-#define SCOPED_CLIPBOARD_WRITER_GLUE_H_
+#ifndef WEBKIT_GLUE_SCOPED_CLIPBOARD_WRITER_GLUE_H_
+#define WEBKIT_GLUE_SCOPED_CLIPBOARD_WRITER_GLUE_H_
 
 #include "ui/base/clipboard/scoped_clipboard_writer.h"
 
-class SkBitmap;
-
 namespace base {
 class SharedMemory;
 }
 
 class ScopedClipboardWriterGlue : public ui::ScopedClipboardWriter {
  public:
-   ScopedClipboardWriterGlue(ui::Clipboard* clipboard)
+  explicit ScopedClipboardWriterGlue(ui::Clipboard* clipboard)
       : ui::ScopedClipboardWriter(clipboard),
         shared_buf_(NULL) {
   }
@@ -29,4 +27,4 @@ class ScopedClipboardWriterGlue : public ui::ScopedClipboardWriter {
   DISALLOW_COPY_AND_ASSIGN(ScopedClipboardWriterGlue);
 };
 
-#endif  // SCOPED_CLIPBOARD_WRITER_GLUE_H_
+#endif  // WEBKIT_GLUE_SCOPED_CLIPBOARD_WRITER_GLUE_H_
diff --git a/webkit/glue/webkit_glue.h b/webkit/glue/webkit_glue.h
index 5c8efba..fe1799c 100644
--- a/webkit/glue/webkit_glue.h
+++ b/webkit/glue/webkit_glue.h
@@ -35,7 +35,6 @@ class PlatformCanvas;
 namespace WebKit {
 class WebFrame;
 class WebString;
-class WebView;
 }
 
 namespace webkit {
@@ -216,6 +215,6 @@ bool IsSingleProcess();
 // ---- END FUNCTIONS IMPLEMENTED BY EMBEDDER ---------------------------------
 
 
-} // namespace webkit_glue
+}  // namespace webkit_glue
 
 #endif  // WEBKIT_GLUE_WEBKIT_GLUE_H_
diff --git a/webkit/glue/webmediaplayer_impl.h b/webkit/glue/webmediaplayer_impl.h
index 06a31fe..2ca0b15 100644
--- a/webkit/glue/webmediaplayer_impl.h
+++ b/webkit/glue/webmediaplayer_impl.h
@@ -59,8 +59,6 @@
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebMediaPlayer.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebMediaPlayerClient.h"
 
-class GURL;
-
 namespace WebKit {
 class WebFrame;
 }
@@ -71,7 +69,6 @@ class MediaLog;
 
 namespace webkit_glue {
 
-class MediaResourceLoaderBridgeFactory;
 class MediaStreamClient;
 class WebMediaPlayerDelegate;
 class WebMediaPlayerProxy;
diff --git a/webkit/quota/quota_database.h b/webkit/quota/quota_database.h
index 7a816f6..1fed8fa 100644
--- a/webkit/quota/quota_database.h
+++ b/webkit/quota/quota_database.h
@@ -21,7 +21,6 @@
 namespace sql {
 class Connection;
 class MetaTable;
-class Statement;
 }
 
 class GURL;
diff --git a/webkit/quota/quota_temporary_storage_evictor.h b/webkit/quota/quota_temporary_storage_evictor.h
index a26924a..eebd700 100644
--- a/webkit/quota/quota_temporary_storage_evictor.h
+++ b/webkit/quota/quota_temporary_storage_evictor.h
@@ -16,10 +16,6 @@
 
 class GURL;
 
-namespace base {
-class MessageLoopProxy;
-}
-
 namespace quota {
 
 class QuotaEvictionHandler;
diff --git a/webkit/support/webkit_support.h b/webkit/support/webkit_support.h
index 650206b..50462f2 100644
--- a/webkit/support/webkit_support.h
+++ b/webkit/support/webkit_support.h
@@ -2,11 +2,10 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef WEBKIT_SUPPORT_WEBIT_SUPPORT_H_
-#define WEBKIT_SUPPORT_WEBIT_SUPPORT_H_
+#ifndef WEBKIT_SUPPORT_WEBKIT_SUPPORT_H_
+#define WEBKIT_SUPPORT_WEBKIT_SUPPORT_H_
 
 #include <string>
-#include <vector>
 
 #include "base/basictypes.h"
 #include "base/string16.h"
@@ -14,11 +13,9 @@
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFileSystem.h"
 #include "ui/base/keycodes/keyboard_codes.h"
 
-class WebURLLoaderMockFactory;
 namespace WebKit {
 class WebApplicationCacheHost;
 class WebApplicationCacheHostClient;
-class WebCString;
 class WebFileSystemCallbacks;
 class WebFrame;
 class WebKitPlatformSupport;
@@ -226,4 +223,4 @@ void EnableWebCoreLogChannels(const std::string& channels);
 
 }  // namespace webkit_support
 
-#endif  // WEBKIT_SUPPORT_WEBIT_CLIENT_IMPL_H_
+#endif  // WEBKIT_SUPPORT_WEBKIT_SUPPORT_H_
diff --git a/webkit/tools/test_shell/simple_resource_loader_bridge.h b/webkit/tools/test_shell/simple_resource_loader_bridge.h
index ba00544..71f9ce5 100644
--- a/webkit/tools/test_shell/simple_resource_loader_bridge.h
+++ b/webkit/tools/test_shell/simple_resource_loader_bridge.h
@@ -11,7 +11,6 @@
 
 class FilePath;
 class GURL;
-class TestShellRequestContext;
 
 class SimpleResourceLoaderBridge {
  public:
diff --git a/webkit/tools/test_shell/test_shell_devtools_agent.h b/webkit/tools/test_shell/test_shell_devtools_agent.h
index 9258442..d8fbea1 100644
--- a/webkit/tools/test_shell/test_shell_devtools_agent.h
+++ b/webkit/tools/test_shell/test_shell_devtools_agent.h
@@ -2,8 +2,10 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef TEST_SHELL_DEVTOOLS_AGENT_H_
-#define TEST_SHELL_DEVTOOLS_AGENT_H_
+#ifndef WEBKIT_TOOLS_TEST_SHELL_TEST_SHELL_DEVTOOLS_AGENT_H_
+#define WEBKIT_TOOLS_TEST_SHELL_TEST_SHELL_DEVTOOLS_AGENT_H_
+
+#include <string>
 
 #include "base/memory/weak_ptr.h"
 #include "base/task.h"
@@ -14,9 +16,8 @@ namespace WebKit {
 
 class WebDevToolsAgent;
 class WebView;
-struct WebDevToolsMessageData;
 
-} // namespace WebKit
+}  // namespace WebKit
 
 class TestShellDevToolsCallArgs;
 class TestShellDevToolsClient;
@@ -61,4 +62,4 @@ class TestShellDevToolsAgent : public WebKit::WebDevToolsAgentClient {
   DISALLOW_COPY_AND_ASSIGN(TestShellDevToolsAgent);
 };
 
-#endif  // TEST_SHELL_DEVTOOLS_AGENT_H_
+#endif  // WEBKIT_TOOLS_TEST_SHELL_TEST_SHELL_DEVTOOLS_AGENT_H_
diff --git a/webkit/tools/test_shell/test_shell_devtools_client.h b/webkit/tools/test_shell/test_shell_devtools_client.h
index b9ef290..a408828 100644
--- a/webkit/tools/test_shell/test_shell_devtools_client.h
+++ b/webkit/tools/test_shell/test_shell_devtools_client.h
@@ -2,8 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef TEST_SHELL_DEVTOOLS_CLIENT_H_
-#define TEST_SHELL_DEVTOOLS_CLIENT_H_
+#ifndef WEBKIT_TOOLS_TEST_SHELL_TEST_SHELL_DEVTOOLS_CLIENT_H_
+#define WEBKIT_TOOLS_TEST_SHELL_TEST_SHELL_DEVTOOLS_CLIENT_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
@@ -14,15 +14,13 @@
 namespace WebKit {
 
 class WebDevToolsFrontend;
-struct WebDevToolsMessageData;
 
-} // namespace WebKit
+}  // namespace WebKit
 
 class TestShellDevToolsCallArgs;
 class TestShellDevToolsAgent;
 
 class TestShellDevToolsClient: public WebKit::WebDevToolsFrontendClient {
-
  public:
   TestShellDevToolsClient(TestShellDevToolsAgent* agent,
                           WebKit::WebView* web_view);
@@ -53,4 +51,4 @@ class TestShellDevToolsClient: public WebKit::WebDevToolsFrontendClient {
   DISALLOW_COPY_AND_ASSIGN(TestShellDevToolsClient);
 };
 
-#endif  // TEST_SHELL_DEVTOOLS_CLIENT_H_
+#endif  // WEBKIT_TOOLS_TEST_SHELL_TEST_SHELL_DEVTOOLS_CLIENT_H_
diff --git a/webkit/tools/test_shell/test_webview_delegate.h b/webkit/tools/test_shell/test_webview_delegate.h
index afad34c..67036bd 100644
--- a/webkit/tools/test_shell/test_webview_delegate.h
+++ b/webkit/tools/test_shell/test_webview_delegate.h
@@ -12,6 +12,7 @@
 #include <map>
 #include <set>
 #include <string>
+#include <vector>
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
@@ -44,7 +45,6 @@
 
 struct WebPreferences;
 class GURL;
-class TestGeolocationService;
 class TestShell;
 class WebWidgetHost;
 
@@ -250,7 +250,7 @@ class TestWebViewDelegate : public WebKit::WebViewClient,
   virtual void DidStopLoadingForPlugin() OVERRIDE {}
   virtual WebKit::WebCookieJar* GetCookieJar() OVERRIDE;
 
-  TestWebViewDelegate(TestShell* shell);
+  explicit TestWebViewDelegate(TestShell* shell);
   virtual ~TestWebViewDelegate();
   void Reset();
 
@@ -329,7 +329,6 @@ class TestWebViewDelegate : public WebKit::WebViewClient,
   }
 
  private:
-
   // Called the title of the page changes.
   // Can be used to update the title of the window.
   void SetPageTitle(const string16& title);