nrwt: running w/ no arguments fails

https://bugs.webkit.org/show_bug.cgi?id=77064

Unreviewed, build fix.

This change fixes a regression introduced in r105944
where running without --platform <foo> or one of the other
platform switches would cause a null deref :(.

* Scripts/webkitpy/layout_tests/run_webkit_tests.py:
(main):

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

diff --git a/Tools/ChangeLog b/Tools/ChangeLog
index cc8ef09..a37f449 100644 (file)
--- a/Tools/ChangeLog
+++ b/Tools/ChangeLog
@@ -1,3 +1,17 @@
+2012-01-25  Dirk Pranke  <dpranke@chromium.org>
+
+        nrwt: running w/ no arguments fails
+        https://bugs.webkit.org/show_bug.cgi?id=77064
+
+        Unreviewed, build fix.
+
+        This change fixes a regression introduced in r105944
+        where running without --platform <foo> or one of the other
+        platform switches would cause a null deref :(.
+
+        * Scripts/webkitpy/layout_tests/run_webkit_tests.py:
+        (main):
+
 2012-01-25  Eric Seidel  <eric@webkit.org>
 
         Automatically CC someone from Apple on any changes to Obj-C public API

diff --git a/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py b/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py
index ee71728..9e17dc4 100755 (executable)
--- a/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py
+++ b/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py
@@ -452,7 +452,7 @@ def parse_args(args=None):
 
 def main():
     options, args = parse_args()
-    if options.platform.startswith('test'):
+    if options.platform and options.platform.startswith('test'):
         # It's a bit lame to import mocks into real code, but this allows the user
         # to run tests against the test platform interactively, which is useful for
         # debugging test failures.