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