[webkit-reviews] review requested: [Bug 42267] Canvas: Make assigning the same fillStyle or strokeStyle a fast no-op : [Attachment 61529] Proposed patch

[bugzilla-daemon at webkit.org]

Andreas Kling <andreas.kling at nokia.com> has asked  for review:
Bug 42267: Canvas: Make assigning the same fillStyle or strokeStyle a fast
no-op
https://bugs.webkit.org/show_bug.cgi?id=42267

Attachment 61529: Proposed patch
https://bugs.webkit.org/attachment.cgi?id=61529&action=review

------- Additional Comments from Andreas Kling <andreas.kling at nokia.com>
Benchmarking with Hixie's fillRect video test:
http://hixie.ch/tests/adhoc/perf/video/002.html

BEFORE:

Elapsed wall-clock time: 905ms (ideal: 640ms).
Elapsed non-idle time: 265ms (ideal: 0ms).
Speed: 16.57fps (ideal: 25.00fps).

AFTER:

Elapsed wall-clock time: 853ms (ideal: 640ms).
Elapsed non-idle time: 213ms (ideal: 0ms).
Speed: 17.58fps (ideal: 25.00fps).