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