ServiceWorker: Returns true for IsMojoForServiceWorkerEnabled

ServiceWorker: Returns true for IsMojoForServiceWorkerEnabled

This patch: 
- changes ServiceWorkerUtils::IsMojoForServiceWorkerEnabled to return always
  true instead of checking the --disable-mojo-service-worker flag (currently it
  already returns true by default)
- removes unit tests with --disable-mojo-service-worker

Layout tests will also be removed at a subsequent patch.

BUG=629701
Review-Url: https://codereview.chromium.org/2627543002
Cr-Commit-Position: refs/heads/master@{#442516}
Committed: https://chromium.googlesource.com/chromium/src/+/41ba1486431391060dd80bce6b6f60ffe0a8df8e

[shimazu, 2017-01-10 03:29:40]

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

[commit-bot: I haz the power, 2017-01-10 03:30:17]

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

[shimazu, 2017-01-10 03:30:17]

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

[shimazu, 2017-01-10 03:30:18]

ptal!

[falken, 2017-01-10 03:46:35]

lgtm but let's make the CL description more clear what's changed. Imagine
someone is investigating a regression and is trying to figure out whether this
patch could have caused it.  I think it's missing that the Mojo path is already
enabled by default and this is just removing the non-Mojo unit test coverage.

I think the key points are:
- Change IsMojoForServiceWorkerEnabled to return true instead of checking the
--disable-mojo-service-worker flag (the default was already true)
- Remove unit test coverage for the --disable-mojo-service-worker flag

[shimazu, 2017-01-10 03:56:32]

Description was changed from

==========
ServiceWorker: Returns true for IsMojoForServiceWorkerEnabled

This is the first patch to remove non-mojo path.
ServiceWorkerUtils::IsMojoForServiceWorkerEnabled returns always true and
non-mojo unittests are also removed. Layout tests will also removed at a
subsequent patch.

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

to

==========
ServiceWorker: Returns true for IsMojoForServiceWorkerEnabled

This patch:
 - changes ServiceWorkerUtils::IsMojoForServiceWorkerEnabled to return always
true instead of checking the --disable-mojo-service-worker flag (currently it
already returns true by default)
 - removes unit tests with --disable-mojo-service-worker

Layout tests will also removed at a subsequent patch.

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

[shimazu, 2017-01-10 03:58:42]

Description was changed from

==========
ServiceWorker: Returns true for IsMojoForServiceWorkerEnabled

This patch:
 - changes ServiceWorkerUtils::IsMojoForServiceWorkerEnabled to return always
true instead of checking the --disable-mojo-service-worker flag (currently it
already returns true by default)
 - removes unit tests with --disable-mojo-service-worker

Layout tests will also removed at a subsequent patch.

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

to

==========
ServiceWorker: Returns true for IsMojoForServiceWorkerEnabled

This patch: 
- changes ServiceWorkerUtils::IsMojoForServiceWorkerEnabled to return always
  true instead of checking the --disable-mojo-service-worker flag (currently it
  already returns true by default)
- removes unit tests with --disable-mojo-service-worker

Layout tests will also be removed at a subsequent patch.

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

[shimazu, 2017-01-10 03:59:16]

On 2017/01/10 03:46:35, falken wrote:
> lgtm but let's make the CL description more clear what's changed. Imagine
> someone is investigating a regression and is trying to figure out whether this
> patch could have caused it.  I think it's missing that the Mojo path is
already
> enabled by default and this is just removing the non-Mojo unit test coverage.
> 
> I think the key points are:
> - Change IsMojoForServiceWorkerEnabled to return true instead of checking the
> --disable-mojo-service-worker flag (the default was already true)
> - Remove unit test coverage for the --disable-mojo-service-worker flag

Thanks for your comment. Updated the description.

[shimazu, 2017-01-10 03:59:23]

The CQ bit was unchecked by shimazu@chromium.org

[shimazu, 2017-01-10 03:59:30]

The CQ bit was checked by shimazu@chromium.org

[commit-bot: I haz the power, 2017-01-10 03:59: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, 2017-01-10 06:18:01]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484020770389340, "parent_rev":
"1cc5c53cff510fe6ba8b256603a01012bd68ba9d", "commit_rev":
"41ba1486431391060dd80bce6b6f60ffe0a8df8e"}

[commit-bot: I haz the power, 2017-01-10 06:18:31]

Description was changed from

==========
ServiceWorker: Returns true for IsMojoForServiceWorkerEnabled

This patch: 
- changes ServiceWorkerUtils::IsMojoForServiceWorkerEnabled to return always
  true instead of checking the --disable-mojo-service-worker flag (currently it
  already returns true by default)
- removes unit tests with --disable-mojo-service-worker

Layout tests will also be removed at a subsequent patch.

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

to

==========
ServiceWorker: Returns true for IsMojoForServiceWorkerEnabled

This patch: 
- changes ServiceWorkerUtils::IsMojoForServiceWorkerEnabled to return always
  true instead of checking the --disable-mojo-service-worker flag (currently it
  already returns true by default)
- removes unit tests with --disable-mojo-service-worker

Layout tests will also be removed at a subsequent patch.

BUG=629701

Review-Url: https://codereview.chromium.org/2627543002
Cr-Commit-Position: refs/heads/master@{#442516}
Committed:
https://chromium.googlesource.com/chromium/src/+/41ba1486431391060dd80bce6b6f...
==========

[commit-bot: I haz the power, 2017-01-10 06:18:32]

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