Media Session: MediaSession.releaseSession() renamed to deactivate()
authoreric.carlson@apple.com <eric.carlson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 28 Aug 2015 18:21:21 +0000 (18:21 +0000)
committereric.carlson@apple.com <eric.carlson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 28 Aug 2015 18:21:21 +0000 (18:21 +0000)
https://bugs.webkit.org/show_bug.cgi?id=148578

Reviewed by Jer Noble.

Source/WebCore:

No new tests, media/session/releasing-sessions.html was updated for the change.

* Modules/mediasession/MediaSession.cpp:
(WebCore::MediaSession::deactivate): Renamed from releaseSession.
(WebCore::MediaSession::releaseSession): Deleted.
* Modules/mediasession/MediaSession.h:
* Modules/mediasession/MediaSession.idl:

* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::setSession): releaseSession -> deactivate.

LayoutTests:

* media/session/releasing-sessions-expected.txt:
* media/session/releasing-sessions.html:

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

LayoutTests/ChangeLog
LayoutTests/media/session/releasing-sessions-expected.txt
LayoutTests/media/session/releasing-sessions.html
Source/WebCore/ChangeLog
Source/WebCore/Modules/mediasession/MediaSession.cpp
Source/WebCore/Modules/mediasession/MediaSession.h
Source/WebCore/Modules/mediasession/MediaSession.idl
Source/WebCore/html/HTMLMediaElement.cpp

index 608c63e..1eeb2b7 100644 (file)
@@ -1,3 +1,13 @@
+2015-08-28  Eric Carlson  <eric.carlson@apple.com>
+
+        Media Session: MediaSession.releaseSession() renamed to deactivate()
+        https://bugs.webkit.org/show_bug.cgi?id=148578
+
+        Reviewed by Jer Noble.
+
+        * media/session/releasing-sessions-expected.txt:
+        * media/session/releasing-sessions.html:
+
 2015-08-28  Joseph Pecoraro  <pecoraro@apple.com>
 
         Web Inspector: Separate creating a style sheet from adding a new rule in the protocol
index d3838d4..48be29d 100644 (file)
@@ -8,7 +8,7 @@ EVENT(playing)
 Media began playing.
 EXPECTED (video.paused == 'false') OK
 EXPECTED (internals.mediaSessionCurrentState(session) == 'active') OK
-RUN(session.releaseSession())
+RUN(session.deactivate())
 EXPECTED (video.paused == 'true') OK
 EXPECTED (internals.mediaSessionCurrentState(session) == 'idle') OK
 END OF TEST
index a91086a..282530c 100644 (file)
@@ -33,7 +33,7 @@
             if (window.internals)
                 testExpected('internals.mediaSessionCurrentState(session)', "active");
 
-            run('session.releaseSession()');
+            run('session.deactivate()');
 
             testExpected('video.paused', true);
 
index 1f633fd..4f12b23 100644 (file)
@@ -1,3 +1,21 @@
+2015-08-28  Eric Carlson  <eric.carlson@apple.com>
+
+        Media Session: MediaSession.releaseSession() renamed to deactivate()
+        https://bugs.webkit.org/show_bug.cgi?id=148578
+
+        Reviewed by Jer Noble.
+
+        No new tests, media/session/releasing-sessions.html was updated for the change.
+
+        * Modules/mediasession/MediaSession.cpp:
+        (WebCore::MediaSession::deactivate): Renamed from releaseSession.
+        (WebCore::MediaSession::releaseSession): Deleted.
+        * Modules/mediasession/MediaSession.h:
+        * Modules/mediasession/MediaSession.idl:
+
+        * html/HTMLMediaElement.cpp:
+        (WebCore::HTMLMediaElement::setSession): releaseSession -> deactivate.
+
 2015-08-28  Joseph Pecoraro  <pecoraro@apple.com>
 
         Web Inspector: Separate creating a style sheet from adding a new rule in the protocol
index 26c60c8..03ca7c5 100644 (file)
@@ -185,18 +185,20 @@ void MediaSession::setMetadata(const Dictionary& metadata)
         page->chrome().client().mediaSessionMetadataDidChange(m_metadata);
 }
 
-void MediaSession::releaseSession()
+void MediaSession::deactivate()
 {
-    // 5.1.3
+    // 5.1.2. Object members
+    // When the deactivate() method is invoked, the user agent must run the following steps:
     // 1. Let media session be the current media session.
-    // 2. Indefinitely pause all of media session's active participating media elements.
-    // 3. Reset media session's active participating media elements to an empty list.
+    // 2. Indefinitely pause all of media session’s audio-producing participants.
+    // 3. Set media session's resume list to an empty list.
+    // 4. Set media session's audio-producing participants to an empty list.
     changeActiveMediaElements([&]() {
         while (!m_activeParticipatingElements.isEmpty())
             m_activeParticipatingElements.takeAny()->pause();
     });
 
-    // 4. Run the media session release algorithm for media session.
+    // 5. Run the media session deactivation algorithm for media session.
     releaseInternal();
 }
 
index df44e50..718c688 100644 (file)
@@ -69,7 +69,7 @@ public:
 
     void setMetadata(const Dictionary&);
 
-    void releaseSession();
+    void deactivate();
 
     // Runs the media session invocation algorithm and returns true on success.
     bool invoke();
index 4d42d56..d7ac061 100644 (file)
@@ -34,7 +34,7 @@
 
     void setMetadata(Dictionary metadata);
 
-    void releaseSession();
+    void deactivate();
 };
 
 enum MediaSessionKind {
index e5d9ee5..6dca71d 100644 (file)
@@ -6569,7 +6569,7 @@ void HTMLMediaElement::setSession(MediaSession* session)
 
         // 7. If old media session is not null and no longer has one or more audio-producing participants, then run the media session deactivation algorithm for old media session.
         if (!m_session->hasActiveMediaElements())
-            m_session->releaseSession();
+            m_session->deactivate();
     }
 
     if (session)