Closed Bug 803330 Opened 7 years ago Closed 3 years ago

"ASSERTION: row count did not change by the amount suggested, check caller" with Page Info

Categories

(Firefox :: Page Info Window, defect)

x86_64
macOS
defect
Not set

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.
Attached file stack
Blocks: fuzz-keys
I can reproduce this by just opening the page info dialog - the page in question doesn't seem to matter.
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: nobody → dkeeler
Attachment #8771168 - Flags: review?(florian) → review+
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.
Pushed by dkeeler@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/75f8e3a05ef8
fix row count assertion failure in page info dialog r=florian
https://hg.mozilla.org/mozilla-central/rev/75f8e3a05ef8
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
You need to log in before you can comment on or make changes to this bug.