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