Fix the loading progress bar for Android.

content/browser/android/web_contents_observer_android.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/android/web_contents_observer_android.h"
 
 #include <string>
 
 #include <jni.h>
 
@@ skipping 119 matching lines @@
   ScopedJavaLocalRef<jstring> jstring_url =
       ConvertUTF8ToJavaString(env, params.url.spec());
   ScopedJavaLocalRef<jstring> jstring_base_url =
       ConvertUTF8ToJavaString(env, params.base_url.spec());
   jboolean jboolean_is_reload = PAGE_TRANSITION_RELOAD == params.transition;
   Java_WebContentsObserverAndroid_didNavigateAnyFrame(
       env, obj.obj(), jstring_url.obj(), jstring_base_url.obj(),
       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 us copy paste errors down in
the future.

+      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,
     int error_code,
     const string16& description,
     const GURL& url) {
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> obj = weak_java_observer_.get(env);
   if (obj.is_null())
     return;
@@ skipping 11 matching lines @@
 }
 
 bool RegisterWebContentsObserverAndroid(JNIEnv* env) {
   if (!HasClass(env, kWebContentsObserverAndroidClassPath)) {
     DLOG(ERROR) << "Unable to find class WebContentsObserverAndroid!";
     return false;
   }
   return RegisterNativesImpl(env);
 }
 }  // namespace content