Fix for the font files being maped multiple times (Fontconfig).

content/common/font_config_ipc_linux.h

 // Copyright (c) 2012 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_FONT_CONFIG_IPC_LINUX_H_
 #define CONTENT_COMMON_FONT_CONFIG_IPC_LINUX_H_
 
 #include "base/compiler_specific.h"
+#include "base/containers/hash_tables.h"
+#include "base/synchronization/lock.h"
 #include "third_party/skia/include/ports/SkFontConfigInterface.h"
 
 #include <string>
 
 namespace content {
 
 // FontConfig implementation for Skia that proxies out of process to get out
 // of the sandbox. See http://code.google.com/p/chromium/wiki/LinuxSandboxIPC
 class FontConfigIPC : public SkFontConfigInterface {
  public:
@@ skipping 11 matching lines @@
   enum Method {
     METHOD_MATCH = 0,
     METHOD_OPEN = 1,
   };
 
   enum {
     kMaxFontFamilyLength = 2048
   };
 
  private:
+  class MappedFontFile;
+  friend class MappedFontFile;
+
+  void RemoveMappedFontFile(MappedFontFile* mapped_font_file);

[Daniel Erat, 2014/11/07 15:07:36]

add a comment:

  // Removes |mapped_font_file| from |mapped_font_files_|.
  // Does not delete the passed-in object.

[Krzysztof Olczyk, 2015/01/14 12:42:23]

Done

+
   const int fd_;
+  base::Lock stream_opening_lock_;

[Daniel Erat, 2014/11/07 15:07:36]

please add a comment documenting what this lock protects

[Krzysztof Olczyk, 2015/01/14 12:42:23]

Done

+  // Maps font identity ID to the memory-mapped file with font data:

[Daniel Erat, 2014/11/07 15:07:36]

s/:/./

[Krzysztof Olczyk, 2015/01/14 12:42:23]

Done

[Krzysztof Olczyk, 2015/01/14 12:42:23]

Done

+  base::hash_map<uint32_t, MappedFontFile*> mapped_font_files_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_COMMON_FONT_CONFIG_IPC_LINUX_H_