Merge pull request #2264 from wordpress-mobile/issue/2222-theme-previ…

…ew-title issue/2222 theme preview title
wordpress-mobile · Jan 29, 2015 · b183edf · b183edf
2 parents 784eb01 + 54e9907
commit b183edf
Showing 1 changed file with 20 additions and 8 deletions.
diff --git a/WordPressUtils/src/main/java/org/wordpress/android/util/WPWebChromeClient.java b/WordPressUtils/src/main/java/org/wordpress/android/util/WPWebChromeClient.java
@@ -7,23 +7,35 @@
 import android.widget.ProgressBar;
 
 public class WPWebChromeClient extends WebChromeClient {
-    private ProgressBar mProgressBar;
-    private Activity mActivity;
+    private final ProgressBar mProgressBar;
+    private final Activity mActivity;
+    private final boolean mAutoUpdateActivityTitle;
 
     public WPWebChromeClient(Activity activity, ProgressBar progressBar) {
+        mActivity = activity;
         mProgressBar = progressBar;
+        mAutoUpdateActivityTitle = true;
+    }
+
+    public WPWebChromeClient(Activity activity,
+                             ProgressBar progressBar,
+                             boolean autoUpdateActivityTitle) {
         mActivity = activity;
+        mProgressBar = progressBar;
+        mAutoUpdateActivityTitle = autoUpdateActivityTitle;
     }
 
     public void onProgressChanged(WebView webView, int progress) {
-        if (!mActivity.isFinishing()) {
+        if (mActivity != null && !mActivity.isFinishing() && mAutoUpdateActivityTitle) {
             mActivity.setTitle(webView.getTitle());
         }
-        if (progress == 100) {
-            mProgressBar.setVisibility(View.GONE);
-        } else {
-            mProgressBar.setVisibility(View.VISIBLE);
-            mProgressBar.setProgress(progress);
+        if (mProgressBar != null) {
+            if (progress == 100) {
+                mProgressBar.setVisibility(View.GONE);
+            } else {
+                mProgressBar.setVisibility(View.VISIBLE);
+                mProgressBar.setProgress(progress);
+            }
         }
     }
 }