Add support for MediaUrlInterceptor

content/browser/media/android/media_player_renderer.cc

 // Copyright 2016 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/browser/media/android/media_player_renderer.h"
 
 #include <memory>
 
 #include "content/browser/android/scoped_surface_request_manager.h"
 #include "content/browser/media/android/media_resource_getter_impl.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/storage_partition.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/content_client.h"
 
 // TODO(tguilbert): Remove this ID once MediaPlayerManager has been deleted
 // and MediaPlayerBridge updated. See comment in header file.
 constexpr int kUnusedAndIrrelevantPlayerId = 0;
 
 namespace content {
 
+static media::MediaUrlInterceptor* media_url_interceptor_ = nullptr;

[watk, 2016/10/28 22:34:48]

This should have the g_ prefix (for global). And I believe the more common
convention is to use an anonymous namespace and remove the static keyword. e.g.,
https://cs.chromium.org/chromium/src/base/logging.cc?sq=package:chromium&l=82...

+
 MediaPlayerRenderer::MediaPlayerRenderer(RenderFrameHost* render_frame_host)
     : render_frame_host_(render_frame_host),
       has_error_(false),
       weak_factory_(this) {}
 
 MediaPlayerRenderer::~MediaPlayerRenderer() {
   CancelScopedSurfaceRequest();
 }
 
 void MediaPlayerRenderer::Initialize(
@@ skipping 122 matching lines @@
     storage::FileSystemContext* file_system_context =
         partition ? partition->GetFileSystemContext() : nullptr;
     media_resource_getter_.reset(
         new MediaResourceGetterImpl(context, file_system_context, host->GetID(),
                                     render_frame_host_->GetRoutingID()));
   }
   return media_resource_getter_.get();
 }
 
 media::MediaUrlInterceptor* MediaPlayerRenderer::GetMediaUrlInterceptor() {
-  // TODO(tguilbert): Offer a RegisterMediaUrlInterceptor equivalent for use in
-  // webview. See crbug.com/636588.
-  return nullptr;
+  return media_url_interceptor_;
 }
 
 void MediaPlayerRenderer::OnTimeUpdate(int player_id,
                                        base::TimeDelta current_timestamp,
                                        base::TimeTicks current_time_ticks) {}
 
 void MediaPlayerRenderer::OnMediaMetadataChanged(int player_id,
                                                  base::TimeDelta duration,
                                                  int width,
                                                  int height,
@@ skipping 56 matching lines @@
 }
 
 void MediaPlayerRenderer::OnDecoderResourcesReleased(int player_id) {
   // Since we are not using a pool of MediaPlayerAndroid instances, this
   // function is not relevant.
 
   // TODO(tguilbert): Throttle requests, via exponential backoff.
   // See crbug.com/636615.
 }
 
+// static
+void MediaPlayerRenderer::RegisterMediaUrlInterceptor(
+    media::MediaUrlInterceptor* media_url_interceptor) {
+  media_url_interceptor_ = media_url_interceptor;
+}
+
 void MediaPlayerRenderer::CancelScopedSurfaceRequest() {
   if (!surface_request_token_)
     return;
 
   ScopedSurfaceRequestManager::GetInstance()->UnregisterScopedSurfaceRequest(
       surface_request_token_);
   surface_request_token_ = base::UnguessableToken();
 }
 
 }  // namespace content