67760 2011-09-07 21:08:11 -0700 layout test media/media-document-audio-repaint.html needs rebaseline 2012-01-10 23:21:50 -0800 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) All All RESOLVED FIXED P3 Normal --- 1 imasaki webkit-unassigned caseq dalecurtis fischman imasaki jamesr kbr scherkus oldest_to_newest 463873 0 imasaki 2011-09-07 21:08:11 -0700 Right now layout test media/media-document-audio-repaint.html (fails on Chromium GPU as you see below. http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20GPU%20Mesa%20-%20chromium.org&tests=media%2Fmedia-document-audio-repaint.html http://trac.webkit.org/browser/trunk/LayoutTests/media/media-document-audio-repaint.html#L8 shows it uses layoutTestController.display(). As for now. we will skip the test on GPU. However, it would be nice if this is implemented. Please let me know if you need more information. 464118 1 scherkus 2011-09-08 10:09:47 -0700 Adding jamesr/kbr to see if they have any ideas on implementing said DRT function I believe the common use case is to get DRT to paint over the entire screen with semi-transparent gray then trigger something that will cause a repaint, presumably testing that we're not painting too much. I'm not sure if this makes sense for GPU compositing or what the equivalent test would be. 464149 2 jamesr 2011-09-08 10:39:07 -0700 LayoutTestController.display() is (obviously) implemented, we have a ton of tests that depend on it. It doesn't have the same incremental repaint behavior as the software path because we don't do incremental composites, we always composite the full screen. You should NOT skip the test unless it crashes. Add the correct expectations. 464158 3 scherkus 2011-09-08 10:48:48 -0700 Thanks for the explanation! imasaki: want to double check our test expectations? the pixel result should look like the CPU version, except have a white background instead of the gray 467233 4 imasaki 2011-09-14 12:24:57 -0700 I double checked and the pixel result good. Rebaseline committed: https://bugs.webkit.org/attachment.cgi?id=105420&action=review 474786 5 imasaki 2011-09-28 09:49:22 -0700 sorry, my bad. The expected result does not look right. I will be working on rebaseline. 494307 6 imasaki 2011-11-01 16:53:24 -0700 I am working on rebaseline for GPU 494418 7 imasaki 2011-11-01 20:38:25 -0700 Rebaselines are committed. https://bugs.webkit.org/show_bug.cgi?id=67760 494550 8 caseq 2011-11-02 04:36:54 -0700 (In reply to comment #7) > Rebaselines are committed. https://bugs.webkit.org/show_bug.cgi?id=67760 The last rebaseline caused some text mistmatches: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20GPU%20Mesa%20-%20chromium.org&showExpectations=true&tests=media%2Fmedia-document-audio-repaint.html So I've done another one: http://trac.webkit.org/changeset/99053 531193 9 dalecurtis 2012-01-06 11:21:51 -0800 I'm rebaselining this test as a result of another change, https://bugs.webkit.org/show_bug.cgi?id=75505 . I suspect this can be closed once that one goes in. 531501 10 dalecurtis 2012-01-06 16:27:02 -0800 This can now be closed.