2011-02-20 Dirk Pranke <dpranke@chromium.org>
[WebKit.git] / 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 ==