Do not skip ink for ideographic scripts

third_party/WebKit/Source/platform/fonts/GlyphBuffer.h

 /*
  * Copyright (C) 2006, 2009, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2007-2008 Torch Mobile Inc.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 123 matching lines @@
     //        yield correctly positioned RTL glyphs without any post-shape
     //        munging).
     SECURITY_DCHECK(!m_offsets.isEmpty());
     for (unsigned i = 0; i + 1 < m_offsets.size(); ++i)
       m_offsets[i] = totalWidth - m_offsets[i + 1];
     m_offsets.back() = totalWidth - afterOffset;
 
     m_offsets.reverse();
   }
 
+  void saveSkipInkExceptions() {
+    m_saveSkipInkExceptions = true;
+    m_skipInkExceptions.reserveInitialCapacity(2048);
+  }
+
+  bool shouldSaveSkipInkExceptions() const { return m_saveSkipInkExceptions; }
+
+  void addIsSkipInkException(bool value) { m_skipInkExceptions.append(value); }

[drott, 2017/01/02 15:07:46]

Can we add a DCHECK_EQ(m_offsets.size(), m_skipInkExceptions.size()); here?

+
+  Vector<bool>& skipInkExceptions() { return m_skipInkExceptions; }

[drott, 2017/01/02 15:07:46]

Can we add the same debug assertion here, before returning?
DCHECK_EQ(m_offsets.size(), m_skipInkExceptions.size())

+
  protected:
   Vector<const SimpleFontData*, 2048> m_fontData;
   Vector<Glyph, 2048> m_glyphs;
 
   // Glyph positioning: either x-only offsets, or interleaved x,y offsets
   // (depending on the buffer-wide positioning mode). This matches the
   // glyph positioning format used by Skia.
   Vector<float, 2048> m_offsets;
+
+  Vector<bool> m_skipInkExceptions;

[drott, 2017/01/02 15:07:46]

Could you add a comment like:
// Flag vector of identical size to m_offset, true when glyph is to be exempted
from ink skipping, false otherwise.

Also, minor, but how about making this a std::unique_ptr initially nullptr and
allocating it when saveSkipInkExceptions is called, then removing the boolean?
And checking for existence of the vector in shouldSaveSkipInkExceptions()? 

+
+  bool m_saveSkipInkExceptions = false;
 };
 
 }  // namespace blink
 
 #endif