Shrink Document::parseQualifiedName a bit.

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.
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) Research In Motion Limited 2010-2011. All rights reserved.
@@ skipping 4189 matching lines @@
     }
 
     const UChar* characters = name.characters16();
 
     if (isValidNameASCII(characters, length))
         return true;
 
     return isValidNameNonASCII(characters, length);
 }
 
+enum QualifiedNameStatus {
+    QNValid,
+    QNMultipleColons,
+    QNInvalidStartChar,
+    QNInvalidChar,
+    QNEmptyPrefix,
+    QNEmptyLocalName
+};
+
+struct ParseQualifiedNameResult {
+    QualifiedNameStatus status;
+    UChar32 character;
+    ParseQualifiedNameResult() { }
+    explicit ParseQualifiedNameResult(QualifiedNameStatus status) : status(status) { }
+    ParseQualifiedNameResult(QualifiedNameStatus status, UChar32 character) : status(status), character(character) { }
+};
+
 template<typename CharType>
-static bool parseQualifiedNameInternal(const AtomicString& qualifiedName, const CharType* characters, unsigned length, AtomicString& prefix, AtomicString& localName, ExceptionState& exceptionState)
+static ParseQualifiedNameResult parseQualifiedNameInternal(const AtomicString& qualifiedName, const CharType* characters, unsigned length, AtomicString& prefix, AtomicString& localName)
 {
     bool nameStart = true;
     bool sawColon = false;
     int colonPos = 0;
 
     for (unsigned i = 0; i < length;) {
         UChar32 c;
         U16_NEXT(characters, i, length, c)
         if (c == ':') {
-            if (sawColon) {
-                exceptionState.throwDOMException(NamespaceError, "The qualified name provided ('" + qualifiedName + "') contains multiple colons.");
-                return false; // multiple colons: not allowed
-            }
+            if (sawColon)
+                return ParseQualifiedNameResult(QNMultipleColons);
             nameStart = true;
             sawColon = true;
             colonPos = i - 1;
         } else if (nameStart) {
-            if (!isValidNameStart(c)) {
-                StringBuilder message;
-                message.appendLiteral("The qualified name provided ('");
-                message.append(qualifiedName);
-                message.appendLiteral("') contains the invalid name-start character '");
-                message.append(c);
-                message.appendLiteral("'.");
-                exceptionState.throwDOMException(InvalidCharacterError, message.toString());
-                return false;
-            }
+            if (!isValidNameStart(c))
+                return ParseQualifiedNameResult(QNInvalidStartChar, c);
             nameStart = false;
         } else {
-            if (!isValidNamePart(c)) {
-                StringBuilder message;
-                message.appendLiteral("The qualified name provided ('");
-                message.append(qualifiedName);
-                message.appendLiteral("') contains the invalid character '");
-                message.append(c);
-                message.appendLiteral("'.");
-                exceptionState.throwDOMException(InvalidCharacterError, message.toString());
-                return false;
-            }
+            if (!isValidNamePart(c))
+                return ParseQualifiedNameResult(QNInvalidChar, c);
         }
     }
 
     if (!sawColon) {
         prefix = nullAtom;
         localName = qualifiedName;
     } else {
         prefix = AtomicString(characters, colonPos);
-        if (prefix.isEmpty()) {
-            exceptionState.throwDOMException(NamespaceError, "The qualified name provided ('" + qualifiedName + "') has an empty namespace prefix.");
-            return false;
-        }
+        if (prefix.isEmpty())
+            return ParseQualifiedNameResult(QNEmptyPrefix);
         int prefixStart = colonPos + 1;
         localName = AtomicString(characters + prefixStart, length - prefixStart);
     }
 
-    if (localName.isEmpty()) {
-        exceptionState.throwDOMException(NamespaceError, "The qualified name provided ('" + qualifiedName + "') has an empty local name.");
-        return false;
-    }
+    if (localName.isEmpty())
+        return ParseQualifiedNameResult(QNEmptyLocalName);
 
-    return true;
+    return ParseQualifiedNameResult(QNValid);
 }
 
 bool Document::parseQualifiedName(const AtomicString& qualifiedName, AtomicString& prefix, AtomicString& localName, ExceptionState& exceptionState)
 {
     unsigned length = qualifiedName.length();
 
     if (!length) {
         exceptionState.throwDOMException(InvalidCharacterError, "The qualified name provided is empty.");
         return false;
     }
 
+    ParseQualifiedNameResult returnValue;
     if (qualifiedName.is8Bit())
-        return parseQualifiedNameInternal(qualifiedName, qualifiedName.characters8(), length, prefix, localName, exceptionState);
-    return parseQualifiedNameInternal(qualifiedName, qualifiedName.characters16(), length, prefix, localName, exceptionState);
+        returnValue = parseQualifiedNameInternal(qualifiedName, qualifiedName.characters8(), length, prefix, localName);
+    else
+        returnValue = parseQualifiedNameInternal(qualifiedName, qualifiedName.characters16(), length, prefix, localName);
+    if (returnValue.status == QNValid)
+        return true;
+
+    StringBuilder message;
+    message.appendLiteral("The qualified name provided ('");
+    message.append(qualifiedName);
+    message.appendLiteral("') ");
+
+    if (returnValue.status == QNMultipleColons) {
+        message.appendLiteral("contains multiple colons.");
+    } else if (returnValue.status == QNInvalidStartChar) {
+        message.appendLiteral("contains the invalid name-start character '");
+        message.append(returnValue.character);
+        message.appendLiteral("'.");
+    } else if (returnValue.status == QNInvalidChar) {
+        message.appendLiteral("contains the invalid character '");
+        message.append(returnValue.character);
+        message.appendLiteral("'.");
+    } else if (returnValue.status == QNEmptyPrefix) {
+        message.appendLiteral("has an empty namespace prefix.");
+    } else {
+        ASSERT(returnValue.status == QNEmptyLocalName);
+        message.appendLiteral("has an empty local name.");
+    }
+
+    if (returnValue.status == QNInvalidStartChar || returnValue.status == QNInvalidChar)
+        exceptionState.throwDOMException(InvalidCharacterError, message.toString());
+    else
+        exceptionState.throwDOMException(NamespaceError, message.toString());
+    return false;
 }
 
 void Document::setEncodingData(const DocumentEncodingData& newData)
 {
     // It's possible for the encoding of the document to change while we're decoding
     // data. That can only occur while we're processing the <head> portion of the
     // document. There isn't much user-visible content in the <head>, but there is
     // the <title> element. This function detects that situation and re-decodes the
     // document's title so that the user doesn't see an incorrectly decoded title
     // in the title bar.
@@ skipping 1554 matching lines @@
 using namespace blink;
 void showLiveDocumentInstances()
 {
     WeakDocumentSet& set = liveDocumentSet();
     fprintf(stderr, "There are %u documents currently alive:\n", set.size());
     for (WeakDocumentSet::const_iterator it = set.begin(); it != set.end(); ++it) {
         fprintf(stderr, "- Document %p URL: %s\n", *it, (*it)->url().string().utf8().data());
     }
 }
 #endif