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