WordPress.org
Get WordPress

Make WordPress Core

Changeset 29189


Ignore:
Timestamp:
07/16/2014 04:13:47 PM (12 years ago)
Author:
wonderboymusic
Message:

Cleanup after [29179]:

  • Cleanup players when the editor is hidden - window scope is unique to each frame
  • Add the editor body class to each iframe sandbox
  • Remove unneeded code from wp-mediaelement.js

See #28905.

Location:
trunk/src/wp-includes/js
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-includes/js/mce-view.js

    r29187 r29189  
    131131                                        node = $( node ).find( '.wpview-content' )[0];
    132132
    133                                         node.innerHTML = '';
     133                                        if ( node ) {
     134                                                node.innerHTML = '';
     135                                        }
    134136
    135137                                        iframe = dom.add( node, 'iframe', {
    … …  
    153155                                                                '<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />' +
    154156                                                        '</head>' +
    155                                                         '<body style="padding: 0; margin: 0;" class="' + /* editor.getBody().className + */ '">' +
     157                                                        '<body data-context="iframe-sandbox" style="padding: 0; margin: 0;" class="' + editor.getBody().className + '">' +
    156158                                                                content +
    157159                                                        '</body>' +
    … …  
    508510                                this.fetching = false;
    509511
    510                                 _.bindAll( this, 'createIframe', 'setNode', 'fetch' );
     512                                _.bindAll( this, 'createIframe', 'setNode', 'fetch', 'pausePlayers' );
    511513                                $( this ).on( 'ready', this.setNode );
    512514                        },
    513515
    514                         setNode: function () {
     516                        setNode: function ( event, editor, node ) {
     517                                this.node = node;
     518                                editor.on( 'hide', this.pausePlayers );
     519
    515520                                if ( this.parsed ) {
    516521                                        this.createIframe( this.parsed );
    … …  
    565570                                }
    566571                                return this.parsed;
     572                        },
     573
     574                        pausePlayers: function() {
     575                                var p, win = $( 'iframe', this.node ).get(0).contentWindow;
     576                                if ( win.mejs ) {
     577                                        for ( p in win.mejs.players ) {
     578                                                win.mejs.players[p].pause();
     579                                                win.mejs.players[p].remove();
     580                                        }
     581                                }
    567582                        }
    568583                },

  • trunk/src/wp-includes/js/mediaelement/wp-mediaelement.js

    r28370 r29189  
    77        $(function () {
    88                var settings = {};
    9 
    10                 if ( $( document.body ).hasClass( 'mce-content-body' ) ) {
    11                         return;
    12                 }
    139
    1410                if ( typeof _wpmejsSettings !== 'undefined' ) {
Note: See TracChangeset for help on using the changeset viewer.