garden-o-matic command should explain that it is starting a local server
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 21 Sep 2011 14:58:15 +0000 (14:58 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 21 Sep 2011 14:58:15 +0000 (14:58 +0000)
https://bugs.webkit.org/show_bug.cgi?id=68486

Reviewed by Dimitri Glazkov.

In the user study, both participants didn't understand why the
garden-o-matic command appeared to hang.  This patch adds an
explaination of what's going on.  It also adds a confirmation prompt
before launching the browser to give the user a chance to read the
text.  If that's too much of a speed bump, we can iterate.

* Scripts/webkitpy/tool/commands/gardenomatic.py:

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

Tools/ChangeLog
Tools/Scripts/webkitpy/tool/commands/gardenomatic.py

index 331b1b5..9c61ea3 100644 (file)
@@ -1,3 +1,18 @@
+2011-09-21  Adam Barth  <abarth@webkit.org>
+
+        garden-o-matic command should explain that it is starting a local server
+        https://bugs.webkit.org/show_bug.cgi?id=68486
+
+        Reviewed by Dimitri Glazkov.
+
+        In the user study, both participants didn't understand why the
+        garden-o-matic command appeared to hang.  This patch adds an
+        explaination of what's going on.  It also adds a confirmation prompt
+        before launching the browser to give the user a chance to read the
+        text.  If that's too much of a speed bump, we can iterate.
+
+        * Scripts/webkitpy/tool/commands/gardenomatic.py:
+
 2011-09-21  Chang Shu  <cshu@webkit.org>
 
         [Qt] Implement KeyDown function for WebKit2 EventSender.
index f6f4ad9..18df09d 100644 (file)
@@ -33,6 +33,10 @@ class GardenOMatic(AbstractDeclarativeCommand):
     url = "http://build.webkit.org/TestFailures/garden-o-matic.html"
 
     def execute(self, options, args, tool):
-        self._tool.user.open_url(self.url)
+        print "This command runs a local HTTP server that changes your working copy"
+        print "based on the actions you take in the web-based UI."
+        if tool.user.confirm("Open the web-based UI before starting the local server?"):
+            self._tool.user.open_url(self.url)
         httpd = GardeningHTTPServer(httpd_port=8127, config={'tool': tool})
+        print "Local HTTP server started."
         httpd.serve_forever()