RESOLVED FIXED 29037
media/controls-after-reload.html crashes occasionally 
https://bugs.webkit.org/show_bug.cgi?id=29037
Summary media/controls-after-reload.html crashes occasionally
Eric Seidel (no email)
Reported 2009-09-08 09:06:09 PDT
Created attachment 39192 [details] Stack trace from 9/3/09, while trying to land bug 28406 media/controls-after-reload.html crashes occasionally This is likely the same root cause as bug 28845, but since the stack trace looks different, I'm filing a separate bug.
Attachments
Stack trace from 9/3/09, while trying to land bug 28406 (56.55 KB, text/plain)
2009-09-08 09:06 PDT, Eric Seidel (no email) 		no flags
Details
Seen again on 9/08/09 with different stack trace while trying to land bug 28934. (46.32 KB, text/plain)
2009-09-08 10:02 PDT, Eric Seidel (no email) 		no flags
Details
Hang report from local run-webkit-tests controls-after-reload.html run 9/22/09 (17.96 KB, text/plain)
2009-09-22 14:54 PDT, Eric Seidel (no email) 		no flags
Details
Patch v1 (1.11 KB, patch)
2009-09-22 14:57 PDT, Eric Seidel (no email) 		no flags
DetailsFormatted DiffDiff
crashlog for posterity (64.01 KB, application/octet-stream)
2009-09-22 15:51 PDT, Eric Seidel (no email) 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1 2009-09-08 09:55:54 PDT
Hum... I wonder why bugzilla fails to detect crash-reports as text/plain. Maybe they have a BOM.
Eric Seidel (no email)
Comment 2 2009-09-08 10:02:08 PDT
Created attachment 39195 [details] Seen again on 9/08/09 with different stack trace while trying to land bug 28934.
Eric Seidel (no email)
Comment 3 2009-09-09 16:19:34 PDT
I'm not sure the second stack trace is related. But looking at these stack traces and hang reports from the other bugs, it looks like that the CVDisplayLink is deallocated already. Possibly something further up the stack as well.
Eric Carlson
Comment 4 2009-09-09 17:00:32 PDT
I don't know about the CVDisplayLink, but it is *definitely* not normal to see 15 threads blocked in JVTCompEncodeFrame.
Eric Seidel (no email)
Comment 5 2009-09-18 15:18:49 PDT
Looks like we're still seeing this one: http://build.webkit.org/results/Leopard%20Intel%20Release%20(Tests)/r48536%20(5140)/results.html So bug 28845's workaround was probably not enough. I don't have a stack trace from the buildbot sadly.
Eric Seidel (no email)
Comment 6 2009-09-21 18:52:02 PDT
This one is still crashing on the bots: http://build.webkit.org/results/Leopard%20Intel%20Release%20(Tests)/r48615%20(5210)/results.html
Eric Seidel (no email)
Comment 7 2009-09-22 14:26:59 PDT
WebKitTools/Scripts/run-webkit-tests --iterations 100 media/controls-after-reload.html Is 100% reproducible failure for me. If Simon/Eric thinks this is the same issue as was seen in bug 29037 then we should just skip this test on Leopard.
Simon Fraser (smfr)
Comment 8 2009-09-22 14:29:51 PDT
Got a crash log?
Eric Seidel (no email)
Comment 9 2009-09-22 14:48:47 PDT
(In reply to comment #8) > Got a crash log? The failures I'm seeing today are timeouts. I'll run it again and see if I can get you an up-to-date crash logs. There are some crash logs already attached to this bug too.
Eric Seidel (no email)
Comment 10 2009-09-22 14:54:25 PDT
Created attachment 39947 [details] Hang report from local run-webkit-tests controls-after-reload.html run 9/22/09
Simon Fraser (smfr)
Comment 11 2009-09-22 14:55:09 PDT
That's the same as the other CVDisplayLink issue. This bug should just be duped.
Eric Seidel (no email)
Comment 12 2009-09-22 14:57:40 PDT
Created attachment 39949 [details] Patch v1
Eric Seidel (no email)
Comment 13 2009-09-22 14:58:11 PDT
Ok, posted a patch to skip this test on Leopard for the same reason as the other one.
WebKit Commit Bot
Comment 14 2009-09-22 15:33:40 PDT
Comment on attachment 39949 [details] Patch v1 Clearing flags on attachment: 39949 Committed r48654: <http://trac.webkit.org/changeset/48654>
WebKit Commit Bot
Comment 15 2009-09-22 15:33:44 PDT
All reviewed patches have been landed. Closing bug.
Eric Seidel (no email)
Comment 16 2009-09-22 15:51:07 PDT
Entertaining that the bot went out with a bang: http://build.webkit.org/results/Leopard%20Intel%20Release%20(Tests)/r48653%20(5248)/results.html That's the last revision before the checkin to skip this test. :) I also have a crash log too which I'll upload for posterity.
Eric Seidel (no email)
Comment 17 2009-09-22 15:51:36 PDT
Created attachment 39955 [details] crashlog for posterity
Note You need to log in before you can comment on or make changes to this bug.