SharedWorker: Remove unnecessary indirection in SharedWorkerMessageFilter

SharedWorker: Remove unnecessary indirection in SharedWorkerMessageFilter

This is a cleanup CL and should not change behavior.

BUG=612308
Review-Url: https://codereview.chromium.org/2650783004
Cr-Commit-Position: refs/heads/master@{#445665}
Committed: https://chromium.googlesource.com/chromium/src/+/726b3a462b704b0398e5686dc67c6fbecfbf9353

[nhiroki, 2017-01-24 04:31:26]

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

[nhiroki, 2017-01-24 04:31:27]

PTAL, thanks!

https://codereview.chromium.org/2650783004/diff/1/content/browser/shared_work...
File content/browser/shared_worker/shared_worker_message_filter.cc (right):

https://codereview.chromium.org/2650783004/diff/1/content/browser/shared_work...
content/browser/shared_worker/shared_worker_message_filter.cc:82:
IPC_MESSAGE_HANDLER(WorkerProcessHostMsg_AllowIndexedDB, OnAllowIndexedDB)
I couldn't replace RequestFileSystemAccessSync and AllowIndexedDB with
MESSAGE_FORWARD. I tried it but got template errors about a discrepancy between
function argument list and actual given arguments. Presumably, a synchronous
message doesn't support MESSAGE_FORWARD or needs special treatment?

Anyway, I'm now interested only in WorkerHostMsg_XXX for UseCounter support and
mojofication, and it'd be ok to leave them as are.

[nhiroki, 2017-01-24 04:31:36]

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

[commit-bot: I haz the power, 2017-01-24 04:31:54]

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, 2017-01-24 05:41:45]

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

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

Dry run: This issue passed the CQ dry run.

[horo, 2017-01-24 06:17:28]

lgtm

[nhiroki, 2017-01-24 06:22:16]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2017-01-24 06:22:41]

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-24 06:26:29]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485238936768020, "parent_rev":
"ec5c65e074b72a641beced4cec2b32824e50730f", "commit_rev":
"726b3a462b704b0398e5686dc67c6fbecfbf9353"}

[commit-bot: I haz the power, 2017-01-24 06:27:34]

Description was changed from

==========
SharedWorker: Remove unnecessary indirection in SharedWorkerMessageFilter

This is a cleanup CL and should not change behavior.

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

to

==========
SharedWorker: Remove unnecessary indirection in SharedWorkerMessageFilter

This is a cleanup CL and should not change behavior.

BUG=612308

Review-Url: https://codereview.chromium.org/2650783004
Cr-Commit-Position: refs/heads/master@{#445665}
Committed:
https://chromium.googlesource.com/chromium/src/+/726b3a462b704b0398e5686dc67c...
==========

[commit-bot: I haz the power, 2017-01-24 06:27:36]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/726b3a462b704b0398e5686dc67c...