2010-03-02 Dirk Pranke <dpranke@chromium.org>
authordpranke@chromium.org <dpranke@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 2 Mar 2010 23:32:55 +0000 (23:32 +0000)
committerdpranke@chromium.org <dpranke@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 2 Mar 2010 23:32:55 +0000 (23:32 +0000)
commite079d0c4f11d4b4b360011dda5b0d84eb32ac81a
tree31fed3b365211d6bbd69db8632383f8e37f702dd
parentf3bbd12194dba14691197f35825f397b2d1f3036
2010-03-02  Dirk Pranke  <dpranke@chromium.org>

        Reviewed by Dimitri Glazkov.

        new-run-webkit-tests on chromium-linux tests to see if
        layout_test_helper exists, but we don't use layout_test_helper on
        linux. The test derefences a None object, and we crash. This fixes
        that.

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

        * Scripts/webkitpy/layout_tests/port/chromium.py:

 2010-03-02  Dirk Pranke  <dpranke@chromium.org>

        Reviewed by Dimitri Glazkov.

        r55388 (bug 35553) worked around a bug in Python's subprocess.Popen()
        that was causing DRT to hang on exit in new-run-webkit-tests.
        Unfortunately, that workaround doesn't work on chromium-win
        (and the script fails completely). The good news is that the check
        isn't actually necessary, and so this change makes it conditional.

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

        * Scripts/webkitpy/layout_tests/port/chromium.py:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@55435 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebKitTools/ChangeLog
WebKitTools/Scripts/webkitpy/layout_tests/port/chromium.py