Closed Bug 1024274 Opened 10 years ago Closed 10 years ago

Win64 REFTEST TEST-UNEXPECTED-PASS | file:///C:/slave/test/build/tests/reftest/tests/layout/style/crashtests/413274-1.xhtml | assertion count 0 is less than expected 2 assertions

Categories

(Core :: Layout, defect)

32 Branch
x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 576435

People

(Reporter: away, Unassigned)

References

Details

The "expected" asserts that we're not getting on win64 are:

###!!! ASSERTION: How can this ever happen?: '*aUnconstrainedSize <= scriptLevelSize', file s:\central\layout\style\nsRuleNode.cpp, line 2765

###!!! ASSERTION: scriptminsize should never be making things bigger: 'aFont->mScriptUnconstrainedSize <= aFont->mSize', file s:\central\layout\style\nsRuleNode.cpp, line 3728
*aUnconstrainedSize = 0x18138400 (403932160)
scriptLevelSize = 0xc0000000 (-1073741824)

Some floating-point math may have gone wrong. The asserts on Win32 go away if I remove the X86 perf shortcut in NSToCoordRound.
What this boils down to is that on 32-bit Windows, NSToCoordRound(nscoord_MAX) is negative, despite the assurances in NS_lroundup30: http://hg.mozilla.org/mozilla-central/annotate/c482c28b35b6/xpcom/ds/nsMathUtils.h#l38
Let's just remove the lround30 code altogether.
Blocks: 413787
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.