Don't use Build.VERSION_CODES as it's not in 1.5

[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 33beae7..34d80ce 100644 (file)
--- a/android/src/com/google/zxing/client/android/HelpActivity.java
+++ b/android/src/com/google/zxing/client/android/HelpActivity.java
@@ -39,7 +39,13 @@ public final class HelpActivity extends Activity {
 
   private static final String TAG = HelpActivity.class.getSimpleName();
 
-  private static final String[] BUGGY_MODEL_SUBSTRINGS = {"Behold II", "Pulse", "Geeksphone"};
+  // 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");
 
@@ -48,8 +54,11 @@ public final class HelpActivity extends Activity {
   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;
 
@@ -82,8 +91,11 @@ public final class HelpActivity extends Activity {
     webView = (WebView)findViewById(R.id.help_contents);
     webView.setWebViewClient(new HelpClient());
 
+    // 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) {
+    if (icicle != null && icicle.getBoolean(WEBVIEW_STATE_PRESENT, false)) {
       webView.restoreState(icicle);
     } else if (intent != null) {
       String page = intent.getStringExtra(REQUESTED_PAGE_KEY);
@@ -96,16 +108,15 @@ public final class HelpActivity extends Activity {
       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);
-  }
 
-  @Override
-  public void onResume() {
-    super.onResume();
-    checkBuggyDevice();
+    if (!initialized) {
+      initialized = true;
+      checkBuggyDevice();
+    }
   }
 
   private void checkBuggyDevice() {
@@ -118,7 +129,7 @@ public final class HelpActivity extends Activity {
           builder.setMessage(R.string.msg_buggy);
           builder.setPositiveButton(R.string.button_ok, groupsListener);
           builder.setNegativeButton(R.string.button_cancel, null);
-          builder.create().show();
+          builder.show();
           break;
         }
       }
@@ -127,7 +138,11 @@ public final class HelpActivity extends Activity {
 
   @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
@@ -147,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;
+      }
+    }
+  }
 }