Moved text decoding to the parser thread

Source/core/html/parser/HTMLIdentifier.h

Index: Source/core/html/parser/HTMLIdentifier.h
diff --git a/Source/core/html/parser/HTMLIdentifier.h b/Source/core/html/parser/HTMLIdentifier.h
index 003401314234ae5f0dfa3c35500594457900f15d..92af4ed97237364c660db60fd07c78a0fb632c53 100644
--- a/Source/core/html/parser/HTMLIdentifier.h
+++ b/Source/core/html/parser/HTMLIdentifier.h
@@ -56,6 +56,19 @@ public:
             m_string = String(vector);
     }
 
+    HTMLIdentifier(const AtomicString& str, CharacterWidth width)

[abarth-chromium, 2013/11/27 18:18:08]

str -> string

Please use complete works in variable names.

It seems very strange that we would want to take an AtomicString parameter here.
 Why not a String parameter?

[oystein (OOO til 10th of July), 2013/11/27 18:46:33]

Done.
If I need to handle both the 8bit and the 16bit case, then yeah a String
parameter would make sense. If I was correct in the assumption below then maybe
it should be a QualifiedName parameter.

+        : m_string(findIfKnown(str.characters16(), str.length()))

[abarth-chromium, 2013/11/27 18:18:08]

How do you know it's safe to call characters16?  What if the string consists of
8 bit characters?

[oystein (OOO til 10th of July), 2013/11/27 18:46:33]

This is only ever used by the QualifiedName -> HTMLIdentifier conversion in
PageSerializer.cpp, I was assuming that'd always be unicode but if that's
incorrect I can add some handling of both (why do we have so many
almost-identical string classes, and keep having to convert between them?).

+    {
+        if (m_string.impl())
+            return;
+        if (width == Likely8Bit)
+            m_string = StringImpl::create8BitIfPossible(str.characters16(), str.length());
+        else if (width == Force8Bit)
+            m_string = String::make8BitFrom16BitSource(str.characters16(), str.length());
+        else
+            m_string = str;
+    }
+
     // asString should only be used on the main thread.
     const String& asString() const;
     // asStringImpl() is safe to call from any thread.
@@ -65,6 +78,9 @@ public:
 
     bool isSafeToSendToAnotherThread() const { return m_string.isSafeToSendToAnotherThread(); }
 
+    bool operator==(const QualifiedName&) const;
+    bool operator!=(const QualifiedName& b) const;

[abarth-chromium, 2013/11/27 18:18:08]

Why does one of these functions have a formal parameter name while the other
does not?

I'd prefer we omitted these operators.  It's better for callers to be explicit
about what comparison operators they want to perform.

[oystein (OOO til 10th of July), 2013/11/27 18:46:33]

Hm what ambiguity is there when comparing a QualifiedName object and a
HTMLIdentifier object?

+
 #ifndef NDEBUG
     static bool isKnown(const StringImpl*);
 #endif