Change AlternativeServiceInfo to a class

Change AlternativeServiceInfo to a class

This is in preparation for adding a private field: advertised_versions
to AlternativeServiceInfo.

BUG=689762
Review-Url: https://codereview.chromium.org/2932503005
Cr-Commit-Position: refs/heads/master@{#478453}
Committed: https://chromium.googlesource.com/chromium/src/+/422ce3567ed22a4611f0ab5caccbc0cf123040df

[Zhongyi Shi, 2017-06-09 06:40:15]

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

[commit-bot: I haz the power, 2017-06-09 06:40:21]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Zhongyi Shi, 2017-06-09 06:44:29]

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

[Zhongyi Shi, 2017-06-09 06:44:29]

bnc@: PTAL, thanks!

This is originated from https://codereview.chromium.org/2901093004/, where we
added a new field to AlternativeServiceInfo: advertised_versions. And we want to
make that field private in particular so that versions will always be sorted in
ascending order when set, a struct is not sufficient any more. Thus we need to
change AlternativeServiceInfo to a class. To make the other cl more clean, I
separate the refactoring from struct to class in this code review.

[commit-bot: I haz the power, 2017-06-09 07:52:02]

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

[commit-bot: I haz the power, 2017-06-09 07:52:03]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Bence, 2017-06-09 18:01:56]

LGTM.  Thank you for taking the extra effort to factor this change out,
https://crrev.com/2901093004 will now be much cleaner this way!

[Zhongyi Shi, 2017-06-09 22:58:35]

The CQ bit was checked by zhongyi@chromium.org

[commit-bot: I haz the power, 2017-06-09 22:59:02]

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, 2017-06-09 23:28:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497049115274190, "parent_rev":
"be6445469de9d55cbed8ad2cddb6ac204116214d", "commit_rev":
"422ce3567ed22a4611f0ab5caccbc0cf123040df"}

[commit-bot: I haz the power, 2017-06-09 23:29:11]

Description was changed from

==========
Change AlternativeServiceInfo to a class

This is in preparation for adding a private field: advertised_versions
to AlternativeServiceInfo.

BUG=689762
==========

to

==========
Change AlternativeServiceInfo to a class

This is in preparation for adding a private field: advertised_versions
to AlternativeServiceInfo.

BUG=689762

Review-Url: https://codereview.chromium.org/2932503005
Cr-Commit-Position: refs/heads/master@{#478453}
Committed:
https://chromium.googlesource.com/chromium/src/+/422ce3567ed22a4611f0ab5caccb...
==========

[commit-bot: I haz the power, 2017-06-09 23:29:13]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/422ce3567ed22a4611f0ab5caccb...