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

Unified Diff: content/browser/android/web_contents_observer_android.cc

Issue 11825007: Fix the loading progress bar for Android. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 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 side-by-side diff with in-line comments
Download patch
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,
« no previous file with comments | « content/browser/android/web_contents_observer_android.h ('k') | content/browser/web_contents/web_contents_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698