Bug 48454

Summary: compositing/tiling/huge-layer* flakey
Product: WebKit Reporter: Adam Roben (:aroben) <aroben>
Component: Layout and RenderingAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, dino, gyuyoung.kim, jberlin, rakuco, simon.fraser, thorton, webkit.review.bot
Priority: P2 Keywords: InRadar, LayoutTestFailure, PlatformOnly
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
URL: http://build.webkit.org/results/Windows%20Release%20(Tests)/r70637%20(5661)/results.html
See Also: https://bugs.webkit.org/show_bug.cgi?id=167537
Attachments:
Description Flags
Patch
none
Patch
none
Patch simon.fraser: review+, webkit.review.bot: commit-queue-

Description Adam Roben (:aroben) 2010-10-27 12:33:57 PDT
The following two tests sometimes fail on Windows:

compositing/tiling/huge-layer-resize.html
compositing/tiling/huge-layer-with-layer-children-resize.html

See the URL for the diffs. I've added these tests to the Skipped file.
Comment 1 Adam Roben (:aroben) 2010-10-27 12:34:23 PDT
Here's an example of compositing/tiling/huge-layer-with-layer-children-resize.html passing: <http://build.webkit.org/results/Windows%20Debug%20(Tests)/r70645%20(21640)/results.html>.
Comment 2 Adam Roben (:aroben) 2010-10-27 12:34:59 PDT
The tests seem to work OK in the browser.
Comment 3 Adam Roben (:aroben) 2010-10-27 12:45:14 PDT
<rdar://problem/8601084>
Comment 5 Simon Fraser (smfr) 2011-06-23 15:50:06 PDT
I think we should make layerTreeAsText() eagerly update the layer's tiling status to avoid this.
Comment 6 Jessie Berlin 2011-06-23 16:06:30 PDT
Disabled the tests until we can implement Simon's suggestion in http://trac.webkit.org/changeset/89631
Comment 7 Simon Fraser (smfr) 2012-10-24 16:54:55 PDT
Created attachment 170515 [details]
Patch
Comment 8 mitz 2012-10-24 22:00:39 PDT
Comment on attachment 170515 [details]
Patch

rs=me
Comment 9 WebKit Review Bot 2012-10-24 23:17:02 PDT
Comment on attachment 170515 [details]
Patch

Attachment 170515 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/14593106

New failing tests:
compositing/tiling/huge-layer-img.html
platform/chromium/virtual/softwarecompositing/tiling/huge-layer-img.html
Comment 10 Simon Fraser (smfr) 2012-10-25 12:32:35 PDT
Created attachment 170711 [details]
Patch
Comment 11 WebKit Review Bot 2012-10-25 13:57:01 PDT
Comment on attachment 170711 [details]
Patch

Attachment 170711 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/14559736

New failing tests:
compositing/tiling/huge-layer-img.html
platform/chromium/virtual/softwarecompositing/tiling/huge-layer-img.html
Comment 12 Simon Fraser (smfr) 2012-10-25 16:40:26 PDT
Created attachment 170767 [details]
Patch
Comment 13 Simon Fraser (smfr) 2012-10-26 08:53:38 PDT
Comment on attachment 170767 [details]
Patch

Carry mitz's r+ forward.
Comment 14 WebKit Review Bot 2012-10-26 09:06:32 PDT
Comment on attachment 170767 [details]
Patch

Rejecting attachment 170767 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
yer-with-layer-children-expected.txt
patching file LayoutTests/platform/mac/compositing/tiling/huge-layer-with-layer-children-resize-expected.txt
patching file LayoutTests/platform/qt-5.0-wk2/TestExpectations
Hunk #2 FAILED at 360.
1 out of 2 hunks FAILED -- saving rejects to file LayoutTests/platform/qt-5.0-wk2/TestExpectations.rej

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Simon Fras..." exit_code: 1 cwd: /mnt/git/webkit-commit-queue

Full output: http://queues.webkit.org/results/14614058
Comment 15 Simon Fraser (smfr) 2012-10-26 10:31:07 PDT
http://trac.webkit.org/changeset/132673