Enable media android tests.

Enable media android tests and make --disable-gesture-requirement-for-media-playback default for media measurement.

Issues with nexus 4 and 7 are not repro-ing. Plus with earlier changes to media actions,
tests should skip when video error happens instead of marking the bot red.

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

[shadi, 2014-01-24 19:29:19]

PTAL

https://codereview.chromium.org/145033014/diff/1/tools/perf/benchmarks/media.py
File tools/perf/benchmarks/media.py (left):

https://codereview.chromium.org/145033014/diff/1/tools/perf/benchmarks/media....
tools/perf/benchmarks/media.py:72: # Disable MSE tests on windows 8
crbug.com/330910
These should have been removed when MSE tests were enabled.

[dtu, 2014-01-24 22:51:57]

lgtm

[commit-bot: I haz the power, 2014-01-25 00:10:00]

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

[commit-bot: I haz the power, 2014-01-25 00:10:03]

Failed to apply patch for tools/perf/benchmarks/media.py:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file tools/perf/benchmarks/media.py
  Hunk #2 FAILED at 35.
  Hunk #3 succeeded at 60 (offset 2 lines).
  1 out of 3 hunks FAILED -- saving rejects to file
tools/perf/benchmarks/media.py.rej

Patch:       tools/perf/benchmarks/media.py
Index: tools/perf/benchmarks/media.py
diff --git a/tools/perf/benchmarks/media.py b/tools/perf/benchmarks/media.py
index
cc540dffdaabcd2555184ec92dfcd79ca0eb15e9..f0a65f057e8763cb0c8846ea6069ceeccb451184
100644
--- a/tools/perf/benchmarks/media.py
+++ b/tools/perf/benchmarks/media.py
@@ -2,8 +2,6 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
-import sys
-
 from telemetry import test
 from telemetry.page import page_measurement
 
@@ -37,19 +35,12 @@ class MediaAndroid(test.Test):
   """Obtains media metrics for key user scenarios on Android."""
   test = media.Media
   tag = 'android'
-  # Disable media-tests on Android: crbug/329691
-  enabled = not sys.platform.startswith('linux')
   page_set = 'page_sets/tough_video_cases.json'
   # Exclude crowd* media files (50fps 2160p).
   options = {
       'page_filter_exclude': '.*crowd.*'
   }
 
-  def CustomizeBrowserOptions(self, options):
-    # Needed to run media actions in JS in Android.
-    options.AppendExtraBrowserArgs(
-        '--disable-gesture-requirement-for-media-playback')
-
 class MediaChromeOS(test.Test):
   """Obtains media metrics for key user scenarios on ChromeOS."""
   test = media.Media
@@ -60,16 +51,8 @@ class MediaChromeOS(test.Test):
       'page_filter_exclude': '.*crowd.*'
   }
 
-  def CustomizeBrowserOptions(self, options):
-    # Needed to run media actions in JS in Android.
-    options.AppendExtraBrowserArgs(
-        '--disable-gesture-requirement-for-media-playback')
-
 class MediaSourceExtensions(test.Test):
   """Obtains media metrics for key media source extensions functions."""
-  test = media.Media
-  # Disable MSE media-tests on Android and linux: crbug/329691
-  # Disable MSE tests on windows 8 crbug.com/330910
   test = MSEMeasurement
   page_set = 'page_sets/mse_cases.json'

[commit-bot: I haz the power, 2014-01-25 00:26:08]

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

[commit-bot: I haz the power, 2014-01-25 01:32:54]

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

[commit-bot: I haz the power, 2014-01-25 01:47:03]

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

[commit-bot: I haz the power, 2014-01-25 09:17:38]

Change committed as 247106