Add a flag to disable server experiments in DRP

Add a flag to disable server experiments in DRP

Use this flag to disable server experiments field trial when
running Lo-Fi integration tests.

BUG=619622
Committed: https://crrev.com/c55229da0bb74bc1a1abee81fd248fe1843448d1
Cr-Commit-Position: refs/heads/master@{#399592}

[tbansal1, 2016-06-13 20:20:23]

Description was changed from

==========
Flag to disable tamper detection experiment

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

to

==========
Flag to disable server experiments

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

[tbansal1, 2016-06-13 20:39:55]

Patchset #1 (id:1) has been deleted

[tbansal1, 2016-06-13 20:40:03]

Patchset #1 (id:20001) has been deleted

[tbansal1, 2016-06-13 20:40:11]

Patchset #1 (id:40001) has been deleted

[tbansal1, 2016-06-13 20:40:27]

Patchset #1 (id:60001) has been deleted

[tbansal1, 2016-06-13 20:41:20]

Description was changed from

==========
Flag to disable server experiments

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

to

==========
Add a flag to disable server experiments in DRP

Use this flag to disable server experiments field trial when
running Lo-Fi integration tests.

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

[tbansal1, 2016-06-13 20:49:07]

Patchset #1 (id:80001) has been deleted

[tbansal1, 2016-06-13 20:57:23]

Patchset #1 (id:100001) has been deleted

[tbansal1, 2016-06-13 20:57:46]

Patchset #1 (id:120001) has been deleted

[tbansal1, 2016-06-13 21:10:10]

Description was changed from

==========
Add a flag to disable server experiments in DRP

Use this flag to disable server experiments field trial when
running Lo-Fi integration tests.

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

to

==========
Add a flag to disable server experiments in DRP

Use this flag to disable server experiments field trial when
running Lo-Fi integration tests.

BUG=619622
==========

[tbansal1, 2016-06-13 21:10:28]

Patchset #1 (id:140001) has been deleted

[tbansal1, 2016-06-13 21:13:40]

tbansal@chromium.org changed reviewers:
+ bustamante@chromium.org, kundaji@chromium.org

[tbansal1, 2016-06-13 21:13:41]

PTAL. Thanks.

[bustamante, 2016-06-13 21:18:59]

On 2016/06/13 21:13:41, tbansal1 wrote:
> PTAL. Thanks.

lgtm

For changes to tools/chrome_proxy/integration_tests/chrome_proxy_measurements.py

[Not at Google. Contact bengr, 2016-06-13 21:50:45]

lgtm

https://codereview.chromium.org/2062963003/diff/160001/components/data_reduct...
File components/data_reduction_proxy/core/common/data_reduction_proxy_params.cc
(right):

https://codereview.chromium.org/2062963003/diff/160001/components/data_reduct...
components/data_reduction_proxy/core/common/data_reduction_proxy_params.cc:126:
const std::string lo_fi_value =
Keep reference with '&'?

[tbansal1, 2016-06-13 22:00:07]

https://codereview.chromium.org/2062963003/diff/160001/components/data_reduct...
File components/data_reduction_proxy/core/common/data_reduction_proxy_params.cc
(right):

https://codereview.chromium.org/2062963003/diff/160001/components/data_reduct...
components/data_reduction_proxy/core/common/data_reduction_proxy_params.cc:126:
const std::string lo_fi_value =
On 2016/06/13 21:50:45, kundaji wrote:
> Keep reference with '&'?

Done.

[tbansal1, 2016-06-13 22:03:58]

The CQ bit was checked by tbansal@chromium.org

[tbansal1, 2016-06-13 22:03:59]

The patchset sent to the CQ was uploaded after l-g-t-m from
bustamante@chromium.org, kundaji@chromium.org
Link to the patchset: https://codereview.chromium.org/2062963003/#ps200001
(title: "kundaji comments")

[commit-bot: I haz the power, 2016-06-13 22:04:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2062963003/200001

[commit-bot: I haz the power, 2016-06-13 23:04:38]

Description was changed from

==========
Add a flag to disable server experiments in DRP

Use this flag to disable server experiments field trial when
running Lo-Fi integration tests.

BUG=619622
==========

to

==========
Add a flag to disable server experiments in DRP

Use this flag to disable server experiments field trial when
running Lo-Fi integration tests.

BUG=619622
==========

[commit-bot: I haz the power, 2016-06-13 23:04:38]

Committed patchset #3 (id:200001)

[commit-bot: I haz the power, 2016-06-13 23:04:50]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-13 23:06:39]

Description was changed from

==========
Add a flag to disable server experiments in DRP

Use this flag to disable server experiments field trial when
running Lo-Fi integration tests.

BUG=619622
==========

to

==========
Add a flag to disable server experiments in DRP

Use this flag to disable server experiments field trial when
running Lo-Fi integration tests.

BUG=619622

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

[commit-bot: I haz the power, 2016-06-13 23:06:40]

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