Index: chrome/browser/android/most_visited_sites.cc |
diff --git a/chrome/browser/android/most_visited_sites.cc b/chrome/browser/android/most_visited_sites.cc |
index a41f6c83c7c456fd3c7d41653259aa6e1a236c4f..1e138f191eca86a2e035ca5a7c323b381fda238b 100644 |
--- a/chrome/browser/android/most_visited_sites.cc |
+++ b/chrome/browser/android/most_visited_sites.cc |
@@ -171,6 +171,18 @@ MostVisitedSites::Suggestion::Suggestion(const base::string16& title, |
MostVisitedSites::Suggestion::Suggestion(const base::string16& title, |
const GURL& url, |
+ MostVisitedSource source, |
+ const base::FilePath& large_icon_path) |
+ : title(title), |
+ url(url), |
+ source(source), |
+ large_icon_path(large_icon_path), |
+ provider_index(-1) { |
+ DCHECK_EQ(MostVisitedSites::WHITELIST, source); |
+} |
+ |
+MostVisitedSites::Suggestion::Suggestion(const base::string16& title, |
+ const GURL& url, |
MostVisitedSource source) |
: title(title), url(url), source(source), provider_index(-1) {} |
@@ -572,8 +584,9 @@ MostVisitedSites::CreateWhitelistEntryPointSuggestions( |
continue; |
} |
- whitelist_suggestions.push_back(make_scoped_ptr(new Suggestion( |
- whitelist->title(), whitelist->entry_point(), WHITELIST))); |
+ whitelist_suggestions.push_back(make_scoped_ptr( |
+ new Suggestion(whitelist->title(), whitelist->entry_point(), WHITELIST, |
+ whitelist->large_icon_path()))); |
if (whitelist_suggestions.size() >= num_whitelist_suggestions) |
break; |
} |
@@ -802,17 +815,20 @@ void MostVisitedSites::NotifyMostVisitedURLsObserver() { |
} |
std::vector<base::string16> titles; |
std::vector<std::string> urls; |
+ std::vector<std::string> large_icon_paths; |
titles.reserve(num_suggestions); |
urls.reserve(num_suggestions); |
for (const auto& suggestion : current_suggestions_) { |
titles.push_back(suggestion->title); |
urls.push_back(suggestion->url.spec()); |
+ large_icon_paths.push_back(suggestion->large_icon_path.value()); |
} |
JNIEnv* env = AttachCurrentThread(); |
DCHECK_EQ(titles.size(), urls.size()); |
Java_MostVisitedURLsObserver_onMostVisitedURLsAvailable( |
env, observer_.obj(), ToJavaArrayOfStrings(env, titles).obj(), |
- ToJavaArrayOfStrings(env, urls).obj()); |
+ ToJavaArrayOfStrings(env, urls).obj(), |
+ ToJavaArrayOfStrings(env, large_icon_paths).obj()); |
} |
void MostVisitedSites::OnPopularSitesAvailable(bool success) { |