Enable EncryptedMediaTest (in content_browsertests) on Android.

Enable EncryptedMediaTest (in content_browsertests) on Android.

Encrytped media playback with <video src=...> is not supported on Android. Those
tests will not run on Android.

BUG=304956, 300035
TEST=This CL enables tests which was previously disabled.

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

[xhwang, 2013-10-07 21:41:22]

ddorwin/shadi: PTAL
qinmin: FYI

https://codereview.chromium.org/26243003/diff/4001/content/browser/media/encr...
File content/browser/media/encrypted_media_browsertest.cc (right):

https://codereview.chromium.org/26243003/diff/4001/content/browser/media/encr...
content/browser/media/encrypted_media_browsertest.cc:111:
INSTANTIATE_TEST_CASE_P(MSE_ClearKey, EncryptedMediaTest,
I prefix the tests with SRC/MSE so we can easily tell what the test is.
Previously we only get /0 /1 suffixes which is hard to read.

[ddorwin, 2013-10-07 21:46:21]

lgtm % comments

https://codereview.chromium.org/26243003/diff/4001/content/browser/media/encr...
File content/browser/media/encrypted_media_browsertest.cc (right):

https://codereview.chromium.org/26243003/diff/4001/content/browser/media/encr...
content/browser/media/encrypted_media_browsertest.cc:106: // TODO(xhwang):
Encrypted media playback with SRC is not supported on Android.
What are you planning "TODO" about this? IF nothing, maybe just remove it.

(Should the TODO be above 105?)

https://codereview.chromium.org/26243003/diff/4001/content/browser/media/encr...
content/browser/media/encrypted_media_browsertest.cc:158: // TODO(xhwang):
Encrypted media playback with SRC is not supported on Android.
Just change it to MSE. There's no real reason this is SRC other than that's
"simplest".

[shadi, 2013-10-07 22:01:56]

LGTM Thanks!

[xhwang, 2013-10-07 22:05:24]

Comments addressed. The side-by-side view still shows "bad old chunk" even if I
tried to upload for 4 times. Please click the file name to see the difference.

https://codereview.chromium.org/26243003/diff/4001/content/browser/media/encr...
File content/browser/media/encrypted_media_browsertest.cc (right):

https://codereview.chromium.org/26243003/diff/4001/content/browser/media/encr...
content/browser/media/encrypted_media_browsertest.cc:106: // TODO(xhwang):
Encrypted media playback with SRC is not supported on Android.
On 2013/10/07 21:46:22, ddorwin wrote:
> What are you planning "TODO" about this? IF nothing, maybe just remove it.
> 
> (Should the TODO be above 105?)

Done.

https://codereview.chromium.org/26243003/diff/4001/content/browser/media/encr...
content/browser/media/encrypted_media_browsertest.cc:158: // TODO(xhwang):
Encrypted media playback with SRC is not supported on Android.
On 2013/10/07 21:46:22, ddorwin wrote:
> Just change it to MSE. There's no real reason this is SRC other than that's
> "simplest".

Done.

[ddorwin, 2013-10-07 22:07:32]

lgtm

[commit-bot: I haz the power, 2013-10-08 07:20:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xhwang@chromium.org/26243003/24001

[commit-bot: I haz the power, 2013-10-08 17:53:35]

Retried try job too often on win7_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-10-08 18:02:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xhwang@chromium.org/26243003/24001

[commit-bot: I haz the power, 2013-10-08 19:23:07]

Change committed as 227565

[frankf, 2013-10-08 23:56:52]

On 2013/10/08 19:23:07, I haz the power (commit-bot) wrote:
> Change committed as 227565

These tests are failing:
http://build.chromium.org/p/chromium.fyi/builders/Android%20Tests%20%28ICS%20...

I reverted this CL. Note that content_browsertests only run on android_fyi_dbg
trybots which you didn't run.