44e60fb48a60ba8933cf3f044e12c1a7f4ca5a9c
[WebKit-https.git] / WebKit / ChangeLog
1 2004-11-18  Chris Blumenberg  <cblu@apple.com>
2
3         * WebView.subproj/WebHTMLRepresentation.m:
4         (-[WebHTMLRepresentation currentForm]): removed stray ";"
5
6 2004-11-18  Chris Blumenberg  <cblu@apple.com>
7
8         Fixed development build failure.
9
10         * Misc.subproj/WebIconDatabase.m:
11         (+[WebIconDatabase sharedIconDatabase]): call LOG not Log
12
13 2004-11-18  Chris Blumenberg  <cblu@apple.com>
14
15         <rdar://problem/3885708> save memory in icon DB by not using NSSets when holding 1 object
16         
17         Reviewed by sullivan.
18
19         * Misc.subproj/WebIconDatabase.m:
20         (+[WebIconDatabase sharedIconDatabase]): added timing code
21         (-[WebIconDatabase _clearDictionaries]): new
22         (-[WebIconDatabase _loadIconDictionaries]): call _clearDictionaries in 2 places before we bail, use _web_setObjectUsingSetIfNecessary:forKey: when adding site URLs to the iconURLToURLs dictionary
23         (-[WebIconDatabase _updateFileDatabase]): fixed comment
24         (-[WebIconDatabase _setIconURL:forURL:]): use _web_setObjectUsingSetIfNecessary:forKey: when adding site URLs to the iconURLToURLs dictionary
25         (-[WebIconDatabase _releaseIconForIconURLString:]): handle NSString objects retured from iconURLToURLs
26         (-[NSMutableDictionary _web_setObjectUsingSetIfNecessary:forKey:]): new, puts a set on the dictionary when there are 2 or more object for s key
27
28 2004-11-17  Richard Williamson   <rjw@apple.com>
29
30         Fixed <rdar://problem/3885073> REGRESSION: Tab images at top of news.com.com replicated and squished
31
32         Correctly account for scaled image size and clipping.
33
34         Reviewed by Maciej.
35
36         * WebCoreSupport.subproj/WebImageData.h:
37         * WebCoreSupport.subproj/WebImageData.m:
38         * WebCoreSupport.subproj/WebImageRenderer.m:
39         (-[WebImageRenderer drawImageInRect:fromRect:compositeOperator:context:]):
40
41 2004-11-17  Maciej Stachowiak  <mjs@apple.com>
42
43         Reviewed by Richard.
44
45         <rdar://problem/3885076> Don't make IDN calls for all-ascii URLs to save about 3 pages
46         at Safari startup.
47         
48         * Misc.subproj/WebNSURLExtras.m:
49         (mapHostNames): If encoding and not decoding, then bail early if the URL is all ascii.
50         (-[NSString _web_mapHostNameWithRange:encode:makeString:]): Remove earlier special-case
51         check for localhost, no longer needed.
52
53 2004-11-17  Richard Williamson   <rjw@apple.com>
54
55         Fixed <rdar://problem/3863601> Legacy font cache code in [WebTextRendererFactory createSharedFactory] may be unnecesary
56
57         and added call to SPI for
58
59         <rdar://problem/3884448> WebKit should turn on CG local font cache
60
61         currently disabled until a Tiger build shows up with the SPI.
62
63         Reviewed by David Harrison.
64
65         * WebCoreSupport.subproj/WebTextRendererFactory.m:
66         (+[WebTextRendererFactory createSharedFactory]):
67
68 2004-11-17  Richard Williamson   <rjw@apple.com>
69
70         Fixed <rdar://problem/3882212> REGRESSION: Images clipped instead of scaled
71         Fixed <rdar://problem/3884088> Crash terminating image load
72
73         Also added code to turn off color correction for images created
74         via CGImageSources.  This code is currently disabled because CG  
75         can't change the color space of images loaded progressively.
76         Further, according to Dave Hayward, CG will no longer attempt
77         to color correct images that don't have embedded profiles as of
78         Tiger 8A306.
79
80         Reviewed by Chris.
81
82         * WebCoreSupport.subproj/WebImageData.m:
83         (-[WebImageData _commonTermination]):
84         (-[WebImageData dealloc]):
85         (-[WebImageData _invalidateImageProperties]):
86         (-[WebImageData imageAtIndex:]):
87         (-[WebImageData incrementalLoadWithBytes:length:complete:]):
88         (-[WebImageData propertiesAtIndex:]):
89
90 2004-11-16  Chris Blumenberg  <cblu@apple.com>
91
92         Fixed: <rdar://problem/3882034> REGRESSION: Context menu incorrect for PDF content
93
94         Reviewed by darin.
95
96         * WebView.subproj/WebPDFView.m:
97         (-[WebPDFView hitTest:]): return self if the current event is a context menu event
98         (-[WebPDFView menuForEvent:]): use the PDFView subview
99
100 2004-11-15  Chris Blumenberg  <cblu@apple.com>
101
102         Fixed: <rdar://problem/3880410> save 5 dirty pages by soft-linking against PDFKit framework
103
104         Reviewed by john.
105
106         * WebKit.pbproj/project.pbxproj:
107         * WebView.subproj/WebPDFRepresentation.m:
108         (+[WebPDFRepresentation PDFDocumentClass]): new
109         (-[WebPDFRepresentation finishedLoadingWithDataSource:]): use PDFDocumentClass
110         * WebView.subproj/WebPDFView.h:
111         * WebView.subproj/WebPDFView.m:
112         (+[WebPDFView PDFKitBundle]): new
113         (+[WebPDFView PDFViewClass]): new
114         (-[WebPDFView initWithFrame:]): create a PDFView subview
115         (-[WebPDFView dealloc]): release the PDFView subview
116         (-[WebPDFView PDFSubview]): new
117
118 2004-11-15  Chris Blumenberg  <cblu@apple.com>
119
120         Fixed: <rdar://problem/3879891> WebKit should link against PDFKit instead of Quartz
121
122         Reviewed by darin.
123
124         * WebKit.pbproj/project.pbxproj: link against PDFKit if it is present instead of Quartz.framework
125
126 2004-11-15  Richard Williamson   <rjw@apple.com>
127
128         Fixed missing retain of image property data.
129
130         Reviewed by John.
131
132         * WebCoreSupport.subproj/WebImageData.h:
133         * WebCoreSupport.subproj/WebImageData.m:
134         (-[WebImageData dealloc]):
135         (-[WebImageData _invalidateImages]):
136         (-[WebImageData imageAtIndex:]):
137         (-[WebImageData propertiesAtIndex:]):
138         (-[WebImageData _frameDuration]):
139
140 2004-11-15  Richard Williamson   <rjw@apple.com>
141
142         Cache image properties and frame durations.
143         Create NSImage and TIFF representations from CGImage, lazily, as needed for
144         dragging and element info dictionary.
145
146         Reviewed by John.
147
148         * WebCoreSupport.subproj/WebImageData.h:
149         * WebCoreSupport.subproj/WebImageData.m:
150         (-[WebImageData dealloc]):
151         (-[WebImageData size]):
152         (-[WebImageData propertiesAtIndex:]):
153         (-[WebImageData _frameDurationAt:]):
154         (-[WebImageData _frameDuration]):
155         * WebCoreSupport.subproj/WebImageRenderer.h:
156         * WebCoreSupport.subproj/WebImageRenderer.m:
157         (-[WebImageRenderer dealloc]):
158         (-[WebImageRenderer TIFFRepresentation]):
159         (-[WebImageRenderer image]):
160
161 2004-11-14  Maciej Stachowiak  <mjs@apple.com>
162
163         Reviewed by John.
164
165         <rdar://problem/3879226> WebKit needlessly uses extra memory to store icon refcounts as NSNumbers       
166
167         * Misc.subproj/WebIconDatabase.m:
168         (-[WebIconDatabase init]):
169         (-[WebIconDatabase _setIconURL:forURL:]):
170         (-[WebIconDatabase _retainIconForIconURLString:]):
171         (-[WebIconDatabase _releaseIconForIconURLString:]):
172         (-[WebIconDatabase _retainFutureIconForURL:]):
173         (-[WebIconDatabase _releaseFutureIconForURL:]):
174         * Misc.subproj/WebIconDatabasePrivate.h:
175
176 2004-11-15  John Sullivan  <sullivan@apple.com>
177
178         Reviewed by Ken.
179         
180         - fixed <rdar://problem/3879513> leak in [WebArchive _propertyListRepresentation] copying HTML to pasteboard
181
182         * WebView.subproj/WebArchive.m:
183         (-[WebArchive _propertyListRepresentation]):
184         the array holding the subresources was not released after use, oops!
185
186 2004-11-12  Chris Blumenberg  <cblu@apple.com>
187
188         Fixed: <rdar://problem/3874577> Opening restricted (parental) content in new window/tab reveals Safari's "Resources" folder
189         
190         Reviewed by john.
191
192         * WebView.subproj/WebDefaultContextMenuDelegate.m:
193         (-[WebDefaultUIDelegate openFrameInNewWindow:]): use the unreachable URL if there is one
194
195 === Safari-171 ===
196
197 2004-11-11  Richard Williamson   <rjw@apple.com>
198
199         Report actual size (not partial size) but use partial size
200         when drawing.
201
202         Reviewed by Maciej.
203
204         * WebCoreSupport.subproj/WebImageData.h:
205         * WebCoreSupport.subproj/WebImageData.m:
206         (-[WebImageData size]):
207
208 2004-11-11  Darin Adler  <darin@apple.com>
209
210         Reviewed by John.
211
212         - added _wasFirstResponderAtMouseDownTime method to bridge so we can fix
213           <rdar://problem/3846152> REGRESSION (125-166): can't drag text out of <input type=text> fields
214           with a subsequent change to WebCore.
215
216         * WebCoreSupport.subproj/WebBridge.m:
217         (wasFirstResponderAtMouseDownTime:): Added. Calls _wasFirstResponderAtMouseDownTime
218         on the WebHTMLView.
219         (_getPreSmartSet): Move global inside the function, add (void) for cleanliness.
220         (_getPostSmartSet): Ditto.
221
222         * WebView.subproj/WebHTMLView.m:
223         (-[WebHTMLViewPrivate dealloc]): Release firstResponderAtMouseDownTime.
224         (-[WebHTMLView _setMouseDownEvent:]): Early exit if event is not changing.
225         Set firstResponderAtMouseDownTime to the first responder.
226         (-[WebHTMLView mouseDown:]): Release firstResponderAtMouseDownTime after handling
227         the mouseDown event.
228         (-[WebHTMLView _wasFirstResponderAtMouseDownTime:]): Added. Uses the
229         firstResponderAtMouseDownTime field.
230         * WebView.subproj/WebHTMLViewInternal.h: Added firstResponderAtMouseDownTime field
231         and _wasFirstResponderAtMouseDownTime method.
232
233         * English.lproj/StringsNotToBeLocalized.txt: Update for recent changes.
234
235 2004-11-11  Richard Williamson   <rjw@apple.com>
236
237         Reviewed by Chris.
238
239         Work-around to minimize impact of 3876764.  Cache frame durations
240         after first call.  So we'll still leak 1K for each animated
241         image, but that's better than 1K each time the frame is drawn!
242         * WebCoreSupport.subproj/WebImageData.h:
243         * WebCoreSupport.subproj/WebImageData.m:
244         (-[WebImageData _frameDuration]):
245
246         Simplified animation cleanup code.  Fixed leak due to 
247         incorrect key passed to CFDictionaryRemoveValue.
248         
249         (+[WebImageData stopAnimationsInView:]):
250         (-[WebImageData addAnimatingRenderer:inView:]):
251         (-[WebImageData removeAnimatingRenderer:]):
252         (-[WebImageData _stopAnimation]):
253
254 2004-11-11  Darin Adler  <darin@apple.com>
255
256         Reviewed by Maciej.
257
258         - fixed <rdar://problem/3605906> Flash scrolled off the top and bottom cause CPU spin when combined with something dirty on the visible part of the page
259
260         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView drawRect:]):
261         Work around AppKit bug by using rectangles from getRectsBeingDrawn:count: instead of
262         using the passed-in rectangle.
263
264 2004-11-11  Richard Williamson   <rjw@apple.com>
265
266         Work-arounds to make new ImageIO code work correctly.  Still
267         disabled for now.  Requires at least Tiger 300.  Testing does
268         show a 3% improvement in PLT tests!  That's huge!
269
270         Reviewed by John.
271
272         * WebCoreSupport.subproj/WebImageData.m:
273         (-[WebImageData imageAtIndex:]):
274         (-[WebImageData incrementalLoadWithBytes:length:complete:]):
275         (-[WebImageData isNull]):
276
277 2004-11-10  Chris Blumenberg  <cblu@apple.com>
278
279         Fixed: <rdar://problem/3396872> ICONS: icon DB inconsistencies can cause slowness during startup, idle and quit
280
281         Reviewed by john.
282
283         * Misc.subproj/WebFileDatabase.m:
284         (-[WebFileDatabase objectForKey:]): added logging code
285         * Misc.subproj/WebIconDatabase.m:
286         (-[WebIconDatabase init]): use alloc, init rather than autorelease, retain
287         (-[WebIconDatabase _loadIconDictionaries]): use 1 object for mapping icon URLs to site URLs and vice versa rather than 3. This avoids inconsistencies and is faster.
288         (-[WebIconDatabase _updateFileDatabase]): write 1 object out
289
290 2004-11-09  David Hyatt  <hyatt@apple.com>
291
292         Fix for 3873234, Safari UI is unresponsive when parsing multiple HTML docs and 3873233, Safari hangs when
293         loading large local files.
294         
295         Reviewed by mjs
296
297         * WebCoreSupport.subproj/WebBridge.m:
298         (-[WebBridge tokenizerProcessedData]):
299         * WebView.subproj/WebDataSource.m:
300         (-[WebDataSource _receivedMainResourceError:complete:]):
301         (-[WebDataSource isLoading]):
302
303 2004-11-09  Richard Williamson   <rjw@apple.com>
304
305         Fixed <rdar://problem/3870964> 8A300: Safari not recognizing a PDF link (it displays raw data)
306
307         Add "text/pdf" as an acceptable PDF MIME type.
308
309         Reviewed by Chris.
310
311         * WebView.subproj/WebDataSource.m:
312         (+[WebDataSource _repTypesAllowImageTypeOmission:]):
313         * WebView.subproj/WebFrameView.m:
314         (+[WebFrameView _viewTypesAllowImageTypeOmission:]):
315
316 2004-11-08  Chris Blumenberg  <cblu@apple.com>
317
318         Fixed: <rdar://problem/3783904> Return key behavior is confusingly different between popup menus and autofill menus
319
320         Reviewed by john.
321
322         * WebCoreSupport.subproj/WebBridge.m:
323         (-[WebBridge control:textView:shouldHandleEvent:]): new
324         * WebView.subproj/WebFormDelegate.h:
325         * WebView.subproj/WebFormDelegate.m:
326         (-[WebFormDelegate control:textView:shouldHandleEvent:inFrame:]): new
327
328 2004-11-05  Chris Blumenberg  <cblu@apple.com>
329
330         Fixed: <rdar://problem/3854218> Safari is sometimes really slow because of increased null events to plug-ins
331
332         * Plugins.subproj/WebBaseNetscapePluginView.m: reverted null event interval to 0.02
333
334 2004-11-05  Chris Blumenberg  <cblu@apple.com>
335
336         Fixed: <rdar://problem/3838413> REGRESSION (Mail): "Smart" word paste adds spaces before/after special characters
337
338         Reviewed by rjw.
339
340         * WebCoreSupport.subproj/WebBridge.m:
341         (_getPreSmartSet): copied from AppKit
342         (_getPostSmartSet): ditto
343         (-[WebBridge isCharacterSmartReplaceExempt:isPreviousCharacter:]): new
344
345 2004-11-05  Richard Williamson   <rjw@apple.com>
346
347         Fixed <rdar://problem/3810702> _checkNavigationPolicyForRequest:dataSource:formState:andCall:withSelector: ASSERTS when reentered from Xcode's man page viewer
348
349         Reviewed by Maciej (a long time ago).
350
351         * WebView.subproj/WebFrame.m:
352         (-[WebFrame _loadDataSource:withLoadType:formState:]):
353
354         Fixed <rdar://problem/3845307> WebKit needs to export _HIWebViewRegisterClass so HIWebViews can work in Carbon nib files
355
356         As suggested in the bug, the fix is to actually call 
357         HIWebViewRegisterClass in WebKitInitForCarbon, rather than
358         exporting the symbol.
359
360         Reviewed by Chris.
361
362         * Carbon.subproj/CarbonUtils.m:
363         (WebInitForCarbon):
364         * Carbon.subproj/HIWebView.m:
365         * WebKit.pbproj/project.pbxproj:
366
367 === Safari-170 ===
368
369 2004-11-05  Darin Adler  <darin@apple.com>
370
371         Reviewed by Chris.
372
373         - fixed <rdar://problem/3857151> Assertion failure in "trackingRectOwner" while moving mouse over Slashdot.org page
374
375         * WebView.subproj/WebHTMLView.m:
376         (-[WebHTMLView _addTrackingRect:owner:userData:assumeInside:useTrackingNum:]): Changed to no longer call
377         addTrackingRect to do the work for consistency with the new method below. Not too much copied and pasted code.
378         (-[WebHTMLView _addTrackingRects:owner:userDataList:assumeInsideList:trackingNums:count:]): Added an override
379         for this new method in Tiger. No harm in implementing it on Panther, although it won't be called.
380         (-[WebHTMLView _removeTrackingRects:count:]): Ditto.
381
382 2004-11-04  David Hyatt  <hyatt@apple.com>
383
384         Make sure the dominant line direction is properly set for RTL runs so that spaces will reverse.
385
386         Change xHeight to measure the ascent of the x glyph, since the xHeight metrics appear to be
387         totally bogus in both CG and AppKit.
388         
389         Reviewed by darin
390
391         * WebCoreSupport.subproj/WebTextRenderer.m:
392         (-[WebTextRenderer xHeight]):
393         (-[WebTextRenderer _createATSUTextLayoutForRun:style:]):
394         (-[WebTextRenderer _trapezoidForRun:style:atPoint:]):
395         (-[WebTextRenderer _ATSU_drawHighlightForRun:style:geometry:]):
396         (-[WebTextRenderer _ATSU_drawRun:style:geometry:]):
397         (-[WebTextRenderer _ATSU_pointToOffset:style:position:reversed:includePartialGlyphs:]):
398
399 2004-11-02  Maciej Stachowiak  <mjs@apple.com>
400
401         Reviewed by Dave Hyatt (when I originally coded it).
402
403         Redid WebKit part of fix for:
404
405         <rdar://problem/3759187> REGRESSION (Mail): implement firstRectForCharacterRange:
406        
407         * WebView.subproj/WebHTMLView.m:
408         (-[WebHTMLView firstRectForCharacterRange:]): Call the appropriate new bridge method,
409         and translate to screen coordinates.
410
411 2004-11-02  John Sullivan  <sullivan@apple.com>
412
413         Reviewed by Hyatt.
414         
415         - [NSFont menuFontOfSize:], called from WebStringTruncator, was taking > 9% of the time creating a 
416         very large bookmarks menu, so I cached this one NSFont object.
417
418         * Misc.subproj/WebStringTruncator.m:
419         (defaultMenuFont):
420         new function, caches the font used when no font is specified
421         (+[WebStringTruncator centerTruncateString:toWidth:]):
422         call new function
423
424 2004-11-02  Ken Kocienda  <kocienda@apple.com>
425
426         Reviewed by Hyatt
427
428         WebCore now implements a command to insert a block in response to typing a return key, and
429         some names were improved in the course of this work.
430
431         * WebView.subproj/WebHTMLView.m:
432         (-[WebHTMLView insertNewline:]): Now calls insertLineBreak on bridge object.
433         (-[WebHTMLView insertLineBreak:]): New method.
434         (-[WebHTMLView insertParagraphSeparator:]): Now implemented.
435         * WebView.subproj/WebView.m: 
436
437 === Safari-169 ===
438
439 2004-10-29  Chris Blumenberg  <cblu@apple.com>
440
441         * WebKit.exp: added _WebPlugInModeKey, forgot to add it earlier
442
443 2004-10-29  Darin Adler  <darin@apple.com>
444
445         - fixed <rdar://problem/3855573> Remove reference to "WebScriptMethods" from WebScriptObject.h comments
446
447         * Plugins.subproj/WebScriptObject.h: Removed unneeded #ifdef protection for multiple includes (since
448         this is an Objective-C header and we use #import for those). Fixed comments as requested in the bug
449         report to match the contents of the file.
450
451 2004-10-27  Ken Kocienda  <kocienda@apple.com>
452
453         Reviewed by Chris
454
455         Added new SPI for Mail so it can get the behavior it needs when the user hits
456         the return key with the selection in quoted content.
457
458         * WebView.subproj/WebView.m
459         * WebView.subproj/WebViewPrivate.h
460
461 2004-10-26  Chris Blumenberg  <cblu@apple.com>
462
463         Fixed exception that Darin encountered in Mail.
464
465         Reviewed by darin.
466
467         * Plugins.subproj/WebPluginController.m:
468         (+[WebPluginController plugInViewWithArguments:fromPluginPackage:]): if the plug-in returns a nil view, return nil
469
470 2004-10-25  Chris Blumenberg  <cblu@apple.com>
471
472         Darin made an internal notification have the Web prefix.
473
474         Reviewed by me.
475
476         * Plugins.subproj/WebBaseNetscapePluginView.m:
477         (-[WebBaseNetscapePluginView addWindowObservers]):
478         (-[WebBaseNetscapePluginView removeWindowObservers]):
479         (ConsoleConnectionChangeNotifyProc):
480
481 2004-10-25  John Sullivan  <sullivan@apple.com>
482
483         Reviewed by Chris.
484         
485         - Cleanup from fix for <rdar://problem/3851676> bookmarks should not hold onto a WebHistoryItem object;
486           eliminated notificationsSuppressed mechanism, which was used only by WebBookmark
487
488         * History.subproj/WebHistoryItem.m:
489         removed notificationsSuppressed ivar from private data object
490         (-[WebHistoryItem setAlternateTitle:]):
491         remove notificationsSuppressed guard
492         (-[WebHistoryItem setURLString:]):
493         ditto
494         (-[WebHistoryItem setOriginalURLString:]):
495         ditto
496         (-[WebHistoryItem setTitle:]):
497         ditto
498         (-[WebHistoryItem _setLastVisitedTimeInterval:]):
499         ditto
500         (-[WebHistoryItem setNotificationsSuppressed:]):
501         removed this method
502         (-[WebHistoryItem notificationsSuppressed]):
503         ditto
504         
505         * History.subproj/WebHistoryItemPrivate.h:
506         removed notificationsSuppressed and setNotificationsSuppressed
507
508 2004-10-22  Chris Blumenberg  <cblu@apple.com>
509
510         Fixed: <rdar://problem/3851491> installedPlugins being called for a page without plugins
511
512         Reviewed by mjs.
513
514         * WebView.subproj/WebFrameView.m:
515         (+[WebFrameView _canShowMIMETypeAsHTML:]): call _viewTypesAllowImageTypeOmission instead of using ivar since the ivar is nil until _viewTypesAllowImageTypeOmission is called, this was causing [WebView canShowMIMEType:] to check plug-ins 
516
517 === Safari-168 ===
518
519 2004-10-22  Ken Kocienda  <kocienda@apple.com>
520
521         Reviewed by me
522
523         * WebKit.pbproj/project.pbxproj: Add GCC_ENABLE_OBJC_GC and GCC_FAST_OBJC_DISPATCH flags.
524
525 2004-10-21  Darin Adler  <darin@apple.com>
526
527         Reviewed by John.
528
529         - fixed <rdar://problem/3847994> REGRESSION: reproducible exception in WebImageRenderer releasePatternColor; afterwards get crash or no more browsing
530
531         * WebCoreSupport.subproj/WebImageRenderer.m:
532         (-[WebInternalImage createRendererIfNeeded]): Replaced retainOrCopyIfNeeded with this.
533         This returns nil if a copied renderer isn't needed, and returns a new renderer if a copy is.
534         The old version was sometimes returning a WebInternalImage and other times a WebImageRenderer.
535         (-[WebImageRenderer retainOrCopyIfNeeded]): Returns the result of createRendererIfNeeded
536         or retains self and returns self.
537
538 2004-10-20  Darin Adler  <darin@apple.com>
539
540         Reviewed by Dave.
541
542         - fixed <rdar://problem/3470715> Pattern cache can get huge with use of css background-image in Safari
543
544         * WebCoreSupport.subproj/WebImageRenderer.h: Change WebImageRenderer to be a subclass of NSObject
545         rather than NSImage and contain a pointer to a WebInternalImage.
546         * WebCoreSupport.subproj/WebImageRenderer.m:
547         (-[WebInternalImage releasePatternColor]): Added. Releases patternColor.
548         (-[WebImageRenderer initWithMIMEType:]): Added. Makes WebInternalImage and then self.
549         (-[WebImageRenderer initWithData:MIMEType:]): Ditto.
550         (-[WebImageRenderer initWithContentsOfFile:]): Ditto.
551         (-[WebImageRenderer dealloc]): Added. Calls releasePatternColor and then releases WebInternalImage.
552         (-[WebImageRenderer image]): Added. Returns pointer to image.
553         (-[WebImageRenderer MIMEType]): Added. Calls through to image.
554         (-[WebImageRenderer TIFFRepresentation]): Ditto.
555         (-[WebImageRenderer frameCount]): Ditto.
556         (-[WebImageRenderer setOriginalData:]): Added. Sets image data pointer.
557         (+[WebImageRenderer stopAnimationsInView:]): Added. Calls through to image.
558         (-[WebImageRenderer incrementalLoadWithBytes:length:complete:]): Ditto.
559         (-[WebImageRenderer size]): Ditto.
560         (-[WebImageRenderer resize:]): Ditto.
561         (-[WebImageRenderer drawImageInRect:fromRect:]): Ditto.
562         (-[WebImageRenderer drawImageInRect:fromRect:compositeOperator:context:]): Ditto.
563         (-[WebImageRenderer stopAnimation]): Ditto.
564         (-[WebImageRenderer tileInRect:fromPoint:context:]): Ditto.
565         (-[WebImageRenderer isNull]): Ditto.
566         (-[WebImageRenderer retainOrCopyIfNeeded]): Ditto.
567         (-[WebImageRenderer increaseUseCount]): Ditto.
568         (-[WebImageRenderer decreaseUseCount]): Ditto.
569         (-[WebImageRenderer flushRasterCache]): Ditto.
570         (-[WebImageRenderer imageRef]): Ditto.
571         (-[WebImageRenderer copyWithZone:]): Ditto.
572
573         * Misc.subproj/WebNSViewExtras.m: (-[NSView _web_dragImage:rect:event:pasteboard:source:offset:]):
574         Update for slight changes to WebImageRenderer API.
575         * WebCoreSupport.subproj/WebImageRendererFactory.m:
576         (-[WebImageRendererFactory imageRendererWithMIMEType:]): Ditto.
577         (-[WebImageRendererFactory imageRendererWithData:MIMEType:]): Ditto.
578         (-[WebImageRendererFactory imageRendererWithSize:]): Ditto.
579         (-[WebImageRendererFactory imageRendererWithName:]): Ditto.
580         * WebView.subproj/WebImageView.m: (-[WebImageView image]): Ditto.
581
582 2004-10-20  Chris Blumenberg  <cblu@apple.com>
583         
584         Fixed: <rdar://problem/3846943> REGRESSION: JNLP files are rendered instead of downloaded
585
586         Reviewed by john.
587
588         * Plugins.subproj/WebBasePluginPackage.h:
589         * Plugins.subproj/WebBasePluginPackage.m:
590         (-[WebBasePluginPackage isJavaPlugIn]): new
591         * Plugins.subproj/WebPluginDatabase.m:
592         (-[WebPluginDatabase refresh]): don't register the Java plug-in for a document view since Java file should be downloaded when not embedded.
593
594 2004-10-20  Chris Blumenberg  <cblu@apple.com>
595
596         Fixed:
597         <rdar://problem/3842030> WebKit needs to pass the mode (NP_FULL, NP_EMBED, etc) when calling plugInViewWithArguments
598         <rdar://problem/3792852> Safari is loading the new QuickTime Cocoa plugin on Panther
599
600         Reviewed by darin.
601
602         * Plugins.subproj/WebPluginDocumentView.m:
603         (-[WebPluginDocumentView setDataSource:]): pass "full" as the mode
604         * Plugins.subproj/WebPluginPackage.m:
605         (-[WebPluginPackage initWithPath:]): load plug-in with the "webplugin" extension
606         * WebCoreSupport.subproj/WebBridge.m:
607         (-[WebBridge pluginViewWithPackage:attributeNames:attributeValues:baseURL:]): pass "embed" as the mode
608         * WebKit.pbproj/project.pbxproj:
609
610 2004-10-19  Vicki Murley <vicki@apple.com>
611
612         - bump WebKit version to 167.1, so that we can do a quick dot submission for <rdar://problem/3843951> 
613
614         * WebKit.pbproj/project.pbxproj:
615
616 2004-10-19  Darin Adler  <darin@apple.com>
617
618         Change suggested by Maciej during code review.
619
620         * WebCoreSupport.subproj/WebTextRenderer.m: Changed rounding hack table to be const so it can be in shared instead
621         of private memory, and doesn't require an initialization function.
622         (+[WebTextRenderer initialize]): Removed initialization.
623
624 2004-10-19  Darin Adler  <darin@apple.com>
625
626         Reviewed by Maciej.
627
628         - fixed <rdar://problem/3838934> Safari stops loading pages after rangeOfCharacterFromSet nil argument exception
629         - fixed <rdar://problem/3843951> REGRESSION (166-167): Safari crashes in widthForNextCharacter (belkin.com, at startup for others)
630         - fixed <rdar://problem/3841049> REGRESSION (109-110): control characters render as square boxes
631
632         * WebCoreSupport.subproj/WebTextRenderer.m:
633         (isSpace): Merged in isAlternateSpace, never used.
634         (setupRoundingHackCharacterTable): Fixed size of table, was 1 entry too short. Got rid of unneeded call to bzero,
635         since globals start out zeroed automatically.
636         (isRoundingHackCharacter): Fixed backwards logic causing the crash in widthForNextCharacter.
637         Also removed explicit compare with 1; check for non-zero is just fine.
638         (fontContainsString): Change code so we'll just skip the font if the covered character set returns nil rather than
639         throwing an exception like the old version did. This should make bug 3838934 go away, although perhaps covering up
640         the underlying problem.
641         (-[WebTextRenderer _convertCharacters:length:toGlyphs:]): Removed unused skipControlCharacters: parameter and also
642         the unnecessary code to copy the buffer to change newline characters and non-break spaces to spaces.
643         (-[WebTextRenderer _convertUnicodeCharacters:length:toGlyphs:]): Removed unused local.
644         (-[WebTextRenderer _extendCharacterToGlyphMapToInclude:]): Added code to set up special cases for control characters,
645         \n and non-break spaces.
646         (-[WebTextRenderer _createATSUTextLayoutForRun:]): Added comment about the cases this code does not handle that
647         are handled by the CG case.
648         (widthForNextCharacter): Call isSpace instead of checking specifically for the space character here. The old code
649         would not handle cases with '\n' coming across from WebCore properly.
650
651 2004-10-18  Chris Blumenberg  <cblu@apple.com>
652
653         Fixed: <rdar://problem/3840916> GC: -[WebNetscapePluginPackage initWithPath:] leaks an NSURL
654
655         Reviewed by kevin.
656
657         * Plugins.subproj/WebNetscapePluginPackage.m:
658         (-[WebNetscapePluginPackage initWithPath:]): use executablePath on NSBundle instead of CFBundleCopyExecutableURL
659
660 2004-10-18  Chris Blumenberg  <cblu@apple.com>
661
662         * DOM.subproj/DOMPrivate.h: change to copied header that was never committed
663
664 2004-10-18  John Sullivan  <sullivan@apple.com>
665
666         Reviewed by Ken.
667         
668         - fixed <rdar://problem/3810183> Make WebHTMLView respect return value of webView:doCommandBySelector:
669
670         * WebView.subproj/WebHTMLView.m:
671         (-[WebHTMLView doCommandBySelector:]):
672         only do default action if delegate returns NO; this works with Mail as of Tiger 8A275.
673
674 === Safari-167 ===
675
676 2004-10-14  Ken Kocienda  <kocienda@apple.com>
677
678         Reviewed by John
679
680         Final fix for these bugs:
681         
682         <rdar://problem/3806306> HTML editing puts spaces at start of line
683         <rdar://problem/3814252> HTML editing groups space with word causing wrapping
684
685         This change sets some new CSS properties that have been added to WebCore to 
686         enable whitespace-handling and line-breaking features that make WebView work
687         more like a text editor.
688
689         * WebView.subproj/WebHTMLRepresentation.m:
690         (-[WebHTMLRepresentation finishedLoadingWithDataSource:]): Turn on special editing
691         CSS properties when loading an HTML document into a WebView that is editable.
692         * WebView.subproj/WebView.m:
693         (-[WebView setEditable:]): Add and remove special editing CSS properties in current
694         document being displayed.
695
696 2004-10-14  Richard Williamson   <rjw@apple.com>
697
698         Fixed <rdar://problem/3823026> making isRoundingHackCharacter use -O3 and an 8-bit lookup-table will speed "XBS" test up by 3% (actually < 1%)
699
700         Careful testing shows a small performance gain on very large text files.  
701         I saw large variations in timings, but taking the lowest PLT timing 
702         with and without this change showed a 0.9% gain.  Note the cvs-base showed
703         no improvement.  The improvement was for the large page attached to the
704         bug.
705
706         Reviewed by Ken.
707
708         * WebCoreSupport.subproj/WebTextRenderer.m:
709         (setupRoundingHackCharacterTable):
710         (isRoundingHackCharacter):
711         (+[WebTextRenderer initialize]):
712
713 2004-10-14  Ken Kocienda  <kocienda@apple.com>
714
715         Reviewed by me
716
717         Fix build breakage. These three functions need to return the values from their
718         calls to WebCGColorSpaceCreateXXX.
719
720         * WebCoreSupport.subproj/WebGraphicsBridge.m:
721         (-[WebGraphicsBridge createRGBColorSpace])
722         (-[WebGraphicsBridge createGrayColorSpace])
723         (-[WebGraphicsBridge createCMYKColorSpace])
724
725 2004-10-13  Richard Williamson   <rjw@apple.com>
726
727         Addressed concerns in <rdar://problem/3803117> RESP: High complexity in icu uidna_IDNToASCII called by [NSString(WebNSURLExtras) _web_mapHostNameWithRange:encode:makeString:]
728
729         In practice I saw NO improvement in performance.  Although,
730         special-case tests could possibly show improvement.  Anyway,
731         the changes don't hurt performance.
732
733         Reviewed by Maciej.
734
735         * Misc.subproj/WebNSURLExtras.m:
736         (-[NSString _web_mapHostNameWithRange:encode:makeString:]):
737
738 2004-10-13  Maciej Stachowiak  <mjs@apple.com>
739
740         Reviewed by Ken.
741
742         <rdar://problem/3824626> Change to do colormatching for DeviceRGB colorspace causes ~11% Safari slowdown
743         
744         - I fixed this by turning off all colormatching for WebKit
745         content. We might turn it back on later. For now, it's possible to
746         turn it on temporarily by defining COLORMATCH_EVERYTHING.
747         
748         * WebCoreSupport.subproj/WebGraphicsBridge.m:
749         (-[WebGraphicsBridge setFocusRingStyle:radius:color:]):
750         (-[WebGraphicsBridge additionalPatternPhase]):
751         (-[WebGraphicsBridge createRGBColorSpace]):
752         (-[WebGraphicsBridge createGrayColorSpace]):
753         (-[WebGraphicsBridge createCMYKColorSpace]):
754         * WebCoreSupport.subproj/WebImageData.m:
755         * WebCoreSupport.subproj/WebImageRenderer.h:
756         * WebCoreSupport.subproj/WebImageRenderer.m:
757         (-[WebImageRenderer _adjustSizeToPixelDimensions]):
758         (-[WebImageRenderer incrementalLoadWithBytes:length:complete:]):
759         (-[WebImageRenderer _adjustColorSpace]):
760         (-[WebImageRenderer drawClippedToValidInRect:fromRect:]):
761         (-[WebImageRenderer tileInRect:fromPoint:context:]):
762         (_createImageRef):
763         (WebCGColorSpaceCreateRGB):
764         (WebCGColorSpaceCreateGray):
765         (WebCGColorSpaceCreateCMYK):
766         * WebKitPrefix.h:
767
768 2004-10-13  Richard Williamson   <rjw@apple.com>
769
770         Don't fill background with transparency unless debug flag
771         is enabled.
772
773         Reviewed by Hyatt.
774
775         * WebView.subproj/WebHTMLView.m:
776         (-[WebHTMLView drawRect:]):
777
778 2004-10-12  Richard Williamson   <rjw@apple.com>
779
780         Fixed <rdar://problem/3829705> Need to remove filling w/ transparency when not drawing backgroundy.
781
782         Reviewed by Ken.
783
784         * WebView.subproj/WebHTMLView.m:
785         (-[WebHTMLView _transparentBackground]):
786         (-[WebHTMLView _setTransparentBackground:]):
787         (-[WebHTMLView drawRect:]):
788         * WebView.subproj/WebHTMLViewInternal.h:
789         * WebView.subproj/WebHTMLViewPrivate.h:
790
791 2004-10-11  Chris Blumenberg  <cblu@apple.com>
792
793         Fixed: <rdar://problem/3802039> 8A259: Can't use Grab services to grab selection from screen
794
795         Reviewed by john.
796
797         * WebView.subproj/WebHTMLView.m:
798         (+[WebHTMLView initialize]): register service "return types" which are types that can be inserted into a WebView
799         (-[WebHTMLView writeSelectionToPasteboard:types:]): service protocol method, be sure to only write specified types
800         (-[WebHTMLView readSelectionFromPasteboard:]): new, service protocol method, insert types
801         (-[WebHTMLView validRequestorForSendType:returnType:]): moved, handle return types
802
803 2004-10-11  Darin Adler  <darin@apple.com>
804
805         Reviewed by John.
806
807         - fixed <rdar://problem/3834130> nil-object-in-dictionary exception seen in -[WebView _elementAtWindowPoint:]
808
809         * WebView.subproj/WebView.m: (-[WebView _elementAtWindowPoint:]): Added a check for nil frame.
810
811 2004-10-11  Darin Adler  <darin@apple.com>
812
813         Reviewed by John.
814
815         - fixed <rdar://problem/3834166> <input type=file> sends onchange even when the same file is chosen twice
816
817         * WebCoreSupport.subproj/WebFileButton.m: (-[WebFileButton chooseFilename:]):
818         Do nothing if filename is the same as before.
819
820 2004-10-11  Ken Kocienda  <kocienda@apple.com>
821
822         Reviewed by Darin
823
824         * WebView.subproj/WebHTMLView.m:
825         (-[WebTextCompleteController doCompletion]): bridge call to get caret rect at a node
826         now takes an affinity: caretRectAtNode:offset:affinity:.
827
828 2004-10-10  Ken Kocienda  <kocienda@apple.com>
829
830         Reviewed by Chris
831
832         Fix for this bug:
833         
834         <rdar://problem/3814236> REGRESSION (Mail): Can't set the color of text in Mail compose window using drag/drag from color panel
835
836         * WebView.subproj/WebHTMLView.m:
837         (+[WebHTMLView _insertablePasteboardTypes]): Add NSColorPboardType to list.
838         (-[WebHTMLView _isNSColorDrag:]): New helper. Determines if drag is an NSColor drag.
839         (-[WebHTMLView draggingUpdatedWithDraggingInfo:actionMask:]): Add a case for NSColor drags,
840         else do what we did before.
841         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): Add a case for NSColor drags, which creates
842         a CSS style containing color info and calls the bridge to apply the style. Otherwise, do what we did before.
843
844 2004-10-11  Darin Adler  <darin@apple.com>
845
846         Reviewed by Ken.
847
848         - fixed <rdar://problem/3833848> REGRESSION (133-134): each keydown event is getting sent multiple times
849
850         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView performKeyEquivalent:]):
851         Don't send an event through WebCore if it has already been through once.
852
853 2004-10-10  John Sullivan  <sullivan@apple.com>
854
855         Reviewed by Ken.
856         
857         - fixed <rdar://problem/3777253> Crash in redirect mechanism trying to display error page for bad scheme
858
859         * WebView.subproj/WebMainResourceClient.m:
860         (-[WebMainResourceClient connection:willSendRequest:redirectResponse:]):
861         add retain/autorelease to the request returned from call to super. In this case, the return value
862         was being dealloc'ed before being returned.
863
864 2004-10-09  Chris Blumenberg  <cblu@apple.com>
865
866         Fixed: 
867         <rdar://problem/3625352> up and down arrow and page up/down keys don't work to scroll overflow:auto/scroll/overlay areas
868         <rdar://problem/3397658> scroll wheel does not work to scroll overflow:auto/scroll/overlay areas (RSS)
869
870         Reviewed by hyatt.
871
872         * Plugins.subproj/WebBaseNetscapePluginStream.m:
873         (-[WebBaseNetscapePluginStream initWithRequestURL:pluginPointer:notifyData:sendNotification:]): fixed typo in comment
874         * Plugins.subproj/WebNetscapePluginStream.m:
875         (-[WebNetscapePluginStream initWithRequest:pluginPointer:notifyData:sendNotification:]): ditto
876         * WebView.subproj/WebFramePrivate.h:
877         * WebView.subproj/WebFrameView.m:
878         (-[WebFrameView _bridge]): new
879         (-[WebFrameView scrollToBeginningOfDocument:]): call the bridge to scroll, if that fails, scroll the document view
880         (-[WebFrameView scrollToEndOfDocument:]): ditto
881         (-[WebFrameView _pageVertically:]): ditto
882         (-[WebFrameView _pageHorizontally:]): ditto
883         (-[WebFrameView _scrollLineVertically:]): ditto
884         (-[WebFrameView _scrollLineHorizontally:]): ditto
885         * WebView.subproj/WebHTMLView.m:
886         (-[WebHTMLView scrollWheel:]): call the bridge to scroll, if that fails, pass to next responder
887
888 === Safari-166 ===
889
890 2004-10-05  Chris Blumenberg  <cblu@apple.com>
891
892         Fixed: <rdar://problem/3827002> assertion failure in WebBaseNetscapePluginStream on abc.go.com
893
894         Reviewed by rjw.
895
896         * Plugins.subproj/WebBaseNetscapePluginStream.m:
897         (-[WebBaseNetscapePluginStream initWithRequestURL:pluginPointer:notifyData:sendNotification:]): avoid assertion failure in dealloc by temporarily setting isTerminated to YES in case we are released in this method
898         * Plugins.subproj/WebNetscapePluginStream.m:
899         (-[WebNetscapePluginStream initWithRequest:pluginPointer:notifyData:sendNotification:]): ditto
900
901 2004-10-05  John Sullivan  <sullivan@apple.com>
902
903         * WebCoreSupport.subproj/WebBridge.m:
904         (-[WebBridge pluginViewWithPackage:attributeNames:attributeValues:baseURL:]):
905         initialize "arguments" var to nil to satisfy compiler on deployment build.
906
907 2004-10-05  Richard Williamson   <rjw@apple.com>
908
909         Fixed <rdar://problem/3825442> first click lost for Dashboard
910         Allow dashboard to force acceptsFirstMouse:
911
912         Reviewed by Chris.
913
914         * WebView.subproj/WebHTMLView.m:
915         (-[WebHTMLView acceptsFirstMouse:]):
916         * WebView.subproj/WebView.m:
917         (-[WebView _dashboardBehavior:]):
918         * WebView.subproj/WebViewInternal.h:
919         * WebView.subproj/WebViewPrivate.h:
920
921         * WebCoreSupport.subproj/WebImageRenderer.h:
922         Comment change only
923
924 2004-10-05  Chris Blumenberg  <cblu@apple.com>
925
926         Fixed: <rdar://problem/3760920> Need to record plugin view instances
927         
928         Reviewed by rjw.
929
930         * Plugins.subproj/WebPluginController.h:
931         * Plugins.subproj/WebPluginController.m:
932         (+[WebPluginController plugInViewWithArguments:fromPluginPackage:]): new, creates plug-in view and adds it to global list
933         (+[WebPluginController isPlugInView:]): new, checks if the plug-in view is in the global list
934         (-[WebPluginController destroyAllPlugins]): remove the plug-in from the global list 
935         * Plugins.subproj/WebPluginDocumentView.m:
936         (-[WebPluginDocumentView setDataSource:]): call [WebPluginController plugInViewWithArguments:fromPluginPackage:]
937         * WebCoreSupport.subproj/WebBridge.m:
938         (-[WebBridge pluginViewWithPackage:attributeNames:attributeValues:baseURL:]): ditto
939         * WebView.subproj/WebFrame.m:
940         (-[WebFrame _reloadForPluginChanges]): call [WebPluginController isPlugInView:]
941         * WebView.subproj/WebHTMLView.m:
942         (-[WebHTMLView addSubview:]): ditto
943
944 2004-10-05  David Hyatt  <hyatt@apple.com>
945
946         Fix to make selection more like NSTextView.  All gap painting is now done by WebCore, so WebKit no longer
947         needs to try to fill gaps around text.
948         
949         Reviewed by kocienda
950
951         * WebCoreSupport.subproj/WebTextRenderer.m:
952         (-[WebTextRenderer _CG_drawHighlightForRun:style:geometry:]):
953         (-[WebTextRenderer _ATSU_drawHighlightForRun:style:geometry:]):
954
955 2004-10-05  Darin Adler  <darin@apple.com>
956
957         Reviewed by John.
958
959         - fixed <rdar://problem/3577255> custom file icon shows up upside down in <input type=file>
960
961         * WebCoreSupport.subproj/WebFileButton.m: (-[WebFileButton setFilename:]): Added a call to
962         setFlipped that fixes the problem, even though I don't know why.
963
964 2004-10-04  Darin Adler  <darin@apple.com>
965
966         Reviewed by Maciej.
967
968         - fixed <rdar://problem/3814237> REGRESSION (Mail): Copy/paste style does not set color in Mail compose window
969
970         * WebView.subproj/WebHTMLView.m:
971         (-[WebHTMLView _selectionStartFontAttributesAsRTF]): Changed to call new bridge method
972         named fontAttributesForSelectionStart, deleted the method this used to use, and renamed
973         this to have the word "start" in it.
974         (-[WebHTMLView copyFont:]): Updated for name change.
975
976         * English.lproj/StringsNotToBeLocalized.txt: Updated for recent changes.
977
978 2004-10-04  Chris Blumenberg  <cblu@apple.com>
979
980         * WebView.subproj/WebFrameInternal.h: removed constant declarations that I committed by mistake
981
982 2004-10-04  Chris Blumenberg  <cblu@apple.com>
983
984         Fixed: <rdar://problem/3798948> NPP_URLNotify is not called if plug-in calls NPN_*URLNotfy
985         Fixed a number of FIXME's related to notifying plug-ins of loaded pages.
986
987         Reviewed by rjw.
988
989         * Plugins.subproj/WebBaseNetscapePluginStream.h: 
990                 - replaced URL ivar with requestURL and responseURL ivars since we need to pass both to plug-ins
991                 - added sendNotification boolean. Relying on notifyData not being NULL was not information to know whether to call NPP_URLNotify or not.
992                 - added isTerminated boolean because determining whether or not stream.ndata is NULL is not enough to know if the stream has been cancelled.
993         * Plugins.subproj/WebBaseNetscapePluginStream.m:
994         (+[WebBaseNetscapePluginStream reasonForError:]): new, factored out from receivedError:
995         (-[WebBaseNetscapePluginStream initWithRequestURL:pluginPointer:notifyData:sendNotification:]): new
996         (-[WebBaseNetscapePluginStream dealloc]): release new ivars
997         (-[WebBaseNetscapePluginStream finalize]): added assert
998         (-[WebBaseNetscapePluginStream setRequestURL:]): new
999         (-[WebBaseNetscapePluginStream setResponseURL:]): new
1000         (-[WebBaseNetscapePluginStream startStreamResponseURL:expectedContentLength:lastModifiedDate:MIMEType:]): renamed, use responseURL as it basically did before
1001         (-[WebBaseNetscapePluginStream startStreamWithResponse:]): call renamed method
1002         (-[WebBaseNetscapePluginStream destroyStream]): 
1003                 - do nothing if terminated
1004                 - call NPP_StreamAsFile and NPP_DestroyStream if stream.ndata is not NULL
1005                 - call NPP_URLNotify if sendNotification is YES regardless of value of notifyData
1006         (-[WebBaseNetscapePluginStream receivedError:]): call reasonForError
1007         (-[WebBaseNetscapePluginStream deliverData]): use renamed ivar
1008         * Plugins.subproj/WebBaseNetscapePluginView.h:
1009                 - added observingFrameLoadNotification boolean
1010                 - renamed dictionary ivar to pendingFrameLoads which has WebFrame keys and WebPluginRequest values
1011         * Plugins.subproj/WebBaseNetscapePluginView.m:
1012         (-[WebBaseNetscapePluginView addFrameLoadObserver]): new
1013         (-[WebBaseNetscapePluginView removeFrameLoadObserver]): new
1014         (-[WebBaseNetscapePluginView stop]): call removeFrameLoadObserver
1015         (-[WebBaseNetscapePluginView initWithFrame:]): use renamed pendingFrameLoads ivar
1016         (-[WebBaseNetscapePluginView dealloc]): ditto
1017         (-[WebBaseNetscapePluginView requestWithURLCString:]): set referrer on the request just as IE does
1018         (-[WebBaseNetscapePluginView evaluateJavaScriptPluginRequest:]): 
1019                 - call NPP_URLNotify depending of value of sendNotification
1020                 - call new init method on WebBaseNetscapePluginStream rather then setting variables individually
1021         (-[WebBaseNetscapePluginView webFrame:didFinishLoadWithReason:]): new, calls NPP_URLNotify at the right time with the right value
1022         (-[WebBaseNetscapePluginView webFrame:didFinishLoadWithError:]): new, delegate method called from WebFrame
1023         (-[WebBaseNetscapePluginView loadPluginRequest:]): call addFrameLoadObserver
1024         (-[WebBaseNetscapePluginView loadRequest:inTarget:withNotifyData:sendNotification:]): take new sendNotification parameter and pass it
1025         (-[WebBaseNetscapePluginView getURLNotify:target:notifyData:]): pass YES for sendNotification
1026         (-[WebBaseNetscapePluginView getURL:target:]): pass NO for sendNotification
1027         (-[WebBaseNetscapePluginView _postURL:target:len:buf:file:notifyData:sendNotification:allowHeaders:]): take new sendNotification parameter and pass it
1028         (-[WebBaseNetscapePluginView postURLNotify:target:len:buf:file:notifyData:]): pass YES for sendNotification
1029         (-[WebBaseNetscapePluginView postURL:target:len:buf:file:]): pass NO for sendNotification
1030         (-[WebPluginRequest initWithRequest:frameName:notifyData:sendNotification:]): take new sendNotification parameter
1031         (-[WebPluginRequest sendNotification]): new
1032         * Plugins.subproj/WebBaseNetscapePluginViewPrivate.h:
1033         * Plugins.subproj/WebNetscapePluginEmbeddedView.m:
1034         (-[WebNetscapePluginEmbeddedView didStart]): set referrer on the request just as IE does
1035         * Plugins.subproj/WebNetscapePluginRepresentation.m:
1036         (-[WebNetscapePluginRepresentation receivedData:withDataSource:]): set the request URL on the stream
1037         * Plugins.subproj/WebNetscapePluginStream.h:
1038         * Plugins.subproj/WebNetscapePluginStream.m:
1039         (-[WebNetscapePluginStream initWithRequest:pluginPointer:notifyData:sendNotification:]): take new sendNotification parameter and pass it
1040         (-[WebNetscapePluginStream dealloc]): use renamed ivar
1041         (-[WebNetscapePluginStream start]): ditto
1042         * WebView.subproj/WebFrame.m:
1043         (-[WebFrame _setState:]): removed notification posting code. This was only used by WebBaseNetscapePluginView and it was the wrong notification to send.
1044         (-[WebFrame _checkLoadCompleteForThisFrame]): call internal load delegate to tell it that the load has finished
1045         (-[WebFrame _loadItem:withLoadType:]): ditto
1046         (-[WebFrame _continueFragmentScrollAfterNavigationPolicy:formState:]): ditto
1047         (-[WebFrame _setInternalLoadDelegate:]): new
1048         (-[WebFrame _internalLoadDelegate]): new
1049         * WebView.subproj/WebFrameInternal.h:
1050         * WebView.subproj/WebFramePrivate.h:
1051
1052 2004-10-04  Chris Blumenberg  <cblu@apple.com>
1053
1054         Fixed: <rdar://problem/3758113> REGRESSION: Macromedia ColdFusion page doesn't show main content
1055
1056         After bumping up our plug-in version, Flash now sends 2 CRLF's between the headers and body of their POST request. Our code was not prepared for this.
1057
1058         Reviewed by darin.
1059
1060         * Plugins.subproj/WebBaseNetscapePluginView.m:
1061         (-[NSData _web_locationAfterFirstBlankLine]): looks for 2 CRLF's, not for 2 LF's
1062
1063 2004-10-04  Darin Adler  <darin@apple.com>
1064
1065         Reviewed by Ken.
1066
1067         - fixed a potential storage leak when we turn on CGImageRef image rendering
1068
1069         * WebCoreSupport.subproj/WebImageRenderer.m: (-[WebImageRenderer dealloc]): Fix potential storage leak
1070         by adding [super dealloc], but leak was not real yet because the code is commented out.
1071
1072         - make paste style work with color as part of fix to <rdar://problem/3814237> REGRESSION (Mail):
1073           Copy/paste style does not set color in Mail compose window
1074
1075         * WebView.subproj/WebHTMLView.m:
1076         (-[WebHTMLView _selectionFontAttributes]): Change structure so it's easy to add more attributes.
1077         For now I haven't added any yet.
1078         (-[WebHTMLView _colorAsString:]): Moved this earlier in the file.
1079         (-[WebHTMLView _shadowAsString:]): Ditto.
1080         (-[WebHTMLView _styleFromFontAttributes:]): Add background color, foreground color, and text shadow.
1081
1082 2004-09-30  Richard Williamson   <rjw@apple.com>
1083
1084         Fixed <rdar://problem/3821215> NPN hasMethod and hasProperty functions should take NPObjects, not NPClass
1085
1086         Also changed dashboard regions dictionary to use "control"
1087         for scroller region label, instead of "scroller, per
1088         request from ouch. 
1089                 
1090         Reviewed by Chris.
1091
1092         * Plugins.subproj/npruntime.h:
1093         * WebView.subproj/WebView.m:
1094         (-[WebView _addScrollerDashboardRegions:from:]):
1095
1096 2004-09-30  Chris Blumenberg  <cblu@apple.com>
1097
1098         Fixed: <rdar://problem/3498680> switching back and forth between tabs stops calling anything in a plug-in
1099
1100         Reviewed by darin.
1101
1102         * Plugins.subproj/WebBaseNetscapePluginView.m:
1103         (-[WebBaseNetscapePluginView sendEvent:]): call setWindowIfNecessary because the window may have changed
1104         (-[WebBaseNetscapePluginView updateAndSetWindow]): new
1105         (-[WebBaseNetscapePluginView setWindowIfNecessary]): was setWindow, this method now just sets the window
1106         (-[WebBaseNetscapePluginView start]): call updateAndSetWindow
1107         (-[WebBaseNetscapePluginView viewDidMoveToWindow]): ditto
1108         (-[WebBaseNetscapePluginView viewHasMoved:]): ditto
1109
1110 2004-09-30  Chris Blumenberg  <cblu@apple.com>
1111
1112         Fixed: <rdar://problem/3498668> switching out of tab doesn't send loseFocusEvent to plug-in
1113
1114         Reviewed by rjw.
1115
1116         * Plugins.subproj/WebBaseNetscapePluginView.h:
1117         * Plugins.subproj/WebBaseNetscapePluginView.m:
1118         (-[WebBaseNetscapePluginView setHasFocus:]): new, sends events to plug-in
1119         (-[WebBaseNetscapePluginView becomeFirstResponder]): call setHasFocus
1120         (-[WebBaseNetscapePluginView resignFirstResponder]): ditto 
1121         (-[WebBaseNetscapePluginView viewWillMoveToWindow:]): ditto
1122
1123 2004-09-30  Chris Blumenberg  <cblu@apple.com>
1124
1125         Fixed:
1126         Assertion failure when loading standalone netscape plug-in content.
1127         Document loads of WebKit plug-in content should be cancelled since the plug-in does its own loading.
1128
1129         Reviewed by john.
1130
1131         * Misc.subproj/WebKitErrors.m: removed deprecated method
1132         * Misc.subproj/WebKitErrorsPrivate.h:
1133         * Plugins.subproj/WebNetscapePluginDocumentView.m:
1134         (-[WebNetscapePluginDocumentView setDataSource:]): fixed the assertion statement
1135         * Plugins.subproj/WebPluginDocumentView.h:
1136         * Plugins.subproj/WebPluginDocumentView.m:
1137         (-[WebPluginDocumentView dealloc]): remove retained plug-in
1138         (-[WebPluginDocumentView setDataSource:]): retain the plug-in, cancel the laod
1139
1140 2004-09-29  Chris Blumenberg  <cblu@apple.com>
1141         
1142         Fixed:
1143         <rdar://problem/3763832> Safari-155: Non-Embeded movies fail to open in Cocoa QT plug-in
1144         <rdar://problem/3820517> "*** -[WebPluginPackage NPP_New]: selector not recognized [self = 0x5552c10]"
1145
1146         Reviewed by rjw.
1147
1148         * History.subproj/WebHistoryItem.m:
1149         * Misc.subproj/WebNSViewExtras.h:
1150         * Misc.subproj/WebNSViewExtras.m:
1151         (-[NSView _web_firstResponderCausesFocusDisplay]):
1152         (-[NSView _webView]):
1153         (-[NSView _frame]):
1154         (-[NSView _bridge]):
1155         (-[NSView _dataSource]):
1156         * Plugins.subproj/WebBasePluginPackage.h:
1157         * Plugins.subproj/WebBasePluginPackage.m:
1158         (-[WebBasePluginPackage hash]):
1159         (-[WebBasePluginPackage isQuickTimePlugIn]):
1160         * Plugins.subproj/WebNetscapePluginDocumentView.m:
1161         (-[WebNetscapePluginDocumentView setDataSource:]):
1162         * Plugins.subproj/WebNetscapePluginRepresentation.m:
1163         * Plugins.subproj/WebPluginController.h:
1164         * Plugins.subproj/WebPluginController.m:
1165         (-[WebPluginController initWithDocumentView:]):
1166         (-[WebPluginController addPlugin:]):
1167         (-[WebPluginController destroyAllPlugins]):
1168         (-[WebPluginController webPlugInContainerLoadRequest:inFrame:]):
1169         (-[WebPluginController webPlugInContainerShowStatus:]):
1170         (-[WebPluginController webPlugInContainerSelectionColor]):
1171         (-[WebPluginController webFrame]):
1172         * Plugins.subproj/WebPluginDatabase.h:
1173         * Plugins.subproj/WebPluginDatabase.m:
1174         (-[WebPluginDatabase pluginForKey:withEnumeratorSelector:]):
1175         (-[WebPluginDatabase refresh]):
1176         (WebPluginDocumentView::while):
1177         * Plugins.subproj/WebPluginDocumentView.h: Added.
1178         * Plugins.subproj/WebPluginDocumentView.m: Added.
1179         (-[WebPluginDocumentView initWithFrame:]):
1180         (-[WebPluginDocumentView dealloc]):
1181         (-[WebPluginDocumentView drawRect:]):
1182         (-[WebPluginDocumentView setDataSource:]):
1183         (-[WebPluginDocumentView setNeedsLayout:]):
1184         (-[WebPluginDocumentView layout]):
1185         (-[WebPluginDocumentView currentWindow]):
1186         (-[WebPluginDocumentView viewWillMoveToWindow:]):
1187         (-[WebPluginDocumentView viewDidMoveToWindow]):
1188         (-[WebPluginDocumentView viewWillMoveToHostWindow:]):
1189         (-[WebPluginDocumentView viewDidMoveToHostWindow]):
1190         (-[WebPluginDocumentView receivedData:withDataSource:]):
1191         (-[WebPluginDocumentView receivedError:withDataSource:]):
1192         (-[WebPluginDocumentView finishedLoadingWithDataSource:]):
1193         (-[WebPluginDocumentView canProvideDocumentSource]):
1194         (-[WebPluginDocumentView documentSource]):
1195         (-[WebPluginDocumentView title]):
1196         * Plugins.subproj/npapi.m:
1197         (NPN_ReleaseVariantValue):
1198         (NPN_GetStringIdentifier):
1199         (NPN_GetStringIdentifiers):
1200         (NPN_GetIntIdentifier):
1201         (NPN_IdentifierIsString):
1202         (NPN_UTF8FromIdentifier):
1203         (NPN_IntFromIdentifier):
1204         (NPN_CreateObject):
1205         (NPN_RetainObject):
1206         (NPN_ReleaseObject):
1207         (NPN_Invoke):
1208         (NPN_InvokeDefault):
1209         (NPN_Evaluate):
1210         (NPN_GetProperty):
1211         (NPN_SetProperty):
1212         (NPN_RemoveProperty):
1213         (NPN_HasProperty):
1214         (NPN_HasMethod):
1215         (NPN_SetException):
1216         (NPN_Call):
1217         * WebCoreSupport.subproj/WebBridge.m:
1218         (-[WebBridge frameRequiredForMIMEType:URL:]):
1219         * WebCoreSupport.subproj/WebViewFactory.m:
1220         * WebKit.exp:
1221         * WebKit.pbproj/project.pbxproj:
1222         * WebView.subproj/WebDebugDOMNode.m:
1223         * WebView.subproj/WebDocumentInternal.h:
1224         * WebView.subproj/WebHTMLView.m:
1225         (-[WebHTMLView initWithFrame:]):
1226         * WebView.subproj/WebHTMLViewPrivate.h:
1227         * WebView.subproj/WebImageRepresentation.h:
1228         * WebView.subproj/WebRenderNode.m:
1229         * WebView.subproj/WebView.m:
1230
1231 2004-09-29  Richard Williamson   <rjw@apple.com>
1232
1233         Fixed <rdar://problem/3779998> bringing window to front or sending to back does not send focus/blur events to JavaScript window object
1234
1235         The fix has two parts, 1) make onblur and onfocus work for windows, 
1236         and 2), allow the dashboard to override WebKit's special key/non-key
1237         behaviors.
1238
1239         Reviewed by Maciej.
1240
1241         * Plugins.subproj/WebBaseNetscapePluginView.m:
1242         (-[WebBaseNetscapePluginView restartNullEvents]):
1243         * WebView.subproj/WebHTMLView.m:
1244         (-[WebHTMLView addMouseMovedObserver]):
1245         (-[WebHTMLView removeMouseMovedObserver]):
1246         * WebView.subproj/WebView.m:
1247         (-[WebView _dashboardBehavior:]):
1248         * WebView.subproj/WebViewInternal.h:
1249         * WebView.subproj/WebViewPrivate.h:
1250
1251 2004-09-29  Maciej Stachowiak  <mjs@apple.com>
1252
1253         Reviewed by John.
1254
1255         - consolidated OS version checks into prefix header
1256
1257         * Misc.subproj/WebFileDatabase.m:
1258         (-[WebFileDatabase _createLRUList:]):
1259         (+[WebFileDatabase _syncLoop:]):
1260         * Misc.subproj/WebKitErrors.m:
1261         (registerErrors):
1262         * Misc.subproj/WebNSObjectExtras.h:
1263         (WebNSRetainCFRelease):
1264         * Misc.subproj/WebNSPasteboardExtras.m:
1265         (-[NSPasteboard _web_declareAndWriteDragImage:URL:title:archive:source:]):
1266         * Misc.subproj/WebUnicode.m:
1267         (_unicodeDirection):
1268         * WebCoreSupport.subproj/WebImageData.h:
1269         * WebCoreSupport.subproj/WebImageRenderer.h:
1270         * WebCoreSupport.subproj/WebKeyGenerator.h:
1271         * WebCoreSupport.subproj/WebNewKeyGeneration.c:
1272         * WebKitPrefix.h:
1273         * WebView.subproj/WebDataSource.m:
1274         (+[WebDataSource _repTypesAllowImageTypeOmission:]):
1275         (-[WebDataSource isLoading]):
1276         * WebView.subproj/WebFrameView.m:
1277         (+[WebFrameView _viewTypesAllowImageTypeOmission:]):
1278         * WebView.subproj/WebHTMLView.m:
1279         * WebView.subproj/WebPDFRepresentation.h:
1280         * WebView.subproj/WebPDFRepresentation.m:
1281         * WebView.subproj/WebPDFView.h:
1282         * WebView.subproj/WebPDFView.m:
1283
1284 2004-09-29  Ken Kocienda  <kocienda@apple.com>
1285
1286         Reviewed by Hyatt
1287
1288         Fix for this bug:
1289         
1290         <rdar://problem/3818296> REGRESSION (Mail): centerSelectionInVisibleArea does not work correctly
1291
1292         * WebView.subproj/WebHTMLView.m:
1293         (-[WebHTMLView centerSelectionInVisibleArea:]): Now calls new centerSelectionInVisibleArea
1294         bridge function instead of ensureCaretVisible. Now handles caret selections and range
1295         selections correctly.
1296
1297 2004-09-28  Chris Blumenberg  <cblu@apple.com>
1298
1299         Added timing code so that Doug can time RTF conversion. 
1300
1301         * WebView.subproj/WebHTMLView.m:
1302         (-[WebHTMLView writeSelectionWithPasteboardTypes:toPasteboard:]):
1303         (-[WebHTMLView _attributeStringFromDOMRange:]):
1304
1305 2004-09-28  Richard Williamson   <rjw@apple.com>
1306
1307         <rdar://problem/3817421> add getter for dashboard regions (debugging)
1308         
1309         <rdar://problem/3817417> NSScrollView need autoregions for dashboard
1310         Also KWQScrollBars
1311
1312         Reviewed by Hyatt.
1313
1314         * WebCoreSupport.subproj/WebBridge.m:
1315         (-[WebBridge dashboardRegionsChanged:]):
1316         * WebView.subproj/WebView.m:
1317         (-[WebView _setInitiatedDrag:]):
1318         (-[WebView _addScrollerDashboardRegions:from:]):
1319         (-[WebView _addScrollerDashboardRegions:]):
1320         (-[WebView _dashboardRegions]):
1321         * WebView.subproj/WebViewPrivate.h:
1322
1323 2004-09-27  John Sullivan  <sullivan@apple.com>
1324
1325         - fixed <rdar://problem/3814705> 8A266: Safari authentication dialog "remember password" text should match Mail
1326
1327         * Panels.subproj/English.lproj/WebAuthenticationPanel.nib:
1328         changed "Remember this password" to "Remember this password in my keychain";
1329         this will need to go through CCC for this week's build.
1330
1331 2004-09-27  Chris Blumenberg  <cblu@apple.com>
1332
1333         Fixed: <rdar://problem/3594754> change null event interval from 20 ms to 10 ms to match speed on Windows
1334
1335         Reviewed by John.
1336
1337         * Plugins.subproj/WebBaseNetscapePluginView.m:
1338
1339 2004-09-27  Chris Blumenberg  <cblu@apple.com>
1340
1341         Fixed: <rdar://problem/3502138> text files don't remember scroll position when going back or reloading
1342
1343         Reviewed by john.
1344
1345         * WebView.subproj/WebTextView.m:
1346         (-[WebTextView layout]): implemented, call sizeToFit, without this scrollPoint: won't work
1347
1348 2004-09-27  John Sullivan  <sullivan@apple.com>
1349
1350         Reviewed by Ken.
1351
1352         - WebKit part of fix for <rdar://problem/3734466> ER: Support standard editing keystrokes 
1353         like Cmd-B while editing rich text
1354
1355         * WebView.subproj/WebHTMLView.m:
1356         (-[WebHTMLView _toggleBold]):
1357         new method, toggles font-weight from "bold" to "normal"
1358         (-[WebHTMLView _toggleItalic]):
1359         new method, toggles font-style from "italic" to "normal"
1360         (-[WebHTMLView _handleStyleKeyEquivalent:]):
1361         new method, if the new preference is set and we're in an editable state, check for standard
1362         key equivalents for toggling styles (just command-B and command-I for now).
1363         (-[WebHTMLView performKeyEquivalent:]):
1364         Moved in file, now calls _handleStyleKeyEquivalent:
1365         
1366         * WebView.subproj/WebPreferenceKeysPrivate.h:
1367         new preference key WebKitRespectStandardStyleKeyEquivalentsPreferenceKey
1368         * WebView.subproj/WebPreferences.m:
1369         (+[WebPreferences initialize]):
1370         initial value of WebKitRespectStandardStyleKeyEquivalentsPreferenceKey is NO (maybe we'll
1371         change our minds about this, but this is more guaranteed to be backward-compatible)
1372         (-[WebPreferences respectStandardStyleKeyEquivalents]):
1373         read WebKitRespectStandardStyleKeyEquivalentsPreferenceKey
1374         (-[WebPreferences setRespectStandardStyleKeyEquivalents:]):
1375         write WebKitRespectStandardStyleKeyEquivalentsPreferenceKey
1376         
1377         * WebView.subproj/WebPreferencesPrivate.h:
1378         declare getter and setter
1379         
1380         * English.lproj/StringsNotToBeLocalized.txt:
1381         updated for these and other recent changes
1382
1383 2004-09-27  Chris Blumenberg  <cblu@apple.com>
1384
1385         Fixed: <rdar://problem/3806649> assertion failure after control-click of webcam
1386
1387         Reviewed by john.
1388
1389         * WebView.subproj/WebDefaultContextMenuDelegate.m:
1390         (-[WebDefaultUIDelegate contextMenuItemsForElement:]): don't provide "Copy Image" if the image is not fully loaded
1391
1392 2004-09-27  Chris Blumenberg  <cblu@apple.com>
1393
1394         Fixed: <rdar://problem/3814810> REGRESSION (125-164): Exception adding nil to dictionary in dragging code
1395
1396         Reviewed by john.
1397
1398         * WebView.subproj/WebHTMLView.m:
1399         (-[WebHTMLView _setMouseDownEvent:]): new
1400         (-[WebHTMLView acceptsFirstMouse:]): call _setMouseDownEvent
1401         (-[WebHTMLView shouldDelayWindowOrderingForEvent:]): ditto
1402         (-[WebHTMLView mouseDown:]): ditto
1403         (-[WebHTMLView _delegateDragSourceActionMask]): removed temp fix, assert that the mouse event is not nil
1404
1405 2004-09-27  Darin Adler  <darin@apple.com>
1406
1407         * WebKit.pbproj/project.pbxproj: Added WebDashboardRegion.h as a private header.
1408
1409 2004-09-24  Chris Blumenberg  <cblu@apple.com>
1410
1411         Reviewed by rjw.
1412
1413         * WebView.subproj/WebHTMLView.m:
1414         (-[WebHTMLView _selectedArchive]): added timing code for copying markup
1415
1416 === Safari-165 ===
1417
1418 2004-09-24  Chris Blumenberg  <cblu@apple.com>
1419
1420         Temp fix for: <rdar://problem/3814810> REGRESSION (125-164): Exception adding nil to dictionary in dragging code
1421
1422         Reviewed by john.
1423
1424         * WebView.subproj/WebHTMLView.m:
1425         (-[WebHTMLView _delegateDragSourceActionMask]): return none if the mouse down event is nil
1426
1427 2004-09-24  Ken Kocienda  <kocienda@apple.com>
1428
1429         Reviewed by John
1430
1431         Fix for this bug:
1432         
1433         <rdar://problem/3813917> REGRESSION (125-163): The font panel will change the font of any web page
1434         
1435         We were doing a laughably bad job at preventing edits in documents that were not editable.
1436         This change fixes the specific case of the bug mentioned above, and makes an attempt to
1437         fix similar bugs by checking for whether the view is in editing mode before making edits.
1438
1439         * WebView.subproj/WebHTMLView.m:
1440         (-[WebHTMLView _canEdit]): Renamed from _canType. Used in many more places in the code now.
1441         (-[WebHTMLView _isMoveDrag]): Change around code to make the meaning more clear. This one
1442         was actually performing a correct check before.
1443         (-[WebHTMLView keyDown:]): Renamed _canType to _canEdit.
1444         (-[WebHTMLView paste:]): Don't beep if can't paste. This matches AppKit. Any paste-related beeps
1445         will come from failure to handle key equivalent. Menu validation will kick in to dim menu.
1446         (-[WebHTMLView _applyStyleToSelection:]): Bail if !_canEdit.
1447         (-[WebHTMLView pasteAsPlainText:]): Ditto.
1448         (-[WebHTMLView _alignSelectionUsingCSSValue:]): Ditto.
1449         (-[WebHTMLView insertNewline:]): Ditto.
1450         (-[WebHTMLView insertParagraphSeparator:]): Ditto.
1451         (-[WebHTMLView _changeWordCaseWithSelector:]): Ditto.
1452         (-[WebHTMLView _deleteWithDirection:granularity:killRing:]): Ditto.
1453         (-[WebHTMLView complete:]): Ditto.
1454         (-[WebHTMLView _changeSpellingToWord:]): Ditto. Some code rearranging to eliminate bridge local variable.
1455         (-[WebHTMLView ignoreSpelling:]): Ditto.
1456         (-[WebHTMLView yank:]): Bail if !_canEdit.
1457         (-[WebHTMLView yankAndSelect:]): Ditto.
1458         (-[WebHTMLView deleteToMark:]): Ditto.
1459         (-[WebHTMLView swapWithMark:]): Ditto.
1460         (-[WebHTMLView transpose:]): Ditto.
1461         (-[WebHTMLView _updateFontPanel]): Ditto. Some code rearranging to eliminate bridge local variable.
1462         (-[WebHTMLView setMarkedText:selectedRange:]): Bail if !_canEdit.
1463         (-[WebHTMLView _insertText:selectInsertedText:]): Ditto. Some code rearranging to eliminate bridge local variable.
1464         * WebView.subproj/WebHTMLViewPrivate.h: Renamed _canType to _canEdit.
1465
1466 2004-09-24  Ken Kocienda  <kocienda@apple.com>
1467
1468         Reviewed by me
1469
1470         * WebCoreSupport.subproj/WebDashboardRegion.h: Check in file copied from WebCore.
1471
1472 2004-09-23  Maciej Stachowiak  <mjs@apple.com>
1473
1474         Reviewed by Darin.
1475
1476         <rdar://problem/3685235> REGRESSION (Mail): links are not properly editable
1477         
1478         * WebView.subproj/WebDefaultUIDelegate.m: By default, don't allow
1479         link dragging if the element under the mouse pointer is
1480         editable. This way, you can drag-select starting inside a link.
1481
1482 2004-09-23  John Sullivan  <sullivan@apple.com>
1483
1484         Reviewed by Chris.
1485         
1486         - WebKit part of fix for <rdar://problem/3415264> 
1487         Default encoding should initially be set to current system encoding
1488
1489         * WebView.subproj/WebPreferences.m:
1490         (-[WebPreferences _setInitialDefaultTextEncodingToSystemEncoding]):
1491         new SPI that sets the initial value of the default text encoding to
1492         be the system encoding, with a special-case conversion of MacRoman->Latin1.
1493         This is not done automatically for WebKit clients for fear of breaking them.
1494         
1495         * WebView.subproj/WebPreferencesPrivate.h:
1496         declare new SPI
1497
1498 2004-09-23  Darin Adler  <darin@apple.com>
1499
1500         Reviewed by Ken.
1501
1502         - fixed <rdar://problem/3811584> REGRESSION (85-125): iframe.document undefined in function called from button onclick; works from img onclick
1503
1504         The fix is to not let "defers callbacks" have any effect on loading "about:blank".
1505         I also had to fix one bug in WebCore that could then be reproduced by going to "about:blank"
1506         while using a button.
1507
1508         * WebView.subproj/WebMainResourceClient.m:
1509         (-[WebMainResourceClient connection:didReceiveResponse:]): Loosen asserts to allow this callback
1510         for the specific case of "about:blank" even if the defers callbacks flag is true.
1511         (-[WebMainResourceClient connectionDidFinishLoading:]): Ditto.
1512         (-[WebMainResourceClient loadWithRequestNow:]): Added NSURLRequest return value. Loosened asserts
1513         as above. Changed code to return a new request if we get a new request back that is not empty
1514         when the defers callbacks flag is true.
1515         (-[WebMainResourceClient loadWithRequest:]): If the defers callbacks flag is set, but the
1516         URL is one that gives us an empty document, then do the work right away, don't defer it.
1517
1518 2004-09-23  Darin Adler  <darin@apple.com>
1519
1520         - fixed B&I builds by checking in generated file
1521
1522         * WebCoreSupport.subproj/WebDashboardRegion.h: Added.
1523
1524 2004-09-22  Richard Williamson   <rjw@apple.com>
1525
1526         Pass dashboard regions to UI delegate.
1527
1528         Reviewed by Hyatt.
1529
1530         * WebCoreSupport.subproj/WebBridge.m:
1531         (-[WebBridge dashboardRegionsChanged:]):
1532         * WebView.subproj/WebUIDelegatePrivate.h:
1533         * copy-webcore-files-to-webkit:
1534
1535 2004-09-22  Chris Blumenberg  <cblu@apple.com>
1536
1537         Fixed build that I just broke.
1538
1539         * WebView.subproj/WebHTMLView.m:
1540         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]):
1541
1542 2004-09-22  Chris Blumenberg  <cblu@apple.com>
1543
1544         <rdar://problem/3812091> REGRESSION (Mail): double-clicked word is not smart inserted on drag
1545
1546         Reviewed by john.
1547
1548         * WebView.subproj/WebHTMLView.m:
1549         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): pass value for smartMove
1550
1551 2004-09-22  Chris Blumenberg  <cblu@apple.com>
1552
1553         Fixed:
1554         <rdar://problem/3667301> Frequent crashes in Mail when viewing HTML messages (CFURLGetByteRangeForComponent)
1555         <rdar://problem/3810354> WebResourceLoadDelegate can't refuse requests by returning nil; code asserts/crashes instead
1556
1557         Reviewed by rjw.
1558
1559         * WebCoreSupport.subproj/WebSubresourceClient.m:
1560         (+[WebSubresourceClient startLoadingResource:withRequest:customHeaders:referrer:forDataSource:]): removed broken code that handled loadWithRequest returning NO
1561         * WebView.subproj/WebBaseResourceHandleDelegate.m:
1562         (-[WebBaseResourceHandleDelegate loadWithRequest:]): if nil is returned from the client for willSendRequest, report the cancelled error and return
1563
1564 2004-09-22  Ken Kocienda  <kocienda@apple.com>
1565
1566         Reviewed by Darin
1567
1568         * WebView.subproj/WebFrameView.m:
1569         (-[WebFrameView _webcore_effectiveFirstResponder]): New function to yield the correct responder
1570         to check for firstResponder-ness before calling makeFirstResonder. This helps to prevent 
1571         unwanted firstResponder switching.
1572         * WebView.subproj/WebView.m:
1573         (-[WebView _webcore_effectiveFirstResponder]): Ditto.
1574
1575 2004-09-21  Chris Blumenberg  <cblu@apple.com
1576
1577         Fixed:
1578         <rdar://problem/3735071> REGRESSION (Mail): WebCore Editing must do smart paste
1579         <rdar://problem/3799163> REGRESSION (Mail): Deleting a word doesn't delete whitespace
1580
1581         Reviewed by darin.
1582
1583         * WebView.subproj/WebDataSource.m:
1584         (-[WebDataSource _replaceSelectionWithArchive:selectReplacement:]): pass NO for smartReplace
1585         * WebView.subproj/WebHTMLView.m:
1586         (-[WebHTMLView _pasteWithPasteboard:allowPlainText:]): pass parameter for smartReplace using _canSmartReplaceWithPasteboard
1587         (-[WebHTMLView _changeSpellingFromMenu:]): pass NO for smartReplace
1588         (-[WebHTMLView pasteboardTypesForSelection]): include WebSmartPastePboardType when _canSmartCopyOrDelete return YES
1589         (-[WebHTMLView writeSelectionWithPasteboardTypes:toPasteboard:]): ditto
1590         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): pass parameter for smartReplace using _canSmartReplaceWithPasteboard
1591         (-[WebHTMLView delete:]): call _deleteSelection
1592         (-[WebHTMLView cut:]): don't call delegate twice, call _deleteRange to delete
1593         (-[WebHTMLView pasteAsPlainText:]): pass parameter for smartReplace using _canSmartReplaceWithPasteboard
1594         (-[WebHTMLView _changeWordCaseWithSelector:]): pass NO for smartReplace
1595         (-[WebHTMLView deleteBackward:]): call _deleteSelection when there is a selected range
1596         (-[WebHTMLView _changeSpellingToWord:]): pass NO for smartReplace
1597         (-[WebHTMLView deleteToMark:]): pass NO for smartDeleteOK
1598         (-[WebHTMLView transpose:]): pass NO for smartReplace
1599         (-[WebHTMLView _shouldDeleteRange:]): moved
1600         (-[WebHTMLView _deleteRange:preflight:killRing:prepend:smartDeleteOK:]): moved, handle smartDelete
1601         (-[WebHTMLView _deleteWithDirection:granularity:killRing:]): moved
1602         (-[WebHTMLView _deleteSelection]): new
1603         (-[WebHTMLView _canSmartReplaceWithPasteboard]): new
1604         (-[WebHTMLView _canSmartCopyOrDelete]): new
1605         (-[WebHTMLView setMarkedText:selectedRange:]): pass NO for smartReplace
1606         (-[WebHTMLView _discardMarkedText]): call _deleteSelection
1607         (-[WebTextCompleteController _insertMatch:]): pass NO for smartReplace
1608         (-[WebTextCompleteController endRevertingChange:moveLeft:]): pass NO for smartReplace
1609         * WebView.subproj/WebHTMLViewInternal.h:
1610         * WebView.subproj/WebView.m:
1611         (-[WebView _commonInitializationWithFrameName:groupName:]): set smartInsertDeleteEnabled to YES
1612         (-[WebView replaceSelectionWithNode:]): pass NO for smartReplace
1613         (-[WebView replaceSelectionWithText:]): pass NO for smartReplace
1614         (-[WebView replaceSelectionWithMarkupString:]): pass NO for smartReplace
1615         (-[WebView deleteSelection]): call _deleteSelection on WebHTMLView
1616
1617 2004-09-21  John Sullivan  <sullivan@apple.com>
1618
1619         Reviewed by Darin.
1620
1621         - WebKit part of fix for <rdar://problem/3618274> REGRESSION (125-135): 
1622         Option-tab doesn't always work as expected
1623
1624         * WebCoreSupport.subproj/WebBridge.m:
1625         (-[WebBridge _retrieveKeyboardUIModeFromPreferences:]):
1626         Don't set WebCoreKeyboardAccessFull when setting WebCoreKeyboardAccessTabsToLinks
1627         after all; these need to be tested independently to get the option-tab behavior
1628         correct.
1629
1630 2004-09-21  John Sullivan  <sullivan@apple.com>
1631
1632         * WebView.subproj/WebHTMLView.m:
1633         (-[WebHTMLView doCommandBySelector:]):
1634         Commented out part of previous change; it breaks Mail editing until Mail
1635         fixes bug 3810158.
1636
1637 2004-09-21  John Sullivan  <sullivan@apple.com>
1638
1639         Reviewed by Ken.
1640
1641         * WebView.subproj/WebHTMLView.m:
1642         (-[WebHTMLView doCommandBySelector:]):
1643         Fix build failure from previous checkin, d'oh! Didn't set up webview variable.
1644
1645 2004-09-21  John Sullivan  <sullivan@apple.com>
1646
1647         Reviewed by Ken.
1648         
1649         - fixed <rdar://problem/3809477> WebHTMLView needs to pass doCommandBySelector through delegate
1650
1651         * WebView.subproj/WebHTMLView.m:
1652         (-[WebHTMLView doCommandBySelector:]):
1653         Call through to editing delegate. If editing delegate returns YES, don't call super.
1654
1655         * WebView.subproj/WebDefaultEditingDelegate.m:
1656         (-[WebDefaultEditingDelegate webView:doCommandBySelector:]):
1657         default implementation (which was never called) was returning YES, but it should
1658         return NO to signal that it didn't handle the selector.
1659         
1660 2004-09-21  Chris Blumenberg  <cblu@apple.com>
1661
1662         Fixed: <rdar://problem/3647229> Safari does not play inline Windows Media Content on some sites (miggy.net and ministryofsound.com)
1663
1664         Reviewed by john.
1665
1666         * Plugins.subproj/WebBaseNetscapePluginView.m:
1667         * Plugins.subproj/WebNetscapePluginEmbeddedView.m:
1668         (-[WebNetscapePluginEmbeddedView initWithFrame:plugin:URL:baseURL:MIMEType:attributeKeys:attributeValues:]): set the plug-in before calling setting the attributes, so we can avoid passing certain attributes to the WMP plug-in that cause it to crash
1669
1670 2004-09-20  Chris Blumenberg  <cblu@apple.com>
1671
1672         Fixed: <rdar://problem/3781290> REGRESSION (Mail): Crash in ReplaceSelectionCommandImpl attaching file to new message
1673
1674         Reviewed by kocienda.
1675
1676         * WebView.subproj/WebView.m:
1677         (-[WebView setEditable:]): call updateSelectionFromEmpty on the bridge if there is no selection
1678
1679 2004-09-20  Chris Blumenberg  <cblu@apple.com>
1680
1681         Changes to implement renamed bridge methods.
1682
1683         Reviewed by darin.
1684
1685         * ChangeLog:
1686         * DOM.subproj/WebDOMOperations.m:
1687         (-[DOMDocument URLWithAttributeString:]): call renamed bridge method
1688         * WebCoreSupport.subproj/WebBridge.m:
1689         (-[WebBridge pluginViewWithPackage:attributeNames:attributeValues:baseURL:]): take 2 parameter arrays rather than 1 which will have to be parsed
1690         (-[WebBridge viewForPluginWithURL:attributeNames:attributeValues:MIMEType:]): ditto
1691
1692 2004-09-20  Darin Adler  <darin@apple.com>
1693
1694         Reviewed by Chris.
1695
1696         * WebView.subproj/WebFramePrivate.h: Added back.
1697         * WebView.subproj/WebFrameViewPrivate.h: Removed.
1698
1699 2004-09-20  Darin Adler  <darin@apple.com>
1700
1701         Reviewed by Ken.
1702
1703         - fixed <rdar://problem/3655360> REGRESSION (Mail): Ctrl-V emacs key binding, -pageDown: method, unimplemented (and pageUp, and selection-modifying versions)
1704         - fixed <rdar://problem/3792138> REGRESSION (Mail): Spell checker doesn't check current selected word
1705
1706         * WebView.subproj/WebFrameViewInternal.h: Moved WebFrameViewPrivate inside the WebFrameView.m file.
1707         Removed a bunch of methods that don't need to be seen in other files, and added _verticalPageScrollDistance.
1708         * WebView.subproj/WebFrameViewPrivate.h: Removed. Renamed to WebFrameViewInternal.h.
1709         * WebView.subproj/WebFrameView.m:
1710         (-[WebFrameView _verticalKeyboardScrollDistance]): Move in the file because of internal vs. private.
1711         (-[WebFrameView _shouldDrawBorder]): Ditto.
1712         (-[WebFrameView _tile]): Ditto.
1713         (-[WebFrameView _verticalPageScrollDistance]): Added. Separate method so it can be called
1714         by the code to implement pageDown:.
1715         (-[WebFrameView _drawBorder]): Move in the file because of internal vs. private.
1716         (-[WebFrameView _goBack]): Ditto.
1717         (-[WebFrameView _goForward]): Ditto.
1718         (-[WebFrameView _scrollVerticallyBy:]): Ditto.
1719         (-[WebFrameView _scrollHorizontallyBy:]): Ditto.
1720         (-[WebFrameView _horizontalKeyboardScrollDistance]): Ditto.
1721         (-[WebFrameView _horizontalPageScrollDistance]): Added. Separate method for consistency with
1722         vertical method above.
1723         (-[WebFrameView _pageVertically:]): Moved and changed to use _verticalPageScrollDistance.
1724         (-[WebFrameView _pageHorizontally:]): Moved and changed to use _horizontalPageScrollDistance.
1725         (-[WebFrameView _scrollLineVertically:]): Move in the file because of internal vs. private.
1726         (-[WebFrameView _scrollLineHorizontally:]): Ditto.
1727         (-[WebFrameView scrollPageUp:]): Ditto.
1728         (-[WebFrameView scrollPageDown:]): Ditto.
1729         (-[WebFrameView scrollLineUp:]): Ditto.
1730         (-[WebFrameView scrollLineDown:]): Ditto.
1731         (-[WebFrameView _firstResponderIsControl]): Ditto.
1732         (-[WebFrameView keyDown:]): Changed to eliminate _pageLeft, _lineLeft, _pageRight, and _lineRight.
1733
1734         * WebView.subproj/WebDataSource.m: Use WebFrameView.h instead of WebFrameViewPrivate.h.
1735
1736         * WebView.subproj/WebHTMLView.m:
1737         (-[WebHTMLView _alterCurrentSelection:verticalDistance:]): Added.
1738         (-[WebHTMLView moveToBeginningOfDocument:]): Use WebSelectToDocumentBoundary.
1739         (-[WebHTMLView moveToBeginningOfDocumentAndModifySelection:]): Ditto.
1740         (-[WebHTMLView moveToEndOfDocument:]): Ditto.
1741         (-[WebHTMLView moveToEndOfDocumentAndModifySelection:]): Ditto.
1742         (-[WebHTMLView moveParagraphBackwardAndModifySelection:]): Added.
1743         (-[WebHTMLView moveParagraphForwardAndModifySelection:]): Added.
1744         (-[WebHTMLView pageUp:]): Added.
1745         (-[WebHTMLView pageDown:]): Added.
1746         (-[WebHTMLView pageUpAndModifySelection:]): Added.
1747         (-[WebHTMLView pageDownAndModifySelection:]): Added.
1748         (-[WebHTMLView showGuessPanel:]): Changed to call advanceToNextMisspellingStartingJustBeforeSelection.
1749         This fixes the problem with spell checking.
1750
1751         * WebView.subproj/WebImageView.m:
1752         (-[WebImageView webView]): Changed to use _web_parentWebView.
1753         (-[WebImageView menuForEvent:]): Changed to use [self webView].
1754         (-[WebImageView mouseDown:]): Ditto.
1755         (-[WebImageView mouseDragged:]): Ditto.
1756         (-[WebImageView draggedImage:endedAt:operation:]): Ditto.
1757
1758         * WebView.subproj/WebTextView.m:
1759         (-[WebTextView _textSizeMultiplierFromWebView]): Changed to use _web_parentWebView.
1760         (-[WebTextView menuForEvent:]): Ditto.
1761         (-[WebTextView drawPageBorderWithSize:]): Ditto.
1762         (-[WebTextView knowsPageRange:]): Ditto.
1763
1764         * Plugins.subproj/WebPluginDatabase.m: Updated filename of WebFrameViewInternal.h.
1765         * WebCoreSupport.subproj/WebBridge.m: Ditto.
1766         * WebKit.pbproj/project.pbxproj: Ditto.
1767         * WebView.subproj/WebFrame.m: Ditto.
1768         * WebView.subproj/WebView.m: Ditto.
1769         * Misc.subproj/WebNSViewExtras.m: Ditto.
1770
1771 2004-09-20  Darin Adler  <darin@apple.com>
1772
1773         Reviewed by Ken.
1774
1775         - added helper method _web_parentWebView so fewer files need to get at WebFrame private methods
1776
1777         * Misc.subproj/WebNSViewExtras.h: Added _web_parentWebView.
1778         * Misc.subproj/WebNSViewExtras.m: (-[NSView _web_parentWebView]): Added.
1779
1780 === Safari-164 ===
1781
1782 2004-09-17  Chris Blumenberg  <cblu@apple.com>
1783         
1784         Fixed:
1785         <rdar://problem/3805757> don't unnecessarily put RTFD on the pasteboard
1786         <rdar://problem/3805756> strip attachments before generating RTF
1787
1788         Reviewed by john.
1789
1790         * WebView.subproj/WebHTMLView.m:
1791         (-[WebHTMLView writeSelectionWithPasteboardTypes:toPasteboard:]): only put RTFD on the pasteboard if it has attachments, strip attachments when writing RTF
1792
1793 2004-09-16  Darin Adler  <darin@apple.com>
1794
1795         Reviewed by John.
1796
1797         - fixed <rdar://problem/3804648> 8A262: Safari crashed in -[WebView(WebPrivate) _editingDelegateForwarder] inside QuickTime Cocoa Plug-in during WebView deallocation
1798
1799         * WebView.subproj/WebView.m: (-[WebView _editingDelegateForwarder]):
1800         Check _private for nil before dereferencing it.
1801
1802 2004-09-16  Chris Blumenberg  <cblu@apple.com>
1803
1804         Fixed: <rdar://problem/3779150> REGRESSION: images not copied when copying HTML in Safari and pasting into TextEdit
1805
1806         Reviewed by john.
1807
1808         * WebView.subproj/WebHTMLView.m:
1809         (-[WebHTMLView writeSelectionWithPasteboardTypes:toPasteboard:]): use RTFDFromRange:: for RTFD
1810         * WebView.subproj/WebHTMLViewPrivate.h:
1811
1812 2004-09-15  Darin Adler  <darin@apple.com>
1813
1814         Reviewed by John.
1815
1816         - fixed assertion I saw using the font panel
1817
1818         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView _colorAsString:]):
1819         Convert color space before trying to get R, G, and B components.
1820
1821 2004-09-15  Chris Blumenberg  <cblu@apple.com>
1822
1823         Fixed: <rdar://problem/3802232> REGRESSION (Mail): WebCore Editing must do smart copy
1824         
1825         Reviewed by kocienda.
1826
1827         * WebView.subproj/WebHTMLView.m:
1828         (-[WebHTMLView _writeSelectionToPasteboard:]): call instance method not class method to get pasteboard types since the types depends on the current selection granularity
1829         (-[WebHTMLView pasteboardTypesForSelection]): if the selection granularity is "word" include the smart pasteboard type
1830         (-[WebHTMLView writeSelectionWithPasteboardTypes:toPasteboard:]): put nil on the pasteboard for smart copy
1831
1832 2004-09-14  Darin Adler  <darin@apple.com>
1833
1834         Reviewed by Maciej.
1835
1836         - fixed <rdar://problem/3788894> REGRESSION (Mail): ctrl-t emacs key binding does not work (transpose)
1837         - fixed <rdar://problem/3798946> REGRESSION (Mail): Cursor does not disappear when typing
1838
1839         * WebView.subproj/WebHTMLView.m:
1840         (-[WebHTMLView keyDown:]): Hide cursor by calling setHiddenUntilMouseMoves:YES.
1841         (-[WebHTMLView transpose:]): Added.
1842
1843 2004-09-14  Richard Williamson   <rjw@apple.com>
1844
1845         1.  Add class parameter to object allocation function.  This is somewhat redundant, given that
1846         the allocation function is in the class function vector, but people wanted to use the same
1847         allocation function for different classes.
1848         
1849         2.  Renamed NPN_Class to NPN_Invoke to match the name in the function vector.
1850         
1851         3.  Add support for a default function on an object.  This is a feature that ActiveX supports,
1852         and will allow JavaScript code to be written that will look exactly the same for both ActiveX
1853         plugins and Netscape or WebKit plugins.  There are implementations included for the 'C' and
1854         'Objective-C' bindings.
1855         
1856         There bugs are covered by
1857         
1858         <rdar://problem/3776343> Support for default functions in the JavaScript bindings
1859         <rdar://problem/3779186> NPN_Call needs to be renamed to NPN_Invoke
1860         <rdar://problem/3674754> Need to implement latest npruntime.h
1861
1862         Reviewed by John.
1863
1864         * Plugins.subproj/WebNetscapePluginPackage.m:
1865         (-[WebNetscapePluginPackage load]):
1866         * Plugins.subproj/WebScriptObject.h:
1867         * Plugins.subproj/npfunctions.h:
1868         * Plugins.subproj/npruntime.h:
1869
1870 2004-09-13  Richard Williamson   <rjw@apple.com>
1871
1872         D'oh.  How many times can I screw up a simple fix!
1873
1874         * WebCoreSupport.subproj/WebTextRenderer.m:
1875         (-[WebTextRenderer initWithFont:usingPrinterFont:]):
1876
1877 2004-09-13  Richard Williamson   <rjw@apple.com>
1878
1879         Fixed snafu from 3782533 checkin.
1880         
1881         Reviewed by John.
1882
1883         * WebCoreSupport.subproj/WebTextRenderer.m:
1884         (-[WebTextRenderer initWithFont:usingPrinterFont:]):
1885
1886 2004-09-12  Chris Blumenberg  <cblu@apple.com>
1887
1888         Support for: <rdar://problem/3794790> drop rate or time remaining from download status when window is too small to fit it
1889
1890         Reviewed by john.
1891
1892         * Misc.subproj/WebStringTruncator.h:
1893         * Misc.subproj/WebStringTruncator.m:
1894         (+[WebStringTruncator widthOfString:font:]): new
1895
1896 2004-09-10  Richard Williamson   <rjw@apple.com>
1897
1898         Fixed <rdar://problem/3782533> CrashTracer: .1459 crashes at com.apple.WebKit: -[WebTextRenderer initWithFont:usingPrinterFont:] + 0x138
1899
1900         We were explicitly failing when we encountered deprecated fonts.
1901         (Those with unsupported glyph packings).
1902         Deprecated fonts should only appear on a system that have 
1903         stuff migrated from OS 9.  Ugh, thats probably why we've never seen
1904         the problem here.
1905         
1906         Reviewed by John.
1907
1908         * WebCoreSupport.subproj/WebTextRenderer.m:
1909         (-[WebTextRenderer initWithFont:usingPrinterFont:]):
1910
1911 2004-09-10  John Sullivan  <sullivan@apple.com>
1912
1913         Reviewed by Chris.
1914         
1915         - added _isFrameSet as a private method, so it can be used in WebBrowser.
1916         This is needed to merge the fix for 3123987 to SUPanNavy.
1917
1918         * WebView.subproj/WebFrame.m:
1919         (-[WebFrame _isFrameSet]):
1920         new method
1921         
1922         * WebView.subproj/WebFramePrivate.h:
1923         declare new method
1924
1925 2004-09-09  Chris Blumenberg  <cblu@apple.com>
1926
1927         Support for: <rdar://problem/3795485> debug menu item to enable RSS animation on first layout
1928
1929         Reviewed by rjw.
1930
1931         * WebCoreSupport.subproj/WebBridge.m:
1932         (-[WebBridge didFirstLayout]): new
1933         * WebView.subproj/WebDefaultFrameLoadDelegate.m:
1934         (-[WebDefaultFrameLoadDelegate webView:didFirstLayoutInFrame:]): new
1935         * WebView.subproj/WebViewPrivate.h:
1936
1937 2004-09-09  Richard Williamson   <rjw@apple.com>
1938
1939         Alternate implementation of image rendering.  Use CGImageRefs instead
1940         of NSImages.
1941         
1942         Mostly works, but currently disabled because of issues w/ CG.
1943         
1944         Reviewed by Chris.
1945
1946         * ChangeLog:
1947         * WebCoreSupport.subproj/WebImageData.h: Added.
1948         * WebCoreSupport.subproj/WebImageData.m: Added.
1949         (-[WebImageData _commonTermination]):
1950         (-[WebImageData dealloc]):
1951         (-[WebImageData finalize]):
1952         (-[WebImageData copyWithZone:]):
1953         (-[WebImageData numberOfImages]):
1954         (-[WebImageData currentFrame]):
1955         (-[WebImageData _invalidateImages]):
1956         (-[WebImageData imageAtIndex:]):
1957         (-[WebImageData incrementalLoadWithBytes:length:complete:]):
1958         (drawPattern):
1959         (-[WebImageData tileInRect:fromPoint:context:]):
1960         (-[WebImageData isNull]):
1961         (-[WebImageData size]):
1962         (-[WebImageData _frameDuration]):
1963         (-[WebImageData _repetitionCount]):
1964         (-[WebImageData isAnimationFinished]):
1965         (+[WebImageData stopAnimationsInView:]):
1966         (-[WebImageData addAnimatingRenderer:inView:]):
1967         (-[WebImageData removeAnimatingRenderer:]):
1968         (-[WebImageData _stopAnimation]):
1969         (-[WebImageData _nextFrame:]):
1970         (-[WebImageData animate]):
1971         * WebCoreSupport.subproj/WebImageRenderer.h:
1972         * WebCoreSupport.subproj/WebImageRenderer.m:
1973         (-[WebImageRenderer initWithMIMEType:]):
1974         (-[WebImageRenderer initWithData:MIMEType:]):
1975         (-[WebImageRenderer initWithContentsOfFile:]):
1976         (-[WebImageRenderer dealloc]):
1977         (-[WebImageRenderer copyWithZone:]):
1978         (-[WebImageRenderer retainOrCopyIfNeeded]):
1979         (-[WebImageRenderer resize:]):
1980         (-[WebImageRenderer size]):
1981         (-[WebImageRenderer MIMEType]):
1982         (-[WebImageRenderer frameCount]):
1983         (-[WebImageRenderer isNull]):
1984         (-[WebImageRenderer incrementalLoadWithBytes:length:complete:]):
1985         (-[WebImageRenderer drawImageInRect:fromRect:]):
1986         (-[WebImageRenderer drawImageInRect:fromRect:compositeOperator:context:]):
1987         (-[WebImageRenderer tileInRect:fromPoint:context:]):
1988         (-[WebImageRenderer _startOrContinueAnimationIfNecessary]):
1989         (+[WebImageRenderer stopAnimationsInView:]):
1990         (-[WebImageRenderer stopAnimation]):
1991         (-[WebImageRenderer targetAnimationRect]):
1992         (-[WebImageRenderer increaseUseCount]):
1993         (-[WebImageRenderer decreaseUseCount]):
1994         (-[WebImageRenderer flushRasterCache]):
1995         (-[WebImageRenderer imageRef]):
1996         (-[WebImageRenderer TIFFRepresentation]):
1997         (-[WebImageRenderer image]):
1998         * WebCoreSupport.subproj/WebImageRendererFactory.m:
1999         (-[WebImageRendererFactory imageRendererWithMIMEType:]):
2000         (-[WebImageRendererFactory imageRendererWithData:MIMEType:]):
2001         (-[WebImageRendererFactory imageRendererWithSize:]):
2002         (-[WebImageRendererFactory imageRendererWithName:]):
2003         * WebKit.pbproj/project.pbxproj:
2004         * WebView.subproj/WebImageView.m:
2005         (-[WebImageView image]):
2006
2007 === Safari-163 ===
2008
2009 2004-09-09  Maciej Stachowiak  <mjs@apple.com>
2010
2011         - rolled out last two changes, they seem to cause a performance regression
2012
2013         * WebView.subproj/WebHTMLView.m:
2014         (-[WebHTMLView firstRectForCharacterRange:]):
2015
2016 2004-09-09  Maciej Stachowiak  <mjs@apple.com>
2017
2018         - fixed build
2019
2020         * WebView.subproj/WebHTMLView.m:
2021         (-[WebHTMLView firstRectForCharacterRange:]):
2022
2023 2004-09-08  Maciej Stachowiak  <mjs@apple.com>
2024
2025         Reviewed by Dave.
2026
2027         WebKit part of fix for:
2028         
2029         <rdar://problem/3759187> REGRESSION (Mail): implement firstRectForCharacterRange:
2030
2031         * WebView.subproj/WebHTMLView.m:
2032         (-[WebHTMLView firstRectForCharacterRange:]): Call the appropriate new bridge method,
2033         and translate to screen coordinates.
2034
2035 2004-09-09  Darin Adler  <darin@apple.com>
2036
2037         Reviewed by Ken.
2038
2039         - fixed <rdar://problem/3790526> mark-related methods not implemented (needed for people with them in their key bindings files)
2040
2041         * WebKit.pbproj/project.pbxproj: Update MACOSX_DEPLOYMENT_TARGET to 10.3 and add -fobjc-exceptions
2042         so we can use new exceptions.
2043
2044         * WebView.subproj/WebHTMLView.m:
2045         (-[WebHTMLView setMark:]): Added.
2046         (unionDOMRanges): Added.
2047         (-[WebHTMLView deleteToMark:]): Added.
2048         (-[WebHTMLView selectToMark:]): Added.
2049         (-[WebHTMLView swapWithMark:]): Added.
2050         (-[WebHTMLView markedRange]): Updated for change to bridge method names.
2051         (-[WebHTMLView hasMarkedText]): Ditto.
2052         (-[WebHTMLView unmarkText]): Ditto.
2053         (-[WebHTMLView _selectMarkedText]): Ditto.
2054         (-[WebHTMLView _selectRangeInMarkedText:]): Ditto.
2055         (-[WebHTMLView setMarkedText:selectedRange:]): Ditto.
2056         (-[WebHTMLView _insertText:selectInsertedText:]): Removed check for empty string. An empty string
2057         should not be filtered out here. We need to allow inserting an empty string.
2058         (-[WebHTMLView _selectionIsInsideMarkedText]): Updated for change to bridge method names.
2059         (-[WebHTMLView _updateSelectionForInputManager]): Ditto.
2060
2061         * WebView.subproj/WebView.m: (-[WebView searchFor:direction:caseSensitive:wrap:]):
2062         Changed to use selectionDOMRange instead of selectionStart.
2063
2064         * WebView.subproj/WebHTMLRepresentation.h: Removed unused setSelectionFrom method.
2065         * WebView.subproj/WebHTMLRepresentation.m: Ditto.
2066
2067 2004-09-08  John Sullivan  <sullivan@apple.com>
2068
2069         Reviewed by Darin.
2070         
2071         - fixed the localization aspect of: 
2072         <rdar://problem/3790011> undoable operations all say "Undo" in the menu, no specific action names
2073         
2074         We now have all the strings ready for localization; we just don't actually use them yet.
2075
2076         * English.lproj/Localizable.strings:
2077         updated for this change
2078         
2079         * WebCoreSupport.subproj/WebBridge.m:
2080         (-[WebBridge setUndoActionNamePlaceholder]):
2081         added this placeholder method whose purpose is to hold localizable strings for all the Undo
2082         action names that NSTextView uses. Later we will use some or all of these, but we can do
2083         that part after the localization freeze.
2084
2085 2004-09-08  Chris Blumenberg  <cblu@apple.com>
2086
2087         Fixed: <rdar://problem/3778785> REGRESSION (Mail): copying from MS word and pasting into editable region leaves internal clipboard data
2088
2089         Reviewed by kocienda.
2090
2091         * WebView.subproj/WebHTMLView.m:
2092         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): ignore Microsoft's header meta data
2093
2094 2004-09-08  Chris Blumenberg  <cblu@apple.com>
2095
2096         Fixed: <rdar://problem/3792893> WebBaseResourceHandleDelegate always returns cached data for subresource loads
2097
2098         Reviewed by rjw.
2099
2100         * WebView.subproj/WebBaseResourceHandleDelegate.m:
2101         (-[WebBaseResourceHandleDelegate _canUseResourceForRequest:]): new
2102         (-[WebBaseResourceHandleDelegate loadWithRequest:]): call _canUseResourceForRequest:
2103
2104 2004-09-08  Chris Blumenberg  <cblu@apple.com>
2105
2106         Forgot to add this in previous check-in.        
2107
2108         * English.lproj/WebViewEditingContextMenu.nib: Added.
2109
2110 2004-09-08  Chris Blumenberg  <cblu@apple.com>
2111
2112         Fixed: <rdar://problem/3791240> WebKit uses the NSTextViewContextMenu nib from inside AppKit
2113
2114         Reviewed by john.
2115
2116         * English.lproj/StringsNotToBeLocalized.txt:
2117         * WebKit.pbproj/project.pbxproj:
2118         * WebView.subproj/WebDefaultContextMenuDelegate.m:
2119         (-[WebDefaultUIDelegate editingContextMenuItemsForElement:]): use our copy of the nib
2120
2121 2004-09-07  Darin Adler  <darin@apple.com>
2122
2123         * English.lproj/StringsNotToBeLocalized.txt: Update for recent changes.
2124
2125 2004-09-07  Chris Blumenberg  <cblu@apple.com>
2126
2127         Fixed: <rdar://problem/3790143> exception raised when dragging a URL with 2-byte characters
2128
2129         (checked in with last check-in)
2130
2131 2004-09-07  Chris Blumenberg  <cblu@apple.com>
2132
2133         Fixed:
2134         <rdar://problem/3080103> Need to pass cmd-modified keys to plug-ins
2135         <rdar://problem/3751509> can't use safari edit menu to copy and paste with Vantage Learning's My Access
2136
2137         Reviewed by john.
2138
2139         * Plugins.subproj/WebBaseNetscapePluginView.m: fake up command-key events for cut, copy, paste and select all so these menu items work for plug-ins
2140         (-[WebBaseNetscapePluginView sendModifierEventWithKeyCode:character:]):
2141         (-[WebBaseNetscapePluginView cut:]):
2142         (-[WebBaseNetscapePluginView copy:]):
2143         (-[WebBaseNetscapePluginView paste:]):
2144         (-[WebBaseNetscapePluginView selectAll:]):
2145
2146 2004-09-07  Darin Adler  <darin@apple.com>
2147
2148         - fixed deployment build
2149
2150         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView _deleteWithDirection:granularity:killRing:]):
2151         Initialize prepend variable.
2152
2153 2004-09-06  Darin Adler  <darin@apple.com>
2154
2155         Reviewed by John.
2156
2157         - fixed <rdar://problem/3696542> REGRESSION (Mail): Editable WebKit doesn't support underline yet (in the iChat profile window, at least)
2158         - fixed <rdar://problem/3780249> REGRESSION (Mail): copy style/paste style doesn't work in HTML editing in Mail
2159         - fixed <rdar://problem/3788857> REGRESSION (Mail): Home and End keys don't work in message composer
2160         - fixed <rdar://problem/3788884> REGRESSION (Mail): ctrl-d emacs key binding does not work (delete forward)
2161         - fixed <rdar://problem/3788890> REGRESSION (Mail): ctrl-k emacs key binding does not work (delete to end of paragraph)
2162         - fixed <rdar://problem/3788899> REGRESSION (Mail): ctrl-y emacs key binding does not work (yank)
2163         - fixed <rdar://problem/3788901> REGRESSION (Mail): ctrl-o emacs key binding does not work (insert newline in front of insertion point)
2164         - fixed <rdar://problem/3788908> REGRESSION (Mail): ctrl-left-arrow emacs key binding does not work (move to beginning of line)
2165         - fixed <rdar://problem/3788913> REGRESSION (Mail): ctrl-right-arrow emacs key binding does not work (move to end of line)
2166         - implemented a first cut at other attribute changes from Text Panel besides underline (bugs?)
2167         - dealt with a couple of FIXMEs in WebHTMLView.m
2168         - updated list of not-yet-implemented methods in WebHTMLView.m
2169         - fixed many deletion operations to call the correct editing delegate methods
2170
2171         * WebView.subproj/WebFrameViewPrivate.h: Remove _scrollToTopLeft and _scrollToBottomLeft.
2172         No one was calling them anyway, so they should really have been marked internal and not private.
2173         * WebView.subproj/WebFrameView.m:
2174         (-[WebFrameView scrollToBeginningOfDocument:]): Renamed _scrollToTopLeft to this, so the home key
2175         would start working with the key bindings machinery.
2176         (-[WebFrameView scrollToEndOfDocument:]): Same thing, for end key.
2177         (-[WebFrameView keyDown:]): Update for name changes, and also make sure we don't try to grab
2178         control-arrow keys here (probably not necessary, but good anyway).
2179
2180         * WebView.subproj/WebHTMLViewInternal.h: Added keyDownEvent field, and startNewKillRingSequence
2181         and nextResponderDisabledOnce flags.
2182         * WebView.subproj/WebHTMLView.m:
2183         Rearrange declarations at the top of the file so that external things are up with
2184         the #import directives and things inside this file are declared below.
2185         (-[WebHTMLView _shouldReplaceSelectionWithText:givenAction:]): Ditto.
2186         (-[WebHTMLView _calculatePrintHeight]): Moved up into the "internal to file" category.
2187         (-[WebHTMLView _updateTextSizeMultiplier]): Ditto.
2188         (-[WebHTMLView _selectedRange]): Added.
2189         (-[WebHTMLView _openLinkFromMenu:]): Left this method lying around even though I deleted the
2190         other APPKIT_CODE_FOR_REFERENCE in case this shows up in the context menu we are now sharing
2191         with the AppKit. Chris will look at this later, and he can delete it then.
2192         (+[WebHTMLView initialize]): Call _NSInitializeKillRing.
2193         (-[WebHTMLView _documentRange]): Added.
2194         (-[WebHTMLView string]): Call the bridge to get the plain text rather than making an attributed
2195         string and then getting the text from there.
2196         (-[WebHTMLView becomeFirstResponder]): Set startNewKillRingSequence flag, so that new deletions
2197         will create a new kill ring entry.
2198         (-[WebHTMLView moveToBeginningOfDocument:]): Use backward direction instead of left direction.
2199         (-[WebHTMLView moveToBeginningOfDocumentAndModifySelection:]): Ditto.
2200         (-[WebHTMLView moveToBeginningOfLine:]): Ditto.
2201         (-[WebHTMLView moveToBeginningOfLineAndModifySelection:]): Ditto.
2202         (-[WebHTMLView moveToBeginningOfParagraph:]): Ditto, also use WebSelectToParagraphBoundary.
2203         (-[WebHTMLView moveToBeginningOfParagraphAndModifySelection:]): Ditto.
2204         (-[WebHTMLView moveToEndOfDocument:]): Use forward direction instead of right direction.
2205         (-[WebHTMLView moveToEndOfDocumentAndModifySelection:]): Ditto.
2206         (-[WebHTMLView moveToEndOfLine:]): Ditto.
2207         (-[WebHTMLView moveToEndOfLineAndModifySelection:]): Ditto.
2208         (-[WebHTMLView moveToEndOfParagraph:]): Ditto, also use WebSelectToParagraphBoundary.
2209         (-[WebHTMLView moveToEndOfParagraphAndModifySelection:]): Ditto.
2210         (-[WebHTMLView _shouldDeleteRange:]): Added.
2211         (-[WebHTMLView _deleteRange:preflight:killRing:prepend:]): Added.
2212         (-[WebHTMLView delete:]): Changed to call new _deleteRange method.
2213         (-[WebHTMLView cut:]): Changed to preflight property and call new _deleteRange method.
2214         (-[WebHTMLView _selectionFontAttributes]): Added.
2215         (-[WebHTMLView _selectionFontAttributesAsRTF]): Added.
2216         (-[WebHTMLView _fontAttributesFromFontPasteboard]): Added.
2217         (-[WebHTMLView _emptyStyle]): Added.
2218         (-[WebHTMLView _styleFromFontAttributes:]): Added.
2219         (-[WebHTMLView _applyStyleToSelection:]): Added.
2220         (-[WebHTMLView copyFont:]): Implemented.
2221         (-[WebHTMLView pasteFont:]): Implemented.
2222         (-[WebHTMLView _originalFontA]): Added.
2223         (-[WebHTMLView _originalFontB]): Added.
2224         (-[WebHTMLView _addToStyle:fontA:fontB:]): Added. Has code from the method that figures out
2225         what the font manager is doing for changeFont:, now needed for changeAttribute: too.
2226         (-[WebHTMLView _styleFromFontManagerOperation]): Renamed and now calls shared methods.
2227         (-[WebHTMLView changeFont:]): Call shared method, still does the same thing.
2228         (-[WebHTMLView _colorAsString:]): Added. Has code from the method we were using with the
2229         color panel before.
2230         (-[WebHTMLView _shadowAsString:]): Added.
2231         (-[WebHTMLView _styleForAttributeChange:]): Added.
2232         (-[WebHTMLView changeAttributes:]): Implemented.
2233         (-[WebHTMLView _styleFromColorPanelWithSelector:]): Renamed and now calls shared methods.
2234         (-[WebHTMLView _changeCSSColorUsingSelector:inRange:]): Call method by new name.
2235         (-[WebHTMLView changeDocumentBackgroundColor:]): Call method by new name.
2236         (-[WebHTMLView changeColor:]): Changed around a bit; still doesn't work yet.
2237         (-[WebHTMLView _alignSelectionUsingCSSValue:]): Call shared methods.
2238         (-[WebHTMLView indent:]): Removed, since NSTextView doesn't implement this method. Added to list
2239         of methods to possibly implement later in the file.
2240         (-[WebHTMLView insertTab:]): Call insertText: to save code and so we get WebViewInsertActionTyped
2241         instead of WebViewInsertActionPasted.
2242         (-[WebHTMLView changeCaseOfLetter:]): Removed, since NSTextView doesn't implement this method.
2243         Added to list of methods to possibly implement later in the file.
2244         (-[WebHTMLView _deleteWithDirection:granularity:killRing:]): Added.
2245         (-[WebHTMLView deleteForward:]): Implemented. This makes Control-D work.
2246         (-[WebHTMLView deleteBackwardByDecomposingPreviousCharacter:]): Implemented by just calling
2247         deleteBackward for now; probably better than doing nothing.
2248         (-[WebHTMLView deleteWordForward:]): Changed to call new _delete method above. Fixes things
2249         so that we delete the selection if there is one, get the appropriate delegate calls, handle
2250         the kill ring properly, and don't do any selection if we can't delete.
2251         (-[WebHTMLView deleteWordBackward:]): Ditto.
2252         (-[WebHTMLView deleteToBeginningOfLine:]): Ditto.
2253         (-[WebHTMLView deleteToEndOfLine:]): Ditto.
2254         (-[WebHTMLView deleteToBeginningOfParagraph:]): Ditto.
2255         (-[WebHTMLView deleteToEndOfParagraph:]): Ditto. Added additional behavior needed since this
2256         is bound to Control-K, so it's not really just delete to end of paragraph.
2257         (-[WebHTMLView insertNewlineIgnoringFieldEditor:]): Added. Calls insertNewline:.
2258         (-[WebHTMLView insertTabIgnoringFieldEditor:]): Added. Calls insertTab:.
2259         (-[WebHTMLView subscript:]): Added.
2260         (-[WebHTMLView superscript:]): Added.
2261         (-[WebHTMLView unscript:]): Added.
2262         (-[WebHTMLView underline:]): Added.
2263         (-[WebHTMLView yank:]): Added.
2264         (-[WebHTMLView yankAndSelect:]): Added. Calls _insertText.
2265         (-[WebHTMLView _arrowKeyDownEventSelectorIfPreprocessing:]): Added. Part of workaround for
2266         control-arrow key trouble.
2267         (-[WebHTMLView respondsToSelector:]): Added. More of workaround.
2268         (-[WebHTMLView nextResponder:]): Added. More of workaround.
2269         (-[WebHTMLView _selectionChanged]): Set startNewKillRingSequence flag, so that new deletions
2270         will create a new kill ring entry.
2271         (-[WebHTMLView _updateFontPanel]): Remove a bunch of code here that wasn't working very well
2272         because it walked a DOM range incorrectly, and instead use the new method that does all the
2273         right stuff on the other side of the bridge.
2274         (-[WebHTMLView _insertText:selectInsertedText:]): Added new helper method for use by both
2275         insertText and yankAndSelect, with most of the guts of insertText and one additional parameter.
2276         (-[WebHTMLView insertText:]): Call the new _insertText.
2277
2278         * WebView.subproj/WebView.m: Use macros to make the forwarding from WebView more terse.
2279         Updated the list to include a few methods it didn't before.
2280
2281 2004-09-06  John Sullivan  <sullivan@apple.com>
2282
2283         Reviewed by Darin.
2284         
2285         - put preference keys in a private header file so Safari can use them for Managed Preferences
2286
2287         * WebView.subproj/WebPreferenceKeysPrivate.h:
2288         new Private header file, includes the #defines for NSUserDefaults preference keys
2289         
2290         * WebView.subproj/WebPreferences.m:
2291         moved the preference key #defines out of here; now imports WebPreferenceKeysPrivate.h
2292
2293         * WebKit.pbproj/project.pbxproj:
2294         updated for new file
2295
2296 2004-09-03  Chris Blumenberg  <cblu@apple.com>
2297
2298         Fixed: <rdar://problem/3782543> CrashTracer: ...87 crashes at com.apple.WebKit: -[WebNetscapePluginPackage initWithPath:] + 0x18c
2299
2300         Reviewed by john.
2301
2302         * Plugins.subproj/WebNetscapePluginPackage.m:
2303         (-[WebNetscapePluginPackage initWithPath:]): make sure the file is at least 8 bytes long before calling memcmp
2304
2305 2004-09-03  Chris Blumenberg  <cblu@apple.com>
2306         
2307         Fixed:
2308         <rdar://problem/3788328> assertion failure when moving an image
2309         <rdar://problem/3783628> REGRESSION (Mail): when I try to reorder an image, the image is duplicated
2310
2311         Reviewed by NOBODY (OOPS!).
2312
2313         * DOM.subproj/WebDOMOperations.m:
2314         (-[DOMDocument _createRangeWithNode:]): new, convenience
2315         (-[DOMDocument _documentRange]): use _ createRangeWithNode:
2316         * DOM.subproj/WebDOMOperationsPrivate.h:
2317         * WebView.subproj/WebHTMLView.m:
2318         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]): set the selection to the image when starting the drag. This allows "move selection" to work and this matches NSText's behavior
2319
2320 === Safari-162 ===
2321
2322 2004-09-02  Richard Williamson   <rjw@apple.com>
2323
2324         Support for patterns in <canvas>.  
2325         (These changes attempt to create a CGImageRef from a WebImageRenderer that
2326         is used by the pattern drawing function.)
2327         
2328         Reviewed by Hyatt.
2329
2330         * WebCoreSupport.subproj/WebImageRenderer.h:
2331         * WebCoreSupport.subproj/WebImageRenderer.m:
2332         (-[WebImageRenderer dealloc]):
2333         (-[WebImageRenderer finalize]):
2334         (-[WebImageRenderer imageRef]):
2335         (_createImageRef):
2336         * WebView.subproj/WebPDFView.h:
2337         * WebView.subproj/WebPDFView.m:
2338
2339 2004-09-01  Chris Blumenberg  <cblu@apple.com>
2340
2341         Fixed deployment build failure.
2342
2343         * WebView.subproj/WebDefaultContextMenuDelegate.m:
2344         (-[WebDefaultUIDelegate menuItemWithTag:]):
2345
2346 2004-08-31  Chris Blumenberg  <cblu@apple.com>
2347
2348         Fixed: 
2349         <rdar://problem/3699498> Context menu for editable WebViews should provide items like Cut and Paste
2350         <rdar://problem/3781535> REGRESSION (Mail): no context menu after ctrl-clicking a misspelled word
2351
2352         Reviewed by kocienda.
2353
2354         * English.lproj/Localizable.strings:
2355         * WebView.subproj/WebDefaultContextMenuDelegate.m:
2356         (-[WebDefaultUIDelegate menuItemWithTag:]): updated to handle new menu items
2357         (-[WebDefaultUIDelegate contextMenuItemsForElement:]): renamed from webView:contextMenuItemsForElement:defaultMenuItems:
2358         (-[WebDefaultUIDelegate editingContextMenuItemsForElement:]): new
2359         (-[WebDefaultUIDelegate webView:contextMenuItemsForElement:defaultMenuItems:]): moved, now call contextMenuItemsForElement: or editingContextMenuItemsForElement:
2360         * WebView.subproj/WebDefaultUIDelegate.h:
2361         * WebView.subproj/WebHTMLView.m:
2362         (-[WebHTMLView _isSelectionMisspelled]): new
2363         (-[WebHTMLView _guessesForMisspelledSelection]): new
2364         (-[WebHTMLView _changeSpellingFromMenu:]): new
2365         (-[WebHTMLView _ignoreSpellingFromMenu:]): new
2366         (-[WebHTMLView _learnSpellingFromMenu:]): new
2367         * WebView.subproj/WebHTMLViewPrivate.h:
2368         * WebView.subproj/WebUIDelegate.h:
2369
2370 2004-08-31  Darin Adler  <darin@apple.com>
2371
2372         - fixed B&I build failure
2373
2374         * WebView.subproj/WebHTMLView.m: (-[WebTextCompleteController _buildUI]):
2375         Work around unwanted warning by adding a cast.
2376
2377 2004-08-27  Maciej Stachowiak  <mjs@apple.com>
2378
2379         Reviewed by John.
2380
2381         <rdar://problem/3778314> REGRESSION: Can't proceed to survey questions on Lominger's Apple website
2382         
2383         Because we will stop parsing when there is a pending redirection,
2384         avoid setting one if no navigation would actually take place
2385         because the number of steps is out of range.
2386
2387         * WebCoreSupport.subproj/WebBridge.m:
2388         (-[WebBridge canGoBackOrForward:]):
2389
2390 2004-08-30  Darin Adler  <darin@apple.com>
2391
2392         Reviewed by John.
2393
2394         - part of fix for <rdar://problem/3637519> REGRESSION (125-128): unrepro crash in QListBox::sizeForNumberOfLines at istweb.apple.com
2395
2396         * WebCoreSupport.subproj/WebTextRendererFactory.m: (-[WebTextRendererFactory clearCaches]):
2397         Call [super clearCaches].
2398
2399 2004-08-30  Darin Adler  <darin@apple.com>
2400
2401         Reviewed by Chris.
2402
2403         - did work to prepare for uploading files incrementally when submitting forms
2404
2405         * History.subproj/WebHistoryItem.m:
2406         (-[WebHistoryItem _setFormInfoFromRequest:]): Use NSArray instead of NSData for form data.
2407         (-[WebHistoryItem formData]): Ditto.
2408         * History.subproj/WebHistoryItemPrivate.h: Ditto.
2409         * WebCoreSupport.subproj/WebBridge.m:
2410         (-[WebBridge startLoadingResource:withURL:customHeaders:postData:]): Ditto.
2411         (-[WebBridge syncLoadResourceWithURL:customHeaders:postData:finalURL:responseHeaders:statusCode:]): Ditto.
2412         (-[WebBridge postWithURL:referrer:target:data:contentType:triggeringEvent:form:formValues:]): Ditto.
2413         * WebCoreSupport.subproj/WebSubresourceClient.h: Ditto.
2414         * WebCoreSupport.subproj/WebSubresourceClient.m:
2415         (+[WebSubresourceClient startLoadingResource:withURL:customHeaders:postData:referrer:forDataSource:]): Ditto.
2416         * WebView.subproj/WebFrame.m:
2417         (-[WebFrame _loadItem:withLoadType:]): Ditto.
2418         (-[WebFrame _postWithURL:referrer:target:data:contentType:triggeringEvent:form:formValues:]): Ditto.
2419         * WebView.subproj/WebFramePrivate.h: Ditto.
2420
2421         * WebView.subproj/WebFormDataStream.h: Added.
2422         * WebView.subproj/WebFormDataStream.m:
2423         (-[WebFormDataStream initWithFormDataArray:]): Placeholder; not done yet.
2424         (-[WebFormDataStream formDataArray]): Ditto.
2425         * WebKit.pbproj/project.pbxproj: Added WebFormDataStream files.
2426
2427 2004-08-30  John Sullivan  <sullivan@apple.com>
2428
2429         Reviewed by Ken.
2430
2431         - WebKit part of fix for <rdar://problem/3607720> myFrame.print() prints the window 
2432         but should only print the frame
2433
2434         * WebView.subproj/WebUIDelegatePrivate.h:
2435         declare new delegate method that includes which frame to print
2436
2437         * WebView.subproj/WebDefaultUIDelegate.m:
2438         (-[WebDefaultUIDelegate webView:printFrameView:]):
2439         implement default (no-op) version of new delegate method
2440
2441         * WebCoreSupport.subproj/WebBridge.m:
2442         (-[WebBridge print]):
2443         call new delegate method if available, otherwise call old delegate method, for
2444         backward compatibility.
2445
2446 2004-08-27  Maciej Stachowiak  <mjs@apple.com>
2447
2448         Reviewed by John. 
2449         Checked in by Ken
2450         
2451         Ken comments: It looks like Maciej forgot to land this when he checked in the
2452         WebCore portion of this change.
2453
2454         <rdar://problem/3778314> REGRESSION: Can't proceed to survey questions on Lominger's Apple website
2455             
2456         Because we will stop parsing when there is a pending redirection,
2457         avoid setting one if no navigation would actually take place
2458         because the number of steps is out of range.
2459     
2460         * WebCoreSupport.subproj/WebBridge.m:
2461         (-[WebBridge canGoBackOrForward:]):
2462
2463 2004-08-27  Ken Kocienda  <kocienda@apple.com>
2464
2465         Reviewed by Chris
2466
2467         Fix for this bug:
2468         
2469         <rdar://problem/3756997> WebKit aggressive in making pasted text into a URL, even when it's not much like a URL
2470
2471         * WebView.subproj/WebHTMLView.m:
2472         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): Don't try so hard to coerce data
2473         on the pasteboard into a URL, and do not make an anchor with an href for any URLs that are 
2474         explicitly present on the pasteboard. Also, move URL pasteboard type check beneath the RTF
2475         checks.
2476
2477 === Safari-161 ===
2478
2479 2004-08-26  Chris Blumenberg  <cblu@apple.com>
2480
2481         Fixed build failure on Panther.
2482
2483         Reviewed by john.
2484
2485         * WebView.subproj/WebHTMLView.m:
2486         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): ifdef'd out call to AppKit SPI
2487         (-[WebHTMLView _attributeStringFromDOMRange:]): ditto
2488
2489 2004-08-26  Chris Blumenberg  <cblu@apple.com>
2490
2491         Tweak to last check-in.
2492
2493         * WebView.subproj/WebHTMLView.m:
2494         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]):
2495
2496 2004-08-26  Chris Blumenberg  <cblu@apple.com>
2497
2498         Fixed: 
2499         <rdar://problem/3546412> support for pasting and drag and dropping of RTF and RTFD to editable WebViews
2500         <rdar://problem/3745345> use AppKit for converting from DOM to RTF
2501
2502         Reviewed by rjw.
2503
2504         * DOM.subproj/WebDOMOperations.m:
2505         (-[DOMDocument _documentRange]): new
2506         * DOM.subproj/WebDOMOperationsPrivate.h:
2507         * WebView.subproj/WebHTMLView.m:
2508         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): call AppKit SPI to get a document fragment from an attributed string
2509         (-[WebHTMLView string]): added a FIXME
2510         (-[WebHTMLView _attributeStringFromDOMRange:]): new, calls AppKit SPI that creates an attributed string from a DOM Range
2511         (-[WebHTMLView attributedString]): call _attributeStringFromDOMRange:, fallback to old code if it returns nil
2512         (-[WebHTMLView selectedAttributedString]): ditto
2513
2514 2004-08-26  Chris Blumenberg  <cblu@apple.com>
2515
2516         Fixed: <rdar://problem/3774178> Plugin hooks for selected state aren't being called
2517
2518         Reviewed by darin.
2519
2520         * WebCoreSupport.subproj/WebBridge.m:
2521         (-[WebBridge setIsSelected:forView:]): don't forget colons in method names
2522
2523 2004-08-26  Chris Blumenberg  <cblu@apple.com>
2524
2525         Fixed: <rdar://problem/3768439> can't click in WebView in Carbon WebKit apps (GetEventPlatformEventRecord returns false)
2526
2527         Reviewed by rjw.
2528
2529         * Carbon.subproj/HIWebView.m:
2530         (Click): use WebGetEventPlatformEventRecord not GetEventPlatformEventRecord
2531         (MouseUp): ditto
2532         (MouseMoved): ditto
2533         (MouseDragged): ditto
2534         (MouseWheelMoved): ditto
2535         (WindowHandler): ditto
2536         (HIWebViewEventHandler): ditto
2537         (UpdateObserver): ditto
2538         (WebGetEventPlatformEventRecord): Call GetEventPlatformEventRecord, if that fails fallback to the current event. This is code Eric Schlegel to me to use. 
2539
2540 2004-08-24  Chris Blumenberg  <cblu@apple.com>
2541
2542         Fixed with help from Trey:
2543         <rdar://problem/3764856> REGRESSION !25-154): Safari accepts mouse clicks (follows links) when not key
2544
2545         Reviewed by rjw.
2546
2547         * WebView.subproj/WebHTMLView.m:
2548         (-[WebHTMLView _isSelectionEvent:]): brought back from CVS
2549         (-[WebHTMLView acceptsFirstMouse:]): only call eventMayStartDrag if _isSelectionEvent returns YES since we only want to allow selection dragging on the first mouse down
2550         (-[WebHTMLView shouldDelayWindowOrderingForEvent:]): ditto
2551
2552 2004-08-24  Ken Kocienda  <kocienda@apple.com>
2553
2554         Reviewed by Hyatt
2555
2556         Improved the checks used to see if certain operations can be done based
2557         on the state of the selection and whether the selection is editable. I
2558         added some helpers and improved some others to assist in making these
2559         determinations.
2560         
2561         This helps to fix this bug:
2562         <rdar://problem/3764987> Crash after adding newline to quoted text
2563         
2564         Since some editing methods expect the the selection to be in a certain state 
2565         in order to work, these checks help obviate crashes like 3764987.
2566
2567         * WebView.subproj/WebHTMLView.m:
2568         (-[WebHTMLView _writeSelectionToPasteboard:]): _haveSelection name changed to _hasSelection.
2569         (-[WebHTMLView _canCopy]): Checks to see if state is appropriate to perform this operation.
2570         (-[WebHTMLView _canCut]): Ditto. Function added.
2571         (-[WebHTMLView _canDelete]): Ditto. Function refined.
2572         (-[WebHTMLView _canPaste]): Ditto. Function refined.
2573         (-[WebHTMLView _canType]): Ditto. Function added.
2574         (-[WebHTMLView _hasSelection]): Name changed from _haveSelection.
2575         (-[WebHTMLView _hasSelectionOrInsertionPoint]): Added.
2576         (-[WebHTMLView _isEditable]): Added.
2577         (-[WebHTMLView takeFindStringFromSelection:]): _haveSelection name changed to _hasSelection.
2578         (-[WebHTMLView validateUserInterfaceItem:]): Ditto
2579         (-[WebHTMLView validRequestorForSendType:returnType:]): Ditto
2580         (-[WebHTMLView keyDown:]):
2581         (-[WebHTMLView copy:]): Uses new _canCopy check.
2582         (-[WebHTMLView cut:]): Uses new _canCut check.
2583         (-[WebHTMLView delete:]): Now uses _canDelete check.
2584         (-[WebHTMLView paste:]): Now uses _canPaste check.
2585         (-[WebHTMLView _updateFontPanel]): _haveSelection name changed to _hasSelection.
2586         * WebView.subproj/WebHTMLViewPrivate.h:
2587         * WebView.subproj/WebView.m:
2588         (-[WebView writeSelectionWithPasteboardTypes:toPasteboard:]): Use selectionState check to
2589         determine whether or not operation can be done.
2590
2591 2004-08-24  Richard Williamson   <rjw@apple.com>
2592
2593         Fixed <rdar://problem/3770469> Some PDFs open with line of previous page above PDF view
2594         
2595         Set height resize flags on WebPDFView.
2596                 
2597         Reviewed by Chris.
2598
2599         * WebView.subproj/WebPDFView.m:
2600         (-[WebPDFView initWithFrame:]):
2601
2602 2004-08-24  David Hyatt  <hyatt@apple.com>
2603
2604         Add Atom and RSS MIME types to set of supported XML types.
2605         
2606         Reviewed by rjw
2607
2608         * WebView.subproj/WebDataSource.m:
2609         (+[WebDataSource _repTypesAllowImageTypeOmission:]):
2610         * WebView.subproj/WebFrameView.m:
2611         (+[WebFrameView _viewTypesAllowImageTypeOmission:]):
2612
2613 2004-08-23  Maciej Stachowiak  <mjs@apple.com>
2614
2615         Reviewed by Kevin.
2616
2617         - remove annoying ERROR spew and replace with comment
2618         
2619         * WebView.subproj/WebHTMLView.m:
2620         (-[WebHTMLView validAttributesForMarkedText]):
2621
2622 2004-08-23  Maciej Stachowiak  <mjs@apple.com>
2623
2624         Reviewed by Richard.
2625
2626         - a bit of code cleanup
2627
2628         * WebView.subproj/WebDataSource.m:
2629         (-[WebDataSource _stringWithData:]): Call textEncodingName instead of
2630         copying it's code.
2631         (-[WebDataSource textEncodingName]): Tweak formatting.
2632
2633 2004-08-23  Chris Blumenberg  <cblu@apple.com>
2634
2635         Fixed build.
2636
2637         * WebKit.pbproj/project.pbxproj: make sure we're doing -f on a file, not a directory
2638
2639 2004-08-23  Chris Blumenberg  <cblu@apple.com>
2640
2641         Fixed: <rdar://problem/3674867> use new Security framework SPI's to reenable cert acquisition
2642
2643         Reviewed by john.
2644
2645         * WebCoreSupport.subproj/WebKeyGeneration.cpp:
2646         * WebCoreSupport.subproj/WebKeyGeneration.h:
2647         * WebCoreSupport.subproj/WebKeyGenerator.h:
2648         * WebCoreSupport.subproj/WebKeyGenerator.m:
2649         (-[WebKeyGenerator signedPublicKeyAndChallengeStringWithStrengthIndex:challenge:pageURL:]): re-enabled cert code, call Panther version on Panther, Tiger version on Tiger
2650         (-[WebKeyGenerator addCertificatesToKeychainFromData:]): ditto
2651         * WebCoreSupport.subproj/WebNewKeyGeneration.c: Added.
2652         (gnrAddContextAttribute):
2653         (gnrGetSubjPubKey):
2654         (gnrNullAlgParams):
2655         (gnrSign):
2656         (gnrFreeCssmData):
2657         (nssArraySize):
2658         (signedPublicKeyAndChallengeString):
2659         (addCertificateToKeychainFromData):
2660         (addCertificatesToKeychainFromData):
2661         * WebCoreSupport.subproj/WebNewKeyGeneration.h: Added.
2662         * WebKit.pbproj/project.pbxproj:
2663
2664 2004-08-20  David Hyatt  <hyatt@apple.com>
2665
2666         Fix the directionality of the unicode hyphen so that on Panther it now matches Tiger.
2667         
2668         Reviewed by darin
2669
2670         * Misc.subproj/WebUnicode.m:
2671         (_unicodeDirection):
2672
2673 2004-08-20  Richard Williamson   <rjw@apple.com>
2674
2675             Implemented new JNI abstraction.  We no longer invoke Java methods
2676             directly with JNI, rather we call into the plugin.  This allows the
2677             plugin to dispatch the call to the appropriate VM thread.  This
2678             change should (will?) fix a whole class of threading related problems with
2679             the Java VM.
2680
2681         Reviewed by Hyatt.
2682
2683         * WebCoreSupport.subproj/WebBridge.m:
2684         (-[WebBridge getAppletInView:]):
2685
2686 2004-08-20  Trey Matteson  <trey@apple.com>
2687
2688         3655407 - Editing: -complete: method unimplemented (WebKit editing API)
2689
2690         This feature is mostly implemented.  The only loose ends I know of are:
2691
2692         3769654 - text insertions done via complete: should preserve case of full replacement string
2693         3769652 - positioning of complete: popup window wrong for right-to-left languages
2694
2695         Reviewed by John
2696
2697         * WebView.subproj/WebHTMLView.m:
2698         (-[WebHTMLViewPrivate dealloc]):  Free new object.
2699         (-[WebHTMLView menuForEvent:]):  Bail on completion session.
2700         (-[WebHTMLView windowDidResignKey:]):  Ditto.
2701         (-[WebHTMLView windowWillClose:]):  Ditto.
2702         (-[WebHTMLView mouseDown:]):  Ditto.
2703         (-[WebHTMLView resignFirstResponder]):  Ditto.
2704         (-[WebHTMLView keyDown:]):  Bail on completion session if WebCore takes the event.
2705         Give the CompleteController a crack at the key event.
2706         (-[WebHTMLView _expandSelectionToGranularity:]):  Adopt method rename
2707         (-[WebHTMLView complete:]):  Make CompleteController, tell it to do its thing.
2708         (-[WebHTMLView checkSpelling:]):  Add ERROR().
2709         (-[WebHTMLView showGuessPanel:]): Add ERROR().
2710         (-[WebHTMLView _changeSpellingToWord:]): Add ERROR().
2711         (-[WebHTMLView ignoreSpelling:]): Add ERROR().
2712
2713         (-[WebTextCompleteController initWithHTMLView:]):
2714         (-[WebTextCompleteController dealloc]):
2715         (-[WebTextCompleteController _insertMatch:]): Stick the new string into the doc.
2716         (-[WebTextCompleteController _buildUI]):  Make popup window.
2717         (-[WebTextCompleteController _placePopupWindow:]):  Position popup window.
2718         (-[WebTextCompleteController doCompletion]):  Lookup matches, display window.
2719         (-[WebTextCompleteController endRevertingChange:moveLeft:]):  Bail on complete: session.
2720         (-[WebTextCompleteController filterKeyDown:]):  Process keys while popup is up.
2721         (-[WebTextCompleteController _reflectSelection]):  Handle choice in popup.
2722         (-[WebTextCompleteController tableAction:]):  Handle double click in popup
2723         (-[WebTextCompleteController numberOfRowsInTableView:]):  Fill table with matches.
2724         (-[WebTextCompleteController tableView:objectValueForTableColumn:row:]):  Ditto.
2725         (-[WebTextCompleteController tableViewSelectionDidChange:]):  Handle selection change.
2726         * WebView.subproj/WebHTMLViewInternal.h:
2727
2728 2004-08-20  John Sullivan  <sullivan@apple.com>
2729
2730         * English.lproj/StringsNotToBeLocalized.txt:
2731         checked this in; the only difference is that
2732         the strings were out of order.
2733
2734 2004-08-19  Richard Williamson   <rjw@apple.com>
2735
2736         Continue to call old pollForAppletInView: in Tiger until
2737         we get an plugin that supports the new API.
2738                 
2739         Reviewed by Chris.
2740
2741         * WebCoreSupport.subproj/WebBridge.m:
2742         (-[WebBridge getAppletInView:]):
2743         * WebKit.pbproj/project.pbxproj:
2744
2745 === Safari-158 ===
2746
2747 2004-08-19  Chris Blumenberg  <cblu@apple.com>
2748
2749         Fixed typo in comment.
2750
2751         * Misc.subproj/WebIconDatabase.m:
2752         (-[WebIconDatabase _updateFileDatabase]):
2753
2754 2004-08-19  Chris Blumenberg  <cblu@apple.com>
2755
2756         Fixed: <rdar://problem/3566336> CrashTracer: .2403 crashes at com.apple.WebKit: -[WebFileDatabase performSetObject:forKey:] + 0x94
2757
2758         Reviewed by trey.
2759
2760         * Misc.subproj/WebIconDatabase.m:
2761         (-[WebIconDatabase _updateFileDatabase]): pass WebFileDatabase copies of the mutable dictionaries or else they may be accessed on a separate thread as the main thread is modifying them
2762
2763 2004-08-18  Richard Williamson   <rjw@apple.com>
2764
2765         Replace horrible pollForAppletInView: with new
2766         webPlugInGetApplet.  The details of how the applet instance
2767         is provided now belong to the Java team.  Yeh.
2768                 
2769         Reviewed by Chris.
2770
2771         * WebCoreSupport.subproj/WebBridge.m:
2772         (-[WebBridge getAppletInView:]):
2773
2774 2004-08-17  Richard Williamson   <rjw@apple.com>
2775
2776         Fixed <rdar://problem/3759093> Need PDF context menu: "Open in Preview" or other external app
2777
2778         Added support for "Open with ..." in PDF view.
2779                 
2780         Reviewed by Chris and Trey.
2781
2782         * ChangeLog:
2783         * English.lproj/Localizable.strings:
2784         * English.lproj/StringsNotToBeLocalized.txt:
2785         * WebView.subproj/WebPDFView.h:
2786         * WebView.subproj/WebPDFView.m:
2787         (-[WebPDFView initWithFrame:]):
2788         (-[WebPDFView dealloc]):
2789         (applicationInfoForMIMEType):
2790         (-[WebPDFView path]):
2791         (-[WebPDFView menuForEvent:]):
2792         (-[WebPDFView setDataSource:]):
2793         (-[WebPDFView layout]):
2794         (-[WebPDFView viewDidMoveToHostWindow]):
2795         (-[WebPDFView openWithFinder:]):
2796
2797 2004-08-17  Richard Williamson   <rjw@apple.com>
2798
2799         JNI needs both the jmethodID and return type.  Changed API to 
2800         pass both.
2801                 
2802         Reviewed by Chris.
2803
2804         * Plugins.subproj/WebJavaPlugIn.h:
2805
2806 2004-08-17  Trey Matteson  <trey@apple.com>
2807
2808         3764147 - failure of subframe to load leaves links in parent doc broken
2809
2810         Reviewed by Maciej.
2811
2812         * WebView.subproj/WebDataSource.m:
2813         (-[WebDataSource _receivedMainResourceError:complete:]):
2814         Need to call [bridge end] in the case of an error, so WC can clean up.
2815
2816 === Safari-157 ===
2817
2818 2004-08-15  Richard Williamson   <rjw@apple.com>
2819
2820         More changes to np headers.
2821         
2822         Reviewed by Darin.
2823
2824         * Plugins.subproj/WebNetscapePluginPackage.m:
2825         (-[WebNetscapePluginPackage load]):
2826         * Plugins.subproj/npapi.h:
2827         * Plugins.subproj/npfunctions.h:
2828         * Plugins.subproj/npruntime.h:
2829
2830 2004-08-13  Darin Adler  <darin@apple.com>
2831
2832         Reviewed by Chris.
2833
2834         - fixed <rdar://problem/3760924> Carbon path passed in NPP_StreamAsFile must be in local character set, not UTF-8
2835
2836         * Plugins.subproj/WebBaseNetscapePluginStream.m:
2837         (CarbonPathFromPOSIXPath): Added. New function that uses the path pieces from an FSSpec.
2838         This has many advantages; the big one that fixes the bug is that it gives the mangled names
2839         that work even for files that have names that can't otherwise be encoded in Carbon-style
2840         path names. I didn't write this from scratch: I started with the method in Foundation
2841         and just changed it to use FSSpec.
2842         (-[WebBaseNetscapePluginStream destroyStream]): Remove code that used NSString and just call
2843         CarbonPathFromPOSIXPath instead.
2844
2845         * Plugins.subproj/npapi.h: The Revision tag wanted to touch this file.
2846         I think we are going to have some trouble with this; I'd like to take that out.
2847
2848         * English.lproj/StringsNotToBeLocalized.txt: Updated.
2849
2850 2004-08-12  Trey Matteson  <trey@apple.com>
2851
2852         3761329 - query result links all dead in ingrammicro.com (sometimes)
2853         3761328 - links in some docs dead when doc is loaded from WebArchive
2854
2855         Nasty problem.  It turns out that these result pages are a parent frame with two child
2856         frames.  One of the child frames is a 1 or 2 byte text document.  When the text child
2857         is the last doc to complete, because we use a Text rep instead of an HTMLRep, it turns
2858         out we never send [bridge end] from WebKit.  That mistake results in checkCompleted not
2859         being called enough in the part, and we never realize the load is done.  WebCore does
2860         not allow redirects to happen until the entire load is complete,  and  the links on this
2861         results page are actually little pieces of JS that set location to a generated URL.  Since
2862         redirects are not allowed these links all silently fail to do anything, and the doc never
2863         achieves a completed state where they will work.
2864
2865         Solution is to make sure we always call [bridge end] from the DataSource instead of only
2866         the HTML rep doing it.
2867
2868         Reviewed by Richard and Darin.
2869
2870         * WebView.subproj/WebDataSource.m:
2871         (-[WebDataSource _finishedLoading]):  Call [bridge end] here for all kinds of docs.
2872         * WebView.subproj/WebHTMLRepresentation.m:
2873         (-[WebHTMLRepresentation finishedLoadingWithDataSource:]):  Don't call is just for HTMLReps.
2874
2875 2004-08-12  Richard Williamson   <rjw@apple.com>
2876
2877         Bring npruntime.h and friends closer to compliance with
2878         latest spec.
2879
2880         Reviewed by Maciej.
2881
2882         * Plugins.subproj/npapi.h:
2883         * Plugins.subproj/npfunctions.h:
2884         * Plugins.subproj/npruntime.h:
2885         (_NPString::):
2886         (_NPString::_NPVariant::):
2887         * copy-webcore-files-to-webkit:
2888
2889 2004-08-12  Chris Blumenberg  <cblu@apple.com>
2890
2891         Fixed: <rdar://problem/3761097> should be able to option-drag selection so HTML can be copied within a page
2892
2893         Reviewed by rjw.
2894
2895         * WebView.subproj/WebHTMLView.m:
2896         (-[WebHTMLView _isMoveDrag]): new, take into account the option key
2897         (-[WebHTMLView draggingUpdatedWithDraggingInfo:actionMask:]): call _isMoveDrag
2898         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): call _isMoveDrag
2899
2900 2004-08-12  Richard Williamson   <rjw@apple.com>
2901
2902         Quick fix for 3760903.  The real fix is described in 3760920.  Needed
2903         by Java plugin guys so they can be unblocked for feature freeze.
2904                 
2905         Reviewed by Chris.
2906
2907         * WebView.subproj/WebFrame.m:
2908         (-[WebFrame _reloadForPluginChanges]):
2909         * WebView.subproj/WebHTMLView.m:
2910         (-[WebHTMLView addSubview:]):
2911
2912 2004-08-12  Chris Blumenberg  <cblu@apple.com>
2913
2914         Fixed: <rdar://problem/3760898> error pages in subframes attempt to load appledata URLs when reloaded
2915
2916         Reviewed by darin.
2917
2918         * WebView.subproj/WebFrame.m:
2919         (-[WebFrame _createItem:]): when setting the original URL of the history item, use the unreachable URL
2920
2921 === Safari-156 ===
2922
2923 2004-08-11  Adele Amchan  <adele@apple.com>
2924
2925         Reviewed by me, bug fix by Darin.
2926
2927         - fixed <rdar://problem/3736477> Pages don't load if hard drive is named with non-ASCII Symbol
2928
2929         * Plugins.subproj/WebBaseNetscapePluginStream.m:
2930         (-[WebBaseNetscapePluginStream startStreamWithURL:expectedContentLength:lastModifiedDate:MIMEType:]):
2931         Use UTF8String instead of cString to convert the MIME type to a C string. Safer, since it can't
2932         ever fail due to encoding problems even though this string should always be ASCII.
2933         (-[WebBaseNetscapePluginStream destroyStream]): Use stringWithUTF8String to convert the path name
2934         to an NSString, since stringWithCString is deprecated (doesn't really matter since the path is always
2935         all ASCII). Fix the bug by calling fileSystemRepresentation on the NSString to turn it into a C
2936         string form. Even though the POSIX path can't have any non-ASCII characters in it, the Carbon path
2937         can, so we need to use this instead of cString which can fail depending on characters and encoding.
2938
2939 2004-08-09  Maciej Stachowiak  <mjs@apple.com>
2940
2941         Reviewed by Trey.
2942
2943         WebKit part of:
2944
2945         - made basic marked text highlighting work to complete basic level of <rdar://problem/3704359> input method support not yet implemented for HTML editing
2946
2947         * WebView.subproj/WebHTMLView.m:
2948         (-[WebHTMLView markedRange]): Use new bridge calls instead of internal marked range storage.
2949         (-[WebHTMLView hasMarkedText]): Likewise.
2950         (-[WebHTMLView unmarkText]): Likewise.
2951         (-[WebHTMLView _selectMarkedText]): Likewise.
2952         (-[WebHTMLView _selectRangeInMarkedText:]): Likewise.
2953         (-[WebHTMLView _selectionIsInsideMarkedText]): Likewise.
2954         (-[WebHTMLView _updateSelectionForInputManager]): Likewise.
2955         (-[WebHTMLView setMarkedText:selectedRange:]): Use direct bridge call instead
2956         of private _selectMarkedDOMRange: method, which would now be trivial.
2957         * WebView.subproj/WebHTMLViewInternal.h: Remove unneeded
2958
2959 2004-08-10  Darin Adler  <darin@apple.com>
2960
2961         Reviewed by Ken.
2962
2963         - change name of WebMakeCollectable to WebNSRetainCFRelease so it fits into the
2964           "NS and CF retain counts are separate" mental model, rather than the "think about
2965           how garbage collection works" one
2966
2967         * Misc.subproj/WebKitNSStringExtras.m: (+[NSString _web_encodingForResource:]): Rename.
2968         * Misc.subproj/WebNSObjectExtras.h: (WebNSRetainCFRelease): Ditto.
2969         * Misc.subproj/WebNSURLExtras.m:
2970         (+[NSURL _web_URLWithData:relativeToURL:]): Ditto.
2971         (-[NSURL _web_URLWithLowercasedScheme]): Ditto.
2972
2973 2004-08-09  Trey Matteson  <trey@apple.com>
2974
2975         3756599 - REGRESSION: hit assertion in KWQPageState invalidate
2976
2977         By inspection I found a flaw in the recently added logic, although
2978         I have no steps to repro.  My theoretical explanation is that we would
2979         get two errors and go through _receivedMainResourceError: twice, which
2980         would cause the pageState to be invalidated twice, which is the only
2981         way I can see to hit the assert.
2982
2983         Reviewed by Darin
2984
2985         * WebView.subproj/WebFrame.m:
2986         (-[WebFrame _receivedMainResourceError:]):  Clear the pageState out
2987         of the history item after it's been invalidated by WebCore.
2988
2989 2004-08-09  Richard Williamson   <rjw@apple.com>
2990
2991         Inspired by Trey we have a much better approach for conditionally
2992         linking Quart.framework.  Instead of multiple targets we use 
2993         `` to invoke some inline script to extend COMMON_LDFLAGS as
2994         necessary.  Thanks Trey!
2995         
2996         Reviewed by Trey.
2997
2998         * WebKit.pbproj/project.pbxproj:
2999
3000 2004-08-09  Trey Matteson  <trey@apple.com>
3001
3002         Hookup UI for "Continuous Spelling" menu item.
3003
3004         Reviewed by Richard.
3005
3006         * WebView.subproj/WebView.m:
3007         (-[WebView validateUserInterfaceItem:]):  Enable and check the
3008         menu item to reflect our state.
3009         (-[WebView toggleContinuousSpellChecking:]):  Change type to
3010         IBAction, just cosmetic.
3011         * WebView.subproj/WebViewPrivate.h:
3012
3013 2004-08-08  Trey Matteson  <trey@apple.com>
3014
3015         3745023 - Safari crashes trying to access anchor while downloading
3016
3017         I bet this is behind a few other crashers as well.  In this bug the start of the
3018         download leaves a KWQPageState hanging around, and when that is freed it damages
3019         the part and view.  If you're still using that page, you're dead.
3020
3021         The fix is to properly invalidate the PageState when we receive an error before
3022         reaching WebFrameCommitted state.  Normally this happens when a page is reheated
3023         from the PageState, but in this case we never manage to leave the page to begin
3024         with, although we've already created the PageState.
3025
3026         Other errors besides the synthetic one download generates would have caused similar
3027         crashing.  Another example would be clicking on a second link before the load
3028         caused by clicking on the first link reached committed state.
3029
3030         Reviewed by Richard
3031
3032         * WebView.subproj/WebDataSource.m:
3033         (-[WebDataSource _receivedMainResourceError:complete:]):  Let the frame do the
3034         main work (since it has access to the pageCache state).  Also renamed to make
3035         it clear that this is about an error for the main resource.
3036         * WebView.subproj/WebDataSourcePrivate.h:
3037         * WebView.subproj/WebFrame.m:
3038         (-[WebFrame _receivedMainResourceError:]):  Let WC know about the failure, as
3039         the DataSource used to, but now pass the pageCache state along too.
3040         * WebView.subproj/WebFramePrivate.h:
3041         * WebView.subproj/WebMainResourceClient.m:
3042         (-[WebMainResourceClient receivedError:]):  Call renamed method.
3043         (-[WebMainResourceClient cancelWithError:]):  Ditto.
3044
3045 2004-08-08  Vicki Murley  <vicki@apple.com>
3046
3047         Reviewed by vicki (changes by rjw)
3048  
3049         - make "weak" linking with Quartz work with buildit
3050
3051         * WebKit.pbproj/project.pbxproj: 
3052
3053 2004-08-06  Ken Kocienda  <kocienda@apple.com>
3054
3055         Reviewed by Maciej
3056
3057         Finish off spellchecking support to HTML editing. Includes work to
3058         enable continuous spellchecking.
3059
3060         * WebCoreSupport.subproj/WebBridge.m:
3061         (-[WebBridge isContinuousSpellCheckingEnabled]): Simple bridge method.
3062         * WebCoreSupport.subproj/WebTextRenderer.m:
3063         (-[WebTextRenderer drawLineForMisspelling:withWidth:]): New method to add
3064         AppKit-style misspelling underline.
3065
3066 === Safari-155 ===
3067
3068 2004-08-05  Darin Adler  <darin@apple.com>
3069
3070         * WebView.subproj/WebHTMLView.m: Added a list of methods that NSTextView implements that we don't.
3071         All inside #if 0.
3072
3073 2004-08-05  Chris Blumenberg  <cblu@apple.com>
3074
3075         Fixed unnecessary import of NSURLFileTypeMappings.
3076
3077         * WebView.subproj/WebMainResourceClient.m:
3078
3079 2004-08-05  Richard Williamson   <rjw@apple.com>
3080
3081         Make builds conditionally include -framework Quartz.
3082                 
3083         Reviewed by Chris.
3084
3085         * WebKit.pbproj/project.pbxproj:
3086
3087 2004-08-04  Darin Adler  <darin@apple.com>
3088
3089         - fixed broken Deployment build
3090
3091         * Carbon.subproj/HIWebView.m: (Click):
3092         Remove code that checks err variable before setting it.
3093
3094 2004-08-03  Richard Williamson   <rjw@apple.com>
3095
3096         First cut at dirt simple PDF support.  This feature
3097         depends on Quartz.framework (parent of PDFKit), which only exists
3098         on Tiger.  So, we "weak" link against Quartz.
3099                 
3100         We do very basic PDF rendering.  Coming up are support for
3101         "Find..." and linearized PDF (incremental).  Linearized PDF 
3102         support will require API changes in PDFKit.
3103                 
3104         No UI is added to Safari, yet.
3105                 
3106         Reviewed by John.
3107
3108         * WebKit.pbproj/project.pbxproj:
3109         * WebView.subproj/WebDataSource.m:
3110         (+[WebDataSource _repTypesAllowImageTypeOmission:]):
3111         * WebView.subproj/WebFrameView.m:
3112         (+[WebFrameView _viewTypesAllowImageTypeOmission:]):
3113         * WebView.subproj/WebPDFRepresentation.h: Added.
3114         * WebView.subproj/WebPDFRepresentation.m: Added.
3115         (-[WebPDFRepresentation finishedLoadingWithDataSource:]):
3116         (-[WebPDFRepresentation canProvideDocumentSource]):
3117         (-[WebPDFRepresentation documentSource]):
3118         (-[WebPDFRepresentation title]):
3119         * WebView.subproj/WebPDFView.h: Added.
3120         * WebView.subproj/WebPDFView.m: Added.
3121         (-[WebPDFView initWithFrame:]):
3122         (-[WebPDFView setDataSource:]):
3123         (-[WebPDFView dataSourceUpdated:]):
3124         (-[WebPDFView setNeedsLayout:]):
3125         (-[WebPDFView layout]):
3126         (-[WebPDFView viewWillMoveToHostWindow:]):
3127         (-[WebPDFView viewDidMoveToHostWindow]):
3128
3129         Copied fix from Jaguar carbon/cocoa work.
3130                 
3131         Reviewed by Darin (Jaguar version)
3132
3133         * Carbon.subproj/CarbonWindowAdapter.h:
3134         * Carbon.subproj/CarbonWindowAdapter.m:
3135         (-[CarbonWindowAdapter relinquishFocus]):
3136         * Carbon.subproj/HIWebView.m:
3137         (Click):
3138         (OwningWindowChanged):
3139         (WindowHandler):
3140
3141 2004-08-03  David Hyatt  <hyatt@apple.com>
3142
3143         Add the text/xsl MIME type as one that can be displayed.
3144         
3145         Reviewed by john
3146
3147         * WebView.subproj/WebDataSource.m:
3148         (+[WebDataSource _repTypesAllowImageTypeOmission:]):
3149         * WebView.subproj/WebFrameView.m:
3150         (+[WebFrameView _viewTypesAllowImageTypeOmission:]):
3151
3152 2004-08-03  Maciej Stachowiak  <mjs@apple.com>
3153
3154         Reviewed by Ken.
3155
3156         - implemented enough of the NSTextInput protocol and added the
3157         proper calls to NSInputManager to allow input methods to
3158         work. However, the text is not marked yet.
3159
3160         * WebView.subproj/WebHTMLViewInternal.h: Added new fields to track
3161         marked range.
3162         * WebView.subproj/WebHTMLView.m:
3163         (-[WebHTMLView markedRange]): Implemented
3164         (-[WebHTMLView conversationIdentifier]): Implemented
3165         (-[WebHTMLView hasMarkedText]): Implemented
3166         (-[WebHTMLView unmarkText]): Implemented
3167         (-[WebHTMLView setMarkedText:selectedRange:]): Implemented - does
3168         not yet handle attributes in attributed strings
3169         (-[WebHTMLView insertText:]): Modified to handle replacing or abandoning
3170         the marked text when set.
3171         (-[WebHTMLView _selectMarkedText]): new helper method, self-explanatory
3172         (-[WebHTMLView _setMarkedDOMRange:]): hitto.
3173         (-[WebHTMLView _selectRangeInMarkedText:]): ditto
3174         (-[WebHTMLView _discardMarkedText]): ditto
3175         (-[WebHTMLView _selectionIsInsideMarkedText]): ditto
3176         (-[WebHTMLView _updateSelectionForInputManager]): ditto
3177
3178         - did a bit of refactoring while I was in here.
3179
3180         * WebView.subproj/WebHTMLView.m:
3181         (-[WebHTMLView _shouldReplaceSelectionWithText:givenAction:]): added this new
3182         helper method to simplify the common case of calling the shouldInsertText: delegate
3183         (-[WebHTMLView pasteAsPlainText:]): use it
3184         (-[WebHTMLView insertTab:]): ditto
3185         (-[WebHTMLView insertNewline:]): ditto
3186         (-[WebHTMLView _changeWordCaseWithSelector:]): ditto
3187         (-[WebHTMLView _changeSpellingToWord:]): ditto
3188         (-[WebHTMLView _selectionChanged]): ditto
3189
3190 2004-08-03  Chris Blumenberg  <cblu@apple.com>
3191
3192         Fixed: <rdar://problem/3572737> Images not resizing at www.bmx-test.com (spoof nofix)
3193
3194         Reviewed by darin.
3195
3196         * WebCoreSupport.subproj/WebImageRendererFactory.m:
3197         (-[WebImageRendererFactory supportedMIMETypes]): hard code image/pjpeg to the list of image mime types that we can handle
3198
3199 2004-08-03  Darin Adler  <darin@apple.com>
3200
3201         Reviewed by Ken.
3202
3203         - fixed <rdar://problem/3740937> ER: A way to turn a DOMRange into text (equivalent of -innerText)
3204
3205         * DOM.subproj/DOMPrivate.h: Added.
3206         * WebKit.pbproj/project.pbxproj: Added DOMPrivate.h.
3207         * copy-webcore-files-to-webkit: Added DOMPrivate.h.
3208
3209 2004-08-02  John Sullivan  <sullivan@apple.com>
3210
3211         Reviewed by Darin.
3212
3213         WebKit part of fix for <rdar://problem/3631868> NSToolbar adoption: 
3214         Tab key should cycle around toolbar and page content
3215
3216         * WebView.subproj/WebHTMLView.m:
3217         removed overrides of nextKeyView and previousKeyView
3218         (-[WebHTMLView nextValidKeyView]):
3219         call super only if we can't move the focus within the frame hierarchy
3220         (-[WebHTMLView previousValidKeyView]):
3221         ditto
3222         * WebView.subproj/WebHTMLViewInternal.h:
3223         removed nextKeyViewAccessShouldMoveFocus ivar
3224
3225 2004-08-02  Ken Kocienda  <kocienda@apple.com>
3226
3227         Reviewed by John
3228
3229         Update name of firstResponderIsSelfOrDescendantView, adding _web_ prefix to this SPI call.
3230         
3231         Do some work to make caret blinking in newly-created editable WebView's that are 
3232         similar in structure to Blot more "automatic".
3233
3234         * Misc.subproj/WebNSViewExtras.h: Change name of firstResponderIsSelfOrDescendantView. Add _web_ prefix.
3235         * Misc.subproj/WebNSViewExtras.m:
3236         (-[NSView _web_firstResponderIsSelfOrDescendantView]): Name change.
3237         (-[NSView _web_firstResponderCausesFocusDisplay]): Adds an additional check for whether the view's
3238         is first responder. This helps to make the focus setting in viewDidMoveToWindow work right.
3239         * WebView.subproj/WebHTMLView.m:
3240         (-[WebHTMLView updateFocusDisplay]): Use _web_firstResponderCausesFocusDisplay now instead of 
3241         firstResponderIsSelfOrDescendantView.
3242         (-[WebHTMLView viewDidMoveToWindow]): Schedule call to updateFocusDisplay for the next crank of the
3243         run loop. The reason is that placing the caret in the just-installed view requires the HTML/XML 
3244         document to be available on the WebCore side, but it is not at the time this code is running. 
3245         However, it will be there on the next crank of the run loop. Doing this helps to make a 
3246         blinking caret appear in a new, empty window "automatic".
3247         (-[WebHTMLView performKeyEquivalent:]):  _web_firstResponderIsSelfOrDescendantView name change.
3248         * WebView.subproj/WebView.m:
3249         (-[WebView _performResponderOperation:with:]): _web_firstResponderIsSelfOrDescendantView name change.
3250
3251 2004-07-29  Maciej Stachowiak  <mjs@apple.com>
3252
3253         Reviewed by John.
3254
3255         Added stubbed-out versions of all the NSTextInput protocol
3256         methods, and put comments in places where we will need to call the
3257         input manager to make input methods work 100%.
3258         
3259         * WebCoreSupport.subproj/WebBridge.m:
3260         (-[WebBridge respondToChangedSelection]):
3261         * WebView.subproj/WebHTMLView.m:
3262         (-[WebHTMLView mouseDown:]):
3263         (-[WebHTMLView mouseDragged:]):
3264         (-[WebHTMLView mouseUp:]):
3265         (-[WebHTMLView _interceptEditingKeyEvent:]):
3266         (-[WebHTMLView keyDown:]):
3267         (-[WebHTMLView _selectionChanged]):
3268         (-[WebHTMLView _delegateDragSourceActionMask]):
3269         (-[WebHTMLView validAttributesForMarkedText]):
3270         (-[WebHTMLView characterIndexForPoint:]):
3271         (-[WebHTMLView firstRectForCharacterRange:]):
3272         (-[WebHTMLView selectedRange]):
3273         (-[WebHTMLView markedRange]):
3274         (-[WebHTMLView attributedSubstringFromRange:]):
3275         (-[WebHTMLView conversationIdentifier]):
3276         (-[WebHTMLView hasMarkedText]):
3277         (-[WebHTMLView unmarkText]):
3278         (-[WebHTMLView setMarkedText:selectedRange:]):
3279         (-[WebHTMLView doCommandBySelector:]):
3280         (-[WebHTMLView insertText:]):
3281         * WebView.subproj/WebHTMLViewInternal.h:
3282         * WebView.subproj/WebView.m:
3283
3284 === Safari-154 ===
3285
3286 2004-07-29  Darin Adler  <darin@apple.com>
3287
3288         Reviewed by Ken.
3289
3290         - fill in some unimplemented methods
3291
3292         * WebView.subproj/WebHTMLView.m:
3293         (-[WebHTMLView moveToBeginningOfDocument:]): Implement, using new document granularity.
3294         (-[WebHTMLView moveToBeginningOfDocumentAndModifySelection:]): Ditto.
3295         (-[WebHTMLView moveToBeginningOfParagraph:]): Implement, even though the underlying WebCore
3296         code is not yet implemented.
3297         (-[WebHTMLView moveToBeginningOfParagraphAndModifySelection:]): Ditto.
3298         (-[WebHTMLView moveToEndOfDocument:]): More of the same.
3299         (-[WebHTMLView moveToEndOfDocumentAndModifySelection:]): Ditto.
3300         (-[WebHTMLView moveToEndOfParagraph:]): More of the same.
3301         (-[WebHTMLView moveToEndOfParagraphAndModifySelection:]): Ditto.
3302         (-[WebHTMLView pageDown:]): Added comment describing desired behavior.
3303         (-[WebHTMLView pageUp:]): Ditto.
3304         (-[WebHTMLView copyFont:]): Ditto.
3305         (-[WebHTMLView pasteFont:]): Ditto.
3306         (-[WebHTMLView changeAttributes:]): Ditto.
3307         (-[WebHTMLView indent:]): Ditto.
3308
3309 2004-07-28  Chris Blumenberg  <cblu@apple.com>
3310
3311         Fixed: <rdar://problem/3737864> Can not download image from web by using drag and drop
3312
3313         Reviewed by john.
3314
3315         * Misc.subproj/WebNSPasteboardExtras.m:
3316         (-[NSPasteboard _web_declareAndWriteDragImage:URL:title:archive:source:]): On Tiger, to use NSFilesPromisePboardType with other pasteboard types, set the file types of the promise on the pasteboard as a property list instead of mucking with the NSFilePromiseDragSource class
3317
3318 2004-07-28  Maciej Stachowiak  <mjs@apple.com>
3319
3320         Reviewed by John.
3321
3322         - avoid triggering an assertion when using dead keys (like option-e)
3323
3324         * WebView.subproj/WebHTMLView.m:
3325         (-[WebHTMLView insertText:]): Don't pass zero-length strings to
3326         WebCore. Dead-key input seems to insert empty strings as a side
3327         effect.
3328
3329 2004-07-28  Trey Matteson  <trey@apple.com>
3330
3331         Spellchecking, Part I.  Basic spellcheck is working.  Spelling panel is hooked up.
3332
3333         At this point, no special marking of misspellings, no grammar check, no context
3334         menu integration, no "check continually" mode.
3335
3336         Reviewed by Ken.
3337
3338         * WebCoreSupport.subproj/WebBridge.m:
3339         (-[WebBridge spellCheckerDocumentTag]):  Typical bridge glue.
3340         * WebView.subproj/WebHTMLView.m:
3341         (-[WebHTMLView validateUserInterfaceItem:]):  Validate various spelling actions.
3342         (-[WebHTMLView checkSpelling:]):  Call WC for real work, update panel.
3343         (-[WebHTMLView showGuessPanel:]):  Show panel, call WC for real work.
3344         (-[WebHTMLView _changeSpellingToWord:]):  Apply correction to our doc.
3345         (-[WebHTMLView changeSpelling:]):  Simple pass through to above method.
3346         (-[WebHTMLView ignoreSpelling:]):  Tell checker to ignore the word.
3347         * WebView.subproj/WebView.m:
3348         (-[WebView _close]):  Call AK's closeSpellDocumentWithTag: for proper cleanup.
3349
3350 2004-07-27  John Sullivan  <sullivan@apple.com>
3351
3352         Reviewed by Trey.
3353         
3354         WebKit part of fix for <rdar://problem/3622268> Reload failed pages when a 
3355         network problem is corrected, inc. using Network Diagnostics
3356
3357         * WebView.subproj/WebFrame.m:
3358         (-[WebFrame reload]):
3359         This method did not handle unreachableURLs at all. The reason Safari's Reload
3360         did work with unreachableURLs was that Safari does "reloadObeyingLocationField"
3361         which never actually calls -[WebFrame reload]. Fixed by creating a fresh
3362         request for the previously-unreachable URL.
3363
3364 2004-07-26  Richard Williamson   <rjw@apple.com>
3365
3366         Fixed 3739737.  When setting the focus to a NSView, set the focus
3367         carbon focus to kControlIndicatorPart.  kControlIndicatorPart is a
3368         placeholder value for use to indicate that Cocoa has the focus.  
3369         
3370         Reviewed by Ken.
3371
3372         * Carbon.subproj/CarbonWindowAdapter.m:
3373         (-[CarbonWindowAdapter makeFirstResponder:]):
3374         * Carbon.subproj/HIWebView.m:
3375         (Click):
3376         (SetFocusPart):
3377
3378 2004-07-26  Chris Blumenberg  <cblu@apple.com>
3379
3380         Fixed: <rdar://problem/3689734> dragging to an editable WebView does not scroll document
3381
3382         Reviewed by rjw.
3383
3384         * WebView.subproj/WebView.m:
3385         (-[WebView documentViewAtWindowPoint:]): new, factored from draggingDocumentViewAtWindowPoint:
3386         (-[WebView _draggingDocumentViewAtWindowPoint:]): call documentViewAtWindowPoint:
3387         (-[WebView _autoscrollForDraggingInfo:timeDelta:]): forward call to the document view
3388         (-[WebView _shouldAutoscrollForDraggingInfo:]): ditto
3389
3390 === Safari-153 ===
3391
3392 2004-07-23  Ken Kocienda  <kocienda@apple.com>
3393
3394         Reviewed by Trey
3395
3396         Fix for this bug:
3397         
3398         <rdar://problem/3738920> Caret blinks in inactive window
3399
3400         As part of the fix, I cleaned up the way we handle special drawing that needs
3401         to be done in the HTML view that is first responder in the key window (e.g the
3402         drawing of text selection highlight and caret blinking).
3403
3404         * WebView.subproj/WebHTMLView.m:
3405         (-[WebHTMLView updateFocusDisplay]): New method that centralizes the changes
3406         we need to make when the first responder-ness of the view or key-ness of the
3407         window changes.
3408         (-[WebHTMLView viewDidMoveToWindow]): Call new updateFocusDisplay helper.
3409         (-[WebHTMLView windowDidBecomeKey:]): Ditto.
3410         (-[WebHTMLView windowDidResignKey:]): Ditto.
3411         (-[WebHTMLView becomeFirstResponder]): Ditto.
3412         (-[WebHTMLView resignFirstResponder]): Ditto.
3413
3414 2004-07-22  Darin Adler  <darin@apple.com>
3415
3416         * Plugins.subproj/npruntime.h: Update with new version from newer JavaScriptCore.
3417
3418 2004-07-21  Ken Kocienda  <kocienda@apple.com>
3419
3420         Reviewed by Trey
3421
3422         * WebCoreSupport.subproj/WebBridge.m: Remove interceptEditingKeyEvent
3423         bridge call over. This method of handling editing key events is now
3424         obsolete.
3425         * WebView.subproj/WebHTMLView.m:
3426         (-[WebHTMLView performKeyEquivalent:]): Send key event over to the 
3427         DOM if this view is or contains the first responder. This now
3428         happens unconditionally.
3429         (-[WebHTMLView keyDown:]): Send key event to the DOM, then see if
3430         the web view wants to interpret it an an editing key event. This
3431         is the new place to intercept key events for editing.
3432
3433 2004-07-21  Ken Kocienda  <kocienda@apple.com>
3434
3435         Reviewed by John
3436
3437         Add implementations for these methods. Formerly, they logged an error.
3438
3439         * WebView.subproj/WebHTMLView.m:
3440         (-[WebHTMLView moveToBeginningOfLine:]):
3441         (-[WebHTMLView moveToBeginningOfLineAndModifySelection:]):
3442         (-[WebHTMLView moveToEndOfLine:]):
3443         (-[WebHTMLView moveToEndOfLineAndModifySelection:]):
3444
3445 2004-07-21  Ken Kocienda  <kocienda@apple.com>
3446
3447         Reviewed by John
3448
3449         Added some more handlers for standard Cocoa key bindings. 
3450         These are "secrets" of NSText, meaning they are not public API, but we choose to mimic.
3451
3452         * WebView.subproj/WebHTMLView.m:
3453         (-[WebHTMLView moveToBeginningOfDocumentAndModifySelection:]):
3454         (-[WebHTMLView moveToBeginningOfLineAndModifySelection:]):
3455         (-[WebHTMLView moveToBeginningOfParagraphAndModifySelection:]):
3456         (-[WebHTMLView moveToEndOfDocumentAndModifySelection:]):
3457         (-[WebHTMLView moveToEndOfLineAndModifySelection:]):
3458         (-[WebHTMLView moveToEndOfParagraph:]):
3459         (-[WebHTMLView moveToEndOfParagraphAndModifySelection:]):
3460         * WebView.subproj/WebView.h: Add these declarations to the comment in the header listing
3461         the responder-like methods we support.
3462         * WebView.subproj/WebView.m:
3463         (-[WebView moveToBeginningOfParagraphAndModifySelection:]):
3464         (-[WebView moveToEndOfParagraphAndModifySelection:]):
3465         (-[WebView moveToBeginningOfLineAndModifySelection:]):
3466         (-[WebView moveToEndOfLineAndModifySelection:]):
3467         (-[WebView moveToBeginningOfDocumentAndModifySelection:]):
3468         (-[WebView moveToEndOfDocumentAndModifySelection:]):
3469
3470 2004-07-20  Ken Kocienda  <kocienda@apple.com>
3471
3472         Reviewed by Richard
3473
3474         * Misc.subproj/WebNSEventExtras.h: Added helper that returns whether
3475         a key event has a binding in the key binding manager.
3476         * Misc.subproj/WebNSEventExtras.m: 
3477         (-[NSEvent _web_keyBindingManagerHasBinding]): New helper mentioned above.
3478         * Plugins.subproj/npruntime.h:
3479         * WebView.subproj/WebHTMLView.m:
3480         (-[WebHTMLView performKeyEquivalent:]): Add a check of whether the key event 
3481         has a binding in the key binding manager. This works around the fact that
3482         NSResponder's interpretKeyEvents does not return a value telling whether
3483         or not the key was handled. This now makes it possible for us to trap
3484         modified key events we know we can handle (like those command-key + arrow events 
3485         used for text navigation), while letting all others pass.
3486
3487 2004-07-20  Chris Blumenberg  <cblu@apple.com>
3488
3489         Fixed:
3490         <rdar://problem/3580589> REGRESSION (1.1-1.2): can't open a new window for an image that has not loaded
3491         <rdar://problem/3612691> Missing image icons (blue ?) lack context menu
3492
3493         Reviewed by john.
3494
3495         * WebView.subproj/WebDefaultContextMenuDelegate.m:
3496         (-[WebDefaultUIDelegate webView:contextMenuItemsForElement:defaultMenuItems:]): if there is no image, but there is an image URL, provide image context menu items besides "Copy Image" 
3497         * WebView.subproj/WebHTMLView.m:
3498         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]): don't drag an image with an actual image
3499         (-[WebHTMLView _mayStartDragAtEventLocation:]): ditto
3500
3501 2004-07-20  Maciej Stachowiak  <mjs@apple.com>
3502
3503         Reviewed by Chris.
3504
3505         <rdar://problem/3721690> REGRESSION (125.7-148u) clicking on links at macosx.apple.com/Builds does not load new page in frame
3506         
3507         * Plugins.subproj/npruntime.h:
3508         * WebCoreSupport.subproj/WebBridge.m:
3509         (-[WebBridge canTargetLoadInFrame:]): Don't apply the restrictions to frames
3510         that are in the same window (Mozilla does this too).
3511
3512 2004-07-20  Richard Williamson   <rjw@apple.com>
3513
3514                 Fix for 3728558.  Fixed the key event handling in the carbon/cocoa
3515                 integration code.  This does not fix the arrow keys not working on
3516                 initial focus problem also mentioned in the bug.
3517                 
3518                 Bumped the version of the NP function structures.
3519                 
3520         Reviewed by John.
3521
3522         * Carbon.subproj/CarbonUtils.m:
3523         (WebInitForCarbon):
3524         (PoolCleaner):
3525         * Carbon.subproj/HIWebView.m:
3526         (OwningWindowChanged):
3527         (WindowHandler):
3528         * Plugins.subproj/npapi.h:
3529
3530 2004-07-20  Trey Matteson  <trey@apple.com>
3531
3532         3733698 REGRESSION: sometimes dragging photos on homepage.mac.com leads to an assertion
3533
3534         Relax an assertion, as we ran into a valid case where it's not true.
3535
3536         Reviewed by Chris.
3537
3538         * WebView.subproj/WebHTMLView.m:
3539         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
3540         Removing this assert allows for the oddball case of a drag gesture that
3541         starts on one element, but then the element is no longer there when
3542         the drag is about to start.
3543
3544 2004-07-20  Ken Kocienda  <kocienda@apple.com>
3545
3546         Reviewed by Hyatt
3547
3548         Fix for this bug:
3549         
3550         <rdar://problem/3707505> HTMLCompose: blinking cursor in both an address text field and the message body
3551
3552         * WebView.subproj/WebHTMLView.m:
3553         (-[WebHTMLView setCaretVisible:]): New helper. Calls over bridge to do the work.
3554         (-[WebHTMLView windowDidBecomeKey:]): This function cannot just assume that self is first responder
3555         (and do things like adjusting text background color and restoring focus rings). 
3556         First-responder-ness needs to be checked first. Now it is.
3557         (-[WebHTMLView windowDidResignKey:]): Ditto.
3558         (-[WebHTMLView becomeFirstResponder]): Call new helper to make caret visible.
3559         (-[WebHTMLView resignFirstResponder]): Call new helper to make caret invisible.
3560
3561 2004-07-12  Richard Williamson   <rjw@apple.com>
3562
3563         Fixed 3721917.  The RealPlayer plugin doesn't support the new 
3564         NPPVpluginScriptableNPObject variable passed to NPP_GetValue and
3565         incorrectly returns NPERR_NO_ERROR.  We interpret this to the
3566         mean the variable has been set.  The variable has not been set 
3567         and will consequently be uninitialized.
3568
3569         Reviewed by NOBODY (OOPS!).
3570
3571         * Plugins.subproj/WebBaseNetscapePluginView.m:
3572         (-[WebBaseNetscapePluginView pluginScriptableObject]):
3573
3574 2004-07-19  Ken Kocienda  <kocienda@apple.com>
3575
3576         Reviewed by Richard
3577
3578         Fix for this bug:
3579         
3580         <rdar://problem/3707504> HTMLCompose: key events are stolen by Web(HTML)View
3581
3582         * Misc.subproj/WebNSViewExtras.h:
3583         * Misc.subproj/WebNSViewExtras.m:
3584         (-[NSView firstResponderIsSelfOrDescendantView]): New helper to 
3585         (-[WebHTMLView performKeyEquivalent:]): Do not pass key events through to the editing
3586         key handler unless the WebHTMLView is first responder or contains the first responder.
3587         This prevents the "stealing" of key events mentioned in the bug.
3588         * WebView.subproj/WebView.m:
3589         (-[WebView _performResponderOperation:with:]): Uses the new firstResponderIsSelfOrDescendantView
3590         helper. The code I replaced used the same logic as the new helper.
3591
3592 === Safari-152 ===
3593
3594 2004-07-12  Maciej Stachowiak  <mjs@apple.com>
3595
3596         Reviewed by Kevin.
3597
3598         <rdar://problem/3673988>: (REGRESSION(141-144): connection assertion failure at http://traffic.511.org/sfgate)
3599         
3600         * WebView.subproj/WebBaseResourceHandleDelegate.m:
3601         (-[WebBaseResourceHandleDelegate connectionDidFinishLoading:]):
3602         Don't check connection consistency if this load got cancelled
3603         while finishing, because in this case we have cleared the
3604         connection field already. This can happen when a redirect fires
3605         from an onload handler.
3606
3607 2004-07-12  Maciej Stachowiak  <mjs@apple.com>
3608
3609         Reviewed by John.
3610
3611         <rdar://problem/3632570>: (REGRESSION: SocialText.net wiki keeps asking for auth, even though pages load)
3612
3613         * Misc.subproj/WebIconLoader.m:
3614         (-[WebIconLoader didReceiveAuthenticationChallenge:]): Ignore the challenge - we don't
3615         want an auth panel for favicons.
3616         (-[WebIconLoader didCancelAuthenticationChallenge:]): Ignore cancel, since we are ignoring
3617         the challenge.
3618
3619 2004-07-09  Chris Blumenberg  <cblu@apple.com>
3620         
3621         Allowed my change for 3715785 to compile on Jaguar.
3622
3623         Reviewed by kocienda.
3624
3625         * WebCoreSupport.subproj/WebBridge.m:
3626         (-[WebBridge canTargetLoadInFrame:]): don't use the DOM API since it doesn't exist on Jaguar, instead call the new domain method on the bridge
3627
3628 2004-07-09  Ken Kocienda  <kocienda@apple.com>
3629
3630         Reviewed by John
3631
3632         * Plugins.subproj/npruntime.h: Updated license to lawyer-approved 
3633         joint Apple-Mozilla BSD-style license.
3634
3635 2004-07-08  Chris Blumenberg  <cblu@apple.com>
3636
3637         Fixed: <rdar://problem/3720728> REGRESSION (125.8-146): Crash moving mouse over plugin at manray-photo.com
3638         
3639         Reviewed by john.
3640
3641         * Plugins.subproj/WebBaseNetscapePluginView.m:
3642         (-[WebBaseNetscapePluginView pluginScriptableObject]): don't call NPP_GetValue unless the plug-in implements it
3643
3644 2004-07-08  Chris Blumenberg  <cblu@apple.com>
3645
3646         Fixed: <rdar://problem/3706296> VIP: ifilm.com crashing reproducibly with Safari
3647
3648         Reviewed by kocienda.
3649
3650         * Plugins.subproj/WebNetscapePluginPackage.m:
3651         (-[WebNetscapePluginPackage launchRealPlayer]): don't release a NULL appURL
3652
3653 2004-07-08  Chris Blumenberg  <cblu@apple.com>
3654
3655         Fixed: <rdar://problem/3650140> reproducible assertion failure going to plugin page with JavaScript disabled
3656
3657         Reviewed by john.
3658
3659         * Plugins.subproj/WebBaseNetscapePluginView.m:
3660         (-[WebBaseNetscapePluginView evaluateJavaScriptPluginRequest:]): don't call NPP_NewStream and other stream methods if there is no JS result to deliver. This is what Mozilla does.
3661         (-[WebBaseNetscapePluginView loadRequest:inTarget:withNotifyData:]): Return NPERR_GENERIC_ERROR if JS is disabled. This is what Mozilla does.
3662
3663 === Safari-151 ===
3664
3665 2004-07-07  Trey Matteson  <trey@apple.com>
3666
3667         3719051 - Safari doesn't update form inputs when a page was refreshed by javascript window.location
3668         ... and at least 5 other cases in Radar
3669
3670         Very similar problem to the Harvard PIN bug.  We need to be sure
3671         to not carry any state over when we are processing a client
3672         redirect, which reuses the same WebHistoryItem.
3673
3674         Reviewed by John.
3675
3676         * WebView.subproj/WebFrame.m:
3677         (-[WebFrame _transitionToCommitted:]):  Comment
3678         (-[WebFrame _opened]):  Clear form and scroll state on client redirect.
3679
3680 2004-07-06  Chris Blumenberg  <cblu@apple.com>
3681
3682         Fixed: <rdar://problem/3715785> multiple frame injection vulnerability reported by Secunia, affects almost all browsers
3683         
3684         Reviewed by john, trey, kocienda.
3685
3686         * WebCoreSupport.subproj/WebBridge.m:
3687         (-[WebBridge canTargetLoadInFrame:]): new method, return YES if the requesting frame is local, the target frame is an entire window or if the domain of the parent of the targeted frame equals this domain
3688         (-[WebBridge loadURL:referrer:reload:userGesture:target:triggeringEvent:form:formValues:]): call canTargetLoadInFrame: to make sure we can load the request
3689         (-[WebBridge postWithURL:referrer:target:data:contentType:triggeringEvent:form:formValues:]): ditto
3690
3691 2004-07-06  John Sullivan  <sullivan@apple.com>
3692
3693         Reviewed by Trey.
3694
3695         - fixed <rdar://problem/3717147> folder icon used for error page in back/forward 
3696         menu when iTunes is not installed
3697
3698         * WebView.subproj/WebFrame.m:
3699         (-[WebFrame _transitionToCommitted:]):
3700         When displaying the error page for an unreachable URL, leave the requested URL
3701         in the WebHistoryItem; don't clobber it with a bogus one that represents the
3702         error page. This not only avoids the wrong-icon problem, but should also help
3703         in cases where an unreachable URL becomes reachable later on.
3704
3705 2004-07-06  Trey Matteson  <trey@apple.com>
3706
3707         3716053 - www.theage.com.au has extra back/forward items due to ads
3708
3709         The real change was in WebKit.  Here was are just renaming a method and folding
3710         all the WebFrameLoadTypeOnLoadEvent uses to be WebFrameLoadTypeInternal, since there
3711         was never any difference anyway.
3712
3713         Reviewed by Richard
3714
3715         * WebCoreSupport.subproj/WebBridge.m:
3716         (-[WebBridge loadURL:referrer:reload:userGesture:target:triggeringEvent:form:formValues:]):  rename part of the method.
3717         * WebView.subproj/WebFrame.m:  Nuke WebFrameLoadTypeOnLoadEvent.
3718         (-[WebFrame _transitionToCommitted:]):
3719         (-[WebFrame _checkLoadCompleteForThisFrame]):
3720         (-[WebFrame _loadItem:withLoadType:]):
3721         (-[WebFrame _itemForRestoringDocState]):
3722         * WebView.subproj/WebFramePrivate.h:
3723
3724 2004-07-06  Ken Kocienda  <kocienda@apple.com>
3725
3726         Reviewed by Trey
3727         
3728         Only register the editing delegate for those notifications for which it implements the callbacks.
3729
3730         * WebView.subproj/WebView.m:
3731         (-[WebView registerForEditingDelegateNotification:selector:]):
3732         (-[WebView setEditingDelegate:]):
3733
3734 2004-07-06  Trey Matteson  <trey@apple.com>
3735
3736         3294652 - Failed drag of links doesn't slide back
3737
3738         The only reason for this is that because of some hacks, we lie to
3739         AK about the drag image offset, which means we slide back to
3740         slightly the wrong place.  But it's very minor, so we should
3741         just fix it.
3742
3743         Reviewed by Ken.
3744
3745         * WebView.subproj/WebHTMLView.m:
3746         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
3747         Ask for slideback.
3748
3749 2004-07-02  Darin Adler  <darin@apple.com>
3750
3751         Reviewed by Dave.
3752
3753         - fixed problem where tabsToLinks and privateBrowsingEnabled did not work with non-
3754           standard WebPreferences objects
3755
3756         * WebView.subproj/WebPreferences.m:
3757         (-[WebPreferences _valueForKey:]): New helper.
3758         (-[WebPreferences _stringValueForKey:]): Use _valueForKey.
3759         (-[WebPreferences _integerValueForKey:]): Use _valueForKey.
3760         (-[WebPreferences _boolValueForKey:]): Use _valueForKey.
3761         (-[WebPreferences tabsToLinks]): Use _boolValueForKey; this is the bug fix.
3762         (-[WebPreferences privateBrowsingEnabled]): Ditto.
3763         (+[WebPreferences _setIBCreatorID:]): Use copy instead of retain for keeping an NSString.
3764
3765 2004-07-01  Trey Matteson  <trey@apple.com>
3766
3767         3556159 - Crashes in -[WebFrame(WebPrivate) _transitionToCommitted:] at www.mastercardbusiness.com
3768
3769         We know from the line number of the crash that it is due to [self parentFrame]==nil.
3770
3771         Looking at the HTML and that of the related bugs, they do special stuff with onload
3772         handlers.  It is no longer repro, presumably because the includes JS files changed,
3773         as the bugs only included the top level HTML.  I suspect that the problem is that the
3774         WebFrameLoadTypeOnLoadEvent case was added, and in some weird sequence specific to
3775         MasterCard, they hit a case where we would be in WebFrameLoadTypeOnLoadEvent mode but
3776         not have a parent frame.
3777
3778         So we guard in the code against hitting a nil parentFrame, and log an error just in
3779         case this ever crops up again and we can learn more about it.
3780
3781         Reviewed by Richard.
3782
3783         * WebView.subproj/WebFrame.m:
3784         (-[WebFrame _transitionToCommitted:]):  Guard against nil parentFrame.
3785
3786 2004-07-01  John Sullivan  <sullivan@apple.com>
3787
3788         Reviewed by Trey.
3789         
3790         - fixed these bugs:
3791         <rdar://problem/3709110> REGRESSION (Tiger): Pressing Tab key to move focus 
3792         onto links skips every other link
3793         <rdar://problem/3692576> focus ring is in odd place after clicking RSS button 
3794         with "Tab to links" enabled
3795         
3796         WebHTMLView has some trickery by which we advance the focused link when nextKeyView
3797         or previousKeyView is called within nextValidKeyView or previousValidKeyView. This
3798         broke in Tiger because AppKit now (sometimes at least) calls nextKeyView more than
3799         once within nextValidKeyView. Fixed 3709110 by making sure we only advance the focus 
3800         once within a call to nextValidKeyView or previousValidKeyView.
3801         
3802         Also, this same trickery didn't work right with hidden views. Fixed 3692576 by checking
3803         whether the view is hidden and bypassing the focus-moving trickery in that case.
3804
3805         * WebView.subproj/WebHTMLViewInternal.h:
3806         renamed inNextValidKeyView -> nextKeyViewAccessShouldMoveFocus
3807         
3808         * WebView.subproj/WebHTMLView.m:
3809         (-[WebHTMLView nextKeyView]):
3810         now clears nextKeyViewAccessShouldMoveFocus
3811         (-[WebHTMLView previousKeyView]):
3812         ditto
3813         (-[WebHTMLView nextValidKeyView]):
3814         now doesn't set focus-moving trigger ivar if view is hidden or has hidden ancestor
3815         (-[WebHTMLView previousValidKeyView]):
3816         ditto
3817
3818 2004-06-30  Trey Matteson  <trey@apple.com>
3819
3820         Dragging within a web view should be allowed to start when the window isn't key.
3821
3822         A few months ago, Chris made this work, but it relied on the fact that all dragging
3823         was done in WebKit.  When WebCore got involved in dragging, it was broken.  Now we
3824         have a new scheme that gets it working again that properly involves WebCore.
3825
3826         The general idea is that when AK asks us whether to accept the first mouse and do
3827         "delayed window ordering", we must consult WC to see if we might start a drag.  In
3828         addition, instead of these drags in non-active windows being started as a special
3829         case in WK, they go through the normal WK-WC drag machinery.  Finally to work in
3830         frames we have to drill to the deepest hit view in acceptsFirstMouse, because previous
3831         hacks to hitTest make the top-most WebHTMLView field all events for its view tree
3832         (which leads to it fielding all acceptFirstMouse messages too).
3833
3834         Reviewed by John.
3835
3836         * WebCoreSupport.subproj/WebBridge.m:
3837         (-[WebBridge mayStartDragAtEventLocation:]):  Glue change for new arg type.
3838         * WebView.subproj/WebHTMLView.m:
3839         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
3840         firstMouseDownEvent ivar is no longer needed.
3841         (-[WebHTMLView _mayStartDragAtEventLocation:]):  Receives a location instead of a drag
3842         event, since we need to do this work when we have no drag event.  This means the
3843         check of the delay for text dragging is moved down to WebCore.
3844         (-[WebHTMLView acceptsFirstMouse:]):  Respond based on whether we might do a drag.
3845         This includes drilling to the deepest view the event hits, whereas we used to only
3846         respond considering the topmost WebHTMLView.
3847         (-[WebHTMLView shouldDelayWindowOrderingForEvent:]):  Ditto.
3848         (-[WebHTMLView mouseDown:]):  Get rid of special case where some activating
3849         mouseDown events weren't sent to WC.  We need to go through the whole pipeline
3850         now to get a drag started properly.
3851         (-[WebHTMLView mouseDragged:]):  Ditto, let WC start the drag.
3852         (-[WebHTMLView mouseUp:]):  firstMouseDownEvent ivar is no longer needed.
3853         * WebView.subproj/WebHTMLViewInternal.h:
3854         * WebView.subproj/WebHTMLViewPrivate.h:
3855
3856 2004-06-25  Trey Matteson  <trey@apple.com>
3857
3858         Added new utility method.
3859
3860         Reviewed by John.
3861
3862         * WebView.subproj/WebFrame.m:
3863         (-[WebFrame _isDescendantOfFrame:]):   New code.
3864         * WebView.subproj/WebFramePrivate.h:
3865
3866 2004-06-24  Richard Williamson   <rjw@apple.com>
3867
3868                 Fixed <rdar://problem/3710313>: CGContext not zeroed when WebImageRenderer is copied
3869                 The context ivar of WebImageRenderer wasn't being nil when the object was copied.
3870
3871         Reviewed by Darin.
3872
3873         * WebCoreSupport.subproj/WebImageRenderer.m:
3874         (-[WebImageRenderer copyWithZone:]):
3875         (-[WebImageRenderer dealloc]):
3876         (-[WebImageRenderer drawImageInRect:fromRect:compositeOperator:context:]):
3877
3878 2004-06-24  Trey Matteson  <trey@apple.com>
3879
3880         3672725 - Assertion failure in URLCompletion code with particular set of bookmarks
3881
3882         Problem was caused by a URL with unescaped unicodes getting into
3883         the Bookmarks file, presumably from import from IE.  We now test
3884         for this case and convert the data on the way in as if it were
3885         user-entered.
3886
3887         Reviewed by John and Darin.
3888
3889         * History.subproj/WebHistoryItem.m:
3890         (-[WebHistoryItem initFromDictionaryRepresentation:]):
3891
3892 2004-06-24  Trey Matteson  <trey@apple.com>
3893
3894         3704950 drag image in DB ConfigBar has horizontal graphics turd WebCore JavaScript
3895
3896         When we generate a drag image (or a selection image too, for that matter) we
3897         translate the CTM using a CG call.  Later, WebImageRenderer adjusts the pattern
3898         phase based on the CTM of the focused view, which doesn't include our translate.
3899         So we must inform WebKit about the additional phase adjustment.
3900
3901         Reviewed by Richard
3902
3903         * WebCoreSupport.subproj/WebGraphicsBridge.h:
3904         * WebCoreSupport.subproj/WebGraphicsBridge.m:
3905         (-[WebGraphicsBridge setAdditionalPatternPhase:]):  New trivial setter.
3906         (-[WebGraphicsBridge additionalPatternPhase]):      ...and getter.
3907         * WebCoreSupport.subproj/WebImageRenderer.m:
3908         (-[WebImageRenderer tileInRect:fromPoint:context:]):  Take any additional phase
3909         adjustment into account when setting phase.
3910
3911 2004-06-24  Trey Matteson  <trey@apple.com>
3912
3913         3693420 - onbeforecut and onbeforepaste need real implementaion
3914
3915         Reviewed by Chris.
3916
3917         * WebView.subproj/WebHTMLView.m:
3918         (-[WebHTMLView validateUserInterfaceItem:]):  Check with WebCore
3919         to see if cut, copy, paste should be enabled.
3920
3921 2004-06-24  John Sullivan  <sullivan@apple.com>
3922
3923         Reviewed by Darin.
3924         
3925         - fixed <rdar://problem/3706792> 8A161: Choosing text encoding for 
3926         error page opens finder window!?
3927
3928         * WebView.subproj/WebFrame.m:
3929         (-[WebFrame _reloadAllowingStaleDataWithOverrideEncoding:]):
3930         take unreachableURL into account
3931
3932 2004-06-23  Richard Williamson   <rjw@apple.com>
3933
3934         Implemented changes for latest npruntime.h.
3935         Made npruntime.h public.
3936                 
3937         Reviewed by Chris.
3938
3939         * Plugins.subproj/WebBaseNetscapePluginView.h:
3940         * Plugins.subproj/WebBaseNetscapePluginView.m:
3941         (-[WebBaseNetscapePluginView pluginScriptableObject]):
3942         (-[WebBaseNetscapePluginView forceRedraw]):
3943         (-[WebBaseNetscapePluginView getVariable:value:]):
3944         * Plugins.subproj/WebBaseNetscapePluginViewPrivate.h:
3945         * Plugins.subproj/WebNetscapePluginPackage.m:
3946         (-[WebNetscapePluginPackage load]):
3947         * Plugins.subproj/npapi.h:
3948         * Plugins.subproj/npapi.m:
3949         (NPN_GetValue):
3950         * Plugins.subproj/npfunctions.h:
3951         * Plugins.subproj/npruntime.h:
3952         * WebKit.pbproj/project.pbxproj:
3953
3954 2004-06-21  Darin Adler  <darin@apple.com>
3955
3956         * English.lproj/StringsNotToBeLocalized.txt: Updated for recent changes.
3957
3958 2004-06-21  Chris Blumenberg  <cblu@apple.com>
3959
3960         Made WebKitErrorDescriptionPlugInCancelledConnection have its own description string
3961         (Not for software update branch)
3962
3963         Reviewed by john.
3964
3965         * English.lproj/Localizable.strings:
3966         * Misc.subproj/WebKitErrors.m:
3967
3968 2004-06-21  Chris Blumenberg  <cblu@apple.com>
3969
3970         Made WebKitErrorDescriptionPlugInCancelledConnection properly reuse an already localized string to avoid loc changes.
3971
3972         Reviewed by john.
3973
3974         * ChangeLog:
3975         * Misc.subproj/WebKitErrors.m:
3976
3977 2004-06-21  Chris Blumenberg  <cblu@apple.com>
3978
3979         Fixed: <rdar://problem/3701269> change in error handling behavior from 10.3.3 to 10.3.4 breaks unreleased Adobe PDF plug-in
3980
3981         Reviewed by john.
3982
3983         * Misc.subproj/WebKitErrors.m:
3984         (registerErrors): register string for WebKitErrorPlugInCancelledConnection
3985         * Misc.subproj/WebKitErrorsPrivate.h:
3986         * Plugins.subproj/WebBaseNetscapePluginStream.h:
3987         * Plugins.subproj/WebBaseNetscapePluginStream.m:
3988         (-[WebBaseNetscapePluginStream startStreamWithURL:expectedContentLength:lastModifiedDate:MIMEType:]): use renamed WEB_REASON_PLUGIN_CANCELLED constant
3989         (-[WebBaseNetscapePluginStream destroyStream]): ditto
3990         * Plugins.subproj/WebNetscapePluginRepresentation.m:
3991         (-[WebNetscapePluginRepresentation cancelWithReason:]): if the reason is WEB_REASON_PLUGIN_CANCELLED, cancel the load with WebKitErrorPlugInCancelledConnection
3992         * Plugins.subproj/WebNetscapePluginStream.m:
3993         (-[WebNetscapePluginStream cancelWithReason:]): ditto
3994
3995 === Safari-146 ===
3996
3997 2004-06-17  Richard Williamson   <rjw@apple.com>
3998
3999                 Changed private headers to be pubic for npapi.h and npfunctions.h.  That
4000                 API was approved but never marked as public in the project file.  This change does
4001                 not make npruntime.h public, which contains the not yet approved changes
4002                 for script-ability of netscape plugins.  Unfortunately, that API will not
4003                 be public for WWDC.
4004                                 
4005         Reviewed by Chris.
4006
4007         * WebKit.pbproj/project.pbxproj:
4008
4009 2004-06-17  Trey Matteson  <trey@apple.com>
4010
4011         3698514 - coordinates in ondragstart and ondrag events are wrong
4012
4013         This part fixes the ondrag coords.  I thought Cocoa passed us the mouse location
4014         in draggedImage:movedTo:, but no, it's the position of the dragged image.
4015         WebCore needs the mouse location, so to calc that we must save away the
4016         offset of the mouse relative to the image when we kick off the drag.
4017
4018         Reviewed by Maciej.
4019
4020         * Misc.subproj/WebNSViewExtras.h:
4021         * Misc.subproj/WebNSViewExtras.m:
4022         (-[NSView _web_dragImage:rect:event:pasteboard:source:offset:]): Add the ability
4023         to return the offset of the cursor wrt to the drag image, since this routine
4024         generates its own drag image and positions it.
4025         * WebView.subproj/WebHTMLView.m:
4026         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
4027         Calc the offset of the cursor wrt to the drag image in the myriad of ways
4028         that we kick off the drag.      
4029         (-[WebHTMLView draggedImage:movedTo:]):  Adjust the location by the
4030         offset we save when we kicked off the drag.
4031         (-[WebHTMLView draggedImage:endedAt:operation:]):  Ditto.
4032         * WebView.subproj/WebHTMLViewInternal.h:
4033         * WebView.subproj/WebImageView.m:
4034         (-[WebImageView mouseDragged:]):  Pass nil for new arg, we don't care.
4035
4036 2004-06-16  David Hyatt  <hyatt@apple.com>
4037
4038         In order to support truncation in Emerson, enhance pointToOffset so that it needn't include partial
4039         character glyphs (the left half of a glyph).
4040         
4041         Reviewed by mjs
4042
4043         * WebCoreSupport.subproj/WebTextRenderer.m:
4044         (-[WebTextRenderer pointToOffset:style:position:reversed:includePartialGlyphs:]):
4045         (-[WebTextRenderer _ATSU_pointToOffset:style:position:reversed:includePartialGlyphs:]):
4046         (-[WebTextRenderer _CG_pointToOffset:style:position:reversed:includePartialGlyphs:]):
4047
4048 === Safari-145 ===
4049
4050 2004-06-16  Darin Adler  <darin@apple.com>
4051
4052         - fixed <rdar://problem/3696081> REGRESSION: <WebKit/CarbonUtils.h> doesn't build any more
4053
4054         Reviewed by Trey.
4055
4056         * Carbon.subproj/CarbonUtils.h: Fixed include. Need to include <ApplicationServices/ApplicationServices.h>
4057         to get CGImageRef; can't include an individual header.
4058
4059 2004-06-15  Richard Williamson   <rjw@apple.com>
4060
4061         Fixed <rdar://problem/3695875>: Objective-C instances that are exported to JavaScript are too promiscuous
4062         
4063         Flip the policy for exposing Objective-C methods and properties.
4064         
4065         Reviewed by Trey.
4066         
4067         * Plugins.subproj/WebPluginPackage.m:
4068         (+[NSObject isSelectorExcludedFromWebScript:]):  Just return YES.
4069         (+[NSObject isKeyExcludedFromWebScript:]):  Just return YES.
4070
4071 2004-06-15  Trey Matteson  <trey@apple.com>
4072
4073         3639321 - Harvard PIN authentication ends up sending PIN as clear text with POST action
4074
4075         When going back/forward to an item, if we went there originally via a POST, 
4076         we ask the user about rePOSTing, and if they say yes, we resend the POST. 
4077         This rePOST case is triggered by the form data that we saved on the b/f item.
4078         In the case of this bug, the overall navigation was accomplished by a POST,
4079         then a redirect, causing a GET.  When a load of type redirect achieves the Committed
4080         stage, we replace the current URL in the b/f item with the new URL (instead of
4081         adding a new item to the b/f list).
4082
4083         The bug is that at the same time we should also update the form data in
4084         the b/f item to match that of the new request.  I think this will normally
4085         mean nil'ing it out, unless there's some way for the result of the redirect
4086         to be another POST.
4087  
4088         The security leak occurred because we did not clear the form data on the item,
4089         so when going back or forward to the page, we would go into the rePOSTing code,
4090         even though we eventually reached that page via a GET (caused by the redirect).
4091         So we would do a POST to the redirect URL containing the private data sent in
4092         the original POST.
4093
4094         Reviewed by mjs and rjw.
4095
4096         * History.subproj/WebHistoryItem.m:
4097         (-[WebHistoryItem _setFormInfoFromRequest:]):  New method, just wraps 3 old set methods.
4098         (-[WebHistoryItem formData]):  Diff being dumb, no change.
4099         (-[WebHistoryItem formContentType]):  Ditto
4100         (-[WebHistoryItem formReferrer]): Ditto
4101         * History.subproj/WebHistoryItemPrivate.h:
4102         * WebView.subproj/WebFrame.m:
4103         (-[WebFrame _createItem:]):  Call new WebHistoryItem method - no change in real behavior
4104         (-[WebFrame _transitionToCommitted:]):  Clear out the form data at the key time, to fix the bug.
4105
4106 2004-06-15  Chris Blumenberg  <cblu@apple.com>