Publicize registration of MediaUrlInterceptor

content/public/browser/android/media_url_interceptor.cc

Index: content/public/browser/android/media_url_interceptor.cc
diff --git a/content/public/browser/android/media_url_interceptor.cc b/content/public/browser/android/media_url_interceptor.cc
new file mode 100644
index 0000000000000000000000000000000000000000..db9914d698f66f4db3b099ae041b3ee47734ae40
--- /dev/null
+++ b/content/public/browser/android/media_url_interceptor.cc
@@ -0,0 +1,17 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "content/public/browser/android/media_url_interceptor.h"
+
+#include "content/browser/media/android/browser_media_player_manager.h"
+
+namespace content {
+
+void MediaUrlInterceptor::Register(
+    media::MediaUrlInterceptor* media_url_interceptor) {
+  content::BrowserMediaPlayerManager::RegisterMediaUrlInterceptor(

[Ted C, 2015/03/17 21:06:39]

Any reason to not add a content/public .h for browser_media_player_manager (and
only have this method defined there)?

[mnaganov (inactive), 2015/03/18 11:17:48]

My concern was that this would imply introducing an abstract
BrowserMediaPlayerManager class and renaming the existing
BrowserMediaPlayerManager into BrowserMediaPlayerManagerImpl for no good reason.

But as the method we need to export is static, a compromise solution I've
employed is adding public/browser/android/browser_media_player_manager.h/cc, but
defining a plain function there, so no class renaming is needed. Does this look
better to you?

+      media_url_interceptor);
+}
+
+}  // namespace content