[promises] Don't allocate new array before filling up existing array

[promises] Don't allocate new array before filling up existing array

Previously we created 3 FixedArrays and then filled them up with
values. This meant that during the creation of the second and third
FixedArray, there were one and two FixedArrays respectively, without
any values in it which broke the FixedArrayVerify.

This patch fills each FixedArray with the correct values before
creating new ones.

BUG=chromium:672051
Committed: https://crrev.com/87b84a341d06d53fbc1f189eede8b2709d864afe
Cr-Commit-Position: refs/heads/master@{#41564}

[gsathya, 2016-12-07 20:20:56]

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

[commit-bot: I haz the power, 2016-12-07 20:21:06]

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

[gsathya, 2016-12-07 20:21:59]

Description was changed from

==========
[promises] Don't allocate new array before filling up existing array

Previously we created 3 FixedArrays and then filled them up with
values. This meant that during the creation of the second and third
FixedArray, there were one and two FixedArrays respectively, without
any values in it which broke the FixedArrayVerify.

This patch fills each FixedArray with the correct values before
creating new ones.

BUG=chromium:672051
==========

to

==========
[promises] Don't allocate new array before filling up existing array

Previously we created 3 FixedArrays and then filled them up with
values. This meant that during the creation of the second and third
FixedArray, there were one and two FixedArrays respectively, without
any values in it which broke the FixedArrayVerify.

This patch fills each FixedArray with the correct values before
creating new ones.

BUG=chromium:672051
==========

[gsathya, 2016-12-07 20:21:59]

gsathya@chromium.org changed reviewers:
+ adamk@chromium.org, ishell@google.com, jochen@chromium.org

[commit-bot: I haz the power, 2016-12-07 20:46:01]

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

[commit-bot: I haz the power, 2016-12-07 20:46:01]

Dry run: This issue passed the CQ dry run.

[Igor Sheludko, 2016-12-07 20:48:10]

ishell@chromium.org changed reviewers:
+ ishell@chromium.org

[Igor Sheludko, 2016-12-07 20:48:10]

lgtm

[gsathya, 2016-12-07 20:48:58]

The CQ bit was checked by gsathya@chromium.org

[commit-bot: I haz the power, 2016-12-07 20:49:01]

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-07 20:50:57]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481143738023190, "parent_rev":
"06c897e0e53cc9248b344871aa8f4c5a67c6976a", "commit_rev":
"412c30773a57300114728d65c335b41805556d29"}

[commit-bot: I haz the power, 2016-12-07 20:51:04]

Description was changed from

==========
[promises] Don't allocate new array before filling up existing array

Previously we created 3 FixedArrays and then filled them up with
values. This meant that during the creation of the second and third
FixedArray, there were one and two FixedArrays respectively, without
any values in it which broke the FixedArrayVerify.

This patch fills each FixedArray with the correct values before
creating new ones.

BUG=chromium:672051
==========

to

==========
[promises] Don't allocate new array before filling up existing array

Previously we created 3 FixedArrays and then filled them up with
values. This meant that during the creation of the second and third
FixedArray, there were one and two FixedArrays respectively, without
any values in it which broke the FixedArrayVerify.

This patch fills each FixedArray with the correct values before
creating new ones.

BUG=chromium:672051
==========

[commit-bot: I haz the power, 2016-12-07 20:51:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-07 20:51:19]

Description was changed from

==========
[promises] Don't allocate new array before filling up existing array

Previously we created 3 FixedArrays and then filled them up with
values. This meant that during the creation of the second and third
FixedArray, there were one and two FixedArrays respectively, without
any values in it which broke the FixedArrayVerify.

This patch fills each FixedArray with the correct values before
creating new ones.

BUG=chromium:672051
==========

to

==========
[promises] Don't allocate new array before filling up existing array

Previously we created 3 FixedArrays and then filled them up with
values. This meant that during the creation of the second and third
FixedArray, there were one and two FixedArrays respectively, without
any values in it which broke the FixedArrayVerify.

This patch fills each FixedArray with the correct values before
creating new ones.

BUG=chromium:672051

Committed: https://crrev.com/87b84a341d06d53fbc1f189eede8b2709d864afe
Cr-Commit-Position: refs/heads/master@{#41564}
==========

[commit-bot: I haz the power, 2016-12-07 20:51:20]

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