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

Unified Diff: webkit/glue/simple_webmimeregistry_impl.cc

Issue 165178: Revert 22813 - Merge 22027 Fix a DCHECK we hit when a mime type isn't ASCII.... (Closed) Base URL: svn://chrome-svn/chrome/branches/195/src/
Patch Set: Created 11 years, 4 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/glue/simple_webmimeregistry_impl.cc
===================================================================
--- webkit/glue/simple_webmimeregistry_impl.cc (revision 22814)
+++ webkit/glue/simple_webmimeregistry_impl.cc (working copy)
@@ -14,30 +14,18 @@
using WebKit::WebString;
using WebKit::WebMimeRegistry;
-namespace {
-
-// Convert a WebString to ASCII, falling back on an empty string in the case
-// of a non-ASCII string.
-const std::string& AsASCII(const WebString& string) {
- if (!IsStringASCII(string))
- return EmptyString();
- return UTF16ToASCII(string);
-}
-
-} // namespace
-
namespace webkit_glue {
WebMimeRegistry::SupportsType SimpleWebMimeRegistryImpl::supportsImageMIMEType(
const WebString& mime_type) {
- if (!net::IsSupportedImageMimeType(AsASCII(mime_type).c_str()))
+ if (!net::IsSupportedImageMimeType(UTF16ToASCII(mime_type).c_str()))
return WebMimeRegistry::IsNotSupported;
return WebMimeRegistry::IsSupported;
}
WebMimeRegistry::SupportsType SimpleWebMimeRegistryImpl::supportsJavaScriptMIMEType(
const WebString& mime_type) {
- if (!net::IsSupportedJavascriptMimeType(AsASCII(mime_type).c_str()))
+ if (!net::IsSupportedJavascriptMimeType(UTF16ToASCII(mime_type).c_str()))
return WebMimeRegistry::IsNotSupported;
return WebMimeRegistry::IsSupported;
}
@@ -45,12 +33,12 @@
WebMimeRegistry::SupportsType SimpleWebMimeRegistryImpl::supportsMediaMIMEType(
const WebString& mime_type, const WebString& codecs) {
// Not supporting the container is a flat-out no.
- if (!net::IsSupportedMediaMimeType(AsASCII(mime_type).c_str()))
+ if (!net::IsSupportedMediaMimeType(UTF16ToASCII(mime_type).c_str()))
return IsNotSupported;
// If we don't recognize the codec, it's possible we support it.
std::vector<std::string> parsed_codecs;
- net::ParseCodecString(AsASCII(codecs).c_str(), &parsed_codecs);
+ net::ParseCodecString(UTF16ToASCII(codecs).c_str(), &parsed_codecs);
if (!net::AreSupportedMediaCodecs(parsed_codecs))
return MayBeSupported;
@@ -60,7 +48,7 @@
WebMimeRegistry::SupportsType SimpleWebMimeRegistryImpl::supportsNonImageMIMEType(
const WebString& mime_type) {
- if (!net::IsSupportedNonImageMimeType(AsASCII(mime_type).c_str()))
+ if (!net::IsSupportedNonImageMimeType(UTF16ToASCII(mime_type).c_str()))
return WebMimeRegistry::IsNotSupported;
return WebMimeRegistry::IsSupported;
}
@@ -84,7 +72,7 @@
WebString SimpleWebMimeRegistryImpl::preferredExtensionForMIMEType(
const WebString& mime_type) {
FilePath::StringType file_extension;
- net::GetPreferredExtensionForMimeType(AsASCII(mime_type),
+ net::GetPreferredExtensionForMimeType(UTF16ToASCII(mime_type),
&file_extension);
return FilePathStringToWebString(file_extension);
}
Property changes on: webkit\glue\simple_webmimeregistry_impl.cc
___________________________________________________________________
Modified: svn:mergeinfo
Reverse-merged /trunk/src/webkit/glue/simple_webmimeregistry_impl.cc:r22027
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698