Don't use hidden #post_ID input field to get mce-view embed postID

[rhurling]

The mce-view embeds use postID = $( '#post_ID' ).val() || 0 to get the ID of the current post (see https://core.trac.wordpress.org/browser/trunk/src/wp-includes/js/mce-view.js#L731).
Everything else I could find used wp.media.view.settings.post.id.

I think it would be better to either use wp.media.view.settings.post.id in the callbacks that use the current postID variable or switch to something that is localized to the script (brought up by @iseulde; ​https://wordpress.slack.com/archives/core-editor/p1437654722000160).

I will add a patch which uses wp.media.view.settings.post.id, because I'm not sure what else I would have to change to use a localize the script.

[rhurling]

Patch which removes the postID variable and uses wp.media.view.settings.post.id

[iseulde]

There's also wp.media.model.settings.post.id. We can't use any of these because the script does not depend on media-*.js. It needs its own setting.

It looks like your patch is reversed or something.

[rhurling]

Refreshed patch, since the first didn't seemed to work

[iseulde]

Fixes dependencies as well.

[iseulde]

In 33426:

TinyMCE: views: use media setting to get post ID

Also fix dependency declarations and confusing variable names.

Props rhurling.
Fixes #33096.

[iseulde]

In 33428:

JSHint for [33426]

Oops. :|

See #33096.