Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1323)

Unified Diff: Source/modules/encoding/TextEncoder.cpp

Issue 145973021: Implement "replacement" text encoding. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Verify replacement name Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/modules/encoding/TextEncoder.cpp
diff --git a/Source/modules/encoding/TextEncoder.cpp b/Source/modules/encoding/TextEncoder.cpp
index 1a26de62da91172943bc0777b555554af6340709..b04edde6843bff4ff18f87d6499557d0017cb2c0 100644
--- a/Source/modules/encoding/TextEncoder.cpp
+++ b/Source/modules/encoding/TextEncoder.cpp
@@ -55,12 +55,12 @@ PassRefPtr<TextEncoder> TextEncoder::create(const String& utfLabel, ExceptionSta
return 0;
}
- return adoptRef(new TextEncoder(encoding.name()));
+ return adoptRef(new TextEncoder(encodingLabel, name));
}
-TextEncoder::TextEncoder(const String& encoding)
- : m_encoding(encoding)
- , m_codec(newTextCodec(m_encoding))
+TextEncoder::TextEncoder(const String& label, const String& name)
+ : m_encoding(name)
+ , m_codec(newTextCodec(label))
{
}
@@ -70,7 +70,7 @@ TextEncoder::~TextEncoder()
String TextEncoder::encoding() const
{
- String name = String(m_encoding.name()).lower();
+ String name = m_encoding.lower();
ASSERT(name == "utf-8" || name == "utf-16le" || name == "utf-16be");
return name;
}

Powered by Google App Engine
This is Rietveld 408576698