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