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