Closed
Bug 803330
Opened 12 years ago
Closed 8 years ago
"ASSERTION: row count did not change by the amount suggested, check caller" with Page Info
Categories
(Firefox :: Page Info Window, defect)
Tracking
()
RESOLVED
FIXED
Firefox 50
Tracking | Status | |
---|---|---|
firefox50 | --- | fixed |
People
(Reporter: jruderman, Assigned: keeler)
References
(Blocks 1 open bug)
Details
(Keywords: assertion, testcase)
Attachments
(3 files)
1. Load the testcase
2. ⌘I (open page info)
3. ⌘` (switch back to the browser window)
4. ⌘I (reset page info)
###!!! ASSERTION: row count did not change by the amount suggested, check caller: 'rowCount == mRowCount', file layout/xul/base/src/tree/src/nsTreeBodyFrame.cpp, line 1819
Not sure if this is a Page Info bug or a XUL tree bug.
Reporter | ||
Comment 1•12 years ago
|
||
Assignee | ||
Comment 2•8 years ago
|
||
I can reproduce this by just opening the page info dialog - the page in question doesn't seem to matter.
Assignee | ||
Comment 3•8 years ago
|
||
When calling rowCountChanged on an nsITreeBoxObject, the implementation assumes
that the row count has actually changed. pageInfoTreeView violated this
assumption in addRows by first calling rowCountChanged and then updating its row
count. This patch fixes the issue by using the power of decomposition. That is,
there's already a perfectly fine function called 'addRow' that can be called
once for each row being added.
Review commit: https://reviewboard.mozilla.org/r/64418/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/64418/
Attachment #8771168 -
Flags: review?(florian)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → dkeeler
Updated•8 years ago
|
Attachment #8771168 -
Flags: review?(florian) → review+
Comment 4•8 years ago
|
||
Comment on attachment 8771168 [details]
bug 803330 - fix row count assertion failure in page info dialog
https://reviewboard.mozilla.org/r/64418/#review61872
I wonder if the rowCountChanged calls are likely to have a performance impact, but given that addRows is currently only used for meta tags, and media elements already call addRow once for each item, it's likely not going to matter.
Assignee | ||
Comment 5•8 years ago
|
||
Thanks!
Pushed by dkeeler@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/75f8e3a05ef8
fix row count assertion failure in page info dialog r=florian
Comment 7•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox50:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
You need to log in
before you can comment on or make changes to this bug.
Description
•