Title: [235197] trunk/Source/WebCore
Revision
235197
Author
eric.carl...@apple.com
Date
2018-08-22 13:40:22 -0700 (Wed, 22 Aug 2018)

Log Message

Log more often during AirPlay state changes
https://bugs.webkit.org/show_bug.cgi?id=188854

Always log some infrequent but important AirPlay state changes to make debugging easier.

Reviewed by Jer Noble.

* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::webkitShowPlaybackTargetPicker): Low always.
(WebCore::HTMLMediaElement::mediaPlayerCurrentPlaybackTargetIsWirelessChanged): Ditto.
(WebCore::HTMLMediaElement::setWirelessPlaybackTarget): Ditto.
(WebCore::HTMLMediaElement::setShouldPlayToPlaybackTarget): Ditto.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (235196 => 235197)


--- trunk/Source/WebCore/ChangeLog	2018-08-22 20:32:06 UTC (rev 235196)
+++ trunk/Source/WebCore/ChangeLog	2018-08-22 20:40:22 UTC (rev 235197)
@@ -1,3 +1,18 @@
+2018-08-22  Eric Carlson  <eric.carl...@apple.com>
+
+        Log more often during AirPlay state changes
+        https://bugs.webkit.org/show_bug.cgi?id=188854
+
+        Always log some infrequent but important AirPlay state changes to make debugging easier.
+
+        Reviewed by Jer Noble.
+
+        * html/HTMLMediaElement.cpp:
+        (WebCore::HTMLMediaElement::webkitShowPlaybackTargetPicker): Low always.
+        (WebCore::HTMLMediaElement::mediaPlayerCurrentPlaybackTargetIsWirelessChanged): Ditto.
+        (WebCore::HTMLMediaElement::setWirelessPlaybackTarget): Ditto.
+        (WebCore::HTMLMediaElement::setShouldPlayToPlaybackTarget): Ditto.
+
 2018-08-21  Ryosuke Niwa  <rn...@webkit.org>
 
         Focus navigation order in slot fallback contents is wrong

Modified: trunk/Source/WebCore/html/HTMLMediaElement.cpp (235196 => 235197)


--- trunk/Source/WebCore/html/HTMLMediaElement.cpp	2018-08-22 20:32:06 UTC (rev 235196)
+++ trunk/Source/WebCore/html/HTMLMediaElement.cpp	2018-08-22 20:40:22 UTC (rev 235197)
@@ -5738,7 +5738,7 @@
 #if ENABLE(WIRELESS_PLAYBACK_TARGET)
 void HTMLMediaElement::webkitShowPlaybackTargetPicker()
 {
-    INFO_LOG(LOGIDENTIFIER);
+    ALWAYS_LOG(LOGIDENTIFIER);
     if (processingUserGestureForMedia())
         removeBehaviorsRestrictionsAfterFirstUserGesture();
     m_mediaSession->showPlaybackTargetPicker();
@@ -5758,7 +5758,7 @@
 {
     m_isPlayingToWirelessTarget = m_player && m_player->isCurrentPlaybackTargetWireless();
 
-    INFO_LOG(LOGIDENTIFIER, "webkitCurrentPlaybackTargetIsWireless = ", m_isPlayingToWirelessTarget);
+    ALWAYS_LOG(LOGIDENTIFIER, m_isPlayingToWirelessTarget);
     ASSERT(m_player);
     configureMediaControls();
     scheduleEvent(eventNames().webkitcurrentplaybacktargetiswirelesschangedEvent);
@@ -5831,7 +5831,7 @@
 
 void HTMLMediaElement::setWirelessPlaybackTarget(Ref<MediaPlaybackTarget>&& device)
 {
-    INFO_LOG(LOGIDENTIFIER);
+    ALWAYS_LOG(LOGIDENTIFIER);
     if (m_player)
         m_player->setWirelessPlaybackTarget(WTFMove(device));
 }
@@ -5852,7 +5852,7 @@
 
 void HTMLMediaElement::setShouldPlayToPlaybackTarget(bool shouldPlay)
 {
-    INFO_LOG(LOGIDENTIFIER, "shouldPlay = ", shouldPlay);
+    ALWAYS_LOG(LOGIDENTIFIER, shouldPlay);
 
     if (m_player)
         m_player->setShouldPlayToPlaybackTarget(shouldPlay);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to