MSHTML: Buffer not updated immediately when focus changes #5040

Closed
nvaccessAuto opened this Issue Apr 16, 2015 · 3 comments

1 participant

@nvaccessAuto

Reported by jteh on 2015-04-16 06:51
In MSHTML, if hidden content is shown and then immediately focused, we don't force an immediate update. This means that in browse mode, the content might not yet be present, causing incorrect reporting.

Str:
1. Open the attached test case in IE. Ensure Javascript isn't blocked.
2. Ensure you are in browse mode.
3. Press the button.

  • Expected: NVDA should say: "This will get focus."
  • Actual: NVDA says: "blank"

We fixed this for Gecko in 6f04af1, but never did it for MSHTML.

@nvaccessAuto

Attachment showFocus.html added by jteh on 2015-04-16 06:52
Description:
Test case.

@nvaccessAuto

Comment 1 by Michael Curran <mick@... on 2015-06-19 20:01
In [5cfc707]:
```CommitTicketReference repository="" revision="5cfc707f3e3ba3753c8c6eff295e5f1ff1636f56"
Merge branch 't5040' into next. Incubates #5040

Changes:
Added labels: incubating
@nvaccessAuto

Comment 2 by James Teh <jamie@... on 2015-07-10 05:26
In [85bd3c4]:
```CommitTicketReference repository="" revision="85bd3c4e78e7e98fdb1422a43e0242cca111d85b"
In browse mode in Internet Explorer and other MSHTML controls, the correct content is now reported when an element appears or changes and is immediately focused.

MSHTML vbufBackend: force an instant update of the content for onfocus events in case the focus moves to something that was newly created/changed.
Fixes #5040.

Changes:
Removed labels: incubating
State: closed
@nvaccessAuto nvaccessAuto added this to the 2015.3 milestone Nov 10, 2015
@jcsteh jcsteh added a commit that referenced this issue Nov 23, 2015
@jcsteh jcsteh In browse mode in Internet Explorer and other MSHTML controls, the co…
…rrect content is now reported when an element appears or changes and is immediately focused.

MSHTML vbufBackend: force an instant update of the content  for onfocus events in case the focus moves to something that was newly created/changed.
Fixes #5040.
85bd3c4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment