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

Side by Side Diff: content/browser/accessibility/browser_accessibility_win.cc

Issue 703123002: AXRoleDescriptions are not correctly exposed for dl, dd, dt tags (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebasing changes Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/accessibility/browser_accessibility_win.h" 5 #include "content/browser/accessibility/browser_accessibility_win.h"
6 6
7 #include <UIAutomationClient.h> 7 #include <UIAutomationClient.h>
8 #include <UIAutomationCoreApi.h> 8 #include <UIAutomationCoreApi.h>
9 9
10 #include "base/strings/string_number_conversions.h" 10 #include "base/strings/string_number_conversions.h"
(...skipping 3461 matching lines...) Expand 10 before | Expand all | Expand 10 after
3472 ia_role_ = ROLE_SYSTEM_GROUPING; 3472 ia_role_ = ROLE_SYSTEM_GROUPING;
3473 ia2_role_ = IA2_ROLE_SECTION; 3473 ia2_role_ = IA2_ROLE_SECTION;
3474 break; 3474 break;
3475 case ui::AX_ROLE_DEFINITION: 3475 case ui::AX_ROLE_DEFINITION:
3476 role_name_ = html_tag; 3476 role_name_ = html_tag;
3477 ia2_role_ = IA2_ROLE_PARAGRAPH; 3477 ia2_role_ = IA2_ROLE_PARAGRAPH;
3478 ia_state_ |= STATE_SYSTEM_READONLY; 3478 ia_state_ |= STATE_SYSTEM_READONLY;
3479 break; 3479 break;
3480 case ui::AX_ROLE_DESCRIPTION_LIST_DETAIL: 3480 case ui::AX_ROLE_DESCRIPTION_LIST_DETAIL:
3481 role_name_ = html_tag; 3481 role_name_ = html_tag;
3482 ia_role_ = ROLE_SYSTEM_TEXT;
3482 ia2_role_ = IA2_ROLE_PARAGRAPH; 3483 ia2_role_ = IA2_ROLE_PARAGRAPH;
3483 ia_state_ |= STATE_SYSTEM_READONLY;
3484 break; 3484 break;
3485 case ui::AX_ROLE_DESCRIPTION_LIST: 3485 case ui::AX_ROLE_DESCRIPTION_LIST:
3486 role_name_ = html_tag; 3486 role_name_ = html_tag;
3487 ia_role_ = ROLE_SYSTEM_LIST; 3487 ia_role_ = ROLE_SYSTEM_LIST;
3488 ia_state_ |= STATE_SYSTEM_READONLY; 3488 ia_state_ |= STATE_SYSTEM_READONLY;
3489 break; 3489 break;
3490 case ui::AX_ROLE_DESCRIPTION_LIST_TERM: 3490 case ui::AX_ROLE_DESCRIPTION_LIST_TERM:
3491 ia_role_ = ROLE_SYSTEM_LISTITEM; 3491 ia_role_ = ROLE_SYSTEM_LISTITEM;
3492 ia_state_ |= STATE_SYSTEM_READONLY; 3492 ia_state_ |= STATE_SYSTEM_READONLY;
3493 break; 3493 break;
(...skipping 364 matching lines...) Expand 10 before | Expand all | Expand 10 after
3858 // The role should always be set. 3858 // The role should always be set.
3859 DCHECK(!role_name_.empty() || ia_role_); 3859 DCHECK(!role_name_.empty() || ia_role_);
3860 3860
3861 // If we didn't explicitly set the IAccessible2 role, make it the same 3861 // If we didn't explicitly set the IAccessible2 role, make it the same
3862 // as the MSAA role. 3862 // as the MSAA role.
3863 if (!ia2_role_) 3863 if (!ia2_role_)
3864 ia2_role_ = ia_role_; 3864 ia2_role_ = ia_role_;
3865 } 3865 }
3866 3866
3867 } // namespace content 3867 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/accessibility/browser_accessibility_cocoa.mm ('k') | content/test/data/accessibility/dl.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698