168624 - Microsoft Narrator hangs Mozilla

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect, P1)

Description

[Skewer]

Procedure: Run Microsoft Narrator (Win+U). Switch to Mozilla and try to do
something.

Expected: Narrator behaves with Mozilla similar to the way it behaves with IE.

Actual: Mozilla hangs (and doesn't recover even if Narrator is closed).

Build: 2002091309 WXP

Comment 1

[Café Internet @ CELAS]

It does not explain why mozilla hangs, but a review of Narrator at
http://www.student.oulu.fi/~vtatila/ms_access_aids.html says:

"Narrator is Microsoft's free and simple screen reader (that comes with Windows
XP) ...application support for other than pretty basic applications is pretty
poor... ...The only practical way to use IE or OE with narrator is to copy the
text you want to read to the clipboard...."

Has it been tested with newer mozilla versions? I don't have XP (or narrator)
here so I cannot test it.

Comment 2

[Lewis Jardine]

I see this behaviour in mozilla 1.5 (20031007). Additionally, narrator appears
to hang and stop reading text until mozilla is end-tasked.

Narrator is installed by default with windows XP; press winkey-u to start it. By
design, there is no way to disable starting of narrator with winkey-u.

In my opinion, this bug should be of severity greater than normal, either
Critical (crashes, loss of data, severe memory leak), or Major (Major loss of
function). In my opinion, being inaccessable to XP's screen reader is a major
loss of function, and mozilla locking up is an instance of 'crashes', and maybe
'loss of data'. Similarly, locking up the system's screen reader is probably
also a 'crash'.

Comment 3

[Asa Dotzler [:asa]]

->accessibility

Comment 4

[Aaron Leventhal]

Wow, still happening in Mozilla 1.8a2
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a2) Gecko/20040603

Comment 5

[Aaron Leventhal]

Attachment: 1) Stop infinite loops by returning CHILDID_SELF and VT_EMPTY get_accFocus() when appropriate, 2) Get rid of unnecessary GetFocusedNode() and just use gLastFocusedNode

Comment 6

[Philip K. Warren]

I get the following error building this patch on AIX+GTK2:

"/home/pkw/sb/mozilla/trunk-devel/mozilla/accessible/src/atk/nsAccessibleHyperText.cpp",
line 509.25: 1540-0274 (S) The name lookup for "gLastFocusedNode" did not find a
declaration.
gmake[3]: *** [nsAccessibleHyperText.o] Error 1

Comment 7

[Aaron Leventhal]

Attachment: Fix build error for atk by makeing nsAccessNode::gLastFocusedNode public

Comment 8

[Philip K. Warren]

Comment on attachment 151156 [details] [diff] [review]
Fix build error for atk by makeing nsAccessNode::gLastFocusedNode public

Everything looks ok to me.

Comment 9

[Darin Fisher]

Comment on attachment 151156 [details] [diff] [review]
Fix build error for atk by makeing nsAccessNode::gLastFocusedNode public

I'm sorry, I don't know this code well enough to provide you with a reasonable
review.  Since focus stuff is involved, I'm going to defer to bryner.

Comment 10

[Aaron Leventhal]

Checking in accessible/src/atk/nsAccessibleHyperText.cpp;
/cvsroot/mozilla/accessible/src/atk/nsAccessibleHyperText.cpp,v  <-- 
nsAccessibleHyperText.cpp
new revision: 1.20; previous revision: 1.19
done
Checking in accessible/src/atk/nsHTMLLinkAccessibleWrap.cpp;
/cvsroot/mozilla/accessible/src/atk/nsHTMLLinkAccessibleWrap.cpp,v  <-- 
nsHTMLLinkAccessibleWrap.cpp
new revision: 1.14; previous revision: 1.13
done
Checking in accessible/src/base/nsAccessNode.h;
/cvsroot/mozilla/accessible/src/base/nsAccessNode.h,v  <--  nsAccessNode.h
new revision: 1.15; previous revision: 1.14
done
Checking in accessible/src/base/nsAccessibilityAtomList.h;
/cvsroot/mozilla/accessible/src/base/nsAccessibilityAtomList.h,v  <-- 
nsAccessibilityAtomList.h
new revision: 1.5; previous revision: 1.4
done
Checking in accessible/src/base/nsAccessibilityService.cpp;
/cvsroot/mozilla/accessible/src/base/nsAccessibilityService.cpp,v  <-- 
nsAccessibilityService.cpp
new revision: 1.111; previous revision: 1.110
done
Checking in accessible/src/base/nsAccessible.cpp;
/cvsroot/mozilla/accessible/src/base/nsAccessible.cpp,v  <--  nsAccessible.cpp
new revision: 1.109; previous revision: 1.108
done
Checking in accessible/src/base/nsAccessible.h;
/cvsroot/mozilla/accessible/src/base/nsAccessible.h,v  <--  nsAccessible.h
new revision: 1.48; previous revision: 1.47
done
Checking in accessible/src/msaa/nsAccessibleWrap.cpp;
/cvsroot/mozilla/accessible/src/msaa/nsAccessibleWrap.cpp,v  <-- 
nsAccessibleWrap.cpp
new revision: 1.12; previous revision: 1.11
done
Checking in accessible/src/msaa/nsDocAccessibleWrap.cpp;
/cvsroot/mozilla/accessible/src/msaa/nsDocAccessibleWrap.cpp,v  <-- 
nsDocAccessibleWrap.cpp
new revision: 1.16; previous revision: 1.15
done
Checking in accessible/src/msaa/nsDocAccessibleWrap.h;
/cvsroot/mozilla/accessible/src/msaa/nsDocAccessibleWrap.h,v  <-- 
nsDocAccessibleWrap.h
new revision: 1.5; previous revision: 1.4
done
Checking in accessible/src/xul/nsXULFormControlAccessible.cpp;
/cvsroot/mozilla/accessible/src/xul/nsXULFormControlAccessible.cpp,v  <-- 
nsXULFormControlAccessible.cpp
new revision: 1.41; previous revision: 1.40
done