Intent to Implement and ship : FetchEvent's Client attribute

37 views
Skip to first unread message

Paritosh Kumar Monu

unread,
Apr 20, 2015, 6:08:17 AM4/20/15
to blin...@chromium.org

Contact emails

parit...@samsung.com


Spec

https://slightlyoff.github.io/ServiceWorker/spec/service_worker/#fetch-event-interface


Summary

Add 'client' member to FetchEvent interface.


Motivation

https://code.google.com/p/chromium/issues/detail?id=470036



Compatibility Risk

Gecko: This is supported in Firefox (under the assumption that the pref to enable serviceworkers is flipped).

WebKit: Not Supported.




Will this feature be supported on all six Blink platforms (Windows, Mac, Linux, Chrome OS, Android, and Android WebView)?

Yes.


OWP launch tracking bug?

https://code.google.com/p/chromium/issues/detail?id=470036



**This post is an upgrade of https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/glY9gVnCq1A/discussion

TAMURA, Kent

unread,
Apr 21, 2015, 4:37:11 AM4/21/15
to Paritosh Kumar Monu, blink-dev
LGTM to ship.  Looks a small addition to the existing feature.

--
TAMURA Kent
Software Engineer, Google


Philip Jägenstedt

unread,
Apr 21, 2015, 11:38:17 AM4/21/15
to TAMURA, Kent, Paritosh Kumar Monu, blink-dev
LGTM2. I don't know if there's some other bits of API or behavior that logically should go together with this change, but trust that will be noticed in code review if nothing else.

To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+...@chromium.org.

Chris Harrelson

unread,
Apr 21, 2015, 12:56:28 PM4/21/15
to Philip Jägenstedt, TAMURA, Kent, Paritosh Kumar Monu, blink-dev
LGTM3
Reply all
Reply to author
Forward
0 new messages