Ensure there are always nice icons for bookmark apps.

extensions/common/image_util.cc

 // Copyright 2014 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 "extensions/browser/image_util.h"
+#include "extensions/common/image_util.h"
 
 #include <stdint.h>
 #include <vector>
 
 #include "base/strings/string_number_conversions.h"
 #include "third_party/skia/include/core/SkColor.h"
 
 namespace extensions {
 namespace image_util {
 
@@ skipping 18 matching lines @@
   // Convert the string to an integer and make sure it is in the correct value
   // range.
   std::vector<uint8_t> color_bytes;
   if (!base::HexStringToBytes(formatted_color, &color_bytes))
     return false;
 
   *result = SkColorSetARGB(255, color_bytes[0], color_bytes[1], color_bytes[2]);
   return true;
 }
 
+std::string GenerateCSSColorString(SkColor color) {
+  char buffer[8];
+
+  sprintf(buffer, "#%02X%02X%02X", SkColorGetR(color), SkColorGetG(color),
+          SkColorGetB(color));
+  return std::string(buffer);

[calamity, 2014/12/09 03:46:12]

Use base::StringPrintf instead. Fixed size buffers make me nervous.

[benwells, 2014/12/09 08:28:35]

Done.

+}
+
 }  // namespace image_util
 }  // namespace extensions