Title: [116179] trunk/Source/WebKit/mac
Revision
116179
Author
[email protected]
Date
2012-05-04 15:08:30 -0700 (Fri, 04 May 2012)

Log Message

REGRESSION (r116127-r116165): 6 tests failing on Lion Debug (Tests)
https://bugs.webkit.org/show_bug.cgi?id=85674

Reviewed by Oliver Hunt.

Null check the results of core([frame DOMDocument]).

* WebView/WebView.mm:
(-[WebView _didStartProvisionalLoadForFrame:]):

Modified Paths

Diff

Modified: trunk/Source/WebKit/mac/ChangeLog (116178 => 116179)


--- trunk/Source/WebKit/mac/ChangeLog	2012-05-04 22:01:07 UTC (rev 116178)
+++ trunk/Source/WebKit/mac/ChangeLog	2012-05-04 22:08:30 UTC (rev 116179)
@@ -1,3 +1,15 @@
+2012-05-04  Jer Noble  <[email protected]>
+
+        REGRESSION (r116127-r116165): 6 tests failing on Lion Debug (Tests)
+        https://bugs.webkit.org/show_bug.cgi?id=85674
+
+        Reviewed by Oliver Hunt.
+
+        Null check the results of core([frame DOMDocument]).
+
+        * WebView/WebView.mm:
+        (-[WebView _didStartProvisionalLoadForFrame:]):
+
 2012-05-02  Jer Noble  <[email protected]>
 
         Taking a visibility:hidden element full screen causes full screen window to disappear.

Modified: trunk/Source/WebKit/mac/WebView/WebView.mm (116178 => 116179)


--- trunk/Source/WebKit/mac/WebView/WebView.mm	2012-05-04 22:01:07 UTC (rev 116178)
+++ trunk/Source/WebKit/mac/WebView/WebView.mm	2012-05-04 22:08:30 UTC (rev 116179)
@@ -1853,7 +1853,8 @@
     [NSApp setWindowsNeedUpdate:YES];
 
 #if ENABLE(FULLSCREEN_API)
-    if (Element* element = core([frame DOMDocument])->webkitCurrentFullScreenElement()) {
+    Document* document = core([frame DOMDocument]);
+    if (Element* element = document ? document->webkitCurrentFullScreenElement() : 0) {
         SEL selector = @selector(webView:closeFullScreenWithListener:);
         if (_private->UIDelegate && [_private->UIDelegate respondsToSelector:selector]) {
             WebKitFullScreenListener *listener = [[WebKitFullScreenListener alloc] initWithElement:element];
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to