Comment 1 Robert O'Callahan (:roc) (email my personal email if necessary) 2012-06-18 19:12:05 PDT

Created attachment 634269 [details] [diff] [review]
fix

Comment 2 Timothy Nikkel (:tnikkel) 2012-06-18 21:34:40 PDT

Comment on attachment 634269 [details] [diff] [review]
fix

I guess content can't do anything silly like create scrollbars by applying an overflow style or something to the root or body element?

Comment 3 Robert O'Callahan (:roc) (email my personal email if necessary) 2012-06-20 17:47:06 PDT

It actually can; html.css has "frameset { overflow: -moz-hidden-unscrollable; }", so content can override it. However, I think that's a feature. Note that currently content can add scrollframes to non-root <frameset>s.

Comment 4 Timothy Nikkel (:tnikkel) 2012-06-20 22:44:38 PDT

Comment on attachment 634269 [details] [diff] [review]
fix

Ok.

Comment 5 Robert O'Callahan (:roc) (email my personal email if necessary) 2012-06-21 22:11:46 PDT

https://hg.mozilla.org/integration/mozilla-inbound/rev/419f0d1b848b

Comment 6 Mounir Lamouri (:mounir) 2012-06-22 02:16:21 PDT

Backed out due to perma orange on Moth, see:
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=8661c74deeb5

https://hg.mozilla.org/integration/mozilla-inbound/rev/ef1c6a327b0d

Comment 7 Robert O'Callahan (:roc) (email my personal email if necessary) 2012-06-22 03:55:30 PDT

This didn't seem to be the cause of the orange. Relanded:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e828ff0dbc94

Comment 8 Ryan VanderMeulen [:RyanVM] 2012-06-23 05:48:07 PDT

https://hg.mozilla.org/mozilla-central/rev/e828ff0dbc94