Use the serviceworker with mojo by default

Use the serviceworker with mojo by default

This patch changes --mojo-service-worker to --disable-mojo-service-worker.
Let's try mojo for the service worker.

BUG=629701
Committed: https://crrev.com/391fa63bcefd25b35addf678356828dded5ae71e
Cr-Commit-Position: refs/heads/master@{#427013}

[shimazu, 2016-10-21 05:16:44]

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

[shimazu, 2016-10-21 05:16:44]

ptal

[horo, 2016-10-21 06:55:07]

lgtm
great work!

[shimazu, 2016-10-21 06:55:52]

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

[commit-bot: I haz the power, 2016-10-21 06:56:05]

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

[shimazu, 2016-10-21 06:57:18]

shimazu@chromium.org changed reviewers:
+ avi@chromium.org

[shimazu, 2016-10-21 06:57:19]

avi@: Could you review changes in the following?
- content/browser/renderer_host/render_process_host_impl.cc
- content/public

[commit-bot: I haz the power, 2016-10-21 10:29:23]

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

[commit-bot: I haz the power, 2016-10-21 10:29:24]

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_...)

[Avi (use Gerrit), 2016-10-21 13:02:19]

On 2016/10/21 06:57:19, shimazu wrote:
> avi@: Could you review changes in the following?
> - content/browser/renderer_host/render_process_host_impl.cc
> - content/public

Those file LGTM.

Good luck!

[shimazu, 2016-10-24 01:21:03]

The CQ bit was checked by shimazu@chromium.org

[commit-bot: I haz the power, 2016-10-24 01:21:13]

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-10-24 01:41:59]

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

[commit-bot: I haz the power, 2016-10-24 01:42:00]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[shimazu, 2016-10-24 02:07:24]

The CQ bit was checked by shimazu@chromium.org

[shimazu, 2016-10-24 02:07:25]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org,
horo@chromium.org
Link to the patchset: https://codereview.chromium.org/2436223004/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2016-10-24 02:07:36]

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-10-24 04:17:57]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-24 04:20:25]

Description was changed from

==========
Use the serviceworker with mojo by default

This patch changes --mojo-service-worker to --disable-mojo-service-worker.
Let's try mojo for the service worker.

BUG=629701
==========

to

==========
Use the serviceworker with mojo by default

This patch changes --mojo-service-worker to --disable-mojo-service-worker.
Let's try mojo for the service worker.

BUG=629701

Committed: https://crrev.com/391fa63bcefd25b35addf678356828dded5ae71e
Cr-Commit-Position: refs/heads/master@{#427013}
==========

[commit-bot: I haz the power, 2016-10-24 04:20:26]

Patchset 2 (id:??) landed as
https://crrev.com/391fa63bcefd25b35addf678356828dded5ae71e
Cr-Commit-Position: refs/heads/master@{#427013}

[dmazzoni, 2016-10-24 16:44:07]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2443983002/ by dmazzoni@chromium.org.

The reason for reverting is: I think this is causing
InstallableManagerBrowserTest.CheckManifest404 to crash, see
http://crbug.com/658702
.