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

Issue 2641563003: Relax another case where screen reader support was enabled. (Closed)

Created:
3 years, 11 months ago by dmazzoni
Modified:
3 years, 11 months ago
Reviewers:
aboxhall
CC:
chromium-reviews, aboxhall+watch_chromium.org, nektar+watch_chromium.org, jam, yuzo+watch_chromium.org, je_julie, darin-cc_chromium.org, dmazzoni+watch_chromium.org, dtseng+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Relax another case where screen reader support was enabled. See previous changes on this bug for context. Take another case where we were enabling full accessibility support, and only enable limited web accessibility support instead. Full screen reader support will be enabled automatically as more advanced APIs are accessed. This change only affects Windows. BUG=672205, 644647 Review-Url: https://codereview.chromium.org/2641563003 Cr-Commit-Position: refs/heads/master@{#444384} Committed: https://chromium.googlesource.com/chromium/src/+/a83d31c69942f949f7970e607f8394e060444ecd

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -1 line) Patch
M content/browser/accessibility/browser_accessibility_manager_win.cc View 1 chunk +6 lines, -1 line 0 comments Download

Messages

Total messages: 12 (8 generated)
dmazzoni
3 years, 11 months ago (2017-01-17 23:31:56 UTC) #4
aboxhall
lgtm
3 years, 11 months ago (2017-01-17 23:33:46 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2641563003/1
3 years, 11 months ago (2017-01-18 16:22:24 UTC) #9
commit-bot: I haz the power
3 years, 11 months ago (2017-01-18 16:26:35 UTC) #12
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/a83d31c69942f949f7970e607f83...

Powered by Google App Engine
This is Rietveld 408576698