IPC interface for font management.

content/common/fontmgr_proxy.h

+// Copyright (c) 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 CONTENT_COMMON_FONTMGR_PROXY_H

[brettw, 2014/03/11 21:38:15]

Can this file be in content/renderer? I think if we need it shared between
different child process types, then it goes in content/child. content/common is
stuff stared between browser and child processes, which I don't think this is.

[bungeman-chromium, 2014/03/13 22:33:31]

Done. Hmmm... yes. As of right now the only user will be the renderer. If we
need to change that later, we can.

+#define CONTENT_COMMON_FONTMGR_PROXY_H
+
+#include "base/values.h"
+#include "content/renderer/render_thread_impl.h"
+#include "third_party/skia/include/ports/SkFontMgr_indirect.h"
+
+class SkData;
+class SkDataTable;
+
+namespace content {
+
+class CONTENT_EXPORT FontMgrProxy : public SkRemotableFontMgr {
+ public:
+  // If we support multiple of these at the same time, this constructor
+  // could take a parameter stating which one to talk to.
+  // There could even be some statics for getting the available ones.
+  FontMgrProxy() { }
+  virtual ~FontMgrProxy() { }
+
+  virtual SkDataTable* getFamilyNames() const OVERRIDE;
+  virtual SkRemotableFontIdentitySet* getIndexIdentities(int index) const OVERRIDE;

[brettw, 2014/03/11 21:38:15]

Long lines (in this case I'd normally just wrap the arg on the next line.

[bungeman-chromium, 2014/03/13 22:33:31]

Done.

+  virtual SkFontIdentity matchIndexStyle(int familyIndex, const SkFontStyle&) const OVERRIDE;
+
+  virtual SkRemotableFontIdentitySet* matchName(const char familyName[]) const OVERRIDE;
+  virtual SkFontIdentity matchNameStyle(const char familyName[], const SkFontStyle&) const OVERRIDE;
+  virtual SkFontIdentity matchNameStyleCharacter(const char familyName[],
+                                                 const SkFontStyle&,
+                                                 const char bpc47[],
+                                                 uint32_t character) const
+                                                 OVERRIDE;
+
+  virtual SkStream* getData(int dataId) const OVERRIDE;
+
+ private:
+  typedef SkRemotableFontMgr INHERITED;

[brettw, 2014/03/11 21:38:15]

I'm not sure what this is for.

[bungeman-chromium, 2014/03/13 22:33:31]

Done. Left-over from the initial implementation. Skia has a debug ref-counting
setup where this is used, but this isn't in Skia, so doesn't need it. Removed.

+};
+
+}  // namespace content
+
+#endif  // SKIA_EXT_FONTMGR_PROXY_H