Supporting high dpi favicons in Instant Extended.

ui/gfx/favicon_size.h

 // Copyright (c) 2011 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_GFX_FAVICON_SIZE_H_
 #define UI_GFX_FAVICON_SIZE_H_
 
 #include "ui/base/ui_export.h"
 
 namespace gfx {
 
 // Size (along each axis) of the favicon.
 UI_EXPORT extern const int kFaviconSize;
 
+// Default size of the favicon.
+UI_EXPORT extern const int kSizeInDip;

[sky, 2013/07/01 13:48:00]

AFAICT all of these are only used in one place. Why do we need them here?
Additionally kSizeInDip is not descriptive. Size of what?

[pedro (no code reviews), 2013/07/01 22:33:30]

I moved this constants to where they're being used, and renamed them to
something more descriptive.

+
+// 2x size of the favicon.
+UI_EXPORT extern const int kSizeInDip2x;
+
+// 4x size of the favicon.
+UI_EXPORT extern const int kSizeInDip4x;
+
 // If the width or height is bigger than the favicon size, a new width/height
 // is calculated and returned in width/height that maintains the aspect
 // ratio of the supplied values.
 UI_EXPORT void CalculateFaviconTargetSize(int* width, int* height);
 
 }  // namespace gfx
 
 #endif  // UI_GFX_FAVICON_SIZE_H_