DevTools: devtools message plumbing between worker and page processes

content/worker/webworkerclient_proxy.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 "content/worker/webworkerclient_proxy.h"
 
 #include "base/command_line.h"
 #include "base/message_loop.h"
 #include "content/common/content_switches.h"
 #include "content/common/file_system/file_system_dispatcher.h"
@@ skipping 25 matching lines @@
 using WebKit::WebWorkerClient;
 
 // How long to wait for worker to finish after it's been told to terminate.
 #define kMaxTimeForRunawayWorkerMs 3000
 
 WebWorkerClientProxy::WebWorkerClientProxy(int route_id,
                                            WebWorkerStubBase* stub)
     : route_id_(route_id),
       appcache_host_id_(0),
       stub_(stub),
-      ALLOW_THIS_IN_INITIALIZER_LIST(kill_process_factory_(this)) {
+      ALLOW_THIS_IN_INITIALIZER_LIST(kill_process_factory_(this)),
+      devtools_delegate_(0) {

[jam, 2011/05/25 17:50:12]

nit: chrome style is to use NULL for pointers

[yurys, 2011/05/26 09:11:49]

Done.

 }
 
 WebWorkerClientProxy::~WebWorkerClientProxy() {
 }
 
 void WebWorkerClientProxy::postMessageToWorkerObject(
     const WebString& message,
     const WebMessagePortChannelArray& channels) {
   std::vector<int> message_port_ids(channels.size());
   std::vector<int> routing_ids(channels.size());
@@ skipping 109 matching lines @@
 void WebWorkerClientProxy::openFileSystem(
     WebKit::WebFileSystem::Type type,
     long long size,
     bool create,
     WebKit::WebFileSystemCallbacks* callbacks) {
   ChildThread::current()->file_system_dispatcher()->OpenFileSystem(
       stub_->url().GetOrigin(), static_cast<fileapi::FileSystemType>(type),
       size, create, new WebFileSystemCallbackDispatcher(callbacks));
 }
 
+void WebWorkerClientProxy::dispatchDevToolsMessage(const WebString& message) {
+  if (devtools_delegate_)
+    devtools_delegate_->SendDevToolsMessage(message);
+}
+
+void WebWorkerClientProxy::set_devtools_delegate(DevToolsDelegate* delegate) {

[jam, 2011/05/25 17:50:12]

note: chrome style is to put simple getters/setters in the header

[yurys, 2011/05/26 09:11:49]

Done.

+  devtools_delegate_ = delegate;
+}
+
 bool WebWorkerClientProxy::Send(IPC::Message* message) {
   return WorkerThread::current()->Send(message);
 }
 
 void WebWorkerClientProxy::EnsureWorkerContextTerminates() {
   // Avoid a worker doing a while(1) from never exiting.
   if (CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kWebWorkerShareProcesses)) {
     // Can't kill the process since there could be workers from other
     // renderer process.
     NOTIMPLEMENTED();
     return;
   }
 
   // This shuts down the process cleanly from the perspective of the browser
   // process, and avoids the crashed worker infobar from appearing to the new
   // page. It's ok to post several of theese, because the first executed task
   // will exit the message loop and subsequent ones won't be executed.
   MessageLoop::current()->PostDelayedTask(FROM_HERE,
       kill_process_factory_.NewRunnableMethod(
           &WebWorkerClientProxy::workerContextDestroyed),
           kMaxTimeForRunawayWorkerMs);
 }