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