Fix and enable mojo:media_pipeline_integration_apptests.

mojo/tools/data/apptests

 # This file contains a list of Mojo gtest unit tests.
 #
 # This must be valid Python. It may use the |config| global that will be a
 # mopy.config.Config object, and must set a |tests| global that will contain the
 # list of tests to run.
 #
 # The entries in |tests| are dictionaries of the following form:
 #   {
 #     # Required URL for apptest.
 #     'test': 'mojo:test_app_url',
@@ skipping 65 matching lines @@
       'test': 'mojo:mandoline_browser_apptests',
       'type': 'gtest_isolated',
     },
     # TODO(sky): reenable when 569367 is fixed.
 #    {
 #      'test': 'mojo:mash_wm_apptests',
 #      'type': 'gtest_isolated',
 #      'args': ['--use-x11-test-config',
 #               '--override-use-gl-with-osmesa-for-tests']
 #    },
-    # TODO(xhwang): Fix and enable mojo:media_pipeline_integration_apptests.
-    # http://crbug.com/501417
     {
       'test': 'mojo:media_apptests',
       'type': 'gtest_isolated',
     },
+    {
+      'test': 'mojo:media_pipeline_integration_apptests',

[sky, 2016/01/21 20:31:39]

Is this depended on by the target the builders look for to determine if
mojo_apptests should run?

[jrummell, 2016/01/26 19:47:41]

I think so, assuming media_apptests was done properly. The dependency has
previously been checked into media/mojo/services/BUILD.gn (in group("tests"),
the first file in this CL, lines 258-270).

I also checked, and the tests did run on the trybots (for
linux_chromium_rel_ng).

[sky, 2016/01/26 23:27:23]

I think it safer to update the mojo_apptests in src/BUILD.gn (I recently added
it, there is a comment in this file to that effect if you sync to tip of tree).

[jrummell, 2016/01/27 00:55:27]

Done.

+      'type': 'gtest_isolated',
+    },
     # TODO(crbug.com/560626): Fix and enable mus_apptests on Android.
     mus_apptests,
     {
       'test': 'mojo:sql_apptests',
       'type': 'gtest_isolated',
     },
     # TODO(sky): enable when works (http://crbug.com/577274 and likely
     # http://crbug.com/569367).
 #    {
 #      'test': 'mojo:views_apptests',
 #      'type': 'gtest_isolated',
 #      'args': ['--use-x11-test-config',
 #               '--override-use-gl-with-osmesa-for-tests', '--enable-logging']
 #    },
     {
       'test': 'mojo:web_view_apptests',
       'type': 'gtest_isolated',
       'args': ['--use-x11-test-config',
                '--override-use-gl-with-osmesa-for-tests']
     },
   ]