Patch from Eli Fidler to properly decode data: urls.
authorstaikos <staikos@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 9 Jul 2007 21:25:12 +0000 (21:25 +0000)
committerstaikos <staikos@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 9 Jul 2007 21:25:12 +0000 (21:25 +0000)
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24124 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKitQt/Api/qwebnetworkinterface.cpp
WebKitQt/ChangeLog

index 5c7302f..99c265e 100644 (file)
@@ -1,6 +1,7 @@
 /*
   Copyright (C) 2006 Enrico Ros <enrico.ros@m31engineering.it>
   Copyright (C) 2007 Trolltech ASA
+  Copyright (C) 2007 Staikos Computing Services Inc.  <info@staikos.net>
   
   This library is free software; you can redistribute it and/or
   modify it under the terms of the GNU Library General Public
@@ -622,8 +623,7 @@ void QWebNetworkInterfacePrivate::parseDataUrl(QWebNetworkJob* job)
 
     int index = data.indexOf(',');
     if (index != -1) {
-        header = data.mid(5, index - 5);
-        header = header.toLower();
+        header = data.mid(5, index - 5).toLower();
         //qDebug() << "header=" << header;
         data = data.mid(index+1);
         //qDebug() << "data=" << data;
@@ -633,14 +633,13 @@ void QWebNetworkInterfacePrivate::parseDataUrl(QWebNetworkJob* job)
             base64 = true;
             header = header.left(header.length() - 7);
             //qDebug() << "mime=" << header;
-        }        
+        }
     } else {
         data = QByteArray();
     }
+    data = QUrl::fromPercentEncoding(data).toLatin1();
     if (base64) {
         data = QByteArray::fromBase64(data);
-    } else {
-        data = QUrl::fromPercentEncoding(data).toLatin1();
     }
 
     if (header.isEmpty()) 
index a1a0b63..a57282b 100644 (file)
@@ -1,3 +1,12 @@
+2007-07-10  Eli Fidler  <eli@staikos.net>
+
+        Reviewed by George Staikos.
+
+        Properly url-decode data urls.
+
+        * Api/qwebnetworkinterface.cpp:
+        (QWebNetworkInterfacePrivate::parseDataUrl):
+
 2007-07-09  Adam Treat  <adam@staikos.net>
 
         Reviewed by George Staikos.