Use polling to check focused node instead of sleep.

Use polling to check focused node instead of sleep.

BUG=477632
Committed: https://crrev.com/188e3ae5a5fcf75d937751b62c5e0ade3509b74f
Cr-Commit-Position: refs/heads/master@{#329713}

[bcwhite, 2015-05-13 16:26:45]

bcwhite@chromium.org changed reviewers:
+ aurimas@chromium.org, nyquist@chromium.org

[bcwhite, 2015-05-13 16:26:45]

aurimas@chromium.org: Please review changes in 
ImeTest

nyquist@chromium.org: Please review changes in 
DOMUtils

[nyquist, 2015-05-13 16:50:43]

https://codereview.chromium.org/1143443002/diff/20001/content/public/test/and...
File
content/public/test/android/javatests/src/org/chromium/content/browser/test/util/DOMUtils.java
(right):

https://codereview.chromium.org/1143443002/diff/20001/content/public/test/and...
content/public/test/android/javatests/src/org/chromium/content/browser/test/util/DOMUtils.java:158:
public static String getFocusedNode(final WebContents webContents)
Nit: Unnecessary final.

https://codereview.chromium.org/1143443002/diff/20001/content/public/test/and...
content/public/test/android/javatests/src/org/chromium/content/browser/test/util/DOMUtils.java:168:
if (id != null && id.length() >= 2 && id.charAt(0) == '"') {
Could you add a comment here to describe that you're stripping the first and
last quote?
Also, would there be any reason to check the last character for '"' as well?

[aurimas (slooooooooow), 2015-05-13 17:14:32]

ImeTest.java LTGM

[bcwhite, 2015-05-13 17:24:28]

On 2015/05/13 17:14:32, aurimas wrote:
> ImeTest.java LTGM

Could I get a few letters exchanged...?  ;-)

[aurimas (slooooooooow), 2015-05-13 17:37:14]

On 2015/05/13 at 17:24:28, bcwhite wrote:
> On 2015/05/13 17:14:32, aurimas wrote:
> > ImeTest.java LTGM
> 
> Could I get a few letters exchanged...?  ;-)

Bah, LGTM

[nyquist, 2015-05-13 17:45:12]

lgtm, but I'm no content owner.

[bcwhite, 2015-05-13 18:19:34]

On 2015/05/13 17:45:12, nyquist wrote:
> lgtm, but I'm no content owner.

You're in content/public/test/android/OWNERS which is the closest OWNERS file. 
Do you want me to ask someone else?

[nyquist, 2015-05-13 20:09:52]

Oh my. No, that's fine. Feel free to go ahead and submit.

[bcwhite, 2015-05-13 20:11:16]

The CQ bit was checked by bcwhite@chromium.org

[bcwhite, 2015-05-13 20:11:17]

The patchset sent to the CQ was uploaded after l-g-t-m from
nyquist@chromium.org, aurimas@chromium.org
Link to the patchset: https://codereview.chromium.org/1143443002/#ps60001
(title: "add focus check for another test")

[commit-bot: I haz the power, 2015-05-13 20:11:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1143443002/60001

[commit-bot: I haz the power, 2015-05-13 20:54:19]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-13 20:55:20]

Patchset 4 (id:??) landed as
https://crrev.com/188e3ae5a5fcf75d937751b62c5e0ade3509b74f
Cr-Commit-Position: refs/heads/master@{#329713}