Revert of Moved text decoding to the parser thread

Revert of https://codereview.chromium.org/74513003/
Reason for revert: https://code.google.com/p/chromium/issues/detail?id=329603

TBR=abarth@chromium.org,japhet@chromium.org,eseidel@chromium.org
NOTREECHECKS=true
NOTRY=true

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=164116

[oystein (OOO til 10th of July), 2013-12-18 20:03:00]

Created Revert of Moved text decoding to the parser thread

[commit-bot: I haz the power, 2013-12-18 20:03:04]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Nate Chapin, 2013-12-18 20:06:45]

lgtm

[commit-bot: I haz the power, 2013-12-18 20:06:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oysteine@chromium.org/118573002/1

[commit-bot: I haz the power, 2013-12-18 20:07:12]

Failed to apply patch for Source/core/page/PageSerializer.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/page/PageSerializer.cpp
  Hunk #1 FAILED at 48.
  Hunk #2 succeeded at 77 (offset 1 line).
  1 out of 2 hunks FAILED -- saving rejects to file
Source/core/page/PageSerializer.cpp.rej

Patch:       Source/core/page/PageSerializer.cpp
Index: Source/core/page/PageSerializer.cpp
diff --git a/Source/core/page/PageSerializer.cpp
b/Source/core/page/PageSerializer.cpp
index
8e3e4ee6aabc03406b8a021f57ec9d5f2c9bf499..d41f871976383f86d0b3e5e170d4357621bda884
100644
--- a/Source/core/page/PageSerializer.cpp
+++ b/Source/core/page/PageSerializer.cpp
@@ -48,13 +48,13 @@
 #include "core/editing/MarkupAccumulator.h"
 #include "core/fetch/FontResource.h"
 #include "core/fetch/ImageResource.h"
-#include "core/frame/Frame.h"
 #include "core/html/HTMLFrameOwnerElement.h"
 #include "core/html/HTMLImageElement.h"
 #include "core/html/HTMLInputElement.h"
 #include "core/html/HTMLLinkElement.h"
 #include "core/html/HTMLStyleElement.h"
-#include "core/html/parser/HTMLParserIdioms.h"
+#include "core/html/parser/HTMLMetaCharsetParser.h"
+#include "core/frame/Frame.h"
 #include "core/page/Page.h"
 #include "core/rendering/RenderImage.h"
 #include "core/rendering/style/StyleFetchedImage.h"
@@ -76,15 +76,15 @@
     HTMLElement* element = toHTMLElement(node);
     if (!element->hasTagName(HTMLNames::metaTag))
         return false;
-    HTMLAttributeList attributes;
+    HTMLMetaCharsetParser::AttributeList attributes;
     if (element->hasAttributes()) {
         for (unsigned i = 0; i < element->attributeCount(); ++i) {
             const Attribute* attribute = element->attributeItem(i);
             // FIXME: We should deal appropriately with the attribute if they
have a namespace.
-            attributes.append(std::make_pair(attribute->name().localName(),
attribute->value().string()));
-        }
-    }
-    WTF::TextEncoding textEncoding = encodingFromMetaAttributes(attributes);
+            attributes.append(std::make_pair(attribute->name().toString(),
attribute->value().string()));
+        }
+    }
+    WTF::TextEncoding textEncoding =
HTMLMetaCharsetParser::encodingFromMetaAttributes(attributes);
     return textEncoding.isValid();
 }

[commit-bot: I haz the power, 2013-12-18 21:23:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oysteine@chromium.org/118573002/150001

[commit-bot: I haz the power, 2013-12-18 21:24:16]

Change committed as 164116