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

Side by Side Diff: components/safe_browsing_db/android/safe_browsing_api_handler_bridge.cc

Issue 2810863003: Android: Remove GetApplicationContext: components/ (Closed)
Patch Set: Fix tests Created 3 years, 8 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 "components/safe_browsing_db/android/safe_browsing_api_handler_bridge.h " 5 #include "components/safe_browsing_db/android/safe_browsing_api_handler_bridge.h "
6 6
7 #include <memory> 7 #include <memory>
8 #include <string> 8 #include <string>
9 9
10 #include "base/android/context_utils.h"
11 #include "base/android/jni_android.h" 10 #include "base/android/jni_android.h"
12 #include "base/android/jni_array.h" 11 #include "base/android/jni_array.h"
13 #include "base/android/jni_string.h" 12 #include "base/android/jni_string.h"
14 #include "base/metrics/histogram_macros.h" 13 #include "base/metrics/histogram_macros.h"
15 #include "components/safe_browsing_db/safe_browsing_api_handler_util.h" 14 #include "components/safe_browsing_db/safe_browsing_api_handler_util.h"
16 #include "content/public/browser/browser_thread.h" 15 #include "content/public/browser/browser_thread.h"
17 #include "jni/SafeBrowsingApiBridge_jni.h" 16 #include "jni/SafeBrowsingApiBridge_jni.h"
18 17
19 using base::android::AttachCurrentThread; 18 using base::android::AttachCurrentThread;
20 using base::android::ConvertJavaStringToUTF8; 19 using base::android::ConvertJavaStringToUTF8;
21 using base::android::ConvertUTF8ToJavaString; 20 using base::android::ConvertUTF8ToJavaString;
22 using base::android::GetApplicationContext;
23 using base::android::JavaParamRef; 21 using base::android::JavaParamRef;
24 using base::android::ScopedJavaLocalRef; 22 using base::android::ScopedJavaLocalRef;
25 using base::android::ToJavaIntArray; 23 using base::android::ToJavaIntArray;
26 using content::BrowserThread; 24 using content::BrowserThread;
27 25
28 namespace safe_browsing { 26 namespace safe_browsing {
29 27
30 namespace { 28 namespace {
31 void RunCallbackOnIOThread( 29 void RunCallbackOnIOThread(
32 const SafeBrowsingApiHandler::URLCheckCallbackMeta& callback, 30 const SafeBrowsingApiHandler::URLCheckCallbackMeta& callback,
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
133 // 131 //
134 SafeBrowsingApiHandlerBridge::SafeBrowsingApiHandlerBridge() 132 SafeBrowsingApiHandlerBridge::SafeBrowsingApiHandlerBridge()
135 : checked_api_support_(false) {} 133 : checked_api_support_(false) {}
136 134
137 SafeBrowsingApiHandlerBridge::~SafeBrowsingApiHandlerBridge() {} 135 SafeBrowsingApiHandlerBridge::~SafeBrowsingApiHandlerBridge() {}
138 136
139 bool SafeBrowsingApiHandlerBridge::CheckApiIsSupported() { 137 bool SafeBrowsingApiHandlerBridge::CheckApiIsSupported() {
140 DCHECK_CURRENTLY_ON(BrowserThread::IO); 138 DCHECK_CURRENTLY_ON(BrowserThread::IO);
141 if (!checked_api_support_) { 139 if (!checked_api_support_) {
142 DVLOG(1) << "Checking API support."; 140 DVLOG(1) << "Checking API support.";
143 j_api_handler_ = Java_SafeBrowsingApiBridge_create(AttachCurrentThread(), 141 j_api_handler_ = Java_SafeBrowsingApiBridge_create(AttachCurrentThread());
144 GetApplicationContext());
145 checked_api_support_ = true; 142 checked_api_support_ = true;
146 } 143 }
147 return j_api_handler_.obj() != nullptr; 144 return j_api_handler_.obj() != nullptr;
148 } 145 }
149 146
150 void SafeBrowsingApiHandlerBridge::StartURLCheck( 147 void SafeBrowsingApiHandlerBridge::StartURLCheck(
151 const SafeBrowsingApiHandler::URLCheckCallbackMeta& callback, 148 const SafeBrowsingApiHandler::URLCheckCallbackMeta& callback,
152 const GURL& url, 149 const GURL& url,
153 const std::vector<SBThreatType>& threat_types) { 150 const std::vector<SBThreatType>& threat_types) {
154 DCHECK_CURRENTLY_ON(BrowserThread::IO); 151 DCHECK_CURRENTLY_ON(BrowserThread::IO);
(...skipping 22 matching lines...) Expand all
177 JNIEnv* env = AttachCurrentThread(); 174 JNIEnv* env = AttachCurrentThread();
178 ScopedJavaLocalRef<jstring> j_url = ConvertUTF8ToJavaString(env, url.spec()); 175 ScopedJavaLocalRef<jstring> j_url = ConvertUTF8ToJavaString(env, url.spec());
179 ScopedJavaLocalRef<jintArray> j_threat_types = 176 ScopedJavaLocalRef<jintArray> j_threat_types =
180 SBThreatTypesToJavaArray(env, local_threat_types); 177 SBThreatTypesToJavaArray(env, local_threat_types);
181 178
182 Java_SafeBrowsingApiBridge_startUriLookup(env, j_api_handler_, callback_id, 179 Java_SafeBrowsingApiBridge_startUriLookup(env, j_api_handler_, callback_id,
183 j_url, j_threat_types); 180 j_url, j_threat_types);
184 } 181 }
185 182
186 } // namespace safe_browsing 183 } // namespace safe_browsing
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698