Support dispatching ServiceWorker FetchEvent and receiving response

content/common/DEPS

 include_rules = [
   "+media/audio",
   "+media/base",
   "-webkit/browser",
   "-webkit/child",
   "-webkit/renderer",
 
   # No inclusion of WebKit from the browser, other than strictly enum/POD,
   # header-only types, and some selected common code.
   "-third_party/WebKit",
   "+third_party/WebKit/public/platform/WebCString.h",
   "+third_party/WebKit/public/platform/WebDeviceMotionData.h",
   "+third_party/WebKit/public/platform/WebDeviceOrientationData.h",
   "+third_party/WebKit/public/platform/WebFloatPoint.h",
   "+third_party/WebKit/public/platform/WebFloatRect.h",
   "+third_party/WebKit/public/platform/WebGamepads.h",
   "+third_party/WebKit/public/platform/WebGraphicsContext3D.h",
   "+third_party/WebKit/public/platform/WebHTTPBody.h",
   "+third_party/WebKit/public/platform/WebIDBCursor.h",
   "+third_party/WebKit/public/platform/WebIDBDatabase.h",
   "+third_party/WebKit/public/platform/WebIDBTypes.h",
   "+third_party/WebKit/public/platform/WebReferrerPolicy.h",
   "+third_party/WebKit/public/platform/WebScreenOrientation.h",
   "+third_party/WebKit/public/platform/WebScreenInfo.h",
   "+third_party/WebKit/public/platform/WebServiceWorkerError.h",
+  "+third_party/WebKit/public/platform/WebServiceWorkerResponse.h",

[Charlie Reis, 2014/03/05 21:00:34]

I see that there's already a DEPS exception for WebServiceWorkerError above, so
this is probably fine.  Can you explain why this is needed, though?

[falken, 2014/03/06 15:58:54]

Hmm.. it's not strictly needed but ServiceWorkerResponse is essentially just an
IPC-able version of WebServiceWorkerResponse so it seemed most convenient to
allow passing in WebServiceWorkerResponse to ServiceWorkerResponse's
constructor. The alternative would be to have a function somewhere else that
does the conversion, which I wouldn't mind doing.

WebServiceWorkerResponse is pretty simple, basically a bag of properties, but
not POD so it can't form an IPC message itself. If WebServiceWorkerResponse
changes, so should ServiceWorkerResponse, so the dependency exists regardless of
DEPS. WDYT?

Also, the DEPS change would require approval from a public/platform owner, so
+tkent.

[Charlie Reis, 2014/03/06 18:59:05]

+jam for his opinion.

[jam, 2014/03/06 21:22:43]

this doesn't pass the guideline in the common above, as it's not a strictly
enum/pod header. this wouldn't link in the chrome release build, since the
browser dll (which links in common) wouldn't link in blink

[falken, 2014/03/07 01:56:51]

OK, I removed the DEPS change.

Since just one place is doing the conversion now, I manually convert there. We
can move it to a separate function later if needed.

   "+third_party/WebKit/public/platform/WebStorageArea.h",
   "+third_party/WebKit/public/platform/WebString.h",
   "+third_party/WebKit/public/platform/linux/WebFontFamily.h",
   "+third_party/WebKit/public/platform/linux/WebFontRenderStyle.h",
   "+third_party/WebKit/public/web/mac/WebScrollbarTheme.h",
   "+third_party/WebKit/public/web/WebAXEnums.h",
   "+third_party/WebKit/public/web/WebCompositionUnderline.h",
   "+third_party/WebKit/public/web/WebContentSecurityPolicy.h",
   "+third_party/WebKit/public/web/WebDragOperation.h",
   "+third_party/WebKit/public/web/WebDragStatus.h",
   "+third_party/WebKit/public/web/WebFindOptions.h",
   "+third_party/WebKit/public/web/WebInputEvent.h",
   "+third_party/WebKit/public/web/WebMediaPlayerAction.h",
   "+third_party/WebKit/public/web/WebPageVisibilityState.h",
   "+third_party/WebKit/public/web/WebPluginAction.h",
   "+third_party/WebKit/public/web/WebPopupType.h",
   "+third_party/WebKit/public/web/WebTextDirection.h",
   "+third_party/WebKit/public/web/WebWindowFeatures.h",
 ]
 
 specific_include_rules = {
 # Java bridge code passes NPAPI types to the browser process. Crazy!
   "java_bridge_messages\.h": [
     "+content/child"
   ]
 }