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