Cleanup the code added for https://bugs.webkit.org/show_bug.cgi?id=128125.
authoreric.carlson@apple.com <eric.carlson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Feb 2014 20:29:33 +0000 (20:29 +0000)
committereric.carlson@apple.com <eric.carlson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Feb 2014 20:29:33 +0000 (20:29 +0000)
Not reviewed.

* html/HTMLMediaSession.cpp:
(WebCore::restrictionName): Don't need a "break" after a "return" in a case statement.
* platform/audio/MediaSession.cpp:
(WebCore::stateName): Ditto.
* platform/audio/ios/MediaSessionManagerIOS.mm:

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

Source/WebCore/ChangeLog
Source/WebCore/html/HTMLMediaSession.cpp
Source/WebCore/platform/audio/MediaSession.cpp
Source/WebCore/platform/audio/ios/MediaSessionManagerIOS.mm

index 50c6c5a..7ccab23 100644 (file)
@@ -1,3 +1,15 @@
+2014-02-12  Eric Carlson  <eric.carlson@apple.com>
+
+        Cleanup the code added for https://bugs.webkit.org/show_bug.cgi?id=128125.
+
+        Not reviewed.
+
+        * html/HTMLMediaSession.cpp:
+        (WebCore::restrictionName): Don't need a "break" after a "return" in a case statement.
+        * platform/audio/MediaSession.cpp:
+        (WebCore::stateName): Ditto.
+        * platform/audio/ios/MediaSessionManagerIOS.mm:
+
 2014-02-12  Andreas Kling  <akling@apple.com>
 
         RenderNamedFlowThread should only support RenderElement children.
index 07b01fa..5cca4f2 100644 (file)
@@ -45,7 +45,7 @@ namespace WebCore {
 #if !LOG_DISABLED
 static const char* restrictionName(HTMLMediaSession::BehaviorRestrictions restriction)
 {
-#define CASE(_restriction) case HTMLMediaSession::_restriction: return #_restriction; break;
+#define CASE(restriction) case HTMLMediaSession::restriction: return #restriction
     switch (restriction) {
     CASE(NoRestrictions);
     CASE(RequireUserGestureForLoad);
index 590b533..b2b2593 100644 (file)
@@ -35,7 +35,7 @@ namespace WebCore {
 #if !LOG_DISABLED
 static const char* stateName(MediaSession::State state)
 {
-#define CASE(_state) case MediaSession::_state: return #_state; break;
+#define CASE(state) case MediaSession::state: return #state
     switch (state) {
     CASE(Idle);
     CASE(Playing);
index 681c714..a403dd1 100644 (file)
@@ -64,9 +64,9 @@ using namespace WebCore;
 
 - (id)initWithCallback:(MediaSessionManageriOS*)callback;
 - (void)clearCallback;
-- (void)interruption:(NSNotification*)notification;
-- (void)applicationWillEnterForeground:(NSNotification*)notification;
-- (void)applicationWillResignActive:(NSNotification*)notification;
+- (void)interruption:(NSNotification *)notification;
+- (void)applicationWillEnterForeground:(NSNotification *)notification;
+- (void)applicationWillResignActive:(NSNotification *)notification;
 @end
 
 namespace WebCore {