MediaUrlInterceptor for embedders to handle custom urls.

media/base/android/media_player_bridge.cc

 // Copyright (c) 2012 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 <media/base/android/media_url_interceptor.h>

[qinmin, 2014/07/24 21:05:27]

remote this, the file is included below

[Ignacio Solla, 2014/07/25 10:35:07]

Done.

 #include "media/base/android/media_player_bridge.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop_proxy.h"
 #include "base/strings/string_util.h"
 #include "jni/MediaPlayerBridge_jni.h"
 #include "media/base/android/media_player_manager.h"
 #include "media/base/android/media_resource_getter.h"
+#include "media/base/android/media_url_interceptor.h"
 
 using base::android::ConvertUTF8ToJavaString;
 using base::android::ScopedJavaLocalRef;
 
 // Time update happens every 250ms.
 const int kTimeUpdateInterval = 250;
 
 // blob url scheme.
 const char kBlobScheme[] = "blob";
 
@@ skipping 131 matching lines @@
   SetDataSource(url_.spec());
 }
 
 void MediaPlayerBridge::SetDataSource(const std::string& url) {
   if (j_media_player_bridge_.is_null())
     return;
 
   JNIEnv* env = base::android::AttachCurrentThread();
   CHECK(env);
 
-  // Create a Java String for the URL.
-  ScopedJavaLocalRef<jstring> j_url_string = ConvertUTF8ToJavaString(env, url);
-  ScopedJavaLocalRef<jstring> j_cookies = ConvertUTF8ToJavaString(
-      env, cookies_);
-  ScopedJavaLocalRef<jstring> j_user_agent = ConvertUTF8ToJavaString(
-      env, user_agent_);
+  media::MediaUrlInterceptor* url_interceptor =
+      manager()->GetMediaUrlInterceptor();
+  int fd;
+  int64 offset;
+  int64 size;
+  if (url_interceptor && url_interceptor->Intercept(url, &fd, &offset, &size)) {
+    if (!Java_MediaPlayerBridge_setDataSourceFromFd(

[mnaganov (inactive), 2014/07/25 08:17:49]

I recommend initializing fd, offset and size to "invalid" values and adding
DCHECKs that the values are now valid, if the interceptor has returned 'true'.

[Ignacio Solla, 2014/07/25 10:35:07]

Done (and moved to private method to avoid duplication)

+        env, j_media_player_bridge_.obj(), fd, offset, size)) {
+      OnMediaError(MEDIA_ERROR_FORMAT);
+      return;
+    }
+  } else {
+    // Create a Java String for the URL.
+    ScopedJavaLocalRef<jstring> j_url_string =
+        ConvertUTF8ToJavaString(env, url);
+    ScopedJavaLocalRef<jstring> j_cookies = ConvertUTF8ToJavaString(

[qinmin, 2014/07/24 21:05:27]

j_cookies and j_user_agent declaration can be moved after the dataUri if
statement, so for data uri we don't need to create these 2 variables.

[Ignacio Solla, 2014/07/25 10:35:07]

Done.

+        env, cookies_);
+    ScopedJavaLocalRef<jstring> j_user_agent = ConvertUTF8ToJavaString(
+        env, user_agent_);
 
-  jobject j_context = base::android::GetApplicationContext();
-  DCHECK(j_context);
+    jobject j_context = base::android::GetApplicationContext();
+    DCHECK(j_context);
 
-  const std::string data_uri_prefix("data:");
-  if (StartsWithASCII(url, data_uri_prefix, true)) {
-    if (!Java_MediaPlayerBridge_setDataUriDataSource(
-        env, j_media_player_bridge_.obj(), j_context, j_url_string.obj())) {
+    const std::string data_uri_prefix("data:");
+    if (StartsWithASCII(url, data_uri_prefix, true)) {
+      if (!Java_MediaPlayerBridge_setDataUriDataSource(
+          env, j_media_player_bridge_.obj(), j_context, j_url_string.obj())) {
+        OnMediaError(MEDIA_ERROR_FORMAT);
+      }
+      return;
+    }
+
+    if (!Java_MediaPlayerBridge_setDataSource(
+        env, j_media_player_bridge_.obj(), j_context, j_url_string.obj(),
+        j_cookies.obj(), j_user_agent.obj(), hide_url_log_)) {
       OnMediaError(MEDIA_ERROR_FORMAT);
+      return;
     }
-    return;
-  }
-
-  if (!Java_MediaPlayerBridge_setDataSource(
-      env, j_media_player_bridge_.obj(), j_context, j_url_string.obj(),
-      j_cookies.obj(), j_user_agent.obj(), hide_url_log_)) {
-    OnMediaError(MEDIA_ERROR_FORMAT);
-    return;
   }
 
   request_media_resources_cb_.Run(player_id());
   if (!Java_MediaPlayerBridge_prepareAsync(env, j_media_player_bridge_.obj()))
     OnMediaError(MEDIA_ERROR_FORMAT);
 }
 
 void MediaPlayerBridge::OnDidSetDataUriDataSource(JNIEnv* env, jobject obj,
     jboolean success) {
   if (!success) {
@@ skipping 20 matching lines @@
   if (!username.empty()) {
     replacements.SetUsernameStr(username);
     if (!password.empty())
       replacements.SetPasswordStr(password);
     url_ = url_.ReplaceComponents(replacements);
   }
   ExtractMediaMetadata(url_.spec());
 }
 
 void MediaPlayerBridge::ExtractMediaMetadata(const std::string& url) {
-  manager()->GetMediaResourceGetter()->ExtractMediaMetadata(
-      url,
-      cookies_,
-      user_agent_,
-      base::Bind(&MediaPlayerBridge::OnMediaMetadataExtracted,
-                 weak_factory_.GetWeakPtr()));
+  media::MediaUrlInterceptor* url_interceptor =
+      manager()->GetMediaUrlInterceptor();
+  int fd;
+  int64 offset;
+  int64 size;
+  if (url_interceptor && url_interceptor->Intercept(url, &fd, &offset, &size)) {
+    manager()->GetMediaResourceGetter()->ExtractMediaMetadata(
+          fd, offset, size,

[mnaganov (inactive), 2014/07/25 08:17:49]

nit: 'ExtractMediaMetadata' arguments alignment is incoherent between the 'if'
statement branches.

[Ignacio Solla, 2014/07/25 10:35:07]

Done.

+          base::Bind(&MediaPlayerBridge::OnMediaMetadataExtracted,
+                     weak_factory_.GetWeakPtr()));
+  } else {
+    manager()->GetMediaResourceGetter()->ExtractMediaMetadata(
+        url, cookies_, user_agent_,
+        base::Bind(&MediaPlayerBridge::OnMediaMetadataExtracted,
+                   weak_factory_.GetWeakPtr()));
+  }
 }
 
 void MediaPlayerBridge::OnMediaMetadataExtracted(
     base::TimeDelta duration, int width, int height, bool success) {
   if (success) {
     duration_ = duration;
     width_ = width;
     height_ = height;
   }
   manager()->OnMediaMetadataChanged(
@@ skipping 249 matching lines @@
 
 GURL MediaPlayerBridge::GetFirstPartyForCookies() {
   return first_party_for_cookies_;
 }
 
 bool MediaPlayerBridge::IsSurfaceInUse() const {
   return is_surface_in_use_;
 }
 
 }  // namespace media