WebKit/ChangeLog

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