ui/gfx: Refactor favicon_size.h

- Put it into a gfx namespace.
- Define the constant into the source file instead.
- Rename the function from calc_favicon_target_size to CalculateFaviconTargetSize,
  so it matches with our style guide.
- Move the implementation of the function to the source file.

R=sky@chromium.org

Review URL: http://codereview.chromium.org/8037011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102760 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 25 insertions, 0 deletions

diff --git a/ui/gfx/favicon_size.cc b/ui/gfx/favicon_size.cc
new file mode 100644
index 0000000..d0ba48a
--- /dev/null
+++ b/ui/gfx/favicon_size.cc
@@ -0,0 +1,25 @@
+// 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.
+
+#include "ui/gfx/favicon_size.h"
+
+namespace gfx {
+
+const int kFaviconSize = 16;
+
+void CalculateFaviconTargetSize(int* width, int* height) {
+  if (*width > kFaviconSize || *height > kFaviconSize) {
+    // Too big, resize it maintaining the aspect ratio.
+    float aspect_ratio = static_cast<float>(*width) /
+                         static_cast<float>(*height);
+    *height = kFaviconSize;
+    *width = static_cast<int>(aspect_ratio * *height);
+    if (*width > kFaviconSize) {
+      *width = kFaviconSize;
+      *height = static_cast<int>(*width / aspect_ratio);
+    }
+  }
+}
+
+}  // namespace gfx