WordPress.org

Make WordPress Core

Changeset 29191


Ignore:
Timestamp:
07/16/2014 06:18:18 PM (7 years ago)
Author:
wonderboymusic
Message:

Cleanup after [29179]:

  • Pause players when media is edited via modal
  • Remove players on unind
  • Account for failure when an empty node is passed to an mce.view.View

See #28905.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/js/mce-view.js

    r29189 r29191  
    130130
    131131                    node = $( node ).find( '.wpview-content' )[0];
    132 
    133                     if ( node ) {
    134                         node.innerHTML = '';
    135                     }
     132                    if ( ! node ) {
     133                        return;
     134                    }
     135                    node.innerHTML = '';
    136136
    137137                    iframe = dom.add( node, 'iframe', {
     
    512512                _.bindAll( this, 'createIframe', 'setNode', 'fetch', 'pausePlayers' );
    513513                $( this ).on( 'ready', this.setNode );
     514
     515                $( document ).on( 'media:edit', this.pausePlayers );
    514516            },
    515517
    516518            setNode: function ( event, editor, node ) {
    517                 this.node = node;
     519                if ( node ) {
     520                    this.node = node;
     521                }
    518522                editor.on( 'hide', this.pausePlayers );
    519523
     
    574578            pausePlayers: function() {
    575579                var p, win = $( 'iframe', this.node ).get(0).contentWindow;
    576                 if ( win.mejs ) {
     580                if ( win && win.mejs ) {
    577581                    for ( p in win.mejs.players ) {
    578582                        win.mejs.players[p].pause();
     583                    }
     584                }
     585            },
     586
     587            unsetPlayers: function() {
     588                var p, win = $( 'iframe', this.node ).get(0).contentWindow;
     589                if ( win && win.mejs ) {
     590                    for ( p in win.mejs.players ) {
    579591                        win.mejs.players[p].remove();
    580592                    }
    581593                }
     594            },
     595
     596            unbind: function() {
     597                this.pausePlayers();
     598                this.unsetPlayers();
    582599            }
    583600        },
Note: See TracChangeset for help on using the changeset viewer.