Revert "Make children indexing implicit by using vector."

This reverts commit dd2eac7b1c3d1f4290df4738343bf7dcf4102f96.

Reason for revert:

Findit ( identified CL at revision 631781 as the
culprit for failures in the build cycles as shown on:

Sample Failed Build:

Sample Failed Step: components_unittests

Original change's description:
> Make children indexing implicit by using vector.
> The quad trees we construct are complete by design.
> So we can simply use a vector to index the children of each node.
> This saves space by removing the need to preserve indices (which were ints before).
> However, the trees could now be inflated with additional null leaves (which we didn't save before).
> We believe this results in smaller trees in the aggregate.
> Bug: 929155
> Change-Id: Iaf8a1d61cb239eee32d85aaa518e89363e49a3da
> Reviewed-on:
> Reviewed-by: anthonyvd <>
> Reviewed-by: Moe Ahmadi <>
> Commit-Queue: Alexandre Frechette <>
> Cr-Commit-Position: refs/heads/master@{#631781}

Change-Id: Ia88bf136dbd990b77b76de8c212813359971ed42
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 929155
Cr-Commit-Position: refs/heads/master@{#631801}
2 files changed