Add a new QuicFlagSaver class for saving/restoring the values of QUIC flags in tests.

Add a new QuicFlagSaver class for saving/restoring the values of QUIC flags in tests.

Also move the list of quic flags and their values into quic_flags_list.h which can be #included elsewhere.

Committed: https://crrev.com/b1c5698853e879c9fe7d8375ff7e5988d3e42152
Cr-Commit-Position: refs/heads/master@{#416403}

[Ryan Hamilton, 2016-09-02 21:44:27]

Description was changed from

==========
Add a new QuicFlagSaver class for saving/restoring the values of QUIC flags in
tests.
==========

to

==========
Add a new QuicFlagSaver class for saving/restoring the values of QUIC flags in
tests.

Also move the list of quic flags and their values into quic_flags_list.h which
can be #included elsewhere.
==========

[Ryan Hamilton, 2016-09-02 21:44:54]

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

[Ryan Hamilton, 2016-09-02 21:45:15]

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

[commit-bot: I haz the power, 2016-09-02 21:45:38]

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

[Jana, 2016-09-02 21:53:24]

lgtm

Very clever! I LOVE this use of macros, and the simplification of merge that
this brings. Can you please change the merge doc, and since this is probably
muscle memory for folks like me, send out an email to quic devs about this
change?

[Ryan Hamilton, 2016-09-02 22:01:42]

On 2016/09/02 21:53:24, Jana wrote:
> lgtm
> 
> Very clever! I LOVE this use of macros, and the simplification of merge that
> this brings. Can you please change the merge doc, and since this is probably
> muscle memory for folks like me, send out an email to quic devs about this
> change?

Will do both!

Thanks!

[commit-bot: I haz the power, 2016-09-02 23:57:07]

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

[commit-bot: I haz the power, 2016-09-02 23:57:08]

Dry run: This issue passed the CQ dry run.

[Ryan Hamilton, 2016-09-02 23:59:05]

The CQ bit was checked by rch@chromium.org

[commit-bot: I haz the power, 2016-09-02 23:59:47]

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-03 00:07:05]

Description was changed from

==========
Add a new QuicFlagSaver class for saving/restoring the values of QUIC flags in
tests.

Also move the list of quic flags and their values into quic_flags_list.h which
can be #included elsewhere.
==========

to

==========
Add a new QuicFlagSaver class for saving/restoring the values of QUIC flags in
tests.

Also move the list of quic flags and their values into quic_flags_list.h which
can be #included elsewhere.
==========

[commit-bot: I haz the power, 2016-09-03 00:07:06]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-03 00:11:41]

Description was changed from

==========
Add a new QuicFlagSaver class for saving/restoring the values of QUIC flags in
tests.

Also move the list of quic flags and their values into quic_flags_list.h which
can be #included elsewhere.
==========

to

==========
Add a new QuicFlagSaver class for saving/restoring the values of QUIC flags in
tests.

Also move the list of quic flags and their values into quic_flags_list.h which
can be #included elsewhere.

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

[commit-bot: I haz the power, 2016-09-03 00:11:42]

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