Title: [182936] trunk/Source/WebKit2
Revision
182936
Author
commit-qu...@webkit.org
Date
2015-04-16 18:25:08 -0700 (Thu, 16 Apr 2015)

Log Message

Use UNUSED_PARAM instead of the void casting to suppress unused parameter warnings.
https://bugs.webkit.org/show_bug.cgi?id=143750

Patch by Sungmann Cho <sungmann....@navercorp.com> on 2015-04-16
Reviewed by Darin Adler.

No new tests, no behavior change.

* WebProcess/Notifications/NotificationPermissionRequestManager.cpp:
(WebKit::NotificationPermissionRequestManager::NotificationPermissionRequestManager):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (182935 => 182936)


--- trunk/Source/WebKit2/ChangeLog	2015-04-17 01:12:51 UTC (rev 182935)
+++ trunk/Source/WebKit2/ChangeLog	2015-04-17 01:25:08 UTC (rev 182936)
@@ -1,3 +1,15 @@
+2015-04-16  Sungmann Cho  <sungmann....@navercorp.com>
+
+        Use UNUSED_PARAM instead of the void casting to suppress unused parameter warnings.
+        https://bugs.webkit.org/show_bug.cgi?id=143750
+
+        Reviewed by Darin Adler.
+
+        No new tests, no behavior change.
+
+        * WebProcess/Notifications/NotificationPermissionRequestManager.cpp:
+        (WebKit::NotificationPermissionRequestManager::NotificationPermissionRequestManager):
+
 2015-04-16  Brady Eidson  <beid...@apple.com>
 
         Compiling a content extension fails when user's home directory is on a different volume from /var/tmp.

Modified: trunk/Source/WebKit2/WebProcess/Notifications/NotificationPermissionRequestManager.cpp (182935 => 182936)


--- trunk/Source/WebKit2/WebProcess/Notifications/NotificationPermissionRequestManager.cpp	2015-04-17 01:12:51 UTC (rev 182935)
+++ trunk/Source/WebKit2/WebProcess/Notifications/NotificationPermissionRequestManager.cpp	2015-04-17 01:25:08 UTC (rev 182936)
@@ -57,13 +57,16 @@
     return adoptRef(new NotificationPermissionRequestManager(page));
 }
 
+#if ENABLE(NOTIFICATIONS) || ENABLE(LEGACY_NOTIFICATIONS)
 NotificationPermissionRequestManager::NotificationPermissionRequestManager(WebPage* page)
-#if ENABLE(NOTIFICATIONS) || ENABLE(LEGACY_NOTIFICATIONS)
     : m_page(page)
-#endif
 {
-    (void)page;
 }
+#else
+NotificationPermissionRequestManager::NotificationPermissionRequestManager(WebPage*)
+{
+}
+#endif
 
 #if ENABLE(NOTIFICATIONS)
 void NotificationPermissionRequestManager::startRequest(SecurityOrigin* origin, PassRefPtr<NotificationPermissionCallback> callback)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to