| Index: chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm
|
| diff --git a/chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm b/chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm
|
| index 7491ed2a7c329662ff27d777cb349d81ff205470..ee085b0b72ebf6edfb264ba8861d91b6efafca66 100644
|
| --- a/chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm
|
| +++ b/chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm
|
| @@ -103,11 +103,8 @@ class HistoryMenuBridgeTest : public CocoaTest {
|
| }
|
|
|
| void GotFaviconData(FaviconService::Handle handle,
|
| - bool know_favicon,
|
| - scoped_refptr<RefCountedBytes> data,
|
| - bool expired,
|
| - GURL url) {
|
| - bridge_->GotFaviconData(handle, know_favicon, data, expired, url);
|
| + history::FaviconData favicon) {
|
| + bridge_->GotFaviconData(handle, favicon);
|
| }
|
|
|
| CancelableRequestConsumerTSimple<HistoryMenuBridge::HistoryItem*>&
|
| @@ -366,7 +363,7 @@ TEST_F(HistoryMenuBridgeTest, GotFaviconData) {
|
| // make icons look pretty.
|
| std::vector<unsigned char> raw;
|
| gfx::PNGCodec::EncodeBGRASkBitmap(bitmap, true, &raw);
|
| - scoped_refptr<RefCountedBytes> bytes(new RefCountedBytes(raw));
|
| + scoped_refptr<RefCountedBytes> bytes();
|
|
|
| // Set up the HistoryItem.
|
| HistoryMenuBridge::HistoryItem item;
|
| @@ -374,7 +371,13 @@ TEST_F(HistoryMenuBridgeTest, GotFaviconData) {
|
| GetFaviconForHistoryItem(&item);
|
|
|
| // Pretend to be called back.
|
| - GotFaviconData(item.icon_handle, true, bytes, false, GURL());
|
| + history::FaviconData favicon;
|
| + favicon.known_icon = true;
|
| + favicon.image_data = new RefCountedBytes(raw);
|
| + favicon.expired = false;
|
| + favicon.icon_url = GURL();
|
| + favicon.icon_type = history::FAVICON;
|
| + GotFaviconData(item.icon_handle, favicon);
|
|
|
| // Make sure the callback works.
|
| EXPECT_FALSE(item.icon_requested);
|
|
|