| Index: ios/chrome/browser/ui/webui/history/browsing_history_handler.cc
|
| diff --git a/ios/chrome/browser/ui/webui/history/browsing_history_handler.cc b/ios/chrome/browser/ui/webui/history/browsing_history_handler.cc
|
| index 37c52034e2eb02ad8dc39377df9d9c196834bbfd..da8d6d2823c2121d4651f9a393542c9215a88939 100644
|
| --- a/ios/chrome/browser/ui/webui/history/browsing_history_handler.cc
|
| +++ b/ios/chrome/browser/ui/webui/history/browsing_history_handler.cc
|
| @@ -26,7 +26,6 @@
|
| #include "components/history/core/browser/top_sites.h"
|
| #include "components/history/core/browser/web_history_service.h"
|
| #include "components/keyed_service/core/service_access_type.h"
|
| -#include "components/prefs/pref_service.h"
|
| #include "components/strings/grit/components_strings.h"
|
| #include "components/sync_driver/device_info.h"
|
| #include "components/sync_driver/device_info_tracker.h"
|
| @@ -39,7 +38,6 @@
|
| #include "ios/chrome/browser/history/history_utils.h"
|
| #include "ios/chrome/browser/history/top_sites_factory.h"
|
| #include "ios/chrome/browser/history/web_history_service_factory.h"
|
| -#include "ios/chrome/browser/pref_names.h"
|
| #include "ios/chrome/browser/sync/ios_chrome_profile_sync_service_factory.h"
|
| #include "ios/chrome/browser/ui/show_privacy_settings_util.h"
|
| #include "ios/chrome/browser/ui/webui/history/favicon_source.h"
|
| @@ -143,8 +141,7 @@ BrowsingHistoryHandler::HistoryEntry::HistoryEntry(
|
| const std::string& client_id,
|
| bool is_search_result,
|
| const base::string16& snippet,
|
| - bool blocked_visit,
|
| - const std::string& accept_languages) {
|
| + bool blocked_visit) {
|
| this->entry_type = entry_type;
|
| this->url = url;
|
| this->title = title;
|
| @@ -154,7 +151,6 @@ BrowsingHistoryHandler::HistoryEntry::HistoryEntry(
|
| this->is_search_result = is_search_result;
|
| this->snippet = snippet;
|
| this->blocked_visit = blocked_visit;
|
| - this->accept_languages = accept_languages;
|
| }
|
|
|
| BrowsingHistoryHandler::HistoryEntry::HistoryEntry()
|
| @@ -193,8 +189,7 @@ scoped_ptr<base::DictionaryValue> BrowsingHistoryHandler::HistoryEntry::ToValue(
|
| scoped_ptr<base::DictionaryValue> result(new base::DictionaryValue());
|
| SetUrlAndTitle(result.get());
|
|
|
| - base::string16 domain =
|
| - url_formatter::IDNToUnicode(url.host(), accept_languages);
|
| + base::string16 domain = url_formatter::IDNToUnicode(url.host());
|
| // When the domain is empty, use the scheme instead. This allows for a
|
| // sensible treatment of e.g. file: URLs when group by domain is on.
|
| if (domain.empty())
|
| @@ -611,15 +606,14 @@ void BrowsingHistoryHandler::QueryComplete(const base::string16& search_text,
|
| history::QueryResults* results) {
|
| DCHECK_EQ(0U, query_results_.size());
|
| query_results_.reserve(results->size());
|
| - const std::string accept_languages = GetAcceptLanguages();
|
|
|
| for (size_t i = 0; i < results->size(); ++i) {
|
| history::URLResult const& page = (*results)[i];
|
| // TODO(dubroy): Use sane time (crbug.com/146090) here when it's ready.
|
| - query_results_.push_back(HistoryEntry(
|
| - HistoryEntry::LOCAL_ENTRY, page.url(), page.title(), page.visit_time(),
|
| - std::string(), !search_text.empty(), page.snippet().text(),
|
| - page.blocked_visit(), accept_languages));
|
| + query_results_.push_back(
|
| + HistoryEntry(HistoryEntry::LOCAL_ENTRY, page.url(), page.title(),
|
| + page.visit_time(), std::string(), !search_text.empty(),
|
| + page.snippet().text(), page.blocked_visit()));
|
| }
|
|
|
| // The items which are to be written into results_info_value_ are also
|
| @@ -653,7 +647,6 @@ void BrowsingHistoryHandler::WebHistoryQueryComplete(
|
| const base::DictionaryValue* results_value) {
|
| base::TimeDelta delta = base::TimeTicks::Now() - start_time;
|
| UMA_HISTOGRAM_TIMES("WebHistory.ResponseTime", delta);
|
| - const std::string accept_languages = GetAcceptLanguages();
|
|
|
| // If the response came in too late, do nothing.
|
| // TODO(dubroy): Maybe show a banner, and prompt the user to reload?
|
| @@ -720,7 +713,7 @@ void BrowsingHistoryHandler::WebHistoryQueryComplete(
|
| web_history_query_results_.push_back(
|
| HistoryEntry(HistoryEntry::REMOTE_ENTRY, gurl, title, time,
|
| client_id, !search_text.empty(), base::string16(),
|
| - /* blocked_visit */ false, accept_languages));
|
| + /* blocked_visit */ false));
|
| }
|
| }
|
| }
|
| @@ -803,12 +796,6 @@ static bool DeletionsDiffer(const history::URLRows& deleted_rows,
|
| return false;
|
| }
|
|
|
| -std::string BrowsingHistoryHandler::GetAcceptLanguages() const {
|
| - ios::ChromeBrowserState* browser_state =
|
| - ios::ChromeBrowserState::FromWebUIIOS(web_ui());
|
| - return browser_state->GetPrefs()->GetString(prefs::kAcceptLanguages);
|
| -}
|
| -
|
| void BrowsingHistoryHandler::OnURLsDeleted(
|
| history::HistoryService* history_service,
|
| bool all_history,
|
|
|