Changeset 60981 in webkit

Timestamp:

Jun 10, 2010 4:05:26 PM (14 years ago)

Author:

yael.aharon@nokia.com

Message:

[Qt] Support for loading notification icons
​https://bugs.webkit.org/show_bug.cgi?id=40004

Reviewed by Kenneth Rohde Christiansen.

Take into use the icon that was loaded for the notification.

• WebCoreSupport/NotificationPresenterClientQt.cpp:

(WebCore::NotificationPresenterClientQt::show):

Location:

trunk/WebKit/qt

Files:

• ChangeLog (modified) (1 diff)
• WebCoreSupport/NotificationPresenterClientQt.cpp (modified) (1 diff)

trunk/WebKit/qt/ChangeLog

@@ +1 @@
+2010-06-10  Yael Aharon  <yael.aharon@nokia.com>
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        [Qt] Support for loading notification icons
+        https://bugs.webkit.org/show_bug.cgi?id=40004
+
+        Take into use the icon that was loaded for the notification.
+
+        * WebCoreSupport/NotificationPresenterClientQt.cpp:
+        (WebCore::NotificationPresenterClientQt::show):
+
 2010-06-10  Mahesh Kulkarni  <mahesh.kulkarni@nokia.com>
 

trunk/WebKit/qt/WebCoreSupport/NotificationPresenterClientQt.cpp

@@ -102 +102 @@
     if (dumpNotification)
         dumpShowText(notification);
-    displayNotification(notification, QByteArray());
+    QByteArray iconData;
+    if (notification->iconData())
+        iconData = QByteArray::fromRawData(notification->iconData()->data(), notification->iconData()->size());
+    displayNotification(notification, iconData);
+    notification->releaseIconData();
     return true;
 }