304871 - Hitting enter with screen reader active will not open new window links

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Aaron Leventhal]

Steps:
1. Go to http://www.firefox-browser.de/
2. With screen reader active in browse mode, tab to link
3. Hit Enter

An alert pops up saying that a popup has been blocked. New windows should always
open as a result of an explicit user action like a key press or mouse click

Comment 1

[Aaron Leventhal]

Attachment: Use different interfaces for simulating a click in DoAction()

Comment 2

[Johnny Stenback (:jst)]

Comment on attachment 192853 [details] [diff] [review]
Use different interfaces for simulating a click in DoAction()

sr=jst

Comment 3

[timeless]

Comment on attachment 192853 [details] [diff] [review]
Use different interfaces for simulating a click in DoAction()

  NS_RELEASE(gDoCommandTimer);
drop this:
  gDoCommandTimer = nsnull;

Comment 4

[Aaron Leventhal]

Fixed in both branch and trunk.

Comment 5

[Aaron Leventhal]

Attachment: Missed part of previous patch --- an important part :)

Comment 6

[Boris Zbarsky [:bzbarsky]]

I won't be able to get to this for at least two weeks.

Comment 7

[Aaron Leventhal]

Comment on attachment 193157 [details] [diff] [review]
Missed part of previous patch --- an important part :)

Very simple patch!

Comment 8

[Johnny Stenback (:jst)]

Comment on attachment 193157 [details] [diff] [review]
Missed part of previous patch --- an important part :)

sr=jst

Comment 9

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1527381