Small style stuff

[zxing.git] / android / src / com / google / zxing / client / android / HelpActivity.java

diff --git a/android/src/com/google/zxing/client/android/HelpActivity.java b/android/src/com/google/zxing/client/android/HelpActivity.java
index 49530df..34d80ce 100644 (file)
--- a/android/src/com/google/zxing/client/android/HelpActivity.java
+++ b/android/src/com/google/zxing/client/android/HelpActivity.java
@@ -17,7 +17,13 @@
 package com.google.zxing.client.android;
 
 import android.app.Activity;
+import android.app.AlertDialog;
+import android.content.DialogInterface;
+import android.content.Intent;
+import android.net.Uri;
+import android.os.Build;
 import android.os.Bundle;
+import android.util.Log;
 import android.view.View;
 import android.view.KeyEvent;
 import android.webkit.WebView;
@@ -30,8 +36,29 @@ import android.widget.Button;
  * @author dswitkin@google.com (Daniel Switkin)
  */
 public final class HelpActivity extends Activity {
-  private static final String DEFAULT_URL = "file:///android_asset/html/index.html";
 
+  private static final String TAG = HelpActivity.class.getSimpleName();
+
+  // Actually guessing at the Desire's MODEL for now:
+  private static final String[] BUGGY_MODEL_SUBSTRINGS = {
+      "Desire",
+      "Pulse", // Camera doesn't come on
+      "Geeksphone", // Doesn't support YUV?
+      "supersonic", // aka Evo
+  };
+  private static final Uri BUGGY_URI =
+      Uri.parse("http://code.google.com/p/zxing/wiki/FrequentlyAskedQuestions");
+
+  // Use this key and one of the values below when launching this activity via intent. If not
+  // present, the default page will be loaded.
+  public static final String REQUESTED_PAGE_KEY = "requested_page_key";
+  public static final String DEFAULT_PAGE = "index.html";
+  public static final String WHATS_NEW_PAGE = "whatsnew.html";
+
+  private static final String BASE_URL = "file:///android_asset/html/";
+  private static final String WEBVIEW_STATE_PRESENT = "webview_state_present";
+
+  private static boolean initialized = false;
   private WebView webView;
   private Button backButton;
 
@@ -47,6 +74,15 @@ public final class HelpActivity extends Activity {
     }
   };
 
+  private final DialogInterface.OnClickListener groupsListener =
+      new DialogInterface.OnClickListener() {
+    public void onClick(DialogInterface dialogInterface, int i) {
+      Intent intent = new Intent(Intent.ACTION_VIEW, BUGGY_URI);
+      intent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_WHEN_TASK_RESET);
+      HelpActivity.this.startActivity(intent);
+    }
+  };
+
   @Override
   protected void onCreate(Bundle icicle) {
     super.onCreate(icicle);
@@ -54,27 +90,59 @@ public final class HelpActivity extends Activity {
 
     webView = (WebView)findViewById(R.id.help_contents);
     webView.setWebViewClient(new HelpClient());
-    if (icicle != null) {
+
+    // Froyo has a bug with calling onCreate() twice in a row, which causes the What's New page
+    // that's auto-loaded on first run to appear blank. As a workaround we only call restoreState()
+    // if a valid URL was loaded at the time the previous activity was torn down.
+    Intent intent = getIntent();
+    if (icicle != null && icicle.getBoolean(WEBVIEW_STATE_PRESENT, false)) {
       webView.restoreState(icicle);
+    } else if (intent != null) {
+      String page = intent.getStringExtra(REQUESTED_PAGE_KEY);
+      if (page != null && page.length() > 0) {
+        webView.loadUrl(BASE_URL + page);
+      } else {
+        webView.loadUrl(BASE_URL + DEFAULT_PAGE);
+      }
     } else {
-      webView.loadUrl(DEFAULT_URL);
+      webView.loadUrl(BASE_URL + DEFAULT_PAGE);
     }
 
-    backButton = (Button)findViewById(R.id.back_button);
+    backButton = (Button) findViewById(R.id.back_button);
     backButton.setOnClickListener(backListener);
-
-    Button doneButton = (Button)findViewById(R.id.done_button);
+    View doneButton = findViewById(R.id.done_button);
     doneButton.setOnClickListener(doneListener);
+
+    if (!initialized) {
+      initialized = true;
+      checkBuggyDevice();
+    }
   }
 
-  @Override
-  public void onResume() {
-    super.onResume();
+  private void checkBuggyDevice() {
+    String model = Build.MODEL;
+    Log.i(TAG, "Build model is " + model);
+    if (model != null) {
+      for (String buggyModelSubstring : BUGGY_MODEL_SUBSTRINGS) {
+        if (model.contains(buggyModelSubstring)) {
+          AlertDialog.Builder builder = new AlertDialog.Builder(this);
+          builder.setMessage(R.string.msg_buggy);
+          builder.setPositiveButton(R.string.button_ok, groupsListener);
+          builder.setNegativeButton(R.string.button_cancel, null);
+          builder.show();
+          break;
+        }
+      }
+    }
   }
 
   @Override
   protected void onSaveInstanceState(Bundle state) {
-    webView.saveState(state);
+    String url = webView.getUrl();
+    if (url != null && url.length() > 0) {
+      webView.saveState(state);
+      state.putBoolean(WEBVIEW_STATE_PRESENT, true);
+    }
   }
 
   @Override
@@ -94,6 +162,17 @@ public final class HelpActivity extends Activity {
       setTitle(view.getTitle());
       backButton.setEnabled(view.canGoBack());
     }
-  }
 
+    @Override
+    public boolean shouldOverrideUrlLoading(WebView view, String url) {
+      if (url.startsWith("file")) {
+        // Keep local assets in this WebView.
+        return false;
+      } else {
+        // Open external URLs in Browser.
+        startActivity(new Intent(Intent.ACTION_VIEW, Uri.parse(url)));
+        return true;
+      }
+    }
+  }
 }