DevTools: devtools message plumbing between worker and page processes

chrome/app/chrome_main.cc

 // 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.
 
 #include "chrome/app/chrome_main.h"
 
 #include "app/app_paths.h"
 #include "base/at_exit.h"
 #include "base/command_line.h"
 #include "base/debug/debugger.h"
@@ skipping 16 matching lines @@
 #include "chrome/common/chrome_content_client.h"
 #include "chrome/common/chrome_content_plugin_client.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_paths_internal.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/common/logging_chrome.h"
 #include "chrome/common/profiling.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/renderer/chrome_content_renderer_client.h"
+#include "chrome/worker/chrome_content_worker_client.h"
 #include "content/browser/renderer_host/render_process_host.h"
 #include "content/common/content_client.h"
 #include "content/common/content_counters.h"
 #include "content/common/content_paths.h"
 #include "content/common/main_function_params.h"
 #include "content/common/sandbox_init_wrapper.h"
 #include "content/common/set_process_title.h"
 #include "ipc/ipc_switches.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/base/ui_base_paths.h"
@@ skipping 164 matching lines @@
 // Enable the heap profiler if the appropriate command-line switch is
 // present, bailing out of the app we can't.
 void EnableHeapProfiler(const CommandLine& command_line) {
 #if defined(OS_WIN)
   if (command_line.HasSwitch(switches::kMemoryProfiling))
     if (!LoadMemoryProfiler())
       exit(-1);
 #endif
 }
 
+void InitializeChromeContentWorkerClient() {
+  static chrome::ChromeContentWorkerClient chrome_content_worker_client;
+  content::GetContentClient()->set_worker(&chrome_content_worker_client);
+}
+
 void InitializeChromeContentRendererClient() {
 #if !defined(NACL_WIN64)  // We don't build the renderer code on win nacl64.
   static chrome::ChromeContentRendererClient chrome_content_renderer_client;
   content::GetContentClient()->set_renderer(&chrome_content_renderer_client);
 #endif
+  InitializeChromeContentWorkerClient();

[jam, 2011/05/24 20:21:31]

so far, by design we only have one process getter in the ContentClient interface
that's non NULL at a time (other than single process mode of course).

We supported (in theory only) creating WebWorkerProxy in the worker in case of
workers creating other workers.  But we never actually implemented it.  Once
dedicated workers run in the renderer, this case won't even matter.  So
basically, we should only have ContentWorkerClient in the worker process.

[yurys, 2011/05/25 17:28:29]

I deleted ContentWorkerClient completely since it's not needed if devtools code
moves to content/.

 }
 
 void InitializeChromeContentClient(const std::string& process_type) {
   if (process_type == switches::kPluginProcess) {
     static chrome::ChromeContentPluginClient chrome_content_plugin_client;
     content::GetContentClient()->set_plugin(&chrome_content_plugin_client);
   } else if (process_type == switches::kRendererProcess ||
              process_type == switches::kExtensionProcess) {
     InitializeChromeContentRendererClient();
+  } else if (process_type == switches::kWorkerProcess) {
+    InitializeChromeContentWorkerClient();
   }
 }
 
 void CommonSubprocessInit(const std::string& process_type) {
 #if defined(OS_WIN)
   // HACK: Let Windows know that we have started.  This is needed to suppress
   // the IDC_APPSTARTING cursor from being displayed for a prolonged period
   // while a subprocess is starting.
   PostThreadMessage(GetCurrentThreadId(), WM_NULL, 0, 0);
   MSG msg;
@@ skipping 571 matching lines @@
 
   if (SubprocessNeedsResourceBundle(process_type))
     ResourceBundle::CleanupSharedInstance();
 
   logging::CleanupChromeLogging();
 
   chrome_main::LowLevelShutdown();
 
   return exit_code;
 }