Closed
Bug 680929
Opened 13 years ago
Closed 13 years ago
empty relations returned to atk
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
mozilla9
Tracking | Status | |
---|---|---|
firefox8 | --- | fixed |
People
(Reporter: tbsaunde, Assigned: tbsaunde)
References
Details
Attachments
(1 file)
1.12 KB,
patch
|
davidb
:
review+
asa
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Regression from 641838, we add a relation to the set of relations in RefRelationSet() even if there is 0 targets. This makes orca spew lots of back traces because atleast one target was expected, and otherwise makes using firefox with orca very bad.
Assignee | ||
Comment 1•13 years ago
|
||
Assignee: nobody → trev.saunders
Status: NEW → ASSIGNED
Attachment #554881 -
Flags: review?(surkov.alexander)
Attachment #554881 -
Flags: review?(bolterbugz)
Comment 2•13 years ago
|
||
Comment on attachment 554881 [details] [diff] [review] patch r=me by inspection. Please test it out.
Attachment #554881 -
Flags: review?(surkov.alexander)
Attachment #554881 -
Flags: review?(bolterbugz)
Attachment #554881 -
Flags: review+
Assignee | ||
Comment 3•13 years ago
|
||
(In reply to David Bolter [:davidb] from comment #2) > Comment on attachment 554881 [details] [diff] [review] > patch > > r=me by inspection. Please test it out. yeah, alreadyu did, using it now to :)
Assignee | ||
Comment 4•13 years ago
|
||
landed http://hg.mozilla.org/integration/mozilla-inbound/rev/e11c851aab48
Comment 5•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/e11c851aab48
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Flags: in-testsuite?
Hardware: x86_64 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
Version: unspecified → Trunk
Assignee | ||
Comment 6•13 years ago
|
||
We don't have any machinary for testing the platform specific accessibility api code at the moment. Fortunately that code is generally a very thin wrapper.
Blocks: 641838
Flags: in-testsuite? → in-testsuite-
Comment 7•13 years ago
|
||
Not having the test framework isn't a reason to set in-testsuite-. Just keep the question mark.
Flags: in-testsuite- → in-testsuite?
Assignee | ||
Comment 8•13 years ago
|
||
Comment on attachment 554881 [details] [diff] [review] patch This regression hsould be in aurora two since 641838 landed before we branched but this hadn't yet. This patch is very low risk and and only effects linux with a accessibility app running. We've also found no regressions on trunk from it, so I doubt we'd find any before its released. (In any cse I'm pretty sure this won't introduce any regressions.) In theory we could fix the regrssion by backing out 641838. However that would be a much bigger change, and would would probably involve backing out 678189 and 672515 as well. So I think we really want to take this on aurora. p.s. I really should have asked earlier, but I've been dealing with school
Attachment #554881 -
Flags: review+ → approval-mozilla-aurora?
Comment 9•13 years ago
|
||
Comment on attachment 554881 [details] [diff] [review] patch Reinstating my r+ which mysteriously evaporated.
Attachment #554881 -
Flags: review+
Updated•13 years ago
|
Attachment #554881 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 10•13 years ago
|
||
Landed on tbsaunde's behalf: http://hg.mozilla.org/releases/mozilla-aurora/rev/2ef302863818
status-firefox8:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•