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