|Summary:||REGRESSION (r147454): Youtube annotation links to new window broken|
|Product:||WebKit||Reporter:||Kevin M. Dean <kevin>|
|Component:||Plug-ins||Assignee:||Anders Carlsson <andersca>|
|Severity:||Normal||CC:||allan.jensen, andersca, cevans, commit-queue, jimoase, jochen, jturcotte|
|Version:||528+ (Nightly build)|
|OS:||OS X 10.8|
Description Kevin M. Dean 2013-04-08 22:47:29 PDT
Youtube videos with annotation links that open a new window for the video page are no longer working, while links that load a video into the existing page continue to work fine. Load the link above which is cued up to just before the end of a video where some annotation links appear on the screen. Click the one on the left for Behind the Scenes and it pauses the video but doesn't open the new window as it should, while the link on the right for Water Jet Packs loads into the existing page fine. I'm finding this problem on many other videos where the link opens a new window rather than the loading into the existing page anywhere the link annotation may appear throughout the video.
Comment 1 Kevin M. Dean 2013-04-08 22:51:26 PDT
This bug also seems to affect clicking the "Watch on Youtube" button that appears as a Youtube logo on the lower toolbar on embedded pages like: https://online.hillsdale.edu/page.aspx?pid=1893 The new window at youtube.com doesn't open. This was reported in Bug 114097
Comment 2 Alexey Proskuryakov 2013-04-09 09:56:56 PDT
Regressed in <http://trac.webkit.org/changeset/147454>.
Comment 3 Alexey Proskuryakov 2013-04-09 10:00:51 PDT
*** Bug 114097 has been marked as a duplicate of this bug. ***
Comment 5 jochen 2013-04-09 12:05:24 PDT
that's bad, sorry will investigate & fix
Comment 7 Anders Carlsson 2013-04-09 14:34:27 PDT
Committed r148054: <http://trac.webkit.org/changeset/148054>
Comment 8 Allan Sandfeld Jensen 2013-09-10 06:36:13 PDT
The same needs to be done for WebKit1.
Comment 10 Anders Carlsson 2013-09-10 08:26:03 PDT
Comment on attachment 211195 [details] Patch Looks good. Next time please open a new bug instead of re-using one that's already been closed.
Comment 11 WebKit Commit Bot 2013-09-10 09:01:42 PDT
Comment on attachment 211195 [details] Patch Clearing flags on attachment: 211195 Committed r155447: <http://trac.webkit.org/changeset/155447>
Comment 12 WebKit Commit Bot 2013-09-10 09:01:46 PDT
All reviewed patches have been landed. Closing bug.