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

Issue 2093753002: Reland: Make ChromeVox Next a setting in options page (Closed)

Created:
4 years, 6 months ago by David Tseng
Modified:
4 years, 6 months ago
Reviewers:
CC:
chromium-reviews, extensions-reviews_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, chromium-apps-reviews_chromium.org, dmazzoni+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Reland: Make ChromeVox Next a setting in options page Original cl: https://codereview.chromium.org/2079073002/ The flakeyness seen in the tests is fixed by the change to automation_custom_bindings.js. The child tree id callback can be called when the root automation node has already been attached. Natively, we send the child tree id whenever we see a child tree id within tree data which leads possibly to triggering the entire code path multiple times for the same tree. The reason why this flakeyness appears now is because Background depends on cvox.ChromeVoxBackground explicitly via goog.require. This is correct as ChromeVox Next (Background) needs some modules from Classic like prefs, keymaps, etc to be instantitated and ready. TBR=dmazzoni@chromium.org BUG=618094, 618092, 618090 CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation Committed: https://crrev.com/69cea7b2b8e5e2c1522e4d68ffacdbc77e718aa7 Cr-Commit-Position: refs/heads/master@{#401721}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+312 lines, -265 lines) Patch
M chrome/browser/resources/chromeos/chromevox/BUILD.gn View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/chromevox/background/background.js View 3 chunks +2 lines, -4 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/chromevox/background/options.html View 1 chunk +7 lines, -0 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/chromevox/background/options.js View 1 chunk +5 lines, -23 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/chromevox/background/prefs.js View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/resources/chromeos/chromevox/chromevox/background/tabs_api_handler.js View 2 chunks +4 lines, -2 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/chromevox/injected/user_commands.js View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/resources/chromeos/chromevox/common/chrome_extension_externs.js View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/resources/chromeos/chromevox/common/command_store.js View 1 chunk +3 lines, -1 line 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_util.js View 1 chunk +19 lines, -0 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js View 27 chunks +145 lines, -139 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs View 12 chunks +49 lines, -44 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/chromevox_state.js View 2 chunks +11 lines, -14 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/cursors.js View 2 chunks +7 lines, -4 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js View 7 chunks +6 lines, -14 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/editing_test.extjs View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/i_search.js View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions_test.extjs View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/notifications.js View 1 chunk +14 lines, -0 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js View 3 chunks +5 lines, -3 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js View 3 chunks +4 lines, -3 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/cvox2/background/tabs_automation_handler.js View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/resources/chromeos/chromevox/host/chrome/braille_background.js View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings.grd View 1 chunk +13 lines, -1 line 0 comments Download
M chrome/browser/resources/chromeos/chromevox/testing/chromevox_next_e2e_test_base.js View 1 chunk +1 line, -1 line 0 comments Download
M chrome/renderer/resources/extensions/automation_custom_bindings.js View 2 chunks +4 lines, -1 line 0 comments Download

Messages

Total messages: 10 (7 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2093753002/1
4 years, 6 months ago (2016-06-23 20:57:59 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 6 months ago (2016-06-23 21:25:25 UTC) #8
commit-bot: I haz the power
4 years, 6 months ago (2016-06-23 21:28:01 UTC) #10
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/69cea7b2b8e5e2c1522e4d68ffacdbc77e718aa7
Cr-Commit-Position: refs/heads/master@{#401721}

Powered by Google App Engine
This is Rietveld 408576698