YouTube Flash plug-in replacement facility should more gracefully handle malformed queries Some YouTube Flash embeds use '&' instead of '?' to start the query portion of the URL. Here's an example: http://www.youtube.com/v/dQw4w9WgXcQ&start=4 Our implementation drops everything including and after the &. Instead, we should treat the & like a ? and restore the query when we're done. If we do this, it means embeds for a particular timecode will be preserved.
Created attachment 287604 [details] first attempt
Comment on attachment 287604 [details] first attempt View in context: https://bugs.webkit.org/attachment.cgi?id=287604&action=review > Source/WebCore/Modules/plugins/YouTubePluginReplacement.cpp:203 > +static URL processAndCreateYouTubeURL(const URL& url, bool& isYouTubeShortenedURL, String& outMalformedQuery) "malformedQuery" doesn't seem right, although I don't have any great suggestions. Maybe "unusedParameters"?
Created attachment 287643 [details] same as last time, with a few renames
Comment on attachment 287643 [details] same as last time, with a few renames Clearing flags on attachment: 287643 Committed r205306: <http://trac.webkit.org/changeset/205306>
All reviewed patches have been landed. Closing bug.