When the user navigates to the home page, make sure to set the RLZ string

chrome/browser/browser_main.cc

Index: chrome/browser/browser_main.cc
===================================================================
--- chrome/browser/browser_main.cc	(revision 98807)
+++ chrome/browser/browser_main.cc	(working copy)
@@ -44,6 +44,7 @@
 #include "chrome/browser/first_run/first_run.h"
 #include "chrome/browser/first_run/first_run_browser_process.h"
 #include "chrome/browser/first_run/upgrade_util.h"
+#include "chrome/browser/google/google_url_tracker.h"
 #include "chrome/browser/instant/instant_field_trial.h"
 #include "chrome/browser/jankometer.h"
 #include "chrome/browser/language_usage_metrics.h"
@@ -1880,11 +1881,23 @@
       }
     }
   }
-  // TODO(rogerta): For now, always passing false for google_homeapge_default
-  // argument, so that functionality is disabled.  A follow up CL will set it
-  // correctly.
+
+  bool google_search_homepage = false;
+  PrefService* pref_service = profile->GetPrefs();
+  if (pref_service) {
+    std::string homepage = pref_service->GetString(prefs::kHomePage);

[sky, 2011/08/30 15:56:53]

I don't know enough about the install flow. If you choose google as the home
page do we always put GoogleURLTracker::kDefaultGoogleHomepage in your prefs
regardless of where in the world you might be?

[Roger Tawa OOO till Jul 10th, 2011/08/30 16:19:31]

Yes, I already checked this wit the PSO folks.

+    google_search_homepage =
+        homepage == GoogleURLTracker::kDefaultGoogleHomepage;
+  }
+
   RLZTracker::InitRlzDelayed(is_first_run, master_prefs.ping_delay,
-                             google_search_default, false);
+                             google_search_default, google_search_homepage);
+
+  // Prime the RLZ cache for the home page access point so that its avaiable
+  // for the startup page if needed (i.e., when the startup page is set to
+  // the home page).
+  RLZTracker::GetAccessPointRlz(rlz_lib::CHROME_HOME_PAGE, NULL);

[sky, 2011/08/30 15:56:53]

This sounds like you're trying to work around a race condition, and that it'll
impact startup. What exactly does this do?

[Roger Tawa OOO till Jul 10th, 2011/08/30 16:19:31]

This posts a tasks to the file thread to fetch the rlz string from the registry.
 You can see the code in rlz.cc in this CL.  I don't think this will impact
startup.

+
 #endif  // GOOGLE_CHROME_BUILD
 #endif  // OS_WIN