Applied the IconType.

chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import <Cocoa/Cocoa.h>
 #include <vector>
 
 #include "base/ref_counted_memory.h"
 #include "base/string_util.h"
 #include "base/sys_string_conversions.h"
@@ skipping 85 matching lines @@
         TabNavigation(0, url, GURL(), title, std::string(),
                       PageTransition::LINK));
     return tab;
   }
 
   void GetFaviconForHistoryItem(HistoryMenuBridge::HistoryItem* item) {
     bridge_->GetFaviconForHistoryItem(item);
   }
 
   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*>&
       favicon_consumer() {
     return bridge_->favicon_consumer_;
   }
 
   BrowserTestHelper browser_test_helper_;
   scoped_ptr<MockBridge> bridge_;
 };
@@ skipping 238 matching lines @@
   SkBitmap bitmap;
   bitmap.setConfig(SkBitmap::kARGB_8888_Config, 25, 25);
   bitmap.allocPixels();
   bitmap.eraseRGB(255, 0, 0);
 
   // Convert it to raw PNG bytes. We totally ignore color order here because
   // we just want to test the roundtrip through the Bridge, not that we can
   // 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;
   item.menu_item.reset([[NSMenuItem alloc] init]);
   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);
   EXPECT_TRUE(item.icon.get());
   EXPECT_TRUE([item.menu_item image]);
 }
 
 }  // namespace