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