Improvements to our windows accessibility tree testing.

Improvements to our windows accessibility tree testing.

This cl makes two changes. First it makes private some BrowserAccessibilityWin
accessors. Clients should use the public COM API when accessing state/roles/etc.

Second, I am also modifying AccessibilityTreeFormatterWin to use the public COM
API. This works for the most part, but the formatter still needs access to the
role name as well as some positioning information.

BUG=733437
Review-Url: https://codereview.chromium.org/2938113002
Cr-Commit-Position: refs/heads/master@{#479620}
Committed: https://chromium.googlesource.com/chromium/src/+/5859af4f6c0c07cf97069fdbe5d40afb40a7ad17

[dougt, 2017-06-14 23:35:01]

The CQ bit was checked by dougt@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-14 23:35:14]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-15 00:30:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-15 00:30:34]

Dry run: This issue passed the CQ dry run.

[dougt, 2017-06-15 00:32:53]

dougt@chromium.org changed reviewers:
+ dmazzoni@chromium.org

[dougt, 2017-06-15 00:32:53]

dougt@chromium.org changed reviewers:
+ dmazzoni@chromium.org

[dougt, 2017-06-15 00:32:54]

dmazzoni, ptal.

[dougt, 2017-06-15 00:32:55]

dmazzoni, ptal.

[dmazzoni, 2017-06-15 06:07:16]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2017-06-15 06:07:17]

lgtm

[commit-bot: I haz the power, 2017-06-15 06:07:30]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-15 06:12:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497506836673510, "parent_rev":
"9fa92c5ef42720afd219a0a8bbb8d13e6bba2157", "commit_rev":
"aeb49740a69d59b123dfa0783cb5365dfdeca8db"}

[commit-bot: I haz the power, 2017-06-15 06:12:19]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497506836673510, "parent_rev":
"8bbfb2b22c1656eeaecd850850a279cf1e9dd730", "commit_rev":
"5859af4f6c0c07cf97069fdbe5d40afb40a7ad17"}

[commit-bot: I haz the power, 2017-06-15 06:12:32]

Description was changed from

==========
Improvements to our windows accessibility tree testing.

This cl makes two changes. First it makes private some BrowserAccessibilityWin
accessors. Clients should use the public COM API when accessing state/roles/etc.

Second, I am also modifying AccessibilityTreeFormatterWin to use the public COM
API. This works for the most part, but the formatter still needs access to the
role name as well as some positioning information.

BUG=733437
==========

to

==========
Improvements to our windows accessibility tree testing.

This cl makes two changes. First it makes private some BrowserAccessibilityWin
accessors. Clients should use the public COM API when accessing state/roles/etc.

Second, I am also modifying AccessibilityTreeFormatterWin to use the public COM
API. This works for the most part, but the formatter still needs access to the
role name as well as some positioning information.

BUG=733437

Review-Url: https://codereview.chromium.org/2938113002
Cr-Commit-Position: refs/heads/master@{#479620}
Committed:
https://chromium.googlesource.com/chromium/src/+/5859af4f6c0c07cf97069fdbe5d4...
==========

[commit-bot: I haz the power, 2017-06-15 06:12:36]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5859af4f6c0c07cf97069fdbe5d4...