Closed Bug 676340 Opened 9 years ago Closed 9 years ago

Crash [@ nsAccessible::NativeState() ] (called by nsXULTextFieldAccessible::GetValue)

Categories

(Core :: Disability Access APIs, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla7
Tracking Status
firefox7 --- fixed

People

(Reporter: surkov, Assigned: surkov)

References

Details

(Keywords: access, Whiteboard: [qa-])

Crash Data

Attachments

(1 file)

Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED
Comment on attachment 550454 [details] [diff] [review]
patch

SHOULDN'T YOU GET THE MSAA METHOD AS WELL AS THE XPCOM ONE? BUT i DON'T REALLY CARE SINCE i'LL ET IT SOON ANYWAY IF YOU DON'T
Attachment #550454 - Flags: review?(trev.saunders) → review+
(In reply to comment #1)
> Comment on attachment 550454 [details] [diff] [review] [diff] [details] [review]
> patch
> 
> SHOULDN'T YOU GET THE MSAA METHOD AS WELL AS THE XPCOM ONE? BUT i DON'T
> REALLY CARE SINCE i'LL ET IT SOON ANYWAY IF YOU DON'T

that would be safer, but not technically correct, and yes I wouldn't care much since it goes away.
Duplicate of this bug: 672297
landed http://hg.mozilla.org/mozilla-central/rev/f0075fe638ee
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
Firefox 6 keeps crashing with this.
Comment on attachment 550454 [details] [diff] [review]
patch

can we get it into Firefox 7? Trivial patch, no risk. High crash numbers in accessibility module.
Attachment #550454 - Flags: approval-mozilla-beta?
(In reply to Karel Jára from comment #6)
> Firefox 6 keeps crashing with this.

Firefox 6 was shipped, all we can do is to try to get it into Firefox 7.
This should definitely be reopened or a new bug should be filed od this nsAccessible::NativeState() bug. Recently I hasn’t been able to retrieve my latest crash reports with FF 7 beta. But with FF 6 beta it often did after a download has been completed.
(In reply to Karel Jára from comment #9)

This fix landed on the Nightly version of Firefox which is Firefox 8. The release drivers are currently debating whether to take this patch for Firefox 7.
Attachment #550454 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Marco, could you please land this patch on beta?
(In reply to alexander surkov from comment #11)
> Marco, could you please land this patch on beta?

Marco was on vacation this week, but I landed it as http://hg.mozilla.org/releases/mozilla-beta/rev/61d6b5f9ee9f
Excellent - thanks Trevor.
Target Milestone: mozilla8 → mozilla7
(In reply to Trevor Saunders (:tbsaunde) from comment #12)
> (In reply to alexander surkov from comment #11)
> > Marco, could you please land this patch on beta?
> 
> Marco was on vacation this week, but I landed it as
> http://hg.mozilla.org/releases/mozilla-beta/rev/61d6b5f9ee9f

yes, thank you for doing this!
qa- as no QA fix verification needed
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.