1 2011-01-28 Pratik Solanki <psolanki@apple.com>
3 Reviewed by David Kilzer.
5 Space not necessary for undefined feature
6 https://bugs.webkit.org/show_bug.cgi?id=53317
8 * Scripts/build-webkit:
10 2011-01-28 Chang Shu <chang.shu@nokia.com>
12 Reviewed by Andreas Kling.
14 Reset NetworkAccessManager to clean up credentials from previous tests.
15 This change causes no performance overhead either.
16 https://bugs.webkit.org/show_bug.cgi?id=36688
18 * DumpRenderTree/qt/DumpRenderTreeQt.cpp:
19 (WebCore::DumpRenderTree::resetToConsistentStateBeforeTesting):
20 (WebCore::DumpRenderTree::open):
21 * DumpRenderTree/qt/DumpRenderTreeQt.h:
23 2011-01-28 Zoltan Horvath <zoltan@webkit.org>
25 Reviewed by Andreas Kling.
27 [Qt] Add Zoom Text Only action to MiniBrowser
28 https://bugs.webkit.org/show_bug.cgi?id=53297
30 Add Zoom Text Only action to MiniBrowser's view menu, change zoom levels to avoid unnecessary divides.
32 * MiniBrowser/qt/BrowserWindow.cpp:
33 (BrowserWindow::BrowserWindow):
34 (BrowserWindow::zoomIn):
35 (BrowserWindow::zoomOut):
36 (BrowserWindow::resetZoom):
37 (BrowserWindow::toggleZoomTextOnly):
38 (BrowserWindow::applyZoom):
39 * MiniBrowser/qt/BrowserWindow.h:
41 2011-01-28 Adam Barth <abarth@webkit.org>
43 Reviewed by Eric Seidel.
45 Add webkit-patch roll-chromium-deps
46 https://bugs.webkit.org/show_bug.cgi?id=53288
48 This command updates the Source/WebKit/chromium/DEPS file with the
49 last-known good revision of Chromium (or a revision specified on the
50 command line). I'd eventually like to turn this into a SheriffBot
51 command, but this is the first step.
53 This patch somewhat sprawled because I needed to move a bunch of code
54 out of ChangeLog that should never have been there in the first place.
55 Also, I had to fix a bug in MockUser in order to test the new command.
57 * Scripts/webkitpy/common/checkout/api.py:
58 * Scripts/webkitpy/common/checkout/changelog.py:
59 * Scripts/webkitpy/common/checkout/changelog_unittest.py:
60 * Scripts/webkitpy/common/checkout/deps.py: Added.
61 * Scripts/webkitpy/common/config/urls.py:
62 * Scripts/webkitpy/tool/commands/__init__.py:
63 * Scripts/webkitpy/tool/commands/download_unittest.py:
64 * Scripts/webkitpy/tool/commands/roll.py: Added.
65 * Scripts/webkitpy/tool/commands/roll_unittest.py: Added.
66 * Scripts/webkitpy/tool/commands/upload_unittest.py:
67 * Scripts/webkitpy/tool/mocktool.py:
68 * Scripts/webkitpy/tool/steps/__init__.py:
69 * Scripts/webkitpy/tool/steps/preparechangelogfordepsroll.py: Added.
70 * Scripts/webkitpy/tool/steps/preparechangelogforrevert.py:
71 * Scripts/webkitpy/tool/steps/preparechangelogforrevert_unittest.py: Added.
72 * Scripts/webkitpy/tool/steps/suggestreviewers_unittest.py:
73 * Scripts/webkitpy/tool/steps/updatechromiumdeps.py: Added.
74 * Scripts/webkitpy/tool/steps/validatechangelogs_unittest.py:
76 2011-01-27 Greg Coletta <greg.coletta@nokia.com>
78 Reviewed by Laszlo Gombos.
80 Get rid of prefix header dependency for WebKit2 build system
81 https://bugs.webkit.org/show_bug.cgi?id=50174
83 Change the style checker so that it enforces config.h include for
86 * Scripts/webkitpy/style/checker.py:
88 2011-01-27 Maciej Stachowiak <mjs@apple.com>
90 Reviewed by Darin Adler.
92 execCommand("Paste") doesn't work in WebKitTestRunner
93 https://bugs.webkit.org/show_bug.cgi?id=52785
95 Enable both of the settings needed to allow paste.
97 * WebKitTestRunner/TestController.cpp:
98 (WTR::TestController::resetStateToConsistentValues):
100 2011-01-27 Jacob Dinu <dinu.jacob@nokia.com>
102 Reviewed by Andreas Kling.
104 [Qt] QtTestBrowser: User input lost when toggling use of QGraphicsView
105 https://bugs.webkit.org/show_bug.cgi?id=48440
107 Preserve user input when toggling use of QGraphicsView
109 * QtTestBrowser/launcherwindow.cpp:
110 (LauncherWindow::initializeView):
111 (LauncherWindow::loadFinished):
112 * QtTestBrowser/launcherwindow.h:
113 * QtTestBrowser/mainwindow.cpp:
114 (MainWindow::addressUrl):
115 * QtTestBrowser/mainwindow.h:
117 2011-01-27 Dirk Pranke <dpranke@chromium.org>
119 Reviewed by Mihai Parparita.
121 new-run-webkit-tests: turn off pixel tests correctly by default
122 for webkit-based ports. r70013 (bug 47510) used
123 port.set_option_default() to attempt to set default values, but
124 that didn't work correctly. I have removed set_option_default
125 for now since it was only being used in two places and in three
126 useless unit tests. There is a separate bug open to fix the
127 option parsing (48095), so this workaround is fine for now.
129 https://bugs.webkit.org/show_bug.cgi?id=53217
131 * Scripts/webkitpy/layout_tests/port/webkit.py:
132 * Scripts/webkitpy/layout_tests/port/base.py:
133 * Scripts/webkitpy/layout_tests/port/base_unittest.py:
135 2011-01-27 Balazs Kelemen <kbalazs@webkit.org>
137 Rubber-stamped by Ariya Hidayat.
139 Trivial crash fix in WTR.
141 * WebKitTestRunner/TestController.cpp:
142 (WTR::TestController::initialize): Increase the size
143 of the vector by one to be able to store the null character.
145 2011-01-27 Balazs Kelemen <kbalazs@webkit.org>
147 Reviewed by Csaba Osztrogonác.
149 [Qt][WK2] WTR should be initialized in the same way as DRT
150 https://bugs.webkit.org/show_bug.cgi?id=53240
152 * WebKitTestRunner/InjectedBundle/qt/ActivateFontsQt.cpp:
153 (WTR::activateFonts): Added missing initialization steps.
155 2011-01-27 Joone Hur <joone.hur@collabora.co.uk>
159 Adding myself to committers.py.
161 * Scripts/webkitpy/common/config/committers.py:
163 2011-01-27 Laszlo Gombos <laszlo.1.gombos@nokia.com>
165 Reviewed by Csaba Osztrogonác.
167 [Qt] [Symbian] Move project files into Source
168 https://bugs.webkit.org/show_bug.cgi?id=52891
170 Fix the Symbian build after the project files
171 are moved to Source directory. On Symbian qmake
172 needs to run in the same directory where the main
173 pro files (WebKit.pro, Tools.pro) are located.
175 * Scripts/webkitdirs.pm: Change to the directory where the pro file
176 is located before running qmake for Symbian. Qmake on Symbian
177 does not properly honor the "-o" option, work it around by setting
178 the name of the Makefile to bld.inf.
180 2011-01-27 Zoltan Horvath <zoltan@webkit.org>
182 Reviewed by Andreas Kling.
184 [Qt] Add simple zooming features to MiniBrowser
185 https://bugs.webkit.org/show_bug.cgi?id=53231
187 Add View menu and Zoom In, Zoom Out, Zoom Reset actions to MiniBrowser.
189 * MiniBrowser/qt/BrowserWindow.cpp:
190 (BrowserWindow::BrowserWindow):
191 (BrowserWindow::zoomIn):
192 (BrowserWindow::zoomOut):
193 (BrowserWindow::resetZoom):
194 (BrowserWindow::updateUserAgentList):
195 (BrowserWindow::applyZoom):
196 * MiniBrowser/qt/BrowserWindow.h:
198 2011-01-26 Maciej Stachowiak <mjs@apple.com>
200 Reviewed by Dan Bernstein.
202 WebKitTestRunner needs to support layoutTestController.evaluateInWebInspector
203 https://bugs.webkit.org/show_bug.cgi?id=42319
205 Add evaluateInWebInspector and other APIs needed for inspector tests to run.
207 * WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl:
208 * WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:
209 (WTR::LayoutTestController::showWebInspector):
210 (WTR::LayoutTestController::closeWebInspector):
211 (WTR::LayoutTestController::evaluateInWebInspector):
212 (WTR::LayoutTestController::setTimelineProfilingEnabled):
213 * WebKitTestRunner/InjectedBundle/LayoutTestController.h:
214 * WebKitTestRunner/TestInvocation.cpp:
215 (WTR::shouldOpenWebInspector):
216 (WTR::TestInvocation::invoke):
217 * WebKitTestRunner/WebKitTestRunnerPrefix.h:
219 2011-01-26 Martin Robinson <mrobinson@igalia.com>
221 Reviewed by Xan Lopez.
223 [GTK] Pixel dumps do not include scrollbars in output images
224 https://bugs.webkit.org/show_bug.cgi?id=53216
226 Show scrollbars in pixel dumps. This will fix pixel dumps for tests
227 that have scrollbars.
229 * DumpRenderTree/gtk/PixelDumpSupportGtk.cpp:
230 (createBitmapContextFromWebView): Take the snapshot of the containing GtkScrolledWindow
231 instead of the web view itself.
233 2011-01-26 Dirk Pranke <dpranke@chromium.org>
235 Reviewed by Mihai Parparita.
237 Fix regression introduced in r76322 ... new-run-webkit-tests
238 was attempting to save the image diff output as a UTF-8-encoded
241 https://bugs.webkit.org/show_bug.cgi?id=53210
243 * Scripts/webkitpy/layout_tests/port/webkit.py:
245 2011-01-26 Sheriff Bot <webkit.review.bot@gmail.com>
247 Unreviewed, rolling out r76709.
248 http://trac.webkit.org/changeset/76709
249 https://bugs.webkit.org/show_bug.cgi?id=53194
251 "broke python tests on non-snowleopard platforms" (Requested
252 by dpranke on #webkit).
254 * Scripts/webkitpy/layout_tests/port/factory.py:
255 * Scripts/webkitpy/layout_tests/port/mock_drt.py: Removed.
256 * Scripts/webkitpy/layout_tests/port/mock_drt_unittest.py: Removed.
258 2011-01-26 Dirk Pranke <dpranke@chromium.org>
260 Reviewed by Tony Chang.
262 new-run-webkit-tests: add a "mock DRT" port implementation
263 and a separate class that emulates what we expect the
264 DumpRenderTree behavior to be.
266 This will eventually replace port/dryrun.py and allow us to get
267 better test coverage of the new-run-webkit-tests code as well as
268 a reference for what new-run-webkit-tests expects from DRT.
270 This is the first attempt at this, and it is pretty bare-boned. It
271 really only has been tested on the 'mac' port (and a little on
272 the 'chromium-mac' port.
274 https://bugs.webkit.org/show_bug.cgi?id=53126
276 * Scripts/webkitpy/layout_tests/port/mock_drt.py: Added.
277 * Scripts/webkitpy/layout_tests/port/mock_drt_unittest.py: Added.
278 * Scripts/webkitpy/layout_tests/port/factory.py:
280 2011-01-26 Xianzhu Wang <wangxianzhu@google.com>
282 Reviewed by Tony Chang.
284 Add '--no-timeout' option to Chromium DRT to ease debugging.
285 https://bugs.webkit.org/show_bug.cgi?id=52873
287 * DumpRenderTree/chromium/DumpRenderTree.cpp:
290 2011-01-26 Csaba Osztrogonác <ossy@webkit.org>
292 Reviewed by Andreas Kling.
294 [Qt] Fix qt_minimal build
295 https://bugs.webkit.org/show_bug.cgi?id=53172
297 * QtTestBrowser/locationedit.h:
299 2011-01-26 Mansi Mithal <mansi.mithal@nokia.com>
301 Reviewed by Antonio Gomes.
303 QtTestBrowser should have a UI Setting to disable plugins
304 https://bugs.webkit.org/show_bug.cgi?id=52408
306 Added a new action item named "Disable Plugins"
307 under the "Settings" menu.
309 * QtTestBrowser/launcherwindow.cpp:
310 (LauncherWindow::createChrome):
311 (LauncherWindow::togglePlugins):
312 * QtTestBrowser/launcherwindow.h:
314 2011-01-25 Patrick Gansterer <paroga@webkit.org>
316 Reviewed by Adam Barth.
318 Move main CMakeLists.txt into Source directory
319 https://bugs.webkit.org/show_bug.cgi?id=52888
321 * Scripts/webkitdirs.pm:
323 2011-01-25 Patrick Gansterer <paroga@webkit.org>
325 Reviewed by Eric Seidel.
327 Improve label text in submit-to-ews
328 https://bugs.webkit.org/show_bug.cgi?id=53130
330 * QueueStatusServer/templates/submittoews.html:
332 2011-01-25 Mansi Mithal <mansi.mithal@nokia.com>
334 Reviewed by Antonio Gomes.
336 QtTestBrowser should have a UI Settings to prevent loading images
337 https://bugs.webkit.org/show_bug.cgi?id=52409
339 Added a new action item named "DisableAutoLoadImages"
340 under the "Settings" menu
342 * QtTestBrowser/launcherwindow.cpp:
343 (LauncherWindow::createChrome):
344 (LauncherWindow::toggleAutoLoadImages):
345 * QtTestBrowser/launcherwindow.h:
347 2011-01-25 Dirk Pranke <dpranke@chromium.org>
349 Reviewed by Tony Chang.
351 Minor bug fixes and cleanup for filesystem wrappers, port/* test
352 classes, test_expectations.py. This change adds "test-win" and
353 "test-mac" variants to the test port so that we can better test
354 rebaselining, and adds a MockUser() object for reuse in testing.
356 https://bugs.webkit.org/show_bug.cgi?id=53036
358 * Scripts/webkitpy/common/system/filesystem.py:
359 * Scripts/webkitpy/common/system/filesystem_mock.py:
360 * Scripts/webkitpy/layout_tests/layout_package/test_expectations.py:
361 * Scripts/webkitpy/layout_tests/port/factory.py:
362 * Scripts/webkitpy/layout_tests/port/base.py:
363 * Scripts/webkitpy/layout_tests/port/test.py:
364 * Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
365 * Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests_unittest.py:
366 * Scripts/webkitpy/tool/mocktool.py:
368 2011-01-25 Tony Chang <tony@chromium.org>
370 Reviewed by Dimitri Glazkov.
372 [chromium] remove unused layoutTestController callback
373 https://bugs.webkit.org/show_bug.cgi?id=53103
375 This method is used by a test_shell_test, but not needed by DRT.
377 * DumpRenderTree/chromium/LayoutTestController.cpp:
378 (LayoutTestController::LayoutTestController):
379 * DumpRenderTree/chromium/LayoutTestController.h:
381 2011-01-25 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
383 Reviewed by Andreas Kling.
385 [GTK] Remove 64 bits release bot from the buildbot master
386 https://bugs.webkit.org/show_bug.cgi?id=52899
388 Removes the 64 bits release slave from the buildbot configuration,
389 so that we can use it as an EWS.
391 * BuildSlaveSupport/build.webkit.org-config/config.json:
393 2011-01-24 Maciej Stachowiak <mjs@apple.com>
395 Reviewed by Dan Bernstein.
397 Implement database quota callback to fix storage tests on WebKit2 bot
398 https://bugs.webkit.org/show_bug.cgi?id=53064
400 * WebKitTestRunner/TestController.cpp:
401 (WTR::exceededDatabaseQuota):
402 (WTR::TestController::createOtherPage):
403 (WTR::TestController::initialize):
405 2011-01-24 Maciej Stachowiak <mjs@apple.com>
407 Reviewed by Anders Carlsson.
409 Use designated temp directory for the database for WebKit2
410 https://bugs.webkit.org/show_bug.cgi?id=53052
412 Adopt the new WK2 API for this.
414 * WebKitTestRunner/TestController.cpp:
415 (WTR::TestController::libraryPathForTesting):
416 (WTR::TestController::initialize):
417 * WebKitTestRunner/TestController.h:
418 * WebKitTestRunner/mac/TestControllerMac.mm:
419 (WTR::TestController::platformLibraryPathForTesting):
420 * WebKitTestRunner/qt/TestControllerQt.cpp:
421 (WTR::TestController::platformLibraryPathForTesting):
422 * WebKitTestRunner/win/TestControllerWin.cpp:
423 (WTR::TestController::platformLibraryPathForTesting):
425 2011-01-24 Lucas Forschler <lforschler@apple.com>
427 Reviewed by Stephanie Lewis.
429 Add a new Leopard test bot.
431 * BuildSlaveSupport/build.webkit.org-config/config.json:
433 2011-01-24 Gyuyoung Kim <gyuyoung.kim@samsung.com>
435 Reviewed by Adam Barth.
437 Add EFL Builder to core waterfall
438 https://bugs.webkit.org/show_bug.cgi?id=52704
440 Add EFL Builder to the core waterfall again.
442 * Scripts/webkitpy/common/net/buildbot/buildbot.py:
443 * Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py:
445 2011-01-24 Xianzhu Wang <wangxianzhu@google.com>
447 Reviewed by Tony Chang.
449 new-run-webkit-tests fails to start HTTP server if there are proxy settings
450 https://bugs.webkit.org/show_bug.cgi?id=52872
452 Force disabling proxy to resolve the problem.
454 * Scripts/webkitpy/layout_tests/port/http_server_base.py:
455 * Scripts/webkitpy/layout_tests/port/websocket_server.py:
457 2011-01-24 Andreas Kling <kling@webkit.org>
459 Reviewed by Darin Adler.
461 check-webkit-style: Update exemptions following Source/ move.
462 https://bugs.webkit.org/show_bug.cgi?id=53017
464 * Scripts/webkitpy/style/checker.py:
465 * Scripts/webkitpy/style/checker_unittest.py:
467 2011-01-24 Andras Becsi <abecsi@webkit.org>
469 Rubber-stamped by Csaba Osztrogonác.
471 [Qt] Move project files into Source
472 https://bugs.webkit.org/show_bug.cgi?id=52891
474 * DumpRenderTree/qt/ImageDiff.pro: Add missing "Source" to WebKit.pri path
476 2011-01-24 Mikhail Naganov <mnaganov@chromium.org>
478 Reviewed by Yury Semikhatsky.
480 Web Inspector: Find duplicate strings in localizedStrings.js
482 https://bugs.webkit.org/show_bug.cgi?id=53005
484 * Scripts/check-inspector-strings:
486 2011-01-24 Zoltan Horvath <zoltan@webkit.org>
488 Reviewed by Andreas Kling.
490 [Qt] Regroup MiniBrowser's menubar and add open file action
491 https://bugs.webkit.org/show_bug.cgi?id=53000
493 Add File and Develop menu to MiniBrowser's menubar and add open file action to File menu.
495 * MiniBrowser/qt/BrowserWindow.cpp:
496 (BrowserWindow::BrowserWindow):
497 (BrowserWindow::openFile):
498 (BrowserWindow::~BrowserWindow):
499 * MiniBrowser/qt/BrowserWindow.h:
501 2011-01-24 Csaba Osztrogonác <ossy@webkit.org>
503 Unreviewed fix after r76496
505 [Qt] Move project files into Source
506 https://bugs.webkit.org/show_bug.cgi?id=52891
508 * BuildSlaveSupport/build.webkit.org-config/master.cfg: Remove unnecessary "Source" from the path.
509 * BuildSlaveSupport/built-product-archive: Remove unnecessary "Source" from the path.
510 * Scripts/webkitdirs.pm: Add the accidentally removed slash to the path.
512 2011-01-24 Andras Becsi <abecsi@webkit.org>
514 Reviewed by Csaba Osztrogonác.
516 [Qt] Move project files into Source
517 https://bugs.webkit.org/show_bug.cgi?id=52891
519 * DerivedSources.pro: Renamed from DerivedSources.pro.
520 * DumpRenderTree/qt/DumpRenderTree.pro:
521 * DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro:
522 * MiniBrowser/DerivedSources.pro:
523 * MiniBrowser/qt/MiniBrowser.pro:
524 * QtTestBrowser/QtTestBrowser.pro:
525 * Scripts/webkitdirs.pm:
527 * WebKitTestRunner/DerivedSources.pro:
528 * WebKitTestRunner/InjectedBundle/qt/InjectedBundle.pro:
529 * WebKitTestRunner/qt/WebKitTestRunner.pro:
531 2011-01-22 Geoffrey Garen <ggaren@apple.com>
533 Reviewed by Dan Bernstein.
535 Beefed up --threaded mode to catch even more kinds of errors.
536 https://bugs.webkit.org/show_bug.cgi?id=52971
538 * DumpRenderTree/pthreads/JavaScriptThreadingPthreads.cpp: Use a shared
539 context group to force JSC to mark multiple threads. (This used to be
540 the default, but it changed in SnowLeopard.)
541 (runJavaScriptThread): Do more locking and unlocking, and more allocation,
542 to give threading mistakes more chances to show themselves.
543 (startJavaScriptThreads):
544 (stopJavaScriptThreads):
546 2011-01-22 Robert Hogan <robert@webkit.org>
548 Reviewed by Andreas Kling.
550 [Qt] Support layoutTestController.addURLToRedirect()
551 https://bugs.webkit.org/show_bug.cgi?id=52956
553 * DumpRenderTree/qt/LayoutTestControllerQt.cpp:
554 (LayoutTestController::addURLToRedirect):
555 * DumpRenderTree/qt/LayoutTestControllerQt.h:
557 2011-01-22 Robert Hogan <robert@webkit.org>
559 Reviewed by Kenneth Rohde Christiansen.
561 [Qt] REGRESSION (r72360): http/tests/security/aboutBlank/xss-DENIED-* tests fail
562 https://bugs.webkit.org/show_bug.cgi?id=49802
564 * DumpRenderTree/qt/DumpRenderTreeQt.cpp:
565 (WebCore::DumpRenderTree::resetToConsistentStateBeforeTesting):
566 (WebCore::DumpRenderTree::open):
568 2011-01-21 Adam Barth <abarth@webkit.org>
570 Reviewed by Maciej Stachowiak.
572 Disable mac_unittest.py that fails on some of the Mac bots
573 https://bugs.webkit.org/show_bug.cgi?id=52947
575 As noted in the comment, this test does not appear to be correct
576 because the function this test is testing returns different results
577 depending on which flavor of Mac the test is run on. This patch
578 disable the test. Hopefully we can re-enable the test once we've
579 probably insulated the test from its environment.
581 * Scripts/webkitpy/layout_tests/port/mac_unittest.py:
583 2011-01-21 Dirk Pranke <dpranke@chromium.org>
585 Reviewed by Mihai Parparita.
587 Fix bug introduced in r76322 that caused NRWT to not actually
588 read the Skipped files properly.
590 https://bugs.webkit.org/show_bug.cgi?id=52771
592 * Scripts/webkitpy/layout_tests/port/mac_unittest.py:
593 * Scripts/webkitpy/layout_tests/port/webkit.py:
594 * Scripts/webkitpy/layout_tests/port/webkit_unittest.py:
596 2011-01-21 Sam Weinig <sam@webkit.org>
598 Reviewed by Geoffrey Sean Garen and Mark Rowe.
600 WebKitTestRunner needs the HOME environment variable to be set.
601 <rdar://problem/8896573>
603 * Scripts/old-run-webkit-tests: Set the HOME environment variable if
606 2011-01-21 Dirk Pranke <dpranke@chromium.org>
608 Reviewed by Eric Seidel.
610 Suppress a few remaining integration tests so that none of the
611 layout_test unit tests ever read from the filesystem or launch
612 subprocesses that aren't part of the unit tests.
614 Also fix a minor bug in the printing unit tests that was
615 incorrectly relying on sys.argv.
617 https://bugs.webkit.org/show_bug.cgi?id=52863
619 * Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
620 * Scripts/webkitpy/layout_tests/port/port_testcase.py:
621 * Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
623 2011-01-21 Darin Adler <darin@apple.com>
627 * MiniBrowser/mac/BrowserWindowController.m:
628 (-[BrowserWindowController awakeFromNib]): Pass 0 for runModal function.
630 2011-01-20 Darin Adler <darin@apple.com>
632 Reviewed by Dan Bernstein.
634 WebKit2: Implement showModalDialog
635 https://bugs.webkit.org/show_bug.cgi?id=52855
637 This fixes WebKitTestRunner to compile, but more work is probably
638 needed to get it to pass the tests.
640 * WebKitTestRunner/TestController.cpp:
641 (WTR::TestController::runModal): Added. Calls through to the
642 platform-specific version of runModal.
643 (WTR::TestController::createOtherPage): Changed to be a private
644 static member function so it can refer to runModal, which is
645 a private static member function.
646 (WTR::TestController::initialize): Pass 0 for the runModal
647 function since we don't need to run the main window modal.
648 I suspect this is wrong and will need to change.
649 * WebKitTestRunner/TestController.h: Added declarations for
650 the functions added above.
651 * WebKitTestRunner/mac/TestControllerMac.mm:
652 (WTR::TestController::runModal): Added. Untested implementation.
653 * WebKitTestRunner/qt/TestControllerQt.cpp:
654 (WTR::TestController::runModal): Added.
655 * WebKitTestRunner/win/TestControllerWin.cpp:
656 (WTR::TestController::runModal): Added.
658 2011-01-21 Eric Seidel <eric@webkit.org>
660 Reviewed by Adam Barth.
662 Split Host object off from WebKitPatch for easier re-use
663 https://bugs.webkit.org/show_bug.cgi?id=52284
665 new-run-webkit-tests and other tools are eventually going to want one of these.
666 Basically you need to inherit from this class if you want to pretend to be a
667 self.tool object for Steps/Commands.
669 * Scripts/webkitpy/common/host.py: Added.
670 * Scripts/webkitpy/tool/main.py:
672 2011-01-20 Dirk Pranke <dpranke@chromium.org>
674 Reviewed by Adam Barth.
676 Disable the tests that start and stop the http server and web
677 server, so that we don't have to worry about port collisions on
680 https://bugs.webkit.org/show_bug.cgi?id=52861
682 * Scripts/webkitpy/layout_tests/port/port_testcase.py:
684 2011-01-20 Dirk Pranke <dpranke@chromium.org>
686 Reviewed by Eric Seidel.
688 nrwt: rewrite google_chrome_unittest to use mock filesystem.
690 https://bugs.webkit.org/show_bug.cgi?id=52772
692 * Scripts/webkitpy/layout_tests/port/google_chrome_unittest.py:
694 2011-01-20 Dirk Pranke <dpranke@chromium.org>
696 Reviewed by Eric Siedel.
698 nrwt: clean up almost all remaining port references, remove
699 unnecessary import clauses. The only remaining references are
700 in places where a mock filesystem makes no sense or can't be
701 used, and in one routine in port/google_chrome_unittest that
702 I'll rewrite in a separate patch.
704 https://bugs.webkit.org/show_bug.cgi?id=52771
706 * Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
707 * Scripts/webkitpy/layout_tests/port/chromium_gpu.py:
708 * Scripts/webkitpy/layout_tests/port/chromium_gpu_unittest.py:
709 * Scripts/webkitpy/layout_tests/port/chromium_linux.py:
710 * Scripts/webkitpy/layout_tests/port/chromium_mac.py:
711 * Scripts/webkitpy/layout_tests/port/chromium_unittest.py:
712 * Scripts/webkitpy/layout_tests/port/chromium_win.py:
713 * Scripts/webkitpy/layout_tests/port/chromium_win_unittest.py:
714 * Scripts/webkitpy/layout_tests/port/google_chrome.py:
715 * Scripts/webkitpy/layout_tests/port/google_chrome_unittest.py:
716 * Scripts/webkitpy/layout_tests/port/gtk.py:
717 * Scripts/webkitpy/layout_tests/port/mac.py:
718 * Scripts/webkitpy/layout_tests/port/port_testcase.py:
719 * Scripts/webkitpy/layout_tests/port/qt.py:
720 * Scripts/webkitpy/layout_tests/port/webkit.py:
721 * Scripts/webkitpy/layout_tests/port/win.py:
722 * Scripts/webkitpy/layout_tests/test_types/image_diff.py:
724 2011-01-20 Dirk Pranke <dpranke@chromium.org>
726 Reviewed by Mihai Parparita.
728 remove fs refs from run_webkit_tests, rebaseline_chromium_webkit_tests
730 https://bugs.webkit.org/show_bug.cgi?id=52762
732 * Scripts/webkitpy/layout_tests/run_webkit_tests.py:
733 * Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
735 2011-01-20 Tony Chang <tony@chromium.org>
737 Reviewed by Ojan Vafai.
739 [chromium] fix the paths used for the rebaseline tool
740 https://bugs.webkit.org/show_bug.cgi?id=52857
742 Some of the bots were renamed so the path to grab the results zip
745 * Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
747 2011-01-20 Dirk Pranke <dpranke@chromium.org>
749 Reviewed by Tony Chang.
751 Fix bug introduced in r72688 / bug 52768 that broke the
752 generation of the output filename links in the results.html
753 output. Apparently none of the unit tests actually checked to
754 make sure the generated links were correct :(
756 https://bugs.webkit.org/show_bug.cgi?id=52854
758 * Scripts/webkitpy/layout_tests/layout_package/test_failures.py:
759 * Scripts/webkitpy/layout_tests/layout_package/test_failures_unittest.py:
761 2011-01-20 Alexey Proskuryakov <ap@apple.com>
763 Reviewed by Darin Adler.
765 https://bugs.webkit.org/show_bug.cgi?id=52849
766 Make window.print work with WebKit2
768 * MiniBrowser/mac/BrowserWindowController.m:
769 (-[BrowserWindowController awakeFromNib]):
770 * WebKitTestRunner/TestController.cpp:
771 (WTR::createOtherPage):
772 (WTR::TestController::initialize):
773 Added zeroes to WKPageUIClient structures to avoid build failures.
775 2011-01-20 Dirk Pranke <dpranke@chromium.org>
777 Reviewed by Eric Seidel.
779 nrwt: remove fs refs from printing, test_failures, test_expectations, text_diff
781 https://bugs.webkit.org/show_bug.cgi?id=52756
783 * Scripts/webkitpy/layout_tests/layout_package/printing.py:
784 * Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
785 * Scripts/webkitpy/layout_tests/layout_package/test_expectations.py:
786 * Scripts/webkitpy/layout_tests/layout_package/test_expectations_unittest.py:
787 * Scripts/webkitpy/layout_tests/layout_package/test_failures.py:
788 * Scripts/webkitpy/layout_tests/test_types/text_diff.py:
790 2011-01-20 Mansi Mithal <mansi.mithal@nokia.com>
792 Reviewed by Antonio Gomes.
794 QtTestBrowser should have a seperate menu for Settings
795 https://bugs.webkit.org/show_bug.cgi?id=52817
797 Added a new menu item named "Settings"
798 and moved the "Enable interrupting js scripts" and "Enable js pop up windows" under the "Settings" menu
800 * QtTestBrowser/launcherwindow.cpp:
801 (LauncherWindow::createChrome):
803 2011-01-20 Sam Weinig <sam@webkit.org>
805 Reviewed by John Sullivan.
807 Update update-webkit-localizable-strings with new path to WebKit.
809 * Scripts/update-webkit-localizable-strings:
811 2011-01-20 James Robinson <jamesr@chromium.org>
813 Reviewed by Darin Fisher.
815 Implement mozilla's requestAnimationFrame API
816 https://bugs.webkit.org/show_bug.cgi?id=51218
818 Chromium DumpRenderTree support for window.webkitRequestAnimationFrame.
820 * DumpRenderTree/chromium/WebViewHost.cpp:
821 (invokeScheduleComposite):
822 (WebViewHost::scheduleAnimation):
823 (WebViewHost::paintInvalidatedRegion):
824 * DumpRenderTree/chromium/WebViewHost.h:
826 2011-01-20 Maciej Stachowiak <mjs@apple.com>
828 Reviewed by Adam Roben.
830 WebKitTestRunner needs to support layoutTestController.evaluateScriptInIsolatedWorld
831 https://bugs.webkit.org/show_bug.cgi?id=42327
833 * WebKitTestRunner/InjectedBundle/Bindings/CodeGeneratorTestRunner.pm: Add support
834 for methods that take their normal arguments but also a JSContextRef.
835 * WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl: IDL definition
836 for evaluateScriptInIsolatedWorld.
837 * WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:
838 (WTR::InjectedBundlePage::didClearWindowForFrame): Set a magic variable only if
839 this call is for an isolated world.
840 * WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:
841 (WTR::worldMap): Helper to create a world map.
842 (WTR::LayoutTestController::worldIDForWorld): Map from an ID to a world.
843 (WTR::LayoutTestController::evaluateScriptInIsolatedWorld): The newly
844 added LayoutTestController API.
845 * WebKitTestRunner/InjectedBundle/LayoutTestController.h:
847 2011-01-19 Adam Roben <aroben@apple.com>
849 Convert paths in environment variables back to msys-style inside commit-log-editor
851 When this script gets run from inside git commit, msys-style paths in the environment will
852 have been turned into Windows-style paths with forward slashes. This screws up functions
853 like File::Spec->rel2abs, which seem to rely on $PWD having an msys-style path. We convert
854 the paths back to msys-style before doing anything else.
856 Fixes <http://webkit.org/b/48527> commit-log-editor uses full paths for section headers when
857 using msysgit's Perl and multiple ChangeLogs have been edited
859 Reviewed by David Kilzer.
861 * Scripts/commit-log-editor: Call fixEnvironment before doing anything else.
862 (fixEnvironment): Added. When run in msys in conjunction with git (i.e., when invoked from
863 inside git commit), convert Windows-style paths in the environment back to msys-style paths.
865 2011-01-20 Zoltan Horvath <zoltan@webkit.org>
867 [Win] Unreviewed build fix after r76248.
869 * DumpRenderTree/ForwardingHeaders/wtf/FastAllocBase.h: Added.
871 2011-01-20 Zoltan Horvath <zoltan@webkit.org>
873 Reviewed by Csaba Osztrogonác.
875 Refactoring of the custom allocation framework
876 https://bugs.webkit.org/show_bug.cgi?id=49897
878 Inheriting from FastAllocBase can result in objects getting larger (bug #33896, #46589).
879 The modification replaces Noncopyable and FastAllocBase classes and these inherits with their
880 equivalent macro implementation at the necessary places.
881 2011-01-20 Yi Shen <yi.4.shen@nokia.com>
885 Adding myself to committers.py.
887 * Scripts/webkitpy/common/config/committers.py:
889 2011-01-20 Adam Barth <abarth@webkit.org>
891 Reviewed by Tony Chang.
893 Chromium builders don't delete unversioned directories when DEPS change
894 https://bugs.webkit.org/show_bug.cgi?id=52745
896 If we pass this option to gclient sync, gclient will delete directories
897 that are no longer part of DEPS instead of leaving them around to
898 confuse us later. Apparently, the downstream buildbots use this option
901 * Scripts/update-webkit-chromium:
903 2011-01-19 Dirk Pranke <dpranke@chromium.org>
905 Unreviewed, build fix - I pulled this patch in to fix an issue
906 revealed by r76195 running on some new bots). Also fix a couple
907 of other issues revealed in testing.
909 nrwt: remove fs refs from layout_package/json*
911 https://bugs.webkit.org/show_bug.cgi?id=52754
913 * Scripts/webkitpy/layout_tests/layout_package/json_results_generator.py:
914 * Scripts/webkitpy/layout_tests/layout_package/json_layout_results_generator.py:
915 * Scripts/webkitpy/layout_tests/layout_package/json_results_generator_unittest.py:
916 * Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
917 * Scripts/webkitpy/layout_tests/port/base.py:
919 2011-01-19 Dirk Pranke <dpranke@chromium.org>
921 Unreviewed, build fix.
923 Fix a couple of minor bugs discovered while bringing up new bots
926 * Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:
927 * Scripts/webkitpy/layout_tests/run_webkit_tests.py:
929 2011-01-19 James Robinson <jamesr@chromium.org>
931 Unreviewed, rolling out r76194.
932 http://trac.webkit.org/changeset/76194
933 https://bugs.webkit.org/show_bug.cgi?id=51218
935 Caused mysterious compile failure on the chromium win
936 build.webkit.org bots
938 * DumpRenderTree/chromium/WebViewHost.cpp:
939 (WebViewHost::paintInvalidatedRegion):
940 * DumpRenderTree/chromium/WebViewHost.h:
942 2011-01-19 Dirk Pranke <dpranke@chromium.org>
944 Reviewed by Mihai Parparita.
946 remove fs refs from test_runner, dump_render_tree_thread
948 https://bugs.webkit.org/show_bug.cgi?id=52753
950 * Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:
951 * Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
952 * Scripts/webkitpy/layout_tests/layout_package/test_runner_unittest.py:
953 * Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
955 2011-01-19 James Robinson <jamesr@chromium.org>
957 Reviewed by Darin Fisher.
959 Implement mozilla's requestAnimationFrame API
960 https://bugs.webkit.org/show_bug.cgi?id=51218
962 Chromium DumpRenderTree support for window.webkitRequestAnimationFrame.
964 * DumpRenderTree/chromium/WebViewHost.cpp:
965 (invokeScheduleComposite):
966 (WebViewHost::scheduleAnimation):
967 (WebViewHost::paintInvalidatedRegion):
968 * DumpRenderTree/chromium/WebViewHost.h:
970 2011-01-19 Dirk Pranke <dpranke@chromium.org>
972 Unreviewed, build fix.
974 Fix a few incorrect "self._filesystem" references introduced in
977 * Scripts/webkitpy/layout_tests/port/chromium.py:
978 * Scripts/webkitpy/layout_tests/port/chromium_linux.py:
980 2011-01-19 Dirk Pranke <dpranke@chromium.org>
982 Reviewed by Mihai Parparita.
984 add a .sep property, abspath(), isabs(), mtime(), and
985 open_text_file_for_writing() to filesystem modules. Some of
986 these properties are not needed in this patch but will be needed
987 in subsequent patches (I'm doing this to avoid having to track
988 multiple versions of a single file).
990 Also, change most of the port/* modules to use the filesystem
991 objects instead of referencing the filesystem directly.
993 https://bugs.webkit.org/show_bug.cgi?id=52748
995 * Scripts/webkitpy/common/system/filesystem.py:
996 * Scripts/webkitpy/common/system/filesystem_mock.py:
997 * Scripts/webkitpy/layout_tests/port/__init__.py:
998 * Scripts/webkitpy/layout_tests/port/base.py:
999 * Scripts/webkitpy/layout_tests/port/base_unittest.py:
1000 * Scripts/webkitpy/layout_tests/port/chromium.py:
1001 * Scripts/webkitpy/layout_tests/port/chromium_unittest.py:
1002 * Scripts/webkitpy/layout_tests/port/config.py:
1003 * Scripts/webkitpy/layout_tests/port/test.py:
1004 * Scripts/webkitpy/layout_tests/port/test_files_unittest.py:
1006 2011-01-19 Maciej Stachowiak <mjs@apple.com>
1008 Reviewed by Anders Carlsson.
1010 If resetting between tests times out, treat it as a WebProcess crash
1011 https://bugs.webkit.org/show_bug.cgi?id=52757
1013 * WebKitTestRunner/TestController.cpp:
1014 (WTR::TestController::runTest):
1016 2011-01-19 Alexey Proskuryakov <ap@apple.com>
1018 Reviewed by Darin Adler.
1020 https://bugs.webkit.org/show_bug.cgi?id=52739
1021 Make it possible for a WebKit2 client to print headers and footers
1023 Added dummy implementations for new printing UIClient calls.
1025 * MiniBrowser/mac/BrowserWindowController.m:
1026 (-[BrowserWindowController awakeFromNib]):
1027 * WebKitTestRunner/TestController.cpp:
1028 (WTR::createOtherPage):
1029 (WTR::TestController::initialize):
1031 2011-01-19 Dirk Pranke <dpranke@chromium.org>
1033 Reviewed by Ojan Vafai.
1035 Change webkitpy/layout_tests/port/test.py to use the
1036 in-memory filesystem for cleaner unit testing. This
1037 change allows us to kill a lot of code that was
1038 specific to the test port, at the cost of being a
1039 little less clear about how things would work if
1040 you didn't have a filesystem.
1042 https://bugs.webkit.org/show_bug.cgi?id=52605
1044 * Scripts/webkitpy/layout_tests/port/test.py:
1046 2011-01-19 Dirk Pranke <dpranke@chromium.org>
1048 Unreviewed, build fix.
1050 Fix test failure caused by committing attachment 79368 instead
1051 of attachment 79461 on bug 52604.
1053 * Scripts/webkitpy/common/system/filesystem_mock.py:
1055 2011-01-19 Dirk Pranke <dpranke@chromium.org>
1057 Reviewed by Mihai Parparita.
1059 new-run-webkit-tests: remove use of os.walk, use mock filesystem for better
1060 unit testing. os.walk() is too much of a hassle to implement on
1061 top of the in-memory mock filesystem and adding the necessary
1062 interface to files_under() gives clients a cleaner API anyway
1063 (for this particular usage model).
1065 https://bugs.webkit.org/show_bug.cgi?id=52691
1067 * Scripts/webkitpy/common/system/filesystem.py:
1068 * Scripts/webkitpy/common/system/filesystem_mock.py:
1069 * Scripts/webkitpy/layout_tests/port/test_files.py:
1070 * Scripts/webkitpy/layout_tests/port/test_files_unittest.py:
1072 2011-01-19 Dirk Pranke <dpranke@chromium.org>
1074 Reviewed by Tony Chang.
1076 Change more modules in the layout-tests code to use the
1077 filesystem wrapper for cleaner unit testing.
1079 This patch also adds the glob() wrapper to the filesystem
1082 https://bugs.webkit.org/show_bug.cgi?id=52604
1084 * Scripts/webkitpy/layout_tests/port/test_files.py:
1085 * Scripts/webkitpy/layout_tests/port/test_files_unittest.py:
1086 * Scripts/webkitpy/layout_tests/test_types/test_type_base.py:
1087 * Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:
1088 * Scripts/webkitpy/common/system/filesystem.py:
1089 * Scripts/webkitpy/common/system/filesystem_mock.py:
1091 2011-01-19 Levi Weintraub <leviw@chromium.org>
1095 Updating my email addresses and irc nick.
1097 * Scripts/webkitpy/common/config/committers.py:
1099 2011-01-19 Tony Chang <tony@chromium.org>
1101 Reviewed by Mihai Parparita.
1103 [chromium] [linux] if check-sys-deps fails, output the failure reason
1104 https://bugs.webkit.org/show_bug.cgi?id=52671
1106 * Scripts/webkitpy/common/system/executive_mock.py: Add support for
1107 error handler functions.
1108 * Scripts/webkitpy/layout_tests/port/chromium.py:
1109 output the error text from --check-sys-deps
1110 * Scripts/webkitpy/layout_tests/port/chromium_unittest.py:
1112 2011-01-19 Aparna Nandyal <aparna.nand@wipro.com>
1114 Reviewed by Andreas Kling.
1116 [Qt] QtTestBrowser crashes when closing if Facebook is opened
1117 https://bugs.webkit.org/show_bug.cgi?id=52554
1119 QtNetworkAccessManager is created in a thread and set as
1120 member variable in QWebPage. When the thread is destroyed
1121 the object is still used. So fix will set the object to
1122 NULL once the thread is destroyed.
1124 * QtTestBrowser/launcherwindow.cpp:
1125 (LauncherWindow::~LauncherWindow):
1127 2011-01-18 Maciej Stachowiak <mjs@apple.com>
1129 Rubber stamped by Csaba Osztrogonac
1131 * WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp: Removed unncessary inculde of WKStringCF.h
1133 2011-01-18 Dirk Pranke <dpranke@chromium.org>
1135 Reviewed by Tony Chang.
1137 new-run-webkit-tests: add a --build-directory command line argument
1139 Official Google Chrome builds use a non-standard build directory
1140 location. This patch adds a --build-directory argument that
1141 allows that location to be specified on the command line. Only
1142 Chromium-based builds will use this flag for now, but anyone can
1145 There are no unit tests for this since it's difficult to test
1146 until mock filesystems are fully supported in the code. This was
1147 tested by hand for now.
1149 https://bugs.webkit.org/show_bug.cgi?id=52694
1151 * Scripts/webkitpy/layout_tests/port/chromium_linux.py:
1152 * Scripts/webkitpy/layout_tests/port/chromium_mac.py:
1153 * Scripts/webkitpy/layout_tests/port/chromium_win.py:
1154 * Scripts/webkitpy/layout_tests/run_webkit_tests.py:
1156 2011-01-18 Maciej Stachowiak <mjs@apple.com>
1158 Not reviewed. Bot fix.
1160 Remove an inadvertently committed debugging print.
1162 * DumpRenderTree/mac/LayoutTestControllerMac.mm:
1163 (LayoutTestController::notifyDone):
1165 2011-01-18 Maciej Stachowiak <mjs@apple.com>
1167 Reviewed by Sam Weinig.
1169 WebKitTestRunner should track loading more like DumpRenderTree
1170 https://bugs.webkit.org/show_bug.cgi?id=52692
1172 Change load tracking to track the current top loading frame, in the manner of DumpRenderTree.
1173 This makes some tests that call notifyDone multiple times pass.
1175 * WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:
1176 (WTR::InjectedBundle::InjectedBundle):
1177 (WTR::InjectedBundle::done):
1178 * WebKitTestRunner/InjectedBundle/InjectedBundle.h:
1179 (WTR::InjectedBundle::topLoadingFrame):
1180 (WTR::InjectedBundle::setTopLoadingFrame):
1181 * WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:
1182 (WTR::InjectedBundlePage::InjectedBundlePage):
1183 (WTR::InjectedBundlePage::stopLoading):
1184 (WTR::InjectedBundlePage::didStartProvisionalLoadForFrame):
1185 (WTR::InjectedBundlePage::didFailProvisionalLoadWithErrorForFrame):
1186 (WTR::InjectedBundlePage::didFinishLoadForFrame):
1187 (WTR::InjectedBundlePage::didFailLoadWithErrorForFrame):
1188 * WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:
1189 * WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:
1190 (WTR::LayoutTestController::notifyDone):
1191 * WebKitTestRunner/TestController.cpp:
1192 (WTR::TestController::didReceiveMessageFromInjectedBundle):
1194 2011-01-18 Mihai Parparita <mihaip@chromium.org>
1196 Reviewed by Kent Tamura.
1198 [Chromium] NRWT looking for DRT binary in the wrong path on Mac
1199 https://bugs.webkit.org/show_bug.cgi?id=52678
1201 Update path now that WebKit is inside Source.
1203 * Scripts/webkitpy/layout_tests/port/chromium_mac.py:
1205 2011-01-18 Robert Hogan <robert@webkit.org>
1207 Reviewed by Simon Hausmann.
1209 [Qt] http/tests/incremental/slow-utf8-text.pl fails
1210 https://bugs.webkit.org/show_bug.cgi?id=44282
1212 * DumpRenderTree/qt/DumpRenderTreeQt.cpp:
1213 (WebCore::DumpRenderTree::dump):
1215 2011-01-18 Dirk Pranke <dpranke@chromium.org>
1217 Reviewed by Mihai Parparita.
1219 Fix a small race condition that could cause the unit tests for
1220 new-run-webkit-tests to fail in a multithreaded environment.
1221 There was a small window in the code where one thread would
1222 check to see if a thread had raised an exception, do something,
1223 and then check to see if the thread was alive. If the other
1224 thread raised an exception and exited in between these two
1225 steps, the exception would be lost. Flipping the order of the
1226 two checks fixes the race, since we will now always check for an
1227 exception on the thread before exiting.
1229 https://bugs.webkit.org/show_bug.cgi?id=51579
1231 * Scripts/webkitpy/layout_tests/layout_package/message_broker.py:
1233 2011-01-18 Eric Seidel <eric@webkit.org>
1235 Reviewed by Adam Barth.
1237 commit-queue dies when test archiving fails
1238 https://bugs.webkit.org/show_bug.cgi?id=52617
1240 I looked at the machine and it had 10 archives already
1241 thus find_unused_name was returning None. I've upped
1242 the limit to 100 (per bug) and tested the case where
1243 find_unused_name returns None (making archive return None).
1245 * Scripts/webkitpy/common/system/workspace.py:
1246 * Scripts/webkitpy/common/system/workspace_unittest.py:
1247 * Scripts/webkitpy/tool/bot/commitqueuetask.py:
1248 * Scripts/webkitpy/tool/bot/commitqueuetask_unittest.py:
1249 * Scripts/webkitpy/tool/commands/queues.py:
1251 2011-01-18 Dirk Pranke <dpranke@chromium.org>
1253 Reviewed by Ojan Vafai.
1255 Update rebaseline-chromium-webkit-tests unit tests to use a mock
1256 filesystem. Also fix a couple of "with" statements missed by
1259 https://bugs.webkit.org/show_bug.cgi?id=52487
1261 * Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests_unittest.py:
1263 2011-01-18 Sergio Villar Senin <svillar@igalia.com>
1265 Reviewed by Martin Robinson.
1267 [Gtk] Build fixes for gtk3 build
1268 https://bugs.webkit.org/show_bug.cgi?id=52648
1270 * DumpRenderTree/gtk/EventSender.cpp:
1271 (beginDragWithFilesCallback):
1273 2011-01-18 Dirk Pranke <dpranke@chromium.org>
1275 Reviewed by Mihai Parparita.
1277 Update rebaseline-chromium-webkit-tests to use filesystem objects
1278 instead of direct references to os.path, shutil, tempfile, etc.
1280 This patch doesn't change anything, but will allow subsequent
1281 patches to change the unit tests to no longer use the real
1284 This patch adds a bunch more methods to the filesystem object as
1287 https://bugs.webkit.org/show_bug.cgi?id=52482
1289 * Scripts/webkitpy/common/system/filesystem.py:
1290 * Scripts/webkitpy/common/system/filesystem_mock.py:
1291 * Scripts/webkitpy/tool/commands/queues.py:
1292 * Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
1293 * Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests_unittest.py:
1295 2011-01-18 Dirk Pranke <dpranke@chromium.org>
1297 Reviewed by Mihai Parparita.
1299 new-run-webkit-tests shouldn't crash when you try to run a
1300 directory containing only skipped files.
1302 https://bugs.webkit.org/show_bug.cgi?id=52478
1304 * Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
1305 * Scripts/webkitpy/layout_tests/port/test.py:
1306 * Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
1308 2011-01-18 Adam Bergkvist <adam.bergkvist@ericsson.com> and Martin Robinson <mrobinson@igalia.com>
1310 Reviewed by Gustavo Noronha Silva.
1312 [GTK] The GTK+ EventSender needs support for beginDragWithFiles
1313 https://bugs.webkit.org/show_bug.cgi?id=40833
1315 * DumpRenderTree/gtk/EventSender.cpp: Add a beginDragWithFiles implementation for the
1317 (dragWithFilesDragDataGetCallback): Added.
1318 (dragWithFilesDragEndCallback): Added.
1319 (beginDragWithFilesCallback): Added.
1321 2011-01-18 Gyuyoung Kim <gyuyoung.kim@samsung.com>
1323 Reviewed by Adam Barth.
1325 ews: Add additional watcher to efl ews
1326 https://bugs.webkit.org/show_bug.cgi?id=52613
1328 Add gyuyoung.kim@samsung.com to watcher list of efl-ews.
1330 * Scripts/webkitpy/tool/commands/earlywarningsystem.py:
1332 2011-01-17 Tony Gentilcore <tonyg@chromium.org>
1334 Reviewed by Alexey Proskuryakov.
1336 Fix some headers with missing or misspelled #ifndef guards
1337 https://bugs.webkit.org/show_bug.cgi?id=52545
1339 * DumpRenderTree/chromium/TestEventPrinter.h:
1340 * DumpRenderTree/chromium/WebPreferences.h:
1342 2011-01-17 Jessie Berlin <jberlin@apple.com>
1344 Reviewed by Anders Carlsson.
1346 REGRESSION: Assertion failure in FrameLoader::continueLoadAfterWillSubmitForm() when
1347 navigating back to an unreachable URL
1348 https://bugs.webkit.org/show_bug.cgi?id=52388
1350 Add a third parameter (unreachableURL) to queueLoadHTMLString.
1352 * DumpRenderTree/LayoutTestController.cpp:
1353 (queueLoadHTMLStringCallback):
1354 (LayoutTestController::queueLoadHTMLString):
1355 (LayoutTestController::queueLoadAlternateHTMLString):
1356 * DumpRenderTree/LayoutTestController.h:
1358 * DumpRenderTree/WorkQueueItem.h:
1359 (LoadHTMLStringItem::LoadHTMLStringItem):
1361 * DumpRenderTree/chromium/LayoutTestController.cpp:
1362 (WorkItemLoadHTMLString::WorkItemLoadHTMLString):
1363 (WorkItemLoadHTMLString::run):
1364 (LayoutTestController::queueLoadHTMLString):
1366 * DumpRenderTree/gtk/WorkQueueItemGtk.cpp:
1367 (LoadHTMLStringItem::invoke):
1368 * DumpRenderTree/mac/WorkQueueItemMac.mm:
1369 (LoadHTMLStringItem::invoke):
1370 * DumpRenderTree/win/WorkQueueItemWin.cpp:
1371 (LoadHTMLStringItem::invoke):
1373 2011-01-17 Dan Bernstein <mitz@apple.com>
1375 Rubber-stamped by Mark Rowe.
1377 Update xcodeproj svn:ignore to include xcuserdata.
1379 * DumpRenderTree/DumpRenderTree.xcodeproj: Modified property svn:ignore.
1380 * MiniBrowser/MiniBrowser.xcodeproj: Modified property svn:ignore.
1381 * TestWebKitAPI/TestWebKitAPI.xcodeproj: Modified property svn:ignore.
1382 * WebKitLauncher/WebKitLauncher.xcodeproj: Modified property svn:ignore.
1383 * WebKitTestRunner/WebKitTestRunner.xcodeproj: Modified property svn:ignore.
1385 2011-01-17 Csaba Osztrogonác <ossy@webkit.org>
1387 Unreviewed fix after r75908.
1389 Move WebKit into Source
1390 https://bugs.webkit.org/show_bug.cgi?id=52530
1392 * BuildSlaveSupport/build.webkit.org-config/master.cfg: Add Source to the path for Qt API tests.
1394 2011-01-17 Csaba Osztrogonác <ossy@webkit.org>
1398 Remove accidentally committed debug printing after r75908.
1400 * Scripts/webkitdirs.pm:
1402 2011-01-17 John Knottenbelt <jknotten@chromium.org>
1406 Add myself to committers list
1407 https://bugs.webkit.org/show_bug.cgi?id=52566
1409 * Scripts/webkitpy/common/config/committers.py:
1411 2011-01-17 John Knottenbelt <jknotten@chromium.org>
1413 Reviewed by Jeremy Orlow.
1415 GeolocationController should call stopUpdating on destruction
1416 https://bugs.webkit.org/show_bug.cgi?id=52216
1418 fast/dom/Geolocation/window-close-crash.html requires that a
1419 Geolocation watch be started in a secondary window. Consequently,
1420 we need to allow geolocation permission and provide a mock
1421 geolocation position for the secondary window's
1422 GeolocationClientMock.
1424 * DumpRenderTree/chromium/LayoutTestController.cpp:
1425 (LayoutTestController::setGeolocationPermission):
1426 (LayoutTestController::setMockGeolocationPosition):
1427 (LayoutTestController::setMockGeolocationError):
1428 * DumpRenderTree/chromium/TestShell.h:
1429 (TestShell::windowList):
1431 2011-01-16 Adam Barth <abarth@webkit.org>
1433 Update more include paths to reflect WebKit move.
1435 * DumpRenderTree/chromium/EventSender.cpp:
1436 (EventSender::keyDown):
1437 (EventSender::needsShiftModifier):
1438 * DumpRenderTree/chromium/WebThemeEngineDRTMac.h:
1439 * DumpRenderTree/chromium/WebThemeEngineDRTMac.mm:
1441 2011-01-16 Kent Tamura <tkent@chromium.org>
1443 Unreviewed, Chromium build fix.
1445 * Scripts/update-webkit-chromium: Webkit/chromium -> Source/WebKit/chromium
1447 2011-01-16 Martin Robinson <mrobinson@igalia.com>
1449 Build fix for GTK+ after source directory movement.
1451 * GNUmakefile.am: Correct include paths.
1453 2011-01-16 Adam Barth <abarth@webkit.org>
1455 Teach svn-apply about WebKit move.
1457 * Scripts/VCSUtils.pm:
1459 2011-01-16 Adam Barth <abarth@webkit.org>
1461 Rubber-stamped by Eric Seidel.
1463 Move WebKit into Source
1464 https://bugs.webkit.org/show_bug.cgi?id=52530
1466 * DumpRenderTree/mac/InternalHeaders/WebKit/WebTypesInternal.h:
1467 * DumpRenderTree/qt/DumpRenderTreeQt.cpp:
1468 * DumpRenderTree/qt/DumpRenderTreeQt.h:
1469 * DumpRenderTree/qt/GCControllerQt.cpp:
1470 * DumpRenderTree/qt/LayoutTestControllerQt.cpp:
1471 * DumpRenderTree/qt/PlainTextControllerQt.cpp:
1472 * DumpRenderTree/qt/TextInputControllerQt.cpp:
1473 * DumpRenderTree/wscript:
1474 * QtTestBrowser/launcherwindow.h:
1475 * Scripts/build-webkit:
1476 * Scripts/webkitdirs.pm:
1478 2011-01-15 Adam Barth <abarth@webkit.org>
1480 Teach svn-apply about WebKit2 move.
1482 * Scripts/VCSUtils.pm:
1484 2011-01-15 Adam Barth <abarth@webkit.org>
1486 Rubber-stamped by Eric Seidel.
1488 Move WebKit2 into Source
1489 https://bugs.webkit.org/show_bug.cgi?id=52438
1491 * MiniBrowser/qt/MiniBrowser.pro:
1492 * Scripts/build-webkit:
1493 * Scripts/do-file-rename:
1494 * Scripts/do-webcore-rename:
1495 * Scripts/test-webkitpy:
1496 * Scripts/webkitdirs.pm:
1497 * Scripts/webkitpy/style/checker.py:
1498 * WebKitTestRunner/DerivedSources.pro:
1499 * WebKitTestRunner/InjectedBundle/qt/InjectedBundle.pro:
1500 * WebKitTestRunner/qt/WebKitTestRunner.pro:
1502 2011-01-15 Jochen Eisinger <jochen@chromium.org>
1504 Reviewed by David Levin.
1506 [chromium] remove obsolete createView method
1507 https://bugs.webkit.org/show_bug.cgi?id=52437
1509 * DumpRenderTree/chromium/WebViewHost.cpp:
1510 (WebViewHost::createView):
1511 * DumpRenderTree/chromium/WebViewHost.h:
1513 2011-01-14 Brian Weinstein <bweinstein@apple.com>
1515 Reviewed by Geoff Garen.
1517 ShouldLoadResourceForFrame should use strings, not URLs.
1518 https://bugs.webkit.org/show_bug.cgi?id=52476
1520 * WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:
1521 (WTR::InjectedBundlePage::shouldLoadResourceForFrame): Update to use a WKStringRef.
1522 * WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:
1524 2011-01-14 Tony Gentilcore <tonyg@chromium.org>
1526 Reviewed by David Levin.
1528 Add Tony Gentilcore as a reviewer
1529 https://bugs.webkit.org/show_bug.cgi?id=52480
1531 * Scripts/webkitpy/common/config/committers.py:
1533 2011-01-14 David Levin <levin@chromium.org>
1535 Reviewed by Eric Seidel.
1537 check-webkit-style: _FunctionState should keep track of where the function name starts.
1538 https://bugs.webkit.org/show_bug.cgi?id=52454
1540 * Scripts/webkitpy/style/checkers/cpp.py: Added function_name_start_position to
1541 _FunctionState and logic for setting it correctly.
1542 * Scripts/webkitpy/style/checkers/cpp_unittest.py: Added tests.
1544 2011-01-14 Vincent Scheib <scheib@chromium.org>
1546 Reviewed by David Levin.
1548 Add support to DumpRenderTree to use the GPU rather than software renderer
1549 https://bugs.webkit.org/show_bug.cgi?id=52333
1551 * DumpRenderTree/chromium/DumpRenderTree.cpp:
1553 * Scripts/webkitpy/layout_tests/port/chromium.py:
1554 * Scripts/webkitpy/layout_tests/run_webkit_tests.py:
1556 2011-01-14 Tony Chang <tony@chromium.org>
1558 Reviewed by Kent Tamura.
1560 [chromium] update fonts.conf to fix 6 linux tests in drt
1561 https://bugs.webkit.org/show_bug.cgi?id=52411
1563 * DumpRenderTree/chromium/fonts.conf: Update to match the copy in
1564 chromium's src/webkit/tools/test_shell/resources/fonts.conf
1566 2011-01-14 Eric Seidel <eric@webkit.org>
1568 Original patch from Peter Gal <galpater@inf.u-szeged.hu>
1570 Reviewed by Adam Barth.
1572 Fix when running Tools/Scripts/check-webkit-style without arguments.
1573 https://bugs.webkit.org/show_bug.cgi?id=52261#c16
1575 * Scripts/webkitpy/common/checkout/scm.py: Handle None for changed_files argument better in Git.create_patch method.
1576 * Scripts/webkitpy/common/checkout/scm_unittest.py:
1578 2011-01-04 Jochen Eisinger <jochen@chromium.org>
1580 Reviewed by David Levin.
1582 Store the tempdir object in a variable so it doesn't get deleted too early.
1583 https://bugs.webkit.org/show_bug.cgi?id=51875
1585 * Scripts/update-webkit-auxiliary-libs:
1586 * Scripts/update-webkit-support-libs:
1588 2011-01-14 Eric Seidel <eric@webkit.org>
1590 Unreviewed. Make it work in python 2.5.
1592 webkit-patch suggest-reviewers dies when ChangeLogs are missing
1593 https://bugs.webkit.org/show_bug.cgi?id=49158
1595 * Scripts/webkitpy/common/checkout/api_unittest.py:
1597 2011-01-14 Sheriff Bot <webkit.review.bot@gmail.com>
1599 Unreviewed, rolling out r75774.
1600 http://trac.webkit.org/changeset/75774
1601 https://bugs.webkit.org/show_bug.cgi?id=52431
1603 gtk builds were broken (Requested by loislo2 on #webkit).
1605 * Scripts/webkitpy/common/checkout/scm.py:
1607 2011-01-14 David Levin <levin@chromium.org>
1609 Reviewed by Shinichiro Hamaji.
1611 check-webkit-style: _FunctionState should detect pure functions.
1612 https://bugs.webkit.org/show_bug.cgi?id=52428
1614 * Scripts/webkitpy/style/checkers/cpp.py: Added detection for if a function declaration
1615 is a pure virtual function.
1616 * Scripts/webkitpy/style/checkers/cpp_unittest.py: Added tests for verifying that pure
1617 virtual functions are correctly detected.
1619 2011-01-13 Ilya Tikhonovsky <loislo@chromium.org>
1621 Reviewed by Yury Semikhatsky.
1623 Just small fix for problem with check-webkit-style.
1624 It just didn't work for this patch.
1625 Old code can't concatenate a list of strings with None.
1627 * Scripts/webkitpy/common/checkout/scm.py:
1629 2011-01-13 David Levin <levin@chromium.org>
1631 Reviewed by Shinichiro Hamaji.
1633 check-webkit-style: _FunctionState should use Position to know where items are.
1634 https://bugs.webkit.org/show_bug.cgi?id=52424
1636 * Scripts/webkitpy/style/checkers/cpp.py:
1637 (_FunctionState.*): Changed _FunctionState to use Position instead of line numbers.
1638 (detect_functions): Changed to pass Position's to _FunctionState.begin and
1639 did some minor clean-up.
1640 (*): Other changes are simply about converting to use the Position's in_FunctionState
1641 instead of line numbers.
1642 * Scripts/webkitpy/style/checkers/cpp_unittest.py: Changed the test code
1643 to verify the positions stored in _FunctionState.
1645 2011-01-13 Eric Seidel <eric@webkit.org>
1647 Reviewed by David Levin.
1649 webkit-patch suggest-reviewers dies when ChangeLogs are missing
1650 https://bugs.webkit.org/show_bug.cgi?id=49158
1652 This is not the most elegant, but it is a very safe fix to this bug.
1653 One advantage of catching ScriptError like this instead of adding a
1654 new added_or_modified_files or fixing all changed_files callers
1655 to use a more specific change_files variant, is that we catch
1656 all kinds of ScriptErrors which might cause our (non-essential)
1657 suggest-reviewers code to fail out. This should make passing
1658 --suggest-reviewers to webkit-patch upload much more robust
1659 and may even make it possible for us to make it default.
1661 The root of the problem here is that SCM.changed_files includes
1662 deleted ChangeLog paths (from moves, etc) which then when we ask
1663 SVN/Git for the contents of the file at that revision, the command
1664 errors out and Executive.run_command raises a ScriptError.
1666 In the future we might fix this differently by making all current
1667 callers of chagned_files use a more specific method for requesting
1668 what types of changes they're interested in (adds, modifies, deletes, etc.)
1670 * Scripts/webkitpy/common/checkout/api.py:
1671 * Scripts/webkitpy/common/checkout/api_unittest.py:
1673 2011-01-13 Dan Bernstein <mitz@apple.com>
1675 Reviewed by Alexey Proskuryakov.
1677 <rdar://problem/8827065> REGRESSION (r71884): Cross-origin XHR fails if willSendRequest changes the URL
1678 https://bugs.webkit.org/show_bug.cgi?id=52419
1680 Allow to further control the behavior of the willSendRequest delegate callback by making it
1681 set a different URL in the new request.
1683 * DumpRenderTree/LayoutTestController.cpp:
1684 (addURLToRedirectCallback): Added.
1685 (LayoutTestController::staticFunctions): Added addURLToRedirect.
1686 (LayoutTestController::addURLToRedirect): Added. Adds the redirection to the map.
1687 (LayoutTestController::redirectionDestinationForURL): Added this getter.
1688 * DumpRenderTree/LayoutTestController.h:
1689 * DumpRenderTree/mac/ResourceLoadDelegate.mm:
1690 (-[ResourceLoadDelegate webView:resource:willSendRequest:redirectResponse:fromDataSource:]): Check
1691 if there is a redirection for the request’s URL, established by addURLToRedirect(), and if so,
1692 change the URL in the new request to the redirect destination.
1694 2011-01-13 Eric Seidel <eric@webkit.org>
1696 Reviewed by Mihai Parparita.
1698 webkit-patch suggest-reviewers fails due to invalid utf8 in ChangeLog files
1699 https://bugs.webkit.org/show_bug.cgi?id=52416
1701 Example: webkit-patch suggest-reviewers -g ca3890fe74d94d85d6bfa48f9ea497b094d8e717
1702 UnicodeDecodeError: 'utf8' codec can't decode bytes in position 536-538: invalid data
1704 I also added FIXMEs about the changed_files returning deleted files problem
1705 which is causing the other most common exception for suggest-reviewers
1706 (and is probably breaking sheriff-bots blame detection).
1708 * Scripts/webkitpy/common/checkout/api.py:
1709 * Scripts/webkitpy/common/checkout/api_unittest.py:
1710 * Scripts/webkitpy/common/checkout/scm.py:
1712 2011-01-13 Eric Seidel <eric@webkit.org>
1714 Reviewed by Adam Barth.
1716 results archives from commit-queue are too large to upload
1717 https://bugs.webkit.org/show_bug.cgi?id=52405
1719 * Scripts/webkitpy/common/system/filesystem.py:
1720 * Scripts/webkitpy/common/system/filesystem_mock.py:
1721 * Scripts/webkitpy/tool/commands/queues.py:
1723 2011-01-13 Alejandro G. Castro <alex@igalia.com>
1725 Reviewed by Martin Robinson.
1727 [gtk] Zero-sized font does not yet work
1728 https://bugs.webkit.org/show_bug.cgi?id=49793
1730 Set minimum-font-size to 0 for the DRT.
1732 * DumpRenderTree/gtk/DumpRenderTree.cpp:
1733 (resetDefaultsToConsistentValues):
1735 2011-01-13 Csaba Osztrogonác <ossy@webkit.org>
1737 Reviewed by Darin Adler.
1739 Make old-run-webkit-tests more robust on buildbots
1740 https://bugs.webkit.org/show_bug.cgi?id=52364
1742 * BuildSlaveSupport/build.webkit.org-config/master.cfg: Pass --exit-after-n-failures 500 to old-run-webkit-tests script.
1744 2011-01-13 Csaba Osztrogonác <ossy@webkit.org>
1746 Reviewed by Darin Adler.
1748 Make old-run-webkit-tests more robust on buildbots
1749 https://bugs.webkit.org/show_bug.cgi?id=52364
1751 * Scripts/old-run-webkit-tests: Ensure that testResultsDirectory is empty.
1753 2011-01-13 Tony Chang <tony@chromium.org>
1755 Reviewed by Kent Tamura.
1757 [chromium] fix plugins/plugin-javascript-access.html on Linux and Mac
1758 https://bugs.webkit.org/show_bug.cgi?id=52332
1760 Forked Info.plist is not longer needed.
1762 * DumpRenderTree/chromium/TestNetscapePlugIn/Info.plist: Removed.
1764 2011-01-13 Mikhail Naganov <mnaganov@chromium.org>
1766 Reviewed by Timothy Hatcher.
1768 Web Inspector: Cleanup after r75613 -- make all UI-facing strings
1769 to be passed directly to WebInspector.UIString
1771 https://bugs.webkit.org/show_bug.cgi?id=52365
1773 * Scripts/check-inspector-strings:
1775 2011-01-13 Sheriff Bot <webkit.review.bot@gmail.com>
1777 Unreviewed, rolling out r74881.
1778 http://trac.webkit.org/changeset/74881
1779 https://bugs.webkit.org/show_bug.cgi?id=52357
1781 The EFL builder has been red for days (Requested by abarth on
1784 * Scripts/webkitpy/common/net/buildbot/buildbot.py:
1785 * Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py:
1787 2011-01-12 Maciej Stachowiak <mjs@apple.com>
1789 Reviewed by David Levin.
1791 Add beforeunload support to WebKitTestRunner
1792 https://bugs.webkit.org/show_bug.cgi?id=52353
1794 This makes a number of currently failing tests pass.
1796 * WebKitTestRunner/TestController.cpp:
1797 (WTR::runBeforeUnloadConfirmPanel):
1798 (WTR::createOtherPage):
1799 (WTR::TestController::initialize):
1801 2011-01-12 Maciej Stachowiak <mjs@apple.com>
1803 Reviewed by Sam Weinig.
1805 Detect, identify and recover from WebProcess crashes in run-webkit-tests
1806 https://bugs.webkit.org/show_bug.cgi?id=52330
1808 * Scripts/old-run-webkit-tests: Detect when the regression test tool reports
1809 a web process crash.
1810 * WebKitTestRunner/TestController.cpp:
1811 (WTR::TestController::initialize): Set up a processDidCrash
1813 (WTR::TestController::processDidCrash): Report that the Web process
1815 * WebKitTestRunner/TestController.h:
1817 2011-01-12 David Levin <levin@chromium.org>
1819 Reviewed by Shinichiro Hamaji.
1821 check-webkit-style -- close_expression function doesn't work correctly.
1822 https://bugs.webkit.org/show_bug.cgi?id=52272
1824 * Scripts/webkitpy/style/checkers/cpp.py:
1825 (Position.__str__): Added a way to convert it to a string which is useful in tests.
1826 (Position.__cmp__): Added a way to compare Position which is useful in tests and
1827 generally useful (for upcoming code changes).
1828 (close_expression): Changed to use Position for input and output.
1829 Also, fixed many bugs such as only working correctly for parenthesis,
1830 not working correctly if given an offset in a line and not finding
1831 the real closing element if there were multiple closing elements in
1833 (detect_functions): Adjusted due to the change in arguments for
1835 * Scripts/webkitpy/style/checkers/cpp_unittest.py:
1836 (CppStyleTestBase.assert_positions_equal): Added a way to verify that
1837 two positions are the same.
1838 (CppStyleTest.test_position): Added tests for the __str_ and __cmp__ methods.
1839 (CppStyleTest.test_close_expression): Added tests to catch the issues
1842 2011-01-12 Sheriff Bot <webkit.review.bot@gmail.com>
1844 Unreviewed, rolling out r75576.
1845 http://trac.webkit.org/changeset/75576
1846 https://bugs.webkit.org/show_bug.cgi?id=52304
1848 broke rebaseline-chromium-webkit-test (Requested by thakis on
1851 * Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
1852 * Scripts/webkitpy/layout_tests/layout_package/test_expectations.py:
1853 * Scripts/webkitpy/layout_tests/layout_package/test_expectations_unittest.py:
1854 * Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
1855 * Scripts/webkitpy/layout_tests/port/base.py:
1856 * Scripts/webkitpy/layout_tests/port/chromium.py:
1857 * Scripts/webkitpy/layout_tests/port/port_testcase.py:
1858 * Scripts/webkitpy/layout_tests/port/test.py:
1859 * Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
1860 * Scripts/webkitpy/layout_tests/run_webkit_tests.py:
1861 * Scripts/webkitpy/style/checkers/test_expectations.py:
1862 * Scripts/webkitpy/style/checkers/test_expectations_unittest.py:
1864 2011-01-12 Maciej Stachowiak <mjs@apple.com>
1866 Reviewed by Darin Adler.
1868 Add a no-build option to run-webkit-tests, to avoid building DRT/WKTR
1869 https://bugs.webkit.org/show_bug.cgi?id=52307
1871 * Scripts/old-run-webkit-tests:
1873 2011-01-12 Koan-Sin Tan <koansin.tan@gmail.com>
1875 Reviewed by Martin Robinson.
1877 [Gtk] WebKitGtk+ doesn't build on Mac OS X 10.6
1878 https://bugs.webkit.org/show_bug.cgi?id=50867
1880 Fixed library extention for Mac in webkitdirs.pm and type conflict in
1881 TestNetscapePlugin.cpp
1883 * GNUmakefile.am: Don't use AppKit when compiling TestNetscapePlugin for GTK+
1884 * Scripts/webkitdirs.pm:
1886 2011-01-12 Mikhail Naganov <mnaganov@chromium.org>
1888 Reviewed by Pavel Feldman.
1890 Add check-inspector-strings script.
1892 https://bugs.webkit.org/show_bug.cgi?id=52295
1894 * Scripts/check-inspector-strings: Added.
1896 2011-01-12 Adam Roben <aroben@apple.com>
1898 Advertise the .testnetscape file extension on Windows
1900 Fixes <http://webkit.org/b/52298>
1901 plugins/no-mime-with-valid-extension.html fails on Windows
1903 Reviewed by Dan Bernstein.
1905 * DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin.rc: Added a
1906 FileExtents field to map the .testnetscape file extension to the
1907 application/x-webkit-test-netscape MIME type, just as we do on Mac.
1909 2011-01-12 Adam Barth <abarth@webkit.org>
1911 Reviewed by Eric Seidel.
1913 Make webkit-patch support subdirectories for SVN checkouts
1914 https://bugs.webkit.org/show_bug.cgi?id=52261
1916 os.relpath does not exist on Python 2.5, so we need to call our version
1919 * Scripts/webkitpy/common/checkout/scm.py:
1921 2011-01-11 Maciej Stachowiak <mjs@apple.com>
1923 Reviewed by Eric Seidel.
1925 Close additional pages opened during a test once the test completes.
1926 https://bugs.webkit.org/show_bug.cgi?id=52276
1928 * WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:
1929 (WTR::InjectedBundle::done):
1931 2011-01-11 David Levin <levin@chromium.org>
1933 Reviewed by Eric Seidel.
1935 Errors should be separated from the command line in bot messages.
1936 https://bugs.webkit.org/show_bug.cgi?id=52275
1938 * Scripts/webkitpy/common/system/executive.py:
1939 (ScriptError.message_with_output): Added a newline to separate the command
1940 line from the errors.
1941 * Scripts/webkitpy/common/system/executive_unittest.py:
1942 (ScriptErrorTest.test_message_with_output): Added tests for the various code
1943 paths in message_with_output.
1945 2011-01-11 Eric Seidel <eric@webkit.org>
1949 commit-queue should know how to upload archived results (for test flakes or general failures)
1950 https://bugs.webkit.org/show_bug.cgi?id=52048
1952 The zips are mostly empty due to forgetting -r.
1953 Expected diffs were not being pulled from the archive due
1954 to the archive having longer paths than I realized.
1956 * Scripts/webkitpy/common/system/workspace.py:
1957 * Scripts/webkitpy/common/system/workspace_unittest.py:
1958 * Scripts/webkitpy/tool/bot/flakytestreporter.py:
1959 * Scripts/webkitpy/tool/bot/flakytestreporter_unittest.py:
1961 2011-01-11 Dirk Pranke <dpranke@chromium.org>
1963 Reviewed by Ojan Vafai.
1965 The current modifier parsing code in test_expectations is
1966 fragile and hard-coded, so it's not easy to understand the logic
1967 or easily add new types of modifiers (like GPU vs. CPU testing
1968 for graphics tests, or 32-bit vs. 64-bit differences).
1970 This is the first of two patches that will add in more generic
1971 support and then eliminate the GPU-specific test expectations
1974 This patch adds two standalone objects for handling modifiers. The
1975 rules for interpreting modifiers, precedence, and conflicts are
1976 given in the docstring to the ModifierMatcher class, which
1977 returns ModifierMatchResult objects.
1979 This patch also adds routines to the Port interface and a
1980 default set of values in the base object, in order to obtain the
1981 values needed on a given test run. These values are then passed
1982 to the expectation parser. This also allows us to clean up the
1983 logic used to lint all of the different configurations in a
1984 single test_expectations.txt file.
1986 The next patch will merge in the separate GPU expectations file.
1988 https://bugs.webkit.org/show_bug.cgi?id=51222
1990 * Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
1991 * Scripts/webkitpy/layout_tests/layout_package/test_expectations.py:
1992 * Scripts/webkitpy/layout_tests/layout_package/test_expectations_unittest.py:
1993 * Scripts/webkitpy/layout_tests/port/base.py:
1994 * Scripts/webkitpy/layout_tests/port/base_unittest.py:
1995 * Scripts/webkitpy/layout_tests/port/chromium.py:
1996 * Scripts/webkitpy/layout_tests/port/port_testcase.py:
1997 * Scripts/webkitpy/layout_tests/port/test.py:
1998 * Scripts/webkitpy/layout_tests/run_webkit_tests.py:
1999 * Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
2001 2011-01-11 Maciej Stachowiak <mjs@apple.com>
2003 Reviewed by Adam Barth.
2005 Make webkit-patch support subdirectories for SVN checkouts
2006 https://bugs.webkit.org/show_bug.cgi?id=52261
2008 * Scripts/webkitpy/common/checkout/scm.py:
2009 * Scripts/webkitpy/style_references.py:
2010 * Scripts/webkitpy/tool/main.py:
2011 * Scripts/webkitpy/tool/steps/preparechangelog.py:
2013 2011-01-11 Eric Seidel <eric@webkit.org>
2017 commit-queue should know how to upload archived results (for test flakes or general failures)
2018 https://bugs.webkit.org/show_bug.cgi?id=52048
2020 I changed the API for archive_last_layout_test_results w/o updating the implementation. Oops.
2021 This fixes an exception seen on the commit-queue when attempting to report flaky tests.
2023 * Scripts/webkitpy/common/system/workspace.py:
2024 * Scripts/webkitpy/common/system/workspace_unittest.py:
2025 * Scripts/webkitpy/tool/commands/queues.py:
2027 2011-01-11 Sam Weinig <sam@webkit.org>
2029 Roll r75474 back in.
2031 2011-01-11 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>
2033 Reviewed by Kenneth Rohde Christiansen.
2035 [Qt] Remove uses of QFuture since it isn't supported on all platforms.
2036 https://bugs.webkit.org/show_bug.cgi?id=51204
2038 * QtTestBrowser/webpage.h:
2039 (QtNAMThread::QtNAMThread):
2040 (QtNAMThread::networkAccessManager):
2043 2011-01-11 Peter Varga <pvarga@webkit.org>
2047 Add myself to committers' list.
2049 * Scripts/webkitpy/common/config/committers.py:
2051 2011-01-11 Eric Seidel <eric@webkit.org>
2053 Reviewed by Adam Barth.
2055 commit-queue should know how to upload archived results (for test flakes or general failures)
2056 https://bugs.webkit.org/show_bug.cgi?id=52048
2058 Now the queue will always upload results. Either the entire zip, or just
2059 the diffs.txt in the case of text failures.
2061 This should make understanding flakes much easier, and paves the way
2062 for having the EWS run layout tests (and upload failures).
2064 In order to upload .zip files I had to teach bugzilla.py to autodetect
2065 mime types from the filename. Since mimetypes.py doesn't include a mapping
2066 for .patch files, I have it add one before calling guess_type.
2068 We may find that always uploading the whole zip instead of just the -diffs.txt
2069 file is preferable, but for now I'm keeping the old behavior because it makes
2070 quickly understanding text failures easy.
2072 * Scripts/webkitpy/common/net/bugzilla/bugzilla.py:
2073 * Scripts/webkitpy/common/system/workspace.py: Added.
2074 * Scripts/webkitpy/common/system/workspace_unittest.py: Added.
2075 * Scripts/webkitpy/tool/bot/commitqueuetask.py:
2076 * Scripts/webkitpy/tool/bot/commitqueuetask_unittest.py:
2077 * Scripts/webkitpy/tool/bot/flakytestreporter.py:
2078 * Scripts/webkitpy/tool/bot/flakytestreporter_unittest.py:
2079 * Scripts/webkitpy/tool/commands/queues.py:
2080 * Scripts/webkitpy/tool/commands/queues_unittest.py:
2082 2011-01-10 Sheriff Bot <webkit.review.bot@gmail.com>
2084 Unreviewed, rolling out r75464.
2085 http://trac.webkit.org/changeset/75464
2086 https://bugs.webkit.org/show_bug.cgi?id=52193
2088 Followup rollout for 75474 (Requested by Ossy on #webkit).
2090 * MiniBrowser/mac/BrowserWindowController.m:
2091 (-[BrowserWindowController awakeFromNib]):
2092 * MiniBrowser/win/BrowserView.cpp:
2093 (BrowserView::create):
2094 * WebKitTestRunner/TestController.cpp:
2095 (WTR::createOtherPage):
2096 (WTR::TestController::initialize):
2098 2011-01-10 Laszlo Gombos <laszlo.1.gombos@nokia.com>
2100 Reviewed by Csaba Osztrogonác.
2102 [Qt] Baseline qt_minimal configuration
2103 https://bugs.webkit.org/show_bug.cgi?id=51313
2105 * DumpRenderTree/qt/DumpRenderTreeQt.cpp:
2106 (WebCore::DumpRenderTree::resetToConsistentStateBeforeTesting): Add
2107 QT_NO_UNDOSTACK guard.
2109 * QtTestBrowser/launcherwindow.cpp:
2110 (LauncherWindow::createChrome): Add QT_NO_SHORTCUT and QT_NO_PRINTER
2112 (LauncherWindow::applyZoom): Add QT_NO_ANIMATION guard.
2113 (LauncherWindow::screenshot): Add QT_NO_FILEDIALOG guard.
2114 (LauncherWindow::selectElements): Add QT_NO_INPUTDIALOG guard.
2115 (LauncherWindow::showUserAgentDialog): Add QT_NO_COMBOBOX guard.
2117 * QtTestBrowser/launcherwindow.h:
2118 * QtTestBrowser/locationedit.cpp: Add QT_NO_INPUTDIALOG guard.
2119 * QtTestBrowser/locationedit.h: Ditto.
2121 * QtTestBrowser/mainwindow.cpp:
2122 (MainWindow::buildUI): Add QT_NO_INPUTDIALOG, QT_NO_SHORTCUT and
2123 QT_NO_UNDOSTACK guards.
2124 (MainWindow::setAddressUrl): Add QT_NO_INPUTDIALOG guard.
2125 (MainWindow::changeLocation): Ditto.
2126 (MainWindow::openFile): Add QT_NO_FILEDIALOG guard.
2127 (MainWindow::openLocation): Add QT_NO_INPUTDIALOG guard.
2129 * QtTestBrowser/webpage.cpp:
2130 (WebPage::openUrlInDefaultBrowser): Add QT_NO_DESKTOPSERVICES guard.
2131 (WebPage::authenticationRequired): Add QT_NO_LINEEDIT guard.
2133 * QtTestBrowser/webview.cpp:
2134 (WebViewGraphicsBased::animatedFlip): Add QT_NO_ANIMATION guard.
2135 (WebViewGraphicsBased::animatedYFlip): Ditto.
2137 2011-01-10 Sam Weinig <sam@webkit.org>
2141 * MiniBrowser/mac/BrowserWindowController.m:
2142 (-[BrowserWindowController awakeFromNib]):
2143 * MiniBrowser/win/BrowserView.cpp:
2144 (BrowserView::create):
2145 * WebKitTestRunner/TestController.cpp:
2146 (WTR::createOtherPage):
2147 (WTR::TestController::initialize):
2149 2011-01-10 Eric Seidel <eric@webkit.org>
2151 Reviewed by Ojan Vafai.
2153 style-queue messages are way too long for big patches
2154 https://bugs.webkit.org/show_bug.cgi?id=52161
2156 We definitely could build much fancier list-to-string-with-limit functions
2157 but this should be sufficient for our needs at the moment.
2159 * Scripts/webkitpy/common/system/executive.py:
2160 * Scripts/webkitpy/common/system/executive_unittest.py:
2162 2011-01-10 Tony Chang <tony@chromium.org>
2164 Unreviewed, rolling out r75398.
2165 http://trac.webkit.org/changeset/75398
2166 https://bugs.webkit.org/show_bug.cgi?id=52008
2168 Some layout tests are crashing on Chromium Win.
2170 * DumpRenderTree/chromium/WebViewHost.cpp:
2171 (WebViewHost::updatePaintRect):
2172 * DumpRenderTree/chromium/WebViewHost.h:
2174 2011-01-10 Adam Roben <aroben@apple.com>
2178 It isn't needed now that r75289 has been rolled out.
2180 * Scripts/run-javascriptcore-tests:
2182 2011-01-10 Maciej Stachowiak <mjs@apple.com>
2184 Reviewed by Anders Carlsson.
2186 WebKitTestRunner uses crazy amounts of CPU, making layout tests slow and flaky
2187 https://bugs.webkit.org/show_bug.cgi?id=52160
2189 * WebKitTestRunner/mac/TestControllerMac.mm:
2190 (WTR::TestController::platformRunUntil): Instead of busy-looping, wait for the timeout.
2191 Anything that can make runUntil() conditions turn true will happen on a run loop cycle
2194 2011-01-10 Tony Chang <tony@chromium.org>
2196 Reviewed by Mihai Parparita.
2198 [chromium] fix for animated gif layout tests
2199 https://bugs.webkit.org/show_bug.cgi?id=52008
2201 * DumpRenderTree/chromium/WebViewHost.cpp: Schedule a paint when we update the paint rect
2202 to trigger gif animations.
2203 (WebViewHostPaintTask::WebViewHostPaintTask):
2204 (WebViewHostPaintTask::runIfValid):
2205 (WebViewHost::updatePaintRect):
2206 * DumpRenderTree/chromium/WebViewHost.h:
2207 (WebViewHost::taskList):
2209 2011-01-10 Alejandro G. Castro <alex@igalia.com>
2211 Reviewed by Xan Lopez.
2213 [GTK] Fix gtk2 compilation for master
2214 https://bugs.webkit.org/show_bug.cgi?id=51885
2216 * DumpRenderTree/gtk/PixelDumpSupportGtk.cpp:
2217 (createBitmapContextFromWebView): Replaced the old
2218 gdk_drawable_get_size with gdk_pixmap_get_size.
2220 2011-01-10 Patrick Gansterer <paroga@webkit.org>
2222 Reviewed by Laszlo Gombos.
2224 [WINCE] Reenable NPAPI after r75203
2225 https://bugs.webkit.org/show_bug.cgi?id=52109
2227 Remove the !isWinCE() check for ENABLE_NETSCAPE_PLUGIN_API and replace
2228 it with !isEfl(), because EFL port does not support NPAPI (yet).
2230 * Scripts/build-webkit:
2232 2011-01-10 Adam Roben <aroben@apple.com>
2234 Skip some asserting JS tests
2236 See <http://webkit.org/b/52156>.
2238 * Scripts/run-javascriptcore-tests:
2240 2011-01-10 Martin Robinson <mrobinson@igalia.com>
2242 Reviewed by Xan Lopez.
2244 [GTK] build-webkit disable Blob support via arguments passed to automake
2245 https://bugs.webkit.org/show_bug.cgi?id=52097
2247 * Scripts/build-webkit: Enable Blob support when building WebKitGTK+ via
2250 2011-01-10 Adam Roben <aroben@apple.com>
2252 Use the same name and description for TestNetscapePlugin on Windows and
2255 Fixes <http://webkit.org/b/52151>
2256 fast/frames/iframe-reparenting-plugins.html fails on Windows
2258 Reviewed by Steve Falkenburg.
2260 * DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin.rc: Copied
2261 the name and description from Mac's Info.plist.
2263 2011-01-10 Siddharth Mathur <siddharth.mathur@nokia.com>
2265 Reviewed by Laszlo Gombos.
2267 [Qt][WK2] Guard gcc-specific syntax
2269 Non-gcc based compliers choke on the "-include"
2270 syntax for preinclude, place guard around it.
2272 * WebKitTestRunner/InjectedBundle/qt/InjectedBundle.pro:
2273 * WebKitTestRunner/qt/WebKitTestRunner.pro:
2275 2011-01-10 Andreas Kling <kling@webkit.org>
2277 Reviewed by Simon Hausmann.
2279 [Qt] [WK2] MiniBrowser does not load pages opened in new windows
2280 https://bugs.webkit.org/show_bug.cgi?id=52111
2282 When getting a createNewPage() callback from QWKPage, create the
2283 new page with the same QWKContext as the page that is passed to
2286 * MiniBrowser/qt/BrowserView.h:
2287 * MiniBrowser/qt/BrowserWindow.cpp:
2289 (BrowserWindow::BrowserWindow):
2290 * MiniBrowser/qt/BrowserWindow.h:
2292 2011-01-10 Andreas Kling <kling@webkit.org>
2294 Reviewed by Kenneth Rohde Christiansen.
2296 [Qt] MiniBrowser: Remove superfluous createNewPage() callback
2298 BrowserView set its own useless createNewPage() callback on QWKPage.
2299 This was immediately overridden by BrowserWindow anyway, so remove
2300 the BrowserView one.
2302 * MiniBrowser/qt/BrowserView.cpp:
2303 (BrowserView::BrowserView):
2305 2011-01-10 John Knottenbelt <jknotten@chromium.org>
2307 Reviewed by Jeremy Orlow.
2309 [Chromium] Remove non-client-based Geolocation code
2310 https://bugs.webkit.org/show_bug.cgi?id=50921
2312 * DumpRenderTree/chromium/LayoutTestController.cpp:
2313 (LayoutTestController::setGeolocationPermission):
2314 (LayoutTestController::setMockGeolocationPosition):
2315 (LayoutTestController::setMockGeolocationError):
2316 * DumpRenderTree/chromium/WebViewHost.cpp:
2317 (WebViewHost::geolocationClientMock):
2318 (WebViewHost::reset):
2319 * DumpRenderTree/chromium/WebViewHost.h:
2321 2011-01-10 Eric Seidel <eric@webkit.org>
2323 Reviewed by Adam Barth.
2325 Commit bot tried to land r? cq+ patch
2326 https://bugs.webkit.org/show_bug.cgi?id=36638
2328 It should now be possible to post a patch with r? and cq+ and the
2329 commit-queue will correctly wait for the r+ before attempting to land the patch.
2330 (This was a feature requested by several users.)
2332 One caveat: We don't actually reject r-, cq+ patches from the queue
2333 (we just ignore them), but I think that's OK for now.
2335 I also noticed that setting r- once a commit-queue node had grabbed
2336 a patch would not properly cancel the landing. That's fixed and
2337 tested in this patch.
2339 * Scripts/webkitpy/tool/bot/commitqueuetask.py:
2340 * Scripts/webkitpy/tool/bot/commitqueuetask_unittest.py:
2341 * Scripts/webkitpy/tool/bot/feeders.py:
2342 * Scripts/webkitpy/tool/bot/feeders_unittest.py:
2344 2011-01-08 Benjamin Poulain <benjamin.poulain@nokia.com>
2346 Reviewed by Kenneth Rohde Christiansen.
2348 [Qt] QtWebKit crashes when using XMLHttpRequest from the unload event
2349 https://bugs.webkit.org/show_bug.cgi?id=51934
2351 When accessing the network from the destructor of QWebPage, the network access manager
2352 was already deleted because the object WebPage was already destructed.
2354 This solve the problem by linking the lifetime of the network access manager to WebPage's QObject.
2355 The object is now destructed in the destructor of QObject.
2357 * QtTestBrowser/webpage.cpp:
2359 (WebPage::setQnamThreaded):
2360 * QtTestBrowser/webpage.h:
2361 (QtNAMThread::QtNAMThread):
2363 2011-01-08 Eric Seidel <eric@webkit.org>
2365 Reviewed by Ojan Vafai.
2367 test_failures.py should not mention TestShell (long live DumpRenderTree!)
2368 https://bugs.webkit.org/show_bug.cgi?id=52067
2370 I ran test-webkitpy, but didn't bother adding a unit test for this string change.
2371 https://bugs.webkit.org/show_bug.cgi?id=51138#c3 is an example of why this is needed.
2373 * Scripts/webkitpy/layout_tests/layout_package/test_failures.py:
2375 2011-01-08 Eric Seidel <eric@webkit.org>
2377 Reviewed by Mihai Parparita.
2379 commit-queue hangs when the ChangeLog diff is bad
2380 https://bugs.webkit.org/show_bug.cgi?id=52072
2382 * Scripts/webkitpy/tool/steps/validatechangelogs.py:
2383 * Scripts/webkitpy/tool/steps/validatechangelogs_unittest.py:
2385 2011-01-08 Adam Barth <abarth@webkit.org>
2387 Teach svn-apply how to re-write WebCore to Source/WebCore.
2389 * Scripts/VCSUtils.pm:
2391 2011-01-07 Adam Barth <abarth@webkit.org>
2393 Rubber-stamped by Eric Seidel.
2395 Move WebCore to Source
2396 https://bugs.webkit.org/show_bug.cgi?id=52050
2398 Update the tools to understand WebCore's new location.
2400 * DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro:
2401 * DumpRenderTree/wscript:
2403 * Scripts/build-webkit:
2404 * Scripts/do-file-rename:
2405 * Scripts/do-webcore-rename:
2406 * Scripts/generate-qt-inspector-resource:
2407 * Scripts/prepare-ChangeLog:
2408 * Scripts/run-bindings-tests:
2409 * Scripts/update-iexploder-cssproperties:
2410 * Scripts/update-sources-list.py:
2411 * Scripts/update-webkit-localizable-strings:
2412 * Scripts/webkitdirs.pm:
2413 * Scripts/webkitpy/common/config/build.py:
2414 * Scripts/webkitpy/common/config/build_unittest.py:
2415 * Scripts/webkitpy/layout_tests/deduplicate_tests_unittest.py:
2416 * Scripts/webkitpy/style/checker.py:
2417 * Scripts/webkitpy/style/checker_unittest.py:
2418 * WebKitTestRunner/DerivedSources.pro:
2419 * WebKitTestRunner/InjectedBundle/qt/InjectedBundle.pro:
2420 * iExploder/iexploder-1.7.2/tools/update_html_tags_from_sources.sh:
2421 * wx/browser/wscript:
2422 * wx/build/settings.py:
2424 2011-01-07 Maciej Stachowiak <mjs@apple.com>
2426 Reviewed by Sam Weinig.
2428 Make WebKitTestRunner rest between loads more robust to avoid test crashes
2429 https://bugs.webkit.org/show_bug.cgi?id=52086
2431 * WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:
2432 (WTR::InjectedBundle::didReceiveMessage):
2433 * WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:
2434 (WTR::LayoutTestController::notifyDone):
2435 * WebKitTestRunner/TestController.cpp:
2436 (WTR::TestController::resetStateToConsistentValues):
2437 (WTR::TestController::run):
2439 2011-01-07 James Robinson <jamesr@chromium.org>
2441 Revert "Implement mozilla's animationTime property"
2442 https://bugs.webkit.org/show_bug.cgi?id=51952
2444 This approach isn't quite right.
2446 * DumpRenderTree/chromium/WebViewHost.cpp:
2447 (WebViewHost::paintInvalidatedRegion):
2449 2011-01-07 Adam Roben <aroben@apple.com>
2453 * BuildSlaveSupport/build.webkit.org-config/master.cfg:
2454 (RunWebKitTests.start): Don't try to concatenate a list and a string.
2456 2011-01-07 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>
2458 Reviewed by Csaba Osztrogonác.
2460 run-qtwebkit-tests should be able to kill a testsuite.
2462 Add new option in the script that can setup a timeout for a test.
2463 If the test execution takes more then specified time then the test
2464 would be terminated.
2466 [Qt] run-qtwebkit-tests needs timeout
2467 https://bugs.webkit.org/show_bug.cgi?id=51894
2469 * BuildSlaveSupport/build.webkit.org-config/master.cfg:
2470 * Scripts/run-qtwebkit-tests:
2472 2011-01-07 Csaba Osztrogonác <ossy@webkit.org>
2474 Unreviewed fix after r75233.
2476 Based on idea from Eric Seidel.
2478 Add remote zip file handling to webkitpy.
2479 https://bugs.webkit.org/show_bug.cgi?id=50901
2481 * Scripts/webkitpy/common/system/fileset.py: Make python 2.5 happy.
2483 2011-01-07 Shane Stephens <shanestephens@google.com>
2485 Reviewed by Eric Seidel.
2487 svn-apply: should support git binary delta diffs
2488 https://bugs.webkit.org/show_bug.cgi?id=38864
2490 * Scripts/VCSUtils.pm:
2491 * Scripts/svn-apply:
2493 2011-01-06 Julie Parent <jparent@chromium.org>
2495 Reviewed by Eric Seidel.
2497 Test Result Server not removing old results.
2498 https://bugs.webkit.org/show_bug.cgi?id=52012
2500 * TestResultServer/model/jsonresults.py:
2501 Fix off-by-one: Truncate if >= num_runs, not just >.
2502 * TestResultServer/model/jsonresults_unittest.py:
2503 Add unit test to test that merging in a new result with the same value
2504 as the last result causes previous result to fall off.
2506 2011-01-06 James Kozianski <koz@chromium.org>
2508 Reviewed by Mihai Parparita.
2510 Add classes to provide a consistent interface to a set of files.
2511 https://bugs.webkit.org/show_bug.cgi?id=50901
2513 These classes allow us to write code that is agnostic to whether a
2514 particular set of files resides in a local directory or in a zip file
2515 on a remote machine.
2517 * Scripts/webkitpy/common/system/directoryfileset.py: Added.
2518 * Scripts/webkitpy/common/system/directoryfileset_unittest.py: Added.
2519 * Scripts/webkitpy/common/system/fileset.py: Added.
2520 * Scripts/webkitpy/common/system/filesystem.py:
2521 * Scripts/webkitpy/common/system/filesystem_mock.py:
2522 * Scripts/webkitpy/common/system/zipfileset.py: Added.
2523 * Scripts/webkitpy/common/system/zipfileset_unittest.py: Added.
2525 2011-01-06 Eric Seidel <eric@webkit.org>
2527 Reviewed by Adam Barth.
2529 NetworkTransaction should log the url it is retrying
2530 https://bugs.webkit.org/show_bug.cgi?id=52043
2532 * Scripts/webkitpy/common/net/networktransaction.py:
2533 * Scripts/webkitpy/common/net/networktransaction_unittest.py:
2535 2011-01-06 Steve Falkenburg <sfalken@apple.com>
2538 Import os before using it.
2540 * BuildSlaveSupport/build.webkit.org-config/master.cfg:
2542 2011-01-06 Eric Seidel <eric@webkit.org>
2544 Reviewed by Adam Barth.
2546 StyleQueue fails to report errors when patches have many files
2547 https://bugs.webkit.org/show_bug.cgi?id=52042
2549 This is one of at least two errors preventing style error reporting at the moment.
2550 When we moved to passing the list of files to check explicitly,
2551 we started to exceed the 500 character TEXT field limit for our AppEngine model object.
2552 We could change the type of the QueueStatus.message type, or we could
2553 just teach statusserver.py how to clamp values (and warn when it does).
2554 I chose the latter path.
2556 * Scripts/webkitpy/common/net/statusserver.py:
2558 2011-01-06 Steve Falkenburg <sfalken@apple.com>
2560 Reviewed by Stephanie Lewis.
2562 Fix syntax error in script.
2564 Windows bots need to archive/unarchive builds to/from configuration-specific directories
2565 https://bugs.webkit.org/show_bug.cgi?id=51996
2567 * BuildSlaveSupport/build.webkit.org-config/master.cfg:
2569 2011-01-06 Laszlo Gombos <laszlo.1.gombos@nokia.com>
2571 Unreviewed, fixing WinCE build after r75196.
2573 WinCE port does not (yet) support NPAPI.
2575 * Scripts/build-webkit:
2577 2011-01-06 Xan Lopez <xlopez@igalia.com>
2579 Reviewed by Darin Adler.
2581 File::Spec->rel2abs in sunspider-compare-results mangles parameters
2582 https://bugs.webkit.org/show_bug.cgi?id=52015
2584 * Scripts/sunspider-compare-results: call rel2abs on our arguments
2585 after the platform has been detected, otherwise the script will
2586 mangle things like '--gtk' into '/a/path/--gtk', breaking the
2589 2011-01-06 Laszlo Gombos <laszlo.1.gombos@nokia.com>
2591 Reviewed by Eric Seidel.
2593 Add ENABLE_NETSCAPE_PLUGIN_API, ENABLE_ORIENTATION_EVENTS and ENABLE_TOUCH_EVENTS flags to build-webkit
2594 https://bugs.webkit.org/show_bug.cgi?id=51346
2596 * Scripts/build-webkit:
2598 2011-01-06 Steve Falkenburg <sfalken@apple.com>
2600 Reviewed by Adam Roben.
2602 Windows bots need to archive/unarchive builds to/from configuration-specific directories
2603 https://bugs.webkit.org/show_bug.cgi?id=51996
2605 After the configuration directory change, we were archiving the incorrect directory for
2606 build product, so we were running tests on the wrong binaries.
2608 This change archives the correct directory, and also unarchives it into a configuration-
2611 * BuildSlaveSupport/build.webkit.org-config/master.cfg: Use a configuration-specific directory on Windows.
2612 * BuildSlaveSupport/built-product-archive: Unarchive/archive to/from proper directory.
2614 2011-01-06 Eric Seidel <eric@webkit.org>
2616 Reviewed by Mihai Parparita.
2618 commit-queue mentions "Text diff mismatch" 4 times instead of once per failure
2619 https://bugs.webkit.org/show_bug.cgi?id=52002
2621 See an example of this here:
2622 https://bugs.webkit.org/show_bug.cgi?id=51314#c2
2624 It's due to the fact that we pass around TestFailure() instance
2625 objects and yet expect them to all compare to be the same.
2627 This is bad. We should either have shared instances of these values
2628 or we should just pass around the class names. There are already hacks
2629 in place to make TestFailure objects pass equality tests, I just added
2630 a __hash__ implementation so they can be used in sets as expected.
2632 * Scripts/webkitpy/common/net/layouttestresults_unittest.py:
2633 * Scripts/webkitpy/layout_tests/layout_package/test_failures.py:
2635 2011-01-06 Evan Martin <evan@chromium.org>
2637 Reviewed by Tony Chang.
2639 [gdb] handle inaccessible memory and UChar*
2640 https://bugs.webkit.org/show_bug.cgi?id=52003
2642 To pretty-print a UChar*, we just grope around in the memory
2643 looking for a terminating NUL. We need to handle the exception
2644 that can occur when we poke into an invalid location.
2646 Additionally, the logic for choosing which pretty-printer would
2647 early return before hitting the point where we'd use the UChar*
2648 printer. (Did this ever work? How did it regress?)
2652 2011-01-05 Brian Weinstein <bweinstein@apple.com>
2654 Reviewed by Ada Chan.
2656 WebKit2: Should be able to call into injected bundle to ask if we should allow resource loads
2657 https://bugs.webkit.org/show_bug.cgi?id=51969
2659 Add shouldLoadResourceForFrame to WTR::InjectedBundlePage.
2661 * WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:
2662 (WTR::InjectedBundlePage::InjectedBundlePage):
2663 (WTR::InjectedBundlePage::shouldLoadResourceForFrame): Stub implementation.
2664 * WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:
2666 2011-01-06 James Robinson <jamesr@chromium.org>
2668 Reviewed by Simon Fraser.
2670 Implement mozilla's animationTime property
2671 https://bugs.webkit.org/show_bug.cgi?id=51952
2673 Chromium DRT support for webkitAnimationTime.
2675 * DumpRenderTree/chromium/WebViewHost.cpp:
2676 (WebViewHost::paintInvalidatedRegion):
2678 2011-01-06 Julie Parent <jparent@chromium.org>
2680 Reviewed by Tony Chang.
2682 Test Result Server always truncates number of results to JSON_RESULTS_MAX_BUILDS
2683 https://bugs.webkit.org/show_bug.cgi?id=51217
2685 * TestResultServer/model/jsonresults.py:
2686 Pass num_runs through to _remove_items_over_max_number_of_builds, and use the value.
2687 Adds missing documentation.
2688 * TestResultServer/model/jsonresults_unittest.py:
2689 Add unit test to test that truncation happens at smaller value than JSON_RESULTS_MAX_BUILDS.
2690 Update test_merge to take a number of builds to truncate at.
2692 2011-01-06 Adam Barth <abarth@webkit.org>
2694 Reviewed by Eric Seidel.
2696 The EWS should log outside of the working directory
2697 https://bugs.webkit.org/show_bug.cgi?id=51986
2699 This patch prepare us for using git clean -x -d -f in the EWS wrapper
2700 script, which will actually clean the working copy back to a pristine
2701 state instead of leaving untracked directories and ignored files.
2703 * Scripts/webkitpy/tool/commands/queues.py:
2704 * Scripts/webkitpy/tool/commands/queues_unittest.py:
2706 2011-01-05 Steve Falkenburg <sfalken@apple.com>
2709 Back out attempted change to built-product-archive.
2711 * BuildSlaveSupport/built-product-archive:
2713 2011-01-05 Steve Falkenburg <sfalken@apple.com>
2716 Include configuration name when unpacking a build archive.
2718 Needed, since we split Windows build results into configuration-specific directories
2719 under WebKitOutputDir.
2721 * BuildSlaveSupport/built-product-archive:
2723 2011-01-05 Steve Falkenburg <sfalken@apple.com>
2725 Reviewed by Darin Adler.
2727 Debug and Release builds on Windows clobber each other
2728 https://bugs.webkit.org/show_bug.cgi?id=49185
2730 Changes the structure of WebKitBuild build products directory so we
2731 completely separate each build configuration into independent directories.
2733 Although we previously had per-configuration directories for obj, this change adds
2734 per-configuration directories for bin, lib, obj, and include. Each configuration's
2735 build products are stored within a directory inside of WebKitBuild.
2737 Most use of $(WebKitOutputDir) in the build files has been replaced by $(ConfigurationBuildDir),
2738 defined in common.vsprops to be $(WebKitOutputDir)\$(ConfigurationName).
2740 * DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePluginCommon.vsprops:
2741 * DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePluginPostBuild.cmd:
2742 * DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePluginPreBuild.cmd:
2743 * DumpRenderTree/win/DumpRenderTreeCommon.vsprops:
2744 * DumpRenderTree/win/DumpRenderTreePostBuild.cmd:
2745 * DumpRenderTree/win/DumpRenderTreePreBuild.cmd:
2746 * DumpRenderTree/win/ImageDiffCommon.vsprops:
2747 * DumpRenderTree/win/ImageDiffPostBuild.cmd:
2748 * DumpRenderTree/win/ImageDiffPreBuild.cmd:
2749 * FindSafari/FindSafari.vcproj:
2750 * MiniBrowser/Configurations/MiniBrowserCommon.vsprops:
2751 * MiniBrowser/MiniBrowserPostBuild.cmd:
2752 * MiniBrowser/MiniBrowserPreBuild.cmd:
2753 * Scripts/webkitdirs.pm:
2754 * TestWebKitAPI/Configurations/TestWebKitAPICommon.vsprops:
2755 * TestWebKitAPI/win/TestWebKitAPIGeneratedCommon.vsprops:
2756 * TestWebKitAPI/win/TestWebKitAPIPostBuild.cmd:
2757 * TestWebKitAPI/win/TestWebKitAPIPreBuild.cmd:
2758 * TestWebKitAPI/win/copy-resources.cmd:
2759 * WebKitAPITest/WebKitAPITestCommon.vsprops:
2760 * WebKitAPITest/WebKitAPITestPostBuild.cmd:
2761 * WebKitAPITest/WebKitAPITestPreBuild.cmd:
2762 * WebKitTestRunner/Configurations/InjectedBundleCommon.vsprops:
2763 * WebKitTestRunner/Configurations/WebKitTestRunnerCommon.vsprops:
2764 * WebKitTestRunner/win/InjectedBundle.vcproj:
2765 * WebKitTestRunner/win/InjectedBundleGeneratedCommon.vsprops:
2766 * WebKitTestRunner/win/InjectedBundlePostBuild.cmd:
2767 * WebKitTestRunner/win/InjectedBundlePreBuild.cmd:
2768 * WebKitTestRunner/win/WebKitTestRunnerPostBuild.cmd:
2769 * WebKitTestRunner/win/WebKitTestRunnerPreBuild.cmd:
2770 * WinLauncher/WinLauncherCommon.vsprops:
2771 * WinLauncher/WinLauncherPostBuild.cmd:
2772 * WinLauncher/WinLauncherPreBuild.cmd:
2774 2011-01-05 Jan Erik Hanssen <jhanssen@sencha.com>
2776 Reviewed by Ariya Hidayat.
2778 [Qt] TestNetscapePlugin doesn't link on OS X
2779 https://bugs.webkit.org/show_bug.cgi?id=51948
2781 Only add a dependency on libX11 on X11 systems
2783 * DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro:
2785 2011-01-05 Jeff Miller <jeffm@apple.com>
2787 Reviewed by Adam Roben.
2789 check-webkit-style should allow unnecessary parameter names in WebKit2 APIs because we're matching CF's header style
2790 https://bugs.webkit.org/show_bug.cgi?id=51970
2792 * Scripts/webkitpy/style/checker.py:
2793 Add -readability/naming for WebKit2 C API directories.
2795 2011-01-05 Eric Seidel <eric@webkit.org>
2797 Unreviewed, just fixing one more log error from my previous python patch.
2799 Move LayoutTestResults over to new-run-webkit-tests TestResult architecture
2800 https://bugs.webkit.org/show_bug.cgi?id=51802
2802 webkit-patch failure-reason was logging about parsing errors
2803 due to my misunderstanding of how BeautifulSoup.findAll worked.
2805 Fixed, and added a unit test.
2807 * Scripts/webkitpy/common/net/layouttestresults.py:
2808 * Scripts/webkitpy/common/net/layouttestresults_unittest.py:
2810 2011-01-05 Benjamin Poulain <benjamin.poulain@nokia.com>
2812 Reviewed by Kenneth Rohde Christiansen.
2814 [Qt] [WK2] Add the possibility to open window from the WebProcess in Minibrowser
2815 https://bugs.webkit.org/show_bug.cgi?id=51951
2817 Set a function to QWKPage::setCreateNewPageFunction() in Minibrowser so there is a way
2818 to create new window from WebKit.
2820 * MiniBrowser/qt/BrowserWindow.cpp:
2822 (BrowserWindow::BrowserWindow):
2823 (BrowserWindow::page): Added to avoid the indirect access to QWKPage spread in the class
2824 (BrowserWindow::newWindow):
2825 (BrowserWindow::updateUserAgentList):
2826 (BrowserWindow::showUserAgentDialog):
2827 * MiniBrowser/qt/BrowserWindow.h: Add the global variable backingStoreTypeForNewWindow to
2828 define the type of the backing store used for new windows.
2829 * MiniBrowser/qt/main.cpp:
2832 2011-01-05 Eric Seidel <eric@webkit.org>
2834 Reviewed by Ojan Vafai.
2836 commit-queue should report failure type when reporting flaky tests
2837 https://bugs.webkit.org/show_bug.cgi?id=51900
2839 This patch was a ridiculous amount of plumbing. In the end I like
2840 the NRWT-style TestResult class even less. But now we're passing
2841 a list of TestResult objects from the queue to the FlakyTestReporter.
2842 Thus the FlakyTestReporter can more easily report what type of failure
2845 In the process, I found that I was not alone in finding TestResult
2846 cumbersome to use. A bunch of code was trying to create a TestResult
2847 object with default values. However since the constructor didn't make
2848 this easy, some places were getting it wrong (including the TestResult unit test)!
2849 I've fixed the TestResult constructor to have default values for non-essential
2852 * Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py:
2853 * Scripts/webkitpy/common/net/layouttestresults.py:
2854 * Scripts/webkitpy/common/net/layouttestresults_unittest.py:
2855 * Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:
2856 * Scripts/webkitpy/layout_tests/layout_package/json_results_generator.py:
2857 * Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
2858 * Scripts/webkitpy/layout_tests/layout_package/test_results.py:
2859 * Scripts/webkitpy/layout_tests/layout_package/test_results_unittest.py:
2860 * Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
2861 * Scripts/webkitpy/tool/bot/commitqueuetask.py:
2862 * Scripts/webkitpy/tool/bot/commitqueuetask_unittest.py:
2863 * Scripts/webkitpy/tool/bot/flakytestreporter.py:
2864 * Scripts/webkitpy/tool/bot/flakytestreporter_unittest.py:
2865 * Scripts/webkitpy/tool/commands/queues.py:
2866 * Scripts/webkitpy/tool/commands/queues_unittest.py:
2868 2011-01-05 Eric Seidel <eric@webkit.org>
2872 Move LayoutTestResults over to new-run-webkit-tests TestResult architecture
2873 https://bugs.webkit.org/show_bug.cgi?id=51802
2875 Add another unit test to cover a previously missing import.
2877 * Scripts/webkitpy/common/net/layouttestresults.py:
2878 * Scripts/webkitpy/common/net/layouttestresults_unittest.py:
2880 2011-01-05 Mihai Parparita <mihaip@chromium.org>
2882 Reviewed by David Levin.
2884 Add mihaip@chromium.org as a reviewer
2885 https://bugs.webkit.org/show_bug.cgi?id=51966
2887 Add myself as a reviewer.
2889 * Scripts/webkitpy/common/config/committers.py:
2891 2011-01-05 James Simonsen <simonjam@chromium.org>
2893 Unreviewed. Adding myself to the list of committers.
2895 * Scripts/webkitpy/common/config/committers.py:
2897 2011-01-05 Eric Seidel <eric@webkit.org>
2901 Move LayoutTestResults over to new-run-webkit-tests TestResult architecture
2902 https://bugs.webkit.org/show_bug.cgi?id=51802
2904 Fix two exceptions seen on the bots after my original commit.
2906 * Scripts/webkitpy/common/net/layouttestresults.py:
2907 * Scripts/webkitpy/common/net/layouttestresults_unittest.py:
2909 2011-01-05 Kundu Suchismita <suchi.kundu@nokia.com>
2911 Reviewed by Andreas Kling.
2913 [Qt]Add local storage settings to QtTestBrowser command line arguments
2914 https://bugs.webkit.org/show_bug.cgi?id=51750
2916 * QtTestBrowser/main.cpp:
2917 (LauncherApplication::handleUserOptions):
2918 Local storage settings can be enable form command line arguments for
2921 2011-01-05 Chris Fleizach <cfleizach@apple.com>
2923 Reviewed by Martin Robinson.
2925 GTK: AX: DRT needs to use correct root object method.
2926 https://bugs.webkit.org/show_bug.cgi?id=51911
2928 * DumpRenderTree/gtk/AccessibilityControllerGtk.cpp:
2929 (AccessibilityController::rootElement):
2931 2011-01-05 Gabor Rapcsanyi <rgabor@inf.u-szeged.hu>
2933 Rubber-stamped by Csaba Osztrogonác.
2935 Based on idea from Péter Gál.
2937 * Scripts/webkitpy/tool/bot/sheriff.py: Remove unnecessary apostrophes around revision numbers.
2939 2011-01-05 Gabor Rapcsanyi <rgabor@inf.u-szeged.hu>
2941 Reviewed by Adam Barth.
2943 sheriff-bot should be able to do multi-revision rollouts
2944 https://bugs.webkit.org/show_bug.cgi?id=51176
2946 * Scripts/webkitpy/tool/bot/irc_command.py:
2947 * Scripts/webkitpy/tool/bot/sheriff.py:
2948 * Scripts/webkitpy/tool/bot/sheriffircbot_unittest.py:
2950 2011-01-05 Eric Seidel <eric@webkit.org>
2952 Reviewed by Adam Barth.
2954 queues.webkit.org should have per-bot status pages
2955 https://bugs.webkit.org/show_bug.cgi?id=51906
2957 Just adds a /queue-status/QUEUE_NAME/bots/BOT_ID page to show
2958 statuses from a single queue. Now that we have many bots servicing
2959 some queues (commit-queue, win-ews, etc.) its easy to have the messages
2960 you care about scroll off the end of the 15-message limit on the main page.
2962 Eventually we should probably rename /queue-status to /queue or /queues,
2963 but that's fodder for another patch.
2965 * QueueStatusServer/handlers/queuestatus.py:
2966 * QueueStatusServer/index.yaml:
2967 * QueueStatusServer/main.py:
2968 * QueueStatusServer/templates/includes/singlequeuestatus.html:
2969 - This probably should use a custom filter instead of hard-coding
2970 the URL scheme here, but I couldn't figure out how to easily
2971 create such a filter. Most filters work with "strings" so we can't
2972 pass the status object. We could add a method to the status
2973 object and call that, but that seemed a bit strange too.
2974 * QueueStatusServer/templates/queuestatus.html:
2976 2011-01-05 Carlos Garcia Campos <cgarcia@igalia.com>
2978 Unreviewed. Adding myself to the list of committers.
2980 * Scripts/webkitpy/common/config/committers.py:
2982 2011-01-04 Koan-Sin Tan <koansin.tan@gmail.com>
2984 Reviewed by David Levin.
2986 check-webkit-style treated some macros with parentheses after #elif as function calls
2987 https://bugs.webkit.org/show_bug.cgi?id=51695
2989 Ingore function call space checking in any preprocessor directives
2990 (things starting with #). Change search() to match() because
2991 preprocessor directives are supposed to be in the beginning of lines.
2993 * Scripts/webkitpy/style/checkers/cpp.py:
2994 * Scripts/webkitpy/style/checkers/cpp_unittest.py: make sure there is no false positives for #elif cases
2996 2011-01-04 Søren Gjesse <sgjesse@chromium.org>
2998 Reviewed by Tony Chang.
3000 Change the --multi-load test option to --stress-opt
3001 https://bugs.webkit.org/show_bug.cgi?id=50751
3003 * DumpRenderTree/chromium/DumpRenderTree.cpp:
3006 * DumpRenderTree/chromium/TestShell.cpp:
3007 (TestShell::TestShell):
3008 * DumpRenderTree/chromium/TestShell.h:
3009 (TestShell::stressOpt):
3010 (TestShell::setStressOpt):
3011 (TestShell::stressDeopt):
3012 (TestShell::setStressDeopt):
3013 (TestShell::javaScriptFlags):
3014 (TestShell::setJavaScriptFlags):
3015 * Scripts/webkitpy/layout_tests/port/chromium.py:
3016 * Scripts/webkitpy/layout_tests/run_webkit_tests.py:
3018 2011-01-04 Chris Fleizach <cfleizach@apple.com>
3020 Reviewed by Sam Weinig.
3022 WK2: Support Accessibility
3023 https://bugs.webkit.org/show_bug.cgi?id=51859
3025 Use rootObject() method to get top of accessibility tree.
3027 * DumpRenderTree/mac/AccessibilityControllerMac.mm:
3028 (AccessibilityController::focusedElement):
3029 (AccessibilityController::rootElement):
3031 2011-01-03 Martin Robinson <mrobinson@igalia.com>
3033 Reviewed by Darin Adler.
3035 Remove the last non-GObject usage of PlatformRefPtr and move the code to GRefPtr
3036 https://bugs.webkit.org/show_bug.cgi?id=51846
3038 * DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp: Change usage of PlatformRefPtr
3041 2011-01-04 Zhe Su <suzhe@chromium.org>
3043 Reviewed by Kenneth Russell.
3045 Fix insertText, setMarkedText and unmarkText methods of
3046 TextInputController to call corresponding methods of WebKit::WebView
3047 rather than WebKit::WebFrame. This change matches the behavior of
3049 insertText corresponds to WebView::confirmComposition(text)
3050 setMarkedText corresponds to WebView::setComposition(...)
3051 unmarkText corresponds to WebView::confirmComposition()
3052 https://bugs.webkit.org/show_bug.cgi?id=51693
3054 * DumpRenderTree/chromium/TextInputController.cpp:
3055 (TextInputController::insertText):
3056 (TextInputController::setMarkedText):
3057 (TextInputController::unmarkText):
3059 2011-01-04 Dihan Wickremasuriya <dihan.wickremasuriya@nokia.com>
3061 Reviewed by Laszlo Gombos.
3063 [Qt] [Symbian] Do not remove 0 byte sized files from productDir
3065 On Symbian productDir points to the source directory. Removing all the
3066 empty files from the productDir directory corrupts the svn repository.
3068 * Scripts/build-webkit:
3070 2011-01-03 Yi Shen <yi.4.shen@nokia.com>
3072 Reviewed by Adam Barth.
3074 [Qt] Add SelectAll option to the context menu for the editor
3075 https://bugs.webkit.org/show_bug.cgi?id=50049
3077 Set shortcut for the SelectAll action.
3079 * QtTestBrowser/mainwindow.cpp:
3080 (MainWindow::buildUI):
3082 2011-01-03 David Levin <levin@chromium.org>
3084 Reviewed by Eric Seidel.
3086 check-webkit-style shouldn't consider "value" automatically to be a meaningless name.
3087 https://bugs.webkit.org/show_bug.cgi?id=51842
3089 * Scripts/webkitpy/style/checkers/cpp.py: Remove the check for value.
3090 * Scripts/webkitpy/style/checkers/cpp_unittest.py: Remove the test for value.
3092 2011-01-03 Patrick Gansterer <paroga@webkit.org>
3094 Reviewed by David Kilzer.
3096 [WINCE] Adds a build slave.
3097 https://bugs.webkit.org/show_bug.cgi?id=50523
3099 * BuildSlaveSupport/build.webkit.org-config/config.json: Add an
3100 entry for a release build of the WinCE port.
3101 * BuildSlaveSupport/build.webkit.org-config/master.cfg: Add platform
3102 flag to build the WinCE port.
3104 2011-01-03 Patrick Gansterer <paroga@webkit.org>
3106 Reviewed by David Kilzer.
3108 Add WinCE support to build-webkit
3109 https://bugs.webkit.org/show_bug.cgi?id=51642
3111 * Scripts/build-webkit:
3112 * Scripts/webkitdirs.pm:
3114 2011-01-03 Eric Seidel <eric@webkit.org>
3116 Reviewed by Adam Barth.
3118 Move LayoutTestResults over to new-run-webkit-tests TestResult architecture
3119 https://bugs.webkit.org/show_bug.cgi?id=51802
3121 I'm not the biggest fan of the test_failures or test_results classes,
3122 but it's better to have one shared set of classes, than separate ones
3123 for new vs. old run-webkit-test result handling.
3125 This moves the ORWT results class "LayoutTestResults" over to using
3126 TestResult and TestFailure classes, making it easy for us to
3127 make all our sheriff-bot and other webkitpy code NRWT ready.
3129 This also makes it a trivial patch to generate results.json information
3130 from ORWT results.html files (for flaky test analysis, etc.) as well
3131 as making it a one-liner to report test failure types when the
3132 commit-queue sees flaky tests.
3134 This patch tried not to add new functionality, but only to replace
3135 the guts of LayoutTestResults, while adding unit tests and hoping
3136 not to break anything.
3138 I also moved callers which assumed User.prompt* were static/class methods
3139 to using them as instance methods (since we'll eventually want to make them such).
3141 In the process of re-writing things, I broke the rebaseline command, so I wrote
3142 a unit test to catch my breakage were I do do so again in the future.
3144 * Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py:
3145 * Scripts/webkitpy/common/net/layouttestresults.py:
3146 * Scripts/webkitpy/common/net/layouttestresults_unittest.py:
3147 * Scripts/webkitpy/layout_tests/layout_package/test_failures.py:
3148 * Scripts/webkitpy/layout_tests/layout_package/test_results.py:
3149 * Scripts/webkitpy/tool/commands/queries.py:
3150 * Scripts/webkitpy/tool/commands/rebaseline.py:
3151 * Scripts/webkitpy/tool/commands/rebaseline_unittest.py:
3152 * Scripts/webkitpy/tool/mocktool.py:
3154 2011-01-03 Adam Barth <abarth@webkit.org>
3156 Reviewed by Simon Fraser.
3158 webkit-patch shouldn't waste time cleaning a working directory that's already clean
3159 https://bugs.webkit.org/show_bug.cgi?id=51840
3161 * Scripts/webkitpy/common/checkout/scm.py:
3163 2011-01-03 Adam Barth <abarth@webkit.org>
3165 Reviewed by Simon Fraser.
3167 webkit-patch should only hide update output if --quiet
3168 https://bugs.webkit.org/show_bug.cgi?id=51838
3170 As requested by smfr.
3172 * Scripts/webkitpy/tool/steps/update.py:
3174 2011-01-03 Adam Barth <abarth@webkit.org>
3176 Reviewed by David Levin.
3178 commit-queue shouldn't reject patches twice if two bots process the same patch
3179 https://bugs.webkit.org/show_bug.cgi?id=51805
3181 Before raising an error, we first check whether the patch is still in
3182 the commit-queue. This check is still racy, of course, but the time
3183 window is much smaller.
3185 * Scripts/webkitpy/tool/bot/commitqueuetask.py:
3187 2011-01-03 Sam Weinig <sam@webkit.org>
3189 Reviewed by Anders Carlsson.
3191 Make run-api-tests less chatty.
3192 https://bugs.webkit.org/show_bug.cgi?id=51831
3194 - Make script quiet by default and add --verbose option (replacing --quiet).
3195 - When not verbose, pipe stdout and stderr to devnull.
3197 * Scripts/run-api-tests:
3199 2011-01-03 Pratik Solanki <psolanki@apple.com>
3201 Unreviewed. Adding myself to committers.py.
3203 * Scripts/webkitpy/common/config/committers.py:
3205 2011-01-03 Koan-Sin Tan <koansin.tan@gmail.com>
3207 Reviewed by Xan Lopez.
3209 "Tool/Scripts/run-launcher --gtk" stopped working after 74855
3210 https://bugs.webkit.org/show_bug.cgi?id=51806
3212 Originally, the $libraryName is 'JavaScriptCore'. When building release
3213 one, the $libraryDir was 'WebKitBuild/Release/JavaScriptCore/../.libs/'
3214 which doesn't exist anymore ('WebKitBuild/Release/JavaScriptCore'
3215 is 'WebKitBuild/Release/Source/JavaScriptCore' now), and
3216 'WebKitBuild/Release/Source/JavaScriptCore/../../.libs' looks weird,
3217 so make it 'WebKitBuild/Release/.libs'
3219 * Scripts/webkitdirs.pm:
3221 2011-01-02 Patrick Gansterer <paroga@webkit.org>
3223 Reviewed by Adam Barth.
3225 Make EFL a core builder
3226 https://bugs.webkit.org/show_bug.cgi?id=51804
3228 * Scripts/webkitpy/common/net/buildbot/buildbot.py:
3229 * Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py:
3231 2011-01-02 Csaba Osztrogonác <ossy@webkit.org>
3233 [Qt] Unreviewed fix after r74855 and r74875.
3235 * BuildSlaveSupport/built-product-archive:
3236 - Create directories explicitly.
3237 - Pass shell=True to subprocess.call().
3239 2011-01-02 Csaba Osztrogonác <ossy@webkit.org>
3241 [Qt] Unreviewed fix after r74855.
3243 Move JavaScriptCore to Source
3244 https://bugs.webkit.org/show_bug.cgi?id=51604
3246 * BuildSlaveSupport/built-product-archive: JavaScriptCore -> Source/JavaScriptCore
3248 2011-01-02 Csaba Osztrogonác <ossy@webkit.org>
3250 [Qt] Unreviewed run-javasriptcore-tests fix after r74855.
3252 Move JavaScriptCore to Source
3253 https://bugs.webkit.org/show_bug.cgi?id=51604
3255 * Scripts/webkitdirs.pm:
3257 2011-01-02 Robert Hogan <robert@webkit.org>
3259 Unreviewed, build fix.
3261 [Qt] Fix DRT build after 74855
3263 * DumpRenderTree/qt/DumpRenderTree.pro:
3265 2011-01-01 Adam Barth <abarth@webkit.org>
3267 Remove the assumption from the Qt and Gtk builds that every project is
3268 in the root directory.
3270 * Scripts/build-webkit:
3272 2011-01-01 Adam Barth <abarth@webkit.org>
3274 Reviewed by Eric Seidel.
3276 Teach svn-apply how to apply patches even though JavaScriptCore has moved
3277 https://bugs.webkit.org/show_bug.cgi?id=51796
3279 After this change, svn-apply will magically apply patches to
3280 JavaScriptCore/foo to Source/JavaScriptCore/foo.
3282 * Scripts/VCSUtils.pm:
3284 2011-01-01 Adam Barth <abarth@webkit.org>
3286 Reviewed by Eric Seidel.
3288 Move JavaScriptCore to Source
3289 https://bugs.webkit.org/show_bug.cgi?id=51604
3291 Update references to JavaScriptCore to point to the new location.
3293 * BuildSlaveSupport/build.webkit.org-config/master.cfg:
3294 * DumpRenderTree/qt/DumpRenderTree.pro:
3295 * DumpRenderTree/qt/ImageDiff.pro:
3296 * DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro:
3297 * DumpRenderTree/wscript:
3298 * Scripts/build-jsc:
3299 * Scripts/build-webkit:
3300 * Scripts/do-file-rename:
3301 * Scripts/do-webcore-rename:
3302 * Scripts/run-javascriptcore-tests:
3303 * Scripts/update-javascriptcore-test-results:
3304 * Scripts/webkitdirs.pm:
3305 * Scripts/webkitpy/common/config/build_unittest.py:
3306 * Scripts/webkitpy/style/checker.py:
3307 * Scripts/webkitpy/style/checker_unittest.py:
3308 * Scripts/webkitpy/style/checkers/cpp_unittest.py:
3309 * WebKitTestRunner/InjectedBundle/qt/InjectedBundle.pro:
3310 * WebKitTestRunner/qt/WebKitTestRunner.pro:
3311 * wx/build/settings.py:
3313 2011-01-01 Adam Barth <abarth@webkit.org>
3315 Reviewed by Eric Seidel.
3317 Move Sources to Source
3318 https://bugs.webkit.org/show_bug.cgi?id=51794
3320 Update scripts to point to the new location.
3322 * Scripts/build-webkit:
3323 * Scripts/do-file-rename:
3324 * Scripts/do-webcore-rename:
3325 * Scripts/webkitpy/common/config/build.py:
3326 * Scripts/webkitpy/common/config/build_unittest.py:
3328 2010-12-31 Kent Tamura <tkent@chromium.org>
3330 Reviewed by Eric Seidel.
3332 [DRT/Chromium] Enable mock spell checker on non-OSX
3333 https://bugs.webkit.org/show_bug.cgi?id=51401
3335 * DumpRenderTree/chromium/WebViewHost.cpp:
3336 (WebViewHost::spellCheck):
3338 2010-12-31 Adam Barth <abarth@webkit.org>
3340 Reviewed by Ariya Hidayat.
3342 Remove Tools/Scripts/wkstyle
3343 https://bugs.webkit.org/show_bug.cgi?id=51774
3345 This script appears to not have been touched in a while and seems to
3346 have been replaced by check-webkit-style.
3348 * Scripts/wkstyle: Removed.
3350 2010-12-31 Ilya Tikhonovsky <loislo@chromium.org>
3352 Unreviewed. Coding style fix.
3354 * DumpRenderTree/chromium/TestShell.h:
3356 2010-12-31 Ilya Tikhonovsky <loislo@chromium.org>
3358 Reviewed by Pavel Feldman.
3360 Web Inspector: [Chromium] Inspector's tests are very slow on Win(Dbg).
3362 It was a problem with Chromium DRT. DevToolsAgent object instance was
3363 attached not only to the inspected page but to the DevTools window too.
3364 As result all the inspector files were interpreted as a content of inspected page
3365 and were transfered to the DevTools scripts panel for debugging etc.
3367 https://bugs.webkit.org/show_bug.cgi?id=51735
3369 * DumpRenderTree/chromium/TestShell.cpp:
3370 (TestShell::TestShell):
3371 (TestShell::bindJSObjectsToWindow):
3372 (TestShell::createNewWindow):
3373 * DumpRenderTree/chromium/TestShell.h:
3375 2010-12-31 Adam Barth <abarth@webkit.org>
3377 Rubber-stamped by Eric Seidel.
3379 Move PageLoadTests to PerformanceTests/PageLoad
3380 https://bugs.webkit.org/show_bug.cgi?id=51771
3382 Update references to PageLoadTests to point to the new location.
3384 * Scripts/run-pageloadtest:
3385 * Scripts/webkitpy/common/config/build.py:
3387 2010-12-31 Adam Barth <abarth@webkit.org>
3389 Rubber-stamped by Eric Seidel.
3391 Move SunSpider into PerformanceTests
3392 https://bugs.webkit.org/show_bug.cgi?id=51769
3394 Update these scripts to point to the new location.
3396 * Scripts/run-sunspider:
3397 * Scripts/sunspider-compare-results:
3399 2010-12-30 Mihai Parparita <mihaip@chromium.org>
3401 Reviewed by Kent Tamura.
3403 [Chromium] Add WebThemeEngineDRTMac so that Chromium DRT scrollbar rendering can match the Mac port's
3404 https://bugs.webkit.org/show_bug.cgi?id=51728
3406 Add implementation of the Mac WebThemeEngine that uses an NSScroller
3407 to render top-level scrollbars. This makes them match the Mac port's
3408 use of an NSScrollView, which means that we'll be able to share more
3411 The new rendering code will not be activated until the
3412 USE_WEB_THEME_ENGINE_TO_PAINT_THUMB #define is fliped in
3413 ScrollbarThemeChromiumMac.
3415 * DumpRenderTree/DumpRenderTree.gypi:
3416 * DumpRenderTree/chromium/TestShellMac.mm:
3418 * DumpRenderTree/chromium/WebThemeEngineDRTMac.h: Added.
3419 * DumpRenderTree/chromium/WebThemeEngineDRTMac.mm: Added.
3420 (+[FakeActiveWindow alwaysActiveWindow]):
3421 (+[FakeActiveWindow alwaysInactiveWindow]):
3422 (-[FakeActiveWindow initWithActiveControls:]):
3423 (-[FakeActiveWindow _hasActiveControls]):
3424 (WebThemeEngineDRTMac::paintScrollbarThumb):
3425 (stateToHIEnableState):
3426 (WebThemeEngineDRTMac::paintHIThemeScrollbarThumb):
3427 (WebThemeEngineDRTMac::paintNSScrollerScrollbarThumb):
3429 2010-12-30 Konstantin Tokarev <annulen@yandex.ru>
3431 Reviewed by David Kilzer.
3433 [Qt] Don't build wtf/TCSystemAlloc.cpp if --system-malloc option is
3435 https://bugs.webkit.org/show_bug.cgi?id=51672
3437 * DumpRenderTree/qt/DumpRenderTree.pro: Replaced USE_SYSTEM_MALLOC
3438 with USE_SYSTEM_MALLOC=1
3439 * WebKitTestRunner/qt/WebKitTestRunner.pro: Replaced USE_SYSTEM_MALLOC
3440 with USE_SYSTEM_MALLOC=1
3442 2010-12-30 Laszlo Gombos <laszlo.1.gombos@nokia.com>
3444 Reviewed by David Kilzer.
3446 [Qt] [Symbian] Fix build-webkit script for Symbian
3447 https://bugs.webkit.org/show_bug.cgi?id=51509
3449 Set the OUTPUT_DIR for Symbian to be the same as the source
3452 * Scripts/webkitdirs.pm:
3454 2010-12-30 Martin Robinson <mrobinson@igalia.com>
3456 Reviewed by Darin Adler.
3458 check-webkit-style should ignore NULL usage in calls to gtk_widget_style_get
3459 https://bugs.webkit.org/show_bug.cgi?id=51758
3461 Add a check-webkit-style exception for gtk_widget_style_get and NULL usage.
3463 * Scripts/webkitpy/style/checkers/cpp.py: Add the exception.
3464 * Scripts/webkitpy/style/checkers/cpp_unittest.py: Add some tests for this.
3466 2010-12-29 Patrick Gansterer <paroga@webkit.org>
3468 Unreviewed WinCE buildfix.
3470 * WinCELauncher/main.cpp: Add missing include.
3472 2010-12-29 Daniel Bates <dbates@rim.com>
3474 Reviewed by Darin Adler.
3476 svn-apply updates date of wrong change log entry for a change log diff that
3477 contains two consecutive entries with the same author and date
3478 https://bugs.webkit.org/show_bug.cgi?id=46061
3480 Fixes an issue where the date of the wrong change log entry may be modified.
3481 Moreover, changes fixChangeLogPatch() to move entries inserted earlier
3482 in a ChangeLog file to the top of the file.
3484 Currently, fixChangeLogPatch() explicitly bails out and returns an unchanged
3485 diff when it detects that the diff inserts a change log entry earlier in a
3486 ChangeLog. It is unusual to land a patch that has such a deliberate ChangeLog
3487 change. With the advent of the commit-queue this functionality of bailing out
3488 and hence landing the patch as-is is harmful to the accuracy of the ChangeLog.
3489 Instead, we should always move the change log entry to the top of the ChangeLog file.
3491 A side-effect of this change is that setChangeLogDateAndReviewer() now updates
3492 the date line of the correct change log entry in a ChangeLog diff.
3494 * Scripts/VCSUtils.pm: Modified fixChangeLogPatch() to move entries inserted earlier to the top.
3496 * Scripts/webkitperl/VCSUtils_unittest/fixChangeLogPatch.pl:
3497 Updated the following unit tests now that we move entries inserted earlier to the top:
3498 - "fixChangeLogPatch: New entry inserted in middle."
3499 (formerly named "fixChangeLogPatch: [no change] New entry inserted in middle.")
3501 - "fixChangeLogPatch: New entry inserted earlier in the file, but after an entry with the same author and date."
3502 (formerly named "fixChangeLogPatch: [no change] New entry inserted earlier in the file, but after an entry with the same author and date.")
3504 * Scripts/webkitperl/VCSUtils_unittest/fixChangeLogPatchThenSetChangeLogDateAndReviewer.pl: Added.
3506 2010-12-29 Konstantin Tokarev <annulen@yandex.ru>
3508 Reviewed by Eric Seidel.
3510 [Qt] Fixed compatibility with gold linker on X11 platforms
3511 https://bugs.webkit.org/show_bug.cgi?id=51700
3513 * DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro:
3514 Linked TestNetscapePlugin with libX11 on Unix platforms
3516 2010-12-29 Ademar de Souza Reis Jr <ademar.reis@openbossa.org>
3518 Reviewed by Darin Adler.
3520 Circular dependency in webkitpy.common.checkout.changelog module
3521 https://bugs.webkit.org/show_bug.cgi?id=50475
3523 Remove automatic import of api.Checkout module when any checkout/
3524 submodule is imported (e.g.: when checkout.scm.Git is imported).
3526 * Scripts/webkitpy/common/checkout/__init__.py:
3528 2010-12-29 Martin Robinson <mrobinson@igalia.com>
3530 Reviewed by Ojan Vafai.
3532 prepare-ChangeLog should support the -g option for specifying the git-commit
3533 https://bugs.webkit.org/show_bug.cgi?id=51708
3535 Add a -g shortcut for the --git-commit prepare-ChangeLog option. This matches
3536 webkit-patch, though the exact meaning of the two options remain different.
3538 * Scripts/prepare-ChangeLog: Add -g.
3540 2010-12-29 Kent Hansen <kent.hansen@nokia.com>
3542 Reviewed by Simon Hausmann.
3544 [Qt] Fix compilation with Qt in namespace
3545 https://bugs.webkit.org/show_bug.cgi?id=51701
3547 * QtTestBrowser/webview.h:
3549 2010-12-28 Daniel Bates <dbates@rim.com>
3551 Reviewed by Sam Weinig.
3553 Substitute // MARK: for compiler-specific #pragma mark
3554 https://bugs.webkit.org/show_bug.cgi?id=51657
3556 Fix future compilation warnings about "#pragma mark" on GTK+ bots by
3557 substituting "// MARK:" for "#pragma mark", which provides
3558 analogous code-bookmarking functionality under Xcode.
3560 Also, for consistency, we should substitute "// MARK:" for compiler-
3561 specific "#pragma mark" in the source files for the Mac port.
3563 * DumpRenderTree/AccessibilityTextMarker.cpp:
3564 * DumpRenderTree/cf/WebArchiveDumpSupport.h:
3565 * DumpRenderTree/mac/AccessibilityTextMarkerMac.mm:
3566 * DumpRenderTree/mac/ObjCController.m:
3567 * MiniBrowser/mac/AppDelegate.m:
3568 * MiniBrowser/mac/BrowserWindowController.m:
3570 2010-12-28 Xan Lopez <xlopez@igalia.com>
3572 Reviewed by Eric Seidel.
3574 Add --no-build option to run-sunspider
3575 https://bugs.webkit.org/show_bug.cgi?id=51658
3577 * Scripts/run-sunspider: add --no-build flag to skip building
3580 2010-12-27 Daniel Bates <dbates@rim.com>
3582 Rubber-stamped by Martin Robinson.
3584 Append the directory Tools/DumpRenderTree to the list of directories that
3585 do-webcore-rename should search so that it can rename DRT files and
3588 * Scripts/do-webcore-rename:
3590 2010-12-27 Eric Seidel <eric@webkit.org>
3592 Reviewed by Adam Barth.
3594 build-webkit should detect missing Java SDK and explain how to install it
3595 https://bugs.webkit.org/show_bug.cgi?id=51651
3597 * Scripts/build-webkit:
3599 2010-12-26 Martin Robinson <mrobinson@igalia.com>
3601 Reviewed by Daniel Bates.
3603 [GTK] Add support for build-jsc
3604 https://bugs.webkit.org/show_bug.cgi?id=51625
3606 Add GTK+ support to build-jsc now that building the jsc executable as
3607 a standalone target is easy.
3609 * Scripts/build-jsc: Call buildGtkProject for the GTK+ case now.
3610 * Scripts/build-webkit: Pass in the proper target name to buildGtkProject.
3611 * Scripts/webkitdirs.pm: Actually process the project parameter in buildGtkProject
3612 and pass it along to buildAutotoolsProject.
3614 2010-12-27 David Levin <levin@chromium.org>
3616 Reviewed by Eric Seidel.
3618 check-webkit-style check for meaningless variable names in function declarations.
3619 https://bugs.webkit.org/show_bug.cgi?id=51523
3621 * Scripts/webkitpy/style/checker.py: Exempted JavaScriptCore/jit/JITStubs.cpp
3622 from the new check and whitespace/parens because the syntax is unusual and
3623 produced a fair number of positives for these checks.
3624 * Scripts/webkitpy/style/checkers/cpp.py:
3625 (_convert_to_lower_with_underscores): Used as a canonical form for type names
3626 and parameter names when determining if the parameter name is useless.
3627 (_create_acronym): Used to check for redundant variable names in cases like "ExceptionCode ec"
3628 (Parameter.lower_with_underscores_name): Gives back the parameter name in a lower_with_underscore
3630 (_check_parameter_name_against_text): Checks to see if the parameter name is in the
3631 text or an acronym of it.
3632 (check_function_definition): Checks function definitions for meaningless variable names.
3633 (process_line): Added call to check_function_definition.
3634 * Scripts/webkitpy/style/checkers/cpp_unittest.py:
3635 (CppFunctionsTest.test_convert_to_lower_with_underscores): Test for _convert_to_lower_with_underscores.
3636 (CppFunctionsTest.test_create_acronym): Test for _create_acronym.
3637 (CppFunctionsTest.test_check_parameter_against_text): Test for _check_parameter_against_text.
3638 (CppStyleTestBase.perform_single_line_lint): Removed the parameter name check
3639 because when only checking a snippet, there are a lot of bogus functions.
3640 (CppStyleTestBase.perform_multi_line_lint): Ditto and removed a bogus filter
3641 that I put there previously and just noticed.
3642 (WebKitStyleTest.test_parameter_names): Tests for the functionality -- both
3643 check_function_definition and process_line.
3645 2010-12-27 Carlos Garcia Campos <cgarcia@igalia.com>
3647 Reviewed by Martin Robinson.
3649 [GTK] Disable flash plugin in GtkLauncher when using gtk3
3650 https://bugs.webkit.org/show_bug.cgi?id=51591
3652 Flash plugin uses gtk2 that is incompatible with gtk3 making
3653 webkit crash when it's built with gtk3.
3655 * GtkLauncher/main.c:
3658 2010-12-25 Adam Barth <abarth@webkit.org>
3660 Reviewed by Eric Seidel.
3662 Move ANGLE to Sources/ThirdParty
3663 https://bugs.webkit.org/show_bug.cgi?id=51605
3665 * Scripts/build-webkit:
3666 - Point to the new ANGLE location.
3668 2010-12-26 David Levin <levin@chromium.org>
3670 Reviewed by Eric Seidel.
3672 check-webkit-style should be able to parse function declaration parameters.
3673 https://bugs.webkit.org/show_bug.cgi?id=51451
3675 * Scripts/webkitpy/style/checkers/cpp.py:
3676 (Position.__init__): Holds simple position information (row, column).
3677 (Parameter.__init__): Holds information about a parameter.
3678 (SingleLineView.__init): Converts multiple lines into a single line for simpler searches.
3679 (SingleLineView.convert_column_to_row): Returns the original row given a column.
3680 (create_skeleton_parameters): Simplifies a parameter list for easier searching.
3681 (find_parameter_name_index): Finds where the parameter name is.
3682 (parameter_list): Generates the list of parameters for a function.
3683 (_FunctionState.begin): Added information to allow determining the parameters
3685 (_FunctionState.get_parameter_list): Returns a tuple of function parameters.
3686 (detect_functions): Improve function detection for operator functions and
3687 determine where the parameters end and pass that to _FunctionState.begin.
3688 * Scripts/webkitpy/style/checkers/cpp_unittest.py:
3689 (CppFunctionsTest.test_parameter): Verifies Parameter functionality.
3690 (CppFunctionsTest.test_single_line_view): Verifies SingleLineView functionality.
3691 (CppFunctionsTest.test_create_skeleton_parameters): Verifies create_skeleton_parameters.
3692 (CppFunctionsTest.test_find_parameter_name_index): Verifies find_parameter_name_index.
3693 (CppFunctionsTest.test_parameter_list): Does some minimal verification for parameter list.
3694 Much more thorough verification is done as part of FunctionDetectionTest.test_parameter_list.
3695 (FunctionDetectionTest.perform_function_detection): Added support for verifying
3696 the parameters found.
3697 (FunctionDetectionTest.test_function_declaration_detection): Added more function
3698 detection test to verify that we catch the operator functions.
3699 (FunctionDetectionTest.test_ignore_macros): Verify that function detection ignores macros.
3700 (FunctionDetectionTest.test_parameter_list): Added tests to verify the parameter parsing.
3701 (CheckForFunctionLengthsTest.test_function_length_check_definition_severity1_for_bad_test_doesnt_break): Removed
3702 because the error. The test is about the bad function name. Fixing the name makes the test exactly like
3703 test_function_length_check_definition_severity1_for_test.