[wp-trac] [WordPress Trac] #28905: Audio/Video MCE views should use iframe sandboxes

[WordPress Trac]

#28905: Audio/Video MCE views should use iframe sandboxes
----------------------------+-------------------------
 Reporter:  wonderboymusic  |       Owner:
     Type:  defect (bug)    |      Status:  reopened
 Priority:  normal          |   Milestone:  4.0
Component:  Media           |     Version:  trunk
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |     Focuses:  javascript
----------------------------+-------------------------
Changes (by azaozz):

 * status:  closed => reopened
 * resolution:  fixed =>


Comment:

 I'm still at a loss here.
 - All audio/video/playlist views seem to use
 `wp_ajax_parse_media_shortcode()`. Is the block about audio and video in
 `wp_ajax_parse_embed()` still needed
 (https://core.trac.wordpress.org/browser/trunk/src/wp-admin/includes/ajax-
 actions.php#L2678).
 - If yes, `wp_ajax_parse_embed()` and `wp_ajax_parse_media_shortcode()`
 seem to repeat quite a bit of code. Merge them?

--
Ticket URL: <https://core.trac.wordpress.org/ticket/28905#comment:36>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform