Closed
Bug 575446
Opened 14 years ago
Closed 14 years ago
Hang and too-much-recursion crash with tree, filter, text-shadow
Categories
(Core :: XUL, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jruderman, Assigned: neil)
References
Details
(Keywords: crash, hang, testcase)
Attachments
(2 files)
240 bytes,
application/vnd.mozilla.xul+xml
|
Details | |
594 bytes,
patch
|
enndeakin
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Comment 1•14 years ago
|
||
After it hangs for a while, it hits a too-much-recursion crash:
...
477 PresShell::HandlePostedReflowCallbacks(int) + 225 (nsPresShell.cpp:4685)
478 PresShell::DidDoReflow(int) + 38 (nsPresShell.cpp:7301)
479 PresShell::ProcessReflowCommands(int) + 503 (nsPresShell.cpp:7569)
480 PresShell::FlushPendingNotifications(mozFlushType) + 840 (nsPresShell.cpp:4816)
481 PresShell::HandlePostedReflowCallbacks(int) + 225 (nsPresShell.cpp:4685)
...
Keywords: crash
Summary: Hang with tree, filter, text-shadow → Hang and too-much-recursion crash with tree, filter, text-shadow
Reporter | ||
Comment 2•14 years ago
|
||
Bug 550306 shows a similar too-much-recursion pattern.
Assignee | ||
Comment 3•14 years ago
|
||
What seems to be going on here is that the tree is trying to mediate the wrong scrollbar - it's finding the scrollbar caused by making the <tree> element overflow: scroll, instead of the XBL scrollbar element. So it keeps on trying to set the maxpos to 0 (since the tree body isn't scrollable) while the overflow: scroll frame tries to set the maxpos to 454526, which appears to be the number of CSS pixels of overflow caused by the outline.
Assignee | ||
Comment 4•14 years ago
|
||
Updated•14 years ago
|
Attachment #454811 -
Flags: review?(enndeakin) → review+
Assignee | ||
Comment 5•14 years ago
|
||
Pushed changeset 4a12cdb66595 to mozilla-central.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•