Bug 36184 - YouTube video resizing doesn't work with OOP plug-ins
: YouTube video resizing doesn't work with OOP plug-ins
Status: RESOLVED FIXED
Product: WebKit
Classification: Unclassified
Component: Plug-ins
: 528+ (Nightly build)
: Macintosh Intel Mac OS X 10.6
: P2 Normal
Assigned To: Alexey Proskuryakov
: InRadar, Regression
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-03-16 12:57 PDT by Alexey Proskuryakov
Modified: 2010-03-16 14:27 PDT (History)
0 users

See Also:


Attachments
proposed fix (6.04 KB, patch)
2010-03-16 13:29 PDT, Alexey Proskuryakov
darin: review+
Details | Formatted Diff | Diff
oops, forgot about WebKitTools (5.62 KB, patch)
2010-03-16 14:03 PDT, Alexey Proskuryakov
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2010-03-16 12:57:09 PDT
Steps to reproduce:

1. Start a youtube video.
2. Click on resize (two-arrow) button to change its size.

Results: depending on original size, the video or its bounding view will be scaled incorrectly, with controls misplaced.

<rdar://problem/7742659>
Comment 1 Alexey Proskuryakov 2010-03-16 13:29:16 PDT
Created attachment 50828 [details]
proposed fix

I've tried using DOM events to make the test work automatically in browser without success.
Comment 2 Darin Adler 2010-03-16 13:37:19 PDT
Comment on attachment 50828 [details]
proposed fix

Does this hurt performance?

r=me
Comment 3 Alexey Proskuryakov 2010-03-16 13:42:12 PDT
Committed <http://trac.webkit.org/changeset/56079>

> Does this hurt performance?

The test I performed was putting six YouTube videos on a page and scrolling. This didn't seem to have been affected.
Comment 4 Alexey Proskuryakov 2010-03-16 14:03:35 PDT
Created attachment 50836 [details]
oops, forgot about WebKitTools
Comment 5 Alexey Proskuryakov 2010-03-16 14:27:02 PDT
Committed WebKitTools changes in <http://trac.webkit.org/changeset/56086>.