Identify service worker version at main resource load time.

content/child/request_extra_data.h

Index: content/child/request_extra_data.h
diff --git a/content/child/request_extra_data.h b/content/child/request_extra_data.h
index 3c98921712b4ae3b96addf554e67cee9b6370f22..5a6b4b0edbcccccac359bf1ab07a4575abbec129 100644
--- a/content/child/request_extra_data.h
+++ b/content/child/request_extra_data.h
@@ -13,11 +13,14 @@
 
 namespace content {
 
-// The RenderView stores an instance of this class in the "extra data" of each
-// ResourceRequest (see RenderFrameImpl::willSendRequest).
+// The RenderView and EmbeddedWorkerContextClient store an instance of this
+// class in the "extra data" of each ResourceRequest (see the willSendRequest
+// methods of RenderFrameImpl and EmbeddedWorkerContextClient willSendRequest).
 class CONTENT_EXPORT RequestExtraData
     : NON_EXPORTED_BASE(public webkit_glue::WebURLRequestExtraDataImpl) {
  public:
+  RequestExtraData();
+  explicit RequestExtraData(int service_worker_provider_id);

[dcheng, 2014/03/17 22:12:10]

I'm not a huge fan of the constructor from int, since there are a number of
other int fields and it's not particularly obvious to me that a constructor from
int "should" be a service worker provider id.

[michaeln, 2014/03/17 22:53:58]

I'd chose this instead of a public setter for just that single data member. I'd
be fine with changing this around to cons up a RequestExtraData struct with just
that one field set differently then the defaults. Did you have anything
particular in mind?

   RequestExtraData(blink::WebPageVisibilityState visibility_state,
                    const blink::WebString& custom_user_agent,
                    bool was_after_preconnect_request,