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 1296 matching lines @@
   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;
   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 <= 0x20 && c != 0x0b) || c == 0x7F) {

[skobes, 2016/11/03 14:31:41]

Can you use named constants for these instead of magic numbers?  Also link to
relevant spec in comments if possible.

       if (builderIndex != 0)
         pendingWhitespace = true;
     } else {
       if (pendingWhitespace) {
         buffer[builderIndex++] = ' ';
         pendingWhitespace = false;
       }
       buffer[builderIndex++] = c;
     }
   }
@@ skipping 5132 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