Implement API for accessing fonts installed locally on the system.

content/browser/font_access/font_access_service_impl.h

+// Copyright 2016 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 CONTENT_BROWSER_FONT_ACCESS_FONT_ACCESS_SERVICE_IMPL_H_
+#define CONTENT_BROWSER_FONT_ACCESS_FONT_ACCESS_SERVICE_IMPL_H_
+
+#include "base/callback.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
+#include "content/browser/font_access/font_getter.h"
+#include "content/common/font_access/font_access_service.mojom.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
+
+namespace content {
+
+class FontAccessServiceImpl : public FontAccessService {
+ public:
+  FontAccessServiceImpl(
+      base::Callback<scoped_ptr<FontCacheMap>(void)> getter,
+      mojo::InterfaceRequest<FontAccessService> request);
+  ~FontAccessServiceImpl() override;
+
+  // Returns a list of FontAccessDescriptions on the callback.
+  void GetFontList(const GetFontListCallback& callback) override;
+
+  // Returns a Mojo SharedBuffer of the font's binary on the callback.
+  void GetFontData(const mojo::String& family,
+                   const mojo::String& style,
+                   const GetFontDataCallback& callback) override;
+
+  static void Create(
+      mojo::InterfaceRequest<FontAccessService> request);
+
+ private:
+  // Used to pass around all the information about a blob from disk.
+  struct BlobInfo {
+    BlobInfo();
+    ~BlobInfo();
+
+    uint64_t size;
+    base::Time last_modified;
+    std::string mime;
+  };

[mlamouri (slow - plz ping), 2016/01/26 17:17:11]

nit: DISALLOW_COPY_AND_ASSIGN?

[Daniel Nishi, 2016/01/26 18:36:10]

I think I use the implicit move, so the macro will cause a compile error.

+
+  // Clobbers the cache and then fetches the font list.
+  void GetFontListFromCache(const GetFontListCallback& callback);
+
+  // Pulls the font binary path from the cache and returns the font binary in a
+  // shared buffer on the callback, if it exists. If it does not exist, returns
+  // an empty shared buffer on the callback.
+  void GetFontDataInternal(const std::string& family,
+                           const std::string& style,
+                           const GetFontDataCallback& callback);
+  // Returns the font binary path if it exists in the cache.
+  // Otherwise, returns an empty string.
+  const std::string FindPathInCache(const std::string family,
+                                    const std::string style);

[mlamouri (slow - plz ping), 2016/01/26 17:17:11]

const std::string&?

[Daniel Nishi, 2016/01/26 18:36:10]

Done.

+
+  void GetFileDataCallback(
+      const FontAccessService::GetFontDataCallback& callback,
+      const std::string path,

[mlamouri (slow - plz ping), 2016/01/26 17:17:11]

ditto

[Daniel Nishi, 2016/01/26 18:36:10]

Done.

+      const BlobInfo info);
+
+  void ClobberCacheAndReply(const base::Closure closure);
+  void SwapCacheAndReply(const base::Closure closure, scoped_ptr<FontCacheMap>);
+
+  static BlobInfo GetFileInfo(const std::string path);
+
+  // Returns a shared buffer with the binary data from a file path.
+  static mojo::ScopedSharedBufferHandle GetSharedBuffer(const std::string path,
+                                                        uint64_t size);
+  static void PassBufferAndSize(
+      const FontAccessService::GetFontDataCallback& callback,
+      uint64_t size,
+      mojo::ScopedSharedBufferHandle handle);
+
+  scoped_ptr<FontCacheMap> font_cache_map_;
+  // The binding between this object and the other end of the pipe.
+  mojo::StrongBinding<FontAccessService> binding_;
+  base::Callback<scoped_ptr<FontCacheMap>(void)> cache_map_getter_;
+  base::WeakPtrFactory<FontAccessServiceImpl> weak_factory_;
+};

[mlamouri (slow - plz ping), 2016/01/26 17:17:11]

DISALLOW_COPY_AND_ASSIGN.

[Daniel Nishi, 2016/01/26 18:36:10]

Done.

+}

[mlamouri (slow - plz ping), 2016/01/26 17:17:11]

nit: leave a blank line before and add `  // namespace content`

[Daniel Nishi, 2016/01/26 18:36:10]

Done.

+
+#endif