[haiku-webkit-commits] r247 - webkit/trunk/WebKit/haiku/WebCoreSupport

  • From: webkit@xxxxxxxxxxxxxxx
  • To: haiku-webkit-commits@xxxxxxxxxxxxx
  • Date: Sun, 28 Feb 2010 15:21:43 +0000

Author: stippi
Date: Sun Feb 28 15:21:43 2010
New Revision: 247
URL: http://mmlr.dyndns.org/changeset/247

Log:
Follow the Gtk port and ignore null resource requests. If we don't we will
crash later on, as the Qt port currently does.

Modified:
   webkit/trunk/WebKit/haiku/WebCoreSupport/FrameLoaderClientHaiku.cpp

Modified: webkit/trunk/WebKit/haiku/WebCoreSupport/FrameLoaderClientHaiku.cpp
==============================================================================
--- webkit/trunk/WebKit/haiku/WebCoreSupport/FrameLoaderClientHaiku.cpp Sun Feb 
28 15:20:49 2010        (r246)
+++ webkit/trunk/WebKit/haiku/WebCoreSupport/FrameLoaderClientHaiku.cpp Sun Feb 
28 15:21:43 2010        (r247)
@@ -426,6 +426,13 @@
     if (!function)
         return;
 
+       // Ignore null requests as the Gtk port does. The Qt port doesn't and I 
have
+       // one URL where Arora crashes the same as WebPositive if not checked 
here.
+    if (request.isNull()) {
+        callPolicyFunction(function, PolicyIgnore);
+        return;
+    }
+
     if (!m_messenger.IsValid() || !isTertiaryMouseButton(action)) {
         callPolicyFunction(function, PolicyUse);
         return;

Other related posts:

  • » [haiku-webkit-commits] r247 - webkit/trunk/WebKit/haiku/WebCoreSupport - webkit