Move app list search result icon sizes to app_list_constants.

ui/app_list/search_result.h

 // 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.
 
 #ifndef UI_APP_LIST_SEARCH_RESULT_H_
 #define UI_APP_LIST_SEARCH_RESULT_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 91 matching lines @@
     return actions_;
   }
   void SetActions(const Actions& sets);
 
   bool is_installing() const { return is_installing_; }
   void SetIsInstalling(bool is_installing);
 
   int percent_downloaded() const { return percent_downloaded_; }
   void SetPercentDownloaded(int percent_downloaded);
 
+  // Returns the size at which this result's icon should be displayed.
+  int GetPreferredIconSize();

[tapted, 2014/08/13 08:09:06]

nit: declare const?

[tapted, 2014/08/13 08:15:22]

also [optional nit] to me, Get..Size() feels like it should return a gfx::Size.
Maybe Get..Dimension()?  (or return a gfx::Size - might make the call sites
neater on balance)

[calamity, 2014/08/13 08:25:09]

Done.

[calamity, 2014/08/13 08:25:09]

Done.

+
   void NotifyItemInstalled();
   void NotifyItemUninstalled();
 
   void AddObserver(SearchResultObserver* observer);
   void RemoveObserver(SearchResultObserver* observer);
 
   // Opens the result.
   virtual void Open(int event_flags);
 
   // Invokes a custom action on the result.
@@ skipping 30 matching lines @@
   int percent_downloaded_;
 
   ObserverList<SearchResultObserver> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(SearchResult);
 };
 
 }  // namespace app_list
 
 #endif  // UI_APP_LIST_SEARCH_RESULT_H_