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

Side by Side Diff: chrome/browser/android/browsing_data/url_filter_bridge.cc

Issue 2209993003: Add missing using statements for JNI types. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/android/browsing_data/url_filter_bridge.h" 5 #include "chrome/browser/android/browsing_data/url_filter_bridge.h"
6 6
7 #include "base/android/jni_android.h" 7 #include "base/android/jni_android.h"
8 #include "base/android/jni_string.h" 8 #include "base/android/jni_string.h"
9 #include "jni/UrlFilterBridge_jni.h" 9 #include "jni/UrlFilterBridge_jni.h"
10 #include "url/gurl.h" 10 #include "url/gurl.h"
11 11
12 using base::android::JavaParamRef;
13
12 UrlFilterBridge::UrlFilterBridge( 14 UrlFilterBridge::UrlFilterBridge(
13 const base::Callback<bool(const GURL&)>& url_filter) 15 const base::Callback<bool(const GURL&)>& url_filter)
14 : url_filter_(url_filter), 16 : url_filter_(url_filter),
15 j_bridge_(Java_UrlFilterBridge_create( 17 j_bridge_(Java_UrlFilterBridge_create(
16 base::android::AttachCurrentThread(), 18 base::android::AttachCurrentThread(),
17 reinterpret_cast<uintptr_t>(this))) {} 19 reinterpret_cast<uintptr_t>(this))) {}
18 20
19 UrlFilterBridge::~UrlFilterBridge() {} 21 UrlFilterBridge::~UrlFilterBridge() {}
20 22
21 void UrlFilterBridge::Destroy(JNIEnv* env, 23 void UrlFilterBridge::Destroy(JNIEnv* env,
22 const JavaParamRef<jobject>& obj) { 24 const JavaParamRef<jobject>& obj) {
23 delete this; 25 delete this;
24 } 26 }
25 27
26 bool UrlFilterBridge::MatchesUrl( 28 bool UrlFilterBridge::MatchesUrl(
27 JNIEnv* env, 29 JNIEnv* env,
28 const JavaParamRef<jobject>& obj, 30 const JavaParamRef<jobject>& obj,
29 const JavaParamRef<jstring>& jurl) const { 31 const JavaParamRef<jstring>& jurl) const {
30 GURL url(base::android::ConvertJavaStringToUTF8(env, jurl)); 32 GURL url(base::android::ConvertJavaStringToUTF8(env, jurl));
31 return url_filter_.Run(url); 33 return url_filter_.Run(url);
32 } 34 }
33 35
34 // static 36 // static
35 bool UrlFilterBridge::Register(JNIEnv* env) { 37 bool UrlFilterBridge::Register(JNIEnv* env) {
36 return RegisterNativesImpl(env); 38 return RegisterNativesImpl(env);
37 } 39 }
OLDNEW
« no previous file with comments | « chrome/browser/android/browsing_data/browsing_data_counter_bridge.cc ('k') | chrome/browser/android/chrome_application.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698