Nvda: Say-all repeats some lines.

[feerrenrut]

Steps to reproduce:

Using alpha-20175,9967903a or later.

1. Open #10885 with Chrome
2. Jump by heading with 'h' until you get to the heading "Description of how this pull request fixes the issue"
3. Start say-all (NVDA+down arrow)

The problem does not require changing the Attempt to cancel speech for expired focus events setting.

Actual behavior:

After a few lines, the following is read twice:
"Tested moving through emails (and across table boundaries) in Gmail"

Expected behavior:

The line should be read once.

System configuration

NVDA installed/portable/running from source:

Installed.

NVDA version:

Alpha build, post #10885 merge: alpha-20175,9967903a
Does not repro with prior alpha: alpha-20168,1cf36fbe

Windows version:

Win 10: 1909

Name and version of other software in use when reproducing the issue:

Chrome 81

Other questions

If addons are disabled, is your problem still occuring?

No addons

[Mary5958]

The same can be reproduced when reading pages in Firefox and with emails in HTML format in Thunderbird.

[gregjozk]

Hello,

one more test case:

• open addons.nvda-project.org
• choose addon developer toolkit
• start say all
• nvda repeats many lines in a list of keyboard shortcuts etc

[amirsol81]

Hi. NVDA also repeats sentences until a "period" is encountered. For instance, open https://www.zdnet.com/article/i-just-dumped-one-of-apples-worst-most-beautiful-products
NVDA repeats many sentences in continuous reading. The first repeated one is "It all must have seemed like a good idea at the time." It is located after the first heading. Simply place the cursor on the heading, or even on the last character of the heading, and press Insert+Down to initiate continuous reading.
This is with NVDA Version alpha-20204,0f54e8ce and Chrome 81/IE 11 on Windows 10 Version 2004.

[tapper82]

Hi same on my windows 10 64.
NVDA
Version: alpha-20206,dba33431
Firefox 78.1

[tapper82]

@derekriemer @josephsl Can you reproduce this?

[Adriani90]

@tapper82 this issue is reproducible and it is already known. See the PR referenced in the issue description for more details.

[leonardder]

This is getting a bit annoying.

@feerrenrut not trying to force you at all, but could a revert be considered if a fix can't be available soon?

[josephsl]

Hi,

Seconded - would like to get this done before beta freeze please.

Thanks.

[tapper82]

+1

[Neurrone]

Agreed with the other calls to revert this, because it is really quite annoying when using say all.

[dingpengyu]

Used version: alpha-20317,6bf3ae5f
has been tested in Simplified Chinese, the problem has been solved.
testing.log

[tapper82]

Come on now this is getting to be a joke. How hard is it to getting it in to your heads that we cant read webpages properly. This is not a bug that can stay in NVDA. It is not fixt. Fix your code or back it out for god sake!

[codeofdusk]

Apparently fixed in #11245.

[amirsol81]

No, I can still duplicate it with Alpha-20317.

[leonardder]

While I agree that this is an annoying issue, Two points:

1. There is an open pr that will solve this, #11245

2. This code is only in Alpha snapshots. Note the following from the development snapshots page:

   Alpha snapshots contain bleeding-edge code. These snapshots are generated straight from NVDA's mainline (master) branch. Once a feature is implemented, it will be merged straight into this branch. Although there are some automated tests, code in this branch may have not been tested at all by users. This should never be used in a production environment!

Comments with a harsh tone of voice like the one written by @tapper82 really aren't helpful in this situation. Please stay kind to each other.

[leonardder]

11245 is not yet merged...

[amirsol81]

Looking forward to it then.