Changeset 27974 for trunk/src/wp-includes/media.php

Timestamp:

04/06/2014 09:33:00 PM (12 years ago)

Author:

wonderboymusic

Message:

When dynamically instantiating MediaElementPlayer instances in JS, preload="metadata" is extremely wonky. preload="none" lets the file play right away.

This has already been addressed for audio|video shortcodes, but also needs to be the case for playlists. When loading a playlist from a DOM cache or via AJAX, and when the sources match those already pre-loaded, the browser will sometimes donkey-punch the HTTP request for the first cached track selected.

Set preload="none" for the empty <audio|video> tag for playlists.

See commits related to #26779.

File:

• trunk/src/wp-includes/media.php (modified) (1 diff)

trunk/src/wp-includes/media.php

@@ -1270 +1270 @@
     <div class="wp-playlist-current-item"></div>
     <?php endif ?>
-    <<?php echo $safe_type ?> controls="controls" preload="metadata" width="<?php
+    <<?php echo $safe_type ?> controls="controls" preload="none" width="<?php
         echo (int) $theme_width;
     ?>"<?php if ( 'video' === $safe_type ):