Bug 46319 – REGRESSION: Lots of fast canvas updates fail to completely render

• Status: RESOLVED
• Resolution: FIXED
• Component: Canvas

[Joel Martin]

This page demonstrates the problem: http://kanaka.github.com/noVNC/webkit_canvas_bug.html

When you press the Start button, the noVNC engine will playback the beginning of a VNC/RFB session. You should see a complete 1024x768 Ubuntu desktop rendered below. However, only the first few rows of pixels will render.

Note that the canvas operations have succeeded, but they are simply not rendering correctly; if you resize the browser slightly, the entire desktop will immediately render. Also, if part of the canvas is out of view, when you scroll it into view, those parts show correct rendering.

I bifurcated using Chromium nightly builds on Linux to determine that the problem was introduced after WebKit revision 66240 and before (or in) WebKit revision 66396.

I have also verified that the problem still exists in WebKit-r67643 on Linux and also in the r67637 WebKit nightly build on Windows XP.

[Joel Martin]

Created attachment 68471 [details]
Picture of canvas failure to render

Here is a picture of the state where the canvas commands have completed, but the canvas is not showing all the data (i.e. the image should continue well past the bottom of the window.

[Antoine Mercadal]

I confirm.

[Alexey Proskuryakov]

Confirmed with a local debug build of r68167 on Mac OS X.

[Matthew Delaney]

I believe this was caused by this change: https://bugs.webkit.org/show_bug.cgi?id=44741 http://trac.webkit.org/changeset/66295

CC-ing Vincent.

[Simon Fraser (smfr)]

<rdar://problem/8471076>

[Vincent Scheib]

I have it reproduced, and will investigate.

[Vincent Scheib]

This was caused by http://trac.webkit.org/changeset/66295, logic error in CanvasRenderingContext2D::putImageData (sourceRect incorrect).

I have a fix in hand and am looking for tests that should have caught this or will write one.

[Vincent Scheib]

Created attachment 69133 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 69133 [details]
Patch

Rejecting patch 69133 from commit-queue.

Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', 'build-and-test', '--no-clean', '--no-update', '--test', '--quiet', '--non-interactive']" exit_code: 2
Last 500 characters of output:
ccessful.
Files=14, Tests=304,  1 wallclock secs ( 0.65 cusr +  0.14 csys =  0.79 CPU)
Running build-dumprendertree
Compiling Java tests
make: Nothing to be done for `default'.
Running tests from /Users/abarth/git/webkit-queue/LayoutTests
Testing 21409 test cases.
fast/repaint/canvas-putImageData.html -> failed

Exiting early after 1 failures. 15031 tests run.
201.57s total testing time

15030 test cases (99%) succeeded
1 test case (<1%) had incorrect layout
3 test cases (<1%) had stderr output

Full output: http://queues.webkit.org/results/4157029

[Vincent Scheib]

Created attachment 69415 [details]
Patch

[Vincent Scheib]

Comment on attachment 69415 [details]
Patch

Previous commit queue failed - caught an error with the test. The expected results had carriage returns in them.

[WebKit Commit Bot]

Comment on attachment 69415 [details]
Patch

Clearing flags on attachment: 69415

Committed r68868: <http://trac.webkit.org/changeset/68868>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Joel Martin]

Awesome! I pulled the nightly and tested with noVNC. Looks good. Thanks.

[Suresh Voruganti]

Top list error for Services, so please cherry pick the fix for Qtwebkit 2.1