Use QUIC for DRP if field trial group starts with Enabled

Use QUIC for DataReductionProxy if user belongs to field
trial group that starts with "Enabled".

Currently, QUIC is used if the field trial group is exactly
"Enabled". This change allows us to have multiple Enabled
groups (e.g., "Enabled", "Enabled2" etc.).

BUG=533703
Committed: https://crrev.com/5cc3f0a27b1c73102bfeee73d6fe24e3400de633
Cr-Commit-Position: refs/heads/master@{#350061}

[tbansal1, 2015-09-19 00:20:00]

tbansal@chromium.org changed reviewers:
+ sclittle@chromium.org

[tbansal1, 2015-09-19 00:20:00]

PTAL. Thanks!

[sclittle, 2015-09-19 00:45:21]

lgtm % nit

https://codereview.chromium.org/1354283002/diff/1/chrome/browser/io_thread_un...
File chrome/browser/io_thread_unittest.cc (right):

https://codereview.chromium.org/1354283002/diff/1/chrome/browser/io_thread_un...
chrome/browser/io_thread_unittest.cc:184: std::string(), false,
nit: Could these be done in 1 line each, e.g.

{std::string(), false},
{"Enabled", true},
...

[tbansal1, 2015-09-21 16:55:52]

tbansal@chromium.org changed reviewers:
+ rch@chromium.org

[tbansal1, 2015-09-21 16:55:53]

rch, PTAL at io_thread_unittest.cc. Thanks!

https://codereview.chromium.org/1354283002/diff/1/chrome/browser/io_thread_un...
File chrome/browser/io_thread_unittest.cc (right):

https://codereview.chromium.org/1354283002/diff/1/chrome/browser/io_thread_un...
chrome/browser/io_thread_unittest.cc:184: std::string(), false,
On 2015/09/19 00:45:21, sclittle wrote:
> nit: Could these be done in 1 line each, e.g.
> 
> {std::string(), false},
> {"Enabled", true},
> ...

git cl format does not like it, and changes it back to multi-line.

[tbansal1, 2015-09-21 17:47:38]

Patchset #2 (id:20001) has been deleted

[Ryan Hamilton, 2015-09-21 17:50:47]

io_thread_unittests.cc: LGTM, mod one nit.

https://codereview.chromium.org/1354283002/diff/40001/chrome/browser/io_threa...
File chrome/browser/io_thread_unittest.cc (right):

https://codereview.chromium.org/1354283002/diff/40001/chrome/browser/io_threa...
chrome/browser/io_thread_unittest.cc:212: EXPECT_EQ(1024 * 1024,
params.quic_socket_receive_buffer_size) << i;
It doesn't seem like you need to test this here, do you?

[tbansal1, 2015-09-21 17:53:15]

Removed extra check.

https://codereview.chromium.org/1354283002/diff/40001/chrome/browser/io_threa...
File chrome/browser/io_thread_unittest.cc (right):

https://codereview.chromium.org/1354283002/diff/40001/chrome/browser/io_threa...
chrome/browser/io_thread_unittest.cc:212: EXPECT_EQ(1024 * 1024,
params.quic_socket_receive_buffer_size) << i;
On 2015/09/21 17:50:47, Ryan Hamilton wrote:
> It doesn't seem like you need to test this here, do you?

Removed.

[tbansal1, 2015-09-21 20:23:46]

Patchset #5 (id:100001) has been deleted

[tbansal1, 2015-09-21 20:23:52]

Patchset #4 (id:80001) has been deleted

[tbansal1, 2015-09-21 20:42:45]

Patchset #4 (id:120001) has been deleted

[tbansal1, 2015-09-21 21:21:28]

Patchset #4 (id:140001) has been deleted

[tbansal1, 2015-09-21 21:21:33]

Patchset #4 (id:160001) has been deleted

[tbansal1, 2015-09-21 21:21:39]

Patchset #4 (id:180001) has been deleted

[tbansal1, 2015-09-21 22:12:47]

The CQ bit was checked by tbansal@chromium.org

[tbansal1, 2015-09-21 22:12:47]

The patchset sent to the CQ was uploaded after l-g-t-m from
sclittle@chromium.org, rch@chromium.org
Link to the patchset: https://codereview.chromium.org/1354283002/#ps200001
(title: "Rebased")

[commit-bot: I haz the power, 2015-09-21 22:14:09]

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

[commit-bot: I haz the power, 2015-09-21 23:38:56]

Committed patchset #4 (id:200001)

[commit-bot: I haz the power, 2015-09-21 23:40:27]

Patchset 4 (id:??) landed as
https://crrev.com/5cc3f0a27b1c73102bfeee73d6fe24e3400de633
Cr-Commit-Position: refs/heads/master@{#350061}