Don't create MostVisitedSites in incognito

chrome/browser/search/instant_service.cc

Index: chrome/browser/search/instant_service.cc
diff --git a/chrome/browser/search/instant_service.cc b/chrome/browser/search/instant_service.cc
index 4a1c3b142c8ca4076e58b0b8ee88481487c72826..e313d56e7edf93f618dffc7a283982ea57e1a91f 100644
--- a/chrome/browser/search/instant_service.cc
+++ b/chrome/browser/search/instant_service.cc
@@ -103,7 +103,8 @@ InstantService::InstantService(Profile* profile)
   if (base::FeatureList::IsEnabled(kNtpTilesFeature)) {
     most_visited_sites_ =
         ChromeMostVisitedSitesFactory::NewForProfile(profile_);
-    most_visited_sites_->SetMostVisitedURLsObserver(this, 8);
+    if (most_visited_sites_)
+      most_visited_sites_->SetMostVisitedURLsObserver(this, 8);

[sfiera, 2017/02/01 11:24:57]

FYI, merge conflict with https://crrev.com/2619993002

[Marc Treib, 2017/02/01 12:42:46]

Thanks for the heads-up!
Since that one doesn't look like it'll land immediately, I guess I'll just go
ahead and land this. Should be trivial to resolve either way.

[mastiz, 2017/02/01 13:03:39]

Don't worry about conflicts, I can resolve those if needed. In fact, I'm not
even sure I'll go ahead with the CL above, thanks for bringing it up anyway.

   } else {
     top_sites_ = TopSitesFactory::GetForProfile(profile_);
     if (top_sites_) {