As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 764249 - Youtube HTML5 resize buttons(Small/Large player) in video controls are missing
: Youtube HTML5 resize buttons(Small/Large player) in video controls are missing
: regression
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: 16 Branch
: x86 All
: -- normal with 1 vote (vote)
: mozilla16
Assigned To: Luke Wagner [:luke]
: Jason Orendorff [:jorendorff]
Depends on:
Blocks: 762473
  Show dependency treegraph
Reported: 2012-06-12 19:53 PDT by Alice0775 White
Modified: 2012-06-18 07:28 PDT (History)
7 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

fix (2.34 KB, patch)
2012-06-13 01:38 PDT, Luke Wagner [:luke]
dvander: review+
Details | Diff | Splinter Review

Description User image Alice0775 White 2012-06-12 19:53:57 PDT
Build Identifier:
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0a1 ID:20120612030527


Steps to Reproduce:
1. Create clean profile
2. Open

Actual Results:
 Resize buttons(Small/Large player) are missing.

 And An error shows in Error Console as follows:
Security Error: Content at may not load data from

Expected Results:
 Resize buttons(Small/Large player) should be available.

Regressiin Window(m-c)
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0a1 ID:20120608134854
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0a1 ID:20120609030536
Comment 1 User image Wes Kocher (:KWierso) 2012-06-12 20:05:51 PDT
I can still see the button if I screw around a lot with the Style Editor and Inspector tools.
I deleted a bunch of instances of "display:none" with Style Editor, and I can then see the Large Player button, overlaid by the gearbox settings button. Then in Inspector, I removed the "hid" part of the large player button's class attribute, and it shows up fully.
Comment 2 User image Alice0775 White 2012-06-12 22:21:10 PDT
In localbuild
Last Good:7d68b45776ff
First Bad:227b4a4c3c46
Regressed by:
227b4a4c3c46	Luke Wagner — Bug 762473 - Don't apply non-reentrant-closure optimization when the scope is extensible (r=dvander)
Comment 3 User image Luke Wagner [:luke] 2012-06-13 01:38:34 PDT
Created attachment 632600 [details] [diff] [review]

Thanks for the clear report, STR, and bisection!

The last fix failed because, apparently, you can't just conservatively set isOuterFunction/isInnerFunction to false.  This patch achieves the same goal (don't do the optimization when the scope can grow/shrink) using a more simple/brutish method.  I'd work more on this if the whole awful mess wasn't being removed soon (bug 753158).
Comment 5 User image Ryan VanderMeulen [:RyanVM] 2012-06-16 06:52:01 PDT
Comment 6 User image Jim Jeffery not reading bug-mail 1/2/11 2012-06-16 08:12:38 PDT
This may not be fixed, looking at still has no re-size buttons showing.

Tested using latest hourly build which contains the patch:

Test URL in step 2 of comment #0 also shows no size-control buttons. Ctrl+F5 to force a reload does not seem to reveal the buttons, thinking it may be in cache or something.
The size buttons do show in latest dev version of Chrome.
Comment 7 User image Alice0775 White 2012-06-16 08:26:15 PDT
Confirmed, The problem is still reproduced.
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0a1 ID:20120616063925
Comment 8 User image Luke Wagner [:luke] 2012-06-17 22:06:39 PDT
Arg, my mistake: the cset in comment 5 is for another bug, the patch for *this* bug is still unlanded.  Thanks for verifying, I'll land the actual patch next.
Comment 9 User image Luke Wagner [:luke] 2012-06-17 23:58:00 PDT
Backed out wrongly-named patch (will re-land in appropriate bug):

Landed this bug's patch:

Sorry for the trouble.
Comment 10 User image Ed Morley [:emorley] 2012-06-18 07:28:32 PDT

Note You need to log in before you can comment on or make changes to this bug.