Chromium Code Reviews| Index: chrome/browser/ui/search/search_tab_helper.cc |
| diff --git a/chrome/browser/ui/search/search_tab_helper.cc b/chrome/browser/ui/search/search_tab_helper.cc |
| index 2071396b4812a19312f03f65820f297ce966f40e..3fb3bb234a14e00b05c08ab3820d7a8a13368020 100644 |
| --- a/chrome/browser/ui/search/search_tab_helper.cc |
| +++ b/chrome/browser/ui/search/search_tab_helper.cc |
| @@ -11,7 +11,6 @@ |
| #include "base/strings/string16.h" |
| #include "base/strings/string_util.h" |
| #include "chrome/browser/chrome_notification_types.h" |
| -#include "chrome/browser/history/most_visited_tiles_experiment.h" |
| #include "chrome/browser/profiles/profile.h" |
| #include "chrome/browser/search/instant_service.h" |
| #include "chrome/browser/search/instant_service_factory.h" |
| @@ -420,7 +419,6 @@ void SearchTabHelper::ThemeInfoChanged(const ThemeBackgroundInfo& theme_info) { |
| void SearchTabHelper::MostVisitedItemsChanged( |
| const std::vector<InstantMostVisitedItem>& items) { |
| std::vector<InstantMostVisitedItem> items_copy(items); |
|
Jered
2014/08/01 15:34:19
I think you do not need to make a copy now.
Mathieu
2014/08/01 15:46:54
Done. silly me.
|
| - MaybeRemoveMostVisitedItems(&items_copy); |
| ipc_router_.SendMostVisitedItems(items_copy); |
| } |
| @@ -428,18 +426,6 @@ void SearchTabHelper::OmniboxStartMarginChanged(int omnibox_start_margin) { |
| ipc_router_.SetOmniboxStartMargin(omnibox_start_margin); |
| } |
| -void SearchTabHelper::MaybeRemoveMostVisitedItems( |
| - std::vector<InstantMostVisitedItem>* items) { |
| - if (!delegate_) |
| - return; |
| - |
| - if (!history::MostVisitedTilesExperiment::IsDontShowOpenURLsEnabled()) |
| - return; |
| - |
| - history::MostVisitedTilesExperiment::RemoveItemsMatchingOpenTabs( |
| - delegate_->GetOpenUrls(), items); |
| -} |
| - |
| void SearchTabHelper::FocusOmnibox(OmniboxFocusState state) { |
| // TODO(kmadhusu): Move platform specific code from here and get rid of #ifdef. |
| #if !defined(OS_ANDROID) |