Title: [260849] trunk/Source/WebCore
Revision
260849
Author
[email protected]
Date
2020-04-28 14:59:23 -0700 (Tue, 28 Apr 2020)

Log Message

MRMediaRemoteSetNowPlayingApplicationPlaybackStateForOrigin log with no error
https://bugs.webkit.org/show_bug.cgi?id=211145

Reviewed by Jer Noble.

Don't log when MRMediaRemoteSetNowPlayingApplicationPlaybackStateForOrigin() returns 0.

* platform/audio/cocoa/MediaSessionManagerCocoa.mm:
(WebCore::MediaSessionManagerCocoa::setNowPlayingInfo):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (260848 => 260849)


--- trunk/Source/WebCore/ChangeLog	2020-04-28 21:51:37 UTC (rev 260848)
+++ trunk/Source/WebCore/ChangeLog	2020-04-28 21:59:23 UTC (rev 260849)
@@ -1,3 +1,15 @@
+2020-04-28  Simon Fraser  <[email protected]>
+
+        MRMediaRemoteSetNowPlayingApplicationPlaybackStateForOrigin log with no error
+        https://bugs.webkit.org/show_bug.cgi?id=211145
+
+        Reviewed by Jer Noble.
+
+        Don't log when MRMediaRemoteSetNowPlayingApplicationPlaybackStateForOrigin() returns 0.
+
+        * platform/audio/cocoa/MediaSessionManagerCocoa.mm:
+        (WebCore::MediaSessionManagerCocoa::setNowPlayingInfo):
+
 2020-04-28  Ross Kirsling  <[email protected]>
 
         [JSC] Align upon the name isCallable instead of isFunction

Modified: trunk/Source/WebCore/platform/audio/cocoa/MediaSessionManagerCocoa.mm (260848 => 260849)


--- trunk/Source/WebCore/platform/audio/cocoa/MediaSessionManagerCocoa.mm	2020-04-28 21:51:37 UTC (rev 260848)
+++ trunk/Source/WebCore/platform/audio/cocoa/MediaSessionManagerCocoa.mm	2020-04-28 21:59:23 UTC (rev 260849)
@@ -273,7 +273,8 @@
 #if LOG_DISABLED
         UNUSED_PARAM(error);
 #else
-        WTFLogAlways("MRMediaRemoteSetNowPlayingApplicationPlaybackStateForOrigin(playing) failed with error %d", error);
+        if (error)
+            WTFLogAlways("MRMediaRemoteSetNowPlayingApplicationPlaybackStateForOrigin(playing) failed with error %d", error);
 #endif
     });
     MRMediaRemoteSetNowPlayingInfo(info.get());
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to