Fix document title with LINE SEPARATOR/PARAGRAPH SEPARATOR/LINE TABULATION

third_party/WebKit/Source/core/dom/Document.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All
  * rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
@@ skipping 1295 matching lines @@
 static inline String canonicalizedTitle(Document* document,
                                         const String& title) {
   unsigned length = title.length();
   unsigned builderIndex = 0;
   const CharacterType* characters = title.getCharacters<CharacterType>();
 
   StringBuffer<CharacterType> buffer(length);
 
   // Replace control characters with spaces and collapse whitespace.
   bool pendingWhitespace = false;
+  const UChar lineTabulationCharacter = 0x0b;

[tkent, 2016/11/08 03:57:15]

Please use symbols defined in wtf/text/CharacterNames.h.  If it doesn't have
these character names, please add them to it.

+  const UChar spaceCharacter = 0x20;
+  const UChar deleteCharacter = 0x7f;
   for (unsigned i = 0; i < length; ++i) {
     UChar32 c = characters[i];
-    if (c <= 0x20 || c == 0x7F ||
-        (WTF::Unicode::category(c) &
-         (WTF::Unicode::Separator_Line | WTF::Unicode::Separator_Paragraph))) {
+    if ((c <= spaceCharacter && c != lineTabulationCharacter) ||
+        c == deleteCharacter) {
       if (builderIndex != 0)
         pendingWhitespace = true;
     } else {
       if (pendingWhitespace) {
         buffer[builderIndex++] = ' ';
         pendingWhitespace = false;
       }
       buffer[builderIndex++] = c;
     }
   }
@@ skipping 5147 matching lines @@
 }
 
 void showLiveDocumentInstances() {
   WeakDocumentSet& set = liveDocumentSet();
   fprintf(stderr, "There are %u documents currently alive:\n", set.size());
   for (Document* document : set)
     fprintf(stderr, "- Document %p URL: %s\n", document,
             document->url().getString().utf8().data());
 }
 #endif