Press This: Check embeds list against oembed providers too.

[stephdau]

We brought the concept of checking pressed URLs against core's list of oembed providers, which makes it possible to auto-embed media in the editor on page load, when scanning a Youtube page, etc.

The check currently resides in the get_suggested_content() method. We should widen the scope of this check to the _limit_embed() method, so we can do away with the tests that check the sent $src against page regexes (not iframe ones):

		} else if ( ! preg_match( '/\/\/(m|www)\.youtube\.com\/watch\?/', $src )          // Youtube video page (www or mobile)
		            && ! preg_match( '/\/youtu\.be\/.+$/', $src )                         // Youtu.be video page
		            && ! preg_match( '/\/\/vimeo\.com\/[\d]+$/', $src )                   // Vimeo video page
		            && ! preg_match( '/\/\/(www\.)?dailymotion\.com\/video\/.+$/', $src ) // Daily Motion video page
		            && ! preg_match( '/\/\/soundcloud\.com\/.+$/', $src )                 // SoundCloud audio page
		            && ! preg_match( '/\/\/twitter\.com\/[^\/]+\/status\/[\d]+$/', $src ) // Twitter status page
		            && ! preg_match( '/\/\/vine\.co\/v\/[^\/]+/', $src ) ) {              // Vine video page
			$src = '';
		}

Once this is done, we could have the get_embeds() method also check the page URL, which would then have the embed that is auto-inserted in the editor also be part of the list of suggested media. This way, if someone lands on, say, a TED page, they will have both the auto-inserted embed, as well as a thumbnail to click, should they need it.

Currently, the alternative would be to re-press the page to get the embed again.

If we go that route, we'll also need to tweak the related code in the bookmarklet.

	if ( href.match( /\/\/(www|m)\.youtube\.com\/watch/ ) ||
		href.match( /\/\/vimeo\.com\/(.+\/)?([\d]+)$/ ) ||
		href.match( /\/\/(www\.)?dailymotion\.com\/video\/.+$/ ) ||
		href.match( /\/\/soundcloud\.com\/.+$/ ) ||
		href.match( /\/\/twitter\.com\/[^\/]+\/status\/[\d]+$/ ) ||
		href.match( /\/\/vine\.co\/v\/[^\/]+/ ) ) {

		add( '_embeds[]', href );
	}

I'd do away with it altogether.

[stephdau]

Currently: media is auto-embedded, but not suggested in list.

[stephdau]

32069.diff​ implements the proposed solution.

[azaozz]

In 32828:

Press This: Check the embeds list against all of the oembed providers.
Props stephdau. Fixes #32069.