Temporary fix for installer/visitedlink crash (crbug.com/171475)

chrome/browser/chrome_browser_main.cc

Index: chrome/browser/chrome_browser_main.cc
===================================================================
--- chrome/browser/chrome_browser_main.cc	(revision 181040)
+++ chrome/browser/chrome_browser_main.cc	(working copy)
@@ -89,6 +89,7 @@
 #include "chrome/browser/ui/uma_browsing_activity_observer.h"
 #include "chrome/browser/ui/user_data_dir_dialog.h"
 #include "chrome/browser/ui/webui/chrome_url_data_manager_backend.h"
+#include "chrome/browser/visitedlink/visitedlink_master.h"
 #include "chrome/common/child_process_logging.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
@@ -1256,6 +1257,15 @@
   RLZTracker::InitRlzFromProfileDelayed(profile_, is_first_run_, ping_delay);
 #endif  // defined(ENABLE_RLZ) && !defined(OS_CHROMEOS)
 
+  // Force creation of the VisitedLinkMaster so that it catches the visited
+  // links in the first tab. This is a workaround for crbug.com/171475 and
+  // crbug.com/160025 which require that the VisitedLinkMaster be created after
+  // the importer is done, but before page navigation starts.
+  //
+  // TODO(caitkp): Hopefully remove this in M26 when refactoring of VisitedLink

[brettw, 2013/02/14 18:50:39]

Since you're landing this on the branch only, can this comment be reworded to
say this is a temporary measure only on M25 because it's fixed by yada yada
yada. (This is going on the branch only, right?)

[Cait (Slow), 2013/02/14 20:24:53]

Fixed. and yes, this is going on the branch only.

+  // code lands.
+  VisitedLinkMaster::FromProfile(profile_);
+
   // Configure modules that need access to resources.
   net::NetModule::SetResourceProvider(chrome_common_net::NetResourceProvider);