Add a cache for FillFontFamilyMap.

chrome/browser/font_family_cache.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.
+
+// Caches font family preferences associated with a PrefService. This class
+// relies on the assumption that each concatenation of map_name + '.' + script
+// is a unique string.
+
+#ifndef CHROME_BROWSER_FONT_FAMILY_CACHE_H_
+#define CHROME_BROWSER_FONT_FAMILY_CACHE_H_
+
+#include "base/containers/hash_tables.h"
+#include "base/gtest_prod_util.h"
+#include "base/prefs/pref_change_registrar.h"
+#include "base/prefs/pref_service.h"
+#include "base/strings/string16.h"
+#include "content/public/common/web_preferences.h"
+
+class PrefService;
+
+FORWARD_DECLARE_TEST(FontFamilyCacheTest, Caching);
+
+namespace chrome {
+
+// Identifies this PrefService cache.
+extern const char* kFontFamilyCacheKey;
+
+class FontFamilyCache : public PrefServiceCache {
+ public:
+  explicit FontFamilyCache(PrefService* prefs);
+  ~FontFamilyCache();

[Avi (use Gerrit), 2014/08/05 01:50:26]

Since FontFamilyCache is going to be deleted through its base-class, the
destructor needs to be virtual, as does PrefServiceCache's

[erikchen, 2014/08/05 17:46:17]

ack, thanks. Too much ObjC.

+
+  // Fills |map| with font family preferences.
+  void FillFontFamilyMap(const char* map_name,
+                         content::ScriptFontFamilyMap* map);
+
+ protected:
+  // Exposed and virtual for testing.
+  // Fetches the font without checking the cache.
+  virtual base::string16 FetchFont(const char* script, const char* map_name);
+
+ private:
+  FRIEND_TEST_ALL_PREFIXES(::FontFamilyCacheTest, Caching);
+
+  // Map from script to font.
+  typedef base::hash_map<const char*, base::string16> ScriptFontMap;
+  // Map from font family to ScriptFontMap.
+  typedef base::hash_map<const char*, ScriptFontMap> FontFamilyMap;
+
+  // Checks the cache for the font. If not present, fetches the font and stores
+  // the result in the cache.
+  // This method needs to be very fast, because its called ~20,000 times on a

[Avi (use Gerrit), 2014/08/05 01:50:26]

s/its/it's/

[erikchen, 2014/08/05 17:46:17]

Done.

+  // fresh launch with an empty profile. It's important to avoid unnecessary
+  // object construction, hence the heavy use of const char* and the minimal use
+  // of std::string.
+  base::string16 FetchAndCacheFont(const char* script, const char* map_name);
+
+  // Called when font family preferences changed.
+  // Invalidates the cached entry, and removes the relevant observer.
+  // Note: It is safe to remove the observer from the pref change callback.
+  void OnPrefsChanged(const std::string& pref_name);
+
+  // Cache of font family preferences.
+  FontFamilyMap font_family_map_;
+
+  // Weak reference.
+  // Note: The lifetime of this object is tied to the lifetime of the
+  // PrefService, so there is no worry about an invalid pointer.
+  const PrefService* prefs_;
+
+  // Reacts to profile changes.
+  PrefChangeRegistrar profile_pref_registrar_;
+
+  DISALLOW_COPY_AND_ASSIGN(FontFamilyCache);
+};
+
+}  // namespace chrome
+
+#endif  // CHROME_BROWSER_FONT_FAMILY_CACHE_H_