Use wire values in SpdySettingsIds.

Use wire values in SpdySettingsIds.

- Use wire values in SpdySettingsIds,
- remove SPDY/3.1-only setting values from SpdySettingsIds,
- rename ParseSettingId() to ParseSettingsId(),
- merge IsValidSettingId() into ParseSettingsId(),
- remove SerializeSettingId(),
- factor out SettingsIdToString(); remove default case
  to make sure every enum value is handled.

These values are not persisted to disk (that was a SPDY/3.1 feature),
so the only downside to the value change is Chrome net-log incompatibility.

This lands server side change 140592266 by bnc.

BUG=622737, 488484
Committed: https://crrev.com/08e5b8e42482ea6f11711333306df76c8ff24d80
Cr-Commit-Position: refs/heads/master@{#436200}

[Bence, 2016-12-02 22:08:55]

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

[commit-bot: I haz the power, 2016-12-02 22:10:00]

Dry run: 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-12-02 23:24:20]

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

[commit-bot: I haz the power, 2016-12-02 23:24:21]

Dry run: This issue passed the CQ dry run.

[Bence, 2016-12-03 00:20:26]

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

[Bence, 2016-12-03 00:20:27]

Ryan: PTAL.  Thank you.

[Ryan Hamilton, 2016-12-04 18:03:51]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2016-12-04 18:03:52]

lgtm

[commit-bot: I haz the power, 2016-12-04 18:04:03]

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-12-04 19:48:16]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480874631903400, "parent_rev":
"2e76c4f6f639d1efdc88f971882b02c95237da2f", "commit_rev":
"78dc39f36a1ace5c7239a887f1ddf7508d0a8252"}

[commit-bot: I haz the power, 2016-12-04 19:48:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-04 19:50:33]

Description was changed from

==========
Use wire values in SpdySettingsIds.

- Use wire values in SpdySettingsIds,
- remove SPDY/3.1-only setting values from SpdySettingsIds,
- rename ParseSettingId() to ParseSettingsId(),
- merge IsValidSettingId() into ParseSettingsId(),
- remove SerializeSettingId(),
- factor out SettingsIdToString(); remove default case
  to make sure every enum value is handled.

These values are not persisted to disk (that was a SPDY/3.1 feature),
so the only downside to the value change is Chrome net-log incompatibility.

This lands server side change 140592266 by bnc.

BUG=622737, 488484
==========

to

==========
Use wire values in SpdySettingsIds.

- Use wire values in SpdySettingsIds,
- remove SPDY/3.1-only setting values from SpdySettingsIds,
- rename ParseSettingId() to ParseSettingsId(),
- merge IsValidSettingId() into ParseSettingsId(),
- remove SerializeSettingId(),
- factor out SettingsIdToString(); remove default case
  to make sure every enum value is handled.

These values are not persisted to disk (that was a SPDY/3.1 feature),
so the only downside to the value change is Chrome net-log incompatibility.

This lands server side change 140592266 by bnc.

BUG=622737, 488484

Committed: https://crrev.com/08e5b8e42482ea6f11711333306df76c8ff24d80
Cr-Commit-Position: refs/heads/master@{#436200}
==========

[commit-bot: I haz the power, 2016-12-04 19:50:33]

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