Closed Bug 819291 Opened 8 years ago Closed 7 years ago

crash in nsIContent::GetPrimaryFrame, called from mozilla::a11y::TextAttrsMgr::GetAttributes

Categories

(Core :: Disability Access APIs, defect)

x86
Windows 7
defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla30

People

(Reporter: MarcoZ, Assigned: yzen)

References

Details

(Keywords: crash, Whiteboard: [good first bug][mentor=trev.saunders@gmail.com][lang=c++][tbird crash])

Crash Data

Attachments

(1 file)

This bug was filed from the Socorro interface and is 
report bp-1a663c9b-4797-43a3-9c0c-961c82121207 .
============================================================= 

I encountered this crash once today when starting a new article on my WordPress blog, but it doesn't happen reliably. Filing it just in case it reveals something obvious. Might also be related to problems Trevor is encountering with enhancing text attribute tests.
OS: Windows NT → Windows 7
It seems somehow we have a HyperTextAccessible for something that isn't a element, Alex any idea how that's possible?  How do you eel about an assert that doesn't happen?
(In reply to Trevor Saunders (:tbsaunde) from comment #1)
> It seems somehow we have a HyperTextAccessible for something that isn't a
> element, Alex any idea how that's possible?  How do you eel about an assert
> that doesn't happen?

I don't see alternatives.
Whiteboard: [good first bug][mentor=trev.saunders@gmail.com][lang=c++]
Marco could you try and reproduce with this build? https://tbpl.mozilla.org/?tree=Try&rev=22a25d1c1879
Trevor, I never saw the bug again unfortunately. So reproducing it may become a real challenge. :)
A community remember just reported on Twitter that (s)he was seeing crashes, and when I received report IDs, found that this is the same bug I was seeing. The IDs are:
bp-d422654c-bf81-47e7-a9f3-887ca2130326
bp-05bd58fe-2a34-4a2e-a2b8-ec9842130326
Duplicate of this bug: 857340
Whiteboard: [good first bug][mentor=trev.saunders@gmail.com][lang=c++] → [good first bug][mentor=trev.saunders@gmail.com][lang=c++][tbird crash]
Crash Signature: [@ nsIContent::GetPrimaryFrame()] → [@ nsIContent::GetPrimaryFrame()] [@ mozilla::a11y::TextAttrsMgr::GetAttributes(nsIPersistentProperties*, int*, int*) ]
I am not able to reproduce this bug in latest build, please send me scenario in which case this bug is produceable. and also let me steps to reproduce this bug.
Assignee: nobody → riteshnpatel1994
Had another of those  crashes after ages. I was answering a Facebook friend request, and the dialog where I was asked whether I wanted to confirm or delete had just gone away after I clicked Confirm.
https://crash-stats.mozilla.com/report/index/7e47c858-ee5c-4b25-a15a-a34922140128
Is this bug still taken? If not, I am willing to take it on.
Thanks.
Flags: needinfo?(riteshnpatel1994)
Taking since did not get a response.
Assignee: riteshnpatel1994 → yzenevich
Flags: needinfo?(riteshnpatel1994)
Attached patch 819291 patchSplinter Review
Added an assertion and the handler for null.
Attachment #8375822 - Flags: review?(surkov.alexander)
Comment on attachment 8375822 [details] [diff] [review]
819291 patch

Review of attachment 8375822 [details] [diff] [review]:
-----------------------------------------------------------------

r=me, thanks
Attachment #8375822 - Flags: review?(surkov.alexander) → review+
https://hg.mozilla.org/mozilla-central/rev/55066842727a
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.