WebKit:
[WebKit-https.git] / WebKit / ChangeLog
1 2005-03-09  Richard Williamson   <rjw@apple.com>
2         
3         Fixed <rdar://problem/4032938> Safari: text layout for MS P Gothic font is corrupted
4
5         The AppKit and ATS reports that MS P Gothic is fixed pitch.  It is
6         not!  This is another case of "fixed pitch" being wrong.  I've
7         coalesced all the special cases into our isFontFixedPitch:, and
8         used a dictionary to improve speed.  No performance regression.
9
10         Reviewed by Maciej.
11
12         * WebCoreSupport.subproj/WebTextRenderer.m:
13         (-[WebTextRenderer _computeWidthForSpace]):
14         * WebCoreSupport.subproj/WebTextRendererFactory.m:
15         (-[WebTextRendererFactory clearCaches]):
16         (-[WebTextRendererFactory isFontFixedPitch:]):
17
18 2005-03-09  Darin Adler  <darin@apple.com>
19
20         Reviewed by Maciej.
21
22         <rdar://problem/4040388> REGRESSION (172-173): nonrepro crash in -[NSString(WebNSURLExtras) _web_isUserVisibleURL]
23
24         * Misc.subproj/WebNSURLExtras.m:
25         (-[NSString _web_isUserVisibleURL]): Fixed some pointer expressions that didn't include the index in the expression.
26         (readIDNScriptWhiteListFile): Removed NSLog statements in here since we decided they aren't useful and they will
27         write some messages on Tiger.
28
29 2005-03-09  Darin Adler  <darin@apple.com>
30
31         * DOM.subproj/DOMPrivate.h: Checked in file copied from WebCore.
32
33 2005-03-08  Richard Williamson   <rjw@apple.com>
34
35         Fixed <rdar://problem/4036949> many JPEG images fail to incremental-load due to change in ImageIO JPEG header parsing (to be fixed in WebKit)
36         Fixed <rdar://problem/4042570> Need to check image properties for presence of width/height properties
37
38         ImageIO-55 changed how image properties are created.  They
39         are now created incrementally.  So we need "re-get" the image
40         properties if the properties we care about (width/height) aren't
41         in the property dictionary.
42
43         Reviewed by John.
44
45         * WebCoreSupport.subproj/WebImageData.h:
46         * WebCoreSupport.subproj/WebImageData.m:
47         (-[WebImageData init]):
48         (-[WebImageData fileProperties]):
49         (-[WebImageData propertiesAtIndex:]):
50         (-[WebImageData _isSizeAvailable]):
51         (-[WebImageData incrementalLoadWithBytes:length:complete:callback:]):
52         (-[WebImageData size]):
53
54 2005-03-08  John Sullivan  <sullivan@apple.com>
55
56         A couple of tweaks to the previous patch, from Darin's review.
57
58         * Misc.subproj/WebNSPasteboardExtras.m:
59         (_writableTypesForImageWithoutArchive):
60         remove unnecessary _web prefix
61         (_writableTypesForImageWithArchive):
62         ditto
63         (+[NSPasteboard _web_writableTypesForImageIncludingArchive:]):
64         use mutableCopy rather than initWithArray:, and adjust for name changes
65
66 2005-03-08  John Sullivan  <sullivan@apple.com>
67
68         Reviewed by Ken.
69         
70         - fixed <rdar://problem/4031826> REGRESSION (Mail): standalone images from 
71         Safari can't be pasted into Mail (WebKit part of fix)
72         
73         We were always declaring webarchive-related pasteboard types, even in the standalone
74         image cases where we had no webarchive. Unfortunately, the WebView pasteboard-related
75         API doesn't prevent this kind of thing from happening, because the code that
76         declares the types isn't guaranteed to be anywhere near the code that writes
77         the pasteboard data.
78         
79         After this fix, I discovered that pasting standalone images into Mail still doesn't
80         work right, but the remaining issues seem to be entirely in Mail. I wrote up 4041671
81         to cover these.
82
83         * Misc.subproj/WebNSPasteboardExtras.h:
84         (+[NSPasteboard _web_writableTypesForImageIncludingArchive:]):
85         Added boolean parameter; clients must specify whether or not there's an
86         archive involved, because the array of types is different if there is.
87
88         * Misc.subproj/WebNSPasteboardExtras.m:
89         (_web_writableTypesForImageWithoutArchive):
90         new static function, constructs (once) and returns the array of types
91         for images that don't have archives
92         (_web_writableTypesForImageWithArchive):
93         new static function, constructs (once) and returns the array of types
94         for images that do have archives
95         (+[NSPasteboard _web_writableTypesForImageIncludingArchive:]):
96         added boolean parameter, now calls one of the two new static functions
97         
98         (-[NSPasteboard _web_writeImage:URL:title:archive:types:]):
99         added asserts that we aren't declaring the archive types if we don't have archive data
100         
101         (-[NSPasteboard _web_declareAndWriteDragImage:URL:title:archive:source:]):
102         updated to pass parameter to _web_writableTypesForImageIncludingArchive:
103         
104         * WebView.subproj/WebDefaultContextMenuDelegate.m:
105         (-[WebDefaultUIDelegate copyImageToClipboard:]):
106         updated to pass parameter to _web_writableTypesForImageIncludingArchive:
107
108         * WebView.subproj/WebImageView.m:
109         (-[WebImageView copy:]):
110         updated to pass parameter to _web_writableTypesForImageIncludingArchive:
111
112         * WebView.subproj/WebView.m:
113         (-[WebView pasteboardTypesForElement:]):
114         updated to pass parameter to _web_writableTypesForImageIncludingArchive:
115
116 2005-03-07  Richard Williamson   <rjw@apple.com>
117
118         More bullet proofing for <rdar://problem/4038304> CrashTracer: ....9 crashes at com.apple.WebKit: -[WebTextRenderer initWithFont:usingPrinterFont:] + 840
119
120         Protect against removal of Times and Times New Roman from
121         system.  If these fonts are removed attempt to get system font
122         instead of FATAL_ALWAYS.
123
124         Reviewed by John.
125
126         * WebCoreSupport.subproj/WebTextRenderer.m:
127         (-[WebTextRenderer initWithFont:usingPrinterFont:]):
128
129 === Safari-403 ===
130
131 2005-03-06  Darin Adler  <darin@apple.com>
132
133         - fixed obvious mistake in IDN script code (luckily it hasn't been in a submission yet!)
134
135         * Misc.subproj/WebNSURLExtras.m: (readIDNScriptWhiteListFile): Use "index" to index into the array,
136         not "script", which is the script number, not the 32-bit-word index.
137
138 2005-03-05  Kevin Decker  <kdecker@apple.com>
139
140         Reviewed by Darin.
141
142         Fixed: <rdar://problem/4038529> Infinite progress bar loading webcams and other sites that use multipart/x-mixed-replace
143
144         The previous patch I landed prevented us from loading multipart/x-mixed-replace but did not always update the progress bar accordingly.  
145         This stops websites from having seemingly infinite progress in the browser UI.  
146
147         * WebCoreSupport.subproj/WebSubresourceClient.m:
148         (-[WebSubresourceClient didReceiveResponse:]): If this is "multipart/x-mixed-replace", remove the WebBaseResourceHandleDelegate client from 
149         the datasource's subresource array, otherwise -[WebDataSource isLoading] incorrectly returns YES.   Also it's possible at this point in
150          time we're done loading now (loaded everything else except for the multipart/x-mixed-replace content) so go ahead and check to see if 
151          in fact we're complete. 
152         * WebView.subproj/WebMainResourceClient.m:
153         (-[WebMainResourceClient connection:didReceiveResponse:]): ditto
154
155 2005-03-05  Richard Williamson   <rjw@apple.com>
156
157         Fixed panther build problem.  Shouldn't include
158         changes for 3968753 on panther.
159
160         Reviewed by John.
161
162         * WebView.subproj/WebView.m:
163         (-[WebView _commonInitializationWithFrameName:groupName:]):
164
165 2005-03-05  Darin Adler  <darin@apple.com>
166
167         Reviewed by John.
168
169         - fixed <rdar://problem/4034603> REGRESSION (185-188): RadarWeb can't send enclosures anymore
170
171         * WebView.subproj/WebFormDataStream.m:
172         (closeCurrentStream): Release currentData when closing the stream.
173         (advanceCurrentStream): Set up and retain currentData when the current stream is reading that data, so the
174         data won't be released while in use.
175         (formCreate): Initialize currentData to NULL.
176
177         - fixed <rdar://problem/4037562> Tiger8A402: Help Viewer crashed when viewing help for iChat (infinite recursion in WebView)
178
179         * WebView.subproj/WebView.m: (-[WebView _responderValidateUserInterfaceItem:]):
180         Check for the case where we ourselves are the responder. This avoids an infinite loop.
181         The actual code to perform operations avoids this with a global variable, but this lighter weight
182         solution is sufficient here because validate operations don't call through to the next responder.
183
184 2005-03-04  Richard Williamson   <rjw@apple.com>
185
186         Fixed <rdar://problem/3968753> REGRESSION: Poor performance with differing multiple animated GIFs (was fast in Panther)
187
188         Disable coalesced updates (in CG).  This restores the
189         panther behavior.
190
191         Reviewed by David Harrison.
192
193         * WebView.subproj/WebView.m:
194         (-[WebView _commonInitializationWithFrameName:groupName:]):
195
196 2005-03-04  Adele Amchan  <adele@apple.com>
197
198         Reviewed by Darin.
199
200         * English.lproj/StringsNotToBeLocalized.txt: added new strings "text/x-vcf" and "text/x-csv" to the list
201
202 2005-03-04  Adele Amchan  <adele@apple.com>
203
204         Reviewed by Chris.
205
206         Fix for <rdar://problem/4032982> Sun iPlanet app: not able to import AddressBook CSV format addresses properly
207         Fix for <rdar://problem/4032985> Sun iPlanet app: not able to import vCard format addresses properly
208
209         * WebView.subproj/WebTextView.m:
210         (+[WebTextView unsupportedTextMIMETypes]): added "text/x-csv" and "text/x-vcf" to the list of MIME types that our text view doesn't handle
211
212 2005-03-04  Darin Adler  <darin@apple.com>
213
214         Reviewed by John.
215
216         - fixed <rdar://problem/4036817> REGRESSION: ctrl-y broken when a line + carriage return cut
217
218         * WebView.subproj/WebHTMLView.m:
219         (-[WebHTMLView _deleteRange:killRing:prepend:smartDeleteOK:deletionAction:]): Merged _handleKillRing behavior
220         into this function, since there's now a more-complicated way the startNewKillRingSequence boolean needs to
221         be handled. Set the startNewKillRingSequence boolean after the entire process so changing the selection before
222         and during the editing dosn't clear it. Also change "isTypingAction" parameter to "deletionAction" so we can
223         handle forward delete with this method.
224         (-[WebHTMLView _deleteSelection]): Pass deleteSelectionAction for action rather than NO for isTypingAction,
225         which is the way to say the same thing using the new parameter.
226         (-[WebHTMLView _deleteWithDirection:granularity:killRing:isTypingAction:]): Refactor to use the _deleteRange
227         method above. Also calls _shouldDeleteRange: for the pre-existing selection case; not doing that before was
228         a bug.
229         (-[WebHTMLView deleteToMark:]): Pass deleteSelectionAction for action rather than NO for isTypingAction,
230         which is the way to say the same thing using the new parameter.
231
232 2005-03-04  Darin Adler  <darin@apple.com>
233
234         Reviewed by John.
235
236         - fixed <rdar://problem/4020413> REGRESSION (Mail): can't use fonts with names that start with "#" in Mail (Korean fonts)
237
238         * WebView.subproj/WebHTMLView.m:
239         (-[WebHTMLView _styleFromFontAttributes:]): Quote font name when calling setFontFamily.
240         (-[WebHTMLView _addToStyle:fontA:fontB:]): Ditto.
241
242 2005-03-04  Darin Adler  <darin@apple.com>
243
244         Reviewed by John.
245
246         - fixed <rdar://problem/3965666> IDN spoofing vulnerability caused by Unicode characters that look like ASCII characters
247
248         * Misc.subproj/WebNSURLExtras.m:
249         (readIDNScriptWhiteListFile): Added. Reads file and parses script names.
250         (readIDNScriptWhiteList): Added. Calls readIDNScriptWhiteList on each of the white list locations in succession.
251         (allCharactersInIDNScriptWhiteList): Renamed from containsPossibleLatinLookalikes and changed sense.
252         Now calls readIDNScriptWhiteList first time, and then uses the read-in list to check the scripts.
253         (-[NSString _web_mapHostNameWithRange:encode:makeString:]): Call allCharactersInIDNScriptWhiteList instead of
254         containsPossibleLatinLookalikes.
255         * Resources/IDNScriptWhiteList.txt: Added.
256         * WebKit.pbproj/project.pbxproj: Added IDNScriptWhiteList.txt file.
257
258         * Misc.subproj/WebKitLocalizableStrings.m: Removed. This is simply unused.
259         * English.lproj/StringsNotToBeLocalized.txt: Updated for recent changes.
260
261 2005-03-04  Darin Adler  <darin@apple.com>
262
263         Reviewed by John.
264
265         - fixed <rdar://problem/3937667> REGRESSION (Mail): Zooming a window from titlebar button doesn't paint newly-exposed portions of window
266
267         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView _recursiveDisplayAllDirtyWithLockFocus:visRect:]):
268         Re-set-up the visRect if the bounds changes due to layout.
269
270 === Safari-402 ===
271
272 2005-03-03  Jens Alfke  <jens@apple.com>
273
274         Reviewed by rjw.
275
276         <rdar://problem/3991818> REGRESSION: Images scale while loading
277         The code could crop an image when not all the scanlines were available yet, and it could crop when only a sub-rect of the image was to be drawn; but if it had to do both at once, it got the coordinates wrong. Fixed that.
278
279         * WebCoreSupport.subproj/WebImageData.m:
280         (-[WebImageData incrementalLoadWithBytes:length:complete:callback:]):
281
282 2005-03-01  David Hyatt  <hyatt@apple.com>
283
284         Fix for 3841186, scrollbar shows up disabled when it should not appear at all.  Make sure updateScrollers
285         is never allowed to be re-entrant from any call point by moving the guard inside the function itself.
286         
287         Reviewed by John Sullivan
288
289         * WebView.subproj/WebDynamicScrollBarsView.m:
290         (-[WebDynamicScrollBarsView updateScrollers]):
291         (-[WebDynamicScrollBarsView reflectScrolledClipView:]):
292
293 2005-03-02  Chris Blumenberg  <cblu@apple.com>
294
295         Fixed: <rdar://problem/4029010> Expose method to retrieve drag image for WebView's selection
296
297         Reviewed by sullivan.
298
299         * WebView.subproj/WebHTMLView.m:
300         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]): call _selectionDraggingImage
301         (-[WebHTMLView _selectionDraggingImage]): new SPI for Mail, factored from previous method
302         (-[WebHTMLView _selectionDraggingRect]): new SPI for Mail
303         * WebView.subproj/WebHTMLViewPrivate.h:
304
305 2005-03-02  John Sullivan  <sullivan@apple.com>
306
307         Reviewed by Adele.
308         
309         - fixed <rdar://problem/4023337> Safari stops loading any page (-[NSCFDictionary setObject:forKey:]: 
310         attempt to insert nil key)
311         
312         It is very likely that the exception being hit is caused by the same problem as WebFoundation
313         bug 4018486. This change makes the code robust against this kind of problem regardless.
314
315         * WebView.subproj/WebBaseResourceHandleDelegate.m:
316         (-[WebBaseResourceHandleDelegate saveResource]):
317         Don't call addSubresource if newly-created resource is nil (but do assert on debug builds).
318         Also assert that originalURL and MIMEType are not nil.
319         
320         * WebView.subproj/WebDataSource.m:
321         (-[WebDataSource addSubresource:]):
322         Don't add nil subresource to dictionary, but do assert on debug builds.
323
324 === Safari-401 ===
325
326 >>>>>>> 1.3057
327 2005-03-01  John Sullivan  <sullivan@apple.com>
328
329         Reviewed by Darin.
330         
331         - fixed <rdar://problem/3987482> Format>Style>Italic is not enabled when a 
332         compose window is empty (works in Blot)
333
334         * WebView.subproj/WebHTMLView.m:
335         (-[WebHTMLView becomeFirstResponder]):
336         call _updateFontPanel here so NSFontManager knows the right font for the menu
337         items and the font panel
338
339 2005-03-01  David Harrison  <harrison@apple.com>
340
341         Reviewed by Chris.
342         
343         <rdar://problem/3915560> Mail would like an SPI to enable "smart" text paste/drop
344
345         * WebView.subproj/WebHTMLView.m:
346         (-[WebHTMLView _smartDeleteRangeForProposedRange:]):
347         (-[WebHTMLView _smartInsertForString:replacingRange:beforeString:afterString:]):
348         New.
349
350 2005-02-28  John Sullivan  <sullivan@apple.com>
351
352         Reviewed by Ken.
353         
354         - WebKit part of fix for <rdar://problem/4023490> REGRESSION (125-185): Tabbing through links 
355         on frameset page gets stuck at end (tivofaq.com)
356         
357         This tab-to-links stuff has been in shaky condition ever since AppKit futzed with
358         tabbing behavior in Tiger to add support for including the toolbar in the key loop.
359         I made some changes months ago to compensate for that, but some cases, such as this
360         one, still weren't fixed.
361
362         * WebCoreSupport.subproj/WebBridge.m:
363         (-[WebBridge _nextKeyViewOutsideWebFrameViewsWithValidityCheck:]):
364         new bottleneck method, extracted from nextKeyViewOutsideWebFrameViews; handles
365         nextKeyView or nextValidKeyView depending on parameter.
366         (-[WebBridge nextKeyViewOutsideWebFrameViews]):
367         now calls extracted method
368         (-[WebBridge nextValidKeyViewOutsideWebFrameViews]):
369         new method, calls new bottleneck method
370         
371         * WebView.subproj/WebHTMLView.m:
372         (-[WebHTMLView nextValidKeyView]):
373         when we're stuck at the end of a nextKeyView chain inside a nexted frame, use
374         nextValidKeyViewOutsideWebFrameViews. Make sure we don't end up looking inside
375         the web frame views while doing this.
376
377 2005-02-25  Darin Adler  <darin@apple.com>
378
379         Reviewed by John.
380
381         - fixed <rdar://problem/4025088> window onblur and onfocus don't fire when text field has focus
382
383         * WebView.subproj/WebHTMLView.m:
384         (-[WebHTMLView updateFocusState]): Renamed from updateFocusDisplay. Added call to setWindowHasFocus: method.
385         (-[WebHTMLView viewDidMoveToWindow]): Call method by new name.
386         (-[WebHTMLView windowDidBecomeKey:]): Ditto.
387         (-[WebHTMLView windowDidResignKey:]): Ditto.
388         (-[WebHTMLView becomeFirstResponder]): Ditto.
389         (-[WebHTMLView resignFirstResponder]): Ditto.
390
391 === Safari-400 ===
392
393 2005-02-25  Richard Williamson   <rjw@apple.com>
394
395         Fixed <rdar://problem/4000962> 8A375: Help Viewer displays voiced sound and semi-voiced characters strangely (characters don't seem to be composed)
396
397         Added special case for voiced marks.
398
399         Reviewed by John.
400
401         * WebCoreSupport.subproj/WebTextRenderer.m:
402         (widthForNextCharacter):
403
404 2005-02-25  Darin Adler  <darin@apple.com>
405
406         Reviewed by John.
407
408         - fixed <rdar://problem/4019823> Seed: Control-Y doesn't work
409
410         * WebView.subproj/WebHTMLView.m:
411         (-[WebHTMLView _deleteWithDirection:granularity:killRing:isTypingAction:]): Call _handleKillRing
412         after setting the selection, since it uses the selection to get the text.
413         (-[WebHTMLView _insertText:selectInsertedText:]): Check for empty string to avoid an assertion
414         on the other side of the bridge when you yank the empty string.
415
416 2005-02-24  Richard Williamson   <rjw@apple.com>
417
418         Fixed <rdar://problem/3382926> Bidi neutrals at RTL/LTR boundaries not handled correctly.
419
420         If directionality is specified use that as initial directionality,
421         rather than neutral directionality.     
422
423         Reviewed by Hyatt.
424
425         * WebCoreSupport.subproj/WebTextRenderer.m:
426         (widthForNextCharacter):
427
428 2005-02-24  Adele Amchan  <adele@apple.com>
429
430         Reviewed by Chris.
431        
432         Fix for <rdar://problem/4023393> Safari crashed in khtml::RenderObject::repaintAfterLayoutIfNeeded(QRect const&, QRect const&)
433
434         We were crashing after hitting PageDown when viewing a pdf because 
435         WebKit was calling over to WebCore to scroll overflow areas.
436         Since this only needs to be done if we're dealing with a WebHTMLView, 
437         I added a wrapper function to check the documentView before calling
438         over to WebCore.
439
440         * WebView.subproj/WebFrameView.m:
441         (-[WebFrameView _scrollOverflowInDirection:granularity:]): added wrapper function that checks if documentView is a WebHTMLView 
442         (-[WebFrameView scrollToBeginningOfDocument:]): uses new wrapper function now
443         (-[WebFrameView scrollToEndOfDocument:]): uses new wrapper function now
444         (-[WebFrameView _pageVertically:]): uses new wrapper function now
445         (-[WebFrameView _pageHorizontally:]): uses new wrapper function now
446         (-[WebFrameView _scrollLineVertically:]): uses new wrapper function now
447         (-[WebFrameView _scrollLineHorizontally:]): uses new wrapper function now
448
449 2005-02-24  Richard Williamson   <rjw@apple.com>
450
451         Fixed <rdar://problem/3985889> REGRESSION (125-180): setting <img> src to GIF that already animated does not animate; just shows final frame
452         
453         Reviewed by Hyatt.
454
455         * WebCoreSupport.subproj/WebImageData.h:
456         * WebCoreSupport.subproj/WebImageData.m:
457         (-[WebImageData resetAnimation]):
458         * WebCoreSupport.subproj/WebImageRenderer.m:
459         (-[WebImageRenderer resetAnimation]):
460         (-[WebInternalImage resetAnimation]):
461         (-[WebImageRenderer drawImageInRect:fromRect:compositeOperator:context:]):
462
463 2005-02-24  Kevin Decker  <kdecker@apple.com>
464
465         Reviewed by John.
466
467         Fixed <rdar://problem/3962401> Don't load multipart/x-mixed-replace content to prevent memory leak
468
469         Since we're not going to fix <rdar://problem/3087535> for Tiger, we should not load multipart/x-mixed-replace content.  Pages with such content contain what is essentially an infinite load and therefore may leak.
470
471         * WebView.subproj/WebMainResourceClient.m: 
472         (-[WebMainResourceClient connection:didReceiveResponse:]):  Disabled loading of multipart/x-mixed-replace content until we fully implement server side push. 
473          * WebCoreSupport.subproj/WebSubresourceClient.m: 
474         (-[WebSubresourceClient didReceiveResponse:]): Ditto. Same exact thing for sub resources. 
475
476
477 2005-02-23  John Sullivan  <sullivan@apple.com>
478
479         Reviewed by Ken.
480         
481         - fixed <rdar://problem/4021370> REGRESSION (Tiger): WebKit part of fix for 
482         shift-tab on tivofaq doing the wrong thing
483
484         * WebView.subproj/WebFrameView.m:
485         (-[WebFrameView becomeFirstResponder]):
486         If our previousValidKeyView is nil or self (same as nil modulo AppKit oddness),
487         look out of the box and get the previousValidKeyView of our webview.
488
489 2005-02-23  Darin Adler  <darin@apple.com>
490
491         Reviewed by Hyatt.
492
493         - fixed <rdar://problem/4010196> REGRESSION (125-186+): 8-character timestamps in gmail wrap to 2 lines (width:8ex; font-size:80%)
494
495         * WebCoreSupport.subproj/WebTextRenderer.m: (-[WebTextRenderer xHeight]): Return the maximum
496         of the "x" height and width. Comment in the code explains why in more detail.
497
498 2005-02-22  Richard Williamson   <rjw@apple.com>
499
500         Fixed <rdar://problem/3937203> when a div adds a scrollbar (overflow:auto) we do not get regions
501
502         Compare regions after automatice scroll regions have been
503         added.
504
505         Reviewed by Maciej.
506
507         * WebCoreSupport.subproj/WebBridge.h:
508         * WebCoreSupport.subproj/WebBridge.m:
509         (-[WebBridge dealloc]):
510         (-[WebBridge _compareDashboardRegions:]):
511         (-[WebBridge dashboardRegionsChanged:]):
512
513 2005-02-22  Richard Williamson   <rjw@apple.com>
514
515         Fixed <rdar://problem/4012463> Dashboard widgets don't work with authenticating proxies
516
517         Added new SPI for dashboard that just calls default delegate
518         behavior.
519
520         Reviewed by Maciej.
521
522         * WebView.subproj/WebView.m:
523         (-[WebView handleAuthenticationForResource:challenge:fromDataSource:]):
524         * WebView.subproj/WebViewPrivate.h:
525
526 2005-02-22  Chris Blumenberg  <cblu@apple.com>
527
528         Reviewed by mjs.
529
530         * WebCoreSupport.subproj/WebBridge.m:
531         (-[WebBridge issuePasteAndMatchStyleCommand]): support for new "PasteAndMatchStyle" exec command
532
533 === Safari-188 ===
534
535 2005-02-21  David Harrison  <harrison@apple.com>
536
537         Reviewed by Darin.
538
539         <rdar://problem/3943090> REGRESSION (Mail): Spelling underline incompletely erased following certain steps
540
541         * WebCoreSupport.subproj/WebTextRenderer.m:
542         (-[WebTextRenderer misspellingLineThickness]):
543         (-[WebTextRenderer misspellingLinePatternWidth]):
544         Replaced #defines with these methods, so others can get the same info.
545         
546         (-[WebTextRenderer drawLineForMisspelling:withWidth:]):
547         Keep underline within originally specified bounds.
548         
549
550 2005-02-21  Darin Adler  <darin@apple.com>
551
552         Reviewed by John.
553
554         - fixed <rdar://problem/4016358> don't ever display IDN URLs with characters from "possible Latin look-alike" scripts
555
556         * Misc.subproj/WebNSURLExtras.m:
557         (containsPossibleLatinLookalikes): Added.
558         (-[NSString _web_mapHostNameWithRange:encode:makeString:]): Call containsPossibleLatinLookalikes, and if true,
559         don't decode the host name.
560
561 2005-02-19  Kevin Decker  <kdecker@apple.com>
562
563         Reviewed by Chris.
564
565         Fixed <rdar://problem/4010765> Flash player can be used to arbitrarily open popup windows without user permission
566         
567         Our window.open() policy is to refuse the <script>window.open(...)</script> case and allow the inline the <a href="javascript:window.open('foo')> case.  Clever advertisers at some point realized that by executing their Javascript through the Flash plugin, Safari would always treat their code as the inline case, and thus, they were able to work around our popup blocker. 
568
569         * Plugins.subproj/WebBaseNetscapePluginView.h: Addded currentEventIsUserGesture boolean ivar. 
570         * Plugins.subproj/WebBaseNetscapePluginView.m:
571         (-[WebBaseNetscapePluginView sendEvent:]): If at any point the user clicks or presses a key from within a plugin, set the currentEventIsUserGesture flag to true. This is important to differentiate legitimate window.open() calls originating from plugins;  we still want to allow those. 
572         (-[WebBaseNetscapePluginView initWithFrame:]): In our asynchronous load, pass along currentEventIsUserGesture to the PluginRequest. 
573         (-[WebBaseNetscapePluginView evaluateJavaScriptPluginRequest:]): Inform WebCore if this was a user originated gesture when calling executeScript(). 
574         (-[WebBaseNetscapePluginView loadRequest:inTarget:withNotifyData:sendNotification:]):
575         (-[WebPluginRequest initWithRequest:frameName:notifyData:sendNotification:didStartFromUserGesture:]):
576         (-[WebPluginRequest isCurrentEventUserGesture]): Added. 
577
578 2005-02-18  Chris Blumenberg  <cblu@apple.com>
579
580         Fixed: <rdar://problem/3945271> REGRESSION (Mail): pasted plain text should pick up typing style instead of being unstyled
581
582         Reviewed by kocienda.
583
584         * WebView.subproj/WebDataSource.m:
585         (-[WebDataSource _replaceSelectionWithArchive:selectReplacement:]): pass NO for matchStyle to replaceSelection
586         * WebView.subproj/WebHTMLView.m:
587         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:chosePlainText:]): return new chosePlainText parameter
588         (-[WebHTMLView _pasteWithPasteboard:allowPlainText:]): pass chosePlainText for matchStyle to replaceSelection
589         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): ditto
590
591 2005-02-17  Richard Williamson   <rjw@apple.com>
592
593         Removed code that should not have been checked in from
594         last patch.
595
596         * WebCoreSupport.subproj/WebTextRendererFactory.m:
597         (-[WebTextRendererFactory isFontFixedPitch:]):
598
599 2005-02-17  Richard Williamson   <rjw@apple.com>
600
601         Fixed <rdar://problem/3999467> when Osaka-Mono is specified as fixed width font, Osaka used instead
602
603         Fixed w/o introducing a performance regression.
604
605         Reviewed by Vicki (and earlier by Dave Harrison).
606
607         * WebCoreSupport.subproj/WebTextRenderer.m:
608         (-[WebTextRenderer _computeWidthForSpace]):
609         (widthForNextCharacter):
610         * WebCoreSupport.subproj/WebTextRendererFactory.h:
611         * WebCoreSupport.subproj/WebTextRendererFactory.m:
612         (-[NSFont _web_isFakeFixedPitch]):
613         (-[WebTextRendererFactory isFontFixedPitch:]):
614         (-[WebTextRendererFactory fontWithFamily:traits:size:]):
615
616 2005-02-17  Richard Williamson   <rjw@apple.com>
617
618         Fixed <rdar://problem/3959304> PDF in img tag is not rendered correctly anymore
619
620         We were incorrectly adding image position when flipping coordinates.
621
622         Reviewed by David Harrison.
623
624         * WebCoreSupport.subproj/WebImageData.m:
625         (-[WebImageData _PDFDrawFromRect:toRect:operation:alpha:flipped:context:]):
626
627 2005-02-17  Richard Williamson   <rjw@apple.com>
628
629         Fixed <rdar://problem/4007262> Seed: Flight tracker scrolling moves to the left
630
631         Added SPI for dashboard to disable wheel scrolling of the WebClipView.
632
633         Reviewed by Ken.
634
635         * ChangeLog:
636         * WebView.subproj/WebClipView.m:
637         (-[WebClipView _focusRingVisibleRect]):
638         (-[WebClipView scrollWheel:]):
639         * WebView.subproj/WebView.m:
640         (-[WebViewPrivate init]):
641         (-[WebView drawRect:]):
642         (-[WebView _dashboardBehavior:]):
643         * WebView.subproj/WebViewInternal.h:
644         * WebView.subproj/WebViewPrivate.h:
645
646 === Safari-187 ===
647
648 2005-02-17  Vicki Murley <vicki@apple.com>
649
650         - roll out this change, since it causes a 1.5% performance regression
651
652     2005-02-15  Richard Williamson   <rjw@apple.com>
653         
654         Fixed <rdar://problem/3999467> when Osaka-Mono is specified as fixed width font, Osaka used instead
655         
656         Lie about Osaka-Mono.  Treat it as fixed pitch, even though,
657         strictly speaking it isn't.  (Similar to what we do with
658         Courier New.)
659
660         Reviewed by David Harrison.
661
662         * WebCoreSupport.subproj/WebTextRenderer.m:
663         (-[WebTextRenderer _computeWidthForSpace]):
664         * WebCoreSupport.subproj/WebTextRendererFactory.h:
665         * WebCoreSupport.subproj/WebTextRendererFactory.m:
666         (-[NSFont _web_isFakeFixedPitch]): 
667         (-[WebTextRendererFactory isFontFixedPitch:]):
668         (-[WebTextRendererFactory fontWithFamily:traits:size:]):
669
670 2005-02-17  John Sullivan  <sullivan@apple.com>
671
672         Reviewed by Ken.
673         
674         - fixed <rdar://problem/3997185> The Web view on .Mac Prefs caused System Prefs 
675         to lockup (resolved by re-boot only)
676
677         * WebCoreSupport.subproj/WebBridge.m:
678         (-[WebBridge nextKeyViewOutsideWebFrameViews]):
679         Don't allow recursion here; assert on debug build, return nil on deployment. I
680         couldn't get my machine into a state to repro this problem (and neither could the
681         originator), but it's obvious from the stack crawl that this method was recursing
682         when it shouldn't have.
683
684 2005-02-16  John Sullivan  <sullivan@apple.com>
685
686         Written by Darin, reviewed by me.
687         
688         - WebKit part of fix for <rdar://problem/4007384> 
689         FILTER: Bookmark of RSS with Japanese search word & multiple RSS pages loses filter
690
691         * DOM.subproj/DOMPrivate.h:
692         updated this file, which is a copy of the WebCore version
693
694 2005-02-16  Richard Williamson   <rjw@apple.com>
695
696         Fixed <rdar://problem/3966973> Tiger 8A357: Binary Compatiblity: frequent Webstractor.app crashes [WebImageData _nextFrame]
697
698         Webstractor.app was playing tricks to create thumbnails of
699         pages.  This caused the 'focusView' to be incorrect during
700         animated GIF frame rendering.
701
702         Reviewed by Chris.
703
704         * WebCoreSupport.subproj/WebImageRenderer.m:
705         (-[WebImageRenderer _startOrContinueAnimationIfNecessary]):
706
707 2005-02-16  Vicki Murley  <vicki@apple.com>
708
709         Reviewed by me, code change by Darin.
710
711         - fixed the build on Panther
712
713         * WebView.subproj/WebFormDataStream.m: (webSetHTTPBody):
714         Added a Panther-specific code path that just loads all the data into one big
715         NSData object. This means that bug 3686434 won't be fixed on SUPanWheat; we'll
716         still load the file into memory before sending it to the server on Panther.
717
718 2005-02-15  Richard Williamson   <rjw@apple.com>
719
720         Fixed <rdar://problem/3998368> Tiger8A376: WebTextRenderer assertion failure in Safari while browsing news.bbc.co.uk
721
722         Removed use of FATAL_ALWAYS from getUncachedWidth().  It's unclear
723         why we would trigger the FATAL_ALWAYS.  In the past we've seen
724         the message triggered because of corrupt fonts.  Anyway, in this
725         particular case, we will now return 0 for the character width,
726         rather than exiting.
727
728         Reviewed by David Harrison.
729
730         * WebCoreSupport.subproj/WebTextRenderer.m:
731         (getUncachedWidth):
732
733 2005-02-15  Richard Williamson   <rjw@apple.com>
734
735         Fixed <rdar://problem/3999467> when Osaka-Mono is specified as fixed width font, Osaka used instead
736
737         Lie about Osaka-Mono.  Treat it as fixed pitch, even though,
738         strictly speaking it isn't.  (Similar to what we do with
739         Courier New.)
740
741         Reviewed by David Harrison.
742
743         * WebCoreSupport.subproj/WebTextRenderer.m:
744         (-[WebTextRenderer _computeWidthForSpace]):
745         * WebCoreSupport.subproj/WebTextRendererFactory.h:
746         * WebCoreSupport.subproj/WebTextRendererFactory.m:
747         (-[NSFont _web_isFakeFixedPitch]):
748         (-[WebTextRendererFactory isFontFixedPitch:]):
749         (-[WebTextRendererFactory fontWithFamily:traits:size:]):
750
751 2005-02-14  Darin Adler  <darin@apple.com>
752
753         Reviewed by John.
754
755         - fixed <rdar://problem/3686434> Safari uses too much RAM on file upload, leading to malloc errors and crashes (HP printers)
756
757         * WebView.subproj/WebFormDataStream.h: Added webSetHTTPBody, which creates and connects an appropriate
758         stream to an NSMutableURLRequest.
759         * WebView.subproj/WebFormDataStream.m: Added implementation here.
760
761         * WebCoreSupport.subproj/WebBridge.m:
762         (-[WebBridge syncLoadResourceWithURL:customHeaders:postData:finalURL:responseHeaders:statusCode:]):
763         Use webSetHTTPBody.
764         * WebCoreSupport.subproj/WebSubresourceClient.m:
765         (+[WebSubresourceClient startLoadingResource:withURL:customHeaders:postData:referrer:forDataSource:]):
766         Use webSetHTTPBody.
767         * WebView.subproj/WebFrame.m:
768         (-[WebFrame _loadItem:withLoadType:]): Use webSetHTTPBody.
769         (-[WebFrame _postWithURL:referrer:target:data:contentType:triggeringEvent:form:formValues:]): Ditto.
770
771         * English.lproj/StringsNotToBeLocalized.txt: Updated for this change and other recent changes.
772
773 2005-02-11  Richard Williamson   <rjw@apple.com>
774
775         Fixed <rdar://problem/4002505> 8A378: Endlessly animating gif's on http://www.entropy.ch
776         
777         If animated images had no loop count property we were incorrectly
778         looping forver.  Note, that in the course of fixing this bug
779         I found that ImageIO is incorrectly NOT reporting the loop count
780         for a whole class of animated GIFs.
781
782         Reviewed by Ken Kocienda.
783
784         * WebCoreSupport.subproj/WebImageData.m:
785         (-[WebImageData _repetitionCount]):
786
787 === Safari-186 ===
788
789 2005-02-11  Vicki Murley  <vicki@apple.com>
790
791         Reviewed by Darin.
792
793         - fix deployment build on Panther
794  
795         * WebView.subproj/WebHTMLView.m:
796         (-[WebHTMLView changeBaseWritingDirection:]): ifdef out NSWritingDirectionNatural
797         (-[WebHTMLView toggleBaseWritingDirection:]): fix a spacing issue
798
799 2005-02-10  David Harrison  <harrison@apple.com>
800
801         Reviewed by Richard.
802
803         <rdar://problem/3991652> REGRESSION (Mail): Deleting entire line in reply deletes extra blank line and moves insertion point
804         
805         * WebView.subproj/WebHTMLView.m:
806         (-[WebHTMLView _deleteRange:killRing:prepend:smartDeleteOK:isTypingAction:]):
807         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
808         (-[WebHTMLView _deleteWithDirection:granularity:killRing:isTypingAction:]):
809         (-[WebHTMLView selectToMark:]):
810         (-[WebHTMLView swapWithMark:]):
811         (-[WebHTMLView transpose:]):
812         (-[WebHTMLView _selectMarkedText]):
813         (-[WebHTMLView _selectRangeInMarkedText:]):
814         Adopt new default affinity of NSSelectionAffinityDownstream as of <rdar://problem/3937447>.
815         
816 2005-02-10  Darin Adler  <darin@apple.com>
817
818         Reviewed by Harrison.
819
820         - fixed <rdar://problem/4002084> Setting ResourceLoadDelegate to nil can cause a crash
821
822         * WebView.subproj/WebView.m: (-[WebView _cacheResourceLoadDelegateImplementations]):
823         Set booleans to either YES or NO, rather than setting them only in the YES case.
824
825 2005-02-10  Darin Adler  <darin@apple.com>
826
827         Reviewed by Harrison.
828
829         - fixed <rdar://problem/3991225> Format->Style->Underline menu item does not get checked when selected text is underlined
830
831         * WebView.subproj/WebHTMLView.m:
832         (-[WebHTMLView validateUserInterfaceItem:]): Added tons of additional cases in here for all the
833         "action" style methods in this class that don't always want to be valid. For the ones where state
834         makes sense, added the state-checking code too for the menu item case.
835         (-[WebHTMLView ignoreSpelling:]): Removed unnecessary "editable text only" check since this command
836         would work fine on a non-editable selection.
837         (-[WebHTMLView swapWithMark:]): Ditto.
838         (-[WebHTMLView changeBaseWritingDirection:]): Added. Like toggle, but based on the sender's tag.
839
840 2005-02-08  Darin Adler  <darin@apple.com>
841
842         "Reviewed" by Richard (he told me the file was obsolete).
843
844         - got rid of an obsolete file
845
846         * Plugins.subproj/npsap.h: Removed.
847         * copy-webcore-files-to-webkit: Removed npsap.h.
848
849 2005-02-09  Richard Williamson   <rjw@apple.com>
850
851         Fixed <rdar://problem/4000073> non-screen font error on www.worldofwarcraft.com
852
853         Reviewed by John Sullivan.
854
855         * WebCoreSupport.subproj/WebTextRenderer.m:
856         (-[WebTextRenderer _smallCapsFont]):
857
858 === Safari-185 ===
859
860 2005-02-07  Chris Blumenberg  <cblu@apple.com>
861
862         Fixed: <rdar://problem/3993354> Safari claims to put RTFD on the pasteboard, but doesn't, in some cases
863
864         Reviewed by john.
865
866         * WebView.subproj/WebHTMLView.m:
867         (-[WebHTMLView _stripAttachmentCharactersFromAttributedString:]): moved
868         (-[WebHTMLView _writeSelectionWithPasteboardTypes:toPasteboard:cachedAttributedString:]): take cachedAttributedString parameter in case the attributed string was gotten already
869         (-[WebHTMLView _writeSelectionToPasteboard:]): omit RTFD from the types list when there are no attachments
870
871 2005-02-07  David Harrison  <harrison@apple.com>
872
873         Reviewed by Ken.
874
875         <rdar://problem/3990693> REGRESSION (8A373): ctrl-k now deletes just one character instead of line
876         
877         * WebView.subproj/WebHTMLView.m:
878         (-[WebHTMLView _deleteWithDirection:granularity:killRing:isTypingAction:]):
879         Set the selection so that deleteKeyPressedWithSmartDelete knows what to delete.
880
881 2005-02-06  Darin Adler  <darin@apple.com>
882
883         Reviewed by Maciej.
884
885         - fixed <rdar://problem/3963166> PDFView SPI print method is being deprecated, moving to PDFDocument; please update WebKit
886
887         * WebView.subproj/WebPDFView.m: (-[WebPDFView printOperationWithPrintInfo:]): Target the document instead of the view.
888
889         * English.lproj/StringsNotToBeLocalized.txt: Updated for recent changes and also moved one translation to be a file-specific
890         item rather than a file-independent one.
891
892 2005-02-05  Chris Blumenberg  <cblu@apple.com>
893
894         Fixed: <rdar://problem/3991974> REGRESSION: www.jabra.com world location screen does not work
895
896         Reviewed by hyatt.
897
898         * WebCoreSupport.subproj/WebBridge.m:
899         (-[WebBridge viewForPluginWithURL:attributeNames:attributeValues:MIMEType:]): use the baseURL from the bridge rather than from the response
900
901 2005-02-04  Chris Blumenberg  <cblu@apple.com>
902
903         Fixed: <rdar://problem/3802781> rtf->html pasteboard conversion using xhtml
904
905         Reviewed by kocienda.
906
907         * WebView.subproj/WebHTMLView.m:
908         (+[WebHTMLView _excludedElementsForAttributedStringConversion]): new
909         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): call _excludedElementsForAttributedStringConversion
910
911 2005-02-04  Chris Blumenberg  <cblu@apple.com>
912
913         Fixed: <rdar://problem/3832973> copy text from PowerPoint, paste into Blot (or presumably Mail) and get a single missing image icon
914
915         Reviewed by john.
916
917         * WebView.subproj/WebHTMLView.m:
918         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): prefer RTF and RTFD over images just as NSTextView does
919
920 2005-02-03  Chris Blumenberg  <cblu@apple.com>
921
922         Fixed: <rdar://problem/3555137> REGRESSION (125-173): Flash animation can erase parts of chrome (bookmarks bar & tab bar)
923
924         convertRect:toView: returns incorrect results inside of viewWillMoveToWindow: with a nil window. Workaround this by catching this case.
925
926         Reviewed by darin.
927
928         * Plugins.subproj/WebBaseNetscapePluginView.m:
929         (-[WebBaseNetscapePluginView superviewsHaveSuperviews]): new
930         (-[WebBaseNetscapePluginView saveAndSetPortStateForUpdate:]): clip out the plug-in view when superviewsHaveSuperviews returns NO
931
932 2005-02-03  Chris Blumenberg  <cblu@apple.com>
933
934         Fixed: <rdar://problem/3893513> Sun iPlanet app: when saving to a file it brings up a window with the thing to be saved instead
935
936         Reviewed by adele.
937
938         * WebView.subproj/WebTextView.m:
939         (+[WebTextView unsupportedTextMIMETypes]): added "text/ldif" to the list of MIME types that our text view doesn't handle
940
941 === Safari-183 ===
942
943 2005-02-03  Chris Blumenberg  <cblu@apple.com>
944
945         * English.lproj/StringsNotToBeLocalized.txt: updated
946
947 2005-02-03  Chris Blumenberg  <cblu@apple.com>
948
949         Fixed: <rdar://problem/3989611> Evite style "add vCalendar to calendar" do not work
950
951         Reviewed by adele.
952
953         * WebView.subproj/WebTextView.m:
954         (+[WebTextView unsupportedTextMIMETypes]): added "text/x-vcalendar" to the list of MIME types our text view doesn't handle
955
956 2005-02-03  Vicki Murley  <vicki@apple.com>
957
958         Reviewed by Darin.
959
960         - fix deployment build breakage on Panther
961  
962         * WebView.subproj/WebHTMLView.m:
963         (-[WebHTMLView toggleBaseWritingDirection:]):
964
965 2005-02-02  John Sullivan  <sullivan@apple.com>
966
967         Reviewed by Chris.
968         
969         - WebKit part of fix for <rdar://problem/3980651> REGRESSION (125-180): Huge number of pages printed from certain page, 
970         iFrame involved
971         
972         This also fixes the problems with printing from GMail, yay!
973
974         * WebView.subproj/WebHTMLView.m:
975         (-[WebHTMLView _setPrinting:minimumPageWidth:maximumPageWidth:adjustViewSize:]):
976         Don't adjust margins for header/footer here, because this is called for each subframe.
977         (-[WebHTMLView knowsPageRange:]):
978         Do adjust margins for header/footer here (analogous to WebTextView and WebImageView).
979         Also, round the page height to an integer here (noticed in passing).
980
981 2005-02-02  Chris Blumenberg  <cblu@apple.com>
982
983         Fixed: <rdar://problem/3986546> Cut, delete, and paste menu items are active when a image is opened in window
984
985         Reviewed by john.
986
987         * WebView.subproj/WebView.m:
988         (-[WebView _responderValidateUserInterfaceItem:]): new, has the responder validate the item 
989         (-[WebView validateUserInterfaceItem:]): call VALIDATE for each repsonder selector using FOR_EACH_RESPONDER_SELECTOR macro
990         (-[WebView _performResponderOperation:with:]): call factored out method _responderForResponderOperations
991         (-[WebView _responderForResponderOperations]): new, code from _performResponderOperation:with:
992
993 2005-02-02  Chris Blumenberg  <cblu@apple.com>
994
995         Fixed: <rdar://problem/3983628> control-click on WebView is not selecting the word under the cursor (Mail, non-editable WebView)
996
997         Reviewed by rjw.
998
999         * WebCoreSupport.subproj/WebBridge.m:
1000         (-[WebBridge selectWordBeforeMenuEvent]): new
1001         * WebView.subproj/WebView.m:
1002         (-[WebView _selectWordBeforeMenuEvent]): new SPI
1003         (-[WebView _setSelectWordBeforeMenuEvent:]): new SPI
1004         * WebView.subproj/WebViewInternal.h:
1005         * WebView.subproj/WebViewPrivate.h:
1006
1007 2005-02-02  Chris Blumenberg  <cblu@apple.com>
1008
1009         Fixed: <rdar://problem/3986013> Assertion failure going back after page load error (no apparent problem in nondebug build)
1010
1011         Reviewed by rjw.
1012
1013         * WebView.subproj/WebDataSource.m:
1014         (-[WebDataSource _setPrimaryLoadComplete:]): don't set the data source data when the main client is nil
1015
1016 2005-02-02  Ken Kocienda  <kocienda@apple.com>
1017
1018         Reviewed by Darin
1019
1020         Fix for this bug:
1021         
1022         <rdar://problem/3823109> WebKit should support -toggleBaseWritingDirection: (bidi editing support)
1023
1024         * WebView.subproj/WebHTMLView.m:
1025         (-[WebHTMLView _applyParagraphStyleToSelection:withUndoAction:]): New function that calls through to
1026         new feature that allows callers to force all properties in a style to be applied as block styles.
1027         (-[WebHTMLView _alignSelectionUsingCSSValue:withUndoAction:]): Removed FIXME comment for something that has been fixed.
1028         (-[WebHTMLView toggleBaseWritingDirection:]): Implemented.
1029
1030 2005-02-01  Richard Williamson   <rjw@apple.com>
1031
1032         Fixed <rdar://problem/3977727> WebKit should use new SPI to support faster GIF rendering
1033
1034         Note:  This REQUIRES build >= 3A362 when building on Tiger.
1035
1036         Reviewed by John.
1037
1038         * WebCoreSupport.subproj/WebImageData.m:
1039         (-[WebImageData _imageSourceOptions]):
1040
1041 === Safari-182 ===
1042
1043 2005-02-01  Richard Williamson   <rjw@apple.com>
1044
1045         Added new SPI for <rdar://problem/3967063> need spi on WebView to turn of lcd text for Dashboard
1046
1047         Reviewed by Chris.
1048
1049         * WebCoreSupport.subproj/WebTextRenderer.m:
1050         (_drawGlyphs):
1051         * WebView.subproj/WebView.m:
1052         (-[WebView drawRect:]):
1053         (-[WebView _dashboardBehavior:]):
1054         (+[WebView _setShouldUseFontSmoothing:]):
1055         (+[WebView _shouldUseFontSmoothing]):
1056         * WebView.subproj/WebViewInternal.h:
1057         * WebView.subproj/WebViewPrivate.h:
1058
1059 2005-01-31  Chris Blumenberg  <cblu@apple.com>
1060
1061         Fixed: <rdar://problem/3949806> REGRESSION: Source window fails to refresh correctly on reload
1062
1063         The data method was being called on WebDataSource before the data was set. This fix makes WebDataSource set the data before releasing the main client instead of waiting for the main client to set it when it is dealloced.
1064
1065         Reviewed by darin.
1066
1067         * WebView.subproj/WebDataSource.m:
1068         (-[WebDataSource _setData:]): moved within file
1069         (-[WebDataSource _setPrimaryLoadComplete:]): call _setData: with the data from the main client before letting go of the main client
1070         * WebView.subproj/WebDataSourcePrivate.h:
1071         * WebView.subproj/WebMainResourceClient.m:
1072         (-[WebMainResourceClient releaseResources]): removed call to _setData: since the data source may need the data before releaseResources is called
1073
1074 2005-01-31  Chris Blumenberg  <cblu@apple.com>
1075
1076         Fixed: <rdar://problem/3946285> Seed: Safari crashed by selecting all at internet-moebel.com
1077
1078         Reviewed by john.
1079
1080         * WebView.subproj/WebHTMLView.m:
1081         (-[WebHTMLView _hitViewForEvent:]): new, factored hit test hack to this method
1082         (-[WebHTMLView _updateMouseoverWithEvent:): call _hitViewForEvent:
1083         (-[WebHTMLView acceptsFirstMouse:]): call _setMouseDownEvent: and _isSelectionEvent: on the hit HTMLView or else when it's asked to drag it will assert
1084         (-[WebHTMLView shouldDelayWindowOrderingForEvent:]): ditto
1085
1086 2005-01-31  David Harrison  <harrison@apple.com>
1087
1088         Reviewed by Darin.
1089
1090         <rdar://problem/3961239> AX Setting AXFocused on AXScrollArea of AXWebArea will cause keyboard selection change
1091
1092         * WebView.subproj/WebHTMLView.m:
1093         (-[WebHTMLView maintainsInactiveSelection]):
1094         Keep the selection when the new first respomder is our own scrollview, in both editable and non-editaqble content.
1095
1096 2005-01-31  Jens Alfke  <jens@apple.com>
1097
1098         Reviewed by John.
1099
1100         - Fixed <rdar://problem/3903199> REGRESSION: Large background patterns slide around while loading
1101
1102         * WebCoreSupport.subproj/WebImageData.m:
1103         (drawPattern):
1104         (-[WebImageData tileInRect:fromPoint:context:]):
1105
1106 2005-01-30  Darin Adler  <darin@apple.com>
1107
1108         Reviewed by John.
1109
1110         - fixed <rdar://problem/3965265> Safari displays HTML as source when default encoding is Hebrew (due to direction overrides added by encoding converter)
1111
1112         * WebView.subproj/WebPreferencesPrivate.h: Added _systemCFStringEncoding, and changed
1113         _setInitialDefaultTextEncodingToSystemEncoding to be a class method.
1114         * WebView.subproj/WebPreferences.m:
1115         (+[WebPreferences _systemCFStringEncoding]): Added. New SPI to be used by Safari. Broken out of
1116         _setInitialDefaultTextEncodingToSystemEncoding, but also added cases for MacArabic and MacHebrew.
1117         (+[WebPreferences _setInitialDefaultTextEncodingToSystemEncoding]): Refactor to use _systemCFStringEncoding.
1118
1119 2005-01-28  Jens Alfke  <jens@apple.com>
1120
1121         Reviewed by Richard.
1122
1123         <rdar://problem/3727680> printing some page with WebKit generates a PDF with a 1x1 image with soft mask (causes Acrobat to hang during print spooling)
1124
1125         Detect 1x1 images, extract and cache color of single pixel, reduce draw and tile operations to a color fill (or to a no-op if the pixel is clear.)
1126
1127         * WebCoreSupport.subproj/WebImageData.h:
1128         * WebCoreSupport.subproj/WebImageData.m:
1129         (-[WebImageData _invalidateImages]):
1130         (-[WebImageData _checkSolidColor:]):
1131         (-[WebImageData _cacheImages:allImages:]):
1132         (-[WebImageData _fillSolidColorInRect:compositeOperation:context:]):
1133         (-[WebImageData tileInRect:fromPoint:context:]):
1134
1135 2005-01-28  David Harrison  <harrison@apple.com>
1136
1137         Reviewed by Darin.
1138
1139         <rdar://problem/3584942> AX: Safari Accessibility parent-child mismatch
1140
1141         Use AppKit SPI _accessibilityParentForSubview to return KWQAccObject parent of AppKit AX object.
1142         
1143         * WebView.subproj/WebHTMLView.m:
1144         (-[WebHTMLView _accessibilityParentForSubview:]):
1145         New.
1146
1147 2005-01-28  Chris Blumenberg  <cblu@apple.com>
1148
1149         WebKit side of: <rdar://problem/3951283> can view pages from the back/forward cache that should be disallowed by Parental Controls
1150
1151         Reviewed by john.
1152
1153         * WebView.subproj/WebDefaultPolicyDelegate.m:
1154         (-[WebDefaultPolicyDelegate webView:shouldGoToHistoryItem:]): new private delegate method
1155         * WebView.subproj/WebFrame.m:
1156         (-[WebFrame _goToItem:withLoadType:]): call new delegate method
1157         * WebView.subproj/WebPolicyDelegatePrivate.h:
1158
1159 2005-01-26  Maciej Stachowiak  <mjs@apple.com>
1160
1161         Reviewed by Darin, Hyatt and Ken.
1162
1163         <rdar://problem/3790449> REGRESSION (Mail): underline behavior is flaky because of how CSS handles text-decoration
1164
1165         * WebView.subproj/WebHTMLView.m:
1166         (-[WebHTMLView _styleFromFontAttributes:]): Use new -khtml-text-decorations-in-effect
1167         property
1168         (-[WebHTMLView _styleForAttributeChange:]): likewise
1169         (-[WebHTMLView underline:]): likewise
1170
1171 2005-01-27  Ken Kocienda  <kocienda@apple.com>
1172
1173         Reviewed by Maciej
1174
1175         * WebView.subproj/WebHTMLView.m:
1176         (-[WebHTMLView _handleKillRing:prepend:]): New helper function. Code factored out from 
1177          _deleteRange:killRing:prepend:smartDeleteOK:isTypingAction:.
1178         (-[WebHTMLView _deleteRange:killRing:prepend:smartDeleteOK:isTypingAction:]): No longer takes 
1179         unused preflight argument. Now takes new isTypingAction argument. Uses isTypingAction to
1180         determine which flavor of delete command to call.
1181         (-[WebHTMLView _deleteSelection]): No longer passes unused preflight argument. Now passes new isTypingAction argument.
1182         (-[WebHTMLView _deleteWithDirection:granularity:killRing:isTypingAction:]): Rearranged code to call
1183         correct flavor of delete command, based on whether typing should be preserved. Some other cleanups.
1184         (-[WebHTMLView deleteForward:]): Add _isEditable check.
1185         (-[WebHTMLView deleteBackward:]): Now calls _deleteWithDirection instead of having unique behavior different 
1186         from forward delete.
1187         (-[WebHTMLView deleteWordForward:]): Add new isTypingAction flag to _deleteWithDirection call.
1188         (-[WebHTMLView deleteWordBackward:]): Ditto.
1189         (-[WebHTMLView deleteToBeginningOfLine:]): Ditto.
1190         (-[WebHTMLView deleteToEndOfLine:]): Ditto.
1191         (-[WebHTMLView deleteToBeginningOfParagraph:]): Ditto.
1192         (-[WebHTMLView deleteToEndOfParagraph:]): Ditto.
1193         (-[WebHTMLView deleteToMark:]): Ditto.
1194
1195 === Safari-181 ===
1196
1197 2005-01-27  Chris Blumenberg  <cblu@apple.com>
1198
1199         Fixed: <rdar://problem/3973272> REGRESSION: Safari uses QT plugin to display PNG images
1200
1201         Reviewed by john.
1202
1203         * Plugins.subproj/WebBasePluginPackage.m: use renamed QT bundle ID
1204
1205 2005-01-26  Darin Adler  <darin@apple.com>
1206
1207         Reviewed by John.
1208
1209         - fixed <rdar://problem/3829517> WebView still draws white when setDrawsBackground set to NO and no content loaded yet
1210
1211         * WebView.subproj/WebFrameView.m: (-[WebFrameView drawRect:]): Check drawsBackground, and don't
1212         draw the white "no document" background if it's NO. This fixes things for frames with no document.
1213         * WebView.subproj/WebImageView.m: (-[WebImageView drawRect:]): Same as above. This fixes things
1214         for frames with just an image.
1215         * WebView.subproj/WebFrame.m: (-[WebFrame _updateDrawsBackground]): Call setDrawsBackground: on the
1216         document view if it implements it. This fixes things for frames with plain text.
1217
1218 2005-01-25  John Sullivan  <sullivan@apple.com>
1219
1220         Reviewed by Chris.
1221         
1222         - WebKit part of fix for: <rdar://problem/3970670> Text context menu 
1223           in WebKit needs "Look Up in Dictionary" item
1224
1225         * WebView.subproj/WebDefaultContextMenuDelegate.m:
1226         (localizedMenuTitleFromAppKit):
1227         return English string in the case where AppKit bundle is found but doesn't
1228         contain the expected string. This case will be hit by people testing with
1229         an older AppKit.
1230         (-[WebDefaultUIDelegate menuItemWithTag:]):
1231         create Look Up in Dictionary item
1232         (-[WebDefaultUIDelegate contextMenuItemsForElement:]):
1233         add Look Up in Dictionary item and separator
1234         (-[WebDefaultUIDelegate editingContextMenuItemsForElement:]):
1235         ditto
1236
1237         * WebView.subproj/WebHTMLView.m:
1238         (-[WebHTMLView _lookUpInDictionaryFromMenu:]):
1239         new method, handles bringing up the Dictionary window. Includes FIXMEs
1240         for a couple of the known loose ends.
1241         (-[WebHTMLView validateUserInterfaceItem:]):
1242         handle Look Up in Dictionary item like the other new items
1243         
1244         * WebView.subproj/WebUIDelegatePrivate.h:
1245         added SPI constant for Look Up in Dictionary menu item
1246
1247 2005-01-24  Maciej Stachowiak  <mjs@apple.com>
1248
1249         Fixed Panther build (missing ifdef)
1250
1251         * WebView.subproj/WebDefaultContextMenuDelegate.m:
1252
1253 2005-01-24  John Sullivan  <sullivan@apple.com>
1254
1255         Reviewed by Darin.
1256         
1257         - WebKit part of fix for <rdar://problem/3960231> Text context menu in WebKit needs Spotlight and Google items
1258         
1259         * WebView.subproj/WebDefaultContextMenuDelegate.m:
1260         (localizedMenuTitleFromAppKit):
1261         new function, gets localized string from AppKit so we can avoid adding localized
1262         strings to WebKit at this late date in Tiger. Returns the non-localized English
1263         string if we can't find the localized string in AppKit.
1264         (-[WebDefaultUIDelegate menuItemWithTag:]):
1265         handle the two new menu items by tag name
1266         (-[WebDefaultUIDelegate contextMenuItemsForElement:]):
1267         add menu items for Search in Google and Search in Spotlight to selected-text menu item
1268         for the noneditable case
1269         (-[WebDefaultUIDelegate editingContextMenuItemsForElement:]):
1270         add menu items for Search in Google and Search in Spotlight to selected-text menu item
1271         for the editable case
1272
1273         * WebView.subproj/WebHTMLView.m:
1274         (-[WebHTMLView _searchWithGoogleFromMenu:]):
1275         implement this menu item action method, using same method name and implementation as NSTextView
1276         (-[WebHTMLView _searchWithSpotlightFromMenu:]):
1277         implement this menu item action method, using same basic implementation as NSTextView
1278         (-[WebHTMLView validateUserInterfaceItem:]):
1279         validate new menu items
1280         
1281         * WebView.subproj/WebUIDelegatePrivate.h:
1282         define new tags for new menu items
1283
1284         * English.lproj/StringsNotToBeLocalized.txt:
1285         updated for these changes
1286
1287 2005-01-24  Darin Adler  <darin@apple.com>
1288
1289         * WebView.subproj/WebDataSource.m: (+[WebDataSource _repTypesAllowImageTypeOmission:]):
1290         Do the same check as for view types, so the representation types are consistent.
1291
1292 2005-01-24  Darin Adler  <darin@apple.com>
1293
1294         Reviewed by John.
1295
1296         - fixed <rdar://problem/3791158> REGRESSION (Mail): copyFont: and pasteFont: copy and paste only the NSFont, not other attributes
1297
1298         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView _styleFromFontAttributes:]): Added the last
1299         few loose ends here: strikethrough and underline.
1300
1301         - fixed <rdar://problem/3967393> add a user default that lets you turn off WebKit PDF support
1302
1303         * WebView.subproj/WebFrameView.m: (+[WebFrameView _viewTypesAllowImageTypeOmission:]):
1304         Leave the PDF-handling classes out of the dictionary if the secret default is set.
1305
1306         * English.lproj/StringsNotToBeLocalized.txt: Updated for recent changes.
1307
1308 2005-01-20  Darin Adler  <darin@apple.com>
1309
1310         Reviewed by Kristin Forster.
1311
1312         - fixed <rdar://problem/3964972> update _initWithCGSEvent:eventRef: call in mouse moved workaround (breaks cursors in Carbon WebKit applications)
1313
1314         * Carbon.subproj/HIWebView.m: (MouseMoved): Instead of munging the event record's window number directly
1315         before calling _initWithCGSEvent, on Tiger we call _eventRelativeToWindow on the event after creating it.
1316         Also added a check so that with Macromedia Contribute's workaround in place we don't do anything at all
1317         to the event. Tested with both Contribute and CarbonWeb.
1318
1319         * English.lproj/StringsNotToBeLocalized.txt: Update for recent changes.
1320
1321 2005-01-20  Richard Williamson   <rjw@apple.com>
1322
1323         Fixed <rdar://problem/3815672> REGRESSION (Mail): Japanese text cannot be made bold
1324
1325         The AppKit's font substitution API doesn't match font traits! 
1326         It only find fonts that contain the appropriate glyphs.  This
1327         patch attempts to find the best variation within a family.
1328
1329         Reviewed by Maciej.
1330
1331         * WebCoreSupport.subproj/WebTextRenderer.m:
1332         (-[WebTextRenderer _substituteFontForString:families:]):
1333
1334 === Safari-180 ===
1335
1336 2005-01-20  Ken Kocienda  <kocienda@apple.com>
1337
1338         Reviewed by John
1339
1340         Fix for this bug:
1341         
1342         <rdar://problem/3786659> REGRESSION (Mail): editable WebViews don't work with "size up" and "size down" NSFontManager changes
1343
1344         * WebView.subproj/WebHTMLView.m:
1345         (-[WebHTMLView _addToStyle:fontA:fontB:]): This is the WebKit side of the fix. Replaced unimplemented code blocks
1346         with FIXME's in them for make bigger and make smaller with real working code.
1347
1348 2005-01-19  David Hyatt  <hyatt@apple.com>
1349
1350         Fix for 3513067, spaces being lost when RTL text is rendered.  Make sure not to allow hangers or spaces
1351         in the margin.
1352         
1353         Reviewed by john
1354         
1355         * WebCoreSupport.subproj/WebTextRenderer.m:
1356         (-[WebTextRenderer _createATSUTextLayoutForRun:style:]):
1357
1358 2005-01-19  Darin Adler <darin@apple.com>
1359
1360         Reviewed by vicki
1361
1362         - fixed <rdar://problem/3962559> stopAnimationsInView leaks after cvs-base
1363  
1364         * WebCoreSupport.subproj/WebImageData.m: (+[WebImageData stopAnimationsInView:]): add a release
1365
1366 2005-01-19  Chris Blumenberg  <cblu@apple.com>
1367
1368         Fixed: <rdar://problem/3961809> plug-in code attempts to load empty URL
1369
1370         Reviewed by john.
1371
1372         * Plugins.subproj/WebNetscapePluginEmbeddedView.m:
1373         (-[WebNetscapePluginEmbeddedView didStart]): don't start load if URL is empty
1374
1375 2005-01-17  Darin Adler  <darin@apple.com>
1376
1377         * DOM.subproj/DOMPrivate.h: Check in generated file.
1378
1379 2005-01-17  Darin Adler  <darin@apple.com>
1380
1381         Reviewed by John and Richard.
1382
1383         - fixed <rdar://problem/3907453> printing a multi-page PDF document from Safari doesn't produce correct output
1384
1385         * WebView.subproj/WebFrameViewPrivate.h: Added.
1386         * WebView.subproj/WebFrameView.m:
1387         (-[WebFrameView canPrintHeadersAndFooters]): Added. Returnes NO for documents that can't print headers or footers,
1388         and delegates to the document view to answer the question. Defaults to NO, since only a view that actively does
1389         the work is compatible with our header and footer code.
1390         (-[WebFrameView printOperationWithPrintInfo:]): Added. Returns an NSPrintOperation set up for printing. The reason
1391         we return this rather than an NSView is that in the PDFView case, the print info is changed around before creating
1392         the NSPrintOperation, and also the PDFKit SPI works this way.
1393
1394         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView canPrintHeadersAndFooters]): Added. Returns YES.
1395         * WebView.subproj/WebImageView.m: (-[WebImageView canPrintHeadersAndFooters]): Ditto.
1396         * WebView.subproj/WebTextView.m: (-[WebTextView canPrintHeadersAndFooters]): Ditto.
1397
1398         * WebView.subproj/WebPDFView.m:
1399         (-[WebPDFView canPrintHeadersAndFooters]): Added. Returns NO.
1400         (-[WebPDFView printOperationWithPrintInfo:]): Added. Calls getPrintOperationForPrintInfo: autoRotate:YES on the PDFView.
1401
1402         * WebKit.pbproj/project.pbxproj: Added WebFrameViewPrivate.h as a new private header.
1403
1404 2005-01-13  Chris Blumenberg  <cblu@apple.com>
1405
1406         Fixed: <rdar://problem/3932107> Safari does not load QT Cocoa plug-in if the WebPluginMIMETypes key is not in the info.plist
1407
1408         Fixing this bug required that we allow WebKit plug-ins (as well as Netscape plug-ins) support BP_CreatePluginMIMETypesPreferences
1409         which allows plug-ins create an auxiliary MIME types file.
1410
1411         Reviewed by adele.
1412
1413         * Plugins.subproj/WebBasePluginPackage.h:
1414         * Plugins.subproj/WebBasePluginPackage.m:
1415         (+[WebBasePluginPackage preferredLocalizationName]): moved from WebNetscapePluginPackage
1416         (-[WebBasePluginPackage pListForPath:createFile:]): ditto
1417         (-[WebBasePluginPackage getPluginInfoFromPLists]): ditto
1418         (-[WebBasePluginPackage isLoaded]): return isLoaded ivar
1419         (-[WebBasePluginPackage load]): if loaded, get BP_CreatePluginMIMETypesPreferences symbol
1420         * Plugins.subproj/WebNetscapePluginPackage.h:
1421         * Plugins.subproj/WebNetscapePluginPackage.m:
1422         (-[WebNetscapePluginPackage load]): call super when done so BP_CreatePluginMIMETypesPreferences can be initialized
1423         * Plugins.subproj/WebPluginPackage.m:
1424         (-[WebPluginPackage initWithPath:]): call getPluginInfoFromPLists
1425         (-[WebPluginPackage load]): call super when done so BP_CreatePluginMIMETypesPreferences can be initialized
1426
1427 2005-01-13  Richard Williamson   <rjw@apple.com>
1428
1429         Fixed <rdar://problem/3952809> WebJavaPlugIn.h comments need method name corrected (webPlugInCallJava)
1430
1431         Reviewed by Maciej.
1432
1433         * Plugins.subproj/WebJavaPlugIn.h:
1434
1435 === Safari-179 ===
1436
1437 2005-01-13  Vicki Murley <vicki@apple.com>
1438
1439         Reviewed by Adele.
1440
1441         - fix <rdar://problem/3946836> Safari about box lists 2004 instead of 2005
1442
1443         * WebKit.pbproj/project.pbxproj: bump "2004" to "2005"
1444
1445 2005-01-13  Richard Williamson   <rjw@apple.com>
1446
1447         Fixed <rdar://problem/3951911> REGRESSION: Animated GIF images with loop counts no longer update
1448
1449         Draw last image after animation loop terminates.  (We
1450         were drawing the image at index+1, which doesn't exist!)
1451
1452         Reviewed by Darin.
1453
1454         * WebCoreSupport.subproj/WebImageData.m:
1455         (-[WebImageData _nextFrame:]):
1456
1457 2005-01-13  Richard Williamson   <rjw@apple.com>
1458
1459         Fixed <rdar://problem/3952084> REGRESSION: Links at projectseven.com now draw and update incorrectly during hover
1460
1461         Turn off use of new CGContextStrokeLineSegments API.  We should
1462         turn back on when 3952944 is fixed.
1463
1464         Reviewed by Darin.
1465
1466         * WebCoreSupport.subproj/WebTextRenderer.m:
1467         (-[WebTextRenderer drawLineForCharacters:yOffset:width:color:thickness:]):
1468
1469 2005-01-13  Chris Blumenberg  <cblu@apple.com>
1470
1471         Fixed: <rdar://problem/3937663> repro assertion failure and crash dragging image that has PDF source
1472
1473         Reviewed by adele.
1474
1475         * Misc.subproj/WebNSViewExtras.m:
1476         (-[NSView _web_dragImage:rect:event:pasteboard:source:offset:]): if [WebImageRenderer image] returns nil, fallback to code that uses a file icon as the drag image
1477         * WebCoreSupport.subproj/WebImageRendererFactory.m:
1478         (-[WebImageRendererFactory supportedMIMETypes]): removed code that omits PDF and PostScript from the list since this omission is only needed in WebImageView
1479         * WebView.subproj/WebImageView.m:
1480         (+[WebImageView supportedImageMIMETypes]): added code that omits PDF and PostScript since we don't want WebImageView to render these types
1481
1482 2005-01-10  Maciej Stachowiak  <mjs@apple.com>
1483
1484         Reviewed by Darin.
1485
1486         <rdar://problem/3758033> REGRESSION (Mail): Support attributes in marked text (International input)
1487         
1488         * WebCoreSupport.subproj/WebTextRenderer.m:
1489         (-[WebTextRenderer drawLineForCharacters:yOffset:width:color:thickness:]): Changed to support
1490         underline thickness. Also added a bit of a hack here to move thickness 2 underlines down by
1491         .5 pixels, since the rendering engine can't give a fractional pixel offset.
1492         * WebView.subproj/WebHTMLView.m:
1493         (-[WebHTMLView validAttributesForMarkedText]): Support underline, underline color and marked
1494         clause attributes. Others that NSText supports are unimplemented for now.
1495         (-[WebHTMLView firstRectForCharacterRange:]): Remove needless logging.
1496         (-[WebHTMLView unmarkText]): Updated for new WebCore SPI.
1497         (-[WebHTMLView _extractAttributes:ranges:fromAttributedString:]): New method to pull the attributes
1498         and ranges out of an attributed string.
1499         (-[WebHTMLView setMarkedText:selectedRange:]): Extract attributes and pass to WebCore.
1500         (-[WebHTMLView insertText:]): Add comment noting that we don't really handle attributed strings
1501         here.
1502
1503 2005-01-12  Darin Adler  <darin@apple.com>
1504
1505         Reviewed by Ken.
1506
1507         - fixed <rdar://problem/3848257> WebView will draw more than AppKit asks it to, so views behind won't redraw enough (transparent WebView)
1508
1509         * WebView.subproj/WebHTMLView.m:
1510         (-[WebHTMLView _propagateDirtyRectsToOpaqueAncestors]): As recommended by Troy Stephens, do the
1511         layouts here in this call, since it's before propagating the dirty rects to our ancestors.
1512         This fixes the bug, but we only do it if the WebView is not opaque, because otherwise we can
1513         optimize by only doing layouts you really need, and doing them later on is safe because we
1514         know we don't need to draw any of the views behind us.
1515         (-[WebHTMLView _layoutIfNeeded]): Added. Factored out from the method below.
1516         (-[WebHTMLView _web_layoutIfNeededRecursive]): Added. Like the other "layout if needed" call,
1517         but unconditional.
1518         (-[WebHTMLView _web_layoutIfNeededRecursive:testDirtyRect:]): Factored out the guts into the
1519         _layoutIfNeeded method above. Otherwise unchanged.
1520         (-[NSView _web_layoutIfNeededRecursive]): Added.
1521
1522         * WebView.subproj/WebFrame.m: (-[WebFrame _updateDrawsBackground]): Call setDrawsBackground:NO
1523         on the scroll view when changing the frame to no longer be in "draws background" mode. This
1524         is needed because the frame manages the "draws background" mode of the scroll view. It won't
1525         have any effect if you call setDrawsBackground:NO before starting to use a WebView, but without
1526         it calling setDrawsBackground:NO later won't have an immediate effect (easily visible in Safari).
1527         This was hidden before because the HTML view was filling with transparent color, which blew away
1528         the fill that was done by NSScrollView.
1529
1530         - fixed <rdar://problem/3921129> reproducible crash at www.funnychristmas.com in CFSet manipulation in WebImageData
1531
1532         * WebCoreSupport.subproj/WebImageData.m:
1533         (-[WebImageData _imageSourceOptions]): Changed types so we don't need a cast.
1534         (+[WebImageData stopAnimationsInView:]): Instead of building a set of sets, by putting in the sets with addObject,
1535         build a single set using unionSet, and then iterate the objects instead of having to iterate the sets and then the
1536         objects in each set. The old code ended up sharing the sets with the live code, when the whole idea was to gather
1537         all the renderers because the process of stopping modifies the active sets.
1538
1539 2005-01-12  Richard Williamson   <rjw@apple.com>
1540
1541         Fixed <rdar://problem/3926825> Safari ignores GIF loop count
1542
1543         Get loop count from file properties, not image properties.
1544
1545         Reviewed by Ken Kocienda.
1546
1547         * WebCoreSupport.subproj/WebImageData.h:
1548         * WebCoreSupport.subproj/WebImageData.m:
1549         (-[WebImageData _commonTermination]):
1550         (-[WebImageData fileProperties]):
1551         (-[WebImageData _floatProperty:type:at:]):
1552         (-[WebImageData _floatFileProperty:type:]):
1553         (-[WebImageData _repetitionCount]):
1554
1555 2005-01-11  Chris Blumenberg  <cblu@apple.com>
1556
1557         Fixed: <rdar://problem/3934749> assertion failure in WebBaseNetscapePluginView loading movie
1558
1559         Reviewed by john.
1560
1561         * Plugins.subproj/WebBaseNetscapePluginView.m:
1562         (-[WebBaseNetscapePluginView start]): call canStart before asserting about the webView
1563
1564 2005-01-11  John Sullivan  <sullivan@apple.com>
1565
1566         Reviewed by Darin.
1567         
1568         - fixed <rdar://problem/3446838> REGRESSION (Mail): text decorations don't print 
1569         (e.g. <strike>, underline)
1570
1571         * WebCoreSupport.subproj/WebTextRenderer.m:
1572         (-[WebTextRenderer drawLineForCharacters:yOffset:withWidth:withColor:]):
1573         This bottleneck routine for drawing a line was setting the linewidth to 0 when
1574         the graphics context was not drawing to the screen. Thus, no lines. Now links
1575         are underlined when printing from Safari (as well as Mail).
1576
1577 2005-01-11  Richard Williamson   <rjw@apple.com>
1578
1579         Fixed 3949145.  CG has a much faster API for drawing lines.
1580         Switched over to that new API (CGContextStrokeLineSegments).
1581
1582         Reviewed by John Sullivan.
1583
1584         * WebCoreSupport.subproj/WebTextRenderer.m:
1585         (-[WebTextRenderer drawLineForCharacters:yOffset:withWidth:withColor:]):
1586
1587 2005-01-10  Chris Blumenberg  <cblu@apple.com>
1588
1589         Fixed: <rdar://problem/3948862> REGRESSION: missing images when RTFD is pasted into editable WebView
1590
1591         This problem occurred because we were creating image elements before creating corresponding image resources. The fix is to have AppKit call us back to create the resources before it creates the elements.
1592
1593         Reviewed by john.
1594
1595         * WebView.subproj/WebHTMLView.m:
1596         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): don't deal with subresources since that's now done by the following method
1597         (-[WebHTMLView resourceForData:preferredFilename:]): new handler method called by AppKit
1598
1599 === Safari-178 ===
1600
1601 2005-01-06  David Harrison  <harrison@apple.com>
1602
1603         Reviewed by Dave Hyatt
1604
1605         <rdar://problem/3588548> AX: tabbing does not work correctly with the screen reader and a focused link; need AXFocusedUIElement to work.
1606
1607         * WebView.subproj/WebHTMLView.m:
1608         (-[WebHTMLView accessibilityFocusedUIElement]):
1609         Implement this so that AppKit can use it from NSAccessibilityHandleFocusChanged.
1610
1611 2005-01-05  Chris Blumenberg  <cblu@apple.com>
1612
1613         Fixed: <rdar://problem/3936865> REGRESSION: canvas.drawImage no longer scales properly
1614
1615         Reviewed by john.
1616
1617         * WebCoreSupport.subproj/WebImageData.m: use the height of the inRect instead of the fromRect when setting the origin of the context
1618
1619 2005-01-04  Chris Blumenberg  <cblu@apple.com>
1620
1621         Fixed: <rdar://problem/3928329> WebKit should pass nil for "language" to checkSpellingOfString:
1622
1623         Reviewed by kocienda.
1624
1625         * WebView.subproj/WebHTMLView.m:
1626         (-[WebHTMLView _isSelectionMisspelled]): pass nil not @"" for language
1627
1628 2004-12-21  Maciej Stachowiak  <mjs@apple.com>
1629
1630         Reviewed by Darin.
1631
1632         <rdar://problem/3888931> frame naming allows malicious site to bring up a window when you click on a link in another
1633
1634         Implement a security check on name frame visbility. This is the
1635         same rule as mozilla. You can only target frames by name if you
1636         are in the same window, have the same domain as the frame or an
1637         ancestor, or if it's a top level window have the same domain as
1638         the opener.
1639         
1640         * WebView.subproj/WebFrame.m:
1641         (-[WebFrame _shouldAllowAccessFrom:]):
1642         (-[WebFrame _descendantFrameNamed:sourceFrame:]):
1643         (-[WebFrame findFrameNamed:]):
1644         * WebView.subproj/WebFramePrivate.h:
1645         * WebView.subproj/WebView.m:
1646         (-[WebView _findFrameInThisWindowNamed:sourceFrame:]):
1647         (-[WebView _findFrameNamed:sourceFrame:]):
1648         * WebView.subproj/WebViewPrivate.h:
1649
1650 === Safari-177 ===
1651
1652 === Safari-176 ===
1653
1654 2004-12-20  Richard Williamson   <rjw@apple.com>
1655
1656         Add call to new API.  ImageIO deprecated some older (although
1657         quite new!) API.   This caused us to fail to build on 337 or later.
1658
1659         Developers wanting to build on older versions of Tiger must define
1660         USE_DEPRECATED_IMAGESOURCE_API in WebImageData.m.
1661
1662         Reviewed by Vicki.
1663
1664         * WebCoreSupport.subproj/WebImageData.m:
1665         (-[WebImageData propertiesAtIndex:]):
1666
1667 2004-12-20  Richard Williamson   <rjw@apple.com>
1668
1669         Don't call Tiger SPI on Panther.
1670
1671         Reviewed by Vicki.
1672
1673         * WebCoreSupport.subproj/WebTextRendererFactory.m:
1674         (+[WebTextRendererFactory createSharedFactory]):
1675
1676 2004-12-20  Richard Williamson   <rjw@apple.com>
1677
1678         Fixed <rdar://problem/3884448> WebKit should turn on CG local font cache
1679
1680         Enable mutli-tier font caching.  We should see a performance boost with this
1681         change.
1682
1683         Reviewed by Chris.
1684
1685         * WebCoreSupport.subproj/WebTextRendererFactory.m:
1686         (+[WebTextRendererFactory createSharedFactory]):
1687
1688 2004-12-20  Richard Williamson   <rjw@apple.com>
1689
1690         Fix image decoding to separately decode image meta data from actual image bits.  I
1691         incorrectly consolidated decode of meta data and image bits resulting in a huge
1692         performance regression.
1693
1694         Double size of WebCore cache on lower end machines.  On the PLT run on machines with
1695         256MB of memory, too many images were being evicted, causing a re-decode on the PLT.
1696         Upping the lower limit of the cache size ensure that no images are evicted (this
1697         goes hand-in-hand with the change to the minimum object size from 32K to 40K).
1698
1699         Reviewed by Ken.
1700
1701         * WebCoreSupport.subproj/WebImageData.h:
1702         * WebCoreSupport.subproj/WebImageData.m:
1703         (+[WebImageData initialize]):
1704         (-[WebImageData _commonTermination]):
1705         (-[WebImageData _invalidateImages]):
1706         (-[WebImageData _invalidateImageProperties]):
1707         (-[WebImageData imageAtIndex:]):
1708         (-[WebImageData propertiesAtIndex:]):
1709         (-[WebImageData _cacheImages:allImages:]):
1710         (-[WebImageData decodeData:isComplete:callback:]):
1711         (-[WebImageData incrementalLoadWithBytes:length:complete:callback:]):
1712         * WebView.subproj/WebPreferences.m:
1713         (+[WebPreferences initialize]):
1714
1715 2004-12-20  Richard Williamson   <rjw@apple.com>
1716
1717         Fixed build problem caused by change to ImageIO API.
1718
1719         Reviewed by Adele.
1720
1721         * WebCoreSupport.subproj/WebImageData.m:
1722
1723 2004-12-19  Darin Adler  <darin@apple.com>
1724
1725         Reviewed by Kevin.
1726
1727         - some garbage collection fixes
1728
1729         * Misc.subproj/WebNSObjectExtras.h: (WebCFAutorelease): Replaced the old WebNSRetainCFRelease with this
1730         much-easier-to-understand function cribbed from what David Harrison did in WebCore.
1731
1732         * Misc.subproj/WebKitNSStringExtras.m: (+[NSString _web_encodingForResource:]): Use CFRelease here to
1733         get rid of an unnecessary use of WebNSRetainCFRelease.
1734         * Misc.subproj/WebNSURLExtras.m:
1735         (+[NSURL _web_URLWithData:relativeToURL:]): Use WebCFAutorelease instead of WebNSRetainCFRelease and autorelease.
1736         (-[NSURL _web_URLWithLowercasedScheme]): Ditto.
1737         (-[NSString _web_mapHostNameWithRange:encode:makeString:]): Use WebCFAutorelease here; the old code would not
1738         work correctly under GC.
1739         * Plugins.subproj/WebNetscapePluginPackage.m: (+[WebNetscapePluginPackage preferredLocalizationName]):
1740         Use WebCFAutorelease here; the old code would not work correctly under GC.
1741
1742 2004-12-18  Chris Blumenberg  <cblu@apple.com>
1743
1744         Fixed: <rdar://problem/3766915> PDF content needs search to work
1745
1746         Reviewed by kevin, john.
1747
1748         * WebView.subproj/WebPDFView.m:
1749         (-[WebPDFView searchFor:direction:caseSensitive:wrap:]): implemented
1750         (-[WebPDFView takeFindStringFromSelection:]): new
1751         (-[WebPDFView jumpToSelection:]): new
1752         (-[WebPDFView validateUserInterfaceItem:]): new
1753
1754 2004-12-17  Richard Williamson   <rjw@apple.com>
1755
1756         Make image decoding as lazy as possible for non threaded case; in some cases
1757         can avoid unnecessary decoding work.
1758
1759         Reviewed by Chris.
1760
1761         * WebCoreSupport.subproj/WebImageData.h:
1762         * WebCoreSupport.subproj/WebImageData.m:
1763         (-[WebImageData imageAtIndex:]):
1764         (-[WebImageData propertiesAtIndex:]):
1765         (-[WebImageData incrementalLoadWithBytes:length:complete:callback:]):
1766
1767 2004-12-16  John Sullivan  <sullivan@apple.com>
1768
1769         Reviewed by Chris.
1770         
1771         One of the assertions from my previous checkin fired, so
1772         I made this code more robust.
1773
1774         * WebCoreSupport.subproj/WebBridge.m:
1775         (-[WebBridge _preferences]):
1776         new helper method, returns global preferences if webView is nil,
1777         otherwise returns webView's preferences
1778         (-[WebBridge getObjectCacheSize]):
1779         use new helper method, remove now-unnecessary assert
1780         (-[WebBridge _retrieveKeyboardUIModeFromPreferences:]):
1781         ditto
1782
1783 2004-12-16  John Sullivan  <sullivan@apple.com>
1784
1785         Reviewed by Darin.
1786         
1787         - fixed <rdar://problem/3913523> Mail needs SPI for adding tooltips to links
1788         - cleaned up some calls to +[WebPreferences standardPreferences] that should
1789           have been using -[WebView preferences]
1790           
1791         This adds a (currently SPI-only) new feature that shows the URL of the link
1792         under the mouse in a toolTip. I tested this in Safari, but we're adding this
1793         feature for Mail, and Safari won't use it (unless of course you know the
1794         magic defaults command)
1795
1796         * Plugins.subproj/WebBaseNetscapePluginView.m:
1797         (-[WebBaseNetscapePluginView start]):
1798         use -[WebView preferences] instead of +[WebPreferences standardPreferences]
1799         
1800         * WebCoreSupport.subproj/WebBridge.m:
1801         (-[WebBridge getObjectCacheSize]):
1802         ditto
1803         (-[WebBridge _retrieveKeyboardUIModeFromPreferences:]):
1804         ditto
1805         
1806         * WebView.subproj/WebBaseResourceHandleDelegate.m:
1807         (-[WebBaseResourceHandleDelegate willCacheResponse:]):
1808         ditto
1809         
1810         * WebView.subproj/WebFrame.m:
1811         (-[WebFrame _transitionToCommitted:]):
1812         ditto
1813         (-[WebFrame _loadItem:withLoadType:]):
1814         ditto
1815         
1816         * WebView.subproj/WebHTMLViewInternal.h:
1817         private struct now keeps ivar for cached value of showsURLsInToolTips
1818         so it doesn't have look it up in preferences a zillion times
1819         
1820         * WebView.subproj/WebHTMLView.m:
1821         (-[WebHTMLView _updateMouseoverWithEvent:]):
1822         if private->showsURLsInToolTips is true, set the toolTip from the URL.
1823         Fall back to showing the title attribute in case some element has a title
1824         attribute but no URL.
1825         (-[WebHTMLView _mayStartDragAtEventLocation:]):
1826         use -[WebView preferences] instead of +[WebPreferences standardPreferences]
1827         (-[WebHTMLView _resetCachedWebPreferences:]):
1828         get a fresh value for private->showsURLsInToolTips
1829         (-[WebHTMLView initWithFrame:]):
1830         call _resetCachedWebPreferences the first time, and listen for WebPreferencesChanged notifications
1831         (-[WebHTMLView _handleStyleKeyEquivalent:]):
1832         use -[WebView preferences] instead of +[WebPreferences standardPreferences]
1833         
1834         * WebView.subproj/WebPreferenceKeysPrivate.h:
1835         added WebKitShowsURLsInToolTipsPreferenceKey
1836         
1837         * WebView.subproj/WebPreferences.m:
1838         (+[WebPreferences initialize]):
1839         initialize WebKitShowsURLsInToolTipsPreferenceKey to 0
1840         (-[WebPreferences showsURLsInToolTips]):
1841         return WebKitShowsURLsInToolTipsPreferenceKey value
1842         (-[WebPreferences setShowsURLsInToolTips:]):
1843         set WebKitShowsURLsInToolTipsPreferenceKey value
1844         
1845         * WebView.subproj/WebPreferencesPrivate.h:
1846         add declarations for showsURLsInToolTips and setter
1847         
1848         * WebView.subproj/WebTextView.m:
1849         (-[WebTextView _preferences]):
1850         new helper method that gets preferences from webView if there is a webView,
1851         otherwise gets global preferences
1852         (-[WebTextView setFixedWidthFont]):
1853         use new helper method rather than always using global preferences
1854         (-[WebTextView initWithFrame:]):
1855         observe WebPreferencesChangedNotification instead of unnecessarily general
1856         NSUserDefaultsChangedNotification
1857
1858 2004-12-14  John Sullivan  <sullivan@apple.com>
1859
1860         Reviewed by Ken.
1861         
1862         - rest of WebKit fix for <rdar://problem/3790011> undoable operations all say "Undo" 
1863         in the menu, no specific action names
1864         
1865         I only know of one loose end currently, which I wrote up as <rdar://problem/3920971> Edit menu 
1866         says "Undo Change Attributes" when it should say "Undo Set Color", from font panel
1867
1868         * WebView.subproj/WebHTMLView.m:
1869         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]):
1870         use WebUndoActionSetColor when dragging color swatch
1871         (-[WebHTMLView _applyStyleToSelection:withUndoAction:]):
1872         new WebUndoAction parameter, passed across bridge
1873         (-[WebHTMLView _toggleBold]):
1874         use WebUndoActionSetFont
1875         (-[WebHTMLView _toggleItalic]):
1876         use WebUndoActionSetFont
1877         (-[WebHTMLView pasteFont:]):
1878         use WebUndoActionPasteFont
1879         (-[WebHTMLView changeFont:]):
1880         use WebUndoActionSetFont
1881         (-[WebHTMLView changeAttributes:]):
1882         use WebUndoActionChangeAttributes
1883         (-[WebHTMLView _undoActionFromColorPanelWithSelector:]):
1884         new method, returns WebUndoActionSetBackgroundColor or WebUndoActionSetColor
1885         (-[WebHTMLView _changeCSSColorUsingSelector:inRange:]):
1886         now calls _undoActionFromColorPanelWithSelector
1887         (-[WebHTMLView changeColor:]):
1888         use WebUndoActionSetColor
1889         (-[WebHTMLView _alignSelectionUsingCSSValue:withUndoAction:]):
1890         new WebUndoAction parameter, passed through
1891         (-[WebHTMLView alignCenter:]):
1892         use WebUndoActionCenter
1893         (-[WebHTMLView alignJustified:]):
1894         use WebUndoActionJustify
1895         (-[WebHTMLView alignLeft:]):
1896         use WebUndoActionAlignLeft
1897         (-[WebHTMLView alignRight:]):
1898         use WebUndoActionAlignRight
1899         (-[WebHTMLView subscript:]):
1900         use WebUndoActionAlignSubscript
1901         (-[WebHTMLView superscript:]):
1902         use WebUndoActionAlignSuperscript
1903         (-[WebHTMLView unscript:]):
1904         use WebUndoActionAlignUnscript
1905         (-[WebHTMLView underline:]):
1906         use WebUndoActionAlignUnderline
1907         
1908         * WebView.subproj/WebView.m:
1909         (-[WebView setTypingStyle:]):
1910         pass WebUndoActionUnspecified through as new parameter since we don't have any more specific info
1911         (-[WebView applyStyle:]):
1912         ditto
1913
1914 2004-12-14  Richard Williamson   <rjw@apple.com>
1915
1916         Helper method to get URL of plugin view.
1917
1918         Reviewed by Chris.
1919
1920         * Misc.subproj/WebNSViewExtras.m:
1921         (-[NSView _webViewURL]):
1922
1923 2004-12-14  Vicki Murley <vicki@apple.com>
1924
1925         Reviewed by rjw.
1926  
1927         <rdar://problem/3855573> Remove reference to "WebScriptMethods" from WebScriptObject.h comments
1928
1929         * Plugins.subproj/WebPlugin.h: changed instances of "WebScriptMethods" to "WebScripting" in this
1930         file as well, as requested in the bug report
1931
1932 2004-12-13  Maciej Stachowiak  <mjs@apple.com>
1933
1934         Reviewed by John.
1935
1936         <rdar://problem/3912488> Mail throws an exception after backspacing "away" inline input
1937         
1938         * WebView.subproj/WebHTMLView.m:
1939         (-[WebHTMLView setMarkedText:selectedRange:]): Don't try to set a selection if
1940         we end up with no marked text, since that case fails and is unnecessary.
1941
1942 2004-12-14  John Sullivan  <sullivan@apple.com>
1943
1944         Reviewed by Ken.
1945
1946         - WebKit part of plumbing of fix for <rdar://problem/3790011> undoable operations all say "Undo" in the menu, 
1947         no specific action names
1948
1949         * WebCoreSupport.subproj/WebBridge.m:
1950         (-[WebBridge nameForUndoAction:]):
1951         renamed from setUndoActionNamePlaceholder, replaced arbitrary integers with enum values, and
1952         handled new "unspecified" case as a fallback
1953
1954 2004-12-13  Richard Williamson   <rjw@apple.com>
1955
1956         Fixed <rdar://problem/3887767> LiveConnect doesn't propagate Java exceptions back to JavaScript (prevents security suite from running)
1957
1958         Reviewed by John.
1959
1960         * Plugins.subproj/WebJavaPlugIn.h:
1961
1962 2004-12-13  John Sullivan  <sullivan@apple.com>
1963
1964         Reviewed by Maciej.
1965         
1966         - fixed <rdar://problem/3744583> Safari can not quit when a webpage has a login sheet 
1967         that can't be cancelled.
1968         
1969         The proper fix for this would be to change the class of the NSPanel in the nib file.
1970         But since this would require a localization change, I did a run-time hack instead.
1971         I'll file a bug about fixing this when we're out of localization freeze.
1972
1973         * Panels.subproj/WebAuthenticationPanel.m:
1974         (-[WebAuthenticationPanel replacePanelWithSubclassHack]):
1975         new method, creates a new panel that is identical to the original one except that
1976         it's our subclass, and moves all the subviews of the original panel into the new one.
1977         (-[WebAuthenticationPanel loadNib]):
1978         call replacePanelWithSubclassHack
1979         (-[NonBlockingPanel _blocksActionWhenModal:]):
1980         only method of new NSPanel subclass; overrides this SPI to allow the user to quit
1981         when one of these panels/sheets is on-screen
1982
1983 2004-12-10  Richard Williamson   <rjw@apple.com>
1984
1985         Fixed <rdar://problem/3898708> REGRESSION (8A314-8A317): World Clock's short hand not displayed (ImageIO problem with PDF?)
1986         Fixed <rdar://problem/3914012> use CG directly for pdf images not ImageIO
1987
1988         Create a PDF document and draw that instead of using ImageIO to create a rasterized image.
1989
1990         Reviewed by Maciej.
1991
1992         * WebCoreSupport.subproj/WebImageData.h:
1993         * WebCoreSupport.subproj/WebImageData.m:
1994         (-[WebImageData setIsPDF:]):
1995         (-[WebImageData isPDF]):
1996         (-[WebImageData dealloc]):
1997         (-[WebImageData decodeData:isComplete:callback:]):
1998         (-[WebImageData incrementalLoadWithBytes:length:complete:callback:]):
1999         (-[WebImageData size]):
2000         (-[WebImageData animate]):
2001         (-[WebImageData _createPDFWithData:]):
2002         (-[WebImageData _PDFDocumentRef]):
2003         (-[WebImageData _PDFDrawInContext:]):
2004         (-[WebImageData _PDFDrawFromRect:toRect:operation:alpha:flipped:context:]):
2005         * WebCoreSupport.subproj/WebImageRenderer.h:
2006         * WebCoreSupport.subproj/WebImageRenderer.m:
2007         (-[WebImageRenderer size]):
2008         (-[WebImageRenderer incrementalLoadWithBytes:length:complete:callback:]):
2009         (-[WebImageRenderer drawImageInRect:fromRect:compositeOperator:context:]):
2010         (_createImageRef):
2011
2012 2004-12-10  John Sullivan  <sullivan@apple.com>
2013
2014         Reviewed by Ken.
2015         
2016         - fixed <rdar://problem/3855127> Error while printing w/o sheet, then window is left in a bad state, 
2017         if there's no default printer set
2018
2019         * WebView.subproj/WebHTMLView.m:
2020         (-[WebHTMLView beginDocument]):
2021         Our implementation of knowsPageRange puts the WebHTMLView into a special "printing mode". We must
2022         exit the "printing mode" to return to normal behavior. This is normally done in endDocument.
2023         However, it turns out that if there's an exception in [super beginDocument], then endDocument
2024         will not be called (lame-o AppKit API). So, we handle that case by catching the exception and
2025         exiting the "printing mode" in beginDocument when it occurs.
2026
2027 2004-12-09  Richard Williamson   <rjw@apple.com>
2028
2029         Fixed <rdar://problem/3905789> Burn Disc image vibrates rapidly
2030
2031         Restrict our support for animated images to GIF.  We used to
2032         use presence of more than one image in a resource to determine
2033         if an image should be animated.  This caused us to animate icns!
2034         If we ever support any other animated image formats we'll have
2035         to extend.
2036
2037         Reviewed by Hyatt.
2038
2039         * WebCoreSupport.subproj/WebImageData.h:
2040         * WebCoreSupport.subproj/WebImageData.m:
2041         (-[WebImageData shouldAnimate]):
2042         * WebCoreSupport.subproj/WebImageRenderer.m:
2043         (-[WebImageRenderer _startOrContinueAnimationIfNecessary]):
2044
2045 2004-12-09  Richard Williamson   <rjw@apple.com>
2046
2047         Make WebPluginDatabase.h private (Dashboard needs SPI).
2048
2049         * WebKit.pbproj/project.pbxproj:
2050
2051 === Safari-175 ===
2052
2053 2004-12-09  Chris Blumenberg  <cblu@apple.com>
2054
2055         Workaround for this exception being raised during download:
2056         [WebDownload connection:willStopBufferingData:]: selector not recognized
2057
2058         Reviewed by john.
2059
2060         * Misc.subproj/WebDownload.m:
2061         (-[WebDownload connection:willStopBufferingData:]): implement this method so no exception is raised. It is a bug in Foundation that this method is being called, but it's too late to fix Foundation since it has already been submitted this week. This workaround will prevent any problems from affecting users.
2062
2063 2004-12-08  Richard Williamson   <rjw@apple.com>
2064
2065         Fixed <rdar://problem/3911719> REGRESSION: Images no longer scale vertically
2066         Account for scaling correctly when taking into account progressively
2067         loaded images.
2068
2069         Also added implementation of repetition count for animated GIF images.
2070         Also replaced strings with new constants from CFImageProperties.h
2071
2072         Also fixed possible problem with -(NSSize)size implementation,
2073         relevant to Panther only.
2074
2075         Reviewed by Chris.
2076
2077         * WebCoreSupport.subproj/WebImageData.m:
2078         (-[WebImageData _floatProperty:type:at:]):
2079         (-[WebImageData _frameDurationAt:]):
2080         (-[WebImageData _repetitionCount]):
2081         * WebCoreSupport.subproj/WebImageRenderer.m:
2082         (-[WebImageRenderer size]):
2083
2084 2004-12-08  Chris Blumenberg  <cblu@apple.com>
2085
2086         Removed NPN wrappers since these no longer need to be defined to make the QT plug-in work
2087         since 3828925 has been fixed.
2088
2089         Reviewed by john.
2090
2091         * Plugins.subproj/WebNetscapePluginPackage.m:
2092         (-[WebNetscapePluginPackage load]): use under-bar symbols since non-under-bar wrappers have been removed
2093         * Plugins.subproj/npapi.m: removed NPN wrappers
2094         * WebKit.exp: removed symbols
2095
2096 2004-12-08  Ken Kocienda  <kocienda@apple.com>
2097
2098         Reviewed by Darin
2099
2100         * WebView.subproj/WebHTMLView.m:
2101         (-[WebHTMLView drawRect:]): Work around for this bug:
2102         <rdar://problem/3908282> REGRESSION (Mail): No drag image dragging selected text in Blot and Mail
2103         The reason for the workaround is that this method is called explicitly from the code
2104         to generate a drag image, and at that time, getRectsBeingDrawn:count: will return a zero count.
2105         This code change uses the passed-in rect when the count is zero.
2106
2107 2004-12-07  Administrator  <cblu@apple.com>
2108
2109         Support for fix for:
2110         <rdar://problem/3734309> Safari doesn't open folders in title bar menu with non-Roman names using Cmd+click
2111
2112         Reviewed by john.
2113
2114         * Misc.subproj/WebNSURLExtras.h:
2115         * Misc.subproj/WebNSURLExtras.m:
2116         (+[NSURL _web_URLWithUserTypedString:relativeToURL:]): renamed to take relativeToURL parameter
2117         (+[NSURL _web_URLWithUserTypedString:]): call _web_URLWithUserTypedString:relativeToURL: with nil for URL
2118
2119 2004-12-07  Richard Williamson   <rjw@apple.com>
2120
2121         Fixed <rdar://problem/3905564> REGRESSION (Tiger); in History menu, pixel size appears but is wrong for standalone images in Safari.
2122
2123         Reviewed by Chris.
2124
2125         * WebCoreSupport.subproj/WebImageRenderer.m:
2126         (-[WebImageRenderer size]):
2127
2128 2004-12-07  Richard Williamson   <rjw@apple.com>
2129
2130         Support threaded image decoding on machines w/ >= 2 CPUs.
2131
2132         Reviewed by Maciej and Chris.
2133
2134         * Misc.subproj/WebKitSystemBits.h:
2135         * Misc.subproj/WebKitSystemBits.m:
2136         (WebSystemMainMemory):
2137         (WebNumberOfCPUs):
2138         * WebCoreSupport.subproj/WebImageData.h:
2139         * WebCoreSupport.subproj/WebImageData.m:
2140         (+[WebImageData initialize]):
2141         (-[WebImageData init]):
2142         (-[WebImageData _commonTermination]):
2143         (-[WebImageData dealloc]):
2144         (-[WebImageData _invalidateImages]):
2145         (-[WebImageData _imageSourceOptions]):
2146         (-[WebImageData imageAtIndex:]):
2147         (-[WebImageData propertiesAtIndex:]):
2148         (-[WebImageData _createImages]):
2149         (-[WebImageData decodeData:isComplete:callback:]):
2150         (-[WebImageData incrementalLoadWithBytes:length:complete:callback:]):
2151         (drawPattern):
2152         (-[WebImageData tileInRect:fromPoint:context:]):
2153         (-[WebImageData isNull]):
2154         (-[WebImageData size]):
2155         (-[WebImageData _frameDurationAt:]):
2156         (-[WebImageData _frameDuration]):
2157         (+[WebImageData stopAnimationsInView:]):
2158         (-[WebImageData addAnimatingRenderer:inView:]):
2159         (-[WebImageData removeAnimatingRenderer:]):
2160         * WebCoreSupport.subproj/WebImageDecodeItem.h: Added.
2161         * WebCoreSupport.subproj/WebImageDecodeItem.m: Added.
2162         (+[WebImageDecodeItem decodeItemWithImage:data:isComplete:callback:]):
2163         (-[WebImageDecodeItem initWithImage:data:isComplete:callback:]):
2164         (-[WebImageDecodeItem finalize]):
2165         (-[WebImageDecodeItem dealloc]):
2166         * WebCoreSupport.subproj/WebImageDecoder.h: Added.
2167         * WebCoreSupport.subproj/WebImageDecoder.m: Added.
2168         (decoderNotifications):
2169         (+[WebImageDecoder initialize]):
2170         (+[WebImageDecoder notifyMainThread]):
2171         (+[WebImageDecoder sharedDecoder]):
2172         (+[WebImageDecoder performDecodeWithImage:data:isComplete:callback:]):
2173         (+[WebImageDecoder imageDecodesPending]):
2174         (+[WebImageDecoder decodeComplete:status:]):
2175         (-[WebImageDecoder init]):
2176         (-[WebImageDecoder dealloc]):
2177         (-[WebImageDecoder finalize]):
2178         (-[WebImageDecoder removeItem]):
2179         (-[WebImageDecoder addItem:]):
2180         (-[WebImageDecoder decodeItem:]):
2181         (decoderThread):
2182         (startDecoderThread):
2183         * WebCoreSupport.subproj/WebImageRenderer.m:
2184         (-[WebImageRenderer initWithData:MIMEType:]):
2185         (-[WebImageRenderer initWithContentsOfFile:]):
2186         (-[WebImageRenderer incrementalLoadWithBytes:length:complete:callback:]):
2187         (-[WebInternalImage incrementalLoadWithBytes:length:complete:callback:]):
2188         * WebKit.pbproj/project.pbxproj:
2189         * WebView.subproj/WebImageRepresentation.m:
2190         (-[WebImageRepresentation receivedData:withDataSource:]):
2191         (-[WebImageRepresentation receivedError:withDataSource:]):
2192         (-[WebImageRepresentation finishedLoadingWithDataSource:]):
2193
2194 2004-12-07  Chris Blumenberg  <cblu@apple.com>
2195
2196         Fix for performance regression.
2197
2198         Reviewed by kevin.
2199
2200         * WebCoreSupport.subproj/WebBridge.m:
2201         (-[WebBridge objectLoadedFromCacheWithURL:response:data:]): construct the WebResource without copying the data
2202
2203 2004-12-07  Chris Blumenberg  <cblu@apple.com>
2204
2205         Fixed: <rdar://problem/3909243> REGRESSION: large standalone images stop loading part way through
2206
2207         Reviewed by kevin.
2208
2209         * WebView.subproj/WebBaseResourceHandleDelegate.h:
2210         * WebView.subproj/WebMainResourceClient.m:
2211         (-[WebMainResourceClient addData:]): call super so it buffers the data
2212
2213 2004-12-06  Richard Williamson   <rjw@apple.com>
2214
2215         Use the AppKit's font rendering mode.  This fixes 3905347, but we still need to track down
2216         and resolve why metrics have changed for Courier.  This may be caused by changes in
2217         AppKit for 3902394.
2218
2219         Reviewed by John.
2220
2221         * WebCoreSupport.subproj/WebTextRenderer.m:
2222         (_AppkitGetCGRenderingMode):
2223         (getUncachedWidth):
2224         (_drawGlyphs):
2225
2226 2004-12-06  Chris Blumenberg  <cblu@apple.com>
2227
2228         Forgot to commit copied header.
2229
2230         * DOM.subproj/DOMPrivate.h:
2231
2232 2004-12-06  Chris Blumenberg  <cblu@apple.com>
2233
2234         Fixed: <rdar://problem/3907381> NSURLConnection and WebKit buffer 2 copies of incoming data
2235
2236         Reviewed by darin.
2237
2238         * WebView.subproj/WebBaseResourceHandleDelegate.h:
2239         * WebView.subproj/WebBaseResourceHandleDelegate.m:
2240         (+[WebBaseResourceHandleDelegate initialize]): cache check to see if Foundation supports access to its buffered data
2241         (-[WebBaseResourceHandleDelegate addData:]): don't buffer data if Foundation is buffering it for us
2242         (-[WebBaseResourceHandleDelegate saveResource]): when creating a WebResource, pass NO for copyData since we know it won't be mutated
2243         (-[WebBaseResourceHandleDelegate resourceData]): return the buffered data from the connection if it supports it
2244         (-[WebBaseResourceHandleDelegate willStopBufferingData:]): make a mutable copy of the data from NSURLConnection so we can continue buffering 
2245         (-[WebBaseResourceHandleDelegate willCacheResponse:]): removed optimization that used the cached response data to save the resource since that is obsolete by this change
2246         (-[WebBaseResourceHandleDelegate connection:willStopBufferingData:]): new callback from NSURLConnection, informs us that NSURLConnection has given up buffering
2247         * WebView.subproj/WebDataSource.m:
2248         (-[WebDataSource _receivedData:]): removed buffering code since that's done by NSURLConnection and the main client
2249         (-[WebDataSource _setData:]): removed unnecessary cast since the resourceData ivar is now an NSData instead of NSMutableData
2250         (-[WebDataSource data]): return resourceData ivar, else return the resourceData from the main client
2251         * WebView.subproj/WebDataSourcePrivate.h:
2252         * WebView.subproj/WebMainResourceClient.m:
2253         (-[WebMainResourceClient releaseResources]): store resourceData on the data source so it can continue to have data after the main client has gone away 
2254         (-[WebMainResourceClient connection:didReceiveData:lengthReceived:]):don't call [dataSource data] just to get the length of data received since [dataSource data] can now cause data to be copied
2255         (-[WebMainResourceClient connectionDidFinishLoading:]): ditto
2256         * WebView.subproj/WebResource.m:
2257         (-[WebResource initWithData:URL:MIMEType:textEncodingName:frameName:]): call following method with YES for copyData
2258         (-[WebResource _initWithData:URL:MIMEType:textEncodingName:frameName:copyData:]): new initializer, allows caller to choose whether or not the data is copied
2259         * WebView.subproj/WebResourcePrivate.h:
2260
2261 2004-12-06  Richard Williamson   <rjw@apple.com>
2262
2263         Fixed <rdar://problem/3903749> REGRESSION (8A321): WebKit gets incorrect glyph metrics due to change in how AppKit uses CGFont
2264
2265         Use CGFontRef direction when both getting font metrics and drawing
2266         glyphs, instead on depending on [NSFont set].
2267
2268         Reviewed by John.
2269
2270         * WebCoreSupport.subproj/WebTextRenderer.m:
2271         (getUncachedWidth):
2272         (_drawGlyphs):
2273
2274 2004-12-06  Ken Kocienda  <kocienda@apple.com>
2275
2276         Reviewed by Harrison
2277         
2278         Fix for this bug:
2279         
2280         <rdar://problem/3906930> Hitting return key in editable content inserts br elements instead of blocks
2281
2282         * WebView.subproj/WebHTMLView.m:
2283         (-[WebHTMLView insertNewline:]): One-line change to call insert-block rather than insert-br method 
2284         on bridge.
2285
2286 2004-12-04  Darin Adler  <darin@apple.com>
2287
2288         Reviewed by John.
2289
2290         - fixed <rdar://problem/3846079> assertion failure in WebHTMLView(WebPrivate) removeTrackingRect at boots.com
2291         - fixed <rdar://problem/3857737> REGRESSION (165-166): clicking in a text field that's scrolled to the right causes it to scroll all the way left
2292         - fixed <rdar://problem/3861952> REGRESSION (165-166): selection is cleared when you start to scroll a frame
2293
2294         * WebView.subproj/WebHTMLViewInternal.h: Added handlingMouseDown flag.
2295         * WebView.subproj/WebHTMLView.m:
2296         (-[WebHTMLView _addTrackingRect:owner:userData:assumeInside:useTrackingNum:]): Allow passing in a
2297         tracking number of 0, which means no existing tracking number.
2298         (-[WebHTMLView _addTrackingRects:owner:userDataList:assumeInsideList:trackingNums:count:]): Ditto.
2299         (-[WebHTMLView removeTrackingRect:]): Allow removing a tracking number of 0, which is a no-op.
2300         (-[WebHTMLView _removeTrackingRects:count:]): Ditto.
2301         (-[WebHTMLView acceptsFirstResponder]): Changed check to use handlingMouseDown flag instead of mouseDownEvent
2302         field since that field is set up too early in the mouse down event handling process.
2303         (-[WebHTMLView mouseDown:]): Added code to set handlingMouseDown flag.
2304
2305         - fixed part of <rdar://problem/3829808> Safari crashes when adding a DOM node that was removed from an XMLHTTP request result
2306
2307         * WebCoreSupport.subproj/WebBridge.m:
2308         (-[WebBridge syncLoadResourceWithURL:customHeaders:postData:finalURL:responseHeaders:statusCode:]):
2309         Changed code around so that it won't try to create a WebResource when the load fails.
2310
2311         - moved next/previous links into private structure with the rest of WebFrame fields
2312           (We have a rule against putting new fields into obejcts that are part of our public API.)
2313
2314         * WebView.subproj/WebFrame.h: Remove _nextSibling and _previousSibling.
2315         * WebView.subproj/WebFramePrivate.h: Added nextSibling and previousSibling fields to private class.
2316         * WebView.subproj/WebFrame.m: Got rid of some tabs in this file.
2317         (-[WebFrame _addChild:]): Changed code to use fields inside _private.
2318         (-[WebFrame _removeChild:]): Ditto.
2319         (-[WebFrame _nextFrameWithWrap:]): Ditto.
2320         (-[WebFrame _previousFrameWithWrap:]): Ditto.
2321
2322 2004-12-04  Chris Blumenberg  <cblu@apple.com>
2323
2324         New fixes for:
2325         <rdar://problem/3685766> WebDataSource is missing subresources when they use cached WebCore data
2326         <rdar://problem/3722434> REGRESSION?: Assertion failure trying to drag image in iframe (itapema.sc.gov.br)
2327         <rdar://problem/3903173> REGRESSION (172-TOT): assertion failure and crash in slotAllData logging into hotmail account
2328         <rdar://problem/3902749> REGRESSION (Tiger): missing image symbol does not appear
2329
2330         Reviewed by darin, rjw, kocienda.
2331
2332         * WebCoreSupport.subproj/WebBridge.m:
2333         (-[WebBridge objectLoadedFromCacheWithURL:response:data:]):
2334         (-[WebBridge syncLoadResourceWithURL:customHeaders:postData:finalURL:responseHeaders:statusCode:]):
2335         * WebCoreSupport.subproj/WebSubresourceClient.m:
2336         (-[WebSubresourceClient didFinishLoading]):
2337         * WebView.subproj/WebFrame.m:
2338         (-[WebFrame _opened]):
2339         (-[WebFrame _internalLoadDelegate]):
2340         (-[WebFrame _sendResourceLoadDelegateMessagesForURL:response:length:]):
2341         * WebView.subproj/WebFrameInternal.h:
2342
2343 2004-12-04  Darin Adler  <darin@apple.com>
2344
2345         Reviewed by Maciej.
2346
2347         - fixed remaining bit of <rdar://problem/3814237> REGRESSION (Mail): Copy/paste style does not set color in Mail compose window
2348
2349         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView _styleFromFontAttributes:]): When translating from
2350         an attribute dictionary to a CSS declaration, treat missing values according to the defaults defined
2351         in <AppKit/NSAttributedString.h>. Before the code was treating them as "no change", which is incorrect.
2352
2353         * English.lproj/StringsNotToBeLocalized.txt: Add a string from the above change.
2354
2355 === Safari-173 ===
2356
2357 2004-12-03  Ken Kocienda  <kocienda@apple.com>
2358
2359         Reviewed by me
2360
2361         Roll out some recent changes by Chris that caused a performance regression.
2362         Fix is in hand, but it is a little risky this close to a submission. So,
2363         we have decided to roll back the change with the regression and roll in
2364         the new code after we submit.
2365
2366         * WebCoreSupport.subproj/WebBridge.m:
2367         (-[WebBridge objectLoadedFromCacheWithURL:response:size:]):
2368         (-[WebBridge syncLoadResourceWithURL:customHeaders:postData:finalURL:responseHeaders:statusCode:]):
2369         * WebCoreSupport.subproj/WebSubresourceClient.m:
2370         (-[WebSubresourceClient didFinishLoading]):
2371         * WebView.subproj/WebFrame.m:
2372         (-[WebFrame _opened]):
2373         (-[WebFrame _internalLoadDelegate]):
2374         * WebView.subproj/WebFrameInternal.h:
2375
2376 2004-12-02  Richard Williamson   <rjw@apple.com>
2377
2378         Fixed <rdar://problem/3841332> REGRESSION (125.9-167u): repro crash in -[KWQPageState invalidate] involving .Mac images
2379
2380         Ensure that the document is cleared when leaving a non-HTML page.  This ensures that
2381         the b/f cache won't incorrectly trash the previous state when restoring.
2382
2383         Reviewed by John.
2384
2385         * WebView.subproj/WebFrame.m:
2386         (-[WebFrame _setState:]):
2387
2388 2004-12-02  Ken Kocienda  <kocienda@apple.com>
2389
2390         Reviewed by Richard
2391
2392         <rdar://problem/3748323> Problem with -[WebView editableDOMRangeForPoint:] (-isFlipped not taken into account?)
2393         <rdar://problem/3852590> REGRESSION (Mail): Dropped content appears in wrong place if Mail message is scrolled down
2394
2395         When implementing drag and drop, moveDragCaretToPoint: and editableDOMRangeForPoint: are used in
2396         concert to track the mouse and determine a drop location, respectively. However, moveDragCaretToPoint:
2397         did a conversion of the passed-in point to the document view's coordinate space, whereas 
2398         editableDOMRangeForPoint: did not. Now it does.
2399         
2400         Note that I will need to coordinate with Grant to have him roll out some code in Mail that
2401         attempts to work around this problem (unsuccessfully), and actually manages to block the
2402         real fix (which needs to be in WebKit).
2403
2404         * WebView.subproj/WebView.m:
2405         (-[WebView editableDOMRangeForPoint:]): Convert the passed-in point to the document view's coordinate space.
2406
2407 2004-12-02  Richard Williamson   <rjw@apple.com>
2408
2409         Fixed <rdar://problem/3895810> FATAL ERROR: <WebTextRenderer: 0x9328a20> unable to initialize with font "Times-Roman 16.00 pt. S ....
2410
2411         We have a hack to replace Times with Times New Roman if we fail
2412         to setup Times.  If we then fail to setup Times New Roman we
2413         don't attempt to further fallback to the system font.  Added
2414         that additional fallback.
2415
2416         Reviewed by Ken.
2417
2418         * WebCoreSupport.subproj/WebTextRenderer.m:
2419         (+[WebTextRenderer webFallbackFontFamily]):
2420         (-[WebTextRenderer initWithFont:usingPrinterFont:]):
2421
2422 2004-12-02  Richard Williamson   <rjw@apple.com>
2423
2424         Fixed build problem on Tiger8A821.  Private macro and function
2425         we were using have been deprecated,
2426
2427         Reviewed by Vicki.
2428
2429         * WebCoreSupport.subproj/WebTextRenderer.m:
2430         (-[WebTextRenderer initWithFont:usingPrinterFont:]):
2431
2432 2004-12-01  Chris Blumenberg  <cblu@apple.com>
2433
2434         Fixed: <rdar://problem/3879870> Flash Player unable to stop data stream from continuing to download by returning -1 from NPP_Write
2435         Also improved and cleaned-up the plug-in stream termination code.
2436
2437         Reviewed by john.
2438
2439         * Plugins.subproj/WebBaseNetscapePluginStream.h:
2440         * Plugins.subproj/WebBaseNetscapePluginStream.m:
2441         (+[WebBaseNetscapePluginStream reasonForError:]): return NPRES_DONE for a nil error
2442         (-[WebBaseNetscapePluginStream _pluginCancelledConnectionError]): new, factored out from other methods
2443         (-[WebBaseNetscapePluginStream errorForReason:]): new
2444         (-[WebBaseNetscapePluginStream dealloc]): release MIME type
2445         (-[WebBaseNetscapePluginStream setMIMEType:]): new
2446         (-[WebBaseNetscapePluginStream startStreamResponseURL:expectedContentLength:lastModifiedDate:MIMEType:]): call setMIMEType so we can use it in _pluginCancelledConnectionError, call renamed methods
2447         (-[WebBaseNetscapePluginStream _destroyStream]): prepended underscore, replaced some early returns with asserts as the callers are now smarter
2448         (-[WebBaseNetscapePluginStream _destroyStreamWithReason:]): prepended underscore, only call _destroyStream if there is an error or if the load is complete and there is no more data to be streamed
2449         (-[WebBaseNetscapePluginStream cancelLoadWithError:]): new, overridden by subclasses to cancel the actual NSURLConnection
2450         (-[WebBaseNetscapePluginStream destroyStreamWithError:]): new, calls _destroyStreamWithReason
2451         (-[WebBaseNetscapePluginStream finishedLoadingWithData:]): call renamed methods
2452         (-[WebBaseNetscapePluginStream _deliverData]): prepended underscore, call cancelLoadAndDestroyStreamWithError if NPP_Write returns a negative number
2453         * Plugins.subproj/WebBaseNetscapePluginView.m:
2454         (-[WebBaseNetscapePluginView destroyStream:reason:]): call cancelLoadAndDestroyStreamWithError
2455         * Plugins.subproj/WebNetscapePluginRepresentation.m:
2456         (-[WebNetscapePluginRepresentation receivedError:withDataSource:]): call destroyStreamWithError
2457         (-[WebNetscapePluginRepresentation cancelLoadWithError:]): new, override method, tell the data source to stop loading
2458         * Plugins.subproj/WebNetscapePluginStream.m:
2459         (-[WebNetscapePluginStream cancelLoadWithError:]): new, override method, tell the loader to stop
2460         (-[WebNetscapePluginStream stop]): call cancelLoadAndDestroyStreamWithError
2461         (-[WebNetscapePluginConnectionDelegate isDone]): new
2462         (-[WebNetscapePluginConnectionDelegate didReceiveResponse:]): call cancelLoadAndDestroyStreamWithError
2463         (-[WebNetscapePluginConnectionDelegate didFailWithError:]): call destroyStreamWithError
2464
2465 2004-12-01  Kevin Decker  <kdecker@apple.com>
2466
2467         Reviewed by Harrison.
2468
2469         Fixed: <rdar://problem/3228878> potential performance problem in finding in large framesets
2470
2471         Got rid of O(N^2) conditions in _nextSibling and _previousSibling of where we were looking up self in the parent array of frames.
2472
2473         * WebView.subproj/WebFrame.h: Added two new pointers, one for the previous kid and one for the next kid
2474         * WebView.subproj/WebFrame.m: 
2475         (-[WebFrame _addChild:]): Updates the previous frame and the next frame after this child
2476         (-[WebFrame _removeChild:]): ditto
2477         (-[WebFrame _nextSibling]): just return the pointer now
2478         (-[WebFrame _previousSibling]): ditto
2479
2480 2004-11-30  Chris Blumenberg  <cblu@apple.com>
2481
2482         Fixed:
2483         <rdar://problem/3685766> WebDataSource is missing subresources when they use cached WebCore data
2484         <rdar://problem/3722434> REGRESSION?: Assertion failure trying to drag image in iframe (itapema.sc.gov.br)
2485
2486         Reviewed by darin.
2487
2488         * WebCoreSupport.subproj/WebBridge.m:
2489         (-[WebBridge objectLoadedFromCacheWithURL:response:data:]): renamed to pass all data for the resource, moved delegate code to new method
2490         (-[WebBridge syncLoadResourceWithURL:customHeaders:postData:finalURL:responseHeaders:statusCode:]): call renamed method
2491         * WebCoreSupport.subproj/WebSubresourceClient.m:
2492         (-[WebSubresourceClient didFinishLoading]): call renamed method
2493         * WebView.subproj/WebFrame.m:
2494         (-[WebFrame _opened]): call _sendResourceLoadDelegateMessagesForURL:response:length:, not objectLoadedFromCacheWithURL:response:data:
2495         (-[WebFrame _internalLoadDelegate]):
2496         (-[WebFrame _sendResourceLoadDelegateMessagesForURL:response:length:]): moved from objectLoadedFromCacheWithURL:response:data:
2497         * WebView.subproj/WebFrameInternal.h:
2498
2499 2004-11-29  Darin Adler  <darin@apple.com>
2500
2501         Reviewed by John.
2502
2503         - worked around bug in Panther where NSScroller calls _destinationFloatValueForScroller: on superview
2504           without first checking if it's implemented
2505
2506         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView _destinationFloatValueForScroller:]):
2507         Implemented. Calls floatValue on the scroller.
2508
2509         * English.lproj/StringsNotToBeLocalized.txt: Update for recent changes.
2510
2511 2004-11-23  Chris Blumenberg  <cblu@apple.com>
2512
2513         Fixed: <rdar://problem/3890944> disable icon database for Dashboard
2514
2515         Reviewed by kevin.
2516
2517         * Misc.subproj/WebIconDatabase.h:
2518         * Misc.subproj/WebIconDatabase.m:
2519         (-[WebIconDatabase init]): don't create dictionaries if disabled
2520         (-[WebIconDatabase iconForURL:withSize:cache:]): return default icon if disabled
2521         (-[WebIconDatabase iconURLForURL:]): return nil if disabled
2522         (-[WebIconDatabase retainIconForURL:]): return if disabled
2523         (-[WebIconDatabase releaseIconForURL:]): ditto
2524         (-[WebIconDatabase delayDatabaseCleanup]): ditto
2525         (-[WebIconDatabase allowDatabaseCleanup]): ditto
2526         (-[WebIconDatabase _isEnabled]): new
2527         (-[WebIconDatabase _setIcon:forIconURL:]): assert if called when disabled, moved to own category implementation
2528         (-[WebIconDatabase _setHaveNoIconForIconURL:]): ditto
2529         (-[WebIconDatabase _setIconURL:forURL:]): ditto
2530         (-[WebIconDatabase _createFileDatabase]): tweak
2531         (-[WebIconDatabase _applicationWillTerminate:]): moved out of public code
2532         * Misc.subproj/WebIconDatabasePrivate.h:
2533         * Misc.subproj/WebIconLoader.m:
2534         * WebView.subproj/WebDataSource.m:
2535         (-[WebDataSource _updateIconDatabaseWithURL:]): assert if called when icon DB is disabled
2536         (-[WebDataSource _loadIcon]): don't load icon if icon DB is disabled
2537
2538 2004-11-22  David Hyatt  <hyatt@apple.com>
2539
2540         Make sure the WebCore cache grows at 512mb and at 1024mb exactly.
2541         
2542         Reviewed by mjs
2543
2544         * WebCoreSupport.subproj/WebBridge.m:
2545         (-[WebBridge getObjectCacheSize]):
2546
2547 2004-11-22  Richard Williamson   <rjw@apple.com>
2548
2549         Fixed <rdar://problem/3891737> WebPreferences do not work if they are set before set on the WebView
2550
2551         John found this problem and suggested the fix.
2552
2553         Reviewed by John Louch.
2554
2555         * WebView.subproj/WebView.m:
2556         (-[WebView setPreferences:]):
2557
2558 2004-11-22  Ken Kocienda  <kocienda@apple.com>
2559
2560         Reviewed by Harrison
2561
2562         * WebCoreSupport.subproj/WebBridge.m:
2563         (-[WebBridge canPaste]): Call WebView _canPaste.
2564         * WebView.subproj/WebView.m:
2565         (-[WebView _canPaste]): Try to forward to document view's implementation. Only WebHTMLView
2566         answers right now. Returns NO otherwise.
2567         * WebView.subproj/WebViewInternal.h: Add _canPaste method to WebView.
2568
2569 2004-11-22  Maciej Stachowiak  <mjs@apple.com>
2570
2571         Back  out the window closing fix, it seems to be causing crashes.
2572
2573         * WebView.subproj/WebFrame.m:
2574         (-[WebFrame _detachFromParent]):
2575
2576 2004-11-20  Maciej Stachowiak  <mjs@apple.com>
2577  
2578         Reviewed by John.
2579  
2580         <rdar://problem/3710101> _web_userVisibleString makes URL autocomplete roughly 2x slower
2581          
2582         * Misc.subproj/WebNSURLExtras.h:
2583         * Misc.subproj/WebNSURLExtras.m:
2584         (-[NSString _web_isUserVisibleURL]): New SPI to check if a URL
2585         string is already in user-visible form (i.e. converting it to an
2586         NSURL and then back via _web_userVisibleString would not change
2587         anything).
2588  
2589 2004-11-19  Maciej Stachowiak  <mjs@apple.com>
2590
2591         Reviewed by John.
2592
2593         <rdar://problem/3190977> closing window with many tabs in it can be quite slow
2594         
2595         * WebView.subproj/WebFrame.m:
2596         (-[WebFrame _detachFromParent]): autorelease bridge instead of releasing it,
2597         to make window and tab closing more responsive - this way the deallocation happens
2598         after the windoow or tab appears to close.
2599
2600 === Safari-172 ===
2601
2602 2004-11-19  Chris Blumenberg  <cblu@apple.com>
2603
2604         Fixed: <rdar://problem/3880387> REGRESSION: www.shockplay.com site gives "Unexpected server response"
2605
2606         Reviewed by mjs.
2607
2608         * Plugins.subproj/WebBaseNetscapePluginView.m:
2609         (-[NSData _web_locationAfterFirstBlankLine]): support both formats ("\r\n\n" and "\r\n\r\n") for separating header data from body data because Shockwave still sends the prior format
2610
2611 2004-11-19  Ken Kocienda  <kocienda@apple.com>
2612
2613         Reviewed by Harrison
2614
2615         Fix for this bug:
2616
2617         <rdar://problem/3655241> setTypingStyle: does not set the real typing style, and typingStyle does not return it
2618
2619         * WebCoreSupport.subproj/WebBridge.m:
2620         (-[WebBridge respondToChangedContents]): No longer call through to WebKit to set the typing style. The call
2621         was part of the misguided use of the setTypingStyle: and typingStyle as a cache of what was stored on
2622         the WebCore side.
2623         (-[WebBridge respondToChangedSelection]): Ditto.
2624         * WebView.subproj/WebView.m:
2625         (-[WebViewPrivate dealloc]): Object no longer has typingStyle ivar.
2626         (-[WebView setTypingStyle:]): Call over the bridge to set typing style.
2627         (-[WebView typingStyle]): Call over the bridge to retrieve typing style.
2628         * WebView.subproj/WebViewInternal.h: Object no longer has typingStyle ivar.
2629
2630 2004-11-18  John Sullivan  <sullivan@apple.com>
2631
2632         Reviewed by Darin.
2633         
2634         - fixed <rdar://problem/3886042> should save history file as binary XML so serialization, 
2635         parsing, reading and writing is faster
2636
2637         * History.subproj/WebHistory.m:
2638         (-[WebHistoryPrivate _saveHistoryGuts:URL:error:]):
2639         convert dictionary to binary data before saving
2640
2641 2004-11-18  Chris Blumenberg  <cblu@apple.com>
2642
2643         * WebView.subproj/WebHTMLRepresentation.m:
2644         (-[WebHTMLRepresentation currentForm]): removed stray ";"
2645
2646 2004-11-18  Chris Blumenberg  <cblu@apple.com>
2647
2648         Fixed development build failure.
2649
2650         * Misc.subproj/WebIconDatabase.m:
2651         (+[WebIconDatabase sharedIconDatabase]): call LOG not Log
2652
2653 2004-11-18  Chris Blumenberg  <cblu@apple.com>
2654
2655         <rdar://problem/3885708> save memory in icon DB by not using NSSets when holding 1 object
2656         
2657         Reviewed by sullivan.
2658
2659         * Misc.subproj/WebIconDatabase.m:
2660         (+[WebIconDatabase sharedIconDatabase]): added timing code
2661         (-[WebIconDatabase _clearDictionaries]): new
2662         (-[WebIconDatabase _loadIconDictionaries]): call _clearDictionaries in 2 places before we bail, use _web_setObjectUsingSetIfNecessary:forKey: when adding site URLs to the iconURLToURLs dictionary
2663         (-[WebIconDatabase _updateFileDatabase]): fixed comment
2664         (-[WebIconDatabase _setIconURL:forURL:]): use _web_setObjectUsingSetIfNecessary:forKey: when adding site URLs to the iconURLToURLs dictionary
2665         (-[WebIconDatabase _releaseIconForIconURLString:]): handle NSString objects retured from iconURLToURLs
2666         (-[NSMutableDictionary _web_setObjectUsingSetIfNecessary:forKey:]): new, puts a set on the dictionary when there are 2 or more object for s key
2667
2668 2004-11-17  Richard Williamson   <rjw@apple.com>
2669
2670         Fixed <rdar://problem/3885073> REGRESSION: Tab images at top of news.com.com replicated and squished
2671
2672         Correctly account for scaled image size and clipping.
2673
2674         Reviewed by Maciej.
2675
2676         * WebCoreSupport.subproj/WebImageData.h:
2677         * WebCoreSupport.subproj/WebImageData.m:
2678         * WebCoreSupport.subproj/WebImageRenderer.m:
2679         (-[WebImageRenderer drawImageInRect:fromRect:compositeOperator:context:]):
2680
2681 2004-11-17  Maciej Stachowiak  <mjs@apple.com>
2682
2683         Reviewed by Richard.
2684
2685         <rdar://problem/3885076> Don't make IDN calls for all-ascii URLs to save about 3 pages
2686         at Safari startup.
2687         
2688         * Misc.subproj/WebNSURLExtras.m:
2689         (mapHostNames): If encoding and not decoding, then bail early if the URL is all ascii.
2690         (-[NSString _web_mapHostNameWithRange:encode:makeString:]): Remove earlier special-case
2691         check for localhost, no longer needed.
2692
2693 2004-11-17  Richard Williamson   <rjw@apple.com>
2694
2695         Fixed <rdar://problem/3863601> Legacy font cache code in [WebTextRendererFactory createSharedFactory] may be unnecesary
2696
2697         and added call to SPI for
2698
2699         <rdar://problem/3884448> WebKit should turn on CG local font cache
2700
2701         currently disabled until a Tiger build shows up with the SPI.
2702
2703         Reviewed by David Harrison.
2704
2705         * WebCoreSupport.subproj/WebTextRendererFactory.m:
2706         (+[WebTextRendererFactory createSharedFactory]):
2707
2708 2004-11-17  Richard Williamson   <rjw@apple.com>
2709
2710         Fixed <rdar://problem/3882212> REGRESSION: Images clipped instead of scaled
2711         Fixed <rdar://problem/3884088> Crash terminating image load
2712
2713         Also added code to turn off color correction for images created
2714         via CGImageSources.  This code is currently disabled because CG  
2715         can't change the color space of images loaded progressively.
2716         Further, according to Dave Hayward, CG will no longer attempt
2717         to color correct images that don't have embedded profiles as of
2718         Tiger 8A306.
2719
2720         Reviewed by Chris.
2721
2722         * WebCoreSupport.subproj/WebImageData.m:
2723         (-[WebImageData _commonTermination]):
2724         (-[WebImageData dealloc]):
2725         (-[WebImageData _invalidateImageProperties]):
2726         (-[WebImageData imageAtIndex:]):
2727         (-[WebImageData incrementalLoadWithBytes:length:complete:]):
2728         (-[WebImageData propertiesAtIndex:]):
2729
2730 2004-11-16  Chris Blumenberg  <cblu@apple.com>
2731
2732         Fixed: <rdar://problem/3882034> REGRESSION: Context menu incorrect for PDF content
2733
2734         Reviewed by darin.
2735
2736         * WebView.subproj/WebPDFView.m:
2737         (-[WebPDFView hitTest:]): return self if the current event is a context menu event
2738         (-[WebPDFView menuForEvent:]): use the PDFView subview
2739
2740 2004-11-15  Chris Blumenberg  <cblu@apple.com>
2741
2742         Fixed: <rdar://problem/3880410> save 5 dirty pages by soft-linking against PDFKit framework
2743
2744         Reviewed by john.
2745
2746         * WebKit.pbproj/project.pbxproj:
2747         * WebView.subproj/WebPDFRepresentation.m:
2748         (+[WebPDFRepresentation PDFDocumentClass]): new
2749         (-[WebPDFRepresentation finishedLoadingWithDataSource:]): use PDFDocumentClass
2750         * WebView.subproj/WebPDFView.h:
2751         * WebView.subproj/WebPDFView.m:
2752         (+[WebPDFView PDFKitBundle]): new
2753         (+[WebPDFView PDFViewClass]): new
2754         (-[WebPDFView initWithFrame:]): create a PDFView subview
2755         (-[WebPDFView dealloc]): release the PDFView subview
2756         (-[WebPDFView PDFSubview]): new
2757
2758 2004-11-15  Chris Blumenberg  <cblu@apple.com>
2759
2760         Fixed: <rdar://problem/3879891> WebKit should link against PDFKit instead of Quartz
2761
2762         Reviewed by darin.
2763
2764         * WebKit.pbproj/project.pbxproj: link against PDFKit if it is present instead of Quartz.framework
2765
2766 2004-11-15  Richard Williamson   <rjw@apple.com>
2767
2768         Fixed missing retain of image property data.
2769
2770         Reviewed by John.
2771
2772         * WebCoreSupport.subproj/WebImageData.h:
2773         * WebCoreSupport.subproj/WebImageData.m:
2774         (-[WebImageData dealloc]):
2775         (-[WebImageData _invalidateImages]):
2776         (-[WebImageData imageAtIndex:]):
2777         (-[WebImageData propertiesAtIndex:]):
2778         (-[WebImageData _frameDuration]):
2779
2780 2004-11-15  Richard Williamson   <rjw@apple.com>
2781
2782         Cache image properties and frame durations.
2783         Create NSImage and TIFF representations from CGImage, lazily, as needed for
2784         dragging and element info dictionary.
2785
2786         Reviewed by John.
2787
2788         * WebCoreSupport.subproj/WebImageData.h:
2789         * WebCoreSupport.subproj/WebImageData.m:
2790         (-[WebImageData dealloc]):
2791         (-[WebImageData size]):
2792         (-[WebImageData propertiesAtIndex:]):
2793         (-[WebImageData _frameDurationAt:]):
2794         (-[WebImageData _frameDuration]):
2795         * WebCoreSupport.subproj/WebImageRenderer.h:
2796         * WebCoreSupport.subproj/WebImageRenderer.m:
2797         (-[WebImageRenderer dealloc]):
2798         (-[WebImageRenderer TIFFRepresentation]):
2799         (-[WebImageRenderer image]):
2800
2801 2004-11-14  Maciej Stachowiak  <mjs@apple.com>
2802
2803         Reviewed by John.
2804
2805         <rdar://problem/3879226> WebKit needlessly uses extra memory to store icon refcounts as NSNumbers       
2806
2807         * Misc.subproj/WebIconDatabase.m:
2808         (-[WebIconDatabase init]):
2809         (-[WebIconDatabase _setIconURL:forURL:]):
2810         (-[WebIconDatabase _retainIconForIconURLString:]):
2811         (-[WebIconDatabase _releaseIconForIconURLString:]):
2812         (-[WebIconDatabase _retainFutureIconForURL:]):
2813         (-[WebIconDatabase _releaseFutureIconForURL:]):
2814         * Misc.subproj/WebIconDatabasePrivate.h:
2815
2816 2004-11-15  John Sullivan  <sullivan@apple.com>
2817
2818         Reviewed by Ken.
2819         
2820         - fixed <rdar://problem/3879513> leak in [WebArchive _propertyListRepresentation] copying HTML to pasteboard
2821
2822         * WebView.subproj/WebArchive.m:
2823         (-[WebArchive _propertyListRepresentation]):
2824         the array holding the subresources was not released after use, oops!
2825
2826 2004-11-12  Chris Blumenberg  <cblu@apple.com>
2827
2828         Fixed: <rdar://problem/3874577> Opening restricted (parental) content in new window/tab reveals Safari's "Resources" folder
2829         
2830         Reviewed by john.
2831
2832         * WebView.subproj/WebDefaultContextMenuDelegate.m:
2833         (-[WebDefaultUIDelegate openFrameInNewWindow:]): use the unreachable URL if there is one
2834
2835 === Safari-171 ===
2836
2837 2004-11-11  Richard Williamson   <rjw@apple.com>
2838
2839         Report actual size (not partial size) but use partial size
2840         when drawing.
2841
2842         Reviewed by Maciej.
2843
2844         * WebCoreSupport.subproj/WebImageData.h:
2845         * WebCoreSupport.subproj/WebImageData.m:
2846         (-[WebImageData size]):
2847
2848 2004-11-11  Darin Adler  <darin@apple.com>
2849
2850         Reviewed by John.
2851
2852         - added _wasFirstResponderAtMouseDownTime method to bridge so we can fix
2853           <rdar://problem/3846152> REGRESSION (125-166): can't drag text out of <input type=text> fields
2854           with a subsequent change to WebCore.
2855
2856         * WebCoreSupport.subproj/WebBridge.m:
2857         (wasFirstResponderAtMouseDownTime:): Added. Calls _wasFirstResponderAtMouseDownTime
2858         on the WebHTMLView.
2859         (_getPreSmartSet): Move global inside the function, add (void) for cleanliness.
2860         (_getPostSmartSet): Ditto.
2861
2862         * WebView.subproj/WebHTMLView.m:
2863         (-[WebHTMLViewPrivate dealloc]): Release firstResponderAtMouseDownTime.
2864         (-[WebHTMLView _setMouseDownEvent:]): Early exit if event is not changing.
2865         Set firstResponderAtMouseDownTime to the first responder.
2866         (-[WebHTMLView mouseDown:]): Release firstResponderAtMouseDownTime after handling
2867         the mouseDown event.
2868         (-[WebHTMLView _wasFirstResponderAtMouseDownTime:]): Added. Uses the
2869         firstResponderAtMouseDownTime field.
2870         * WebView.subproj/WebHTMLViewInternal.h: Added firstResponderAtMouseDownTime field
2871         and _wasFirstResponderAtMouseDownTime method.
2872
2873         * English.lproj/StringsNotToBeLocalized.txt: Update for recent changes.
2874
2875 2004-11-11  Richard Williamson   <rjw@apple.com>
2876
2877         Reviewed by Chris.
2878
2879         Work-around to minimize impact of 3876764.  Cache frame durations
2880         after first call.  So we'll still leak 1K for each animated
2881         image, but that's better than 1K each time the frame is drawn!
2882         * WebCoreSupport.subproj/WebImageData.h:
2883         * WebCoreSupport.subproj/WebImageData.m:
2884         (-[WebImageData _frameDuration]):
2885
2886         Simplified animation cleanup code.  Fixed leak due to 
2887         incorrect key passed to CFDictionaryRemoveValue.
2888         
2889         (+[WebImageData stopAnimationsInView:]):
2890         (-[WebImageData addAnimatingRenderer:inView:]):
2891         (-[WebImageData removeAnimatingRenderer:]):
2892         (-[WebImageData _stopAnimation]):
2893
2894 2004-11-11  Darin Adler  <darin@apple.com>
2895
2896         Reviewed by Maciej.
2897
2898         - fixed <rdar://problem/3605906> Flash scrolled off the top and bottom cause CPU spin when combined with something dirty on the visible part of the page
2899
2900         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView drawRect:]):
2901         Work around AppKit bug by using rectangles from getRectsBeingDrawn:count: instead of
2902         using the passed-in rectangle.
2903
2904 2004-11-11  Richard Williamson   <rjw@apple.com>
2905
2906         Work-arounds to make new ImageIO code work correctly.  Still
2907         disabled for now.  Requires at least Tiger 300.  Testing does
2908         show a 3% improvement in PLT tests!  That's huge!
2909
2910         Reviewed by John.
2911
2912         * WebCoreSupport.subproj/WebImageData.m:
2913         (-[WebImageData imageAtIndex:]):
2914         (-[WebImageData incrementalLoadWithBytes:length:complete:]):
2915         (-[WebImageData isNull]):
2916
2917 2004-11-10  Chris Blumenberg  <cblu@apple.com>
2918
2919         Fixed: <rdar://problem/3396872> ICONS: icon DB inconsistencies can cause slowness during startup, idle and quit
2920
2921         Reviewed by john.
2922
2923         * Misc.subproj/WebFileDatabase.m:
2924         (-[WebFileDatabase objectForKey:]): added logging code
2925         * Misc.subproj/WebIconDatabase.m:
2926         (-[WebIconDatabase init]): use alloc, init rather than autorelease, retain
2927         (-[WebIconDatabase _loadIconDictionaries]): use 1 object for mapping icon URLs to site URLs and vice versa rather than 3. This avoids inconsistencies and is faster.
2928         (-[WebIconDatabase _updateFileDatabase]): write 1 object out
2929
2930 2004-11-09  David Hyatt  <hyatt@apple.com>
2931
2932         Fix for 3873234, Safari UI is unresponsive when parsing multiple HTML docs and 3873233, Safari hangs when
2933         loading large local files.
2934         
2935         Reviewed by mjs
2936
2937         * WebCoreSupport.subproj/WebBridge.m:
2938         (-[WebBridge tokenizerProcessedData]):
2939         * WebView.subproj/WebDataSource.m:
2940         (-[WebDataSource _receivedMainResourceError:complete:]):
2941         (-[WebDataSource isLoading]):
2942
2943 2004-11-09  Richard Williamson   <rjw@apple.com>
2944
2945         Fixed <rdar://problem/3870964> 8A300: Safari not recognizing a PDF link (it displays raw data)
2946
2947         Add "text/pdf" as an acceptable PDF MIME type.
2948
2949         Reviewed by Chris.
2950
2951         * WebView.subproj/WebDataSource.m:
2952         (+[WebDataSource _repTypesAllowImageTypeOmission:]):
2953         * WebView.subproj/WebFrameView.m:
2954         (+[WebFrameView _viewTypesAllowImageTypeOmission:]):
2955
2956 2004-11-08  Chris Blumenberg  <cblu@apple.com>
2957
2958         Fixed: <rdar://problem/3783904> Return key behavior is confusingly different between popup menus and autofill menus
2959
2960         Reviewed by john.
2961
2962         * WebCoreSupport.subproj/WebBridge.m:
2963         (-[WebBridge control:textView:shouldHandleEvent:]): new
2964         * WebView.subproj/WebFormDelegate.h:
2965         * WebView.subproj/WebFormDelegate.m:
2966         (-[WebFormDelegate control:textView:shouldHandleEvent:inFrame:]): new
2967
2968 2004-11-05  Chris Blumenberg  <cblu@apple.com>
2969
2970         Fixed: <rdar://problem/3854218> Safari is sometimes really slow because of increased null events to plug-ins
2971
2972         * Plugins.subproj/WebBaseNetscapePluginView.m: reverted null event interval to 0.02
2973
2974 2004-11-05  Chris Blumenberg  <cblu@apple.com>
2975
2976         Fixed: <rdar://problem/3838413> REGRESSION (Mail): "Smart" word paste adds spaces before/after special characters
2977
2978         Reviewed by rjw.
2979
2980         * WebCoreSupport.subproj/WebBridge.m:
2981         (_getPreSmartSet): copied from AppKit
2982         (_getPostSmartSet): ditto
2983         (-[WebBridge isCharacterSmartReplaceExempt:isPreviousCharacter:]): new
2984
2985 2004-11-05  Richard Williamson   <rjw@apple.com>
2986
2987         Fixed <rdar://problem/3810702> _checkNavigationPolicyForRequest:dataSource:formState:andCall:withSelector: ASSERTS when reentered from Xcode's man page viewer
2988
2989         Reviewed by Maciej (a long time ago).
2990
2991         * WebView.subproj/WebFrame.m:
2992         (-[WebFrame _loadDataSource:withLoadType:formState:]):
2993
2994         Fixed <rdar://problem/3845307> WebKit needs to export _HIWebViewRegisterClass so HIWebViews can work in Carbon nib files
2995
2996         As suggested in the bug, the fix is to actually call 
2997         HIWebViewRegisterClass in WebKitInitForCarbon, rather than
2998         exporting the symbol.
2999
3000         Reviewed by Chris.
3001
3002         * Carbon.subproj/CarbonUtils.m:
3003         (WebInitForCarbon):
3004         * Carbon.subproj/HIWebView.m:
3005         * WebKit.pbproj/project.pbxproj:
3006
3007 === Safari-170 ===
3008
3009 2004-11-05  Darin Adler  <darin@apple.com>
3010
3011         Reviewed by Chris.
3012
3013         - fixed <rdar://problem/3857151> Assertion failure in "trackingRectOwner" while moving mouse over Slashdot.org page
3014
3015         * WebView.subproj/WebHTMLView.m:
3016         (-[WebHTMLView _addTrackingRect:owner:userData:assumeInside:useTrackingNum:]): Changed to no longer call
3017         addTrackingRect to do the work for consistency with the new method below. Not too much copied and pasted code.
3018         (-[WebHTMLView _addTrackingRects:owner:userDataList:assumeInsideList:trackingNums:count:]): Added an override
3019         for this new method in Tiger. No harm in implementing it on Panther, although it won't be called.
3020         (-[WebHTMLView _removeTrackingRects:count:]): Ditto.
3021
3022 2004-11-04  David Hyatt  <hyatt@apple.com>
3023
3024         Make sure the dominant line direction is properly set for RTL runs so that spaces will reverse.
3025
3026         Change xHeight to measure the ascent of the x glyph, since the xHeight metrics appear to be
3027         totally bogus in both CG and AppKit.
3028         
3029         Reviewed by darin
3030
3031         * WebCoreSupport.subproj/WebTextRenderer.m:
3032         (-[WebTextRenderer xHeight]):
3033         (-[WebTextRenderer _createATSUTextLayoutForRun:style:]):
3034         (-[WebTextRenderer _trapezoidForRun:style:atPoint:]):
3035         (-[WebTextRenderer _ATSU_drawHighlightForRun:style:geometry:]):
3036         (-[WebTextRenderer _ATSU_drawRun:style:geometry:]):
3037         (-[WebTextRenderer _ATSU_pointToOffset:style:position:reversed:includePartialGlyphs:]):
3038
3039 2004-11-02  Maciej Stachowiak  <mjs@apple.com>
3040
3041         Reviewed by Dave Hyatt (when I originally coded it).
3042
3043         Redid WebKit part of fix for:
3044
3045         <rdar://problem/3759187> REGRESSION (Mail): implement firstRectForCharacterRange:
3046        
3047         * WebView.subproj/WebHTMLView.m:
3048         (-[WebHTMLView firstRectForCharacterRange:]): Call the appropriate new bridge method,
3049         and translate to screen coordinates.
3050
3051 2004-11-02  John Sullivan  <sullivan@apple.com>
3052
3053         Reviewed by Hyatt.
3054         
3055         - [NSFont menuFontOfSize:], called from WebStringTruncator, was taking > 9% of the time creating a 
3056         very large bookmarks menu, so I cached this one NSFont object.
3057
3058         * Misc.subproj/WebStringTruncator.m:
3059         (defaultMenuFont):
3060         new function, caches the font used when no font is specified
3061         (+[WebStringTruncator centerTruncateString:toWidth:]):
3062         call new function
3063
3064 2004-11-02  Ken Kocienda  <kocienda@apple.com>
3065
3066         Reviewed by Hyatt
3067
3068         WebCore now implements a command to insert a block in response to typing a return key, and
3069         some names were improved in the course of this work.
3070
3071         * WebView.subproj/WebHTMLView.m:
3072         (-[WebHTMLView insertNewline:]): Now calls insertLineBreak on bridge object.
3073         (-[WebHTMLView insertLineBreak:]): New method.
3074         (-[WebHTMLView insertParagraphSeparator:]): Now implemented.
3075         * WebView.subproj/WebView.m: 
3076
3077 === Safari-169 ===
3078
3079 2004-10-29  Chris Blumenberg  <cblu@apple.com>
3080
3081         * WebKit.exp: added _WebPlugInModeKey, forgot to add it earlier
3082
3083 2004-10-29  Darin Adler  <darin@apple.com>
3084
3085         - fixed <rdar://problem/3855573> Remove reference to "WebScriptMethods" from WebScriptObject.h comments
3086
3087         * Plugins.subproj/WebScriptObject.h: Removed unneeded #ifdef protection for multiple includes (since
3088         this is an Objective-C header and we use #import for those). Fixed comments as requested in the bug
3089         report to match the contents of the file.
3090
3091 2004-10-27  Ken Kocienda  <kocienda@apple.com>
3092
3093         Reviewed by Chris
3094
3095         Added new SPI for Mail so it can get the behavior it needs when the user hits
3096         the return key with the selection in quoted content.
3097
3098         * WebView.subproj/WebView.m
3099         * WebView.subproj/WebViewPrivate.h
3100
3101 2004-10-26  Chris Blumenberg  <cblu@apple.com>
3102
3103         Fixed exception that Darin encountered in Mail.
3104
3105         Reviewed by darin.
3106
3107         * Plugins.subproj/WebPluginController.m:
3108         (+[WebPluginController plugInViewWithArguments:fromPluginPackage:]): if the plug-in returns a nil view, return nil
3109
3110 2004-10-25  Chris Blumenberg  <cblu@apple.com>
3111
3112         Darin made an internal notification have the Web prefix.
3113
3114         Reviewed by me.
3115
3116         * Plugins.subproj/WebBaseNetscapePluginView.m:
3117         (-[WebBaseNetscapePluginView addWindowObservers]):
3118         (-[WebBaseNetscapePluginView removeWindowObservers]):
3119         (ConsoleConnectionChangeNotifyProc):
3120
3121 2004-10-25  John Sullivan  <sullivan@apple.com>
3122
3123         Reviewed by Chris.
3124         
3125         - Cleanup from fix for <rdar://problem/3851676> bookmarks should not hold onto a WebHistoryItem object;
3126           eliminated notificationsSuppressed mechanism, which was used only by WebBookmark
3127
3128         * History.subproj/WebHistoryItem.m:
3129         removed notificationsSuppressed ivar from private data object
3130         (-[WebHistoryItem setAlternateTitle:]):
3131         remove notificationsSuppressed guard
3132         (-[WebHistoryItem setURLString:]):
3133         ditto
3134         (-[WebHistoryItem setOriginalURLString:]):
3135         ditto
3136         (-[WebHistoryItem setTitle:]):
3137         ditto
3138         (-[WebHistoryItem _setLastVisitedTimeInterval:]):
3139         ditto
3140         (-[WebHistoryItem setNotificationsSuppressed:]):
3141         removed this method
3142         (-[WebHistoryItem notificationsSuppressed]):
3143         ditto
3144         
3145         * History.subproj/WebHistoryItemPrivate.h:
3146         removed notificationsSuppressed and setNotificationsSuppressed
3147
3148 2004-10-22  Chris Blumenberg  <cblu@apple.com>
3149
3150         Fixed: <rdar://problem/3851491> installedPlugins being called for a page without plugins
3151
3152         Reviewed by mjs.
3153
3154         * WebView.subproj/WebFrameView.m:
3155         (+[WebFrameView _canShowMIMETypeAsHTML:]): call _viewTypesAllowImageTypeOmission instead of using ivar since the ivar is nil until _viewTypesAllowImageTypeOmission is called, this was causing [WebView canShowMIMEType:] to check plug-ins 
3156
3157 === Safari-168 ===
3158
3159 2004-10-22  Ken Kocienda  <kocienda@apple.com>
3160
3161         Reviewed by me
3162
3163         * WebKit.pbproj/project.pbxproj: Add GCC_ENABLE_OBJC_GC and GCC_FAST_OBJC_DISPATCH flags.
3164
3165 2004-10-21  Darin Adler  <darin@apple.com>
3166
3167         Reviewed by John.
3168
3169         - fixed <rdar://problem/3847994> REGRESSION: reproducible exception in WebImageRenderer releasePatternColor; afterwards get crash or no more browsing
3170
3171         * WebCoreSupport.subproj/WebImageRenderer.m:
3172         (-[WebInternalImage createRendererIfNeeded]): Replaced retainOrCopyIfNeeded with this.
3173         This returns nil if a copied renderer isn't needed, and returns a new renderer if a copy is.
3174         The old version was sometimes returning a WebInternalImage and other times a WebImageRenderer.
3175         (-[WebImageRenderer retainOrCopyIfNeeded]): Returns the result of createRendererIfNeeded
3176         or retains self and returns self.
3177
3178 2004-10-20  Darin Adler  <darin@apple.com>
3179
3180         Reviewed by Dave.
3181
3182         - fixed <rdar://problem/3470715> Pattern cache can get huge with use of css background-image in Safari
3183
3184         * WebCoreSupport.subproj/WebImageRenderer.h: Change WebImageRenderer to be a subclass of NSObject
3185         rather than NSImage and contain a pointer to a WebInternalImage.
3186         * WebCoreSupport.subproj/WebImageRenderer.m:
3187         (-[WebInternalImage releasePatternColor]): Added. Releases patternColor.
3188         (-[WebImageRenderer initWithMIMEType:]): Added. Makes WebInternalImage and then self.
3189         (-[WebImageRenderer initWithData:MIMEType:]): Ditto.
3190         (-[WebImageRenderer initWithContentsOfFile:]): Ditto.
3191         (-[WebImageRenderer dealloc]): Added. Calls releasePatternColor and then releases WebInternalImage.
3192         (-[WebImageRenderer image]): Added. Returns pointer to image.
3193         (-[WebImageRenderer MIMEType]): Added. Calls through to image.
3194         (-[WebImageRenderer TIFFRepresentation]): Ditto.
3195         (-[WebImageRenderer frameCount]): Ditto.
3196         (-[WebImageRenderer setOriginalData:]): Added. Sets image data pointer.
3197         (+[WebImageRenderer stopAnimationsInView:]): Added. Calls through to image.
3198         (-[WebImageRenderer incrementalLoadWithBytes:length:complete:]): Ditto.
3199         (-[WebImageRenderer size]): Ditto.
3200         (-[WebImageRenderer resize:]): Ditto.
3201         (-[WebImageRenderer drawImageInRect:fromRect:]): Ditto.
3202         (-[WebImageRenderer drawImageInRect:fromRect:compositeOperator:context:]): Ditto.
3203         (-[WebImageRenderer stopAnimation]): Ditto.
3204         (-[WebImageRenderer tileInRect:fromPoint:context:]): Ditto.
3205         (-[WebImageRenderer isNull]): Ditto.
3206         (-[WebImageRenderer retainOrCopyIfNeeded]): Ditto.
3207         (-[WebImageRenderer increaseUseCount]): Ditto.
3208         (-[WebImageRenderer decreaseUseCount]): Ditto.
3209         (-[WebImageRenderer flushRasterCache]): Ditto.
3210         (-[WebImageRenderer imageRef]): Ditto.
3211         (-[WebImageRenderer copyWithZone:]): Ditto.
3212
3213         * Misc.subproj/WebNSViewExtras.m: (-[NSView _web_dragImage:rect:event:pasteboard:source:offset:]):
3214         Update for slight changes to WebImageRenderer API.
3215         * WebCoreSupport.subproj/WebImageRendererFactory.m:
3216         (-[WebImageRendererFactory imageRendererWithMIMEType:]): Ditto.
3217         (-[WebImageRendererFactory imageRendererWithData:MIMEType:]): Ditto.
3218         (-[WebImageRendererFactory imageRendererWithSize:]): Ditto.
3219         (-[WebImageRendererFactory imageRendererWithName:]): Ditto.
3220         * WebView.subproj/WebImageView.m: (-[WebImageView image]): Ditto.
3221
3222 2004-10-20  Chris Blumenberg  <cblu@apple.com>
3223         
3224         Fixed: <rdar://problem/3846943> REGRESSION: JNLP files are rendered instead of downloaded
3225
3226         Reviewed by john.
3227
3228         * Plugins.subproj/WebBasePluginPackage.h:
3229         * Plugins.subproj/WebBasePluginPackage.m:
3230         (-[WebBasePluginPackage isJavaPlugIn]): new
3231         * Plugins.subproj/WebPluginDatabase.m:
3232         (-[WebPluginDatabase refresh]): don't register the Java plug-in for a document view since Java file should be downloaded when not embedded.
3233
3234 2004-10-20  Chris Blumenberg  <cblu@apple.com>
3235
3236         Fixed:
3237         <rdar://problem/3842030> WebKit needs to pass the mode (NP_FULL, NP_EMBED, etc) when calling plugInViewWithArguments
3238         <rdar://problem/3792852> Safari is loading the new QuickTime Cocoa plugin on Panther
3239
3240         Reviewed by darin.
3241
3242         * Plugins.subproj/WebPluginDocumentView.m:
3243         (-[WebPluginDocumentView setDataSource:]): pass "full" as the mode
3244         * Plugins.subproj/WebPluginPackage.m:
3245         (-[WebPluginPackage initWithPath:]): load plug-in with the "webplugin" extension
3246         * WebCoreSupport.subproj/WebBridge.m:
3247         (-[WebBridge pluginViewWithPackage:attributeNames:attributeValues:baseURL:]): pass "embed" as the mode
3248         * WebKit.pbproj/project.pbxproj:
3249
3250 2004-10-19  Vicki Murley <vicki@apple.com>
3251
3252         - bump WebKit version to 167.1, so that we can do a quick dot submission for <rdar://problem/3843951> 
3253
3254         * WebKit.pbproj/project.pbxproj:
3255
3256 2004-10-19  Darin Adler  <darin@apple.com>
3257
3258         Change suggested by Maciej during code review.
3259
3260         * WebCoreSupport.subproj/WebTextRenderer.m: Changed rounding hack table to be const so it can be in shared instead
3261         of private memory, and doesn't require an initialization function.
3262         (+[WebTextRenderer initialize]): Removed initialization.
3263
3264 2004-10-19  Darin Adler  <darin@apple.com>
3265
3266         Reviewed by Maciej.
3267
3268         - fixed <rdar://problem/3838934> Safari stops loading pages after rangeOfCharacterFromSet nil argument exception
3269         - fixed <rdar://problem/3843951> REGRESSION (166-167): Safari crashes in widthForNextCharacter (belkin.com, at startup for others)
3270         - fixed <rdar://problem/3841049> REGRESSION (109-110): control characters render as square boxes
3271
3272         * WebCoreSupport.subproj/WebTextRenderer.m:
3273         (isSpace): Merged in isAlternateSpace, never used.
3274         (setupRoundingHackCharacterTable): Fixed size of table, was 1 entry too short. Got rid of unneeded call to bzero,
3275         since globals start out zeroed automatically.
3276         (isRoundingHackCharacter): Fixed backwards logic causing the crash in widthForNextCharacter.
3277         Also removed explicit compare with 1; check for non-zero is just fine.
3278         (fontContainsString): Change code so we'll just skip the font if the covered character set returns nil rather than
3279         throwing an exception like the old version did. This should make bug 3838934 go away, although perhaps covering up
3280         the underlying problem.
3281         (-[WebTextRenderer _convertCharacters:length:toGlyphs:]): Removed unused skipControlCharacters: parameter and also
3282         the unnecessary code to copy the buffer to change newline characters and non-break spaces to spaces.
3283         (-[WebTextRenderer _convertUnicodeCharacters:length:toGlyphs:]): Removed unused local.
3284         (-[WebTextRenderer _extendCharacterToGlyphMapToInclude:]): Added code to set up special cases for control characters,
3285         \n and non-break spaces.
3286         (-[WebTextRenderer _createATSUTextLayoutForRun:]): Added comment about the cases this code does not handle that
3287         are handled by the CG case.
3288         (widthForNextCharacter): Call isSpace instead of checking specifically for the space character here. The old code
3289         would not handle cases with '\n' coming across from WebCore properly.
3290
3291 2004-10-18  Chris Blumenberg  <cblu@apple.com>
3292
3293         Fixed: <rdar://problem/3840916> GC: -[WebNetscapePluginPackage initWithPath:] leaks an NSURL
3294
3295         Reviewed by kevin.
3296
3297         * Plugins.subproj/WebNetscapePluginPackage.m:
3298         (-[WebNetscapePluginPackage initWithPath:]): use executablePath on NSBundle instead of CFBundleCopyExecutableURL
3299
3300 2004-10-18  Chris Blumenberg  <cblu@apple.com>
3301
3302         * DOM.subproj/DOMPrivate.h: change to copied header that was never committed
3303
3304 2004-10-18  John Sullivan  <sullivan@apple.com>
3305
3306         Reviewed by Ken.
3307         
3308         - fixed <rdar://problem/3810183> Make WebHTMLView respect return value of webView:doCommandBySelector:
3309
3310         * WebView.subproj/WebHTMLView.m:
3311         (-[WebHTMLView doCommandBySelector:]):
3312         only do default action if delegate returns NO; this works with Mail as of Tiger 8A275.
3313
3314 === Safari-167 ===
3315
3316 2004-10-14  Ken Kocienda  <kocienda@apple.com>
3317
3318         Reviewed by John
3319
3320         Final fix for these bugs:
3321         
3322         <rdar://problem/3806306> HTML editing puts spaces at start of line
3323         <rdar://problem/3814252> HTML editing groups space with word causing wrapping
3324
3325         This change sets some new CSS properties that have been added to WebCore to 
3326         enable whitespace-handling and line-breaking features that make WebView work
3327         more like a text editor.
3328
3329         * WebView.subproj/WebHTMLRepresentation.m:
3330         (-[WebHTMLRepresentation finishedLoadingWithDataSource:]): Turn on special editing
3331         CSS properties when loading an HTML document into a WebView that is editable.
3332         * WebView.subproj/WebView.m:
3333         (-[WebView setEditable:]): Add and remove special editing CSS properties in current
3334         document being displayed.
3335
3336 2004-10-14  Richard Williamson   <rjw@apple.com>
3337
3338         Fixed <rdar://problem/3823026> making isRoundingHackCharacter use -O3 and an 8-bit lookup-table will speed "XBS" test up by 3% (actually < 1%)
3339
3340         Careful testing shows a small performance gain on very large text files.  
3341         I saw large variations in timings, but taking the lowest PLT timing 
3342         with and without this change showed a 0.9% gain.  Note the cvs-base showed
3343         no improvement.  The improvement was for the large page attached to the
3344         bug.
3345
3346         Reviewed by Ken.
3347
3348         * WebCoreSupport.subproj/WebTextRenderer.m:
3349         (setupRoundingHackCharacterTable):
3350         (isRoundingHackCharacter):
3351         (+[WebTextRenderer initialize]):
3352
3353 2004-10-14  Ken Kocienda  <kocienda@apple.com>
3354
3355         Reviewed by me
3356
3357         Fix build breakage. These three functions need to return the values from their
3358         calls to WebCGColorSpaceCreateXXX.
3359
3360         * WebCoreSupport.subproj/WebGraphicsBridge.m:
3361         (-[WebGraphicsBridge createRGBColorSpace])
3362         (-[WebGraphicsBridge createGrayColorSpace])
3363         (-[WebGraphicsBridge createCMYKColorSpace])
3364
3365 2004-10-13  Richard Williamson   <rjw@apple.com>
3366
3367         Addressed concerns in <rdar://problem/3803117> RESP: High complexity in icu uidna_IDNToASCII called by [NSString(WebNSURLExtras) _web_mapHostNameWithRange:encode:makeString:]
3368
3369         In practice I saw NO improvement in performance.  Although,
3370         special-case tests could possibly show improvement.  Anyway,
3371         the changes don't hurt performance.
3372
3373         Reviewed by Maciej.
3374
3375         * Misc.subproj/WebNSURLExtras.m:
3376         (-[NSString _web_mapHostNameWithRange:encode:makeString:]):
3377
3378 2004-10-13  Maciej Stachowiak  <mjs@apple.com>
3379
3380         Reviewed by Ken.
3381
3382         <rdar://problem/3824626> Change to do colormatching for DeviceRGB colorspace causes ~11% Safari slowdown
3383         
3384         - I fixed this by turning off all colormatching for WebKit
3385         content. We might turn it back on later. For now, it's possible to
3386         turn it on temporarily by defining COLORMATCH_EVERYTHING.
3387         
3388         * WebCoreSupport.subproj/WebGraphicsBridge.m:
3389         (-[WebGraphicsBridge setFocusRingStyle:radius:color:]):
3390         (-[WebGraphicsBridge additionalPatternPhase]):
3391         (-[WebGraphicsBridge createRGBColorSpace]):
3392         (-[WebGraphicsBridge createGrayColorSpace]):
3393         (-[WebGraphicsBridge createCMYKColorSpace]):
3394         * WebCoreSupport.subproj/WebImageData.m:
3395         * WebCoreSupport.subproj/WebImageRenderer.h:
3396         * WebCoreSupport.subproj/WebImageRenderer.m:
3397         (-[WebImageRenderer _adjustSizeToPixelDimensions]):
3398         (-[WebImageRenderer incrementalLoadWithBytes:length:complete:]):
3399         (-[WebImageRenderer _adjustColorSpace]):
3400         (-[WebImageRenderer drawClippedToValidInRect:fromRect:]):
3401         (-[WebImageRenderer tileInRect:fromPoint:context:]):
3402         (_createImageRef):
3403         (WebCGColorSpaceCreateRGB):
3404         (WebCGColorSpaceCreateGray):
3405         (WebCGColorSpaceCreateCMYK):
3406         * WebKitPrefix.h:
3407
3408 2004-10-13  Richard Williamson   <rjw@apple.com>
3409
3410         Don't fill background with transparency unless debug flag
3411         is enabled.
3412
3413         Reviewed by Hyatt.
3414
3415         * WebView.subproj/WebHTMLView.m:
3416         (-[WebHTMLView drawRect:]):
3417
3418 2004-10-12  Richard Williamson   <rjw@apple.com>
3419
3420         Fixed <rdar://problem/3829705> Need to remove filling w/ transparency when not drawing backgroundy.
3421
3422         Reviewed by Ken.
3423
3424         * WebView.subproj/WebHTMLView.m:
3425         (-[WebHTMLView _transparentBackground]):
3426         (-[WebHTMLView _setTransparentBackground:]):
3427         (-[WebHTMLView drawRect:]):
3428         * WebView.subproj/WebHTMLViewInternal.h:
3429         * WebView.subproj/WebHTMLViewPrivate.h:
3430
3431 2004-10-11  Chris Blumenberg  <cblu@apple.com>
3432
3433         Fixed: <rdar://problem/3802039> 8A259: Can't use Grab services to grab selection from screen
3434
3435         Reviewed by john.
3436
3437         * WebView.subproj/WebHTMLView.m:
3438         (+[WebHTMLView initialize]): register service "return types" which are types that can be inserted into a WebView
3439         (-[WebHTMLView writeSelectionToPasteboard:types:]): service protocol method, be sure to only write specified types
3440         (-[WebHTMLView readSelectionFromPasteboard:]): new, service protocol method, insert types
3441         (-[WebHTMLView validRequestorForSendType:returnType:]): moved, handle return types
3442
3443 2004-10-11  Darin Adler  <darin@apple.com>
3444
3445         Reviewed by John.
3446
3447         - fixed <rdar://problem/3834130> nil-object-in-dictionary exception seen in -[WebView _elementAtWindowPoint:]
3448
3449         * WebView.subproj/WebView.m: (-[WebView _elementAtWindowPoint:]): Added a check for nil frame.
3450
3451 2004-10-11  Darin Adler  <darin@apple.com>
3452
3453         Reviewed by John.
3454
3455         - fixed <rdar://problem/3834166> <input type=file> sends onchange even when the same file is chosen twice
3456
3457         * WebCoreSupport.subproj/WebFileButton.m: (-[WebFileButton chooseFilename:]):
3458         Do nothing if filename is the same as before.
3459
3460 2004-10-11  Ken Kocienda  <kocienda@apple.com>
3461
3462         Reviewed by Darin
3463
3464         * WebView.subproj/WebHTMLView.m:
3465         (-[WebTextCompleteController doCompletion]): bridge call to get caret rect at a node
3466         now takes an affinity: caretRectAtNode:offset:affinity:.
3467
3468 2004-10-10  Ken Kocienda  <kocienda@apple.com>
3469
3470         Reviewed by Chris
3471
3472         Fix for this bug:
3473         
3474         <rdar://problem/3814236> REGRESSION (Mail): Can't set the color of text in Mail compose window using drag/drag from color panel
3475
3476         * WebView.subproj/WebHTMLView.m:
3477         (+[WebHTMLView _insertablePasteboardTypes]): Add NSColorPboardType to list.
3478         (-[WebHTMLView _isNSColorDrag:]): New helper. Determines if drag is an NSColor drag.
3479         (-[WebHTMLView draggingUpdatedWithDraggingInfo:actionMask:]): Add a case for NSColor drags,
3480         else do what we did before.
3481         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): Add a case for NSColor drags, which creates
3482         a CSS style containing color info and calls the bridge to apply the style. Otherwise, do what we did before.
3483
3484 2004-10-11  Darin Adler  <darin@apple.com>
3485
3486         Reviewed by Ken.
3487
3488         - fixed <rdar://problem/3833848> REGRESSION (133-134): each keydown event is getting sent multiple times
3489
3490         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView performKeyEquivalent:]):
3491         Don't send an event through WebCore if it has already been through once.
3492
3493 2004-10-10  John Sullivan  <sullivan@apple.com>
3494
3495         Reviewed by Ken.
3496         
3497         - fixed <rdar://problem/3777253> Crash in redirect mechanism trying to display error page for bad scheme
3498
3499         * WebView.subproj/WebMainResourceClient.m:
3500         (-[WebMainResourceClient connection:willSendRequest:redirectResponse:]):
3501         add retain/autorelease to the request returned from call to super. In this case, the return value
3502         was being dealloc'ed before being returned.
3503
3504 2004-10-09  Chris Blumenberg  <cblu@apple.com>
3505
3506         Fixed: 
3507         <rdar://problem/3625352> up and down arrow and page up/down keys don't work to scroll overflow:auto/scroll/overlay areas
3508         <rdar://problem/3397658> scroll wheel does not work to scroll overflow:auto/scroll/overlay areas (RSS)
3509
3510         Reviewed by hyatt.
3511
3512         * Plugins.subproj/WebBaseNetscapePluginStream.m:
3513         (-[WebBaseNetscapePluginStream initWithRequestURL:pluginPointer:notifyData:sendNotification:]): fixed typo in comment
3514         * Plugins.subproj/WebNetscapePluginStream.m:
3515         (-[WebNetscapePluginStream initWithRequest:pluginPointer:notifyData:sendNotification:]): ditto
3516         * WebView.subproj/WebFramePrivate.h:
3517         * WebView.subproj/WebFrameView.m:
3518         (-[WebFrameView&n