Closed
Bug 423681
Opened 16 years ago
Closed 16 years ago
Text events broken in text fields -- effective 3/15/08
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: aaronlev)
References
Details
(Keywords: access, regression)
Attachments
(1 file)
1.45 KB,
patch
|
sicking
:
review+
bzbarsky
:
superreview+
mtschrep
:
approval1.9+
|
Details | Diff | Splinter Review |
Go to http://www.google.com Use accevent to watch events Type letters TEXT_INSERTED events after the first get errors like the following: TEXT_INSERTED Name=[Error getting string:80070057 - The parameter is incorrect.] Role=[Error getting role:80070057 - The parameter is incorrect.] State=[Error getting state:80070057 - The parameter is incorrect.] Attribs@@= NewText=
Flags: blocking1.9?
Assignee | ||
Comment 1•16 years ago
|
||
Broken for the first time in March 15 builds.
Summary: Text events broken in text fields → Text events broken in text fields -- effective 3/15/08
Assignee | ||
Comment 2•16 years ago
|
||
Regression range seems to point to bug 419416: http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2008-03-14+00%3A00&maxdate=2008-03-15+08%3A00&cvsroot=%2Fcvsroot
Assignee | ||
Comment 3•16 years ago
|
||
Actually this was caused by security fix bug 401155. It's preventing a11y requests from being handled.
Assignee | ||
Comment 4•16 years ago
|
||
Attachment #310281 -
Flags: superreview?
Attachment #310281 -
Flags: review?
Assignee | ||
Updated•16 years ago
|
Attachment #310281 -
Flags: superreview?(bzbarsky)
Attachment #310281 -
Flags: superreview?
Attachment #310281 -
Flags: review?(jonas)
Attachment #310281 -
Flags: review?
*please* mark regressions as dependencies. Even if they are just suspected. It is very easy to loose track of them otherwise.
Blocks: 401155
Comment 6•16 years ago
|
||
Comment on attachment 310281 [details] [diff] [review] Let a11y events through even during script unsafe times -- they are coming from win32 Looks ok.
Attachment #310281 -
Flags: superreview?(bzbarsky) → superreview+
So are you sure that this code won't end up firing any events that content code can catch?
Assignee | ||
Comment 8•16 years ago
|
||
No, there might be a case where content code can catch it, but this a11y event only is fired when a screen reader or similar AT is running. I'll look into that issue but we need to get the accessibility builds going again.
Assignee | ||
Comment 9•16 years ago
|
||
Actually for content code to catch it, it would require either require that: 1) A Windows executable caught an MSAA event and in-process used an IAccessible method to perform an action or focus a node 2) The content had privileges to look at accessibility interfaces and catch events that way
Comment on attachment 310281 [details] [diff] [review] Let a11y events through even during script unsafe times -- they are coming from win32 So I take it that's a "no"? If so looks fine to me, other than change NS_WARNING to NS_ERROR as we do want to assert.
Attachment #310281 -
Flags: review?(jonas) → review+
Assignee | ||
Updated•16 years ago
|
Attachment #310281 -
Flags: review?
Assignee | ||
Updated•16 years ago
|
Attachment #310281 -
Flags: approval1.9?
Assignee | ||
Updated•16 years ago
|
Attachment #310281 -
Flags: review?
Updated•16 years ago
|
Attachment #310281 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Updated•16 years ago
|
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Flags: blocking1.9?
You need to log in
before you can comment on or make changes to this bug.
Description
•