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

Unified Diff: media/base/android/media_codec_util.cc

Issue 2237943002: Remove now-unnecessary .obj() in Java method calls. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@switch-context
Patch Set: Rebase *again* :( Created 4 years, 4 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
« no previous file with comments | « media/audio/android/audio_record_input.cc ('k') | media/base/android/media_drm_bridge.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/android/media_codec_util.cc
diff --git a/media/base/android/media_codec_util.cc b/media/base/android/media_codec_util.cc
index efe5afd374f62843930930b3da144c93f0561bb8..e507936f4f8e7c9bfce91189cb72e3b060c3788c 100644
--- a/media/base/android/media_codec_util.cc
+++ b/media/base/android/media_codec_util.cc
@@ -73,7 +73,7 @@ static std::string GetDefaultCodecName(const std::string& mime_type,
JNIEnv* env = AttachCurrentThread();
ScopedJavaLocalRef<jstring> j_mime = ConvertUTF8ToJavaString(env, mime_type);
ScopedJavaLocalRef<jstring> j_codec_name =
- Java_MediaCodecUtil_getDefaultCodecName(env, j_mime.obj(), direction,
+ Java_MediaCodecUtil_getDefaultCodecName(env, j_mime, direction,
require_software_codec);
return ConvertJavaStringToUTF8(env, j_codec_name.obj());
}
@@ -84,7 +84,7 @@ static bool IsDecoderSupportedByDevice(const std::string& android_mime_type) {
JNIEnv* env = AttachCurrentThread();
ScopedJavaLocalRef<jstring> j_mime =
ConvertUTF8ToJavaString(env, android_mime_type);
- return Java_MediaCodecUtil_isDecoderSupportedForDevice(env, j_mime.obj());
+ return Java_MediaCodecUtil_isDecoderSupportedForDevice(env, j_mime);
}
// static
@@ -119,7 +119,7 @@ std::set<int> MediaCodecUtil::GetEncoderColorFormats(
JNIEnv* env = AttachCurrentThread();
ScopedJavaLocalRef<jstring> j_mime = ConvertUTF8ToJavaString(env, mime_type);
ScopedJavaLocalRef<jintArray> j_color_format_array =
- Java_MediaCodecUtil_getEncoderColorFormatsForMime(env, j_mime.obj());
+ Java_MediaCodecUtil_getEncoderColorFormatsForMime(env, j_mime);
if (j_color_format_array.obj()) {
std::vector<int> formats;
@@ -140,7 +140,7 @@ bool MediaCodecUtil::CanDecode(const std::string& codec, bool is_secure) {
if (mime.empty())
return false;
ScopedJavaLocalRef<jstring> j_mime = ConvertUTF8ToJavaString(env, mime);
- return Java_MediaCodecUtil_canDecode(env, j_mime.obj(), is_secure);
+ return Java_MediaCodecUtil_canDecode(env, j_mime, is_secure);
}
// static
« no previous file with comments | « media/audio/android/audio_record_input.cc ('k') | media/base/android/media_drm_bridge.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698