diff options
| -rw-r--r-- | AUTHORS | 1 | ||||
| -rw-r--r-- | net/base/mime_util.cc | 12 | ||||
| -rw-r--r-- | webkit/glue/mimetype_unittest.cc | 6 |
3 files changed, 14 insertions, 5 deletions
@@ -41,3 +41,4 @@ Will Hirsch <chromium@willhirsch.co.uk> Yoav Zilberberg <yoav.zilberberg@gmail.com> Joel Stanley <joel@jms.id.au> Jacob Mandelson <jacob@mandelson.org> +Yuri Gorobets <yuri.gorobets@gmail.com> diff --git a/net/base/mime_util.cc b/net/base/mime_util.cc index f34645f..71a7311 100644 --- a/net/base/mime_util.cc +++ b/net/base/mime_util.cc @@ -37,7 +37,7 @@ class MimeUtil : public PlatformMimeUtil { bool MatchesMimeType(const std::string &mime_type_pattern, const std::string &mime_type) const; -private: + private: friend struct DefaultSingletonTraits<MimeUtil>; MimeUtil() { InitializeMimeTypeMaps(); @@ -52,7 +52,7 @@ private: MimeMappings non_image_map_; MimeMappings javascript_map_; MimeMappings view_source_map_; -}; // class MimeUtil +}; // class MimeUtil struct MimeInfo { const char* mime_type; @@ -196,8 +196,14 @@ static const char* const supported_non_image_types[] = { "text/xml", "text/xsl", "text/plain", + // Many users complained about css files served for + // download instead of displaying in the browser: + // http://code.google.com/p/chromium/issues/detail?id=7192 + // So, by including "text/css" into this list we choose Firefox + // behavior - css files will be displayed: + "text/css", "text/", - "image/svg+xml", // SVG is text-based XML, even though it has an image/ type + "image/svg+xml", // SVG is text-based XML, even though it has an image/ type "application/xml", "application/xhtml+xml", "application/rss+xml", diff --git a/webkit/glue/mimetype_unittest.cc b/webkit/glue/mimetype_unittest.cc index 57fbae6..427c0b1 100644 --- a/webkit/glue/mimetype_unittest.cc +++ b/webkit/glue/mimetype_unittest.cc @@ -48,8 +48,10 @@ TEST_F(MimeTypeTests, MimeTypeTests) { // Firefox 3: Display // Internet Explorer 7: Download // Safari 3.2: Download - // We choose to match Safari. - // "text/css", + // We choose to match Firefox due to the lot of complains + // from the users if css files are downloaded: + // http://code.google.com/p/chromium/issues/detail?id=7192 + "text/css", "text/javascript", "text/plain", "application/x-javascript", |