Bug 58085

Summary: Logic error in HTMLMediaElement::exitFullscreen()
Product: WebKit Reporter: Jer Noble <jer.noble>
Component: WebCore Misc.Assignee: Jer Noble <jer.noble>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, webkit.review.bot
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Patch
none
Patch eric.carlson: review+

Jer Noble
Reported 2011-04-07 14:55:27 PDT
If a script calls exitFullscreen() on an element which is not the current full screen element, it will hit the ASSERT in the legacy full screen media case.
Attachments
Patch (2.20 KB, patch)
2011-04-07 15:03 PDT, Jer Noble 		no flags
DetailsFormatted DiffDiff
Patch (2.19 KB, patch)
2011-04-07 16:44 PDT, Jer Noble 		eric.carlson: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Jer Noble
Comment 1 2011-04-07 14:58:45 PDT
<rdar://problem/9252557>
Jer Noble
Comment 2 2011-04-07 15:03:31 PDT
Created attachment 88710 [details] Patch
WebKit Review Bot
Comment 3 2011-04-07 16:09:39 PDT
Attachment 88710 [details] did not build on chromium: Build output: http://queues.webkit.org/results/8342680
Jer Noble
Comment 4 2011-04-07 16:44:29 PDT
Created attachment 88733 [details] Patch Missing parenthesis reinserted.
Jer Noble
Comment 5 2011-04-08 13:12:16 PDT
Committed r83332: <http://trac.webkit.org/changeset/83332>
Note You need to log in before you can comment on or make changes to this bug.