media: Make media mojo services a component

chrome/browser/chrome_content_browser_client.cc

Index: chrome/browser/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index 8c40a31a94b278d92c167c464f7061fed2508d4a..ff8f5b1cb6a86b9a133fb1905c87a608c1259be0 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -310,7 +310,7 @@
 #endif
 
 #if defined(ENABLE_MOJO_MEDIA_IN_BROWSER_PROCESS)
-#include "media/mojo/services/mojo_media_application_factory.h"
+#include "media/mojo/services/mojo_media_application_factory.h"  // nogncheck

[ddorwin, 2016/05/11 23:02:27]

Why is this now required? Is this a bug in DEPS handling?

[xhwang, 2016/05/12 06:32:24]

// nogncheck is recommended for this kind of conditional includes:

https://chromium.googlesource.com/chromium/src/+/master/tools/gn/docs/referen...

But I actually don't see "nogncheck" used anywhere in chrome/.

+brettw: Do you know what's the status and recommendation of "gn check" in
chrome/?

[xhwang, 2016/05/16 05:41:20]

Per brettw's comment, I think this is the right thing to do.

 #endif
 
 using base::FileDescriptor;