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