Fix devtools unable to start a shared workers.

content/browser/webui/url_data_manager_backend.cc

 // Copyright (c) 2012 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/browser/webui/url_data_manager_backend.h"
 
 #include <set>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 679 matching lines @@
   std::string origin = GetOriginHeaderValue(request);
   if (!origin.empty()) {
     std::string header =
         source->source()->GetAccessControlAllowOriginForOrigin(origin);
     DCHECK(header.empty() || header == origin || header == "*" ||
            header == "null");
     job->set_access_control_allow_origin(header);
   }
 
   // Look up additional request info to pass down.
+  int child_id = -1;
   ResourceRequestInfo::WebContentsGetter wc_getter;
   const ResourceRequestInfo* info = ResourceRequestInfo::ForRequest(request);
-  if (info)
+  if (info) {
+    child_id = info->GetChildID();
     wc_getter = info->GetWebContentsGetterForRequest();
+  }
 
   // Forward along the request to the data source.
   base::MessageLoop* target_message_loop =
       source->source()->MessageLoopForRequestPath(path);
   if (!target_message_loop) {
     job->MimeTypeAvailable(source->source()->GetMimeType(path));
     // Eliminate potentially dangling pointer to avoid future use.
     job = nullptr;
 
     // The DataSource is agnostic to which thread StartDataRequest is called
     // on for this path.  Call directly into it from this thread, the IO
     // thread.
     source->source()->StartDataRequest(
         path, wc_getter,
         base::Bind(&URLDataSourceImpl::SendResponse, source, request_id));
   } else {
     // URLRequestChromeJob should receive mime type before data. This
     // is guaranteed because request for mime type is placed in the
     // message loop before request for data. And correspondingly their
     // replies are put on the IO thread in the same order.
     target_message_loop->task_runner()->PostTask(
         FROM_HERE, base::Bind(&GetMimeTypeOnUI, base::RetainedRef(source), path,
                               job->AsWeakPtr()));
 
     // The DataSource wants StartDataRequest to be called on a specific thread,
     // usually the UI thread, for this path.
     target_message_loop->task_runner()->PostTask(
         FROM_HERE, base::Bind(&URLDataManagerBackend::CallStartRequest,
-                              base::RetainedRef(source), path,
+                              base::RetainedRef(source), path, child_id,
                               wc_getter, request_id));
   }
   return true;
 }
 
 URLDataSourceImpl* URLDataManagerBackend::GetDataSourceFromURL(
     const GURL& url) {
   // The input usually looks like: chrome://source_name/extra_bits?foo
   // so do a lookup using the host of the URL.
   DataSourceMap::iterator i = data_sources_.find(url.host());
   if (i != data_sources_.end())
     return i->second.get();
 
   // No match using the host of the URL, so do a lookup using the scheme for
   // URLs on the form source_name://extra_bits/foo .
   i = data_sources_.find(url.scheme() + "://");
   if (i != data_sources_.end())
     return i->second.get();
 
   // No matches found, so give up.
   return nullptr;
 }
 
 void URLDataManagerBackend::CallStartRequest(
     scoped_refptr<URLDataSourceImpl> source,
     const std::string& path,
+    int child_id,
     const ResourceRequestInfo::WebContentsGetter& wc_getter,
     int request_id) {
-  if (BrowserThread::CurrentlyOn(BrowserThread::UI) && !wc_getter.is_null() &&
-      !wc_getter.Run()) {
-    // Make the request fail if its initiating WebContents is no longer valid.
+  if (BrowserThread::CurrentlyOn(BrowserThread::UI) && child_id != -1 &&
+      !RenderProcessHost::FromID(child_id)) {
+    // Make the request fail if its initiating renderer is no longer valid.
     // This can happen when the IO thread posts this task just before the
-    // WebContents shuts down.
+    // renderer shuts down.
+    // Note we check the process id instead of wc_getter because requests from
+    // workers wouldn't have a WebContents.
     source->SendResponse(request_id, nullptr);
     return;
   }
   source->source()->StartDataRequest(
       path,
       wc_getter,
       base::Bind(&URLDataSourceImpl::SendResponse, source, request_id));
 }
 
 void URLDataManagerBackend::RemoveRequest(URLRequestChromeJob* job) {
@@ skipping 62 matching lines @@
 
 }  // namespace
 
 net::URLRequestJobFactory::ProtocolHandler* CreateDevToolsProtocolHandler(
     ResourceContext* resource_context,
     bool is_incognito) {
   return new DevToolsJobFactory(resource_context, is_incognito);
 }
 
 }  // namespace content