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