Holger Hans Peter Freyther <holger.freyther@trolltech.com>
authorhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 3 Apr 2008 10:30:59 +0000 (10:30 +0000)
committerhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 3 Apr 2008 10:30:59 +0000 (10:30 +0000)
* Treat http and https as special URLs as well. Change the main.cpp and
  DumpRenderTree.cpp to handle http and https URIs as input.
* I'm not doing the clean up and merging of these two checks now.

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

WebKitTools/ChangeLog
WebKitTools/DumpRenderTree/qt/DumpRenderTree.cpp
WebKitTools/DumpRenderTree/qt/main.cpp

index 22b841a..654dbc6 100644 (file)
@@ -2,6 +2,20 @@
 
         Reviewed by Simon.
 
+        * Treat http and https as special URLs as well. Change the main.cpp and
+        DumpRenderTree.cpp to handle http and https URIs as input.
+        * I'm not doing the clean up and merging of these two checks now.
+        
+
+        * DumpRenderTree/qt/DumpRenderTree.cpp:
+        (WebCore::DumpRenderTree::readStdin):
+        * DumpRenderTree/qt/main.cpp:
+        (main):
+
+2008-04-03  Holger Hans Peter Freyther  <holger.freyther@trolltech.com>
+
+        Reviewed by Simon.
+
         * Remove dumpFrameLoadCallbacks from simple-subframe.html because it is
         located in a directory with the name "loading". And this will autoamtically
         enable dumping.
index 37f1a36..22edc84 100644 (file)
@@ -1,6 +1,7 @@
 /*
  * Copyright (C) 2005, 2006 Apple Computer, Inc.  All rights reserved.
  * Copyright (C) 2006 Nikolas Zimmermann <zimmermann@kde.org>
+ * Copyright (C) 2008 Trolltech ASA
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -184,8 +185,14 @@ void DumpRenderTree::readStdin(int /* socket */)
     //fprintf(stderr, "\n    opening %s\n", line.constData());
     if (line.isEmpty())
         quit();
-    QFileInfo fi(line);
-    open(QUrl::fromLocalFile(fi.absoluteFilePath()));
+
+    if (line.startsWith("http:") || line.startsWith("https:"))
+        open(QUrl(line));
+    else {
+        QFileInfo fi(line);
+        open(QUrl::fromLocalFile(fi.absoluteFilePath()));
+    }
+
     fflush(stdout);
 }
 
index 738be7f..769b8fb 100644 (file)
@@ -157,7 +157,9 @@ int main(int argc, char* argv[])
         dumper.open();
     } else {
         if (!args.last().startsWith("/")
-            && !args.last().startsWith("file:")) {
+            && !args.last().startsWith("file:")
+            && !args.last().startsWith("http:")
+            && !args.last().startsWith("https:")) {
             QString path = QDir::currentPath();
             if (!path.endsWith('/'))
                 path.append('/');