Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(183)

Unified Diff: chrome/renderer/resources/extensions/searchbox_api.js

Issue 18344010: Completing high dpi support for Instant Extended. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync and rebase Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/renderer/searchbox/searchbox.h » ('j') | chrome/renderer/searchbox/searchbox.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/resources/extensions/searchbox_api.js
diff --git a/chrome/renderer/resources/extensions/searchbox_api.js b/chrome/renderer/resources/extensions/searchbox_api.js
index 1acb05ad36fcfac749c31e2725fba43e2d2faf5f..c36cac02a891ffb30d28169b9b89b29b3690fa57 100644
--- a/chrome/renderer/resources/extensions/searchbox_api.js
+++ b/chrome/renderer/resources/extensions/searchbox_api.js
@@ -83,16 +83,24 @@ if (!chrome.embeddedSearch) {
function GetMostVisitedItemsWrapper() {
var mostVisitedItems = GetMostVisitedItems();
for (var i = 0, item; item = mostVisitedItems[i]; ++i) {
+ item.faviconUrl = GenerateThumbnailURL(item.renderViewId, item.rid);
// These properties are private data and should not be returned to
// the page. They are only accessible via getMostVisitedItemData().
item.url = null;
item.title = null;
item.domain = null;
item.direction = null;
+ item.renderViewId = null;
}
return mostVisitedItems;
}
+ function GenerateThumbnailURL(renderViewId, rid) {
samarth 2013/07/24 00:11:05 GenerateFaviconURL
pedro (no code reviews) 2013/07/24 01:10:55 Done.
+ return "chrome-search://favicon/size/16@" +
+ window.devicePixelRatio + "x/" +
+ renderViewId + "/" + rid;
+ }
+
// =======================================================================
// Exported functions
// =======================================================================
« no previous file with comments | « no previous file | chrome/renderer/searchbox/searchbox.h » ('j') | chrome/renderer/searchbox/searchbox.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698