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

Unified Diff: third_party/WebKit/Source/core/html/parser/TextResourceDecoder.h

Issue 1721373002: UTF-8 detector for pages missing encoding info (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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: third_party/WebKit/Source/core/html/parser/TextResourceDecoder.h
diff --git a/third_party/WebKit/Source/core/html/parser/TextResourceDecoder.h b/third_party/WebKit/Source/core/html/parser/TextResourceDecoder.h
index 15cd6c6bce1ebe5a85ac2a499dbeb0117289353b..87b847955aeb7160e43d36c8fe05dcfee815f312 100644
--- a/third_party/WebKit/Source/core/html/parser/TextResourceDecoder.h
+++ b/third_party/WebKit/Source/core/html/parser/TextResourceDecoder.h
@@ -80,16 +80,18 @@ public:
private:
- // TextResourceDecoder does three kind of encoding detection:
+ // TextResourceDecoder does four kinds of encoding detection:
// 1. By BOM,
// 2. By Content if |m_contentType| is not |PlainTextContext|
- // (e.g. <meta> tag for HTML), and
- // 3. By detectTextEncoding().
+ // (e.g. <meta> tag for HTML),
+ // 3. By isUTF8Encoded() to detect if the document
+ // is of UTF-8, and
+ // 4. By detectTextEncoding().
enum EncodingDetectionOption {
- // Use 1. + 2. + 3.
+ // Use 1. + 2. + 4.
UseAllAutoDetection,
- // Use 1. + 2.
+ // Use 1. + 2. + 3.
UseContentAndBOMBasedDetection,
// Use None of them.

Powered by Google App Engine
This is Rietveld 408576698