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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: net/base/mime_sniffer_unittest.cc

Issue 2883833002: net: Do not sniff XHTML content (Closed)
Patch Set: . Created 3 years, 7 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
« net/base/mime_sniffer.cc ('K') | « net/base/mime_sniffer.cc ('k') | third_party/WebKit/LayoutTests/external/wpt/dom/nodes/Document-createElement-namespace-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/base/mime_sniffer_unittest.cc
diff --git a/net/base/mime_sniffer_unittest.cc b/net/base/mime_sniffer_unittest.cc
index 590dd77f4a079dcd34befce0c18746fe101c6cd8..d4b704a0e820bc209be9ed853bc8912d732d958e 100644
--- a/net/base/mime_sniffer_unittest.cc
+++ b/net/base/mime_sniffer_unittest.cc
@@ -339,17 +339,16 @@ TEST(MimeSnifferTest, XMLTest) {
EXPECT_EQ("text/plain",
SniffMimeType("<?xml?><notafeed", std::string(), "text/plain"));
- // Positive test for the two instances we upgrade to XHTML.
+ // Positive test for the one instance we upgrade to XHTML.
EXPECT_EQ("application/xhtml+xml",
SniffMimeType("<html xmlns=\"http://www.w3.org/1999/xhtml\">",
std::string(),
"text/xml"));
- EXPECT_EQ("application/xhtml+xml",
- SniffMimeType("<html xmlns=\"http://www.w3.org/1999/xhtml\">",
- std::string(),
- "application/xml"));
// Following our behavior with HTML, don't call other mime types XHTML.
+ EXPECT_EQ("application/xml",
+ SniffMimeType("<html xmlns=\"http://www.w3.org/1999/xhtml\">",
+ std::string(), "application/xml"));
EXPECT_EQ("text/plain",
SniffMimeType("<html xmlns=\"http://www.w3.org/1999/xhtml\">",
std::string(),
« net/base/mime_sniffer.cc ('K') | « net/base/mime_sniffer.cc ('k') | third_party/WebKit/LayoutTests/external/wpt/dom/nodes/Document-createElement-namespace-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698