Chromium Code Reviews| Index: chrome/renderer/searchbox/searchbox_extension.cc |
| diff --git a/chrome/renderer/searchbox/searchbox_extension.cc b/chrome/renderer/searchbox/searchbox_extension.cc |
| index 17e7bf15a118c7bc1cc64b0afbdc8239a520900a..1bfd744c3e8c9c04897a6ad34e562b96d840316f 100644 |
| --- a/chrome/renderer/searchbox/searchbox_extension.cc |
| +++ b/chrome/renderer/searchbox/searchbox_extension.cc |
| @@ -152,8 +152,24 @@ v8::Local<v8::Object> GenerateMostVisitedItem( |
| v8::Int32::New(isolate, render_view_id)); |
| obj->Set(v8::String::NewFromUtf8(isolate, "rid"), |
| v8::Int32::New(isolate, restricted_id)); |
| - obj->Set(v8::String::NewFromUtf8(isolate, "thumbnailUrl"), |
| - GenerateThumbnailURL(isolate, render_view_id, restricted_id)); |
| + |
| + if (!mv_item.thumbnail.spec().empty()) { |
|
Mathieu
2015/07/28 13:32:42
put a comment above this block
fserb
2015/07/28 17:55:30
Done.
|
| + v8::Local<v8::Array> thumbs = v8::Array::New(isolate, 2); |
| + thumbs->Set(0, UTF8ToV8String(isolate, base::StringPrintf( |
| + "chrome-search://thumb2/%s", |
| + mv_item.url.spec().c_str()))); |
| + thumbs->Set(1, UTF8ToV8String(isolate, mv_item.thumbnail.spec())); |
| + obj->Set(v8::String::NewFromUtf8(isolate, "thumbnailUrls"), thumbs); |
| + } else { |
| + obj->Set(v8::String::NewFromUtf8(isolate, "thumbnailUrl"), |
| + GenerateThumbnailURL(isolate, render_view_id, restricted_id)); |
| + } |
| + |
| + if (!mv_item.favicon.spec().empty()) { |
|
Mathieu
2015/07/28 13:32:43
...and this block, saying that Chrome can pre-popu
fserb
2015/07/28 17:55:30
Done.
|
| + obj->Set(v8::String::NewFromUtf8(isolate, "faviconUrl"), |
| + UTF8ToV8String(isolate, mv_item.favicon.spec())); |
| + } |
| + |
| if (IsIconNTPEnabled()) { |
| // Update website http://www.chromium.org/embeddedsearch when we make this |
| // permanent. |