WebKit/ChangeLog

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