SharedWorker: Make sure route_id is always valid in WebSharedWorkerProxy

SharedWorker: Make sure route_id is always valid in WebSharedWorkerProxy

|route_id| passed to the ctor of WebSharedWorkerProxy is always valid because
it's assigned by synchronous IPC ("ViewHostMsg_CreateWorker") in
SharedWorkerRepository::createSharedWorkerConnector before creating the proxy.

BUG=612308
Committed: https://crrev.com/d8b82c0b2a4db7b524f3272b64fea16e2594a6fb
Cr-Commit-Position: refs/heads/master@{#441590}

[nhiroki, 2016-12-28 09:42:03]

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

[commit-bot: I haz the power, 2016-12-28 09:42:13]

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-28 09:45:10]

Description was changed from

==========
SharedWorker: Make sure route_id is always valid

|route_id| passed to the ctor of WebSharedWorkerProxy is always valid because
the id is generated immediately before creating the proxy by synchronous IPC
communication.

BUG=612308
==========

to

==========
SharedWorker: Make sure route_id is always valid in WebSharedWorkerProxy

|route_id| passed to the ctor of WebSharedWorkerProxy is always valid because
it's assigned by synchronous IPC ("ViewHostMsg_CreateWorker") in
SharedWorkerRepository::createSharedWorkerConnector immediately before creating
the proxy.

BUG=612308
==========

[nhiroki, 2016-12-28 09:45:25]

Description was changed from

==========
SharedWorker: Make sure route_id is always valid in WebSharedWorkerProxy

|route_id| passed to the ctor of WebSharedWorkerProxy is always valid because
it's assigned by synchronous IPC ("ViewHostMsg_CreateWorker") in
SharedWorkerRepository::createSharedWorkerConnector immediately before creating
the proxy.

BUG=612308
==========

to

==========
SharedWorker: Make sure route_id is always valid in WebSharedWorkerProxy

|route_id| passed to the ctor of WebSharedWorkerProxy is always valid because
it's assigned by synchronous IPC ("ViewHostMsg_CreateWorker") in
SharedWorkerRepository::createSharedWorkerConnector immediately before creating
the proxy.

BUG=612308
==========

[nhiroki, 2016-12-28 09:47:21]

Description was changed from

==========
SharedWorker: Make sure route_id is always valid in WebSharedWorkerProxy

|route_id| passed to the ctor of WebSharedWorkerProxy is always valid because
it's assigned by synchronous IPC ("ViewHostMsg_CreateWorker") in
SharedWorkerRepository::createSharedWorkerConnector immediately before creating
the proxy.

BUG=612308
==========

to

==========
SharedWorker: Make sure route_id is always valid in WebSharedWorkerProxy

|route_id| passed to the ctor of WebSharedWorkerProxy is always valid because
it's assigned by synchronous IPC ("ViewHostMsg_CreateWorker") in
SharedWorkerRepository::createSharedWorkerConnector before creating the proxy.

BUG=612308
==========

[nhiroki, 2016-12-28 10:08:36]

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

[nhiroki, 2016-12-28 10:08:36]

PTAL (Sorry for pushing a bunch of review requests in your queue :p)

[commit-bot: I haz the power, 2016-12-28 10:34:40]

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

[commit-bot: I haz the power, 2016-12-28 10:34:41]

Dry run: This issue passed the CQ dry run.

[kinuko, 2017-01-05 04:45:50]

lgtm

[nhiroki, 2017-01-05 04:46:28]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2017-01-05 04:46:43]

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, 2017-01-05 05:40:29]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483591588538360, "parent_rev":
"ae0f374bfb4180c38cad0dca1cd8815b5c67fb25", "commit_rev":
"233b67d4b984a435d96c96dfef8b0908dea5f7d6"}

[commit-bot: I haz the power, 2017-01-05 05:41:01]

Description was changed from

==========
SharedWorker: Make sure route_id is always valid in WebSharedWorkerProxy

|route_id| passed to the ctor of WebSharedWorkerProxy is always valid because
it's assigned by synchronous IPC ("ViewHostMsg_CreateWorker") in
SharedWorkerRepository::createSharedWorkerConnector before creating the proxy.

BUG=612308
==========

to

==========
SharedWorker: Make sure route_id is always valid in WebSharedWorkerProxy

|route_id| passed to the ctor of WebSharedWorkerProxy is always valid because
it's assigned by synchronous IPC ("ViewHostMsg_CreateWorker") in
SharedWorkerRepository::createSharedWorkerConnector before creating the proxy.

BUG=612308

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

[commit-bot: I haz the power, 2017-01-05 05:41:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-05 05:44:17]

Description was changed from

==========
SharedWorker: Make sure route_id is always valid in WebSharedWorkerProxy

|route_id| passed to the ctor of WebSharedWorkerProxy is always valid because
it's assigned by synchronous IPC ("ViewHostMsg_CreateWorker") in
SharedWorkerRepository::createSharedWorkerConnector before creating the proxy.

BUG=612308

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

to

==========
SharedWorker: Make sure route_id is always valid in WebSharedWorkerProxy

|route_id| passed to the ctor of WebSharedWorkerProxy is always valid because
it's assigned by synchronous IPC ("ViewHostMsg_CreateWorker") in
SharedWorkerRepository::createSharedWorkerConnector before creating the proxy.

BUG=612308

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

[commit-bot: I haz the power, 2017-01-05 05:44:18]

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