Local salient image storage for enhanced bookmark experiment.

components/bookmarks_enhanced/image_store_util.h

+// 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.
+
+#ifndef COMPONENTS_BOOKMARKS_ENHANCED_IMAGE_STORE_UTIL_H_
+#define COMPONENTS_BOOKMARKS_ENHANCED_IMAGE_STORE_UTIL_H_
+
+#include "base/memory/ref_counted.h"
+#include "base/memory/ref_counted_memory.h"
+#include "ui/gfx/geometry/size.h"
+#include "ui/gfx/image/image.h"
+
+namespace image_store_util {

[sky, 2014/05/06 16:14:20]

Namespaces generally should not match files. Instead use a namespace of
bookmarks_enhanced. In fact all this code should be in bookmark_enhanced
namespace.

[Kibeom Kim (inactive), 2014/05/07 00:09:46]

Done.

+// The two methods below archive and unarchive an image to and from a bag of
+// bytes. There is no API on gfx::Image capable of doing it while preserving the
+// scale of the image.
+
+// Encode the UIImage representation of a gfx::Image.

[sky, 2014/05/06 16:14:20]

UIImage?

[Kibeom Kim (inactive), 2014/05/07 00:09:46]

Done.

+scoped_refptr<base::RefCountedMemory> BytesForImage(gfx::Image image);

[sky, 2014/05/06 16:14:20]

const gfx::Image&

[Kibeom Kim (inactive), 2014/05/07 00:09:46]

Done.

+
+// Decode the UIImage in the bytes and returns a gfx::Image.

[sky, 2014/05/06 16:14:20]

UIImage?

[Kibeom Kim (inactive), 2014/05/07 00:09:46]

Done.

+gfx::Image ImageForBytes(const scoped_refptr<base::RefCountedMemory>& data);
+
+}
+
+#endif  // COMPONENTS_BOOKMARKS_ENHANCED_IMAGE_STORE_UTIL_H_