Log the expected icon size when no suitable icon for a banner is found.

chrome/browser/manifest/manifest_icon_selector.h

 // Copyright 2015 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 CHROME_BROWSER_MANIFEST_MANIFEST_ICON_SELECTOR_H_
 #define CHROME_BROWSER_MANIFEST_MANIFEST_ICON_SELECTOR_H_
 
 #include "base/macros.h"
 #include "content/public/common/manifest.h"
 #include "url/gurl.h"
@@ skipping 24 matching lines @@
   //
   // Any icon returned will be close as possible to |ideal_icon_size_in_dp|
   // with a size not less than |minimum_icon_size_in_dp|.
   //
   // Returns the icon url if a suitable icon is found. An empty URL otherwise.
   static GURL FindBestMatchingIcon(
       const std::vector<content::Manifest::Icon>& icons,
       int ideal_icon_size_in_dp,
       int minimum_icon_size_in_dp);
 
+  static int ConvertIconSizeFromDpToPx(int icon_size_in_dp);
+
  private:
   ManifestIconSelector(int ideal_icon_size_in_px,
                        int minimum_icon_size_in_px);
   virtual ~ManifestIconSelector() {}
 
   // Runs the algorithm to find the best matching icon in the icons listed in
   // the Manifest.
   // Returns the icon url if a suitable icon is found. An empty URL otherwise.
-  int FindBestMatchingIcon(
-      const std::vector<content::Manifest::Icon>& icons,
-      float density);
+  int FindBestMatchingIcon(const std::vector<content::Manifest::Icon>& icons);
 
   // Runs an algorithm only based on icon declared sizes. It will try to find
   // size that is the closest to preferred_icon_size_in_pixels_ but bigger than
   // preferred_icon_size_in_pixels_ if possible.
   // Returns the index of a suitable icon if one is found. -1 otherwise.
   int FindBestMatchingIconForDensity(
       const std::vector<content::Manifest::Icon>& icons,
       float density);
 
   // Returns whether the |preferred_icon_size_in_pixels_| is in |sizes|.
   bool IconSizesContainsPreferredSize(const std::vector<gfx::Size>& sizes);
 
   // Returns whether a size bigger than |minimun_icon_size_in_pixels_| is in
   // |sizes|.
   bool IconSizesContainsBiggerThanMinimumSize(
       const std::vector<gfx::Size>& sizes);
 
   // Returns an array containing the items in |icons| without the unsupported
   // image MIME types.
   static std::vector<content::Manifest::Icon> FilterIconsByType(
       const std::vector<content::Manifest::Icon>& icons);
 
   // Returns whether the 'any' (ie. gfx::Size(0,0)) is in |sizes|.
   static bool IconSizesContainsAny(const std::vector<gfx::Size>& sizes);
 
   const int ideal_icon_size_in_px_;
   const int minimum_icon_size_in_px_;
+  const float density_;
 
   friend class ManifestIconSelectorTest;
 
   DISALLOW_COPY_AND_ASSIGN(ManifestIconSelector);
 };
 
 #endif  // CHROME_BROWSER_MANIFEST_MANIFEST_ICON_SELECTOR_H_