Title: [201233] trunk/Source/WebCore
Revision
201233
Author
jer.no...@apple.com
Date
2016-05-20 16:46:06 -0700 (Fri, 20 May 2016)

Log Message

Inconsistent state in playback controls
https://bugs.webkit.org/show_bug.cgi?id=157962
<rdar://problem/26397571>

Reviewed by Beth Dakin.

Do not use the playbackSessionManager() as the model, that's what the model is for.

* platform/mac/WebPlaybackSessionInterfaceMac.mm:
(WebCore::WebPlaybackSessionInterfaceMac::setClient):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (201232 => 201233)


--- trunk/Source/WebCore/ChangeLog	2016-05-20 23:15:15 UTC (rev 201232)
+++ trunk/Source/WebCore/ChangeLog	2016-05-20 23:46:06 UTC (rev 201233)
@@ -1,3 +1,16 @@
+2016-05-20  Jer Noble  <jer.no...@apple.com>
+
+        Inconsistent state in playback controls
+        https://bugs.webkit.org/show_bug.cgi?id=157962
+        <rdar://problem/26397571>
+
+        Reviewed by Beth Dakin.
+
+        Do not use the playbackSessionManager() as the model, that's what the model is for.
+
+        * platform/mac/WebPlaybackSessionInterfaceMac.mm:
+        (WebCore::WebPlaybackSessionInterfaceMac::setClient):
+
 2016-05-20  Brady Eidson  <beid...@apple.com>
 
         Modern IDB: Properly handle blobs in Workers.

Modified: trunk/Source/WebCore/platform/mac/WebPlaybackSessionInterfaceMac.mm (201232 => 201233)


--- trunk/Source/WebCore/platform/mac/WebPlaybackSessionInterfaceMac.mm	2016-05-20 23:15:15 UTC (rev 201232)
+++ trunk/Source/WebCore/platform/mac/WebPlaybackSessionInterfaceMac.mm	2016-05-20 23:46:06 UTC (rev 201233)
@@ -58,10 +58,8 @@
 {
     m_client = client;
 
-    if (m_client) {
-        float rate = [playBackControlsManager() rate];
-        m_client->rateChanged(!!rate, rate);
-    }
+    if (m_client && m_playbackSessionModel)
+        m_client->rateChanged(m_playbackSessionModel->isPlaying(), m_playbackSessionModel->playbackRate());
 }
 
 void WebPlaybackSessionInterfaceMac::setDuration(double duration)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to