Update EME playback-multiple-sessions test to avoid flakiness

Update EME playback-multiple-sessions test to avoid flakiness

Update the test to not call generateRequest() for either stream
until both "onencrypted" events are received. This is to avoid
the occasional race condition where the first "message" event
arrives before the second "onencrypted" event, and thus messes
up the expected-output log comparison.

BUG=407993
TEST=modified test still pass

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183842

[jrummell, 2014-10-14 21:44:42]

jrummell@chromium.org changed reviewers:
+ ddorwin@chromium.org, sandersd@chromium.org, xhwang@chromium.org

[jrummell, 2014-10-14 21:44:43]

PTAL.

[sandersd (OOO until July 31), 2014-10-14 22:31:05]

lgtm

[ddorwin, 2014-10-16 16:14:32]

LGTM % comments

https://codereview.chromium.org/658633002/diff/1/LayoutTests/media/encrypted-...
File
LayoutTests/media/encrypted-media/encrypted-media-playback-multiple-sessions.html
(right):

https://codereview.chromium.org/658633002/diff/1/LayoutTests/media/encrypted-...
LayoutTests/media/encrypted-media/encrypted-media-playback-multiple-sessions.html:60:
// "message" event from occurring before both "onencrypted"
nit: This comment would probably be better before line 39 to explain why we are
saving the events.

https://codereview.chromium.org/658633002/diff/1/LayoutTests/media/encrypted-...
LayoutTests/media/encrypted-media/encrypted-media-playback-multiple-sessions.html:61:
// events are received).
Include the reason: expected text

[jrummell, 2014-10-16 17:42:56]

Thanks for the reviews.

https://codereview.chromium.org/658633002/diff/1/LayoutTests/media/encrypted-...
File
LayoutTests/media/encrypted-media/encrypted-media-playback-multiple-sessions.html
(right):

https://codereview.chromium.org/658633002/diff/1/LayoutTests/media/encrypted-...
LayoutTests/media/encrypted-media/encrypted-media-playback-multiple-sessions.html:60:
// "message" event from occurring before both "onencrypted"
On 2014/10/16 16:14:32, ddorwin wrote:
> nit: This comment would probably be better before line 39 to explain why we
are
> saving the events.

Done.

https://codereview.chromium.org/658633002/diff/1/LayoutTests/media/encrypted-...
LayoutTests/media/encrypted-media/encrypted-media-playback-multiple-sessions.html:61:
// events are received).
On 2014/10/16 16:14:32, ddorwin wrote:
> Include the reason: expected text

Done.

[jrummell, 2014-10-16 17:43:31]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2014-10-16 17:44:16]

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

[commit-bot: I haz the power, 2014-10-16 18:47:52]

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

[commit-bot: I haz the power, 2014-10-16 18:47:53]

Try jobs failed on following builders:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[jrummell, 2014-10-16 18:55:55]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2014-10-16 18:56:59]

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

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

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

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

Try jobs failed on following builders:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[jrummell, 2014-10-16 23:18:15]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2014-10-16 23:19:14]

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

[commit-bot: I haz the power, 2014-10-16 23:20:12]

Committed patchset #2 (id:20001) as 183842