Avoid unnecessary HashTable resizing during copy construction.

Avoid unnecessary HashTable resizing during copy construction.

We do know the final table size, so reserve its size before starting
to copy over. Thereby avoiding intermediate table allocations.

R=esprehn
BUG=

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

[sof, 2017-02-22 13:29:24]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-22 13:29:43]

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

[sof, 2017-02-22 14:33:13]

sigbjornf@opera.com changed reviewers:
+ esprehn@chromium.org, haraken@chromium.org

[sof, 2017-02-22 14:33:14]

please take a look.

Noticed some allocation churn due to this for resource responses & requests.

[esprehn, 2017-02-22 14:44:52]

lgtm

[sof, 2017-02-22 14:46:57]

Description was changed from

==========
Avoid unnecessary HashTable resizing during copy construction.

We do know the final table size, so reserve its size before starting
to copy over. Thereby avoiding intermediate table allocations.

R=
BUG=
==========

to

==========
Avoid unnecessary HashTable resizing during copy construction.

We do know the final table size, so reserve its size before starting
to copy over. Thereby avoiding intermediate table allocations.

R=esprehn
BUG=
==========

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

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

[commit-bot: I haz the power, 2017-02-22 15:22:10]

Dry run: This issue passed the CQ dry run.

[sof, 2017-02-22 15:23:22]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2017-02-22 15:23:37]

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-02-22 15:27:41]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487777002101160, "parent_rev":
"fe033cc08fe0572cd3ee971800db6482c7838563", "commit_rev":
"c1f114d257fb0edcf6cc54ab871cd48638d8601c"}

[commit-bot: I haz the power, 2017-02-22 15:28:20]

Description was changed from

==========
Avoid unnecessary HashTable resizing during copy construction.

We do know the final table size, so reserve its size before starting
to copy over. Thereby avoiding intermediate table allocations.

R=esprehn
BUG=
==========

to

==========
Avoid unnecessary HashTable resizing during copy construction.

We do know the final table size, so reserve its size before starting
to copy over. Thereby avoiding intermediate table allocations.

R=esprehn
BUG=

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

[commit-bot: I haz the power, 2017-02-22 15:28:21]

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

[haraken, 2017-02-22 23:27:30]

LGTM