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

chrome/browser/browser_main.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/browser_main.h"
 
 #include <algorithm>
 #include <string>
 #include <vector>
 
@@ skipping 26 matching lines @@
 #include "chrome/browser/browser_process_impl.h"
 #include "chrome/browser/browser_shutdown.h"
 #include "chrome/browser/chrome_content_browser_client.h"
 #include "chrome/browser/defaults.h"
 #include "chrome/browser/extensions/extension_protocols.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extensions_startup.h"
 #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"
 #include "chrome/browser/metrics/field_trial_synchronizer.h"
 #include "chrome/browser/metrics/histogram_synchronizer.h"
 #include "chrome/browser/metrics/metrics_log.h"
 #include "chrome/browser/metrics/metrics_service.h"
 #include "chrome/browser/metrics/thread_watcher.h"
 #include "chrome/browser/net/chrome_dns_cert_provenance_checker.h"
 #include "chrome/browser/net/chrome_dns_cert_provenance_checker_factory.h"
@@ skipping 1816 matching lines @@
   if (template_url_service) {
     const TemplateURL* url_template =
         template_url_service->GetDefaultSearchProvider();
     if (url_template) {
       const TemplateURLRef* urlref = url_template->url();
       if (urlref) {
         google_search_default = urlref->HasGoogleBaseURLs();
       }
     }
   }
-  // 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
 
   // Configure modules that need access to resources.
   net::NetModule::SetResourceProvider(chrome_common_net::NetResourceProvider);
 
   // In unittest mode, this will do nothing.  In normal mode, this will create
   // the global GoogleURLTracker and IntranetRedirectDetector instances, which
   // will promptly go to sleep for five and seven seconds, respectively (to
   // avoid slowing startup), and wake up afterwards to see if they should do
@@ skipping 284 matching lines @@
   if (env->GetVar(chrome::kPreReadEnvironmentVariable, &pre_read) &&
       (pre_read == "0" || pre_read == "1")) {
     std::string uma_name(name);
     uma_name += "_PreRead";
     uma_name += pre_read == "1" ? "Enabled" : "Disabled";
     AddPreReadHistogramTime(uma_name.c_str(), time);
   }
 #endif
 }