[Android WebView] Implement initial settings and callback support for Service Workers.

android_webview/browser/net/aw_request_interceptor.cc

Index: android_webview/browser/net/aw_request_interceptor.cc
diff --git a/android_webview/browser/net/aw_request_interceptor.cc b/android_webview/browser/net/aw_request_interceptor.cc
index 30ae888793ad4ec98ef57a42f7d7d574c6055586..fb8b37fec568233b6c288f85b13669878fa5024e 100644
--- a/android_webview/browser/net/aw_request_interceptor.cc
+++ b/android_webview/browser/net/aw_request_interceptor.cc
@@ -114,6 +114,23 @@ class ShouldInterceptRequestAdaptor
   DISALLOW_COPY_AND_ASSIGN(ShouldInterceptRequestAdaptor);
 };
 
+scoped_ptr<AwContentsIoThreadClient> GetCorrespondingIoThreadClient(
+    net::URLRequest* request) {
+  int render_process_id, render_frame_id;
+  if (!content::ResourceRequestInfo::GetRenderFrameForRequest(
+          request, &render_process_id, &render_frame_id)) {
+       return AwContentsIoThreadClient::GetServiceWorkerIoThreadClient();

[mnaganov (inactive), 2016/01/12 00:42:03]

Are you sure it's safe to assume that if a corresponding RenderFrame isn't
found, the request is from a shared worker? How about dedicated web workers for
example? I don't think they have a RenderFrame.

[timvolodine, 2016/01/13 17:07:58]

Just checked: regular web workers do have a valid render frame id so the
callbacks should go the usual way to the webview client. And we don't expose
shared workers in webview, so this approach seems safe enough at this point.

Currently there doesn't seem to be a (straightforward and reliable) way to check
if the request originated from a service worker. Looks like it would require
changes to IPC structs to have this info reliably. Could probably be done as a
separate patch if we really need it..

[mnaganov (inactive), 2016/01/13 17:55:27]

OK, please then add a comment here explaining why we can assume that absence if
frame id means it's a request from SW.

[timvolodine, 2016/01/14 19:43:55]

Done.

+  }
+
+  scoped_ptr<AwContentsIoThreadClient> io_thread_client =
+      AwContentsIoThreadClient::FromID(render_process_id, render_frame_id);
+
+  if (!io_thread_client)
+    return AwContentsIoThreadClient::GetServiceWorkerIoThreadClient();
+
+  return io_thread_client;
+}
+
 }  // namespace
 
 AwRequestInterceptor::AwRequestInterceptor() {}
@@ -129,14 +146,8 @@ net::URLRequestJob* AwRequestInterceptor::MaybeInterceptRequest(
   if (request->GetUserData(kRequestAlreadyHasJobDataKey))
     return nullptr;
 
-  int render_process_id, render_frame_id;
-  if (!content::ResourceRequestInfo::GetRenderFrameForRequest(
-          request, &render_process_id, &render_frame_id)) {
-    return nullptr;
-  }
-
   scoped_ptr<AwContentsIoThreadClient> io_thread_client =
-      AwContentsIoThreadClient::FromID(render_process_id, render_frame_id);
+      GetCorrespondingIoThreadClient(request);
 
   if (!io_thread_client)
     return nullptr;