2010-09-21 Dirk Pranke <dpranke@chromium.org>
authordpranke@chromium.org <dpranke@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 22 Sep 2010 03:13:38 +0000 (03:13 +0000)
committerdpranke@chromium.org <dpranke@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 22 Sep 2010 03:13:38 +0000 (03:13 +0000)
        Unreviewed, build fix.

        new-run-webkit-tests: fix bug introduced in r68008 where if you
        specify --chromium and no --platform, and you're running on
        windows, you use the 'chromium-win' port by default instead of the
        version-specific port. This breaks the buildbots.

        It will be good for this refactoring to settle down so I can
        rewrite the logic for default ports and the unit tests to be
        clearer (and the testing more comprehensive).

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

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

WebKitTools/ChangeLog
WebKitTools/Scripts/webkitpy/layout_tests/port/chromium_win.py

index e04f9bf..4362e25 100644 (file)
@@ -1,5 +1,20 @@
 2010-09-21  Dirk Pranke  <dpranke@chromium.org>
 
+        Unreviewed, build fix.
+
+        new-run-webkit-tests: fix bug introduced in r68008 where if you
+        specify --chromium and no --platform, and you're running on
+        windows, you use the 'chromium-win' port by default instead of the
+        version-specific port. This breaks the buildbots.
+
+        It will be good for this refactoring to settle down so I can
+        rewrite the logic for default ports and the unit tests to be
+        clearer (and the testing more comprehensive).
+
+        * Scripts/webkitpy/layout_tests/port/chromium_win.py:
+
+2010-09-21  Dirk Pranke  <dpranke@chromium.org>
+
         Reviewed by Adam Barth.
 
         Modify the Port interface to take only a series of keyword arguments
index 333cfa4..d2b0265 100644 (file)
@@ -42,7 +42,7 @@ class ChromiumWinPort(chromium.ChromiumPort):
     """Chromium Win implementation of the Port class."""
 
     def __init__(self, **kwargs):
-        kwargs.setdefault('port_name', 'chromium-win')
+        kwargs.setdefault('port_name', 'chromium-win' + self.version())
         chromium.ChromiumPort.__init__(self, **kwargs)
 
     def setup_environ_for_server(self):