Commit 3ab70d23 authored by Dominic Mazzoni's avatar Dominic Mazzoni Committed by Commit Bot

Clean-up after migration from ax_enums.idl to ax_enums.mojom

The idl file should be removed, and a couple of comments got missed.

TBR=dcheng@chromium.org

Bug: 650275
Change-Id: Ic36c71da5234bd9f0808f005c967147a11e45e27
Reviewed-on: https://chromium-review.googlesource.com/889043Reviewed-by: default avatarAaron Leventhal <aleventhal@chromium.org>
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532590}
parent af66ebcd
This diff is collapsed.
...@@ -452,7 +452,7 @@ enum IntAttribute { ...@@ -452,7 +452,7 @@ enum IntAttribute {
kSetSize, kSetSize,
kPosInSet, kPosInSet,
// In the case of AX_ROLE_COLOR_WELL, specifies the selected color. // In the case of Role::kColorWell, specifies the selected color.
kColorValue, kColorValue,
// Indicates the element that represents the current item within a container // Indicates the element that represents the current item within a container
...@@ -571,10 +571,10 @@ enum IntListAttribute { ...@@ -571,10 +571,10 @@ enum IntListAttribute {
// For inline text. This is the pixel position of the end of this // For inline text. This is the pixel position of the end of this
// character within the bounding rectangle of this object, in the // character within the bounding rectangle of this object, in the
// direction given by AX_ATTR_TEXT_DIRECTION. For example, for left-to-right // direction given by StringAttribute::kTextDirection. For example,
// text, the first offset is the right coordinate of the first character // for left-to-right text, the first offset is the right coordinate of
// within the object's bounds, the second offset is the right coordinate // the first character within the object's bounds, the second offset
// of the second character, and so on. // is the right coordinate of the second character, and so on.
kCharacterOffsets, kCharacterOffsets,
// Used for caching. Do not read directly. Use // Used for caching. Do not read directly. Use
......
...@@ -766,8 +766,8 @@ void Combobox::OnBlur() { ...@@ -766,8 +766,8 @@ void Combobox::OnBlur() {
} }
void Combobox::GetAccessibleNodeData(ui::AXNodeData* node_data) { void Combobox::GetAccessibleNodeData(ui::AXNodeData* node_data) {
// AX_ROLE_COMBO_BOX is for UI elements with a dropdown and an editable text // ax::mojom::Role::kComboBox is for UI elements with a dropdown and
// field, which views::Combobox does not have. Use // an editable text field, which views::Combobox does not have. Use
// ax::mojom::Role::kPopUpButton to match an HTML <select> element. // ax::mojom::Role::kPopUpButton to match an HTML <select> element.
node_data->role = ax::mojom::Role::kPopUpButton; node_data->role = ax::mojom::Role::kPopUpButton;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment