Closed
Bug 382756
Opened 17 years ago
Closed 17 years ago
"ASSERTION: When exactly is this supposed to be non-zero?"
Categories
(Core :: Layout, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jruderman, Assigned: sharparrow1)
References
Details
(Keywords: assertion, regression, testcase)
Attachments
(2 files)
|
273 bytes,
application/vnd.mozilla.xul+xml
|
Details | |
|
1.25 KB,
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
Loading the testcase triggers: ###!!! ASSERTION: When exactly is this supposed to be non-zero?: '!viewRect.x && !viewRect.y', file /Users/jruderman/trunk/mozilla/view/src/nsViewManager.cpp, line 490 This assertion was added in bug 380438.
|
Assignee | |
Comment 1•17 years ago
|
||
Okay, I'll have a look at it; the reason I added it as an assertion is because I couldn't come up with any way to test the translation getting used for that case... thanks for the testcase. It's not really critical, though; the worst that could happen is corrupted painting. I guess the situation really is rare if it took a week for someone to hit it with an irregular testcase. To make the testcase assert cross-platform, just add an outline onto the treechildren. It looks like we are in fact doing something wrong here, though; incremental painting leads to gaps in the outline.
Assignee: nobody → sharparrow1
|
|
Assignee | |
Comment 2•17 years ago
|
||
Attachment #270138 -
Flags: review?(roc)
Comment on attachment 270138 [details] [diff] [review] Patch ugh!
Attachment #270138 -
Flags: superreview+
Attachment #270138 -
Flags: review?(roc)
Attachment #270138 -
Flags: review+
|
|
Assignee | |
Comment 4•17 years ago
|
||
Checked in.
Status: NEW → RESOLVED
Closed: 17 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•