Bug 29037

Summary:

media/controls-after-reload.html crashes occasionally

Product:

WebKit

Reporter:

Eric Seidel (no email) <eric>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, eric.carlson, simon.fraser

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Bug Depends on:

28845

Bug Blocks:

Attachments:

Description	Flags
Stack trace from 9/3/09, while trying to land bug 28406	none
Seen again on 9/08/09 with different stack trace while trying to land bug 28934.	none
Hang report from local run-webkit-tests controls-after-reload.html run 9/22/09	none
Patch v1	none
crashlog for posterity	none

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	Details Formatted Diff Diff
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.