jni: Pass method parameters as JavaParamRef in content.

content/browser/android/content_video_view.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 "content/browser/android/content_video_view.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "base/metrics/histogram.h"
@@ skipping 104 matching lines @@
   }
 }
 
 void ContentVideoView::OnExitFullscreen() {
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> content_video_view = GetJavaObject(env);
   if (!content_video_view.is_null())
     Java_ContentVideoView_onExitFullscreen(env, content_video_view.obj());
 }
 
-void ContentVideoView::RecordFullscreenPlayback(
-    JNIEnv*, jobject, bool is_portrait_video, bool is_orientation_portrait) {
+void ContentVideoView::RecordFullscreenPlayback(JNIEnv*,
+                                                const JavaParamRef<jobject>&,
+                                                bool is_portrait_video,
+                                                bool is_orientation_portrait) {
   UMA_HISTOGRAM_BOOLEAN("MobileFullscreenVideo.OrientationPortrait",
                         is_orientation_portrait);
   UMA_HISTOGRAM_BOOLEAN("MobileFullscreenVideo.VideoPortrait",
                         is_portrait_video);
 }
 
 void ContentVideoView::RecordExitFullscreenPlayback(
-    JNIEnv*, jobject, bool is_portrait_video,
+    JNIEnv*,
+    const JavaParamRef<jobject>&,
+    bool is_portrait_video,
     long playback_duration_in_milliseconds_before_orientation_change,
     long playback_duration_in_milliseconds_after_orientation_change) {
   bool orientation_changed = (
       playback_duration_in_milliseconds_after_orientation_change != 0);
   if (is_portrait_video) {
     UMA_HISTOGRAM_COUNTS(
         "MobileFullscreenVideo.PortraitDuration",
         playback_duration_in_milliseconds_before_orientation_change);
     UMA_HISTOGRAM_COUNTS(
         "MobileFullscreenVideo.PortraitRotation", orientation_changed);
@@ skipping 20 matching lines @@
   media::MediaPlayerAndroid* player = manager_->GetFullscreenPlayer();
   if (player && player->IsPlayerReady()) {
     Java_ContentVideoView_onUpdateMediaMetadata(
         env, content_video_view.obj(), player->GetVideoWidth(),
         player->GetVideoHeight(),
         static_cast<int>(player->GetDuration().InMilliseconds()),
         player->CanPause(),player->CanSeekForward(), player->CanSeekBackward());
   }
 }
 
-bool ContentVideoView::IsPlaying(JNIEnv*, jobject obj) {
+bool ContentVideoView::IsPlaying(JNIEnv*, const JavaParamRef<jobject>& obj) {
   media::MediaPlayerAndroid* player = manager_->GetFullscreenPlayer();
   return player ? player->IsPlaying() : false;
 }
 
-void ContentVideoView::ExitFullscreen(
-    JNIEnv*, jobject, jboolean release_media_player) {
+void ContentVideoView::ExitFullscreen(JNIEnv*,
+                                      const JavaParamRef<jobject>&,
+                                      jboolean release_media_player) {
   j_content_video_view_.reset();
   manager_->ExitFullscreen(release_media_player);
 }
 
-void ContentVideoView::SetSurface(JNIEnv* env, jobject obj,
-                                  jobject surface) {
+void ContentVideoView::SetSurface(JNIEnv* env,
+                                  const JavaParamRef<jobject>& obj,
+                                  const JavaParamRef<jobject>& surface) {
   manager_->SetVideoSurface(
       gfx::ScopedJavaSurface::AcquireExternalSurface(surface));
 }
 
-void ContentVideoView::RequestMediaMetadata(JNIEnv* env, jobject obj) {
+void ContentVideoView::RequestMediaMetadata(JNIEnv* env,
+                                            const JavaParamRef<jobject>& obj) {
   base::MessageLoop::current()->PostTask(
       FROM_HERE,
       base::Bind(&ContentVideoView::UpdateMediaMetadata,
                  weak_factory_.GetWeakPtr()));
 }
 
 ScopedJavaLocalRef<jobject> ContentVideoView::GetJavaObject(JNIEnv* env) {
   return j_content_video_view_.get(env);
 }
 
@@ skipping 13 matching lines @@
     return JavaObjectWeakGlobalRef(env, nullptr);
 
   return JavaObjectWeakGlobalRef(
       env,
       Java_ContentVideoView_createContentVideoView(
           env,
           j_content_view_core.obj(),
           reinterpret_cast<intptr_t>(this)).obj());
 }
 }  // namespace content