Bug 90735

Summary: [EFL] REGRESSION (r122035): fullscreen/exit-full-screen-iframe.html is crashing
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit EFLAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: gyuyoung.kim, joone, kenneth, lucas.de.marchi, mrobinson, pnormand, rakuco, tonikitoo, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description Chris Dumez 2012-07-08 01:16:35 PDT 
The crash looks like:

crash log for DumpRenderTree (pid 27037):
STDOUT: <empty>
STDERR: ASSERTION FAILED: this
STDERR: /home/buildslave-1/webkit-buildslave/efl-linux-64-debug/build/Source/WebCore/dom/Node.h(412) : WebCore::Document* WebCore::Node::document() const
STDERR: 1   0x7fca92305f4b
STDERR: 2   0x7fca92305252
STDERR: 3   0x7fca95268636 WebCore::Document::webkitExitFullscreen()
STDERR: 4   0x7fca95268321 WebCore::Document::webkitCancelFullScreen()
STDERR: 5   0x7fca95e0f0bc WebCore::jsDocumentPrototypeFunctionWebkitCancelFullScreen(JSC::ExecState*)
STDERR: 6   0x7fca47033265

We can inspire from http://trac.webkit.org/changeset/122035/trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.cpp to fix the crash.
Comment 1 Chris Dumez 2012-07-08 02:06:33 PDT 
Created attachment 151161 [details]
Patch
Comment 2 Chris Dumez 2012-07-08 02:10:25 PDT 
Created attachment 151162 [details]
Patch

Add [EFL] tag.
Comment 3 WebKit Review Bot 2012-07-08 20:51:43 PDT 
Comment on attachment 151162 [details]
Patch

Clearing flags on attachment: 151162

Committed r122078: <http://trac.webkit.org/changeset/122078>
Comment 4 WebKit Review Bot 2012-07-08 20:51:48 PDT 
All reviewed patches have been landed.  Closing bug.