Deprecate ondisplay()
authorjonlee@apple.com <jonlee@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Mar 2012 22:44:02 +0000 (22:44 +0000)
committerjonlee@apple.com <jonlee@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Mar 2012 22:44:02 +0000 (22:44 +0000)
https://bugs.webkit.org/show_bug.cgi?id=80483
<rdar://problem/10912432>

Reviewed by Jian Li.

Source/WebCore:

The appropriate event listener for notifications is onshow().

* notifications/Notification.h: Wrap ondisplay in ENABLE(LEGACY_NOTIFICATIONS).
* notifications/Notification.idl: Wrap ondisplay in ENABLE_LEGACY_NOTIFICATIONS.

Source/WebKit/chromium:

* src/WebNotification.cpp:
(WebKit::WebNotification::dispatchDisplayEvent): Dispatch the show event.

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

Source/WebCore/ChangeLog
Source/WebCore/notifications/Notification.h
Source/WebCore/notifications/Notification.idl
Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/src/WebNotification.cpp

index 2fc63d7..c8f51da 100644 (file)
@@ -1,3 +1,16 @@
+2012-03-13  Jon Lee  <jonlee@apple.com>
+
+        Deprecate ondisplay()
+        https://bugs.webkit.org/show_bug.cgi?id=80483
+        <rdar://problem/10912432>
+
+        Reviewed by Jian Li.
+
+        The appropriate event listener for notifications is onshow().
+
+        * notifications/Notification.h: Wrap ondisplay in ENABLE(LEGACY_NOTIFICATIONS).
+        * notifications/Notification.idl: Wrap ondisplay in ENABLE_LEGACY_NOTIFICATIONS.
+
 2012-03-14  W. James MacLean  <wjmaclean@chromium.org>
 
         [chromium] Wire up PlatformGestureCurves for CC, ScrollAnimatorNone.
index b8f337c..cbd819b 100644 (file)
@@ -89,9 +89,9 @@ public:
     TextDirection direction() const { return dir() == "rtl" ? RTL : LTR; }
 
     DEFINE_ATTRIBUTE_EVENT_LISTENER(show);
-    // FIXME: The latest Web Notifications standard uses the onshow event listener.
-    // The ondisplay event listener should be removed when implementations change the event listener to onshow.
+#if ENABLE(LEGACY_NOTIFICATIONS)
     DEFINE_ATTRIBUTE_EVENT_LISTENER(display);
+#endif
     DEFINE_ATTRIBUTE_EVENT_LISTENER(error);
     DEFINE_ATTRIBUTE_EVENT_LISTENER(close);
     DEFINE_ATTRIBUTE_EVENT_LISTENER(click);
index e6dd25a..3156a8d 100644 (file)
@@ -41,9 +41,9 @@ module threads {
         void cancel();
 
         attribute EventListener onshow;
-        // FIXME: The latest Web Notifications standard uses the onshow event listener.
-        // The ondisplay event listener should be removed when implementations change the event listener to onshow.
+#if defined(ENABLE_LEGACY_NOTIFICATIONS) && ENABLE_LEGACY_NOTIFICATIONS
         attribute EventListener ondisplay;
+#endif
         attribute EventListener onerror;
         attribute EventListener onclose;
         attribute EventListener onclick;
index af710e4..2d9a3de 100644 (file)
@@ -1,3 +1,14 @@
+2012-03-13  Jon Lee  <jonlee@apple.com>
+
+        Deprecate ondisplay()
+        https://bugs.webkit.org/show_bug.cgi?id=80483
+        <rdar://problem/10912432>
+
+        Reviewed by Jian Li.
+
+        * src/WebNotification.cpp:
+        (WebKit::WebNotification::dispatchDisplayEvent): Dispatch the show event.
+
 2012-03-14  W. James MacLean  <wjmaclean@chromium.org>
 
         [chromium] Wire up PlatformGestureCurves for CC, ScrollAnimatorNone.
index 7a63b59..747376d 100644 (file)
@@ -116,7 +116,10 @@ void WebNotification::detachPresenter()
 
 void WebNotification::dispatchDisplayEvent()
 {
+#if ENABLE(LEGACY_NOTIFICATIONS)
     dispatchEvent("display");
+#endif
+    dispatchEvent("show");
 }
 
 void WebNotification::dispatchErrorEvent(const WebKit::WebString& /* errorMessage */)