Flip quic flags by 12/12/2016 18:30 PST

Flip quic flags by 12/12/2016 18:30 PST

BUG=

[Zhongyi Shi, 2016-12-13 02:41:27]

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

[Ryan Hamilton, 2016-12-13 03:55:09]

https://codereview.chromium.org/2569703003/diff/1/net/quic/core/quic_flags_li...
File net/quic/core/quic_flags_list.h (right):

https://codereview.chromium.org/2569703003/diff/1/net/quic/core/quic_flags_li...
net/quic/core/quic_flags_list.h:105: 
These two flag fips are disabling flags. Can you confirm this is intended?

[Zhongyi Shi, 2016-12-13 05:00:28]

https://codereview.chromium.org/2569703003/diff/1/net/quic/core/quic_flags_li...
File net/quic/core/quic_flags_list.h (right):

https://codereview.chromium.org/2569703003/diff/1/net/quic/core/quic_flags_li...
net/quic/core/quic_flags_list.h:105: 
On 2016/12/13 03:55:09, Ryan Hamilton wrote:
> These two flag fips are disabling flags. Can you confirm this is intended?

Yup. The internal bug 33307008 shows these two flags were default enabled before
finch experiments was turned back. They will be re-enabled once the finch
experiment is live.

[Ryan Hamilton, 2016-12-13 05:40:33]

On 2016/12/13 05:00:28, Zhongyi Shi wrote:
>
https://codereview.chromium.org/2569703003/diff/1/net/quic/core/quic_flags_li...
> File net/quic/core/quic_flags_list.h (right):
> 
>
https://codereview.chromium.org/2569703003/diff/1/net/quic/core/quic_flags_li...
> net/quic/core/quic_flags_list.h:105: 
> On 2016/12/13 03:55:09, Ryan Hamilton wrote:
> > These two flag fips are disabling flags. Can you confirm this is intended?
> 
> Yup. The internal bug 33307008 shows these two flags were default enabled
before
> finch experiments was turned back. They will be re-enabled once the finch
> experiment is live.

LGTM!