commit 8233facddcc51865d612a919d450db6954aa48e3
author Michael Kolb <kolby@google.com> Tue Oct 26 16:08:53 2010 -0700
committer Michael Kolb <kolby@google.com> Wed Nov 17 13:27:05 2010 -0800
tree 049b5f9ac93146d842aa1c28a168e93783ea578a
parent d9fb57958a0bea503b43d740057e3a4623c91d12

Controller Refactor

    Bug: 3170671
    First step towards a model/view/control design in Browser
    introduced Controller object
    started separating UI code
    represent state of the app in one place only

Change-Id: Ica387d6bde2dcf1a4993c3db0cce498cf34ff60f

src/com/android/browser/BrowserHomepagePreference.java

diff --git a/src/com/android/browser/BrowserHomepagePreference.java b/src/com/android/browser/BrowserHomepagePreference.java
index fefe101..9886053 100644
--- a/src/com/android/browser/BrowserHomepagePreference.java
+++ b/src/com/android/browser/BrowserHomepagePreference.java
@@ -21,16 +21,16 @@
 import android.content.DialogInterface;
 import android.os.Bundle;
 import android.preference.EditTextPreference;
-import android.widget.Button;
-import android.widget.EditText;
-import android.widget.LinearLayout;
-import android.widget.Toast;
+import android.util.AttributeSet;
 import android.view.Gravity;
 import android.view.View;
 import android.view.ViewGroup;
 import android.view.Window;
 import android.view.WindowManager;
-import android.util.AttributeSet;
+import android.widget.Button;
+import android.widget.EditText;
+import android.widget.LinearLayout;
+import android.widget.Toast;
 
 public class BrowserHomepagePreference extends EditTextPreference {
     private String mCurrentPage;
@@ -102,7 +102,9 @@
     protected void onDialogClosed(boolean positiveResult) {
         if (positiveResult) {
             String url = getEditText().getText().toString();
-            if (!BrowserActivity.ACCEPTED_URI_SCHEMA.matcher(url).matches()) {
+            if (url.length() > 0
+                    && !UrlUtils.ACCEPTED_URI_SCHEMA.matcher(url)
+                            .matches()) {
                 int colon = url.indexOf(':');
                 int space = url.indexOf(' ');
                 if (colon == -1 && space == -1 && url.length() > 0) {