ShortcutInfo::favicon is now a gfx::ImageFamily instead of gfx::Image.

chrome/browser/ui/cocoa/omnibox/omnibox_view_mac_unittest.mm

 // Copyright (c) 2012 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 "chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.h"
 
 #include "chrome/browser/search/search.h"
 #include "chrome/browser/ui/cocoa/cocoa_profile_test.h"
 #include "chrome/browser/ui/omnibox/omnibox_edit_controller.h"
 #include "chrome/browser/ui/omnibox/omnibox_popup_model.h"
 #include "chrome/browser/ui/omnibox/omnibox_popup_view.h"
 #include "chrome/browser/ui/toolbar/toolbar_model_delegate.h"
 #include "chrome/browser/ui/toolbar/toolbar_model_impl.h"
 #include "testing/platform_test.h"
+#include "ui/gfx/image/image.h"

[pkotwicz, 2013/04/04 18:34:40]

Do you need this change?

[Matt Giuca, 2013/04/05 06:30:13]

Yes. This stopped compiling when I removed the image.h include from one of the
other .h files. The correct solution is for this module to include what it uses.

 
 namespace {
 
 class MockOmniboxEditModel : public OmniboxEditModel {
  public:
   MockOmniboxEditModel(OmniboxView* view,
                        OmniboxEditController* controller,
                        Profile* profile)
       : OmniboxEditModel(view, controller, profile),
         up_or_down_count_(0) {
@@ skipping 167 matching lines @@
 
   // With popup open verify that pressing up and down arrow works.
   popup_view.set_is_open(true);
   model->set_up_or_down_count(0);
   view.OnDoCommandBySelector(@selector(moveDown:));
   EXPECT_EQ(1, model->up_or_down_count());
   model->set_up_or_down_count(0);
   view.OnDoCommandBySelector(@selector(moveUp:));
   EXPECT_EQ(-1, model->up_or_down_count());
 }