Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(50)

Issue 2571963002: Merge to m56: Fix issues with sending tts queue lots of utterances (Closed)

Created:
4 years ago by David Tseng
Modified:
4 years ago
Reviewers:
David Tseng
CC:
chromium-reviews, alemate+watch_chromium.org, oshima+watch_chromium.org, aboxhall+watch_chromium.org, nektar+watch_chromium.org, yuzo+watch_chromium.org, je_julie, arv+watch_chromium.org, dtseng+watch_chromium.org, dmazzoni+watch_chromium.org
Target Ref:
refs/pending/branch-heads/2924
Project:
chromium
Visibility:
Public.

Description

Merge to m56: Fix issues with sending tts queue lots of utterances - $descendants for div's can be very expensive. For example, in Gmail, there is a focusable div that wraps almost the entire contents of the page. We end up generating Output for everything under the div and lag significantly when focus lands here. Fix this by only taking the text content of divs as one large string. $descendants had all output including rich formatting. - live region output on alerts causes double output (and similar lag). For example, pressing '?' in Gmail triggers hundreds of runs inside of LiveRegion. Fix this by ignoring live regions set on alert nodes. TEST=navigate in Gmail. BUG=672955 TBR=dtseng@chromium.org CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2563173002 Cr-Commit-Position: refs/heads/master@{#437925} (cherry picked from commit a740f7ec72a6bf88e269f52c5561dd57ffde1589) Committed: https://chromium.googlesource.com/chromium/src/+/d0f1e3d1b5534b5b963f26f4e9e096ef3bc44bee

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -11 lines) Patch
M chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions.js View 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions_test.extjs View 1 chunk +1 line, -3 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js View 5 chunks +11 lines, -7 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
David Tseng
4 years ago (2016-12-13 23:45:47 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
d0f1e3d1b5534b5b963f26f4e9e096ef3bc44bee (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698