iframe overlaps text until i force a reflow

RESOLVED FIXED

Status

()

Core
Layout: Block and Inline
--
major
RESOLVED FIXED
14 years ago
14 years ago

People

(Reporter: Jesse Ruderman, Assigned: dbaron)

Tracking

({regression, testcase})

Trunk
x86
All
regression, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

14 years ago
When I use the "view style sheets" bookmarklet on a page with a linked style
sheet, the iframe overlaps other text the bookmarklet spits out.  If I force a
reflow (resize window, use Firebird's autoscroll, etc), the iframe jumps to
where it should be.

The testcase doesn't use JavaScript. It's just a simple HTML page with an
iframe.  The testcase is fragile, like bug 214607's: adding space before <html>
or before <body> fixes it.
(Reporter)

Comment 1

14 years ago
Created attachment 128949 [details]
testcase

<html><body> <h3>Style sheets</h3> <iframe
src="http://www.squarefree.com/squarefree.css"></iframe></body>
.
Assignee: other → block-and-inline
Component: Layout → Layout: Block & Inline

Comment 3

14 years ago
Bug occurs in 2003-07-28-10 trunk Linux, but works fine in Mozilla 1.4 Linux
Keywords: regression
OS: Windows XP → All

Comment 4

14 years ago
2003-07-22-05: bug does not occur
2003-07-23-05: the IFRAME is not visible at all
2003-07-24-05: the IFRAME and the text overlaps

bug 79315 seems to fit the problem and is in the time frame
(Assignee)

Comment 5

14 years ago
Maybe the same as bug 214077...
Depends on: 214077
(Assignee)

Comment 6

14 years ago
Actually, this is a widget positioning problem, and I can see two potential
causes of such problems:
 1. we're positioning the views inside-out rather than outside-in in
RelativePositionFrames
 2. we don't call PositionChildViews to reposition any views in descendants of
leaves -- if they have widgets those widgets won't be repositioned.

I think it's the first case that's the problem here.
(Assignee)

Comment 7

14 years ago
Created attachment 129262 [details] [diff] [review]
patch
(Assignee)

Updated

14 years ago
Attachment #129262 - Flags: superreview?(roc+moz)
Attachment #129262 - Flags: review?(roc+moz)
Comment on attachment 129262 [details] [diff] [review]
patch

ah yes. Nice.
Attachment #129262 - Flags: superreview?(roc+moz)
Attachment #129262 - Flags: superreview+
Attachment #129262 - Flags: review?(roc+moz)
Attachment #129262 - Flags: review+
(Assignee)

Updated

14 years ago
Attachment #129262 - Flags: approval1.5b?

Comment 9

14 years ago
Comment on attachment 129262 [details] [diff] [review]
patch

a=asa (on behalf of drivers) for checkin to 1.5beta.
Attachment #129262 - Flags: approval1.5b? → approval1.5b+
(Assignee)

Comment 10

14 years ago
taking
Assignee: block-and-inline → dbaron
(Assignee)

Comment 11

14 years ago
Fix checked in to trunk, 2003-08-06 16:59 -0700.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.