Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1216)

Unified Diff: content/browser/media/android/media_resource_getter_impl.cc

Issue 130363002: Pass Chrome user agent string to Android media player (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: pass the user agent from mp-bridge to media-resource-getter Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/media/android/media_resource_getter_impl.cc
diff --git a/content/browser/media/android/media_resource_getter_impl.cc b/content/browser/media/android/media_resource_getter_impl.cc
index 53a5094dce4b8ace26192018f8e6ef633bf44ef4..afd01e114e758a2d0314224234680bc6f7bf87b8 100644
--- a/content/browser/media/android/media_resource_getter_impl.cc
+++ b/content/browser/media/android/media_resource_getter_impl.cc
@@ -22,6 +22,9 @@
#include "net/url_request/url_request_context_getter.h"
#include "url/gurl.h"
+using base::android::ConvertUTF8ToJavaString;
+using base::android::ScopedJavaLocalRef;
+
namespace content {
static void ReturnResultOnUIThread(
@@ -35,17 +38,21 @@ static void ReturnResultOnUIThread(
// thread to run the callback function.
static void GetMediaMetadata(
const std::string& url, const std::string& cookies,
+ const std::string& user_agent,
const media::MediaResourceGetter::ExtractMediaMetadataCB& callback) {
JNIEnv* env = base::android::AttachCurrentThread();
- base::android::ScopedJavaLocalRef<jstring> j_url_string =
- base::android::ConvertUTF8ToJavaString(env, url);
- base::android::ScopedJavaLocalRef<jstring> j_cookies =
- base::android::ConvertUTF8ToJavaString(env, cookies);
+ ScopedJavaLocalRef<jstring> j_url_string = ConvertUTF8ToJavaString(env, url);
+ ScopedJavaLocalRef<jstring> j_cookies = ConvertUTF8ToJavaString(env, cookies);
jobject j_context = base::android::GetApplicationContext();
- base::android::ScopedJavaLocalRef<jobject> j_metadata =
- Java_MediaResourceGetter_extractMediaMetadata(
- env, j_context, j_url_string.obj(), j_cookies.obj());
+ ScopedJavaLocalRef<jstring> j_user_agent = ConvertUTF8ToJavaString(
+ env, user_agent);
+ ScopedJavaLocalRef<jobject> j_metadata =
+ Java_MediaResourceGetter_extractMediaMetadata(env,
+ j_context,
+ j_url_string.obj(),
+ j_cookies.obj(),
+ j_user_agent.obj());
BrowserThread::PostTask(
BrowserThread::UI, FROM_HERE,
base::Bind(callback, base::TimeDelta::FromMilliseconds(
@@ -261,11 +268,12 @@ void MediaResourceGetterImpl::GetPlatformPathCallback(
void MediaResourceGetterImpl::ExtractMediaMetadata(
const std::string& url, const std::string& cookies,
- const ExtractMediaMetadataCB& callback) {
+ const std::string& user_agent, const ExtractMediaMetadataCB& callback) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
base::SequencedWorkerPool* pool = content::BrowserThread::GetBlockingPool();
pool->PostWorkerTask(
- FROM_HERE, base::Bind(&GetMediaMetadata, url, cookies, callback));
+ FROM_HERE,
+ base::Bind(&GetMediaMetadata, url, cookies, user_agent, callback));
}
// static

Powered by Google App Engine
This is Rietveld 408576698