Fixes a race condition when a shared worker exits while one parent is loading

it.
Changed the shared worker startup code to assign a route ID at the time that we
initially lookup the worker, and pass that same route ID in when we later try
to create the worker, to gracefully handle this race condition.

BUG=29243

TEST=existing tests suffice (can't reproduce race condition in tests)

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

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

1 files changed, 3 insertions, 1 deletions

diff --git a/chrome/renderer/webworker_base.cc b/chrome/renderer/webworker_base.cc
index e78ef64..2bc10b5 100644
--- a/chrome/renderer/webworker_base.cc
+++ b/chrome/renderer/webworker_base.cc
@@ -54,7 +54,8 @@ void WebWorkerBase::CreateWorkerContext(const GURL& script_url,
                                         bool is_shared,
                                         const string16& name,
                                         const string16& user_agent,
-                                        const string16& source_code) {
+                                        const string16& source_code,
+                                        int pending_route_id) {
   DCHECK(route_id_ == MSG_ROUTING_NONE);
   ViewHostMsg_CreateWorker_Params params;
   params.url = script_url;
@@ -62,6 +63,7 @@ void WebWorkerBase::CreateWorkerContext(const GURL& script_url,
   params.name = name;
   params.document_id = document_id_;
   params.render_view_route_id = render_view_route_id_;
+  params.route_id = pending_route_id;
   IPC::Message* create_message = new ViewHostMsg_CreateWorker(
       params, &route_id_);
   child_thread_->Send(create_message);