Fixing regression in putImageData blink perf test

Fixing regression in putImageData blink perf test

Creating the image data object by using createImageData instead of
getImageData to avoid state changes that interfere with the
benchmark's execution.  Also doing a drive-by code simplification

BUG=630670
Committed: https://crrev.com/c3a213e577d12c648fce456d2e0098fe36194abd
Cr-Commit-Position: refs/heads/master@{#409263}

[Justin Novosad, 2016-07-27 20:25:32]

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

[commit-bot: I haz the power, 2016-07-27 20:25:49]

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

[Justin Novosad, 2016-07-27 20:25:58]

junov@chromium.org changed reviewers:
+ xidachen@chromium.org

[Justin Novosad, 2016-07-27 20:25:58]

PTAL

[commit-bot: I haz the power, 2016-07-27 22:06:47]

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

[commit-bot: I haz the power, 2016-07-27 22:06:48]

Dry run: This issue passed the CQ dry run.

[xidachen, 2016-07-28 01:17:59]

https://codereview.chromium.org/2190523003/diff/1/third_party/WebKit/Performa...
File third_party/WebKit/PerformanceTests/Canvas/putImageData.html (right):

https://codereview.chromium.org/2190523003/diff/1/third_party/WebKit/Performa...
third_party/WebKit/PerformanceTests/Canvas/putImageData.html:10: imageData =
ctx2D.createImageData(canvas2D.width, canvas2D.height);
If my understanding is correct, this imageData will be transparent black. In
that case, can we just do var imageData = new ImageData(canvas2d.width,
canvas2d.height)? This will largely eliminate the overhead of this test.

[Justin Novosad, 2016-08-02 17:54:39]

Done

[Justin Novosad, 2016-08-02 17:54:47]

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

[commit-bot: I haz the power, 2016-08-02 17:55:22]

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

[xidachen, 2016-08-02 17:55:36]

lgtm

[Justin Novosad, 2016-08-02 18:28:15]

The CQ bit was unchecked by junov@chromium.org

[Justin Novosad, 2016-08-02 18:28:23]

The CQ bit was checked by junov@chromium.org

[commit-bot: I haz the power, 2016-08-02 18:29:20]

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-08-02 19:11:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-02 19:12:54]

Description was changed from

==========
Fixing regression in putImageData blink perf test

Creating the image data object by using createImageData instead of
getImageData to avoid state changes that interfere with the
benchmark's execution.  Also doing a drive-by code simplification

BUG=630670
==========

to

==========
Fixing regression in putImageData blink perf test

Creating the image data object by using createImageData instead of
getImageData to avoid state changes that interfere with the
benchmark's execution.  Also doing a drive-by code simplification

BUG=630670

Committed: https://crrev.com/c3a213e577d12c648fce456d2e0098fe36194abd
Cr-Commit-Position: refs/heads/master@{#409263}
==========

[commit-bot: I haz the power, 2016-08-02 19:12:58]

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