commit a3f90f06f5b3b7db18f30eca5322796fc6b4c2e3
author Grace Kloba <klobag@google.com> Tue May 26 11:32:53 2009 -0700
committer Grace Kloba <klobag@google.com> Tue May 26 11:32:53 2009 -0700
tree 07b6e040def81a03a4470b03c67261faf4dda9d0
parent 2cac3857e10f6597ed5a33bf0260e1212cdd502a
parent f2c5c1b555904400f4092888ae81b8ed74bfe026

Merge commit 'f2c5c1b555904400f4092888ae81b8ed74bfe026' into manualfix

Conflicts:
	src/com/android/browser/BrowserSettings.java

src/com/android/browser/BrowserActivity.java

diff --git a/src/com/android/browser/BrowserActivity.java b/src/com/android/browser/BrowserActivity.java
index d251f7a..356b2d2 100644
--- a/src/com/android/browser/BrowserActivity.java
+++ b/src/com/android/browser/BrowserActivity.java
@@ -195,6 +195,9 @@
                 try {
                     if (mGls == null) return;
 
+                    if (!homepage.startsWith("http://www.google.")) return;
+                    if (homepage.indexOf('?') == -1) return;
+
                     String hostedUser = mGls.getAccount(GoogleLoginServiceConstants.PREFER_HOSTED);
                     String googleUser = mGls.getAccount(GoogleLoginServiceConstants.REQUIRE_GOOGLE);
 
@@ -214,8 +217,7 @@
 
                     if (googleUser == null || !hostedUser.equals(googleUser)) {
                         String domain = hostedUser.substring(hostedUser.lastIndexOf('@')+1);
-                        homepage = "http://www.google.com/m/a/" + domain + "?client=ms-" +
-                            Partner.getString(BrowserActivity.this.getContentResolver(), Partner.CLIENT_ID);
+                        homepage = homepage.replace("?", "/a/" + domain + "?");
                     }
                 } catch (RemoteException ignore) {
                     // Login service died; carry on

src/com/android/browser/BrowserSettings.java

diff --git a/src/com/android/browser/BrowserSettings.java b/src/com/android/browser/BrowserSettings.java
index 19e8449..32ad19e 100644
--- a/src/com/android/browser/BrowserSettings.java
+++ b/src/com/android/browser/BrowserSettings.java
@@ -57,8 +57,6 @@
  */
 class BrowserSettings extends Observable {
 
-    private static final String DEFAULT_HOME_URL =
-            "http://www.google.com/m?client=ms-";
     // Private variables for settings
     // NOTE: these defaults need to be kept in sync with the XML
     // until the performance of PreferenceManager.setDefaultValues()
@@ -73,7 +71,7 @@
     private boolean saveFormData = true;
     private boolean openInBackground = false;
     private String defaultTextEncodingName;
-    private String homeUrl;
+    private String homeUrl = "";
     private boolean loginInitialized = false;
     private boolean autoFitPage = true;
     private boolean landscapeOnly = false;
@@ -235,8 +233,7 @@
         // Set the default value for the Database path.
         databasePath = ctx.getDir("databases", 0).getPath();
 
-        homeUrl = DEFAULT_HOME_URL +
-                Partner.getString(ctx.getContentResolver(), Partner.CLIENT_ID);
+        homeUrl = getFactoryResetHomeUrl(ctx);
 
         // Load the defaults from the xml
         // This call is TOO SLOW, need to manually keep the defaults
@@ -501,8 +498,17 @@
         p.edit().clear().commit();
         PreferenceManager.setDefaultValues(ctx, R.xml.browser_preferences,
                 true);
-        setHomePage(ctx, DEFAULT_HOME_URL +
-                Partner.getString(ctx.getContentResolver(), Partner.CLIENT_ID));
+        // reset homeUrl
+        setHomePage(ctx, getFactoryResetHomeUrl(ctx));
+    }
+
+    private String getFactoryResetHomeUrl(Context context) {
+        String url = context.getResources().getString(R.string.homepage_base);
+        if (url.indexOf("{CID}") != -1) {
+            url = url.replace("{CID}", Partner.getString(context
+                    .getContentResolver(), Partner.CLIENT_ID));
+        }
+        return url;
     }
 
     // Private constructor that does nothing.