2009-06-26 Eric Seidel <eric@webkit.org>
[WebKit-https.git] / WebKitTools / ChangeLog
1 2009-06-26  Eric Seidel  <eric@webkit.org>
2
3         Reviewed by Tor Arne Vestbø.
4
5         Remove non-sense --update option to land-patches
6         and make land-patches update before every patch application.
7         This makes it slightly less likely that multi-patch landings will fail.
8
9         Also updated git diff command to include staged modifications.
10
11         * Scripts/bugzilla-tool:
12         * Scripts/modules/scm.py:
13
14 2009-06-25  Eric Seidel  <eric@webkit.org>
15
16         Reviewed by Jan Alonzo.
17
18         Fix obsolete_attachment to work when passed a comment.
19         https://bugs.webkit.org/show_bug.cgi?id=26745
20
21         * Scripts/modules/bugzilla.py:
22
23 2009-06-25  Jan Michael Alonzo  <jmalonzo@webkit.org>
24
25         Reviewed by Mark Rowe.
26
27         bugzilla-tool apply-patch throws exception in Linux
28         https://bugs.webkit.org/show_bug.cgi?id=26738
29
30         HEAD is case-sensitive in Linux. Convert uses of head to HEAD in
31         the scm module.
32
33         * Scripts/modules/scm.py:
34
35 2009-06-25  Eric Seidel  <eric@webkit.org>
36
37         Reviewed by Jan Alonzo.
38
39         bugzilla-tool post-diff needs to obsolete old patches before posting
40         https://bugs.webkit.org/show_bug.cgi?id=26740
41
42         I've also added a --no-obsolete to disable this behavior.
43         
44         I also finally updated the bug page parsing to use XML. So much less code!
45
46         * Scripts/bugzilla-tool:
47         * Scripts/modules/bugzilla.py:
48
49 2009-06-25  Eric Seidel  <eric@webkit.org>
50
51         Reviewed by Mark Rowe.
52
53         Call WebKitTools/Scripts scripts by their absolute paths
54         https://bugs.webkit.org/show_bug.cgi?id=26704
55
56         bugzilla-tool used to use whatever build-webkit was in your path.
57         That could end up building the wrong copy of WebKit.
58
59         * Scripts/bugzilla-tool:
60         * Scripts/modules/scm.py:
61
62 2009-06-25  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
63
64         Reviewed by Dave Levin.
65
66         Use unicode() instead of str when reading from bugzilla.
67         Also add Simon Fraser as a reviewer.
68         https://bugs.webkit.org/show_bug.cgi?id=26719
69
70         * Scripts/modules/bugzilla.py:
71
72 2009-06-25  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
73
74         Reviewed by Simon Hausmann.
75
76         Use Q_ASSERT in Qt's DumpRenderTree instead of JSC's ASSERT
77         
78         The WTFReportAssertionFailure function in JSC is not exported when
79         building QtWebKit in both debug and release on Mac, so DRT fails to
80         link.
81         
82         We can revert this patch once the Qt port builds JSC as a separate
83         library, and we add the proper export macros to Assertions.cpp
84
85         * DumpRenderTree/qt/WorkQueue.cpp:
86         (WorkQueue::queue):
87         (WorkQueue::dequeue):
88
89 2009-06-25  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
90
91         Reviewed by Jan Alonzo.
92
93         bugzilla-tool: Add Simon Hausmann as reviewer
94
95         * Scripts/modules/bugzilla.py:
96
97 2009-06-24  Eric Seidel  <eric@webkit.org>
98
99         Reviewed by Dave Levin.
100
101         Support local commits during apply-patches
102         and let land-patches take multiple bug ids.
103         https://bugs.webkit.org/show_bug.cgi?id=26703
104
105         I also restructured parts of land-patches into
106         class methods and static methods in preparation
107         for future code sharing with other commands.
108
109         * Scripts/bugzilla-tool:
110         * Scripts/modules/bugzilla.py:
111         * Scripts/modules/scm.py:
112
113 2009-06-25  Eric Seidel  <eric@webkit.org>
114
115         Reviewed by Tor Arne Vestbø.
116
117         Make svn-apply work with Git too
118         https://bugs.webkit.org/show_bug.cgi?id=26299
119         
120         Add an --force option to svn-apply and otherwise make svn-apply
121         exit non-zero when patch application fails.
122         https://bugs.webkit.org/show_bug.cgi?id=26300
123
124         I did not update svn-unapply, because it makes no sense in a Git world.
125         You don't roll in and out patch files.  You make commits and deal with those.
126         Git users can just git reset --hard to get the same functionality.
127
128         * Scripts/svn-apply:
129
130 2009-06-25  Eric Seidel  <eric@webkit.org>
131
132         Reviewed by Tor Arne Vestbø.
133
134         Remove use of os.system to fix reviewers with unicode chars in their names
135         https://bugs.webkit.org/show_bug.cgi?id=26713
136
137         Also change to latin1 encoding of ø to make python happy.
138
139         * Scripts/modules/bugzilla.py:
140         * Scripts/modules/scm.py:
141
142 2009-06-25  Eric Seidel  <eric@webkit.org>
143
144         Reviewed by Tor Arne Vestbø.
145
146         Add Adam Roben and Tor Arne Vestbø to the reviewers list.
147
148         * Scripts/modules/bugzilla.py:
149
150 2009-06-24  Chris Fleizach  <cfleizach@apple.com>
151
152         Windows build fix.
153
154         * DumpRenderTree/win/AccessibilityUIElementWin.cpp:
155         (AccessibilityUIElement::language):
156
157 2009-06-24  Chris Fleizach  <cfleizach@apple.com>
158
159         Reviewed by Oliver Hunt.
160
161         Bug 26668: AX: need a way to retrieve the language for an element
162         Support ability to retrieve AXLanguage for testing
163
164         * DumpRenderTree/AccessibilityUIElement.cpp:
165         (getLanguageCallback):
166         (AccessibilityUIElement::getJSClass):
167         * DumpRenderTree/AccessibilityUIElement.h:
168         * DumpRenderTree/mac/AccessibilityUIElementMac.mm:
169         (AccessibilityUIElement::language):
170
171 2009-06-24  Eric Seidel  <eric@webkit.org>
172
173         Reviewed by Maciej Stachowiak.
174         
175         Spell Maciej's name right and add Brady Eidson.
176
177         * Scripts/modules/bugzilla.py:
178
179 2009-06-23  Eric Seidel  <eric@webkit.org>
180
181         Reviewed by Dave Levin.
182
183         Fix missing comment message during land-patches and SVN.commit_with_message respecting --dry-run
184         https://bugs.webkit.org/show_bug.cgi?id=26669
185
186         * Scripts/bugzilla-tool:
187         * Scripts/modules/scm.py:
188
189 2009-06-23  Takeshi Yoshino  <tyoshino@google.com>
190
191         Reviewed by Mark Rowe.
192
193         Bug 26537: Builds from command-line fail if custom build product directory is set and ~/Library/Preferences/xcodebuild.plist exists
194         https://bugs.webkit.org/show_bug.cgi?id=26537
195
196         Let determineBaseProductDir subroutine remove
197         ~/Library/Preferences/xcodebuild.plist. It can prevent xcodebuild from
198         respecting global settings such as a custom build products directory
199         (<rdar://problem/5585899>).
200
201         * Scripts/webkitdirs.pm:
202
203 2009-06-23  Eric Seidel  <eric@webkit.org>
204
205         Reviewed by Dave Levin.
206
207         Make SCM.run_command smarter, and make all previous
208         os.system and subprocess.popen use SCM.run_command instead.
209         https://bugs.webkit.org/show_bug.cgi?id=26666
210
211         This makes it easier to handle errors in a standard way throughout all the code.
212         Since this new code raises by default when the exit_code != 0,
213         we should prevent future problems of bugzilla-tool continuing after
214         a git or svn command failed.
215
216         * Scripts/modules/scm.py:
217
218 2009-06-23  Joe Mason  <joe.mason@torchmobile.com>
219
220         Reviewed by Adam Treat.
221
222         https://bugs.webkit.org/show_bug.cgi?id=26664
223         * Scripts/prepare-ChangeLog: Added --git-index mode to list only the
224         changes which are already staged in the index.  Useful to create an
225         entry for what you're about to commit while ignoring unstaged changes.
226
227 2009-06-23  Anders Carlsson  <andersca@apple.com>
228
229         Reviewed by Darin Adler.
230         
231         Update for WebKit changes.
232         
233         * DumpRenderTree/win/UIDelegate.h:
234         (UIDelegate::willPerformDragSourceAction):
235
236 2009-06-23  Eric Seidel  <eric@webkit.org>
237
238         Reviewed by Mark Rowe.
239
240         * Scripts/modules/scm.py: Fix commit_with_message to return the commit output.
241
242 2009-06-23  Eric Seidel  <eric@webkit.org>
243
244         Reviewed by Mark Rowe.
245
246         * Scripts/modules/scm.py: Fix error seen when commiting r44979.
247
248 2009-06-18  Eric Seidel  <eric@webkit.org>
249
250         Reviewed by Dave Levin.
251
252         WebKit needs a script to interact with bugzilla and automate
253         parts of the patch posting and commit processes.
254         https://bugs.webkit.org/show_bug.cgi?id=26283
255
256         This is really a first-draft tool.
257         It's to the point where it's useful to more people than just me now though.
258         Git support works.  SVN support is written, but mostly untested.
259
260         This tool requires BeautifulSoup and mechanize python modules to run:
261         sudo easy_install BeautifulSoup
262         sudo easy_install mechanize
263
264         More important than the tool itself are the Bugzilla, Git and SVN class abstractions
265         which I hope will allow easy writing of future tools.
266
267         The tool currently implements 10 commands, described below.
268
269         Helpers for scripting dealing with the commit queue:
270         bugs-to-commit                 Bugs in the commit queue
271         patches-to-commit              Patches attached to bugs in the commit queue
272
273         Dealing with bugzilla:
274         reviewed-patches BUGID         r+'d patches on a bug
275         apply-patches BUGID            Applies all patches on a bug to the local working directory without committing.
276         land-and-update BUGID          Lands the current working directory diff and updates the bug.
277         land-patches [options] BUGID   Lands all patches on a bug optionally testing them first
278         obsolete-attachments BUGID     Marks all attachments on a bug as obsolete.
279         commit-message                 Prints a commit message suitable for the uncommitted changes.
280
281         These effectively replace git-send-bugzilla:
282         post-diff BUGID                Attaches the current working directory diff to a bug as a patch file.
283         post-commits BUGID COMMITISH   Attaches a range of local commits to a bug as patch files.
284
285         post-diff works for SVN and Git, post-commits only works for SCMs with local-commit support (like Git)
286
287         land-* commands in a Git environment only work with simple patches due to svn-apply bugs:
288         https://bugs.webkit.org/show_bug.cgi?id=26299
289         https://bugs.webkit.org/show_bug.cgi?id=26300
290
291         This script follows python style (similar to how for Obj-C we follow AppKit style)
292         http://www.python.org/doc/essays/styleguide.html
293         The Python community has a strong style culture and the WebKit style guide is silent re: Python.
294
295         I've filed a bug to update the WebKit style guide to mention python:
296         https://bugs.webkit.org/show_bug.cgi?id=26524
297
298         * Scripts/bugzilla-tool: Added.
299
300 2009-06-22  Steve Falkenburg  <sfalken@apple.com>
301
302         Remove errant line of code mistakenly checked in.
303
304         * DumpRenderTree/win/DumpRenderTree.cpp:
305         (main):
306
307 2009-06-22  Steve Falkenburg  <sfalken@apple.com>
308
309         Pass correct value to setShouldPaintNativeControls.
310         
311         Rubber stamped by Mark Rowe.
312
313         * DumpRenderTree/win/DumpRenderTree.cpp:
314         (main):
315
316 2009-06-22  Steve Falkenburg  <sfalken@apple.com>
317
318         Fix last DumpRenderTree change to correctly set preferences flag without crashing.
319         
320         Reviewed by Mark Rowe.
321
322         * DumpRenderTree/win/DumpRenderTree.cpp:
323         (main):
324
325 2009-06-21  Steve Falkenburg  <sfalken@apple.com>
326
327         Set up global native controls flag before creating the first WebView.
328         
329         Reviewed by Darin Adler.
330
331         * DumpRenderTree/win/DumpRenderTree.cpp:
332         (main):
333
334 2009-06-21  Mark Rowe  <mrowe@apple.com>
335
336         Reviewed by Sam Weinig.
337
338         Set a sensible user agent string for the HTTP requests that Sparkle makes (checking for and downloading updates).
339
340         * WebKitLauncher/WebKitNightlyEnablerSparkle.m:
341         (userAgentStringForSparkle):
342         (initializeSparkle):
343
344 2009-06-21  Mark Rowe  <mrowe@apple.com>
345
346         Reviewed by Sam Weinig.
347
348         Add a hook to the WebKit launcher application to allow a link on the nightly build start page to
349         trigger an update via the built-in software update mechanism.
350
351         * WebKitLauncher/WebKitLauncher.xcodeproj/project.pbxproj:
352         * WebKitLauncher/WebKitLauncherURLProtocol.h: Added.
353         * WebKitLauncher/WebKitLauncherURLProtocol.m: Added.
354         (+[WebKitLauncherURLProtocol load]):
355         (+[WebKitLauncherURLProtocol canInitWithRequest:]): Only allow use of the x-webkit-launcher scheme from .webkit.org subdomains.
356         (+[WebKitLauncherURLProtocol canonicalRequestForRequest:]):
357         (-[WebKitLauncherURLProtocol startLoading]):
358         (-[WebKitLauncherURLProtocol stopLoading]):
359         (-[WebKitLauncherURLProtocol handleIsWebKitLauncherAvailableJS]): Return a brief JavaScript snippet that can be used to programatically
360         determine whether the x-webkit-launcher is available and working.
361         (-[WebKitLauncherURLProtocol handleCheckForUpdates]): Trigger a software update on the main thread.
362         (-[WebKitLauncherURLProtocol resourceNotFound]): Fail with a generic "File does not exist" error.
363
364 2009-06-20  Jan Michael Alonzo  <jmalonzo@webkit.org>
365
366         Reviewed by Gustavo Noronha and Xan Lopez.
367
368         [Gtk] Implement DRT XSS auditor support
369         https://bugs.webkit.org/show_bug.cgi?id=26571
370
371         * DumpRenderTree/gtk/DumpRenderTree.cpp:
372         (resetWebViewToConsistentStateBeforeTesting):
373         * DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:
374         (LayoutTestController::setXSSAuditorEnabled):
375
376 2009-06-19  Darin Adler  <darin@apple.com>
377
378         * Scripts/do-webcore-rename: More renaming ideas.
379
380 2009-06-15  Andre Pedralho  <andre.pedralho@openbossa.org>
381
382         Reviewed by Tor Arne Vestbø.
383
384         Only pass --makeargs along if an argument is given.
385
386         * Scripts/build-webkit:
387
388 2009-06-19  Adam Barth  <abarth@webkit.org>
389
390         Unreviewed attempt to fix Windows build.
391
392         * DumpRenderTree/win/DumpRenderTree.cpp:
393         (resetWebViewToConsistentStateBeforeTesting):
394         * DumpRenderTree/win/LayoutTestControllerWin.cpp:
395         (LayoutTestController::setXSSAuditorEnabled):
396
397 2009-06-18  Adam Barth  <abarth@webkit.org>
398
399         Reviewed by Sam Weinig.
400
401         https://bugs.webkit.org/show_bug.cgi?id=26199
402
403         Add support for testing the XSSAuditor.
404
405         * DumpRenderTree/LayoutTestController.cpp:
406         (setXSSAuditorEnabledCallback):
407         (LayoutTestController::staticFunctions):
408         * DumpRenderTree/LayoutTestController.h:
409         * DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:
410         (LayoutTestController::setXSSAuditorEnabled):
411         * DumpRenderTree/mac/DumpRenderTree.mm:
412         (resetWebViewToConsistentStateBeforeTesting):
413         * DumpRenderTree/mac/LayoutTestControllerMac.mm:
414         (LayoutTestController::setXSSAuditorEnabled):
415         * DumpRenderTree/win/DumpRenderTree.cpp:
416         (resetWebViewToConsistentStateBeforeTesting):
417         * DumpRenderTree/win/LayoutTestControllerWin.cpp:
418         (LayoutTestController::setXSSAuditorEnabled):
419         (LayoutTestController::setPopupBlockingEnabled):
420         * DumpRenderTree/wx/LayoutTestControllerWx.cpp:
421         (LayoutTestController::setXSSAuditorEnabled):
422
423 2009-06-18  Darin Adler  <darin@apple.com>
424
425         Rubber stamped by Mark Rowe.
426
427         * DumpRenderTree/mac/DumpRenderTreeWindow.mm:
428         (-[DumpRenderTreeWindow close]): Resolved crashes seen during regression
429         tests. The close method can be called on a window that's already closed
430         so we can't assert here.
431
432 2009-06-17  Steve Falkenburg  <sfalken@apple.com>
433
434         Updated for consolidated WebKit COM interfaces.
435         
436         Reviewed by Adam Roben.
437
438         * DumpRenderTree/win/FrameLoadDelegate.cpp:
439         * DumpRenderTree/win/FrameLoadDelegate.h:
440         * DumpRenderTree/win/UIDelegate.cpp:
441         * DumpRenderTree/win/UIDelegate.h:
442         * WinLauncher/WinLauncher.h:
443
444 2009-06-16  Adam Barth  <abarth@webkit.org>
445
446         Reviewed by Darin Adler.
447
448         https://bugs.webkit.org/show_bug.cgi?id=26437
449
450         Make the commit-log-editor match the ambient line endings in commit
451         messages.
452
453         * Scripts/commit-log-editor:
454
455 2009-06-16  Xan Lopez  <xlopez@igalia.com>
456
457         Reviewed by Gustavo Noronha.
458
459         Update GtkLauncher to recent API changes in the progress property,
460         which now goes from 0.0 to 1.0.
461
462         * GtkLauncher/main.c:
463
464 2009-06-16  Adam Barth  <abarth@webkit.org>
465
466         Reviewed by Darin Adler.
467
468         https://bugs.webkit.org/show_bug.cgi?id=26000
469
470         Teach prepare-ChangeLog to match the line ends that are already present
471         in ChangeLog files.  This helps folks whose use cygwin perl with CR LF
472         line endings on Windows.
473
474         Also, teach prepare-ChangeLog to normalize backslashes in paths.  This
475         helps folks who use Windows SVN prepare correct ChangeLogs.
476
477         * Scripts/prepare-ChangeLog:
478
479 == Rolled over to ChangeLog-2009-06-16 ==