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

Unified Diff: LayoutTests/fast/dom/document-attribute-js-null.html

Issue 1180793002: Document attribute charset and its aliases to return UTF-8 (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Removed defaultCharset changes Created 5 years, 6 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: LayoutTests/fast/dom/document-attribute-js-null.html
diff --git a/LayoutTests/fast/dom/document-attribute-js-null.html b/LayoutTests/fast/dom/document-attribute-js-null.html
index 7d99616ad1d45ca7051ef209798c45ae327ecb75..de379ef30fa730f5dd4c1f48842afaca2692a053 100644
--- a/LayoutTests/fast/dom/document-attribute-js-null.html
+++ b/LayoutTests/fast/dom/document-attribute-js-null.html
@@ -66,7 +66,10 @@
attributes: [
{name: 'xmlVersion', expectedExceptionCode: 9},
{name: 'documentURI', expectedNull: null},
- {name: 'charset', expectedNull: undefined},
+ {name: 'charset', expectedNull: 'UTF-8'},
dominicc (has gone to gerrit) 2015/06/22 01:23:31 I think the case of this is wrong per https://enco
Habib Virji 2015/06/22 14:07:30 Yes it does not match it as utf-8, but these test
philipj_slow 2015/06/23 08:32:56 Actually https://dom.spec.whatwg.org/#dom-document
+ {name: 'defaultCharset', expectedNull: undefined},
+ {name: 'characterSet', expectedNull: 'UTF-8'},
+ {name: 'inputEncoding', expectedNull: 'UTF-8'},
{name: 'selectedStylesheetSet', expectedNull: null}
]
},

Powered by Google App Engine
This is Rietveld 408576698