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