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

Side by Side Diff: components/web_contents_delegate_android/web_contents_delegate_android.cc

Issue 646693003: Removing the CVC parameter from ShowRepostFormWarningDialog Callback. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed review comments. Created 6 years, 2 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/web_contents_delegate_android/web_contents_delegate_android .h" 5 #include "components/web_contents_delegate_android/web_contents_delegate_android .h"
6 6
7 #include <android/keycodes.h> 7 #include <android/keycodes.h>
8 8
9 #include "base/android/jni_android.h" 9 #include "base/android/jni_android.h"
10 #include "base/android/jni_array.h" 10 #include "base/android/jni_array.h"
(...skipping 300 matching lines...) Expand 10 before | Expand all | Expand 10 after
311 return Java_WebContentsDelegateAndroid_takeFocus( 311 return Java_WebContentsDelegateAndroid_takeFocus(
312 env, obj.obj(), reverse); 312 env, obj.obj(), reverse);
313 } 313 }
314 314
315 void WebContentsDelegateAndroid::ShowRepostFormWarningDialog( 315 void WebContentsDelegateAndroid::ShowRepostFormWarningDialog(
316 WebContents* source) { 316 WebContents* source) {
317 JNIEnv* env = AttachCurrentThread(); 317 JNIEnv* env = AttachCurrentThread();
318 ScopedJavaLocalRef<jobject> obj = GetJavaDelegate(env); 318 ScopedJavaLocalRef<jobject> obj = GetJavaDelegate(env);
319 if (obj.is_null()) 319 if (obj.is_null())
320 return; 320 return;
321 ScopedJavaLocalRef<jobject> content_view_core = 321 Java_WebContentsDelegateAndroid_showRepostFormWarningDialog(env, obj.obj());
322 content::ContentViewCore::FromWebContents(source)->GetJavaObject();
323 if (content_view_core.is_null())
324 return;
325 Java_WebContentsDelegateAndroid_showRepostFormWarningDialog(env, obj.obj(),
326 content_view_core.obj());
327 } 322 }
328 323
329 void WebContentsDelegateAndroid::ToggleFullscreenModeForTab( 324 void WebContentsDelegateAndroid::ToggleFullscreenModeForTab(
330 WebContents* web_contents, 325 WebContents* web_contents,
331 bool enter_fullscreen) { 326 bool enter_fullscreen) {
332 JNIEnv* env = AttachCurrentThread(); 327 JNIEnv* env = AttachCurrentThread();
333 ScopedJavaLocalRef<jobject> obj = GetJavaDelegate(env); 328 ScopedJavaLocalRef<jobject> obj = GetJavaDelegate(env);
334 if (obj.is_null()) 329 if (obj.is_null())
335 return; 330 return;
336 Java_WebContentsDelegateAndroid_toggleFullscreenModeForTab( 331 Java_WebContentsDelegateAndroid_toggleFullscreenModeForTab(
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
382 // Native JNI methods 377 // Native JNI methods
383 // ---------------------------------------------------------------------------- 378 // ----------------------------------------------------------------------------
384 379
385 // Register native methods 380 // Register native methods
386 381
387 bool RegisterWebContentsDelegateAndroid(JNIEnv* env) { 382 bool RegisterWebContentsDelegateAndroid(JNIEnv* env) {
388 return RegisterNativesImpl(env); 383 return RegisterNativesImpl(env);
389 } 384 }
390 385
391 } // namespace web_contents_delegate_android 386 } // namespace web_contents_delegate_android
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698