[BlackBerry] Avoid assertion failure in App Cache.
authorstaikos@webkit.org <staikos@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Aug 2012 16:36:02 +0000 (16:36 +0000)
committerstaikos@webkit.org <staikos@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Aug 2012 16:36:02 +0000 (16:36 +0000)
https://bugs.webkit.org/show_bug.cgi?id=93926

Reviewed by Rob Buis.

Short-circuit data url decoding if we know it's not a data URL to
avoid an assertion failure in debug mode.  No functional change.

No new tests since this is just a debug mode assertion failure.

* loader/appcache/ApplicationCacheGroup.cpp:
(WebCore::ApplicationCacheGroup::createResourceHandle):

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

Source/WebCore/ChangeLog
Source/WebCore/loader/appcache/ApplicationCacheGroup.cpp

index 513888e..3de6bd8 100644 (file)
@@ -1,3 +1,18 @@
+2012-08-14  George Staikos  <staikos@webkit.org>
+
+        [BlackBerry] Avoid assertion failure in App Cache.
+        https://bugs.webkit.org/show_bug.cgi?id=93926
+
+        Reviewed by Rob Buis.
+
+        Short-circuit data url decoding if we know it's not a data URL to
+        avoid an assertion failure in debug mode.  No functional change.
+
+        No new tests since this is just a debug mode assertion failure.
+
+        * loader/appcache/ApplicationCacheGroup.cpp:
+        (WebCore::ApplicationCacheGroup::createResourceHandle):
+
 2012-08-14  Thiago Marcos P. Santos  <thiago.santos@intel.com>
 
         [EFL] Do not warn when the default theme is not found
index 4e42938..2dfe34c 100644 (file)
@@ -498,7 +498,9 @@ PassRefPtr<ResourceHandle> ApplicationCacheGroup::createResourceHandle(const KUR
             target = ResourceRequest::targetTypeFromMimeType(mimeType);
     }
     if (target == ResourceRequest::TargetIsUnspecified) {
-        String mimeType = mimeTypeFromDataURL(url);
+        String mimeType;
+        if (url.protocolIsData())
+            mimeType = mimeTypeFromDataURL(url);
         if (!mimeType.isEmpty())
             target = ResourceRequest::targetTypeFromMimeType(mimeType);
     }