bindings: Make v8_union iterate through its members by type name

bindings: Make v8_union iterate through its members by type name

This is being done in preparation for supporting flattening union member
types and deal with nested union types.

A union's flattened member types are represented as a set, so once we start
using it to generate the union V8->C++ conversion code we will need to do so
in a deterministic way.

This specific change just makes the current code iterate through its member
types sorted by name to update existing test expectations and reduce the
size of the upcoming nested union support patch.

BUG=700225
R=bashi@chromium.org,haraken@chromium.org,yukishiino@chromium.org

Review-Url: https://codereview.chromium.org/2742213002
Cr-Commit-Position: refs/heads/master@{#456338}
Committed: https://chromium.googlesource.com/chromium/src/+/a328b46a04ef32a9afdbed84930042c6d5b0d87f

[Raphael Kubo da Costa (rakuco), 2017-03-12 14:56:05]

PTAL. This is groundwork for https://codereview.chromium.org/2746713003

[Raphael Kubo da Costa (rakuco), 2017-03-12 14:56:11]

The CQ bit was checked by raphael.kubo.da.costa@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-12 14:56:15]

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

[haraken, 2017-03-12 15:15:59]

LGTM

[commit-bot: I haz the power, 2017-03-12 16:35:17]

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

[commit-bot: I haz the power, 2017-03-12 16:35:18]

Dry run: This issue passed the CQ dry run.

[bashi, 2017-03-12 23:41:18]

lgtm

[Raphael Kubo da Costa (rakuco), 2017-03-13 08:37:54]

The CQ bit was checked by raphael.kubo.da.costa@intel.com

[commit-bot: I haz the power, 2017-03-13 08:38:00]

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-03-13 08:41:39]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489394274721080, "parent_rev":
"4e533644399b42111d85ab607219494ed719f0cb", "commit_rev":
"a328b46a04ef32a9afdbed84930042c6d5b0d87f"}

[commit-bot: I haz the power, 2017-03-13 08:42:15]

Description was changed from

==========
bindings: Make v8_union iterate through its members by type name

This is being done in preparation for supporting flattening union member
types and deal with nested union types.

A union's flattened member types are represented as a set, so once we start
using it to generate the union V8->C++ conversion code we will need to do so
in a deterministic way.

This specific change just makes the current code iterate through its member
types sorted by name to update existing test expectations and reduce the
size of the upcoming nested union support patch.

BUG=700225
R=bashi@chromium.org,haraken@chromium.org,yukishiino@chromium.org
==========

to

==========
bindings: Make v8_union iterate through its members by type name

This is being done in preparation for supporting flattening union member
types and deal with nested union types.

A union's flattened member types are represented as a set, so once we start
using it to generate the union V8->C++ conversion code we will need to do so
in a deterministic way.

This specific change just makes the current code iterate through its member
types sorted by name to update existing test expectations and reduce the
size of the upcoming nested union support patch.

BUG=700225
R=bashi@chromium.org,haraken@chromium.org,yukishiino@chromium.org

Review-Url: https://codereview.chromium.org/2742213002
Cr-Commit-Position: refs/heads/master@{#456338}
Committed:
https://chromium.googlesource.com/chromium/src/+/a328b46a04ef32a9afdbed849300...
==========

[commit-bot: I haz the power, 2017-03-13 08:42:16]

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

[Yuki, 2017-03-13 08:43:27]

LGTM.