1809635 - Somewhat safer usage of nsITreeView in nsTreeBodyFrame.

Status: RESOLVED FIXED

(Core :: Layout, task)

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1809635 - Somewhat safer usage of tree views in nsTreeBodyFrame. r=smaug

Not convinced this will help with bug 1809492, so different bug... But
these are places where I noticed we call nsITreeView from a member.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/acb661aab657
Somewhat safer usage of tree views in nsTreeBodyFrame. r=smaug

Comment 3

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/acb661aab657

Comment 4

[Emilio Cobos Álvarez (:emilio)]

Comment on attachment 9311727 [details]
Bug 1809635 - Somewhat safer usage of tree views in nsTreeBodyFrame. r=smaug

Beta/Release Uplift Approval Request

• User impact if declined: Should fix or help with bug 1809492.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce: N/A
• List of other uplifts needed: none
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Has been in the tree for a way, fairly low-risk patch.
• String changes made/needed: none
• Is Android affected?: No

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9311727 [details]
Bug 1809635 - Somewhat safer usage of tree views in nsTreeBodyFrame. r=smaug

Crashes are still happening in 110.0b7, so it doesn't appear this patch fixed them unfortunately.