Bug 103431 - [V8] Crash wrapping media element when MediaPlayer isn't available
Summary: [V8] Crash wrapping media element when MediaPlayer isn't available
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-27 09:37 PST by Marshall Greenblatt
Modified: 2012-11-28 03:34 PST (History)
6 users (show)

See Also:

Attachments
Patch (5.15 KB, patch)
2012-11-28 01:45 PST, Dan Carney 		no flags Details | Formatted Diff | Diff
Patch (5.34 KB, patch)
2012-11-28 02:01 PST, Dan Carney 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Marshall Greenblatt 2012-11-27 09:37:31 PST 
1. Ensure a browser using v8 doesn't have access to a media player
2. Visit a page with a <video> tag
3. Infinite recursion

This seems to be a reappearance of https://bugs.webkit.org/show_bug.cgi?id=39907, likely caused by https://bugs.webkit.org/show_bug.cgi?id=102082 ([V8] More efficient wrapper dispatch).
Comment 1 Adam Barth 2012-11-27 09:56:37 PST 
Dan, would you be willing to take a look at this bug?
Comment 2 Dan Carney 2012-11-28 00:13:25 PST 
(In reply to comment #1)
> Dan, would you be willing to take a look at this bug?

On it
Comment 3 Dan Carney 2012-11-28 01:45:48 PST 
Created attachment 176429 [details]
Patch
Comment 4 Dan Carney 2012-11-28 02:01:13 PST 
Created attachment 176430 [details]
Patch
Comment 5 Kentaro Hara 2012-11-28 02:45:53 PST 
Comment on attachment 176430 [details]
Patch

Looks reasonable.
Comment 6 WebKit Review Bot 2012-11-28 03:34:16 PST 
Comment on attachment 176430 [details]
Patch

Clearing flags on attachment: 176430

Committed r135993: <http://trac.webkit.org/changeset/135993>
Comment 7 WebKit Review Bot 2012-11-28 03:34:20 PST 
All reviewed patches have been landed.  Closing bug.