Merge ~abreu-alexandre/oxide:fix-non-mh-build into oxide:master

[Alexandre Abreu]

Fix non media hub build: mediahub_shim source_set conditionally considered.

For non media hub build, it seems that eventhough the source_set for mediahub_shim is only a dep if enable_mediahub is set, it is still being considered along its chain of dependencies,

[Chris Coulson (chrisccoulson)]

review: Approve

diff --git a/shared/BUILD.gn b/shared/BUILD.gn
index b9675b8..0239707 100644
--- a/shared/BUILD.gn
+++ b/shared/BUILD.gn
@@ -116,14 +116,16 @@ version_header("chrome_version") {
   output_file = "$target_gen_dir/common/chrome_version.h"
 }
 
-source_set("mediahub_shim") {
-  configs -= [ "//build/config/gcc:no_exceptions" ]
-  configs += [ "//oxide/build/config/linux:media_hub_client" ]
-
-  sources = [
-    "browser/media/mediahub_player_shim.cc",
-    "browser/media/mediahub_player_shim.h"
-  ]
+if (enable_mediahub) {
+  source_set("mediahub_shim") {
+    configs -= [ "//build/config/gcc:no_exceptions" ]
+    configs += [ "//oxide/build/config/linux:media_hub_client" ]
+
+    sources = [
+      "browser/media/mediahub_player_shim.cc",
+      "browser/media/mediahub_player_shim.h"
+    ]
+  }
 }
 
 component("shared") {