Fix onKeyAdded check causing flakiness in the tests.

Fix onKeyAdded check causing flakiness in the tests, and enabling relevant disabled test.


BUG=168252
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=175113

[shadi, 2013-01-03 22:58:30]

PTAL

[ddorwin, 2013-01-03 23:29:55]

lgtm % suggestion

https://codereview.chromium.org/11753017/diff/1/content/test/data/media/encry...
File content/test/data/media/encrypted_media_player.html (right):

https://codereview.chromium.org/11753017/diff/1/content/test/data/media/encry...
content/test/data/media/encrypted_media_player.html:15: // webkitkeyadded
happens right before playback, we check for it after
I suggest something like:
// keyadded may be fired around the start of playback; check for it after a
delay to avoid timing issues.

[shadi, 2013-01-04 00:09:19]

Enabling the disabled test as well.

PTAL again.

https://codereview.chromium.org/11753017/diff/1/content/test/data/media/encry...
File content/test/data/media/encrypted_media_player.html (right):

https://codereview.chromium.org/11753017/diff/1/content/test/data/media/encry...
content/test/data/media/encrypted_media_player.html:15: // webkitkeyadded
happens right before playback, we check for it after
On 2013/01/03 23:29:55, ddorwin wrote:
> I suggest something like:
> // keyadded may be fired around the start of playback; check for it after a
> delay to avoid timing issues.

Done.

[ddorwin, 2013-01-04 00:11:38]

lgtm

Thanks!

[commit-bot: I haz the power, 2013-01-04 00:14:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shadi@chromium.org/11753017/4002

[commit-bot: I haz the power, 2013-01-04 03:57:45]

Retried try job too often on mac_rel for step(s) browser_tests

[commit-bot: I haz the power, 2013-01-04 03:59:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shadi@chromium.org/11753017/4002

[commit-bot: I haz the power, 2013-01-04 06:54:41]

Change committed as 175113