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

Side by Side Diff: chrome/browser/android/banners/app_banner_manager_android.cc

Issue 2434623002: Remove AppBannerManager code to skip secure context checks for tests. (Closed)
Patch Set: Clean up Created 4 years, 1 month 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 <utility> 5 #include <utility>
6 6
7 #include "chrome/browser/android/banners/app_banner_manager_android.h" 7 #include "chrome/browser/android/banners/app_banner_manager_android.h"
8 8
9 #include "base/android/jni_android.h" 9 #include "base/android/jni_android.h"
10 #include "base/android/jni_string.h" 10 #include "base/android/jni_string.h"
(...skipping 281 matching lines...) Expand 10 before | Expand all | Expand 10 after
292 JNIEnv* env, 292 JNIEnv* env,
293 const JavaParamRef<jclass>& clazz, 293 const JavaParamRef<jclass>& clazz,
294 const JavaParamRef<jobject>& java_web_contents) { 294 const JavaParamRef<jobject>& java_web_contents) {
295 AppBannerManagerAndroid* manager = AppBannerManagerAndroid::FromWebContents( 295 AppBannerManagerAndroid* manager = AppBannerManagerAndroid::FromWebContents(
296 content::WebContents::FromJavaWebContents(java_web_contents)); 296 content::WebContents::FromJavaWebContents(java_web_contents));
297 return manager? ScopedJavaLocalRef<jobject>(manager->GetJavaBannerManager()) 297 return manager? ScopedJavaLocalRef<jobject>(manager->GetJavaBannerManager())
298 : ScopedJavaLocalRef<jobject>(); 298 : ScopedJavaLocalRef<jobject>();
299 } 299 }
300 300
301 // static 301 // static
302 void DisableSecureSchemeCheckForTesting(JNIEnv* env,
303 const JavaParamRef<jclass>& clazz) {
304 AppBannerManager::DisableSecureSchemeCheckForTesting();
305 }
306
307 // static
308 void SetEngagementWeights(JNIEnv* env, 302 void SetEngagementWeights(JNIEnv* env,
309 const JavaParamRef<jclass>& clazz, 303 const JavaParamRef<jclass>& clazz,
310 jdouble direct_engagement, 304 jdouble direct_engagement,
311 jdouble indirect_engagement) { 305 jdouble indirect_engagement) {
312 AppBannerManager::SetEngagementWeights(direct_engagement, 306 AppBannerManager::SetEngagementWeights(direct_engagement,
313 indirect_engagement); 307 indirect_engagement);
314 } 308 }
315 309
316 // static 310 // static
317 void SetTimeDeltaForTesting(JNIEnv* env, 311 void SetTimeDeltaForTesting(JNIEnv* env,
318 const JavaParamRef<jclass>& clazz, 312 const JavaParamRef<jclass>& clazz,
319 jint days) { 313 jint days) {
320 AppBannerManager::SetTimeDeltaForTesting(days); 314 AppBannerManager::SetTimeDeltaForTesting(days);
321 } 315 }
322 316
323 } // namespace banners 317 } // namespace banners
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698