[subresource_filter] |enable_logging| should be part of the IPC with ActivationState

|enable_logging| should be part of the IPC with ActivationState.

BUG=724549
Review-Url: https://codereview.chromium.org/2894163002
Cr-Commit-Position: refs/heads/master@{#473342}
Committed: https://chromium.googlesource.com/chromium/src/+/1315a0fb48bc85823b8072b56abbe38dfcf1679f

[shivanisha, 2017-05-19 17:40:07]

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

[commit-bot: I haz the power, 2017-05-19 17:40:37]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[shivanisha, 2017-05-19 17:40:57]

Description was changed from

==========
enable_logging to be part of the IPC with ActivationState

BUG=
==========

to

==========
|enable_logging| should be part of the IPC with ActivationState.

BUG=
==========

[shivanisha, 2017-05-19 17:40:57]

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

[shivanisha, 2017-05-19 17:41:30]

PTAL, Thanks!

[Charlie Harrison, 2017-05-19 17:44:05]

LGTM but you'll need an IPC owner too

[Charlie Harrison, 2017-05-19 17:45:03]

Would you add a BUG= though?

[shivanisha, 2017-05-19 17:48:48]

Description was changed from

==========
|enable_logging| should be part of the IPC with ActivationState.

BUG=
==========

to

==========
|enable_logging| should be part of the IPC with ActivationState.

BUG=724549
==========

[shivanisha, 2017-05-19 17:51:19]

shivanisha@chromium.org changed reviewers:
+ dcheng@chromium.org

[shivanisha, 2017-05-19 17:51:55]

dcheng@, PTAL for owner's approval, Thanks!

[commit-bot: I haz the power, 2017-05-19 19:37:47]

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

[commit-bot: I haz the power, 2017-05-19 19:37:48]

Dry run: This issue passed the CQ dry run.

[dcheng, 2017-05-19 21:42:44]

The CQ bit was checked by dcheng@chromium.org

[dcheng, 2017-05-19 21:42:45]

lgtm

lgtm

[commit-bot: I haz the power, 2017-05-19 21:43:20]

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-19 21:49:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495230164757830, "parent_rev":
"315a8dfff8c17beac989fbb8b098286d406f4bc9", "commit_rev":
"1315a0fb48bc85823b8072b56abbe38dfcf1679f"}

[commit-bot: I haz the power, 2017-05-19 21:49:14]

Description was changed from

==========
|enable_logging| should be part of the IPC with ActivationState.

BUG=724549
==========

to

==========
|enable_logging| should be part of the IPC with ActivationState.

BUG=724549

Review-Url: https://codereview.chromium.org/2894163002
Cr-Commit-Position: refs/heads/master@{#473342}
Committed:
https://chromium.googlesource.com/chromium/src/+/1315a0fb48bc85823b8072b56abb...
==========

[commit-bot: I haz the power, 2017-05-19 21:49:16]

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