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