Intent to Remove: InstallEvent Interface

[Paritosh]

 

Primary eng (and PM) emails

parit...@samsung.com

 

Link to “Intent to Deprecate” thread

Never used from Script

 

Summary

Removing InstallEvent Interface from ServiceWorker, as this get removed in https://github.com/slightlyoff/ServiceWorker/commit/4d172287dc5f92530ac251f68a670f5f4de9540c

 

Motivation

https://github.com/slightlyoff/ServiceWorker/issues/661

http://code.google.com/p/chromium/issues/detail?id=470032

 

Alternative

We can use Extendable interface for install events.

 

Compatibility Risk

Low. As Never used from script.

Paritosh

[TAMURA, Kent]

> Never used from Script

I'm confused.  Do you mean InstallEvent interface haven't shipped?

--

TAMURA Kent
Software Engineer, Google

[Matt Falkenhagen]

It has shipped and can be used in script as "new InstallEvent()" or "event instanceof InstallEvent".

But most likely, no one does this. The install event is a special lifetime event dispatched to the Service Worker so it can do initialization before receiving functional events. Dispatching yourself an install event wouldn't make sense, and there's no reason outside of tests to do instanceof InstallEvent. So it's should be safe to make this change.

[TAMURA, Kent]

So, in summary,

- The interface will be changed for 'install' event: InstallEvent -> ExtendableEvent, which was a parent interface of InstallEvent

- InstallEvent::activeWorker will be removed

Is this right?

[Matt Falkenhagen]

(reply all this time) 

Correct, but we never shipped InstallEvent::activeWorker.

[TAMURA, Kent]

Thanks.  I finally understand the compatibility risk is very low.  LGTM.

[Dimitri Glazkov]

LGTM2

[Chris Harrelson]

LGTM

LGTM2

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