SharedWorker: Make shared workers keyed by a pair of url and name

SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

Before this change, shared workers throws URLMismatchError when a given
worker's name is equal to an existing shared worker but a given script URL is
different. After this change, a new shared worker is simply created in the case.

BUG=538914
Committed: https://crrev.com/a2f777316c98a1bcc06f2d34120b7bec2c86a237
Cr-Commit-Position: refs/heads/master@{#440729}

[nhiroki, 2016-12-26 10:10:36]

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

[commit-bot: I haz the power, 2016-12-26 10:10:50]

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

[nhiroki, 2016-12-26 10:22:08]

Description was changed from

==========
SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

BUG=538914
==========

to

==========
SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

This is tested by web-platform-tests:
imported/wpt/workers/constructors/SharedWorker/URLMismatchError.htm

BUG=538914
==========

[nhiroki, 2016-12-26 10:49:23]

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

[commit-bot: I haz the power, 2016-12-26 10:49:33]

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

[nhiroki, 2016-12-26 10:50:39]

Description was changed from

==========
SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

This is tested by web-platform-tests:
imported/wpt/workers/constructors/SharedWorker/URLMismatchError.htm

BUG=538914
==========

to

==========
SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

BUG=538914
==========

[nhiroki, 2016-12-26 11:09:20]

Patchset #1 (id:1) has been deleted

[nhiroki, 2016-12-26 11:09:27]

Patchset #1 (id:20001) has been deleted

[nhiroki, 2016-12-26 11:10:27]

nhiroki@chromium.org changed reviewers:
+ haraken@chromium.org, kinuko@chromium.org

[nhiroki, 2016-12-26 11:11:27]

Hi, can you review this?

+kinuko@ for the entire changes
+haraken@ for blink changes

Thanks.

[nhiroki, 2016-12-26 11:15:02]

Description was changed from

==========
SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

BUG=538914
==========

to

==========
SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

Before this change, shared workers threw URLMismatchError when a given worker's
name is equal to an existing shared worker but a given URL is different. After
this change, a new shared worker is created in the case.

BUG=538914
==========

[nhiroki, 2016-12-26 11:16:17]

Description was changed from

==========
SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

Before this change, shared workers threw URLMismatchError when a given worker's
name is equal to an existing shared worker but a given URL is different. After
this change, a new shared worker is created in the case.

BUG=538914
==========

to

==========
SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

Before this change, shared workers throws URLMismatchError when a given
worker's name is equal to an existing shared worker but a given script URL is
different. After this change, a new shared worker is simply created in the case.

BUG=538914
==========

[commit-bot: I haz the power, 2016-12-26 12:08:06]

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

[commit-bot: I haz the power, 2016-12-26 12:08:07]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[nhiroki, 2016-12-26 12:25:05]

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

[commit-bot: I haz the power, 2016-12-26 12:25:19]

Dry run: 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-12-26 13:39:03]

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

[commit-bot: I haz the power, 2016-12-26 13:39:04]

Dry run: This issue passed the CQ dry run.

[nhiroki, 2016-12-27 01:36:37]

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

[commit-bot: I haz the power, 2016-12-27 01:36:43]

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

[kinuko, 2016-12-27 02:21:27]

lgtm

[haraken, 2016-12-27 02:44:31]

LGTM

[commit-bot: I haz the power, 2016-12-27 03:17:30]

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

[commit-bot: I haz the power, 2016-12-27 03:17:31]

Dry run: This issue passed the CQ dry run.

[nhiroki, 2016-12-27 04:00:56]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2016-12-27 04:01:00]

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-12-27 04:04:26]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1482811256745610,
"parent_rev": "1a2d2b2d81d66fd2d3a87d13ec308a258a4e57aa", "commit_rev":
"6e69c0c84105d2727a89a99d384f6402945aad1d"}

[commit-bot: I haz the power, 2016-12-27 04:08:53]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1482811256745610,
"parent_rev": "168d9972f22624bf3a198c4eca30e4ef63024e70", "commit_rev":
"3f1a4b7dcdbf9afaad4d8d72f9a340ada09993fc"}

[commit-bot: I haz the power, 2016-12-27 04:10:22]

Description was changed from

==========
SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

Before this change, shared workers throws URLMismatchError when a given
worker's name is equal to an existing shared worker but a given script URL is
different. After this change, a new shared worker is simply created in the case.

BUG=538914
==========

to

==========
SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

Before this change, shared workers throws URLMismatchError when a given
worker's name is equal to an existing shared worker but a given script URL is
different. After this change, a new shared worker is simply created in the case.

BUG=538914

Review-Url: https://codereview.chromium.org/2604733002
==========

[commit-bot: I haz the power, 2016-12-27 04:10:23]

Committed patchset #3 (id:80001)

[commit-bot: I haz the power, 2017-01-02 15:45:26]

Description was changed from

==========
SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

Before this change, shared workers throws URLMismatchError when a given
worker's name is equal to an existing shared worker but a given script URL is
different. After this change, a new shared worker is simply created in the case.

BUG=538914

Review-Url: https://codereview.chromium.org/2604733002
==========

to

==========
SharedWorker: Make shared workers keyed by a pair of url and name

This patch makes shared workers keyed by a pair of url and name, and deprecates
URLMismatchError that was deleted from the spec:
https://github.com/whatwg/html/pull/175

Before this change, shared workers throws URLMismatchError when a given
worker's name is equal to an existing shared worker but a given script URL is
different. After this change, a new shared worker is simply created in the case.

BUG=538914

Committed: https://crrev.com/a2f777316c98a1bcc06f2d34120b7bec2c86a237
Cr-Commit-Position: refs/heads/master@{#440729}
==========

[commit-bot: I haz the power, 2017-01-02 15:45:27]

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