Reenable CvoxBrailleIntegrationTests that were previously flaky.

Reenable CvoxBrailleIntegrationTests that were previously flaky.

BUG=462294
Committed: https://crrev.com/69f69ee169b1a34f749186524ac5cb65af90ac7b
Cr-Commit-Position: refs/heads/master@{#320733}

[Peter Lundblad, 2015-03-16 14:10:44]

plundblad@chromium.org changed reviewers:
+ dtseng@chromium.org

[dmazzoni, 2015-03-16 15:07:11]

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

[dmazzoni, 2015-03-16 15:07:16]

https://codereview.chromium.org/1011753003/diff/1/chrome/browser/resources/ch...
File
chrome/browser/resources/chromeos/chromevox/host/chrome/braille_integration_test.unitjs
(right):

https://codereview.chromium.org/1011753003/diff/1/chrome/browser/resources/ch...
chrome/browser/resources/chromeos/chromevox/host/chrome/braille_integration_test.unitjs:85:
window.setTimeout(checkFocus, 100);
Rather than setTimeout, use a focus event listener on the window

[Peter Lundblad, 2015-03-16 15:45:52]

PTAL

dmazzoni@chromium.org writes:
> 
>
https://codereview.chromium.org/1011753003/diff/1/chrome/browser/resources/ch...
> File
>
chrome/browser/resources/chromeos/chromevox/host/chrome/braille_integration_test.unitjs
> (right):
> 
>
https://codereview.chromium.org/1011753003/diff/1/chrome/browser/resources/ch...
>
chrome/browser/resources/chromeos/chromevox/host/chrome/braille_integration_test.unitjs:85:
> window.setTimeout(checkFocus, 100);
> Rather than setTimeout, use a focus event listener on the window
> 

Done, assuming that there's no time window where is focused by not
document.

Thanks,
//Peter

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[dmazzoni, 2015-03-16 15:49:28]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2015-03-16 15:49:29]

lgtm

I'm pretty sure window.onfocus and document.hasFocus() are the same so this
looks right to me

[commit-bot: I haz the power, 2015-03-16 15:50:00]

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

[commit-bot: I haz the power, 2015-03-16 16:10:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-16 16:11:19]

Patchset 2 (id:??) landed as
https://crrev.com/69f69ee169b1a34f749186524ac5cb65af90ac7b
Cr-Commit-Position: refs/heads/master@{#320733}