Chromium Code Reviews| Index: content/browser/android/web_contents_observer_android.cc |
| diff --git a/content/browser/android/web_contents_observer_android.cc b/content/browser/android/web_contents_observer_android.cc |
| index 154fb5b3e68eff961a5970b4fab0b36bcd695fb8..aecf77d794cd9a96bca0fbdbe682b7792d0a5f72 100644 |
| --- a/content/browser/android/web_contents_observer_android.cc |
| +++ b/content/browser/android/web_contents_observer_android.cc |
| @@ -137,6 +137,25 @@ void WebContentsObserverAndroid::DidNavigateAnyFrame( |
| jboolean_is_reload); |
| } |
| +void WebContentsObserverAndroid::DidStartProvisionalLoadForFrame( |
| + int64 frame_id, |
| + int64 parent_frame_id, |
| + bool is_main_frame, |
| + const GURL& validated_url, |
| + bool is_error_page, |
| + bool is_iframe_srcdoc, |
| + RenderViewHost* render_view_host) { |
| + JNIEnv* env = AttachCurrentThread(); |
| + ScopedJavaLocalRef<jobject> obj = weak_java_observer_.get(env); |
|
David Trainor- moved to gerrit
2013/01/08 22:22:18
maybe use obj(...); (copy constructor?)
Ted C
2013/01/08 22:43:17
Done.
|
| + if (obj.is_null()) |
| + return; |
| + ScopedJavaLocalRef<jstring> jstring_url = |
|
David Trainor- moved to gerrit
2013/01/08 22:22:18
copy constructor?
Ted C
2013/01/08 22:43:17
Done. Updated other usages in this file to keep u
|
| + ConvertUTF8ToJavaString(env, validated_url.spec()); |
| + Java_WebContentsObserverAndroid_didStartProvisionalLoadForFrame( |
| + env, obj.obj(), frame_id, parent_frame_id, is_main_frame, |
| + jstring_url.obj(), is_error_page, is_iframe_srcdoc); |
| +} |
| + |
| void WebContentsObserverAndroid::DidFailLoadInternal( |
| bool is_provisional_load, |
| bool is_main_frame, |