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