DevTools: Migrate to new WebView initialization (pass devtools client directly).

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

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

10 files changed, 27 insertions, 24 deletions

diff --git a/DEPS b/DEPS
index b378cca..beefa0b 100644
--- a/DEPS
+++ b/DEPS
@@ -1,7 +1,7 @@
 vars = {
   "webkit_trunk":
   "http://svn.webkit.org/repository/webkit/trunk",
-  "webkit_revision": "61756",
+  "webkit_revision": "61766",
   "ffmpeg_revision": "49485",
   "skia_revision": "582",
   "chromium_git": "http://src.chromium.org/git",
diff --git a/chrome/renderer/print_web_view_helper.cc b/chrome/renderer/print_web_view_helper.cc
index 9bab6a3..726ce2d 100644..100755
--- a/chrome/renderer/print_web_view_helper.cc
+++ b/chrome/renderer/print_web_view_helper.cc
@@ -246,7 +246,7 @@ bool PrintWebViewHelper::CopyAndPrint(const ViewMsg_PrintPages_Params& params,
   prefs.javascript_enabled = false;
   prefs.java_enabled = false;
 
-  print_web_view_ = WebView::create(this);
+  print_web_view_ = WebView::create(this, 0);
   prefs.Apply(print_web_view_);
   print_web_view_->initializeMainFrame(this);
 
diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc
index 6b95e29..312161b 100644
--- a/chrome/renderer/render_view.cc
+++ b/chrome/renderer/render_view.cc
@@ -561,14 +561,12 @@ void RenderView::Init(gfx::NativeViewId parent_hwnd,
 
   devtools_agent_.reset(new DevToolsAgent(routing_id, this));
 
-  webwidget_ = WebView::create(this);
+  webwidget_ = WebView::create(this, devtools_agent_.get());
   Singleton<ViewMap>::get()->insert(std::make_pair(webview(), this));
   webkit_preferences_.Apply(webview());
   webview()->initializeMainFrame(this);
   if (!frame_name.empty())
     webview()->mainFrame()->setName(frame_name);
-  webview()->setDevToolsAgent(
-      WebDevToolsAgent::create(webview(), devtools_agent_.get()));
 
   OnSetRendererPrefs(renderer_prefs);
 
diff --git a/webkit/tools/test_shell/test_shell.cc b/webkit/tools/test_shell/test_shell.cc
index 90c8abc..62bacb2 100644
--- a/webkit/tools/test_shell/test_shell.cc
+++ b/webkit/tools/test_shell/test_shell.cc
@@ -583,11 +583,6 @@ WebView* TestShell::CreateWebView() {
   return new_win->webView();
 }
 
-void TestShell::InitializeDevToolsAgent(WebView* webView) {
-  DCHECK(!dev_tools_agent_.get());
-  dev_tools_agent_.reset(new TestShellDevToolsAgent(webView));
-}
-
 void TestShell::ShowDevTools() {
   if (!devtools_shell_) {
     FilePath dir_exe;
diff --git a/webkit/tools/test_shell/test_shell.h b/webkit/tools/test_shell/test_shell.h
index 21d8856..f415d54 100644..100755
--- a/webkit/tools/test_shell/test_shell.h
+++ b/webkit/tools/test_shell/test_shell.h
@@ -92,7 +92,7 @@ public:
     static void CleanupLogging();
 
     // Initialization and clean up of a static member variable.
-    static void InitializeTestShell(bool layout_test_mode, 
+    static void InitializeTestShell(bool layout_test_mode,
                                     bool allow_external_pages);
     static void ShutdownTestShell();
 
@@ -329,7 +329,6 @@ public:
 protected:
     void CreateDevToolsClient(TestShellDevToolsAgent* agent);
     bool Initialize(const GURL& starting_url);
-    void InitializeDevToolsAgent(WebKit::WebView* webView);
     bool IsSVGTestURL(const GURL& url);
     void SizeToSVG();
     void SizeToDefault();
@@ -386,9 +385,9 @@ private:
 
     // True when the app is being run using the --layout-tests switch.
     static bool layout_test_mode_;
-  
+
     // True when we wish to allow test shell to load external pages like
-    // www.google.com even when in --layout-test mode (used for QA to 
+    // www.google.com even when in --layout-test mode (used for QA to
     // produce images of the rendered page)
     static bool allow_external_pages_;
 
diff --git a/webkit/tools/test_shell/test_shell_devtools_agent.cc b/webkit/tools/test_shell/test_shell_devtools_agent.cc
index 704827d..20e8c16 100644..100755
--- a/webkit/tools/test_shell/test_shell_devtools_agent.cc
+++ b/webkit/tools/test_shell/test_shell_devtools_agent.cc
@@ -57,17 +57,18 @@ void TestShellDevToolsAgent::DispatchMessageLoop() {
 #pragma warning(disable : 4355)
 #endif // defined(OS_WIN)
 
-TestShellDevToolsAgent::TestShellDevToolsAgent(WebView* web_view)
+TestShellDevToolsAgent::TestShellDevToolsAgent()
     : call_method_factory_(this),
-      dev_tools_client_(NULL),
-      web_view_(web_view) {
+      dev_tools_client_(NULL) {
   static int dev_tools_agent_counter;
   routing_id_ = ++dev_tools_agent_counter;
   if (routing_id_ == 1)
     WebDevToolsAgent::setMessageLoopDispatchHandler(
         &TestShellDevToolsAgent::DispatchMessageLoop);
-  web_dev_tools_agent_ = WebDevToolsAgent::create(web_view_, this);
-  web_view_->setDevToolsAgent(web_dev_tools_agent_);
+}
+
+void TestShellDevToolsAgent::SetWebView(WebKit::WebView* web_view) {
+  web_view_ = web_view;
 }
 
 void TestShellDevToolsAgent::sendMessageToFrontend(
diff --git a/webkit/tools/test_shell/test_shell_devtools_agent.h b/webkit/tools/test_shell/test_shell_devtools_agent.h
index 8020876..baf7174 100644..100755
--- a/webkit/tools/test_shell/test_shell_devtools_agent.h
+++ b/webkit/tools/test_shell/test_shell_devtools_agent.h
@@ -23,9 +23,11 @@ class TestShellDevToolsClient;
 class TestShellDevToolsAgent : public WebKit::WebDevToolsAgentClient {
 
  public:
-  TestShellDevToolsAgent(WebKit::WebView* web_view);
+  TestShellDevToolsAgent();
   virtual ~TestShellDevToolsAgent() {}
 
+  void SetWebView(WebKit::WebView* web_view);
+
   // WebDevToolsAgentClient implementation.
   virtual void sendMessageToFrontend(
       const WebKit::WebDevToolsMessageData& data);
diff --git a/webkit/tools/test_shell/test_shell_win.cc b/webkit/tools/test_shell/test_shell_win.cc
index 841d724..079a830 100644..100755
--- a/webkit/tools/test_shell/test_shell_win.cc
+++ b/webkit/tools/test_shell/test_shell_win.cc
@@ -34,6 +34,7 @@
 #include "webkit/glue/plugins/plugin_list.h"
 #include "webkit/tools/test_shell/resource.h"
 #include "webkit/tools/test_shell/test_navigation_controller.h"
+#include "webkit/tools/test_shell/test_shell_devtools_agent.h"
 #include "webkit/tools/test_shell/test_shell_switches.h"
 #include "webkit/tools/test_shell/test_webview_delegate.h"
 
@@ -392,13 +393,17 @@ bool TestShell::Initialize(const GURL& starting_url) {
       win_util::SetWindowProc(m_editWnd, TestShell::EditWndProc);
   win_util::SetWindowUserData(m_editWnd, this);
 
+  dev_tools_agent_.reset(new TestShellDevToolsAgent());
+
   // create webview
   m_webViewHost.reset(
-      WebViewHost::Create(m_mainWnd, delegate_.get(), *TestShell::web_prefs_));
+      WebViewHost::Create(m_mainWnd,
+                          delegate_.get(),
+                          dev_tools_agent_.get(),
+                          *TestShell::web_prefs_));
+  dev_tools_agent_->SetWebView(m_webViewHost->webview());
   delegate_->RegisterDragDrop();
 
-  InitializeDevToolsAgent(webView());
-
   // Load our initial content.
   if (starting_url.is_valid())
     LoadURL(starting_url);
diff --git a/webkit/tools/test_shell/webview_host.h b/webkit/tools/test_shell/webview_host.h
index 2f09dec..2fc545f 100644..100755
--- a/webkit/tools/test_shell/webview_host.h
+++ b/webkit/tools/test_shell/webview_host.h
@@ -20,6 +20,7 @@ struct WebPreferences;
 class TestWebViewDelegate;
 
 namespace WebKit {
+class WebDevToolsAgentClient;
 class WebView;
 }
 
@@ -31,6 +32,7 @@ class WebViewHost : public WebWidgetHost {
   // MoveWindow (or equivalent) function.
   static WebViewHost* Create(gfx::NativeView parent_view,
                              TestWebViewDelegate* delegate,
+                             WebKit::WebDevToolsAgentClient* devtoolsClient,
                              const WebPreferences& prefs);
 
   WebKit::WebView* webview() const;
diff --git a/webkit/tools/test_shell/webview_host_win.cc b/webkit/tools/test_shell/webview_host_win.cc
index 3c2e9a8..17c8fa1 100644..100755
--- a/webkit/tools/test_shell/webview_host_win.cc
+++ b/webkit/tools/test_shell/webview_host_win.cc
@@ -19,6 +19,7 @@ static const wchar_t kWindowClassName[] = L"WebViewHost";
 /*static*/
 WebViewHost* WebViewHost::Create(HWND parent_view,
                                  TestWebViewDelegate* delegate,
+                                 WebDevToolsAgentClient* devToolsClient,
                                  const WebPreferences& prefs) {
   WebViewHost* host = new WebViewHost();
 
@@ -41,7 +42,7 @@ WebViewHost* WebViewHost::Create(HWND parent_view,
                              GetModuleHandle(NULL), NULL);
   win_util::SetWindowUserData(host->view_, host);
 
-  host->webwidget_ = WebView::create(delegate);
+  host->webwidget_ = WebView::create(delegate, devToolsClient);
   prefs.Apply(host->webview());
   host->webview()->initializeMainFrame(delegate);