[subresource_filter] Swap Allowed/Block groups in Site Settings

[subresource_filter] Swap Allowed/Block groups in Site Settings

Subresource filter requires that the Allow group comes first.

BUG=689487
Review-Url: https://codereview.chromium.org/2882003003
Cr-Commit-Position: refs/heads/master@{#471910}
Committed: https://chromium.googlesource.com/chromium/src/+/d7c4b64db12ae0c5e19593df15cc64fd0817bc64

[Charlie Harrison, 2017-05-13 22:36:08]

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

[commit-bot: I haz the power, 2017-05-13 22:36:11]

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-05-13 23:20:20]

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

[commit-bot: I haz the power, 2017-05-13 23:20:21]

Dry run: This issue passed the CQ dry run.

[Charlie Harrison, 2017-05-14 00:33:12]

Description was changed from

==========
[subresource_filter] Swap Allowed/Block groups in Site Settings

BUG=689487
==========

to

==========
[subresource_filter] Swap Allowed/Block groups in Site Settings

Subresource filter requires that the Allow group comes first.

BUG=689487
==========

[Charlie Harrison, 2017-05-15 01:57:35]

csharrison@chromium.org changed reviewers:
+ dtrainor@chromium.org

[Charlie Harrison, 2017-05-15 01:57:36]

dtrainor: PTAL? Let me know if you think there's a better way to do this.

[David Trainor- moved to gerrit, 2017-05-15 17:54:26]

dtrainor@chromium.org changed reviewers:
+ twellington@chromium.org

[David Trainor- moved to gerrit, 2017-05-15 17:54:26]

+twellington in case she has any suggestions.  I think this is fine though. 
lgtm % her reply.

[Theresa, 2017-05-15 20:03:34]

lgtm

[Charlie Harrison, 2017-05-15 20:05:12]

Thanks folks :)

[Charlie Harrison, 2017-05-15 20:05:15]

The CQ bit was checked by csharrison@chromium.org

[commit-bot: I haz the power, 2017-05-15 20:05:47]

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-05-15 21:36:17]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494878715763770, "parent_rev":
"8e72a1beacc36f4f0e4d9cb3e4404cf8d896d931", "commit_rev":
"d7c4b64db12ae0c5e19593df15cc64fd0817bc64"}

[commit-bot: I haz the power, 2017-05-15 21:36:24]

Description was changed from

==========
[subresource_filter] Swap Allowed/Block groups in Site Settings

Subresource filter requires that the Allow group comes first.

BUG=689487
==========

to

==========
[subresource_filter] Swap Allowed/Block groups in Site Settings

Subresource filter requires that the Allow group comes first.

BUG=689487

Review-Url: https://codereview.chromium.org/2882003003
Cr-Commit-Position: refs/heads/master@{#471910}
Committed:
https://chromium.googlesource.com/chromium/src/+/d7c4b64db12ae0c5e19593df15cc...
==========

[commit-bot: I haz the power, 2017-05-15 21:36:29]

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