Use Vector<...>::append(const U*, size_t) in SVGPathByteStream

Use Vector<...>::append(const U*, size_t) in SVGPathByteStream

This appears to help larger paths a decent amount, while not hurting
smaller paths. Payload for this copy is 2-26 bytes. For a ~3.5k
character path string this reduced runtime of a
setAttribute('d', <path>) micro-benchmark (w/ mostly 10 byte payloads)
by roughly 17%.

BUG=568735
Committed: https://crrev.com/119dbdc4873edbe693a6b715b9cc53613a7e5721
Cr-Commit-Position: refs/heads/master@{#365365}

[fs, 2015-12-15 14:26:52]

fs@opera.com changed reviewers:
+ pdr@chromium.org

[fs, 2015-12-15 14:26:52]

This seemed like a simple enough change with decent enough improvement - without
starting to molest Vector and do something like
https://codereview.chromium.org/1524813002 (which adds an additional 5% to the
gain.)

[fs, 2015-12-15 14:44:25]

Description was changed from

==========
Use Vector<...>::append(const U*, size_t) in SVGPathByteStream

This appears to help larger paths a decent amount, while not hurting
smaller paths. Payload for this copy is 10-26 bytes. For a ~3.5k
character path string this reduced runtime of a
setAttribute('d', <path>) micro-benchmark by roughly 17%.

BUG=568735
==========

to

==========
Use Vector<...>::append(const U*, size_t) in SVGPathByteStream

This appears to help larger paths a decent amount, while not hurting
smaller paths. Payload for this copy is 2-26 bytes. For a ~3.5k
character path string this reduced runtime of a
setAttribute('d', <path>) micro-benchmark by roughly 17%.

BUG=568735
==========

[fs, 2015-12-15 14:45:52]

Description was changed from

==========
Use Vector<...>::append(const U*, size_t) in SVGPathByteStream

This appears to help larger paths a decent amount, while not hurting
smaller paths. Payload for this copy is 2-26 bytes. For a ~3.5k
character path string this reduced runtime of a
setAttribute('d', <path>) micro-benchmark by roughly 17%.

BUG=568735
==========

to

==========
Use Vector<...>::append(const U*, size_t) in SVGPathByteStream

This appears to help larger paths a decent amount, while not hurting
smaller paths. Payload for this copy is 2-26 bytes. For a ~3.5k
character path string this reduced runtime of a
setAttribute('d', <path>) micro-benchmark (w/ mostly 10 byte payloads)
by roughly 17%.

BUG=568735
==========

[pdr., 2015-12-15 21:52:55]

On 2015/12/15 at 14:26:52, fs wrote:
> This seemed like a simple enough change with decent enough improvement -
without starting to molest Vector and do something like
https://codereview.chromium.org/1524813002 (which adds an additional 5% to the
gain.)

Nice! LGTM

[pdr., 2015-12-15 21:53:01]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2015-12-15 21:55:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1527613006/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1527613006/1

[commit-bot: I haz the power, 2015-12-15 22:45:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-15 22:47:22]

Description was changed from

==========
Use Vector<...>::append(const U*, size_t) in SVGPathByteStream

This appears to help larger paths a decent amount, while not hurting
smaller paths. Payload for this copy is 2-26 bytes. For a ~3.5k
character path string this reduced runtime of a
setAttribute('d', <path>) micro-benchmark (w/ mostly 10 byte payloads)
by roughly 17%.

BUG=568735
==========

to

==========
Use Vector<...>::append(const U*, size_t) in SVGPathByteStream

This appears to help larger paths a decent amount, while not hurting
smaller paths. Payload for this copy is 2-26 bytes. For a ~3.5k
character path string this reduced runtime of a
setAttribute('d', <path>) micro-benchmark (w/ mostly 10 byte payloads)
by roughly 17%.

BUG=568735

Committed: https://crrev.com/119dbdc4873edbe693a6b715b9cc53613a7e5721
Cr-Commit-Position: refs/heads/master@{#365365}
==========

[commit-bot: I haz the power, 2015-12-15 22:47:23]

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