Rename confusing method name.

Rename confusing method name.

ExtensionsService::DisableUserExtensions(ids) doesn't disable the passed
extensions, but all user extensions except the passed ids.

It's very hard to notice that from the caller side, and the actual
code does completely opposite.

This CL renames this confusing method name.

BUG=680429
TEST=Refactoring only.

Review-Url: https://codereview.chromium.org/2646803003
Cr-Commit-Position: refs/heads/master@{#445603}
Committed: https://chromium.googlesource.com/chromium/src/+/294bb3e342259c73d2abddd5b759104d34854b17

[mtomasz, 2017-01-20 02:32:40]

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

[commit-bot: I haz the power, 2017-01-20 02:33:10]

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

[mtomasz, 2017-01-20 02:33:17]

Description was changed from

==========
Rename confusing method name.

ExtensionsService::DisableUserExtensions(ids) doesn't disable the passed
extensions, but all user extensions except the passed ids.

It's very hard to notice that from the caller side, and the actual
code does completely opposite.

This CL renames this confusing method name.

BUG=680429
TEST=Refactoring only.
==========

to

==========
Rename confusing method name.

ExtensionsService::DisableUserExtensions(ids) doesn't disable the passed
extensions, but all user extensions except the passed ids.

It's very hard to notice that from the caller side, and the actual
code does completely opposite.

This CL renames this confusing method name.

BUG=680429
TEST=Refactoring only.
==========

[mtomasz, 2017-01-20 02:33:17]

mtomasz@chromium.org changed reviewers:
+ benwells@chromium.org

[mtomasz, 2017-01-20 02:33:24]

@benwells: PTAL. Thanks.

[mtomasz, 2017-01-20 02:36:28]

mtomasz@chromium.org changed reviewers:
+ battre@chromium.org
- benwells@chromium.org

[mtomasz, 2017-01-20 02:36:40]

@battre: PTAL (@benwells is OOO). Thanks.

[commit-bot: I haz the power, 2017-01-20 03:21:24]

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

[commit-bot: I haz the power, 2017-01-20 03:21:25]

Dry run: This issue passed the CQ dry run.

[battre, 2017-01-20 07:19:34]

LGTM and thank you!

[mtomasz, 2017-01-20 07:24:59]

The CQ bit was checked by mtomasz@chromium.org

[commit-bot: I haz the power, 2017-01-20 07:25:15]

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-20 07:41:55]

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

[commit-bot: I haz the power, 2017-01-20 07:41:56]

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

[mtomasz, 2017-01-23 00:56:14]

mtomasz@chromium.org changed reviewers:
+ lazyboy@chromium.org

[mtomasz, 2017-01-23 00:56:23]

@lazyboy: PTAL. Thanks.

[lazyboy, 2017-01-23 18:43:04]

thanks, lgtm

[mtomasz, 2017-01-24 00:26:35]

The CQ bit was checked by mtomasz@chromium.org

[commit-bot: I haz the power, 2017-01-24 00:27:38]

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 02:17:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485217595043200, "parent_rev":
"3d6e10d81b19b35794e834f6bd602d892b2bf2f9", "commit_rev":
"294bb3e342259c73d2abddd5b759104d34854b17"}

[commit-bot: I haz the power, 2017-01-24 02:18:51]

Description was changed from

==========
Rename confusing method name.

ExtensionsService::DisableUserExtensions(ids) doesn't disable the passed
extensions, but all user extensions except the passed ids.

It's very hard to notice that from the caller side, and the actual
code does completely opposite.

This CL renames this confusing method name.

BUG=680429
TEST=Refactoring only.
==========

to

==========
Rename confusing method name.

ExtensionsService::DisableUserExtensions(ids) doesn't disable the passed
extensions, but all user extensions except the passed ids.

It's very hard to notice that from the caller side, and the actual
code does completely opposite.

This CL renames this confusing method name.

BUG=680429
TEST=Refactoring only.

Review-Url: https://codereview.chromium.org/2646803003
Cr-Commit-Position: refs/heads/master@{#445603}
Committed:
https://chromium.googlesource.com/chromium/src/+/294bb3e342259c73d2abddd5b759...
==========

[commit-bot: I haz the power, 2017-01-24 02:18:52]

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