Create scrollframes for frameset documents

RESOLVED FIXED in mozilla16

Status

()

RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: roc, Assigned: roc)

Tracking

Trunk
mozilla16
x86_64
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

We might as well. Framesets aren't common, the optimization of avoiding scrollframes is insigificant, and it simplifies things across mobile and desktop and reduces the number of different code paths.
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?
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 on attachment 634269 [details] [diff] [review]
fix

Ok.
Attachment #634269 - Flags: review?(tnikkel) → review+
https://hg.mozilla.org/mozilla-central/rev/e828ff0dbc94
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16

Updated

6 years ago
Depends on: 807174

Updated

6 years ago
No longer depends on: 807174
You need to log in before you can comment on or make changes to this bug.