Subresource Filter: Fix browsertest to preserve feature flags

Subresource Filter: Fix browsertest to preserve feature flags

Since the browsertest modifies the feature flags after browser startup,
it can lead to inconsistent and unexpected behavior.

This CL preserves the existing features flags.

BUG=649783
Committed: https://crrev.com/a9d229b2bee6d538cc2c79930c7ad0ae404e25be
Cr-Commit-Position: refs/heads/master@{#420701}

[tommycli, 2016-09-23 19:04:31]

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

[tommycli, 2016-09-23 19:04:51]

tommycli@chromium.org changed reviewers:
+ engedy@chromium.org

[tommycli, 2016-09-23 19:04:51]

engedy: PTAL, this patch implements your suggested fix.

It passed trybots here: https://codereview.chromium.org/2360083003/

Thanks!

[commit-bot: I haz the power, 2016-09-23 19:05:09]

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

[engedy, 2016-09-23 19:22:29]

Description was changed from

==========
Subresource Filter: Fix browsertest to preserve feature flags

Since the browsertest modifies the feature flags after browser startup,
it can lead to inconsistent and unexpected behavior.

This CL preserves the existing features flags.

BUG=626728
==========

to

==========
Subresource Filter: Fix browsertest to preserve feature flags

Since the browsertest modifies the feature flags after browser startup,
it can lead to inconsistent and unexpected behavior.

This CL preserves the existing features flags.

BUG=649783
==========

[engedy, 2016-09-23 19:23:55]

Awesome, thank you, LGTM.  I have filed a bug and updated the CL description
with its number.

[tommycli, 2016-09-23 19:30:29]

The CQ bit was unchecked by tommycli@chromium.org

[tommycli, 2016-09-23 19:30:30]

The CQ bit was checked by tommycli@chromium.org

[tommycli, 2016-09-23 19:30:42]

engedy: thank you for the suggested fix and quick response time!

[commit-bot: I haz the power, 2016-09-23 19:30:53]

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-09-23 19:46:54]

Description was changed from

==========
Subresource Filter: Fix browsertest to preserve feature flags

Since the browsertest modifies the feature flags after browser startup,
it can lead to inconsistent and unexpected behavior.

This CL preserves the existing features flags.

BUG=649783
==========

to

==========
Subresource Filter: Fix browsertest to preserve feature flags

Since the browsertest modifies the feature flags after browser startup,
it can lead to inconsistent and unexpected behavior.

This CL preserves the existing features flags.

BUG=649783
==========

[commit-bot: I haz the power, 2016-09-23 19:46:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-23 19:49:54]

Description was changed from

==========
Subresource Filter: Fix browsertest to preserve feature flags

Since the browsertest modifies the feature flags after browser startup,
it can lead to inconsistent and unexpected behavior.

This CL preserves the existing features flags.

BUG=649783
==========

to

==========
Subresource Filter: Fix browsertest to preserve feature flags

Since the browsertest modifies the feature flags after browser startup,
it can lead to inconsistent and unexpected behavior.

This CL preserves the existing features flags.

BUG=649783

Committed: https://crrev.com/a9d229b2bee6d538cc2c79930c7ad0ae404e25be
Cr-Commit-Position: refs/heads/master@{#420701}
==========

[commit-bot: I haz the power, 2016-09-23 19:49:55]

Patchset 1 (id:??) landed as
https://crrev.com/a9d229b2bee6d538cc2c79930c7ad0ae404e25be
Cr-Commit-Position: refs/heads/master@{#420701}