Mojo C++ bindings: Add mojo::Array<T>::reserve()

Mojo C++ bindings: Add mojo::Array<T>::reserve()

This is needed to avoid patterns like:

    mojo::Array<T> array(set.size());
    size_t i = 0;
    for (const auto& element : set) {
      array[i++] = element;
    }

and instead do:

    mojo::Array<T> array;
    array.reserve(set.size());
    for (const auto& element : set) {
      array.push_back(element);
    }

BUG=None

Committed: https://crrev.com/1b7ca74a52b9ecf2d5e98d327db45a979af4d84c
Cr-Commit-Position: refs/heads/master@{#421762}

[Luis Héctor Chávez, 2016-09-29 02:09:32]

lhchavez@chromium.org changed reviewers:
+ rockot@chromium.org, yzshen@chromium.org

[Luis Héctor Chávez, 2016-09-29 02:09:32]

rockot PTAL
yzshen FYI

[Ken Rockot(use gerrit already), 2016-09-29 05:23:27]

Lgtm but FYI we'd like to get rid of mojo::Array anyway. If you remove
use_new_wrapper_types=false from your mojom rules they'll generate code to use
std::vector instead.

[Luis Héctor Chávez, 2016-09-29 05:25:48]

On 2016/09/29 05:23:27, Ken Rockot wrote:
> Lgtm but FYI we'd like to get rid of mojo::Array anyway. If you remove
> use_new_wrapper_types=false from your mojom rules they'll generate code to use
> std::vector instead.

all right, i'll check this in and will see how hard removing that is next.
thanks!

[Luis Héctor Chávez, 2016-09-29 05:25:56]

The CQ bit was checked by lhchavez@chromium.org

[commit-bot: I haz the power, 2016-09-29 05:26:12]

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-29 06:31:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-29 06:33:36]

Description was changed from

==========
Mojo C++ bindings: Add mojo::Array<T>::reserve()

This is needed to avoid patterns like:

    mojo::Array<T> array(set.size());
    size_t i = 0;
    for (const auto& element : set) {
      array[i++] = element;
    }

and instead do:

    mojo::Array<T> array;
    array.reserve(set.size());
    for (const auto& element : set) {
      array.push_back(element);
    }

BUG=None
==========

to

==========
Mojo C++ bindings: Add mojo::Array<T>::reserve()

This is needed to avoid patterns like:

    mojo::Array<T> array(set.size());
    size_t i = 0;
    for (const auto& element : set) {
      array[i++] = element;
    }

and instead do:

    mojo::Array<T> array;
    array.reserve(set.size());
    for (const auto& element : set) {
      array.push_back(element);
    }

BUG=None

Committed: https://crrev.com/1b7ca74a52b9ecf2d5e98d327db45a979af4d84c
Cr-Commit-Position: refs/heads/master@{#421762}
==========

[commit-bot: I haz the power, 2016-09-29 06:33:37]

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