Merge pull request #2264 from wordpress-mobile/issue/2222-theme-preview-title

issue/2222 theme preview title

Author: daniloercoli

WordPress/src/main/java/org/wordpress/android/ui/themes/ThemePreviewFragment.java

@@ -134,8 +134,11 @@ public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle sa
         mWebView.getSettings().setUserAgentString(DESKTOP_UA);
         mWebView.setScrollBarStyle(View.SCROLLBARS_INSIDE_OVERLAY);
 
-        mWebView.setWebChromeClient(new WPWebChromeClient(getActivity(), (ProgressBar) view.findViewById(
-                R.id.progress_bar)));
+        mWebView.setWebChromeClient(
+                new WPWebChromeClient(
+                        getActivity(),
+                        (ProgressBar) view.findViewById(R.id.progress_bar),
+                        false));
 
         mWebView.setWebViewClient(new WPWebViewClient(mBlog));
 

libs/utils/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);
+            }
         }
     }
 }