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