Holger Hans Peter Freyther <holger.freyther@trolltech.com>
authorhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 18 Jan 2008 10:37:33 +0000 (10:37 +0000)
committerhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 18 Jan 2008 10:37:33 +0000 (10:37 +0000)
* Ask the WebCore::IconDatabase only if our URL is not empty. Otherwise
  we will see a crash in a HashSet.
* It is crashing there because the StringImpl of an empty String is 0.
* We avoid this crash by checking for isEmpty() in WebKit as there is no
  use to ask the iconDatabase for an empty string. We will fallback to the
  defaultIcon.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29614 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/qt/Api/qwebframe.cpp
WebKit/qt/ChangeLog

index 67771fd..8882578 100644 (file)
@@ -239,7 +239,12 @@ QUrl QWebFrame::url() const
 QPixmap QWebFrame::icon() const
 {
     String url = d->frame->loader()->url().string();
-    Image* image = iconDatabase()->iconForPageURL(url, IntSize(16, 16));
+
+    Image* image = 0;
+    if (!url.isEmpty()) {
+        image = iconDatabase()->iconForPageURL(url, IntSize(16, 16));
+    }
+
     if (!image || image->isNull()) {
         image = iconDatabase()->defaultIcon(IntSize(16, 16));
     }
index 8a305e6..181ab3e 100644 (file)
@@ -1,3 +1,18 @@
+2008-01-18  Holger Hans Peter Freyther  <holger.freyther@trolltech.com>
+
+        Reviewed by Simon.
+
+        * Ask the WebCore::IconDatabase only if our URL is not empty. Otherwise
+        we will see a crash in a HashSet.
+        * It is crashing there because the StringImpl of an empty String is 0.
+        * We avoid this crash by checking for isEmpty() in WebKit as there is no
+        use to ask the iconDatabase for an empty string. We will fallback to the
+        defaultIcon.
+        
+
+        * Api/qwebframe.cpp:
+        (QWebFrame::icon):
+
 2008-01-17  Simon Hausmann  <hausmann@webkit.org>
 
         Reviewed by Holger.