service worker: Relax the BadMessageReceived call in OnSetHostedVersionId

service worker: Relax the BadMessageReceived call in OnSetHostedVersionId

The message may be for an embedded worker that had
been detached.

BUG=625042
Committed: https://crrev.com/1dd05b7262de80415eb0d106a0ff0a471d675aad
Cr-Commit-Position: refs/heads/master@{#405748}

[falken, 2016-07-15 06:37:49]

Description was changed from

==========
service worker: Don't BadMessageReceived if a detached worker comes back from
the dead

BUG=625042
==========

to

==========
service worker: Relax the BadMessageReceived call in OnSetHostedVersionId 

The message may be for an embedded worker that had
been detached.

BUG=625042
==========

[falken, 2016-07-15 06:46:15]

The CQ bit was checked by falken@chromium.org to run a CQ dry run

[falken, 2016-07-15 06:46:19]

falken@chromium.org changed reviewers:
+ nhiroki@chromium.org

[falken, 2016-07-15 06:46:20]

ptal

[commit-bot: I haz the power, 2016-07-15 06:46:36]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[nhiroki, 2016-07-15 07:44:56]

lgtm

[falken, 2016-07-15 07:53:05]

falken@chromium.org changed reviewers:
+ dcheng@chromium.org

[falken, 2016-07-15 07:53:05]

dcheng: Can you review messages?

[commit-bot: I haz the power, 2016-07-15 09:24:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-15 09:24:54]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[dcheng, 2016-07-15 09:25:46]

LGTM

https://codereview.chromium.org/2154663002/diff/20001/content/browser/service...
File content/browser/service_worker/service_worker_dispatcher_host.cc (right):

https://codereview.chromium.org/2154663002/diff/20001/content/browser/service...
content/browser/service_worker/service_worker_dispatcher_host.cc:886: // If the
version has a different embedded worker, assume the message is about
Nit: newline above for consistency

[falken, 2016-07-15 11:30:35]

https://codereview.chromium.org/2154663002/diff/20001/content/browser/service...
File content/browser/service_worker/service_worker_dispatcher_host.cc (right):

https://codereview.chromium.org/2154663002/diff/20001/content/browser/service...
content/browser/service_worker/service_worker_dispatcher_host.cc:886: // If the
version has a different embedded worker, assume the message is about
On 2016/07/15 09:25:46, dcheng wrote:
> Nit: newline above for consistency

Done.

[falken, 2016-07-15 11:31:02]

The CQ bit was checked by falken@chromium.org

[falken, 2016-07-15 11:31:02]

The patchset sent to the CQ was uploaded after l-g-t-m from
nhiroki@chromium.org, dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2154663002/#ps40001
(title: "newl")

[commit-bot: I haz the power, 2016-07-15 11:31:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-15 13:07:06]

Description was changed from

==========
service worker: Relax the BadMessageReceived call in OnSetHostedVersionId 

The message may be for an embedded worker that had
been detached.

BUG=625042
==========

to

==========
service worker: Relax the BadMessageReceived call in OnSetHostedVersionId 

The message may be for an embedded worker that had
been detached.

BUG=625042
==========

[commit-bot: I haz the power, 2016-07-15 13:07:07]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-15 13:09:03]

Description was changed from

==========
service worker: Relax the BadMessageReceived call in OnSetHostedVersionId 

The message may be for an embedded worker that had
been detached.

BUG=625042
==========

to

==========
service worker: Relax the BadMessageReceived call in OnSetHostedVersionId

The message may be for an embedded worker that had
been detached.

BUG=625042

Committed: https://crrev.com/1dd05b7262de80415eb0d106a0ff0a471d675aad
Cr-Commit-Position: refs/heads/master@{#405748}
==========

[commit-bot: I haz the power, 2016-07-15 13:09:04]

Patchset 3 (id:??) landed as
https://crrev.com/1dd05b7262de80415eb0d106a0ff0a471d675aad
Cr-Commit-Position: refs/heads/master@{#405748}