Closed
Bug 364867
Opened 18 years ago
Closed 18 years ago
"WARNING: Textframe maps no content" on many websites
Categories
(Core :: Layout, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: mcsmurf, Unassigned)
References
Details
(Keywords: testcase)
Attachments
(2 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
With a current debug trunk build on Win2k I get this warning
WARNING: Textframe maps no content: file h:/mozilla/tree-main/mozilla/layout/gen
eric/nsTextFrame.cpp, line 4945
many times on various websites. I think this started with the landing of the reflow branch, but this is only a rough guess (I do not build debug trunk builds every day).
Reporter | ||
Comment 1•18 years ago
|
||
Comment 2•18 years ago
|
||
This warning appears very frequently, making it difficult to see other console output.
Flags: blocking1.9?
Reporter | ||
Comment 3•18 years ago
|
||
BTW: The warning was added as part of Bug 343445, not as part of reflow branch.
Updated•18 years ago
|
OS: Windows 2000 → All
Comment 4•18 years ago
|
||
Confirming that this occurs in
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a3pre) Gecko/20070302
Minefield/3.0a3pre
Again, the warning occurs so frequently that it's difficult to see other console output.
Comment 5•18 years ago
|
||
this is happening far too often, and disrupts people who aren't working on layout.
To make a positive contribution, I will see if I can figure out a fix for at least some of the situations that cause this. However, I don't know this code.
Attachment #257259 -
Flags: superreview?(roc)
Attachment #257259 -
Flags: review?(roc)
Comment 6•18 years ago
|
||
All instances of this warning I have observed are due to nsTextFrame *aBefore having a ContentOffset of 0 and a ContentLength of 0.
Comment on attachment 257259 [details] [diff] [review]
ifdef 0
I was going to just fix this but that's actually kinda hard and this code will be going away in favour of the new textframe code (which works entirely differently and has no equivalent assertion) so might as well just disable it for now. Sorry for the delay.
Attachment #257259 -
Flags: superreview?(roc)
Attachment #257259 -
Flags: superreview+
Attachment #257259 -
Flags: review?(roc)
Attachment #257259 -
Flags: review+
Comment 8•18 years ago
|
||
Checking in nsTextFrame.cpp;
/cvsroot/mozilla/layout/generic/nsTextFrame.cpp,v <-- nsTextFrame.cpp
new revision: 1.620; previous revision: 1.619
done
Flags: blocking1.9? → blocking1.9-
Comment 9•18 years ago
|
||
Fixed in the best of all possible ways, cvs rm.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•