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