cc: Improve performance of PaintOpBuffer::Playback with no ranges.

cc: Improve performance of PaintOpBuffer::Playback with no ranges.

In this case we were constructing 2 vectors on each call to playback()
which causes us to do malloc/free twice as well. Instead, define a
static vector and pass that which recovers basically all the lost time.

The other changes in NextOp try to improve the codegen, and seem to
maybe have up to 1ms improvement of the remaining delta of 2ms from
before cfc477b43f1c91e84e17180ee937728d6f496aa6, but it's in the
noise. However I think it's nicer with less nesting and only writing
the operator++ once anyways.

BUG=726031
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2905383002
Cr-Original-Commit-Position: refs/heads/master@{#475936}
Review-Url: https://codereview.chromium.org/2916703004 .
Cr-Commit-Position: refs/branch-heads/3112@{#95}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed: https://chromium.googlesource.com/chromium/src/+/feb6693d3c26cef1c1a1bb8ee1b2af015c4fb31a

[danakj, 2017-06-01 17:43:21]

Description was changed from

==========
cc: Improve performance of PaintOpBuffer::Playback with no ranges.

In this case we were constructing 2 vectors on each call to playback()
which causes us to do malloc/free twice as well. Instead, define a
static vector and pass that which recovers basically all the lost time.

The other changes in NextOp try to improve the codegen, and seem to
maybe have up to 1ms improvement of the remaining delta of 2ms from
before cfc477b43f1c91e84e17180ee937728d6f496aa6, but it's in the
noise. However I think it's nicer with less nesting and only writing
the operator++ once anyways.

BUG=726031
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2905383002
Cr-Commit-Position: refs/heads/master@{#475936}
(cherry picked from commit 4deb94b53a551930b3db578c4c339ec0ef95bc6b)
==========

to

==========
cc: Improve performance of PaintOpBuffer::Playback with no ranges.

In this case we were constructing 2 vectors on each call to playback()
which causes us to do malloc/free twice as well. Instead, define a
static vector and pass that which recovers basically all the lost time.

The other changes in NextOp try to improve the codegen, and seem to
maybe have up to 1ms improvement of the remaining delta of 2ms from
before cfc477b43f1c91e84e17180ee937728d6f496aa6, but it's in the
noise. However I think it's nicer with less nesting and only writing
the operator++ once anyways.

BUG=726031
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2905383002
Cr-Original-Commit-Position: refs/heads/master@{#475936}
Review-Url: https://codereview.chromium.org/2916703004 .
Cr-Commit-Position: refs/branch-heads/3112@{#95}
Cr-Branched-From:
b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed:
https://chromium.googlesource.com/chromium/src/+/feb6693d3c26cef1c1a1bb8ee1b2...
==========

[danakj, 2017-06-01 17:43:22]

Committed patchset #1 (id:1) manually as
feb6693d3c26cef1c1a1bb8ee1b2af015c4fb31a.