Changeset 29194

Timestamp:

07/16/2014 07:35:55 PM (10 years ago)

Author:

wonderboymusic

Message:

Cleanup after [29179]:

• Remove wp.media.mixin.pauseAllPlayers() - was only ever intended to be used in MCE views, reaches too far into global scope. Rendered irrelevant since [28364].
• Remove wp.media.mixin.pausePlayers() - added in [28364]. MCE views no longer keep track of player instances and call mixin methods, they refer to their child iframes and call MEjs methods on the iframe's window.mejs.players.

See #28905.

File:

• trunk/src/wp-includes/js/media-audiovideo.js (modified) (2 diffs)

trunk/src/wp-includes/js/media-audiovideo.js

@@ -15 +15 @@
     wp.media.mixin = {
         mejsSettings: baseSettings,
-        /**
-         * Pauses every instance of MediaElementPlayer
-         */
-        pauseAllPlayers: function() {
-            var p;
-            if ( window.mejs && window.mejs.players ) {
-                for ( p in window.mejs.players ) {
-                    window.mejs.players[p].pause();
-                }
-            }
-        },
 
         removeAllPlayers: function() {
@@ -36 +25 @@
                 }
             }
-        },
-
-        /**
-         * Pauses the current object's instances of MediaElementPlayer
-         */
-        pausePlayers: function() {
-            _.each( this.players, function (player) {
-                player.pause();
-            } );
         },