ServiceWorker: Update worker state when waitUntil rejected.

ServiceWorker: Update worker state when waitUntil be rejected.

When "install" and "activate" events lifetime be extended by using
waitUntil() method, the rejection of passing promise will update worker
state to "redundant".

See http://slightlyoff.github.io/ServiceWorker/spec/service_worker/ 
[[Install]] and [[Activate]] algorithms.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=280997

[xiang, 2014-06-27 12:15:29]

PTAL, thanks.

[falken, 2014-06-30 01:59:03]

On 2014/06/27 12:15:29, xiang wrote:
> PTAL, thanks.

This looks right, but can you write a test for this change? I think a Blink-side
layout test should work: register a worker, have it reject the install/activate
event, then check its status. Could you also update the CL description to
mention the part of the spec this implements?

[xiang, 2014-06-30 03:01:28]

On 2014/06/30 01:59:03, falken wrote:
> On 2014/06/27 12:15:29, xiang wrote:
> > PTAL, thanks.
> 
> This looks right, but can you write a test for this change? I think a
Blink-side
> layout test should work: register a worker, have it reject the
install/activate
> event, then check its status. Could you also update the CL description to
> mention the part of the spec this implements?

Updated the description.

And a CL was sent(https://codereview.chromium.org/352423005/) to include these
tests on Blink side.
Please help to review it, thanks.

[falken, 2014-06-30 04:40:54]

On 2014/06/30 03:01:28, xiang wrote:
> On 2014/06/30 01:59:03, falken wrote:
> > On 2014/06/27 12:15:29, xiang wrote:
> > > PTAL, thanks.
> > 
> > This looks right, but can you write a test for this change? I think a
> Blink-side
> > layout test should work: register a worker, have it reject the
> install/activate
> > event, then check its status. Could you also update the CL description to
> > mention the part of the spec this implements?
> 
> Updated the description.

Thanks, one nit: The spec is still evolving so instead of or in addition to the
section numbers, it helps to say [[Install]] and [[Activate]] algorithms.
 
> 
> And a CL was sent(https://codereview.chromium.org/352423005/) to include these
> tests on Blink side.
> Please help to review it, thanks.

Ah, I see that now, thanks. In the future, it helps to link each patch to its
sibling patches for multi-sided patches like this.

[xiang, 2014-07-01 06:26:14]

On 2014/06/30 04:40:54, falken wrote:
> On 2014/06/30 03:01:28, xiang wrote:
> > On 2014/06/30 01:59:03, falken wrote:
> > > On 2014/06/27 12:15:29, xiang wrote:
> > > > PTAL, thanks.
> > > 
> > > This looks right, but can you write a test for this change? I think a
> > Blink-side
> > > layout test should work: register a worker, have it reject the
> > install/activate
> > > event, then check its status. Could you also update the CL description to
> > > mention the part of the spec this implements?
> > 
> > Updated the description.
> 
> Thanks, one nit: The spec is still evolving so instead of or in addition to
the
> section numbers, it helps to say [[Install]] and [[Activate]] algorithms.
>  

Description updated.

> > 
> > And a CL was sent(https://codereview.chromium.org/352423005/) to include
these
> > tests on Blink side.
> > Please help to review it, thanks.
> 
> Ah, I see that now, thanks. In the future, it helps to link each patch to its
> sibling patches for multi-sided patches like this.

OK, I will follow the style, thanks.

[xiang, 2014-07-02 08:18:56]

On 2014/07/01 06:26:14, xiang wrote:
> On 2014/06/30 04:40:54, falken wrote:
> > On 2014/06/30 03:01:28, xiang wrote:
> > > On 2014/06/30 01:59:03, falken wrote:
> > > > On 2014/06/27 12:15:29, xiang wrote:
> > > > > PTAL, thanks.
> > > > 
> > > > This looks right, but can you write a test for this change? I think a
> > > Blink-side
> > > > layout test should work: register a worker, have it reject the
> > > install/activate
> > > > event, then check its status. Could you also update the CL description
to
> > > > mention the part of the spec this implements?
> > > 
> > > Updated the description.
> > 
> > Thanks, one nit: The spec is still evolving so instead of or in addition to
> the
> > section numbers, it helps to say [[Install]] and [[Activate]] algorithms.
> >  
> 
> Description updated.
> 
> > > 
> > > And a CL was sent(https://codereview.chromium.org/352423005/) to include
> these
> > > tests on Blink side.
> > > Please help to review it, thanks.
> > 
> > Ah, I see that now, thanks. In the future, it helps to link each patch to
its
> > sibling patches for multi-sided patches like this.
> 
> OK, I will follow the style, thanks.

ping falken@, how do you think about this CL now? Thanks.

[falken, 2014-07-02 08:24:03]

On 2014/07/02 08:18:56, xiang wrote:
> On 2014/07/01 06:26:14, xiang wrote:
> > On 2014/06/30 04:40:54, falken wrote:
> > > On 2014/06/30 03:01:28, xiang wrote:
> > > > On 2014/06/30 01:59:03, falken wrote:
> > > > > On 2014/06/27 12:15:29, xiang wrote:
> > > > > > PTAL, thanks.
> > > > > 
> > > > > This looks right, but can you write a test for this change? I think a
> > > > Blink-side
> > > > > layout test should work: register a worker, have it reject the
> > > > install/activate
> > > > > event, then check its status. Could you also update the CL description
> to
> > > > > mention the part of the spec this implements?
> > > > 
> > > > Updated the description.
> > > 
> > > Thanks, one nit: The spec is still evolving so instead of or in addition
to
> > the
> > > section numbers, it helps to say [[Install]] and [[Activate]] algorithms.
> > >  
> > 
> > Description updated.
> > 
> > > > 
> > > > And a CL was sent(https://codereview.chromium.org/352423005/) to include
> > these
> > > > tests on Blink side.
> > > > Please help to review it, thanks.
> > > 
> > > Ah, I see that now, thanks. In the future, it helps to link each patch to
> its
> > > sibling patches for multi-sided patches like this.
> > 
> > OK, I will follow the style, thanks.
> 
> ping falken@, how do you think about this CL now? Thanks.

lgtm, assuming the tests will land after this

[xiang, 2014-07-02 08:30:31]

The CQ bit was checked by xiang.long@intel.com

[xiang, 2014-07-02 08:30:52]

The CQ bit was unchecked by xiang.long@intel.com

[xiang, 2014-07-02 08:48:52]

The CQ bit was checked by xiang.long@intel.com

[commit-bot: I haz the power, 2014-07-02 08:50:06]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/xiang.long@intel.com/355163002/60001

[commit-bot: I haz the power, 2014-07-02 10:49:22]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  ios_rel_device on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device/builds...)

[commit-bot: I haz the power, 2014-07-02 10:56:30]

Change committed as 280997