WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
54994
media test failures on SNOWLEOPARD (needs rebaseline)
https://bugs.webkit.org/show_bug.cgi?id=54994
Summary
media test failures on SNOWLEOPARD (needs rebaseline)
Anna Cavender
Reported
2011-02-22 13:51:19 PST
The following tests need rebaseline under Snowleopard: BUGWK54322 MAC SNOWLEOPARD : fast/media/mq-transform-02.html = IMAGE+TEXT BUGWK54322 MAC SNOWLEOPARD : fast/media/mq-transform-03.html = IMAGE+TEXT BUGWK54322 MAC SNOWLEOPARD : media/audio-controls-rendering.html = TEXT BUGWK54322 MAC SNOWLEOPARD : media/video-aspect-ratio.html = IMAGE BUGWK54322 MAC SNOWLEOPARD : media/video-empty-source.html = TEXT BUGWK54322 MAC SNOWLEOPARD : media/video-no-audio.html = TEXT BUGWK54322 MAC SNOWLEOPARD : media/video-volume-slider.html = TEXT BUGWK54322 MAC SNOWLEOPARD : media/video-zoom.html = IMAGE
Attachments
Patch
(320.34 KB, patch)
2011-02-22 13:58 PST
,
Anna Cavender
no flags
Details
Formatted Diff
Diff
Patch
(320.36 KB, patch)
2011-02-23 07:01 PST
,
Anna Cavender
no flags
Details
Formatted Diff
Diff
Patch
(319.68 KB, patch)
2011-02-23 08:22 PST
,
Anna Cavender
no flags
Details
Formatted Diff
Diff
Patch
(1.22 MB, patch)
2011-02-23 15:36 PST
,
Anna Cavender
no flags
Details
Formatted Diff
Diff
Show Obsolete
(3)
View All
Add attachment
proposed patch, testcase, etc.
Anna Cavender
Comment 1
2011-02-22 13:58:54 PST
Created
attachment 83380
[details]
Patch
Anna Cavender
Comment 2
2011-02-23 07:01:23 PST
Created
attachment 83481
[details]
Patch
Anna Cavender
Comment 3
2011-02-23 07:02:10 PST
old patch wouldn't apply. trying with updated files.
WebKit Commit Bot
Comment 4
2011-02-23 08:22:10 PST
Comment on
attachment 83481
[details]
Patch Rejecting
attachment 83481
[details]
from commit-queue. Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-7', 'build-..." exit_code: 2 Last 500 characters of output: ................ fast/loader ..................................................................................................... fast/loader/stateobjects ...................... fast/loading . fast/media ................................................................... fast/media/mq-transform-02.html -> failed Exiting early after 1 failures. 9901 tests run. 240.95s total testing time 9900 test cases (99%) succeeded 1 test case (<1%) had incorrect layout 6 test cases (<1%) had stderr output Full output:
http://queues.webkit.org/results/7980153
Anna Cavender
Comment 5
2011-02-23 08:22:47 PST
Created
attachment 83484
[details]
Patch
Anna Cavender
Comment 6
2011-02-23 08:23:37 PST
I'm not sure why the fast/media tests aren't passing, so I added those lines back into test_expections.txt to see what happens.
Simon Fraser (smfr)
Comment 7
2011-02-23 10:03:28 PST
Chromium doesn't compile with accelerated compositing enabled, so its results for mq-transform-02.html and mq-transform-03.html will differ from Safari on Mac.
Anna Cavender
Comment 8
2011-02-23 15:36:29 PST
Created
attachment 83564
[details]
Patch
Anna Cavender
Comment 9
2011-02-23 15:42:04 PST
Quite a few more media/ tests needing rebaselines from when I first posted this bug, so I added those. I put new baselines in chromium-mac/ and moved any existing Leopard ones to chromium-mac-leopard/. Also, please note, fixes for the two fast/media tests: BUGWK54322 MAC SNOWLEOPARD : fast/media/mq-transform-02.html = IMAGE+TEXT BUGWK54322 MAC SNOWLEOPARD : fast/media/mq-transform-03.html = IMAGE+TEXT are not included as they are actually for CSS media type selector tests and were mistakenly included with media/ bugs. Thanks!
Eric Seidel (no email)
Comment 10
2011-02-23 19:56:55 PST
The commit-queue can't land when the SL tests fail. So it's unlikely that it will be able to land this. There is a chance it will pick it up if it's early enough in the queue and successfully land it, but I wouldn't count on it.
Eric Seidel (no email)
Comment 11
2011-02-23 19:57:13 PST
Oh, this is chromium-only. Then the cq can land it just fine. SInce it uses the Apple Mac build.
Anna Cavender
Comment 12
2011-02-24 08:28:37 PST
Eric, can you see a reason why this patch seems stuck (not landing)? I'm not getting any errors, but it's not getting picked up either ...
Eric Seidel (no email)
Comment 13
2011-02-24 10:43:30 PST
The SL bot was red all day yesterday. The 6 commit-queue instances just can't land anything while the SL tests are red. They're landing since 2am last night (when Ryosuke finally made the tree green).
Eric Seidel (no email)
Comment 14
2011-02-24 10:44:32 PST
(I should clarify: ryosuke was cleaning up someone elses mess, and the tree went red shortly after it went green which is why they didn't all land over night. it's back green now and things are landing again.)
Eric Seidel (no email)
Comment 15
2011-02-24 10:50:32 PST
The queue lands upwards of 12 patches an hour when the tree is green. :) Eventually we'll teach the commit-queue how to land when the tree is red, but for now it's very conservative and will wait until it can see a 100% pass on its machine before landing.
Anna Cavender
Comment 16
2011-02-24 11:02:52 PST
OK, thanks. I see it going through the process now :)
WebKit Commit Bot
Comment 17
2011-02-24 12:46:02 PST
Comment on
attachment 83564
[details]
Patch Clearing flags on attachment: 83564 Committed
r79603
: <
http://trac.webkit.org/changeset/79603
>
WebKit Commit Bot
Comment 18
2011-02-24 12:46:12 PST
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug