Enable media in default GN build.

Roll ffmpeg to pick up build fix. f620cda6 -> 04bb33e9

All hail GN. oRZ

BUG=none
TBR=brettw

Review URL: https://codereview.chromium.org/381903003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@282780 0039d316-1c4b-4281-b951-d872f2087c98

BUILD.gn

@@ -38,6 +38,7 @@ group("root") {
     "//gpu",
     "//google_apis",
     "//ipc",
+    "//media",
     "//mojo",
     "//mojo/common",
     "//mojo/environment:chromium",

DEPS

@@ -220,7 +220,7 @@ deps = {
 
   "src/third_party/ffmpeg":
     Var("chromium_git") +
-    "/chromium/third_party/ffmpeg.git@f620cda6d1661f1b2f3214d17b9a254f5c89225f",
+    "/chromium/third_party/ffmpeg.git@4b7b36f956209fad838ad87d0129130713fbb00c",
 
   "src/third_party/libjingle/source/talk":
     (Var("googlecode_url") % "webrtc") + "/trunk/talk@" +

media/BUILD.gn

@@ -31,7 +31,7 @@ if (is_android) {
 # Original conditional: (OS=="linux" or OS=="freebsd" or OS=="solaris") and embedded!=1
 use_alsa = false
 use_pulseaudio = false
-if (is_posix) {
+if (is_posix && !is_android) {
   use_alsa = true
   if (!use_cras) {
     use_pulseaudio = true
@@ -595,8 +595,6 @@ component("media") {
       "base/android/media_player_manager.h",
       "base/android/media_resource_getter.cc",
       "base/android/media_resource_getter.h",
-      "base/media.cc",
-      "base/media.h",
       "base/media_stub.cc",
       "midi/midi_manager_android.cc",
       "midi/usb_midi_device_android.cc",
@@ -622,7 +620,7 @@ component("media") {
 
   if (is_chromeos) {
     # A simple WebM encoder for animated avatars on ChromeOS.
-    sources += [ 
+    sources += [
       "formats/webm/chromeos/ebml_writer.cc",
       "formats/webm/chromeos/ebml_writer.h",
       "formats/webm/chromeos/webm_encoder.cc",
@@ -1288,48 +1286,50 @@ component("shared_memory_support") {
   deps = [ "//base" ]
 }
 
-test("ffmpeg_unittests") {
-  sources = [ "ffmpeg/ffmpeg_unittest.cc" ]
+if (media_use_ffmpeg) {
+  test("ffmpeg_unittests") {
+    sources = [ "ffmpeg/ffmpeg_unittest.cc" ]
 
-  deps = [
-     ":media",
-     ":media_test_support",
-     "//base",
-     "//base:i18n",
-     "//base/test:test_support",
-     "//testing/gtest",
-     "//third_party/ffmpeg",
-     "//ui/gfx:gfx_test_support",
-  ]
-}
+    deps = [
+       ":media",
+       ":media_test_support",
+       "//base",
+       "//base:i18n",
+       "//base/test:test_support",
+       "//testing/gtest",
+       "//third_party/ffmpeg",
+       "//ui/gfx:gfx_test_support",
+    ]
+  }
 
-test("ffmpeg_regression_tests") {
-  sources = [
-    "base/run_all_unittests.cc",
-    "ffmpeg/ffmpeg_regression_tests.cc",
-    "filters/pipeline_integration_test_base.cc",
-  ]
-  configs += [ ":media_config" ]
-  deps = [
-    ":media",
-    ":media_test_support",
-    "//base/test:test_support",
-    "//testing/gmock",
-    "//testing/gtest",
-    "//third_party/ffmpeg",
-    "//ui/gfx/geometry",
-    "//ui/gfx:gfx_test_support",
-  ]
-# TODO(ajwong): This was in the original gyp, but it seems silly.
-#        ['os_posix==1 and OS!="mac"', {
-#          'conditions': [
-#            ['use_allocator!="none"', {
-#              'dependencies': [
-#                '../base/allocator/allocator.gyp:allocator',
-#              ],
-#            }],
-#          ],
-#        }],
+  test("ffmpeg_regression_tests") {
+    sources = [
+      "base/run_all_unittests.cc",
+      "ffmpeg/ffmpeg_regression_tests.cc",
+      "filters/pipeline_integration_test_base.cc",
+    ]
+    configs += [ ":media_config" ]
+    deps = [
+      ":media",
+      ":media_test_support",
+      "//base/test:test_support",
+      "//testing/gmock",
+      "//testing/gtest",
+      "//third_party/ffmpeg",
+      "//ui/gfx/geometry",
+      "//ui/gfx:gfx_test_support",
+    ]
+  # TODO(ajwong): This was in the original gyp, but it seems silly.
+  #        ['os_posix==1 and OS!="mac"', {
+  #          'conditions': [
+  #            ['use_allocator!="none"', {
+  #              'dependencies': [
+  #                '../base/allocator/allocator.gyp:allocator',
+  #              ],
+  #            }],
+  #          ],
+  #        }],
+  }
 }
 
 if (use_x11) {