Field trial removing tiles from NTP if URL is already open - for 1993 clients

chrome/browser/ui/search/instant_page.cc

 // Copyright 2013 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/ui/search/instant_page.h"
 
 #include "apps/app_launcher.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chrome_notification_types.h"
+#include "chrome/browser/history/most_visited_tiles_experiment.h"
+#include "chrome/browser/history/top_sites.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search/instant_service.h"
 #include "chrome/browser/search/instant_service_factory.h"
 #include "chrome/browser/search/search.h"
+#include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/search/instant_tab.h"
 #include "chrome/browser/ui/search/search_model.h"
 #include "chrome/browser/ui/search/search_tab_helper.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_details.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/frame_navigate_params.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/font.h"
 
+namespace {
+
+// Creates a set containing the canonical URLs of the currently open tabs.
+void GetOpenUrls(const TabStripModel& tabs,
+                 const history::TopSites& top_sites,
+                 std::set<std::string>* urls) {
+  for (int i = 0; i < tabs.count(); ++i) {
+    content::WebContents* web_contents = tabs.GetWebContentsAt(i);
+    if (web_contents)
+      urls->insert(top_sites.GetCanonicalURLString(web_contents->GetURL()));
+  }
+}
+
+}  // namespace
+
 InstantPage::Delegate::~Delegate() {
 }
 
 InstantPage::~InstantPage() {
   if (contents())
     SearchTabHelper::FromWebContents(contents())->model()->RemoveObserver(this);
 
   // |profile_| may be NULL during unit tests.
   if (profile_) {
     InstantService* instant_service =
@@ skipping 141 matching lines @@
   if (is_main_frame)
     delegate_->InstantPageLoadFailed(contents());
 }
 
 void InstantPage::ThemeInfoChanged(const ThemeBackgroundInfo& theme_info) {
   sender()->SendThemeBackgroundInfo(theme_info);
 }
 
 void InstantPage::MostVisitedItemsChanged(
     const std::vector<InstantMostVisitedItem>& items) {
-  sender()->SendMostVisitedItems(items);
+  std::vector<InstantMostVisitedItem> items_copy(items);
+  MaybeRemoveMostVisitedItems(&items_copy);
+
+  sender()->SendMostVisitedItems(items_copy);
 
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_INSTANT_SENT_MOST_VISITED_ITEMS,
       content::Source<InstantPage>(this),
       content::NotificationService::NoDetails());
 }
 
 void InstantPage::ModelChanged(const SearchModel::State& old_state,
                                const SearchModel::State& new_state) {
   if (old_state.instant_support != new_state.instant_support)
@@ skipping 75 matching lines @@
   delegate_->UndoAllMostVisitedDeletions();
 }
 
 void InstantPage::ClearContents() {
   if (contents())
     SearchTabHelper::FromWebContents(contents())->model()->RemoveObserver(this);
 
   sender()->SetContents(NULL);
   Observe(NULL);
 }
+
+void InstantPage::MaybeRemoveMostVisitedItems(
+    std::vector<InstantMostVisitedItem>* items) {
+// The following #if is due to the facts that tabstripmodel cannot be accessed
+// in the same way, that chrome::FindBrowserWithProfile is undefined in Android
+// and, moreover, that this experiment is not designed to run on Android devices
+// due to different NTP presentation.
+#if !defined(OS_ANDROID)
+  if (!history::MostVisitedTilesExperiment::IsDontShowOpenURLsEnabled()) {
+    return;
+  }
+
+  TabStripModel* tab_strip_model = chrome::FindBrowserWithProfile(
+      profile_,
+      chrome::GetActiveDesktop())->tab_strip_model();
+  history::TopSites* top_sites = profile_->GetTopSites();
+  if (!tab_strip_model || !top_sites) {
+    NOTREACHED();
+    return;
+  }
+
+  std::set<std::string> open_urls;
+  GetOpenUrls(*tab_strip_model, *top_sites, &open_urls);
+  history::MostVisitedTilesExperiment::RemoveItemsMatchingOpenTabs(
+      open_urls, items);
+
+#endif
+}