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