Tools/ChangeLog

   1 2011-02-20  Dirk Pranke  <dpranke@chromium.org>
   2
   3         Reviewed by Ojan Vafai.
   4
   5         test-webkitpy: stop skipping a bunch of tests on win32
   6         https://bugs.webkit.org/show_bug.cgi?id=54788
   7
   8         * Scripts/webkitpy/common/net/testoutputset_unittest.py:
   9         * Scripts/webkitpy/test/main.py:
  10
  11 2011-02-20  Dirk Pranke  <dpranke@chromium.org>
  12
  13         Reviewed by Ojan Vafai.
  14
  15         Fix various bugs related to handling of dir separators and real
  16         vs. fake filesystems that were causing test-webkitpy to fail
  17         tests on win32. These bugs have a variety of causes but are
  18         lumped together because they should be easy to review.
  19
  20         https://bugs.webkit.org/show_bug.cgi?id=54700
  21
  22         * Scripts/webkitpy/common/net/testoutput.py:
  23           Here we were using os.path.sep but not escaping it prior to
  24           using in a regexp, which  caused bad things on win32.
  25         * Scripts/webkitpy/common/net/testoutput_unittest.py:
  26           Same here.
  27         * Scripts/webkitpy/layout_tests/port/chromium_unittest.py:
  28           Here we were using the native filesystem instead of the mock
  29           filesystem, making some tests unnecessarily platform dependent.
  30           Also we weren't setting the default configuration for the
  31           TestChromiumWinPort.
  32         * Scripts/webkitpy/layout_tests/update_webgl_conformance_tests.py:
  33           Here we *shouldn't* use os.path.join(), just so the code becomes
  34           less platform dependent.
  35         * Scripts/webkitpy/style/checker.py:
  36           Here we should be using os.path.sep instead of "/"; the tests
  37           require the native filesystem.
  38         * Scripts/webkitpy/style/checkers/cpp_unittest.py:
  39           Here the underlying code is using os.path.sep, so we probably
  40           should as well.
  41         * Scripts/webkitpy/style/test_expectations_unittest.py:
  42           Here we should be using the filesystem hanging of the port
  43           object (which is the mocked filesystem) rather than the native
  44           one.
  45
  46 2011-02-20  Anders Carlsson  <andersca@apple.com>
  47
  48         Reviewed by Maciej Stachowiak.
  49
  50         Crash when a plug-in requests a javascript: url that destroys the plug-in
  51         https://bugs.webkit.org/show_bug.cgi?id=54837
  52         <rdar://problem/9005475>
  53
  54         Add new plug-in test.
  55
  56         * DumpRenderTree/DumpRenderTree.gypi:
  57         * DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:
  58         * DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp:
  59         (PluginTest::NPN_GetURL):
  60         * DumpRenderTree/TestNetscapePlugIn/PluginTest.h:
  61         * DumpRenderTree/TestNetscapePlugIn/Tests/GetURLWithJavaScriptURLDestroyingPlugin.cpp: Added.
  62         (GetURLWithJavaScriptURLDestroyingPlugin::GetURLWithJavaScriptURLDestroyingPlugin):
  63         (GetURLWithJavaScriptURLDestroyingPlugin::NPP_New):
  64         * DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin.vcproj:
  65         * DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro:
  66
  67 2011-02-19  Patrick Gansterer  <paroga@webkit.org>
  68
  69         Reviewed by Andreas Kling.
  70
  71         Make WinCE a core builder
  72         https://bugs.webkit.org/show_bug.cgi?id=54801
  73
  74         * Scripts/webkitpy/common/net/buildbot/buildbot.py:
  75         * Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py:
  76
  77 2011-02-19  Lukasz Slachciak  <l.slachciak@samsung.com>
  78
  79         Reviewed by Andreas Kling.
  80
  81         Fixed problem with launching EWebLauncher with "run-launcher --efl". Incorrect path referring to non-existing .libs directory ws set.
  82         https://webkit.org/b/54778
  83
  84         * Scripts/webkitdirs.pm:
  85
  86 2011-02-18  David Levin  <levin@chromium.org>
  87
  88         Reviewed by Eric Seidel.
  89
  90         check-webkit-style: Misses brace style error when the line with the { has a } in it.
  91         https://bugs.webkit.org/show_bug.cgi?id=54769
  92
  93         * Scripts/webkitpy/style/checkers/cpp.py: Fixed the check for the close brace
  94           to only look after the last open brace, so that the open brace in this line
  95           "} else {" will still be able to trigger the error.
  96         * Scripts/webkitpy/style/checkers/cpp_unittest.py: Added related unit tests.
  97
  98 2011-02-18  Zan Dobersek  <zandobersek@gmail.com>
  99
 100         Reviewed by Martin Robinson.
 101
 102         [Gtk] Re-enable meter tag support
 103         https://bugs.webkit.org/show_bug.cgi?id=54762
 104
 105         Meter tag is supported on the Gtk port, enable it by default.
 106
 107         * Scripts/build-webkit:
 108
 109 2011-02-18  David Levin  <levin@chromium.org>
 110
 111         Reviewed by Eric Seidel.
 112
 113         check-webkit-style falsely complains about WebKitGTK+ public headers
 114         https://bugs.webkit.org/show_bug.cgi?id=54650
 115
 116         * Scripts/webkitpy/style/checker.py: Add some exceptions for the gtk files.
 117         * Scripts/webkitpy/style/checkers/cpp.py: Restrict the checks for WEBKIT_API
 118           to the chromium directory (and improved the checks slightly).
 119         * Scripts/webkitpy/style/checkers/cpp_unittest.py: Added corresponding tests.
 120
 121 2011-02-18  Dirk Pranke  <dpranke@chromium.org>
 122
 123         Reviewed by James Robinson.
 124
 125         new-run-webkit-tests: enable multiple processes by default on
 126         mac. This change removes the artificial restrictions we placed
 127         on the # of child processes we used with the old threading
 128         model, and switches to the new message-based model and multiple
 129         processes, where available. If multiple processes are not
 130         available (Leopard / Python 2.5), then we use the 'inline' model
 131         instead of the 'old-threads' model on the 'mac' port or one
 132         process on the 'chromium-mac' port. We need additional testing
 133         to see if the new 'threads' model works reliably and is worth
 134         supporting, or if we should just live with things being slightly
 135         slow.
 136
 137         https://bugs.webkit.org/show_bug.cgi?id=54596
 138
 139         * Scripts/webkitpy/layout_tests/port/chromium_mac.py:
 140         * Scripts/webkitpy/layout_tests/port/chromium_mac_unittest.py:
 141         * Scripts/webkitpy/layout_tests/port/mac.py:
 142         * Scripts/webkitpy/layout_tests/port/mac_unittest.py:
 143         * Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
 144
 145 2011-02-18  Andrew Wilson  <atwilson@chromium.org>
 146
 147         Unreviewed, rolling out r79047.
 148         http://trac.webkit.org/changeset/79047
 149         https://bugs.webkit.org/show_bug.cgi?id=54596
 150
 151         Broke canary bots - please watch the canaries the next time
 152         you land this
 153
 154         * Scripts/webkitpy/layout_tests/port/chromium_mac.py:
 155         * Scripts/webkitpy/layout_tests/port/mac.py:
 156
 157 2011-02-18  Emil A Eklund  <eae@chromium.org>
 158
 159         Reviewed by Darin Adler.
 160
 161         Crash in EventHandler::sendContextMenuEventForKey
 162         https://bugs.webkit.org/show_bug.cgi?id=54495
 163
 164         Add support for the context menu key (VK_APPS) to EventSender for
 165         platforms that support the key.
 166
 167         * DumpRenderTree/chromium/EventSender.cpp:
 168         (EventSender::keyDown):
 169         * DumpRenderTree/gtk/EventSender.cpp:
 170         (keyDownCallback):
 171         * DumpRenderTree/qt/EventSenderQt.cpp:
 172         (EventSender::keyDown):
 173         * DumpRenderTree/win/EventSender.cpp:
 174         (keyDownCallback):
 175
 176 2011-02-18  Dirk Pranke  <dpranke@chromium.org>
 177
 178         Reviewed by Ojan Vafai.
 179
 180         new-run-webkit-tests: enable multiple processes by default on mac
 181         https://bugs.webkit.org/show_bug.cgi?id=54596
 182
 183         * Scripts/webkitpy/layout_tests/port/chromium_mac.py:
 184         * Scripts/webkitpy/layout_tests/port/mac.py:
 185
 186 2011-02-18  Dirk Pranke  <dpranke@chromium.org>
 187
 188         Reviewed by Adam Roben.
 189
 190         Skip pretty patch unit tests if ruby isn't installed.
 191
 192         https://bugs.webkit.org/show_bug.cgi?id=54699
 193
 194         * Scripts/webkitpy/common/prettypatch_unittest.py:
 195
 196 2011-02-18  Dirk Pranke  <dpranke@chromium.org>
 197
 198         Reviewed by Adam Roben.
 199
 200         Skip mac.test_skipped_file_paths() when running on win32; the
 201         test has hardcoded '/' directory separators and since it's a
 202         port-specific test and we have coverage on other ports it's not
 203         worth it to make the test more generic.
 204
 205         https://bugs.webkit.org/show_bug.cgi?id=54698
 206
 207         * Scripts/webkitpy/layout_tests/port/mac_unittest.py:
 208
 209 2011-02-18  Dirk Pranke  <dpranke@chromium.org>
 210
 211         Reviewed by Adam Roben.
 212
 213         Fix webkitpy.layout_tests.port.config_unittest to work on Win32.
 214
 215         https://bugs.webkit.org/show_bug.cgi?id=54703
 216
 217         * Scripts/webkitpy/layout_tests/port/config_unittest.py:
 218
 219 2011-02-18  Dirk Pranke  <dpranke@chromium.org>
 220
 221         Reviewed by Adam Roben.
 222
 223         Change the default port we're mocking out on win32 when we
 224         run mock_drt_unittest. Normally we'd default to the
 225         port-specific default, but that would be "win", which doesn't
 226         currently work.
 227
 228         https://bugs.webkit.org/show_bug.cgi?id=54708
 229
 230         * Scripts/webkitpy/layout_tests/port/mock_drt_unittest.py:
 231
 232 2011-02-18  Kenneth Russell  <kbr@google.com>
 233
 234         Reviewed by James Robinson.
 235
 236         [chromium] Clean up initialization of minimum timer interval
 237         https://bugs.webkit.org/show_bug.cgi?id=54772
 238
 239         Use new GetForegroundTabTimerInterval function in webkit_support
 240         to reset the minimum timer interval before each test.
 241
 242         * DumpRenderTree/chromium/WebViewHost.cpp:
 243         (WebViewHost::reset):
 244
 245 2011-02-18  Dirk Pranke  <dpranke@chromium.org>
 246
 247         Reviewed by Adam Roben.
 248
 249         r78494 fixed the chromium_win unittest failing under win32,
 250         but not cygwin. This fixes cygwin as well.
 251
 252         https://bugs.webkit.org/show_bug.cgi?id=54652
 253
 254         * Scripts/webkitpy/layout_tests/port/chromium_win.py:
 255
 256 2011-02-18  Dirk Pranke  <dpranke@chromium.org>
 257
 258         Reviewed by Ojan Vafai, Mihai Parparita.
 259
 260         new-run-webkit-tests: r78522 made it impossible to create a
 261         'chromium-win' or 'chromium-mac' port; they would always get the
 262         version tacked on to the end. It turns out that we actually need
 263         to be able to create these ports in order for
 264         rebaseline-chromium-webkit-tests to work correctly given the
 265         broken coverage of our layout bots. As soon as we get SL and
 266         Win 7 bots running reliably on the canaries, we can get rid of
 267         this change. Note that we only need to change the chromium_mac
 268         and win ports because linux doesn't yet use versions and that
 269         script doesn't work with non-chromium ports.
 270
 271         Also, this change fixes a bug where we were using
 272         "platform/chromium-win-win7" and
 273         "platform/chromium-mac-snowleopard" as the baseline dirs for
 274         Win 7 and SL; we should've been using the generic directories
 275         instead since they are the newest versions of the os's.
 276
 277         https://bugs.webkit.org/show_bug.cgi?id=54691
 278
 279         * Scripts/webkitpy/layout_tests/port/chromium_mac.py:
 280         * Scripts/webkitpy/layout_tests/port/chromium_mac_unittest.py:
 281         * Scripts/webkitpy/layout_tests/port/chromium_win.py:
 282         * Scripts/webkitpy/layout_tests/port/chromium_win_unittest.py:
 283         * Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
 284
 285 2011-02-16  MORITA Hajime  <morrita@google.com>
 286
 287         Reviewed by Kent Tamura.
 288
 289         [Chromium][DRT] some PostDelayedTask() accesses possibly dangling this pointer.
 290         http://webkit.org/b/53899
 291
 292         Replaced static invocation wrappers with MethodTask objects,
 293         which are removed on WebViewHost destructor.
 294
 295         * DumpRenderTree/chromium/WebViewHost.cpp:
 296         (WebViewHost::requestCheckingOfText):
 297         (WebViewHost::scheduleAnimation):
 298         (WebViewHost::closeWidgetSoon):
 299         * DumpRenderTree/chromium/WebViewHost.h:
 300         (WebViewHost::HostMethodTask::HostMethodTask):
 301         (WebViewHost::HostMethodTask::runIfValid):
 302         (WebViewHost::taskList):
 303
 304 2011-02-07  Hayato Ito  <hayato@chromium.org>
 305
 306         Reviewed by Ojan Vafai.
 307
 308         Get rid of code which writes test results from test_type's
 309         compare_output() method.
 310         Writing test results now happens in single_test_runner, using
 311         newly introduced test_result_writer module.
 312
 313         Subsequent patches will eliminate test_type/* classes.
 314
 315         https://bugs.webkit.org/show_bug.cgi?id=53908
 316
 317         * Scripts/webkitpy/layout_tests/layout_package/single_test_runner.py:
 318         * Scripts/webkitpy/layout_tests/layout_package/test_result_writer.py: Added.
 319         * Scripts/webkitpy/layout_tests/test_types/image_diff.py:
 320         * Scripts/webkitpy/layout_tests/test_types/test_type_base.py:
 321         * Scripts/webkitpy/layout_tests/test_types/text_diff.py:
 322
 323 2011-02-17  Dirk Pranke  <dpranke@chromium.org>
 324
 325         Reviewed by Adam Roben.
 326
 327         Add blacklist of remaining webkitpy/* modules that fail on
 328         Win32, so that test-webkitpy will at least run cleanly.
 329
 330         https://bugs.webkit.org/show_bug.cgi?id=54709
 331
 332         * Scripts/webkitpy/layout_tests/test/main.py:
 333
 334 2011-02-17  Simon Fraser  <simon.fraser@apple.com>
 335
 336         Reviewed by Sam Weinig.
 337
 338         Set a preference to ensure that scrollbars are in a predictable
 339         state for DRT.
 340
 341         * DumpRenderTree/mac/DumpRenderTree.mm:
 342         (resetDefaultsToConsistentValues):
 343
 344 2011-02-17  Dirk Pranke  <dpranke@chromium.org>
 345
 346         Reviewed by Adam Roben.
 347
 348         test-webkitpy doesn't work with the multiprocessing module on win32
 349         or cygwin, so we skip it for now.
 350
 351         https://bugs.webkit.org/show_bug.cgi?id=54520
 352
 353         * Scripts/webkitpy/layout_tests/layout_package/manager_worker_broker_unittest.py:
 354         * Scripts/webkitpy/layout_tests/layout_package/run_webkit_tests_unittest.py:
 355
 356 2011-02-17  Dirk Pranke  <dpranke@chromium.org>
 357
 358         Reviewed by Adam Roben.
 359
 360         Fix version, baseline_search_path for chromium-win-win7. Also
 361         fix chromium_win_unittests, and fix port_testcase.make_port()
 362         to correctly use the options passed into it and only set
 363         results_directory() if necessary.
 364
 365         https://bugs.webkit.org/show_bug.cgi?id=54652
 366
 367         * Scripts/webkitpy/layout_tests/port/chromium_win.py:
 368         * Scripts/webkitpy/layout_tests/port/chromium_win_unittest.py:
 369         * Scripts/webkitpy/layout_tests/port/port_testcase.py:
 370
 371 2011-02-17  Dirk Pranke  <dpranke@chromium.org>
 372
 373         Reviewed by Ojan Vafai.
 374
 375         Fix a hang in new-run-webkit-tests when run with
 376         --dry-run --worker-model=processes
 377
 378         https://bugs.webkit.org/show_bug.cgi?id=54595
 379
 380         * Scripts/webkitpy/layout_tests/layout_package/test_runner2.py:
 381         * Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
 382
 383 2011-02-17  Kevin Ollivier  <kevino@theolliviers.com>
 384
 385         [wx] Build fixes after recent changes.
 386
 387         * DumpRenderTree/wx/LayoutTestControllerWx.cpp:
 388         (LayoutTestController::setMinimumTimerInterval):
 389         * wx/build/settings.py:
 390
 391 2011-02-17  Csaba Osztrogonác  <ossy@webkit.org>
 392
 393         Reviewed by Andreas Kling.
 394
 395         [Qt] ImageDiff sometimes hangs
 396         https://bugs.webkit.org/show_bug.cgi?id=54641
 397
 398         * DumpRenderTree/qt/ImageDiff.cpp: Clear actualImage and baselineImage after all comparison.
 399         (main):
 400
 401 2011-02-17  Peter Varga  <pvarga@webkit.org>
 402
 403         Reviewed by Csaba Osztrogonác.
 404
 405         The run-sunspider --v8-suite doesn't work
 406         https://bugs.webkit.org/show_bug.cgi?id=54660
 407
 408         * Scripts/run-sunspider:
 409
 410 2011-02-17  Gabor Rapcsanyi  <rgabor@webkit.org>
 411
 412         Reviewed by Csaba Osztrogonác.
 413
 414         [NRWT] Fix environment setup on Qt port.
 415         https://bugs.webkit.org/show_bug.cgi?id=54549
 416
 417         * Scripts/webkitpy/layout_tests/port/qt.py:
 418
 419 2011-02-16  Yael Aharon  <yael.aharon@nokia.com>
 420
 421         Reviewed by Andreas Kling.
 422
 423         [Qt] Enable tiled backing store by default in MiniBrowser.
 424         https://bugs.webkit.org/show_bug.cgi?id=54581
 425
 426         Instead of a flag to turn it on, a new flag is defined to turn
 427         tiled backing store off.
 428
 429         * MiniBrowser/qt/MiniBrowserApplication.cpp:
 430         (MiniBrowserApplication::handleUserOptions):
 431         * MiniBrowser/qt/MiniBrowserApplication.h:
 432         (WindowOptions::WindowOptions):
 433
 434 2011-02-16  Eric Seidel  <eric@webkit.org>
 435
 436         Reviewed by Mihai Parparita.
 437
 438         flaky test archive uploads are always 22 bytes long
 439         https://bugs.webkit.org/show_bug.cgi?id=54593
 440
 441         Speculative fix for the 22-bytes bug.
 442
 443         * Scripts/webkitpy/tool/bot/flakytestreporter.py:
 444
 445 2011-02-16  Anna Cavender  <annacc@chromium.org>
 446
 447         Reviewed by Mihai Parparita.
 448
 449         http/tests/media should be served over HTTP (not from local file)
 450         https://bugs.webkit.org/show_bug.cgi?id=54028
 451
 452         Removed special file:// treatment of http/tests/media
 453
 454         * Scripts/old-run-webkit-tests:
 455         * Scripts/webkitpy/layout_tests/port/base.py:
 456
 457 2011-02-16  Mihai Parparita  <mihaip@chromium.org>
 458
 459         Reviewed by Ojan Vafai.
 460
 461         ChangeLog.parse_latest_entry_from_file does not handle rolled over ChangeLogs
 462         https://bugs.webkit.org/show_bug.cgi?id=54609
 463
 464         r78737 rolled over ChangeLogs, and webkit-patch complains when landing
 465         patches such as this one that are the first to add an entry to the
 466         ChangeLog (since they don't see a second date line to know where the
 467         entry ends).
 468
 469         * Scripts/webkitpy/common/checkout/changelog.py:
 470         * Scripts/webkitpy/common/checkout/changelog_unittest.py:
 471
 472 == Rolled over to ChangeLog-2011-02-16 ==