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