Use BrowserAccessibility::GetClosestPlatformObject consistently

Use BrowserAccessibility::GetClosestPlatformObject consistently

We had two code blocks that were trying to do the same thing as
BrowserAccessibility::GetClosestPlatformObject(), but they should have
just called that function instead.

Also, the two code blocks had a bug in that they considered iframe elements
to be leaf nodes, which could cause problems. GetClosestPlatformObject()
doesn't have that bug because it only searches within the current frame,
so by switching to this function we fix that bug.

BUG=652805
Committed: https://crrev.com/6d2ed04992c82f83d1bbbd39520f3e1867b6b28c
Cr-Commit-Position: refs/heads/master@{#424196}

[dmazzoni, 2016-10-10 17:46:53]

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

[commit-bot: I haz the power, 2016-10-10 17:47:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[aboxhall, 2016-10-10 17:55:28]

aboxhall@chromium.org changed reviewers:
+ aboxhall@chromium.org

[aboxhall, 2016-10-10 17:55:29]

lgtm

[commit-bot: I haz the power, 2016-10-10 18:29:50]

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

[commit-bot: I haz the power, 2016-10-10 18:29:51]

Dry run: This issue passed the CQ dry run.

[dmazzoni, 2016-10-10 18:33:37]

The CQ bit was checked by dmazzoni@chromium.org

[commit-bot: I haz the power, 2016-10-10 18:34:00]

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, 2016-10-10 18:39:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-10 18:43:27]

Description was changed from

==========
Use BrowserAccessibility::GetClosestPlatformObject consistently

We had two code blocks that were trying to do the same thing as
BrowserAccessibility::GetClosestPlatformObject(), but they should have
just called that function instead.

Also, the two code blocks had a bug in that they considered iframe elements
to be leaf nodes, which could cause problems. GetClosestPlatformObject()
doesn't have that bug because it only searches within the current frame,
so by switching to this function we fix that bug.

BUG=652805
==========

to

==========
Use BrowserAccessibility::GetClosestPlatformObject consistently

We had two code blocks that were trying to do the same thing as
BrowserAccessibility::GetClosestPlatformObject(), but they should have
just called that function instead.

Also, the two code blocks had a bug in that they considered iframe elements
to be leaf nodes, which could cause problems. GetClosestPlatformObject()
doesn't have that bug because it only searches within the current frame,
so by switching to this function we fix that bug.

BUG=652805

Committed: https://crrev.com/6d2ed04992c82f83d1bbbd39520f3e1867b6b28c
Cr-Commit-Position: refs/heads/master@{#424196}
==========

[commit-bot: I haz the power, 2016-10-10 18:43:29]

Patchset 1 (id:??) landed as
https://crrev.com/6d2ed04992c82f83d1bbbd39520f3e1867b6b28c
Cr-Commit-Position: refs/heads/master@{#424196}