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

Side by Side Diff: content/browser/frame_host/render_frame_host_android.cc

Issue 2681933002: Add Java wrapper for RenderFrameHost (Closed)
Patch Set: Fix potential dtor problem Created 3 years, 10 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "content/browser/frame_host/render_frame_host_android.h"
6
7 #include "base/android/jni_string.h"
8 #include "base/logging.h"
9 #include "content/browser/frame_host/render_frame_host_impl.h"
10 #include "jni/RenderFrameHostImpl_jni.h"
11
12 using base::android::AttachCurrentThread;
13 using base::android::ConvertUTF8ToJavaString;
14 using base::android::JavaParamRef;
15 using base::android::JavaRef;
16 using base::android::ScopedJavaLocalRef;
17
18 namespace content {
19
20 // static
21 bool RenderFrameHostAndroid::Register(JNIEnv* env) {
22 return RegisterNativesImpl(env);
23 }
24
25 RenderFrameHostAndroid::RenderFrameHostAndroid(
26 RenderFrameHostImpl* render_frame_host)
27 : render_frame_host_(render_frame_host), weak_factory_(this) {
28 JNIEnv* env = AttachCurrentThread();
29 ScopedJavaLocalRef<jobject> obj =
30 Java_RenderFrameHostImpl_create(env, reinterpret_cast<intptr_t>(this));
31 obj_ = JavaObjectWeakGlobalRef(env, obj);
32 }
33
34 RenderFrameHostAndroid::~RenderFrameHostAndroid() {
35 ScopedJavaLocalRef<jobject> jobj = GetJavaObject();
36 if (!jobj.is_null()) {
37 Java_RenderFrameHostImpl_clearNativePtr(AttachCurrentThread(), jobj);
38 obj_.reset();
39 }
40 }
41
42 base::android::ScopedJavaLocalRef<jobject>
43 RenderFrameHostAndroid::GetJavaObject() {
44 JNIEnv* env = base::android::AttachCurrentThread();
45 return obj_.get(env);
46 }
47
48 ScopedJavaLocalRef<jstring> RenderFrameHostAndroid::GetLastCommittedURL(
49 JNIEnv* env,
50 const JavaParamRef<jobject>& obj) const {
51 return ConvertUTF8ToJavaString(
52 env, render_frame_host_->GetLastCommittedURL().spec());
53 }
54
55 // static
56 RenderFrameHost* RenderFrameHost::FromJavaRenderFrameHost(
57 const JavaRef<jobject>& jrender_frame_host_android) {
58 DCHECK_CURRENTLY_ON(BrowserThread::UI);
59 if (jrender_frame_host_android.is_null())
60 return NULL;
boliu 2017/02/16 01:00:58 nit: nullptr everywhere
rwlbuis 2017/03/01 21:53:57 Acknowledged.
61
62 RenderFrameHostAndroid* render_frame_host_android =
63 reinterpret_cast<RenderFrameHostAndroid*>(
64 Java_RenderFrameHostImpl_getNativePointer(
65 AttachCurrentThread(), jrender_frame_host_android));
66 if (!render_frame_host_android)
67 return NULL;
68 return render_frame_host_android->render_frame_host();
69 }
70
71 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698