[Qt] FrameLoaderClientQt::canShowMIMEType should always return true
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 16 Nov 2011 15:12:30 +0000 (15:12 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 16 Nov 2011 15:12:30 +0000 (15:12 +0000)
https://bugs.webkit.org/show_bug.cgi?id=72169

Patch by Jesus Sanchez-Palencia <jesus.palencia@openbossa.org> on 2011-11-16
Reviewed by Simon Hausmann.

Currently FrameLoaderClientQt::canShowMIMEType is called after
dispatchDecidePolicyForResponse. If decidePolicyForResponse returns
PolicyUse and canShowMIMEType returns false, then we have a bug.
Therefore, FrameLoaderClientQt::canShowMIMEType now will always
return true, following the implemenation of WebKit2.

* WebCoreSupport/FrameLoaderClientQt.cpp:
(WebCore::FrameLoaderClientQt::canShowMIMEType):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@100448 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/qt/ChangeLog
Source/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.cpp

index e0263a83dbce97d351f712493403c12aa7be3e01..a23973be53c142d2b053d871bbe3158bbd72b6c4 100644 (file)
@@ -1,3 +1,19 @@
+2011-11-16  Jesus Sanchez-Palencia  <jesus.palencia@openbossa.org>
+
+        [Qt] FrameLoaderClientQt::canShowMIMEType should always return true
+        https://bugs.webkit.org/show_bug.cgi?id=72169
+
+        Reviewed by Simon Hausmann.
+
+        Currently FrameLoaderClientQt::canShowMIMEType is called after
+        dispatchDecidePolicyForResponse. If decidePolicyForResponse returns
+        PolicyUse and canShowMIMEType returns false, then we have a bug.
+        Therefore, FrameLoaderClientQt::canShowMIMEType now will always
+        return true, following the implemenation of WebKit2.
+
+        * WebCoreSupport/FrameLoaderClientQt.cpp:
+        (WebCore::FrameLoaderClientQt::canShowMIMEType):
+
 2011-11-15  Huang Dongsung  <luxtella@company100.net>
 
         [TexMap][Qt] Start PageClientQ[Graphics]Widget::syncTimer only when the syncTimer
index 04b1e3f8902404991a81106d026fcfbc0271cfff..fafc29fa01fe865c50412304c2bef396a179727b 100644 (file)
@@ -650,25 +650,10 @@ bool FrameLoaderClientQt::canShowMIMETypeAsHTML(const String& MIMEType) const
     notImplemented();
     return false;
 }
-    
+
 bool FrameLoaderClientQt::canShowMIMEType(const String& MIMEType) const
 {
-    String type = MIMEType;
-    type.makeLower();
-    if (MIMETypeRegistry::isSupportedImageMIMEType(type))
-        return true;
-
-    if (MIMETypeRegistry::isSupportedNonImageMIMEType(type))
-        return true;
-
-    if (MIMETypeRegistry::isSupportedMediaMIMEType(type))
-        return true;
-
-    if (m_frame && m_frame->settings()  && m_frame->settings()->arePluginsEnabled()
-        && PluginDatabase::installedPlugins()->isMIMETypeRegistered(type))
-        return true;
-
-    return false;
+    return true;
 }
 
 bool FrameLoaderClientQt::representationExistsForURLScheme(const String&) const