Safari-162 stamp for everything except WebBrowser. In these project, CFBundleShortVe...
[WebKit-https.git] / WebKit / ChangeLog
1 === Safari-162 ===
2
3 2004-09-02  Richard Williamson   <rjw@apple.com>
4
5         Support for patterns in <canvas>.  
6         (These changes attempt to create a CGImageRef from a WebImageRenderer that
7         is used by the pattern drawing function.)
8         
9         Reviewed by Hyatt.
10
11         * WebCoreSupport.subproj/WebImageRenderer.h:
12         * WebCoreSupport.subproj/WebImageRenderer.m:
13         (-[WebImageRenderer dealloc]):
14         (-[WebImageRenderer finalize]):
15         (-[WebImageRenderer imageRef]):
16         (_createImageRef):
17         * WebView.subproj/WebPDFView.h:
18         * WebView.subproj/WebPDFView.m:
19
20 2004-09-01  Chris Blumenberg  <cblu@apple.com>
21
22         Fixed deployment build failure.
23
24         * WebView.subproj/WebDefaultContextMenuDelegate.m:
25         (-[WebDefaultUIDelegate menuItemWithTag:]):
26
27 2004-08-31  Chris Blumenberg  <cblu@apple.com>
28
29         Fixed: 
30         <rdar://problem/3699498> Context menu for editable WebViews should provide items like Cut and Paste
31         <rdar://problem/3781535> REGRESSION (Mail): no context menu after ctrl-clicking a misspelled word
32
33         Reviewed by kocienda.
34
35         * English.lproj/Localizable.strings:
36         * WebView.subproj/WebDefaultContextMenuDelegate.m:
37         (-[WebDefaultUIDelegate menuItemWithTag:]): updated to handle new menu items
38         (-[WebDefaultUIDelegate contextMenuItemsForElement:]): renamed from webView:contextMenuItemsForElement:defaultMenuItems:
39         (-[WebDefaultUIDelegate editingContextMenuItemsForElement:]): new
40         (-[WebDefaultUIDelegate webView:contextMenuItemsForElement:defaultMenuItems:]): moved, now call contextMenuItemsForElement: or editingContextMenuItemsForElement:
41         * WebView.subproj/WebDefaultUIDelegate.h:
42         * WebView.subproj/WebHTMLView.m:
43         (-[WebHTMLView _isSelectionMisspelled]): new
44         (-[WebHTMLView _guessesForMisspelledSelection]): new
45         (-[WebHTMLView _changeSpellingFromMenu:]): new
46         (-[WebHTMLView _ignoreSpellingFromMenu:]): new
47         (-[WebHTMLView _learnSpellingFromMenu:]): new
48         * WebView.subproj/WebHTMLViewPrivate.h:
49         * WebView.subproj/WebUIDelegate.h:
50
51 2004-08-31  Darin Adler  <darin@apple.com>
52
53         - fixed B&I build failure
54
55         * WebView.subproj/WebHTMLView.m: (-[WebTextCompleteController _buildUI]):
56         Work around unwanted warning by adding a cast.
57
58 2004-08-27  Maciej Stachowiak  <mjs@apple.com>
59
60         Reviewed by John.
61
62         <rdar://problem/3778314> REGRESSION: Can't proceed to survey questions on Lominger's Apple website
63         
64         Because we will stop parsing when there is a pending redirection,
65         avoid setting one if no navigation would actually take place
66         because the number of steps is out of range.
67
68         * WebCoreSupport.subproj/WebBridge.m:
69         (-[WebBridge canGoBackOrForward:]):
70
71 2004-08-30  Darin Adler  <darin@apple.com>
72
73         Reviewed by John.
74
75         - part of fix for <rdar://problem/3637519> REGRESSION (125-128): unrepro crash in QListBox::sizeForNumberOfLines at istweb.apple.com
76
77         * WebCoreSupport.subproj/WebTextRendererFactory.m: (-[WebTextRendererFactory clearCaches]):
78         Call [super clearCaches].
79
80 2004-08-30  Darin Adler  <darin@apple.com>
81
82         Reviewed by Chris.
83
84         - did work to prepare for uploading files incrementally when submitting forms
85
86         * History.subproj/WebHistoryItem.m:
87         (-[WebHistoryItem _setFormInfoFromRequest:]): Use NSArray instead of NSData for form data.
88         (-[WebHistoryItem formData]): Ditto.
89         * History.subproj/WebHistoryItemPrivate.h: Ditto.
90         * WebCoreSupport.subproj/WebBridge.m:
91         (-[WebBridge startLoadingResource:withURL:customHeaders:postData:]): Ditto.
92         (-[WebBridge syncLoadResourceWithURL:customHeaders:postData:finalURL:responseHeaders:statusCode:]): Ditto.
93         (-[WebBridge postWithURL:referrer:target:data:contentType:triggeringEvent:form:formValues:]): Ditto.
94         * WebCoreSupport.subproj/WebSubresourceClient.h: Ditto.
95         * WebCoreSupport.subproj/WebSubresourceClient.m:
96         (+[WebSubresourceClient startLoadingResource:withURL:customHeaders:postData:referrer:forDataSource:]): Ditto.
97         * WebView.subproj/WebFrame.m:
98         (-[WebFrame _loadItem:withLoadType:]): Ditto.
99         (-[WebFrame _postWithURL:referrer:target:data:contentType:triggeringEvent:form:formValues:]): Ditto.
100         * WebView.subproj/WebFramePrivate.h: Ditto.
101
102         * WebView.subproj/WebFormDataStream.h: Added.
103         * WebView.subproj/WebFormDataStream.m:
104         (-[WebFormDataStream initWithFormDataArray:]): Placeholder; not done yet.
105         (-[WebFormDataStream formDataArray]): Ditto.
106         * WebKit.pbproj/project.pbxproj: Added WebFormDataStream files.
107
108 2004-08-30  John Sullivan  <sullivan@apple.com>
109
110         Reviewed by Ken.
111
112         - WebKit part of fix for <rdar://problem/3607720> myFrame.print() prints the window 
113         but should only print the frame
114
115         * WebView.subproj/WebUIDelegatePrivate.h:
116         declare new delegate method that includes which frame to print
117
118         * WebView.subproj/WebDefaultUIDelegate.m:
119         (-[WebDefaultUIDelegate webView:printFrameView:]):
120         implement default (no-op) version of new delegate method
121
122         * WebCoreSupport.subproj/WebBridge.m:
123         (-[WebBridge print]):
124         call new delegate method if available, otherwise call old delegate method, for
125         backward compatibility.
126
127 2004-08-27  Maciej Stachowiak  <mjs@apple.com>
128
129         Reviewed by John. 
130         Checked in by Ken
131         
132         Ken comments: It looks like Maciej forgot to land this when he checked in the
133         WebCore portion of this change.
134
135         <rdar://problem/3778314> REGRESSION: Can't proceed to survey questions on Lominger's Apple website
136             
137         Because we will stop parsing when there is a pending redirection,
138         avoid setting one if no navigation would actually take place
139         because the number of steps is out of range.
140     
141         * WebCoreSupport.subproj/WebBridge.m:
142         (-[WebBridge canGoBackOrForward:]):
143
144 2004-08-27  Ken Kocienda  <kocienda@apple.com>
145
146         Reviewed by Chris
147
148         Fix for this bug:
149         
150         <rdar://problem/3756997> WebKit aggressive in making pasted text into a URL, even when it's not much like a URL
151
152         * WebView.subproj/WebHTMLView.m:
153         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): Don't try so hard to coerce data
154         on the pasteboard into a URL, and do not make an anchor with an href for any URLs that are 
155         explicitly present on the pasteboard. Also, move URL pasteboard type check beneath the RTF
156         checks.
157
158 === Safari-161 ===
159
160 2004-08-26  Chris Blumenberg  <cblu@apple.com>
161
162         Fixed build failure on Panther.
163
164         Reviewed by john.
165
166         * WebView.subproj/WebHTMLView.m:
167         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): ifdef'd out call to AppKit SPI
168         (-[WebHTMLView _attributeStringFromDOMRange:]): ditto
169
170 2004-08-26  Chris Blumenberg  <cblu@apple.com>
171
172         Tweak to last check-in.
173
174         * WebView.subproj/WebHTMLView.m:
175         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]):
176
177 2004-08-26  Chris Blumenberg  <cblu@apple.com>
178
179         Fixed: 
180         <rdar://problem/3546412> support for pasting and drag and dropping of RTF and RTFD to editable WebViews
181         <rdar://problem/3745345> use AppKit for converting from DOM to RTF
182
183         Reviewed by rjw.
184
185         * DOM.subproj/WebDOMOperations.m:
186         (-[DOMDocument _documentRange]): new
187         * DOM.subproj/WebDOMOperationsPrivate.h:
188         * WebView.subproj/WebHTMLView.m:
189         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): call AppKit SPI to get a document fragment from an attributed string
190         (-[WebHTMLView string]): added a FIXME
191         (-[WebHTMLView _attributeStringFromDOMRange:]): new, calls AppKit SPI that creates an attributed string from a DOM Range
192         (-[WebHTMLView attributedString]): call _attributeStringFromDOMRange:, fallback to old code if it returns nil
193         (-[WebHTMLView selectedAttributedString]): ditto
194
195 2004-08-26  Chris Blumenberg  <cblu@apple.com>
196
197         Fixed: <rdar://problem/3774178> Plugin hooks for selected state aren't being called
198
199         Reviewed by darin.
200
201         * WebCoreSupport.subproj/WebBridge.m:
202         (-[WebBridge setIsSelected:forView:]): don't forget colons in method names
203
204 2004-08-26  Chris Blumenberg  <cblu@apple.com>
205
206         Fixed: <rdar://problem/3768439> can't click in WebView in Carbon WebKit apps (GetEventPlatformEventRecord returns false)
207
208         Reviewed by rjw.
209
210         * Carbon.subproj/HIWebView.m:
211         (Click): use WebGetEventPlatformEventRecord not GetEventPlatformEventRecord
212         (MouseUp): ditto
213         (MouseMoved): ditto
214         (MouseDragged): ditto
215         (MouseWheelMoved): ditto
216         (WindowHandler): ditto
217         (HIWebViewEventHandler): ditto
218         (UpdateObserver): ditto
219         (WebGetEventPlatformEventRecord): Call GetEventPlatformEventRecord, if that fails fallback to the current event. This is code Eric Schlegel to me to use. 
220
221 2004-08-24  Chris Blumenberg  <cblu@apple.com>
222
223         Fixed with help from Trey:
224         <rdar://problem/3764856> REGRESSION !25-154): Safari accepts mouse clicks (follows links) when not key
225
226         Reviewed by rjw.
227
228         * WebView.subproj/WebHTMLView.m:
229         (-[WebHTMLView _isSelectionEvent:]): brought back from CVS
230         (-[WebHTMLView acceptsFirstMouse:]): only call eventMayStartDrag if _isSelectionEvent returns YES since we only want to allow selection dragging on the first mouse down
231         (-[WebHTMLView shouldDelayWindowOrderingForEvent:]): ditto
232
233 2004-08-24  Ken Kocienda  <kocienda@apple.com>
234
235         Reviewed by Hyatt
236
237         Improved the checks used to see if certain operations can be done based
238         on the state of the selection and whether the selection is editable. I
239         added some helpers and improved some others to assist in making these
240         determinations.
241         
242         This helps to fix this bug:
243         <rdar://problem/3764987> Crash after adding newline to quoted text
244         
245         Since some editing methods expect the the selection to be in a certain state 
246         in order to work, these checks help obviate crashes like 3764987.
247
248         * WebView.subproj/WebHTMLView.m:
249         (-[WebHTMLView _writeSelectionToPasteboard:]): _haveSelection name changed to _hasSelection.
250         (-[WebHTMLView _canCopy]): Checks to see if state is appropriate to perform this operation.
251         (-[WebHTMLView _canCut]): Ditto. Function added.
252         (-[WebHTMLView _canDelete]): Ditto. Function refined.
253         (-[WebHTMLView _canPaste]): Ditto. Function refined.
254         (-[WebHTMLView _canType]): Ditto. Function added.
255         (-[WebHTMLView _hasSelection]): Name changed from _haveSelection.
256         (-[WebHTMLView _hasSelectionOrInsertionPoint]): Added.
257         (-[WebHTMLView _isEditable]): Added.
258         (-[WebHTMLView takeFindStringFromSelection:]): _haveSelection name changed to _hasSelection.
259         (-[WebHTMLView validateUserInterfaceItem:]): Ditto
260         (-[WebHTMLView validRequestorForSendType:returnType:]): Ditto
261         (-[WebHTMLView keyDown:]):
262         (-[WebHTMLView copy:]): Uses new _canCopy check.
263         (-[WebHTMLView cut:]): Uses new _canCut check.
264         (-[WebHTMLView delete:]): Now uses _canDelete check.
265         (-[WebHTMLView paste:]): Now uses _canPaste check.
266         (-[WebHTMLView _updateFontPanel]): _haveSelection name changed to _hasSelection.
267         * WebView.subproj/WebHTMLViewPrivate.h:
268         * WebView.subproj/WebView.m:
269         (-[WebView writeSelectionWithPasteboardTypes:toPasteboard:]): Use selectionState check to
270         determine whether or not operation can be done.
271
272 2004-08-24  Richard Williamson   <rjw@apple.com>
273
274         Fixed <rdar://problem/3770469> Some PDFs open with line of previous page above PDF view
275         
276         Set height resize flags on WebPDFView.
277                 
278         Reviewed by Chris.
279
280         * WebView.subproj/WebPDFView.m:
281         (-[WebPDFView initWithFrame:]):
282
283 2004-08-24  David Hyatt  <hyatt@apple.com>
284
285         Add Atom and RSS MIME types to set of supported XML types.
286         
287         Reviewed by rjw
288
289         * WebView.subproj/WebDataSource.m:
290         (+[WebDataSource _repTypesAllowImageTypeOmission:]):
291         * WebView.subproj/WebFrameView.m:
292         (+[WebFrameView _viewTypesAllowImageTypeOmission:]):
293
294 2004-08-23  Maciej Stachowiak  <mjs@apple.com>
295
296         Reviewed by Kevin.
297
298         - remove annoying ERROR spew and replace with comment
299         
300         * WebView.subproj/WebHTMLView.m:
301         (-[WebHTMLView validAttributesForMarkedText]):
302
303 2004-08-23  Maciej Stachowiak  <mjs@apple.com>
304
305         Reviewed by Richard.
306
307         - a bit of code cleanup
308
309         * WebView.subproj/WebDataSource.m:
310         (-[WebDataSource _stringWithData:]): Call textEncodingName instead of
311         copying it's code.
312         (-[WebDataSource textEncodingName]): Tweak formatting.
313
314 2004-08-23  Chris Blumenberg  <cblu@apple.com>
315
316         Fixed build.
317
318         * WebKit.pbproj/project.pbxproj: make sure we're doing -f on a file, not a directory
319
320 2004-08-23  Chris Blumenberg  <cblu@apple.com>
321
322         Fixed: <rdar://problem/3674867> use new Security framework SPI's to reenable cert acquisition
323
324         Reviewed by john.
325
326         * WebCoreSupport.subproj/WebKeyGeneration.cpp:
327         * WebCoreSupport.subproj/WebKeyGeneration.h:
328         * WebCoreSupport.subproj/WebKeyGenerator.h:
329         * WebCoreSupport.subproj/WebKeyGenerator.m:
330         (-[WebKeyGenerator signedPublicKeyAndChallengeStringWithStrengthIndex:challenge:pageURL:]): re-enabled cert code, call Panther version on Panther, Tiger version on Tiger
331         (-[WebKeyGenerator addCertificatesToKeychainFromData:]): ditto
332         * WebCoreSupport.subproj/WebNewKeyGeneration.c: Added.
333         (gnrAddContextAttribute):
334         (gnrGetSubjPubKey):
335         (gnrNullAlgParams):
336         (gnrSign):
337         (gnrFreeCssmData):
338         (nssArraySize):
339         (signedPublicKeyAndChallengeString):
340         (addCertificateToKeychainFromData):
341         (addCertificatesToKeychainFromData):
342         * WebCoreSupport.subproj/WebNewKeyGeneration.h: Added.
343         * WebKit.pbproj/project.pbxproj:
344
345 2004-08-20  David Hyatt  <hyatt@apple.com>
346
347         Fix the directionality of the unicode hyphen so that on Panther it now matches Tiger.
348         
349         Reviewed by darin
350
351         * Misc.subproj/WebUnicode.m:
352         (_unicodeDirection):
353
354 2004-08-20  Richard Williamson   <rjw@apple.com>
355
356             Implemented new JNI abstraction.  We no longer invoke Java methods
357             directly with JNI, rather we call into the plugin.  This allows the
358             plugin to dispatch the call to the appropriate VM thread.  This
359             change should (will?) fix a whole class of threading related problems with
360             the Java VM.
361
362         Reviewed by Hyatt.
363
364         * WebCoreSupport.subproj/WebBridge.m:
365         (-[WebBridge getAppletInView:]):
366
367 2004-08-20  Trey Matteson  <trey@apple.com>
368
369         3655407 - Editing: -complete: method unimplemented (WebKit editing API)
370
371         This feature is mostly implemented.  The only loose ends I know of are:
372
373         3769654 - text insertions done via complete: should preserve case of full replacement string
374         3769652 - positioning of complete: popup window wrong for right-to-left languages
375
376         Reviewed by John
377
378         * WebView.subproj/WebHTMLView.m:
379         (-[WebHTMLViewPrivate dealloc]):  Free new object.
380         (-[WebHTMLView menuForEvent:]):  Bail on completion session.
381         (-[WebHTMLView windowDidResignKey:]):  Ditto.
382         (-[WebHTMLView windowWillClose:]):  Ditto.
383         (-[WebHTMLView mouseDown:]):  Ditto.
384         (-[WebHTMLView resignFirstResponder]):  Ditto.
385         (-[WebHTMLView keyDown:]):  Bail on completion session if WebCore takes the event.
386         Give the CompleteController a crack at the key event.
387         (-[WebHTMLView _expandSelectionToGranularity:]):  Adopt method rename
388         (-[WebHTMLView complete:]):  Make CompleteController, tell it to do its thing.
389         (-[WebHTMLView checkSpelling:]):  Add ERROR().
390         (-[WebHTMLView showGuessPanel:]): Add ERROR().
391         (-[WebHTMLView _changeSpellingToWord:]): Add ERROR().
392         (-[WebHTMLView ignoreSpelling:]): Add ERROR().
393
394         (-[WebTextCompleteController initWithHTMLView:]):
395         (-[WebTextCompleteController dealloc]):
396         (-[WebTextCompleteController _insertMatch:]): Stick the new string into the doc.
397         (-[WebTextCompleteController _buildUI]):  Make popup window.
398         (-[WebTextCompleteController _placePopupWindow:]):  Position popup window.
399         (-[WebTextCompleteController doCompletion]):  Lookup matches, display window.
400         (-[WebTextCompleteController endRevertingChange:moveLeft:]):  Bail on complete: session.
401         (-[WebTextCompleteController filterKeyDown:]):  Process keys while popup is up.
402         (-[WebTextCompleteController _reflectSelection]):  Handle choice in popup.
403         (-[WebTextCompleteController tableAction:]):  Handle double click in popup
404         (-[WebTextCompleteController numberOfRowsInTableView:]):  Fill table with matches.
405         (-[WebTextCompleteController tableView:objectValueForTableColumn:row:]):  Ditto.
406         (-[WebTextCompleteController tableViewSelectionDidChange:]):  Handle selection change.
407         * WebView.subproj/WebHTMLViewInternal.h:
408
409 2004-08-20  John Sullivan  <sullivan@apple.com>
410
411         * English.lproj/StringsNotToBeLocalized.txt:
412         checked this in; the only difference is that
413         the strings were out of order.
414
415 2004-08-19  Richard Williamson   <rjw@apple.com>
416
417         Continue to call old pollForAppletInView: in Tiger until
418         we get an plugin that supports the new API.
419                 
420         Reviewed by Chris.
421
422         * WebCoreSupport.subproj/WebBridge.m:
423         (-[WebBridge getAppletInView:]):
424         * WebKit.pbproj/project.pbxproj:
425
426 === Safari-158 ===
427
428 2004-08-19  Chris Blumenberg  <cblu@apple.com>
429
430         Fixed typo in comment.
431
432         * Misc.subproj/WebIconDatabase.m:
433         (-[WebIconDatabase _updateFileDatabase]):
434
435 2004-08-19  Chris Blumenberg  <cblu@apple.com>
436
437         Fixed: <rdar://problem/3566336> CrashTracer: .2403 crashes at com.apple.WebKit: -[WebFileDatabase performSetObject:forKey:] + 0x94
438
439         Reviewed by trey.
440
441         * Misc.subproj/WebIconDatabase.m:
442         (-[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
443
444 2004-08-18  Richard Williamson   <rjw@apple.com>
445
446         Replace horrible pollForAppletInView: with new
447         webPlugInGetApplet.  The details of how the applet instance
448         is provided now belong to the Java team.  Yeh.
449                 
450         Reviewed by Chris.
451
452         * WebCoreSupport.subproj/WebBridge.m:
453         (-[WebBridge getAppletInView:]):
454
455 2004-08-17  Richard Williamson   <rjw@apple.com>
456
457         Fixed <rdar://problem/3759093> Need PDF context menu: "Open in Preview" or other external app
458
459         Added support for "Open with ..." in PDF view.
460                 
461         Reviewed by Chris and Trey.
462
463         * ChangeLog:
464         * English.lproj/Localizable.strings:
465         * English.lproj/StringsNotToBeLocalized.txt:
466         * WebView.subproj/WebPDFView.h:
467         * WebView.subproj/WebPDFView.m:
468         (-[WebPDFView initWithFrame:]):
469         (-[WebPDFView dealloc]):
470         (applicationInfoForMIMEType):
471         (-[WebPDFView path]):
472         (-[WebPDFView menuForEvent:]):
473         (-[WebPDFView setDataSource:]):
474         (-[WebPDFView layout]):
475         (-[WebPDFView viewDidMoveToHostWindow]):
476         (-[WebPDFView openWithFinder:]):
477
478 2004-08-17  Richard Williamson   <rjw@apple.com>
479
480         JNI needs both the jmethodID and return type.  Changed API to 
481         pass both.
482                 
483         Reviewed by Chris.
484
485         * Plugins.subproj/WebJavaPlugIn.h:
486
487 2004-08-17  Trey Matteson  <trey@apple.com>
488
489         3764147 - failure of subframe to load leaves links in parent doc broken
490
491         Reviewed by Maciej.
492
493         * WebView.subproj/WebDataSource.m:
494         (-[WebDataSource _receivedMainResourceError:complete:]):
495         Need to call [bridge end] in the case of an error, so WC can clean up.
496
497 === Safari-157 ===
498
499 2004-08-15  Richard Williamson   <rjw@apple.com>
500
501         More changes to np headers.
502         
503         Reviewed by Darin.
504
505         * Plugins.subproj/WebNetscapePluginPackage.m:
506         (-[WebNetscapePluginPackage load]):
507         * Plugins.subproj/npapi.h:
508         * Plugins.subproj/npfunctions.h:
509         * Plugins.subproj/npruntime.h:
510
511 2004-08-13  Darin Adler  <darin@apple.com>
512
513         Reviewed by Chris.
514
515         - fixed <rdar://problem/3760924> Carbon path passed in NPP_StreamAsFile must be in local character set, not UTF-8
516
517         * Plugins.subproj/WebBaseNetscapePluginStream.m:
518         (CarbonPathFromPOSIXPath): Added. New function that uses the path pieces from an FSSpec.
519         This has many advantages; the big one that fixes the bug is that it gives the mangled names
520         that work even for files that have names that can't otherwise be encoded in Carbon-style
521         path names. I didn't write this from scratch: I started with the method in Foundation
522         and just changed it to use FSSpec.
523         (-[WebBaseNetscapePluginStream destroyStream]): Remove code that used NSString and just call
524         CarbonPathFromPOSIXPath instead.
525
526         * Plugins.subproj/npapi.h: The Revision tag wanted to touch this file.
527         I think we are going to have some trouble with this; I'd like to take that out.
528
529         * English.lproj/StringsNotToBeLocalized.txt: Updated.
530
531 2004-08-12  Trey Matteson  <trey@apple.com>
532
533         3761329 - query result links all dead in ingrammicro.com (sometimes)
534         3761328 - links in some docs dead when doc is loaded from WebArchive
535
536         Nasty problem.  It turns out that these result pages are a parent frame with two child
537         frames.  One of the child frames is a 1 or 2 byte text document.  When the text child
538         is the last doc to complete, because we use a Text rep instead of an HTMLRep, it turns
539         out we never send [bridge end] from WebKit.  That mistake results in checkCompleted not
540         being called enough in the part, and we never realize the load is done.  WebCore does
541         not allow redirects to happen until the entire load is complete,  and  the links on this
542         results page are actually little pieces of JS that set location to a generated URL.  Since
543         redirects are not allowed these links all silently fail to do anything, and the doc never
544         achieves a completed state where they will work.
545
546         Solution is to make sure we always call [bridge end] from the DataSource instead of only
547         the HTML rep doing it.
548
549         Reviewed by Richard and Darin.
550
551         * WebView.subproj/WebDataSource.m:
552         (-[WebDataSource _finishedLoading]):  Call [bridge end] here for all kinds of docs.
553         * WebView.subproj/WebHTMLRepresentation.m:
554         (-[WebHTMLRepresentation finishedLoadingWithDataSource:]):  Don't call is just for HTMLReps.
555
556 2004-08-12  Richard Williamson   <rjw@apple.com>
557
558         Bring npruntime.h and friends closer to compliance with
559         latest spec.
560
561         Reviewed by Maciej.
562
563         * Plugins.subproj/npapi.h:
564         * Plugins.subproj/npfunctions.h:
565         * Plugins.subproj/npruntime.h:
566         (_NPString::):
567         (_NPString::_NPVariant::):
568         * copy-webcore-files-to-webkit:
569
570 2004-08-12  Chris Blumenberg  <cblu@apple.com>
571
572         Fixed: <rdar://problem/3761097> should be able to option-drag selection so HTML can be copied within a page
573
574         Reviewed by rjw.
575
576         * WebView.subproj/WebHTMLView.m:
577         (-[WebHTMLView _isMoveDrag]): new, take into account the option key
578         (-[WebHTMLView draggingUpdatedWithDraggingInfo:actionMask:]): call _isMoveDrag
579         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): call _isMoveDrag
580
581 2004-08-12  Richard Williamson   <rjw@apple.com>
582
583         Quick fix for 3760903.  The real fix is described in 3760920.  Needed
584         by Java plugin guys so they can be unblocked for feature freeze.
585                 
586         Reviewed by Chris.
587
588         * WebView.subproj/WebFrame.m:
589         (-[WebFrame _reloadForPluginChanges]):
590         * WebView.subproj/WebHTMLView.m:
591         (-[WebHTMLView addSubview:]):
592
593 2004-08-12  Chris Blumenberg  <cblu@apple.com>
594
595         Fixed: <rdar://problem/3760898> error pages in subframes attempt to load appledata URLs when reloaded
596
597         Reviewed by darin.
598
599         * WebView.subproj/WebFrame.m:
600         (-[WebFrame _createItem:]): when setting the original URL of the history item, use the unreachable URL
601
602 === Safari-156 ===
603
604 2004-08-11  Adele Amchan  <adele@apple.com>
605
606         Reviewed by me, bug fix by Darin.
607
608         - fixed <rdar://problem/3736477> Pages don't load if hard drive is named with non-ASCII Symbol
609
610         * Plugins.subproj/WebBaseNetscapePluginStream.m:
611         (-[WebBaseNetscapePluginStream startStreamWithURL:expectedContentLength:lastModifiedDate:MIMEType:]):
612         Use UTF8String instead of cString to convert the MIME type to a C string. Safer, since it can't
613         ever fail due to encoding problems even though this string should always be ASCII.
614         (-[WebBaseNetscapePluginStream destroyStream]): Use stringWithUTF8String to convert the path name
615         to an NSString, since stringWithCString is deprecated (doesn't really matter since the path is always
616         all ASCII). Fix the bug by calling fileSystemRepresentation on the NSString to turn it into a C
617         string form. Even though the POSIX path can't have any non-ASCII characters in it, the Carbon path
618         can, so we need to use this instead of cString which can fail depending on characters and encoding.
619
620 2004-08-09  Maciej Stachowiak  <mjs@apple.com>
621
622         Reviewed by Trey.
623
624         WebKit part of:
625
626         - made basic marked text highlighting work to complete basic level of <rdar://problem/3704359> input method support not yet implemented for HTML editing
627
628         * WebView.subproj/WebHTMLView.m:
629         (-[WebHTMLView markedRange]): Use new bridge calls instead of internal marked range storage.
630         (-[WebHTMLView hasMarkedText]): Likewise.
631         (-[WebHTMLView unmarkText]): Likewise.
632         (-[WebHTMLView _selectMarkedText]): Likewise.
633         (-[WebHTMLView _selectRangeInMarkedText:]): Likewise.
634         (-[WebHTMLView _selectionIsInsideMarkedText]): Likewise.
635         (-[WebHTMLView _updateSelectionForInputManager]): Likewise.
636         (-[WebHTMLView setMarkedText:selectedRange:]): Use direct bridge call instead
637         of private _selectMarkedDOMRange: method, which would now be trivial.
638         * WebView.subproj/WebHTMLViewInternal.h: Remove unneeded
639
640 2004-08-10  Darin Adler  <darin@apple.com>
641
642         Reviewed by Ken.
643
644         - change name of WebMakeCollectable to WebNSRetainCFRelease so it fits into the
645           "NS and CF retain counts are separate" mental model, rather than the "think about
646           how garbage collection works" one
647
648         * Misc.subproj/WebKitNSStringExtras.m: (+[NSString _web_encodingForResource:]): Rename.
649         * Misc.subproj/WebNSObjectExtras.h: (WebNSRetainCFRelease): Ditto.
650         * Misc.subproj/WebNSURLExtras.m:
651         (+[NSURL _web_URLWithData:relativeToURL:]): Ditto.
652         (-[NSURL _web_URLWithLowercasedScheme]): Ditto.
653
654 2004-08-09  Trey Matteson  <trey@apple.com>
655
656         3756599 - REGRESSION: hit assertion in KWQPageState invalidate
657
658         By inspection I found a flaw in the recently added logic, although
659         I have no steps to repro.  My theoretical explanation is that we would
660         get two errors and go through _receivedMainResourceError: twice, which
661         would cause the pageState to be invalidated twice, which is the only
662         way I can see to hit the assert.
663
664         Reviewed by Darin
665
666         * WebView.subproj/WebFrame.m:
667         (-[WebFrame _receivedMainResourceError:]):  Clear the pageState out
668         of the history item after it's been invalidated by WebCore.
669
670 2004-08-09  Richard Williamson   <rjw@apple.com>
671
672         Inspired by Trey we have a much better approach for conditionally
673         linking Quart.framework.  Instead of multiple targets we use 
674         `` to invoke some inline script to extend COMMON_LDFLAGS as
675         necessary.  Thanks Trey!
676         
677         Reviewed by Trey.
678
679         * WebKit.pbproj/project.pbxproj:
680
681 2004-08-09  Trey Matteson  <trey@apple.com>
682
683         Hookup UI for "Continuous Spelling" menu item.
684
685         Reviewed by Richard.
686
687         * WebView.subproj/WebView.m:
688         (-[WebView validateUserInterfaceItem:]):  Enable and check the
689         menu item to reflect our state.
690         (-[WebView toggleContinuousSpellChecking:]):  Change type to
691         IBAction, just cosmetic.
692         * WebView.subproj/WebViewPrivate.h:
693
694 2004-08-08  Trey Matteson  <trey@apple.com>
695
696         3745023 - Safari crashes trying to access anchor while downloading
697
698         I bet this is behind a few other crashers as well.  In this bug the start of the
699         download leaves a KWQPageState hanging around, and when that is freed it damages
700         the part and view.  If you're still using that page, you're dead.
701
702         The fix is to properly invalidate the PageState when we receive an error before
703         reaching WebFrameCommitted state.  Normally this happens when a page is reheated
704         from the PageState, but in this case we never manage to leave the page to begin
705         with, although we've already created the PageState.
706
707         Other errors besides the synthetic one download generates would have caused similar
708         crashing.  Another example would be clicking on a second link before the load
709         caused by clicking on the first link reached committed state.
710
711         Reviewed by Richard
712
713         * WebView.subproj/WebDataSource.m:
714         (-[WebDataSource _receivedMainResourceError:complete:]):  Let the frame do the
715         main work (since it has access to the pageCache state).  Also renamed to make
716         it clear that this is about an error for the main resource.
717         * WebView.subproj/WebDataSourcePrivate.h:
718         * WebView.subproj/WebFrame.m:
719         (-[WebFrame _receivedMainResourceError:]):  Let WC know about the failure, as
720         the DataSource used to, but now pass the pageCache state along too.
721         * WebView.subproj/WebFramePrivate.h:
722         * WebView.subproj/WebMainResourceClient.m:
723         (-[WebMainResourceClient receivedError:]):  Call renamed method.
724         (-[WebMainResourceClient cancelWithError:]):  Ditto.
725
726 2004-08-08  Vicki Murley  <vicki@apple.com>
727
728         Reviewed by vicki (changes by rjw)
729  
730         - make "weak" linking with Quartz work with buildit
731
732         * WebKit.pbproj/project.pbxproj: 
733
734 2004-08-06  Ken Kocienda  <kocienda@apple.com>
735
736         Reviewed by Maciej
737
738         Finish off spellchecking support to HTML editing. Includes work to
739         enable continuous spellchecking.
740
741         * WebCoreSupport.subproj/WebBridge.m:
742         (-[WebBridge isContinuousSpellCheckingEnabled]): Simple bridge method.
743         * WebCoreSupport.subproj/WebTextRenderer.m:
744         (-[WebTextRenderer drawLineForMisspelling:withWidth:]): New method to add
745         AppKit-style misspelling underline.
746
747 === Safari-155 ===
748
749 2004-08-05  Darin Adler  <darin@apple.com>
750
751         * WebView.subproj/WebHTMLView.m: Added a list of methods that NSTextView implements that we don't.
752         All inside #if 0.
753
754 2004-08-05  Chris Blumenberg  <cblu@apple.com>
755
756         Fixed unnecessary import of NSURLFileTypeMappings.
757
758         * WebView.subproj/WebMainResourceClient.m:
759
760 2004-08-05  Richard Williamson   <rjw@apple.com>
761
762         Make builds conditionally include -framework Quartz.
763                 
764         Reviewed by Chris.
765
766         * WebKit.pbproj/project.pbxproj:
767
768 2004-08-04  Darin Adler  <darin@apple.com>
769
770         - fixed broken Deployment build
771
772         * Carbon.subproj/HIWebView.m: (Click):
773         Remove code that checks err variable before setting it.
774
775 2004-08-03  Richard Williamson   <rjw@apple.com>
776
777         First cut at dirt simple PDF support.  This feature
778         depends on Quartz.framework (parent of PDFKit), which only exists
779         on Tiger.  So, we "weak" link against Quartz.
780                 
781         We do very basic PDF rendering.  Coming up are support for
782         "Find..." and linearized PDF (incremental).  Linearized PDF 
783         support will require API changes in PDFKit.
784                 
785         No UI is added to Safari, yet.
786                 
787         Reviewed by John.
788
789         * WebKit.pbproj/project.pbxproj:
790         * WebView.subproj/WebDataSource.m:
791         (+[WebDataSource _repTypesAllowImageTypeOmission:]):
792         * WebView.subproj/WebFrameView.m:
793         (+[WebFrameView _viewTypesAllowImageTypeOmission:]):
794         * WebView.subproj/WebPDFRepresentation.h: Added.
795         * WebView.subproj/WebPDFRepresentation.m: Added.
796         (-[WebPDFRepresentation finishedLoadingWithDataSource:]):
797         (-[WebPDFRepresentation canProvideDocumentSource]):
798         (-[WebPDFRepresentation documentSource]):
799         (-[WebPDFRepresentation title]):
800         * WebView.subproj/WebPDFView.h: Added.
801         * WebView.subproj/WebPDFView.m: Added.
802         (-[WebPDFView initWithFrame:]):
803         (-[WebPDFView setDataSource:]):
804         (-[WebPDFView dataSourceUpdated:]):
805         (-[WebPDFView setNeedsLayout:]):
806         (-[WebPDFView layout]):
807         (-[WebPDFView viewWillMoveToHostWindow:]):
808         (-[WebPDFView viewDidMoveToHostWindow]):
809
810         Copied fix from Jaguar carbon/cocoa work.
811                 
812         Reviewed by Darin (Jaguar version)
813
814         * Carbon.subproj/CarbonWindowAdapter.h:
815         * Carbon.subproj/CarbonWindowAdapter.m:
816         (-[CarbonWindowAdapter relinquishFocus]):
817         * Carbon.subproj/HIWebView.m:
818         (Click):
819         (OwningWindowChanged):
820         (WindowHandler):
821
822 2004-08-03  David Hyatt  <hyatt@apple.com>
823
824         Add the text/xsl MIME type as one that can be displayed.
825         
826         Reviewed by john
827
828         * WebView.subproj/WebDataSource.m:
829         (+[WebDataSource _repTypesAllowImageTypeOmission:]):
830         * WebView.subproj/WebFrameView.m:
831         (+[WebFrameView _viewTypesAllowImageTypeOmission:]):
832
833 2004-08-03  Maciej Stachowiak  <mjs@apple.com>
834
835         Reviewed by Ken.
836
837         - implemented enough of the NSTextInput protocol and added the
838         proper calls to NSInputManager to allow input methods to
839         work. However, the text is not marked yet.
840
841         * WebView.subproj/WebHTMLViewInternal.h: Added new fields to track
842         marked range.
843         * WebView.subproj/WebHTMLView.m:
844         (-[WebHTMLView markedRange]): Implemented
845         (-[WebHTMLView conversationIdentifier]): Implemented
846         (-[WebHTMLView hasMarkedText]): Implemented
847         (-[WebHTMLView unmarkText]): Implemented
848         (-[WebHTMLView setMarkedText:selectedRange:]): Implemented - does
849         not yet handle attributes in attributed strings
850         (-[WebHTMLView insertText:]): Modified to handle replacing or abandoning
851         the marked text when set.
852         (-[WebHTMLView _selectMarkedText]): new helper method, self-explanatory
853         (-[WebHTMLView _setMarkedDOMRange:]): hitto.
854         (-[WebHTMLView _selectRangeInMarkedText:]): ditto
855         (-[WebHTMLView _discardMarkedText]): ditto
856         (-[WebHTMLView _selectionIsInsideMarkedText]): ditto
857         (-[WebHTMLView _updateSelectionForInputManager]): ditto
858
859         - did a bit of refactoring while I was in here.
860
861         * WebView.subproj/WebHTMLView.m:
862         (-[WebHTMLView _shouldReplaceSelectionWithText:givenAction:]): added this new
863         helper method to simplify the common case of calling the shouldInsertText: delegate
864         (-[WebHTMLView pasteAsPlainText:]): use it
865         (-[WebHTMLView insertTab:]): ditto
866         (-[WebHTMLView insertNewline:]): ditto
867         (-[WebHTMLView _changeWordCaseWithSelector:]): ditto
868         (-[WebHTMLView _changeSpellingToWord:]): ditto
869         (-[WebHTMLView _selectionChanged]): ditto
870
871 2004-08-03  Chris Blumenberg  <cblu@apple.com>
872
873         Fixed: <rdar://problem/3572737> Images not resizing at www.bmx-test.com (spoof nofix)
874
875         Reviewed by darin.
876
877         * WebCoreSupport.subproj/WebImageRendererFactory.m:
878         (-[WebImageRendererFactory supportedMIMETypes]): hard code image/pjpeg to the list of image mime types that we can handle
879
880 2004-08-03  Darin Adler  <darin@apple.com>
881
882         Reviewed by Ken.
883
884         - fixed <rdar://problem/3740937> ER: A way to turn a DOMRange into text (equivalent of -innerText)
885
886         * DOM.subproj/DOMPrivate.h: Added.
887         * WebKit.pbproj/project.pbxproj: Added DOMPrivate.h.
888         * copy-webcore-files-to-webkit: Added DOMPrivate.h.
889
890 2004-08-02  John Sullivan  <sullivan@apple.com>
891
892         Reviewed by Darin.
893
894         WebKit part of fix for <rdar://problem/3631868> NSToolbar adoption: 
895         Tab key should cycle around toolbar and page content
896
897         * WebView.subproj/WebHTMLView.m:
898         removed overrides of nextKeyView and previousKeyView
899         (-[WebHTMLView nextValidKeyView]):
900         call super only if we can't move the focus within the frame hierarchy
901         (-[WebHTMLView previousValidKeyView]):
902         ditto
903         * WebView.subproj/WebHTMLViewInternal.h:
904         removed nextKeyViewAccessShouldMoveFocus ivar
905
906 2004-08-02  Ken Kocienda  <kocienda@apple.com>
907
908         Reviewed by John
909
910         Update name of firstResponderIsSelfOrDescendantView, adding _web_ prefix to this SPI call.
911         
912         Do some work to make caret blinking in newly-created editable WebView's that are 
913         similar in structure to Blot more "automatic".
914
915         * Misc.subproj/WebNSViewExtras.h: Change name of firstResponderIsSelfOrDescendantView. Add _web_ prefix.
916         * Misc.subproj/WebNSViewExtras.m:
917         (-[NSView _web_firstResponderIsSelfOrDescendantView]): Name change.
918         (-[NSView _web_firstResponderCausesFocusDisplay]): Adds an additional check for whether the view's
919         is first responder. This helps to make the focus setting in viewDidMoveToWindow work right.
920         * WebView.subproj/WebHTMLView.m:
921         (-[WebHTMLView updateFocusDisplay]): Use _web_firstResponderCausesFocusDisplay now instead of 
922         firstResponderIsSelfOrDescendantView.
923         (-[WebHTMLView viewDidMoveToWindow]): Schedule call to updateFocusDisplay for the next crank of the
924         run loop. The reason is that placing the caret in the just-installed view requires the HTML/XML 
925         document to be available on the WebCore side, but it is not at the time this code is running. 
926         However, it will be there on the next crank of the run loop. Doing this helps to make a 
927         blinking caret appear in a new, empty window "automatic".
928         (-[WebHTMLView performKeyEquivalent:]):  _web_firstResponderIsSelfOrDescendantView name change.
929         * WebView.subproj/WebView.m:
930         (-[WebView _performResponderOperation:with:]): _web_firstResponderIsSelfOrDescendantView name change.
931
932 2004-07-29  Maciej Stachowiak  <mjs@apple.com>
933
934         Reviewed by John.
935
936         Added stubbed-out versions of all the NSTextInput protocol
937         methods, and put comments in places where we will need to call the
938         input manager to make input methods work 100%.
939         
940         * WebCoreSupport.subproj/WebBridge.m:
941         (-[WebBridge respondToChangedSelection]):
942         * WebView.subproj/WebHTMLView.m:
943         (-[WebHTMLView mouseDown:]):
944         (-[WebHTMLView mouseDragged:]):
945         (-[WebHTMLView mouseUp:]):
946         (-[WebHTMLView _interceptEditingKeyEvent:]):
947         (-[WebHTMLView keyDown:]):
948         (-[WebHTMLView _selectionChanged]):
949         (-[WebHTMLView _delegateDragSourceActionMask]):
950         (-[WebHTMLView validAttributesForMarkedText]):
951         (-[WebHTMLView characterIndexForPoint:]):
952         (-[WebHTMLView firstRectForCharacterRange:]):
953         (-[WebHTMLView selectedRange]):
954         (-[WebHTMLView markedRange]):
955         (-[WebHTMLView attributedSubstringFromRange:]):
956         (-[WebHTMLView conversationIdentifier]):
957         (-[WebHTMLView hasMarkedText]):
958         (-[WebHTMLView unmarkText]):
959         (-[WebHTMLView setMarkedText:selectedRange:]):
960         (-[WebHTMLView doCommandBySelector:]):
961         (-[WebHTMLView insertText:]):
962         * WebView.subproj/WebHTMLViewInternal.h:
963         * WebView.subproj/WebView.m:
964
965 === Safari-154 ===
966
967 2004-07-29  Darin Adler  <darin@apple.com>
968
969         Reviewed by Ken.
970
971         - fill in some unimplemented methods
972
973         * WebView.subproj/WebHTMLView.m:
974         (-[WebHTMLView moveToBeginningOfDocument:]): Implement, using new document granularity.
975         (-[WebHTMLView moveToBeginningOfDocumentAndModifySelection:]): Ditto.
976         (-[WebHTMLView moveToBeginningOfParagraph:]): Implement, even though the underlying WebCore
977         code is not yet implemented.
978         (-[WebHTMLView moveToBeginningOfParagraphAndModifySelection:]): Ditto.
979         (-[WebHTMLView moveToEndOfDocument:]): More of the same.
980         (-[WebHTMLView moveToEndOfDocumentAndModifySelection:]): Ditto.
981         (-[WebHTMLView moveToEndOfParagraph:]): More of the same.
982         (-[WebHTMLView moveToEndOfParagraphAndModifySelection:]): Ditto.
983         (-[WebHTMLView pageDown:]): Added comment describing desired behavior.
984         (-[WebHTMLView pageUp:]): Ditto.
985         (-[WebHTMLView copyFont:]): Ditto.
986         (-[WebHTMLView pasteFont:]): Ditto.
987         (-[WebHTMLView changeAttributes:]): Ditto.
988         (-[WebHTMLView indent:]): Ditto.
989
990 2004-07-28  Chris Blumenberg  <cblu@apple.com>
991
992         Fixed: <rdar://problem/3737864> Can not download image from web by using drag and drop
993
994         Reviewed by john.
995
996         * Misc.subproj/WebNSPasteboardExtras.m:
997         (-[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
998
999 2004-07-28  Maciej Stachowiak  <mjs@apple.com>
1000
1001         Reviewed by John.
1002
1003         - avoid triggering an assertion when using dead keys (like option-e)
1004
1005         * WebView.subproj/WebHTMLView.m:
1006         (-[WebHTMLView insertText:]): Don't pass zero-length strings to
1007         WebCore. Dead-key input seems to insert empty strings as a side
1008         effect.
1009
1010 2004-07-28  Trey Matteson  <trey@apple.com>
1011
1012         Spellchecking, Part I.  Basic spellcheck is working.  Spelling panel is hooked up.
1013
1014         At this point, no special marking of misspellings, no grammar check, no context
1015         menu integration, no "check continually" mode.
1016
1017         Reviewed by Ken.
1018
1019         * WebCoreSupport.subproj/WebBridge.m:
1020         (-[WebBridge spellCheckerDocumentTag]):  Typical bridge glue.
1021         * WebView.subproj/WebHTMLView.m:
1022         (-[WebHTMLView validateUserInterfaceItem:]):  Validate various spelling actions.
1023         (-[WebHTMLView checkSpelling:]):  Call WC for real work, update panel.
1024         (-[WebHTMLView showGuessPanel:]):  Show panel, call WC for real work.
1025         (-[WebHTMLView _changeSpellingToWord:]):  Apply correction to our doc.
1026         (-[WebHTMLView changeSpelling:]):  Simple pass through to above method.
1027         (-[WebHTMLView ignoreSpelling:]):  Tell checker to ignore the word.
1028         * WebView.subproj/WebView.m:
1029         (-[WebView _close]):  Call AK's closeSpellDocumentWithTag: for proper cleanup.
1030
1031 2004-07-27  John Sullivan  <sullivan@apple.com>
1032
1033         Reviewed by Trey.
1034         
1035         WebKit part of fix for <rdar://problem/3622268> Reload failed pages when a 
1036         network problem is corrected, inc. using Network Diagnostics
1037
1038         * WebView.subproj/WebFrame.m:
1039         (-[WebFrame reload]):
1040         This method did not handle unreachableURLs at all. The reason Safari's Reload
1041         did work with unreachableURLs was that Safari does "reloadObeyingLocationField"
1042         which never actually calls -[WebFrame reload]. Fixed by creating a fresh
1043         request for the previously-unreachable URL.
1044
1045 2004-07-26  Richard Williamson   <rjw@apple.com>
1046
1047         Fixed 3739737.  When setting the focus to a NSView, set the focus
1048         carbon focus to kControlIndicatorPart.  kControlIndicatorPart is a
1049         placeholder value for use to indicate that Cocoa has the focus.  
1050         
1051         Reviewed by Ken.
1052
1053         * Carbon.subproj/CarbonWindowAdapter.m:
1054         (-[CarbonWindowAdapter makeFirstResponder:]):
1055         * Carbon.subproj/HIWebView.m:
1056         (Click):
1057         (SetFocusPart):
1058
1059 2004-07-26  Chris Blumenberg  <cblu@apple.com>
1060
1061         Fixed: <rdar://problem/3689734> dragging to an editable WebView does not scroll document
1062
1063         Reviewed by rjw.
1064
1065         * WebView.subproj/WebView.m:
1066         (-[WebView documentViewAtWindowPoint:]): new, factored from draggingDocumentViewAtWindowPoint:
1067         (-[WebView _draggingDocumentViewAtWindowPoint:]): call documentViewAtWindowPoint:
1068         (-[WebView _autoscrollForDraggingInfo:timeDelta:]): forward call to the document view
1069         (-[WebView _shouldAutoscrollForDraggingInfo:]): ditto
1070
1071 === Safari-153 ===
1072
1073 2004-07-23  Ken Kocienda  <kocienda@apple.com>
1074
1075         Reviewed by Trey
1076
1077         Fix for this bug:
1078         
1079         <rdar://problem/3738920> Caret blinks in inactive window
1080
1081         As part of the fix, I cleaned up the way we handle special drawing that needs
1082         to be done in the HTML view that is first responder in the key window (e.g the
1083         drawing of text selection highlight and caret blinking).
1084
1085         * WebView.subproj/WebHTMLView.m:
1086         (-[WebHTMLView updateFocusDisplay]): New method that centralizes the changes
1087         we need to make when the first responder-ness of the view or key-ness of the
1088         window changes.
1089         (-[WebHTMLView viewDidMoveToWindow]): Call new updateFocusDisplay helper.
1090         (-[WebHTMLView windowDidBecomeKey:]): Ditto.
1091         (-[WebHTMLView windowDidResignKey:]): Ditto.
1092         (-[WebHTMLView becomeFirstResponder]): Ditto.
1093         (-[WebHTMLView resignFirstResponder]): Ditto.
1094
1095 2004-07-22  Darin Adler  <darin@apple.com>
1096
1097         * Plugins.subproj/npruntime.h: Update with new version from newer JavaScriptCore.
1098
1099 2004-07-21  Ken Kocienda  <kocienda@apple.com>
1100
1101         Reviewed by Trey
1102
1103         * WebCoreSupport.subproj/WebBridge.m: Remove interceptEditingKeyEvent
1104         bridge call over. This method of handling editing key events is now
1105         obsolete.
1106         * WebView.subproj/WebHTMLView.m:
1107         (-[WebHTMLView performKeyEquivalent:]): Send key event over to the 
1108         DOM if this view is or contains the first responder. This now
1109         happens unconditionally.
1110         (-[WebHTMLView keyDown:]): Send key event to the DOM, then see if
1111         the web view wants to interpret it an an editing key event. This
1112         is the new place to intercept key events for editing.
1113
1114 2004-07-21  Ken Kocienda  <kocienda@apple.com>
1115
1116         Reviewed by John
1117
1118         Add implementations for these methods. Formerly, they logged an error.
1119
1120         * WebView.subproj/WebHTMLView.m:
1121         (-[WebHTMLView moveToBeginningOfLine:]):
1122         (-[WebHTMLView moveToBeginningOfLineAndModifySelection:]):
1123         (-[WebHTMLView moveToEndOfLine:]):
1124         (-[WebHTMLView moveToEndOfLineAndModifySelection:]):
1125
1126 2004-07-21  Ken Kocienda  <kocienda@apple.com>
1127
1128         Reviewed by John
1129
1130         Added some more handlers for standard Cocoa key bindings. 
1131         These are "secrets" of NSText, meaning they are not public API, but we choose to mimic.
1132
1133         * WebView.subproj/WebHTMLView.m:
1134         (-[WebHTMLView moveToBeginningOfDocumentAndModifySelection:]):
1135         (-[WebHTMLView moveToBeginningOfLineAndModifySelection:]):
1136         (-[WebHTMLView moveToBeginningOfParagraphAndModifySelection:]):
1137         (-[WebHTMLView moveToEndOfDocumentAndModifySelection:]):
1138         (-[WebHTMLView moveToEndOfLineAndModifySelection:]):
1139         (-[WebHTMLView moveToEndOfParagraph:]):
1140         (-[WebHTMLView moveToEndOfParagraphAndModifySelection:]):
1141         * WebView.subproj/WebView.h: Add these declarations to the comment in the header listing
1142         the responder-like methods we support.
1143         * WebView.subproj/WebView.m:
1144         (-[WebView moveToBeginningOfParagraphAndModifySelection:]):
1145         (-[WebView moveToEndOfParagraphAndModifySelection:]):
1146         (-[WebView moveToBeginningOfLineAndModifySelection:]):
1147         (-[WebView moveToEndOfLineAndModifySelection:]):
1148         (-[WebView moveToBeginningOfDocumentAndModifySelection:]):
1149         (-[WebView moveToEndOfDocumentAndModifySelection:]):
1150
1151 2004-07-20  Ken Kocienda  <kocienda@apple.com>
1152
1153         Reviewed by Richard
1154
1155         * Misc.subproj/WebNSEventExtras.h: Added helper that returns whether
1156         a key event has a binding in the key binding manager.
1157         * Misc.subproj/WebNSEventExtras.m: 
1158         (-[NSEvent _web_keyBindingManagerHasBinding]): New helper mentioned above.
1159         * Plugins.subproj/npruntime.h:
1160         * WebView.subproj/WebHTMLView.m:
1161         (-[WebHTMLView performKeyEquivalent:]): Add a check of whether the key event 
1162         has a binding in the key binding manager. This works around the fact that
1163         NSResponder's interpretKeyEvents does not return a value telling whether
1164         or not the key was handled. This now makes it possible for us to trap
1165         modified key events we know we can handle (like those command-key + arrow events 
1166         used for text navigation), while letting all others pass.
1167
1168 2004-07-20  Chris Blumenberg  <cblu@apple.com>
1169
1170         Fixed:
1171         <rdar://problem/3580589> REGRESSION (1.1-1.2): can't open a new window for an image that has not loaded
1172         <rdar://problem/3612691> Missing image icons (blue ?) lack context menu
1173
1174         Reviewed by john.
1175
1176         * WebView.subproj/WebDefaultContextMenuDelegate.m:
1177         (-[WebDefaultUIDelegate webView:contextMenuItemsForElement:defaultMenuItems:]): if there is no image, but there is an image URL, provide image context menu items besides "Copy Image" 
1178         * WebView.subproj/WebHTMLView.m:
1179         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]): don't drag an image with an actual image
1180         (-[WebHTMLView _mayStartDragAtEventLocation:]): ditto
1181
1182 2004-07-20  Maciej Stachowiak  <mjs@apple.com>
1183
1184         Reviewed by Chris.
1185
1186         <rdar://problem/3721690> REGRESSION (125.7-148u) clicking on links at macosx.apple.com/Builds does not load new page in frame
1187         
1188         * Plugins.subproj/npruntime.h:
1189         * WebCoreSupport.subproj/WebBridge.m:
1190         (-[WebBridge canTargetLoadInFrame:]): Don't apply the restrictions to frames
1191         that are in the same window (Mozilla does this too).
1192
1193 2004-07-20  Richard Williamson   <rjw@apple.com>
1194
1195                 Fix for 3728558.  Fixed the key event handling in the carbon/cocoa
1196                 integration code.  This does not fix the arrow keys not working on
1197                 initial focus problem also mentioned in the bug.
1198                 
1199                 Bumped the version of the NP function structures.
1200                 
1201         Reviewed by John.
1202
1203         * Carbon.subproj/CarbonUtils.m:
1204         (WebInitForCarbon):
1205         (PoolCleaner):
1206         * Carbon.subproj/HIWebView.m:
1207         (OwningWindowChanged):
1208         (WindowHandler):
1209         * Plugins.subproj/npapi.h:
1210
1211 2004-07-20  Trey Matteson  <trey@apple.com>
1212
1213         3733698 REGRESSION: sometimes dragging photos on homepage.mac.com leads to an assertion
1214
1215         Relax an assertion, as we ran into a valid case where it's not true.
1216
1217         Reviewed by Chris.
1218
1219         * WebView.subproj/WebHTMLView.m:
1220         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
1221         Removing this assert allows for the oddball case of a drag gesture that
1222         starts on one element, but then the element is no longer there when
1223         the drag is about to start.
1224
1225 2004-07-20  Ken Kocienda  <kocienda@apple.com>
1226
1227         Reviewed by Hyatt
1228
1229         Fix for this bug:
1230         
1231         <rdar://problem/3707505> HTMLCompose: blinking cursor in both an address text field and the message body
1232
1233         * WebView.subproj/WebHTMLView.m:
1234         (-[WebHTMLView setCaretVisible:]): New helper. Calls over bridge to do the work.
1235         (-[WebHTMLView windowDidBecomeKey:]): This function cannot just assume that self is first responder
1236         (and do things like adjusting text background color and restoring focus rings). 
1237         First-responder-ness needs to be checked first. Now it is.
1238         (-[WebHTMLView windowDidResignKey:]): Ditto.
1239         (-[WebHTMLView becomeFirstResponder]): Call new helper to make caret visible.
1240         (-[WebHTMLView resignFirstResponder]): Call new helper to make caret invisible.
1241
1242 2004-07-12  Richard Williamson   <rjw@apple.com>
1243
1244         Fixed 3721917.  The RealPlayer plugin doesn't support the new 
1245         NPPVpluginScriptableNPObject variable passed to NPP_GetValue and
1246         incorrectly returns NPERR_NO_ERROR.  We interpret this to the
1247         mean the variable has been set.  The variable has not been set 
1248         and will consequently be uninitialized.
1249
1250         Reviewed by NOBODY (OOPS!).
1251
1252         * Plugins.subproj/WebBaseNetscapePluginView.m:
1253         (-[WebBaseNetscapePluginView pluginScriptableObject]):
1254
1255 2004-07-19  Ken Kocienda  <kocienda@apple.com>
1256
1257         Reviewed by Richard
1258
1259         Fix for this bug:
1260         
1261         <rdar://problem/3707504> HTMLCompose: key events are stolen by Web(HTML)View
1262
1263         * Misc.subproj/WebNSViewExtras.h:
1264         * Misc.subproj/WebNSViewExtras.m:
1265         (-[NSView firstResponderIsSelfOrDescendantView]): New helper to 
1266         (-[WebHTMLView performKeyEquivalent:]): Do not pass key events through to the editing
1267         key handler unless the WebHTMLView is first responder or contains the first responder.
1268         This prevents the "stealing" of key events mentioned in the bug.
1269         * WebView.subproj/WebView.m:
1270         (-[WebView _performResponderOperation:with:]): Uses the new firstResponderIsSelfOrDescendantView
1271         helper. The code I replaced used the same logic as the new helper.
1272
1273 === Safari-152 ===
1274
1275 2004-07-12  Maciej Stachowiak  <mjs@apple.com>
1276
1277         Reviewed by Kevin.
1278
1279         <rdar://problem/3673988>: (REGRESSION(141-144): connection assertion failure at http://traffic.511.org/sfgate)
1280         
1281         * WebView.subproj/WebBaseResourceHandleDelegate.m:
1282         (-[WebBaseResourceHandleDelegate connectionDidFinishLoading:]):
1283         Don't check connection consistency if this load got cancelled
1284         while finishing, because in this case we have cleared the
1285         connection field already. This can happen when a redirect fires
1286         from an onload handler.
1287
1288 2004-07-12  Maciej Stachowiak  <mjs@apple.com>
1289
1290         Reviewed by John.
1291
1292         <rdar://problem/3632570>: (REGRESSION: SocialText.net wiki keeps asking for auth, even though pages load)
1293
1294         * Misc.subproj/WebIconLoader.m:
1295         (-[WebIconLoader didReceiveAuthenticationChallenge:]): Ignore the challenge - we don't
1296         want an auth panel for favicons.
1297         (-[WebIconLoader didCancelAuthenticationChallenge:]): Ignore cancel, since we are ignoring
1298         the challenge.
1299
1300 2004-07-09  Chris Blumenberg  <cblu@apple.com>
1301         
1302         Allowed my change for 3715785 to compile on Jaguar.
1303
1304         Reviewed by kocienda.
1305
1306         * WebCoreSupport.subproj/WebBridge.m:
1307         (-[WebBridge canTargetLoadInFrame:]): don't use the DOM API since it doesn't exist on Jaguar, instead call the new domain method on the bridge
1308
1309 2004-07-09  Ken Kocienda  <kocienda@apple.com>
1310
1311         Reviewed by John
1312
1313         * Plugins.subproj/npruntime.h: Updated license to lawyer-approved 
1314         joint Apple-Mozilla BSD-style license.
1315
1316 2004-07-08  Chris Blumenberg  <cblu@apple.com>
1317
1318         Fixed: <rdar://problem/3720728> REGRESSION (125.8-146): Crash moving mouse over plugin at manray-photo.com
1319         
1320         Reviewed by john.
1321
1322         * Plugins.subproj/WebBaseNetscapePluginView.m:
1323         (-[WebBaseNetscapePluginView pluginScriptableObject]): don't call NPP_GetValue unless the plug-in implements it
1324
1325 2004-07-08  Chris Blumenberg  <cblu@apple.com>
1326
1327         Fixed: <rdar://problem/3706296> VIP: ifilm.com crashing reproducibly with Safari
1328
1329         Reviewed by kocienda.
1330
1331         * Plugins.subproj/WebNetscapePluginPackage.m:
1332         (-[WebNetscapePluginPackage launchRealPlayer]): don't release a NULL appURL
1333
1334 2004-07-08  Chris Blumenberg  <cblu@apple.com>
1335
1336         Fixed: <rdar://problem/3650140> reproducible assertion failure going to plugin page with JavaScript disabled
1337
1338         Reviewed by john.
1339
1340         * Plugins.subproj/WebBaseNetscapePluginView.m:
1341         (-[WebBaseNetscapePluginView evaluateJavaScriptPluginRequest:]): don't call NPP_NewStream and other stream methods if there is no JS result to deliver. This is what Mozilla does.
1342         (-[WebBaseNetscapePluginView loadRequest:inTarget:withNotifyData:]): Return NPERR_GENERIC_ERROR if JS is disabled. This is what Mozilla does.
1343
1344 === Safari-151 ===
1345
1346 2004-07-07  Trey Matteson  <trey@apple.com>
1347
1348         3719051 - Safari doesn't update form inputs when a page was refreshed by javascript window.location
1349         ... and at least 5 other cases in Radar
1350
1351         Very similar problem to the Harvard PIN bug.  We need to be sure
1352         to not carry any state over when we are processing a client
1353         redirect, which reuses the same WebHistoryItem.
1354
1355         Reviewed by John.
1356
1357         * WebView.subproj/WebFrame.m:
1358         (-[WebFrame _transitionToCommitted:]):  Comment
1359         (-[WebFrame _opened]):  Clear form and scroll state on client redirect.
1360
1361 2004-07-06  Chris Blumenberg  <cblu@apple.com>
1362
1363         Fixed: <rdar://problem/3715785> multiple frame injection vulnerability reported by Secunia, affects almost all browsers
1364         
1365         Reviewed by john, trey, kocienda.
1366
1367         * WebCoreSupport.subproj/WebBridge.m:
1368         (-[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
1369         (-[WebBridge loadURL:referrer:reload:userGesture:target:triggeringEvent:form:formValues:]): call canTargetLoadInFrame: to make sure we can load the request
1370         (-[WebBridge postWithURL:referrer:target:data:contentType:triggeringEvent:form:formValues:]): ditto
1371
1372 2004-07-06  John Sullivan  <sullivan@apple.com>
1373
1374         Reviewed by Trey.
1375
1376         - fixed <rdar://problem/3717147> folder icon used for error page in back/forward 
1377         menu when iTunes is not installed
1378
1379         * WebView.subproj/WebFrame.m:
1380         (-[WebFrame _transitionToCommitted:]):
1381         When displaying the error page for an unreachable URL, leave the requested URL
1382         in the WebHistoryItem; don't clobber it with a bogus one that represents the
1383         error page. This not only avoids the wrong-icon problem, but should also help
1384         in cases where an unreachable URL becomes reachable later on.
1385
1386 2004-07-06  Trey Matteson  <trey@apple.com>
1387
1388         3716053 - www.theage.com.au has extra back/forward items due to ads
1389
1390         The real change was in WebKit.  Here was are just renaming a method and folding
1391         all the WebFrameLoadTypeOnLoadEvent uses to be WebFrameLoadTypeInternal, since there
1392         was never any difference anyway.
1393
1394         Reviewed by Richard
1395
1396         * WebCoreSupport.subproj/WebBridge.m:
1397         (-[WebBridge loadURL:referrer:reload:userGesture:target:triggeringEvent:form:formValues:]):  rename part of the method.
1398         * WebView.subproj/WebFrame.m:  Nuke WebFrameLoadTypeOnLoadEvent.
1399         (-[WebFrame _transitionToCommitted:]):
1400         (-[WebFrame _checkLoadCompleteForThisFrame]):
1401         (-[WebFrame _loadItem:withLoadType:]):
1402         (-[WebFrame _itemForRestoringDocState]):
1403         * WebView.subproj/WebFramePrivate.h:
1404
1405 2004-07-06  Ken Kocienda  <kocienda@apple.com>
1406
1407         Reviewed by Trey
1408         
1409         Only register the editing delegate for those notifications for which it implements the callbacks.
1410
1411         * WebView.subproj/WebView.m:
1412         (-[WebView registerForEditingDelegateNotification:selector:]):
1413         (-[WebView setEditingDelegate:]):
1414
1415 2004-07-06  Trey Matteson  <trey@apple.com>
1416
1417         3294652 - Failed drag of links doesn't slide back
1418
1419         The only reason for this is that because of some hacks, we lie to
1420         AK about the drag image offset, which means we slide back to
1421         slightly the wrong place.  But it's very minor, so we should
1422         just fix it.
1423
1424         Reviewed by Ken.
1425
1426         * WebView.subproj/WebHTMLView.m:
1427         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
1428         Ask for slideback.
1429
1430 2004-07-02  Darin Adler  <darin@apple.com>
1431
1432         Reviewed by Dave.
1433
1434         - fixed problem where tabsToLinks and privateBrowsingEnabled did not work with non-
1435           standard WebPreferences objects
1436
1437         * WebView.subproj/WebPreferences.m:
1438         (-[WebPreferences _valueForKey:]): New helper.
1439         (-[WebPreferences _stringValueForKey:]): Use _valueForKey.
1440         (-[WebPreferences _integerValueForKey:]): Use _valueForKey.
1441         (-[WebPreferences _boolValueForKey:]): Use _valueForKey.
1442         (-[WebPreferences tabsToLinks]): Use _boolValueForKey; this is the bug fix.
1443         (-[WebPreferences privateBrowsingEnabled]): Ditto.
1444         (+[WebPreferences _setIBCreatorID:]): Use copy instead of retain for keeping an NSString.
1445
1446 2004-07-01  Trey Matteson  <trey@apple.com>
1447
1448         3556159 - Crashes in -[WebFrame(WebPrivate) _transitionToCommitted:] at www.mastercardbusiness.com
1449
1450         We know from the line number of the crash that it is due to [self parentFrame]==nil.
1451
1452         Looking at the HTML and that of the related bugs, they do special stuff with onload
1453         handlers.  It is no longer repro, presumably because the includes JS files changed,
1454         as the bugs only included the top level HTML.  I suspect that the problem is that the
1455         WebFrameLoadTypeOnLoadEvent case was added, and in some weird sequence specific to
1456         MasterCard, they hit a case where we would be in WebFrameLoadTypeOnLoadEvent mode but
1457         not have a parent frame.
1458
1459         So we guard in the code against hitting a nil parentFrame, and log an error just in
1460         case this ever crops up again and we can learn more about it.
1461
1462         Reviewed by Richard.
1463
1464         * WebView.subproj/WebFrame.m:
1465         (-[WebFrame _transitionToCommitted:]):  Guard against nil parentFrame.
1466
1467 2004-07-01  John Sullivan  <sullivan@apple.com>
1468
1469         Reviewed by Trey.
1470         
1471         - fixed these bugs:
1472         <rdar://problem/3709110> REGRESSION (Tiger): Pressing Tab key to move focus 
1473         onto links skips every other link
1474         <rdar://problem/3692576> focus ring is in odd place after clicking RSS button 
1475         with "Tab to links" enabled
1476         
1477         WebHTMLView has some trickery by which we advance the focused link when nextKeyView
1478         or previousKeyView is called within nextValidKeyView or previousValidKeyView. This
1479         broke in Tiger because AppKit now (sometimes at least) calls nextKeyView more than
1480         once within nextValidKeyView. Fixed 3709110 by making sure we only advance the focus 
1481         once within a call to nextValidKeyView or previousValidKeyView.
1482         
1483         Also, this same trickery didn't work right with hidden views. Fixed 3692576 by checking
1484         whether the view is hidden and bypassing the focus-moving trickery in that case.
1485
1486         * WebView.subproj/WebHTMLViewInternal.h:
1487         renamed inNextValidKeyView -> nextKeyViewAccessShouldMoveFocus
1488         
1489         * WebView.subproj/WebHTMLView.m:
1490         (-[WebHTMLView nextKeyView]):
1491         now clears nextKeyViewAccessShouldMoveFocus
1492         (-[WebHTMLView previousKeyView]):
1493         ditto
1494         (-[WebHTMLView nextValidKeyView]):
1495         now doesn't set focus-moving trigger ivar if view is hidden or has hidden ancestor
1496         (-[WebHTMLView previousValidKeyView]):
1497         ditto
1498
1499 2004-06-30  Trey Matteson  <trey@apple.com>
1500
1501         Dragging within a web view should be allowed to start when the window isn't key.
1502
1503         A few months ago, Chris made this work, but it relied on the fact that all dragging
1504         was done in WebKit.  When WebCore got involved in dragging, it was broken.  Now we
1505         have a new scheme that gets it working again that properly involves WebCore.
1506
1507         The general idea is that when AK asks us whether to accept the first mouse and do
1508         "delayed window ordering", we must consult WC to see if we might start a drag.  In
1509         addition, instead of these drags in non-active windows being started as a special
1510         case in WK, they go through the normal WK-WC drag machinery.  Finally to work in
1511         frames we have to drill to the deepest hit view in acceptsFirstMouse, because previous
1512         hacks to hitTest make the top-most WebHTMLView field all events for its view tree
1513         (which leads to it fielding all acceptFirstMouse messages too).
1514
1515         Reviewed by John.
1516
1517         * WebCoreSupport.subproj/WebBridge.m:
1518         (-[WebBridge mayStartDragAtEventLocation:]):  Glue change for new arg type.
1519         * WebView.subproj/WebHTMLView.m:
1520         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
1521         firstMouseDownEvent ivar is no longer needed.
1522         (-[WebHTMLView _mayStartDragAtEventLocation:]):  Receives a location instead of a drag
1523         event, since we need to do this work when we have no drag event.  This means the
1524         check of the delay for text dragging is moved down to WebCore.
1525         (-[WebHTMLView acceptsFirstMouse:]):  Respond based on whether we might do a drag.
1526         This includes drilling to the deepest view the event hits, whereas we used to only
1527         respond considering the topmost WebHTMLView.
1528         (-[WebHTMLView shouldDelayWindowOrderingForEvent:]):  Ditto.
1529         (-[WebHTMLView mouseDown:]):  Get rid of special case where some activating
1530         mouseDown events weren't sent to WC.  We need to go through the whole pipeline
1531         now to get a drag started properly.
1532         (-[WebHTMLView mouseDragged:]):  Ditto, let WC start the drag.
1533         (-[WebHTMLView mouseUp:]):  firstMouseDownEvent ivar is no longer needed.
1534         * WebView.subproj/WebHTMLViewInternal.h:
1535         * WebView.subproj/WebHTMLViewPrivate.h:
1536
1537 2004-06-25  Trey Matteson  <trey@apple.com>
1538
1539         Added new utility method.
1540
1541         Reviewed by John.
1542
1543         * WebView.subproj/WebFrame.m:
1544         (-[WebFrame _isDescendantOfFrame:]):   New code.
1545         * WebView.subproj/WebFramePrivate.h:
1546
1547 2004-06-24  Richard Williamson   <rjw@apple.com>
1548
1549                 Fixed <rdar://problem/3710313>: CGContext not zeroed when WebImageRenderer is copied
1550                 The context ivar of WebImageRenderer wasn't being nil when the object was copied.
1551
1552         Reviewed by Darin.
1553
1554         * WebCoreSupport.subproj/WebImageRenderer.m:
1555         (-[WebImageRenderer copyWithZone:]):
1556         (-[WebImageRenderer dealloc]):
1557         (-[WebImageRenderer drawImageInRect:fromRect:compositeOperator:context:]):
1558
1559 2004-06-24  Trey Matteson  <trey@apple.com>
1560
1561         3672725 - Assertion failure in URLCompletion code with particular set of bookmarks
1562
1563         Problem was caused by a URL with unescaped unicodes getting into
1564         the Bookmarks file, presumably from import from IE.  We now test
1565         for this case and convert the data on the way in as if it were
1566         user-entered.
1567
1568         Reviewed by John and Darin.
1569
1570         * History.subproj/WebHistoryItem.m:
1571         (-[WebHistoryItem initFromDictionaryRepresentation:]):
1572
1573 2004-06-24  Trey Matteson  <trey@apple.com>
1574
1575         3704950 drag image in DB ConfigBar has horizontal graphics turd WebCore JavaScript
1576
1577         When we generate a drag image (or a selection image too, for that matter) we
1578         translate the CTM using a CG call.  Later, WebImageRenderer adjusts the pattern
1579         phase based on the CTM of the focused view, which doesn't include our translate.
1580         So we must inform WebKit about the additional phase adjustment.
1581
1582         Reviewed by Richard
1583
1584         * WebCoreSupport.subproj/WebGraphicsBridge.h:
1585         * WebCoreSupport.subproj/WebGraphicsBridge.m:
1586         (-[WebGraphicsBridge setAdditionalPatternPhase:]):  New trivial setter.
1587         (-[WebGraphicsBridge additionalPatternPhase]):      ...and getter.
1588         * WebCoreSupport.subproj/WebImageRenderer.m:
1589         (-[WebImageRenderer tileInRect:fromPoint:context:]):  Take any additional phase
1590         adjustment into account when setting phase.
1591
1592 2004-06-24  Trey Matteson  <trey@apple.com>
1593
1594         3693420 - onbeforecut and onbeforepaste need real implementaion
1595
1596         Reviewed by Chris.
1597
1598         * WebView.subproj/WebHTMLView.m:
1599         (-[WebHTMLView validateUserInterfaceItem:]):  Check with WebCore
1600         to see if cut, copy, paste should be enabled.
1601
1602 2004-06-24  John Sullivan  <sullivan@apple.com>
1603
1604         Reviewed by Darin.
1605         
1606         - fixed <rdar://problem/3706792> 8A161: Choosing text encoding for 
1607         error page opens finder window!?
1608
1609         * WebView.subproj/WebFrame.m:
1610         (-[WebFrame _reloadAllowingStaleDataWithOverrideEncoding:]):
1611         take unreachableURL into account
1612
1613 2004-06-23  Richard Williamson   <rjw@apple.com>
1614
1615         Implemented changes for latest npruntime.h.
1616         Made npruntime.h public.
1617                 
1618         Reviewed by Chris.
1619
1620         * Plugins.subproj/WebBaseNetscapePluginView.h:
1621         * Plugins.subproj/WebBaseNetscapePluginView.m:
1622         (-[WebBaseNetscapePluginView pluginScriptableObject]):
1623         (-[WebBaseNetscapePluginView forceRedraw]):
1624         (-[WebBaseNetscapePluginView getVariable:value:]):
1625         * Plugins.subproj/WebBaseNetscapePluginViewPrivate.h:
1626         * Plugins.subproj/WebNetscapePluginPackage.m:
1627         (-[WebNetscapePluginPackage load]):
1628         * Plugins.subproj/npapi.h:
1629         * Plugins.subproj/npapi.m:
1630         (NPN_GetValue):
1631         * Plugins.subproj/npfunctions.h:
1632         * Plugins.subproj/npruntime.h:
1633         * WebKit.pbproj/project.pbxproj:
1634
1635 2004-06-21  Darin Adler  <darin@apple.com>
1636
1637         * English.lproj/StringsNotToBeLocalized.txt: Updated for recent changes.
1638
1639 2004-06-21  Chris Blumenberg  <cblu@apple.com>
1640
1641         Made WebKitErrorDescriptionPlugInCancelledConnection have its own description string
1642         (Not for software update branch)
1643
1644         Reviewed by john.
1645
1646         * English.lproj/Localizable.strings:
1647         * Misc.subproj/WebKitErrors.m:
1648
1649 2004-06-21  Chris Blumenberg  <cblu@apple.com>
1650
1651         Made WebKitErrorDescriptionPlugInCancelledConnection properly reuse an already localized string to avoid loc changes.
1652
1653         Reviewed by john.
1654
1655         * ChangeLog:
1656         * Misc.subproj/WebKitErrors.m:
1657
1658 2004-06-21  Chris Blumenberg  <cblu@apple.com>
1659
1660         Fixed: <rdar://problem/3701269> change in error handling behavior from 10.3.3 to 10.3.4 breaks unreleased Adobe PDF plug-in
1661
1662         Reviewed by john.
1663
1664         * Misc.subproj/WebKitErrors.m:
1665         (registerErrors): register string for WebKitErrorPlugInCancelledConnection
1666         * Misc.subproj/WebKitErrorsPrivate.h:
1667         * Plugins.subproj/WebBaseNetscapePluginStream.h:
1668         * Plugins.subproj/WebBaseNetscapePluginStream.m:
1669         (-[WebBaseNetscapePluginStream startStreamWithURL:expectedContentLength:lastModifiedDate:MIMEType:]): use renamed WEB_REASON_PLUGIN_CANCELLED constant
1670         (-[WebBaseNetscapePluginStream destroyStream]): ditto
1671         * Plugins.subproj/WebNetscapePluginRepresentation.m:
1672         (-[WebNetscapePluginRepresentation cancelWithReason:]): if the reason is WEB_REASON_PLUGIN_CANCELLED, cancel the load with WebKitErrorPlugInCancelledConnection
1673         * Plugins.subproj/WebNetscapePluginStream.m:
1674         (-[WebNetscapePluginStream cancelWithReason:]): ditto
1675
1676 === Safari-146 ===
1677
1678 2004-06-17  Richard Williamson   <rjw@apple.com>
1679
1680                 Changed private headers to be pubic for npapi.h and npfunctions.h.  That
1681                 API was approved but never marked as public in the project file.  This change does
1682                 not make npruntime.h public, which contains the not yet approved changes
1683                 for script-ability of netscape plugins.  Unfortunately, that API will not
1684                 be public for WWDC.
1685                                 
1686         Reviewed by Chris.
1687
1688         * WebKit.pbproj/project.pbxproj:
1689
1690 2004-06-17  Trey Matteson  <trey@apple.com>
1691
1692         3698514 - coordinates in ondragstart and ondrag events are wrong
1693
1694         This part fixes the ondrag coords.  I thought Cocoa passed us the mouse location
1695         in draggedImage:movedTo:, but no, it's the position of the dragged image.
1696         WebCore needs the mouse location, so to calc that we must save away the
1697         offset of the mouse relative to the image when we kick off the drag.
1698
1699         Reviewed by Maciej.
1700
1701         * Misc.subproj/WebNSViewExtras.h:
1702         * Misc.subproj/WebNSViewExtras.m:
1703         (-[NSView _web_dragImage:rect:event:pasteboard:source:offset:]): Add the ability
1704         to return the offset of the cursor wrt to the drag image, since this routine
1705         generates its own drag image and positions it.
1706         * WebView.subproj/WebHTMLView.m:
1707         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
1708         Calc the offset of the cursor wrt to the drag image in the myriad of ways
1709         that we kick off the drag.      
1710         (-[WebHTMLView draggedImage:movedTo:]):  Adjust the location by the
1711         offset we save when we kicked off the drag.
1712         (-[WebHTMLView draggedImage:endedAt:operation:]):  Ditto.
1713         * WebView.subproj/WebHTMLViewInternal.h:
1714         * WebView.subproj/WebImageView.m:
1715         (-[WebImageView mouseDragged:]):  Pass nil for new arg, we don't care.
1716
1717 2004-06-16  David Hyatt  <hyatt@apple.com>
1718
1719         In order to support truncation in Emerson, enhance pointToOffset so that it needn't include partial
1720         character glyphs (the left half of a glyph).
1721         
1722         Reviewed by mjs
1723
1724         * WebCoreSupport.subproj/WebTextRenderer.m:
1725         (-[WebTextRenderer pointToOffset:style:position:reversed:includePartialGlyphs:]):
1726         (-[WebTextRenderer _ATSU_pointToOffset:style:position:reversed:includePartialGlyphs:]):
1727         (-[WebTextRenderer _CG_pointToOffset:style:position:reversed:includePartialGlyphs:]):
1728
1729 === Safari-145 ===
1730
1731 2004-06-16  Darin Adler  <darin@apple.com>
1732
1733         - fixed <rdar://problem/3696081> REGRESSION: <WebKit/CarbonUtils.h> doesn't build any more
1734
1735         Reviewed by Trey.
1736
1737         * Carbon.subproj/CarbonUtils.h: Fixed include. Need to include <ApplicationServices/ApplicationServices.h>
1738         to get CGImageRef; can't include an individual header.
1739
1740 2004-06-15  Richard Williamson   <rjw@apple.com>
1741
1742         Fixed <rdar://problem/3695875>: Objective-C instances that are exported to JavaScript are too promiscuous
1743         
1744         Flip the policy for exposing Objective-C methods and properties.
1745         
1746         Reviewed by Trey.
1747         
1748         * Plugins.subproj/WebPluginPackage.m:
1749         (+[NSObject isSelectorExcludedFromWebScript:]):  Just return YES.
1750         (+[NSObject isKeyExcludedFromWebScript:]):  Just return YES.
1751
1752 2004-06-15  Trey Matteson  <trey@apple.com>
1753
1754         3639321 - Harvard PIN authentication ends up sending PIN as clear text with POST action
1755
1756         When going back/forward to an item, if we went there originally via a POST, 
1757         we ask the user about rePOSTing, and if they say yes, we resend the POST. 
1758         This rePOST case is triggered by the form data that we saved on the b/f item.
1759         In the case of this bug, the overall navigation was accomplished by a POST,
1760         then a redirect, causing a GET.  When a load of type redirect achieves the Committed
1761         stage, we replace the current URL in the b/f item with the new URL (instead of
1762         adding a new item to the b/f list).
1763
1764         The bug is that at the same time we should also update the form data in
1765         the b/f item to match that of the new request.  I think this will normally
1766         mean nil'ing it out, unless there's some way for the result of the redirect
1767         to be another POST.
1768  
1769         The security leak occurred because we did not clear the form data on the item,
1770         so when going back or forward to the page, we would go into the rePOSTing code,
1771         even though we eventually reached that page via a GET (caused by the redirect).
1772         So we would do a POST to the redirect URL containing the private data sent in
1773         the original POST.
1774
1775         Reviewed by mjs and rjw.
1776
1777         * History.subproj/WebHistoryItem.m:
1778         (-[WebHistoryItem _setFormInfoFromRequest:]):  New method, just wraps 3 old set methods.
1779         (-[WebHistoryItem formData]):  Diff being dumb, no change.
1780         (-[WebHistoryItem formContentType]):  Ditto
1781         (-[WebHistoryItem formReferrer]): Ditto
1782         * History.subproj/WebHistoryItemPrivate.h:
1783         * WebView.subproj/WebFrame.m:
1784         (-[WebFrame _createItem:]):  Call new WebHistoryItem method - no change in real behavior
1785         (-[WebFrame _transitionToCommitted:]):  Clear out the form data at the key time, to fix the bug.
1786
1787 2004-06-15  Chris Blumenberg  <cblu@apple.com>
1788
1789         Fixed: <rdar://problem/3695724> WebKit plug-ins should only have to implement plugInViewWithArguments:
1790
1791         Reviewed by rjw.
1792
1793         * ChangeLog:
1794         * Plugins.subproj/WebPluginViewFactory.h: mention that plugInViewWithArguments is required
1795         * WebView.subproj/WebFrame.m:
1796         (-[WebFrame _reloadForPluginChanges]): check for plugInViewWithArguments: not webPlugInInitialize
1797         * WebView.subproj/WebHTMLView.m:
1798         (-[WebHTMLView addSubview:]): ditto
1799
1800 2004-06-15  Chris Blumenberg  <cblu@apple.com>
1801
1802         Reviewed by NOBODY (OOPS!).
1803
1804         * WebView.subproj/WebFrame.m:
1805         (-[WebFrame _reloadForPluginChanges]):
1806         * WebView.subproj/WebHTMLView.m:
1807         (-[WebHTMLView addSubview:]):
1808
1809 2004-06-15  Trey Matteson  <trey@apple.com>
1810
1811         3695240 - pasting plain text with newlines in it turns them into spaces
1812
1813         Easy fix, we just need to consume the incoming data as text instead of markup.
1814
1815         Reviewed by John and Darin.
1816
1817         * WebView.subproj/WebHTMLView.m:
1818         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]):
1819
1820 2004-06-15  Darin Adler  <darin@apple.com>
1821
1822         - fixed crash introduced by my earlier change
1823
1824         * WebCoreSupport.subproj/WebImageRenderer.m:
1825         (-[WebImageContext dealloc]): Set _cgsContext to 0 before calling super as before.
1826         (-[WebImageContext finalize]): Ditto.
1827
1828 2004-06-14  Darin Adler  <darin@apple.com>
1829
1830         Reviewed by Maciej.
1831
1832         - fixed some things for GC that Patrick missed, or that happened after the branch
1833
1834         * Carbon.subproj/HIWebView.m:
1835         (HIWebViewConstructor): Use CFRetain instead of retain.
1836         (HIWebViewDestructor): Use CFRelease instead of release.
1837         * Misc.subproj/WebFileDatabase.m:
1838         (-[WebFileDatabase initWithPath:]): Use release instead of dealloc.
1839         * WebCoreSupport.subproj/WebImageRenderer.m:
1840         (-[WebImageContext finalize]): Had [super dealloc] here by mistake; change to [super finalize].
1841         * WebCoreSupport.subproj/WebTextRenderer.m:
1842         (-[WebTextRenderer finalize]): Added.
1843         * WebView.subproj/WebDebugDOMNode.m:
1844         (-[WebDebugDOMNode initWithWebFrameView:]): Use release instead of dealloc.
1845         * WebView.subproj/WebRenderNode.m:
1846         (-[WebRenderNode initWithWebFrameView:]): Use release instead of dealloc.
1847
1848 2004-06-15  Trey Matteson  <trey@apple.com>
1849
1850         Fix ASSERT/crash we get sometimes when dragging link images.  Turns
1851         out there was an uninitialized variable for the image size,
1852         so at random we would try to create huge images that could not
1853         be focused.
1854         
1855         Reviewed by Maciej and Darin.
1856
1857         * WebView.subproj/WebHTMLView.m:
1858         (-[WebHTMLView _dragImageForLinkElement:]):  Trivial fix to
1859         not use += with uninited variable.
1860
1861 2004-06-14  Darin Adler  <darin@apple.com>
1862
1863         Reviewed by me, code changes by Patrick Beard.
1864
1865         - fixed <rdar://problem/3671507>: (WebKit should adopt GC changes and compile with GC enabled)
1866
1867         * WebKit.pbproj/project.pbxproj: Added WebNSObjectExtras.h.
1868         * Misc.subproj/WebNSObjectExtras.h: Added. Includes WebMakeCollectable, a cover for
1869         CFMakeCollectable that returns type id, for less casting, and works on Panther as well
1870         as Tiger. Also declares finalize in NSObject so we can call super without warnings on Panther.
1871
1872         * Carbon.subproj/CarbonWindowAdapter.m: Fixed header and includes a bit.
1873         (-[CarbonWindowAdapter finalize]): Added.
1874         * History.subproj/WebBackForwardList.m:
1875         (-[WebBackForwardList dealloc]): Got count outside loop.
1876         (-[WebBackForwardList finalize]): Added.
1877         * History.subproj/WebHistoryItem.m:
1878         (-[WebHistoryItem finalize]): Added.
1879         * Misc.subproj/WebFileDatabase.m:
1880         (-[WebFileDatabase _createLRUList:]): Use release on Panther, drain on Tiger.
1881         (+[WebFileDatabase _syncLoop:]): Ditto.
1882         (-[WebFileDatabase dealloc]): Removed, since we never deallocate objects of this class, and
1883         the method was untested.
1884         * Misc.subproj/WebKitErrors.m: (registerErrors): Use release on Panther, drain on Tiger.
1885         * Misc.subproj/WebKitNSStringExtras.m: (+[NSString _web_encodingForResource:]): Use
1886         WebMakeCollectable. Also fixed indenting.
1887         * Misc.subproj/WebNSURLExtras.m:
1888         (+[NSURL _web_URLWithData:relativeToURL:]): Use WebMakeCollectable.
1889         (-[NSURL _web_URLWithLowercasedScheme]): Use WebMakeCollectable.
1890         (-[NSString _web_mapHostNameWithRange:encode:makeString:]): Change code to use CFRelease
1891         to balance CFURLCreateStringByReplacingPercentEscapes CFString creation, not release.
1892         * Plugins.subproj/WebBaseNetscapePluginStream.m:
1893         (-[WebBaseNetscapePluginStream finalize]): Added.
1894         * Plugins.subproj/WebBaseNetscapePluginView.m:
1895         (-[WebBaseNetscapePluginView freeAttributeKeysAndValues]): Added. Shared code for dealloc and finalize.
1896         (-[WebBaseNetscapePluginView dealloc]): Call freeAttributeKeysAndValues.
1897         (-[WebBaseNetscapePluginView finalize]): Added.
1898         (-[WebBaseNetscapePluginView requestWithURLCString:]): Changed so that CFString objects are
1899         released with CFRelease, not release.
1900         (-[WebBaseNetscapePluginView loadRequest:inTarget:withNotifyData:]): Fixed storage leak in
1901         error case. Made sure CFString object is released with CFRelease, not release.
1902         (-[WebBaseNetscapePluginView _postURLNotify:target:len:buf:file:notifyData:allowHeaders:]):
1903         Another CFString that needed to a CFRelease, not a release.
1904         (-[WebBaseNetscapePluginView status:]): Ditto.
1905         * Plugins.subproj/WebBasePluginPackage.m:
1906         (-[WebBasePluginPackage pathByResolvingSymlinksAndAliasesInPath:]): Changed code to
1907         CFRelease a CFURLRef, instead of release.
1908         (-[WebBasePluginPackage finalize]): Added.
1909         * WebCoreSupport.subproj/WebBridge.m:
1910         (-[WebBridge fini]): Added. Shared code for dealloc and finalize.
1911         (-[WebBridge dealloc]): Call fini.
1912         (-[WebBridge finalize]): Added.
1913         * WebCoreSupport.subproj/WebGlyphBuffer.m:
1914         (-[WebGlyphBuffer finalize]): Added.
1915         * WebCoreSupport.subproj/WebImageRenderer.m:
1916         (-[WebImageContext initWithBounds:context:]): Use CGContextRetain to avoid cast.
1917         (-[WebImageContext dealloc]): Tweaked.
1918         (-[WebImageContext finalize]): Added.
1919         (-[WebImageRenderer finalize]): Added.
1920         (-[WebPDFDocument finalize]): Added.
1921         * WebView.subproj/WebDataProtocol.m:
1922         (+[NSURL _web_uniqueWebDataURL]): Changed so that CFString object is released with
1923         CFRelease, not release.
1924         * WebView.subproj/WebDataSource.m:
1925         (-[WebDataSourcePrivate dealloc]): Took non-release work out of here.
1926         (-[WebDataSource dealloc]): Moved it here.
1927         (-[WebDataSource finalize]): Added.
1928         (-[WebDataSource isLoading]): Use release on Panther, drain on Tiger.
1929         * WebView.subproj/WebFrame.m:
1930         (-[WebFramePrivate dealloc]): Took non-release work out of here.
1931         (-[WebFrame dealloc]): Moved it here.
1932         (-[WebFrame finalize]): Added.
1933         * WebView.subproj/WebFrameView.m:
1934         (-[WebFrameView finalize]): Added.
1935         * WebView.subproj/WebHTMLRepresentation.m:
1936         (-[WebHTMLRepresentation finalize]): Added.
1937         * WebView.subproj/WebHTMLView.m:
1938         (-[WebHTMLViewPrivate dealloc]): Moved non-release work out of here.
1939         (-[WebHTMLView dealloc]): Moved it here.
1940         (-[WebHTMLView finalize]): Added.
1941         * WebView.subproj/WebImageView.m:
1942         (-[WebImageView finalize]): Added.
1943         * WebView.subproj/WebMainResourceClient.m:
1944         (-[WebMainResourceClient finalize]): Added.
1945         * WebView.subproj/WebTextView.m:
1946         (-[WebTextView finalize]): Added.
1947         * WebView.subproj/WebView.m:
1948         (-[WebView finalize]): Added.
1949
1950 2004-06-14  Ken Kocienda  <kocienda@apple.com>
1951
1952         Reviewed by Darin
1953         
1954         Fix for this bug:
1955         
1956         <rdar://problem/3693202> WebView.h header unnecessarily lists NSResponder methods it overrides
1957
1958         * WebView.subproj/WebView.h: NSResponder overrides now gathered in a comment section.
1959
1960 2004-06-13  Trey Matteson  <trey@apple.com>
1961
1962         Support for DHTML cut/copy/paste.
1963
1964         Reviewed by Chris
1965
1966         * WebView.subproj/WebHTMLView.m:
1967         (-[WebHTMLView copy:]):  Give DHTML first crack at executing the command.
1968         (-[WebHTMLView cut:]):  Ditto
1969         (-[WebHTMLView paste:]):  Ditto
1970
1971 2004-06-14  Chris Blumenberg  <cblu@apple.com>
1972
1973         Fixed: <rdar://problem/3674456> make new drag & drop API compatible with DHTML dragging
1974
1975         Reviewed by trey.
1976
1977         * WebView.subproj/WebHTMLView.m:
1978         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): tweak
1979         * WebView.subproj/WebUIDelegate.h: removed old, added new methods
1980         * WebView.subproj/WebUIDelegatePrivate.h: moved new methods to public header
1981         * WebView.subproj/WebView.h: removed old, added new methods
1982         * WebView.subproj/WebView.m: moved methods around
1983         (-[WebView moveDragCaretToPoint:]): moved
1984         (-[WebView removeDragCaret]): moved
1985         (-[WebView _bridgeAtPoint:]): moved
1986         (-[WebView editableDOMRangeForPoint:]): moved
1987         * WebView.subproj/WebViewInternal.h: add new internal method
1988         * WebView.subproj/WebViewPrivate.h: moved new methods to public header
1989
1990 2004-06-14  Trey Matteson  <trey@apple.com>
1991
1992         Use a different hack, as recommended by Kristin, to force the drag
1993         manager to exit a modal event wait it is in.  The hack is required
1994         to update the drag image on the fly.  Instead of posting a CG
1995         event we post at the AppKit level.
1996
1997         Reviewed by Louch
1998
1999         * WebCoreSupport.subproj/WebGraphicsBridge.m:
2000         (-[WebGraphicsBridge setDraggingImage:at:]):
2001
2002 2004-06-14  Chris Blumenberg  <cblu@apple.com>
2003
2004         Fixed:
2005         <rdar://problem/3692668> REGRESSION: delay when images and links
2006         <rdar://problem/3692675> links should drag when dragging images that are links when drag source action is WebDragSourceActionLink
2007
2008         Reviewed by trey.
2009
2010         * WebView.subproj/WebHTMLView.m:
2011         (-[WebHTMLView _mayStartDragWithMouseDragged:]): cleaned up, only use selection delay for selections, allow links to be dragged when image dragging is disabled 
2012
2013 2004-06-12  Trey Matteson  <trey@apple.com>
2014
2015         Small rearrangement to support dynamic changing of the drag
2016         image during DHTML dragging.
2017
2018         Reviewed by John
2019
2020         * WebCoreSupport.subproj/WebBridge.m:  Bridge glue moved to WebGraphicsBridge.
2021         * WebCoreSupport.subproj/WebGraphicsBridge.m:
2022         (FlipImageSpec):  Code moved from WebHTMLView.m.
2023         (-[WebGraphicsBridge setDraggingImage:at:]):  Ditto.  Plus, we've added a gross
2024         event posting hack to force CG drag manager to update the display when we set
2025         the drag image.
2026         * WebView.subproj/WebHTMLView.m:  Code moved to WebGraphicsBridge.
2027         * WebView.subproj/WebHTMLViewPrivate.h:
2028
2029 2004-06-11  Chris Blumenberg  <cblu@apple.com>
2030
2031         Implemented the remainder of the drag & drop API.
2032
2033         Reviewed by trey.
2034
2035         * Misc.subproj/WebNSPasteboardExtras.h:
2036         * Misc.subproj/WebNSPasteboardExtras.m:
2037         (-[NSPasteboard _web_declareAndWriteDragImage:URL:title:archive:source:]): new
2038         * Misc.subproj/WebNSViewExtras.h:
2039         * Misc.subproj/WebNSViewExtras.m:
2040         (-[NSView _web_dragImage:rect:event:pasteboard:source:]): simplified, this method now just creates a drag image and starts the drag
2041         * WebCoreSupport.subproj/WebBridge.m:
2042         (-[WebBridge allowDHTMLDrag:UADrag:]): now calls _delegateDragSourceActionMask on WebHTMLView to interact with the delegate
2043         * WebView.subproj/WebDefaultUIDelegate.m:
2044         * WebView.subproj/WebHTMLView.m:
2045         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]): call new delegate method, cleaned-up a little
2046         (-[WebHTMLView draggingUpdatedWithDraggingInfo:actionMask:]): call drag caret methods on WebView instead of WebBridge so WebView can make sure only 1 HTML view has a drag cursor
2047         (-[WebHTMLView draggingCancelledWithDraggingInfo:]): ditto
2048         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): ditto
2049         (-[WebHTMLView _delegateDragSourceActionMask]): new, gets drag source action mask from delegate
2050         * WebView.subproj/WebHTMLViewInternal.h:
2051         * WebView.subproj/WebImageView.h:
2052         * WebView.subproj/WebImageView.m:
2053         (-[WebImageView mouseDown:]): get the drag source action mask from the delegate
2054         (-[WebImageView mouseDragged:]): inform the delegate of the drag
2055         * WebView.subproj/WebView.m:
2056         (-[WebViewPrivate dealloc]): assert that dragCursorBridge is nil
2057         (-[WebView _close]): release dragCursorBridge
2058         (-[WebView _bridgeAtPoint:]): new
2059         (-[WebView editableDOMRangeForPoint:]): new API
2060         (-[WebView moveDragCaretToPoint:]): new API
2061         (-[WebView removeDragCaret]): new API
2062         (-[WebView _frameViewAtWindowPoint:]): moved so this can be called internally
2063         * WebView.subproj/WebViewInternal.h:
2064         * WebView.subproj/WebViewPrivate.h:
2065
2066 2004-07-10  Trey Matteson  <trey@apple.com>
2067
2068         Prep work for latest delegate API for dragging.  In addition, I also straightened out all
2069         the cases of DHTML setting a drag image or setting pasteboard data, and how that would
2070         override WebKit's default behavior (which follows how WinIE does things).
2071
2072         Reviewed by Chris.
2073
2074         * Misc.subproj/WebNSViewExtras.h:
2075         * Misc.subproj/WebNSViewExtras.m:
2076         (-[NSView _web_dragImage:archive:rect:URL:title:event:dragImage:dragLocation:writePasteboard:]):
2077         New args to allow WebCore override of dragImage and pasteboard data.
2078         * WebCoreSupport.subproj/WebBridge.m:
2079         (-[WebBridge allowDHTMLDrag:UADrag:]):  New method to return the drag action info to WC.
2080         (-[WebBridge startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
2081         Pass along new args.
2082         * WebView.subproj/WebHTMLView.m:
2083         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
2084         Allow WebCore to override drag image and pasteboard data for any type of drag.
2085         (-[WebHTMLView mouseDragged:]):  Pass NO for new args.
2086         * WebView.subproj/WebHTMLViewPrivate.h:
2087         * WebView.subproj/WebImageView.m:
2088         (-[WebImageView mouseDragged:]):  Pass NO/nil for new args.
2089
2090 === Safari-144 ===
2091
2092 2004-06-10  Kevin Decker  <kdecker@apple.com>
2093
2094         Reviewed by John.
2095
2096         * WebView.subproj/WebResource.m:
2097         (-[WebResource description]):
2098             -added per request of cblu
2099         (-[WebResource _response]):
2100             -added this method to the header
2101         (-[WebResource _stringValue]):
2102             - gives the string value of the NSData representation
2103         * WebView.subproj/WebResourcePrivate.h:
2104
2105 2004-06-10  Darin Adler  <darin@apple.com>
2106
2107         Reviewed by Ken.
2108
2109         * WebCoreSupport.subproj/WebBridge.m: Removed undo-related methods.
2110         (-[WebBridge undoManager]): Added.
2111
2112         * WebKit.pbproj/.cvsignore: Updated for new Xcode files.
2113
2114 2004-06-09  Ken Kocienda  <kocienda@apple.com>
2115
2116         Reviewed by Darin
2117
2118         Fix for this bug:
2119         
2120         <rdar://problem/3672088>: "Editable WebViews should maintain a selection even when they're not firstResponder"
2121
2122         Add some code to determine whether a WebHTMLView should maintain an
2123         inactive selection when the view is not first responder. Traditionally,
2124         these views have not maintained such selections, clearing them when the
2125         view was not first responder. However, for appls embedding this view as
2126         an editing widget, it is desirable to act more like an NSTextView. For
2127         now, however, the view only acts in this way when the web view is set to
2128         be editable with -[WebView setEditable:YES]. This will maintain
2129         traditional behavior for WebKit clients dating back to before this
2130         change, and will likely be a decent switch for the long term, since
2131         clients to ste the web view to be editable probably want it to act like
2132         a "regular" Cocoa view in terms of its selection behavior.
2133
2134         * WebView.subproj/WebHTMLView.m:
2135         (-[WebHTMLView updateTextBackgroundColor]): Add code to tell whether the view is resigning first responder,
2136         and if it is, use the inactive text background color.
2137         (-[WebHTMLView maintainsInactiveSelection]): New helper which does checks to see if the new selection
2138         behavior should be used, or whether we should continue with traditional WebKit behavior.
2139         (-[WebHTMLView resignFirstResponder]): Call new maintainsInactiveSelection helper. If true,
2140         do not clear the selection.
2141         * WebView.subproj/WebHTMLViewInternal.h: Add resigningFirstResponder flag.
2142
2143 2004-06-09  Chris Blumenberg  <cblu@apple.com>
2144
2145         Implemented drag destination portion of the new drag & drop API.
2146
2147         Reviewed by trey.
2148
2149         * WebView.subproj/WebDefaultUIDelegate.m:
2150         (-[WebDefaultUIDelegate webView:dragDestinationActionMaskForDraggingInfo:]): new delegate implementation
2151         (-[WebDefaultUIDelegate webView:willPerformDragDestinationAction:forDraggingInfo:]): ditto
2152         * WebView.subproj/WebDocumentInternal.h:
2153         * WebView.subproj/WebHTMLView.m:
2154         (-[WebHTMLView draggingUpdatedWithDraggingInfo:actionMask:]): handle the action mask
2155         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): ditto
2156         * WebView.subproj/WebUIDelegate.h:
2157         * WebView.subproj/WebUIDelegatePrivate.h:
2158         * WebView.subproj/WebView.m:
2159         (-[WebView _dragOperationForDraggingInfo:]): call new delegate methods
2160         (-[WebView performDragOperation:]): ditto
2161         * WebView.subproj/WebViewInternal.h:
2162         * WebView.subproj/WebViewPrivate.h:
2163
2164 2004-06-09  Richard Williamson   <rjw@apple.com>
2165
2166         Implemented PDF rendering for the drawImage() function
2167         in Context2D.  This allows PDF files to be drawn in scaled
2168         or rotated context without rasterization artifacts.
2169         
2170         The PDF image is currently NOT cached.  Caching can/will be added
2171         as an optimization.  The hooks are already in place to flush
2172         the cache as necessary.
2173         
2174         Reviewed by John.
2175
2176         * WebCoreSupport.subproj/WebImageRenderer.h:
2177         * WebCoreSupport.subproj/WebImageRenderer.m:
2178         (-[WebImageRenderer dealloc]):
2179         (-[WebImageRenderer _needsRasterFlush]):
2180         (-[WebImageRenderer drawClippedToValidInRect:fromRect:]):
2181         (-[WebImageRenderer _PDFDocumentRef]):
2182         (-[WebImageRenderer _PDFDraw]):
2183         (-[WebImageRenderer _PDFDrawFromRect:toRect:operation:alpha:flipped:]):
2184         (-[WebImageRenderer MIMEType]):
2185         (ReleasePDFDocumentData):
2186         (-[WebPDFDocument initWithData:]):
2187         (-[WebPDFDocument dealloc]):
2188         (-[WebPDFDocument documentRef]):
2189         (-[WebPDFDocument mediaBox]):
2190         (-[WebPDFDocument bounds]):
2191         (-[WebPDFDocument adjustCTM:]):
2192         (-[WebPDFDocument setCurrentPage:]):
2193         (-[WebPDFDocument currentPage]):
2194         (-[WebPDFDocument pageCount]):
2195
2196         Added back check for old plugin API.
2197         
2198         * WebView.subproj/WebFrame.m:
2199         (-[WebFrame _reloadForPluginChanges]):
2200         * WebView.subproj/WebHTMLView.m:
2201         (-[WebHTMLView addSubview:]):
2202         * WebCoreSupport.subproj/WebBridge.m:
2203         (-[WebBridge frameRequiredForMIMEType:URL:]):
2204
2205 2004-06-08  Trey Matteson  <trey@apple.com>
2206
2207         In DHTML dragging there is no notion of registering for types, so we'd
2208         like to just pass all types down to WebCore.  It turns out that the per-type
2209         registration doesn't matter as far as the underlying drag service is
2210         concerned, so Cocoa is already getting called for any type.  We just hack and
2211         override a private method to ensure we accept any type.
2212
2213         Reviewed by Chris.
2214
2215         * WebView.subproj/WebView.m:
2216         (-[WebView _hitTest:dragTypes:]):  We accept any drag type if it is within
2217         our view, without overriding a subview's decision.
2218
2219 2004-06-08  Trey Matteson  <trey@apple.com>
2220
2221         A DHTML drag source can now change the dragging image during the drag.  Currently
2222         it may only be set to a static image.
2223
2224         Reviewed by John
2225
2226         * WebCoreSupport.subproj/WebBridge.m:
2227         (-[WebBridge setDraggingImage:at:]):  Typical bridge glue.
2228         * WebView.subproj/WebHTMLView.m:
2229         (FlipImageSpec):  New utility, copied from AppKit.
2230         (-[WebHTMLView _setDraggingImage:at:]):  Mostly copied from AppKit.  Sets the drag
2231         image using CG API.
2232         * WebView.subproj/WebHTMLViewPrivate.h:
2233
2234 2004-06-08  Chris Blumenberg  <cblu@apple.com>
2235
2236         Fixed: <rdar://problem/3640423>: (REGRESSION: can't paste text copied from web page into Excel (due to HTML on the pasteboard?))
2237
2238         Reviewed by darin.
2239
2240         * WebView.subproj/WebHTMLView.m:
2241         (+[WebHTMLView _selectionPasteboardTypes]): don't include NSHTMLPboardType
2242         (-[WebHTMLView _selectedArchive]): removed markup string parameter
2243         (_selectionPasteboardTypes::if): don't put NSHTMLPboardType on the pasteboard
2244         * WebView.subproj/WebHTMLViewPrivate.h:
2245
2246 2004-06-07  Darin Adler  <darin@apple.com>
2247
2248         Reviewed by Chris.
2249
2250         - fixed <rdar://problem/3676761>: (REGRESSION: Text Bigger/Smaller commands are always disabled in TOT)
2251
2252         * WebView.subproj/WebHTMLView.m: Add @interface declaration for WebTextSizing category
2253         on WebHTMLView. This must have gotten lost somewhere along the way.
2254
2255 2004-06-05  Trey Matteson  <trey@apple.com>
2256
2257         WebKit no longer causes an endless stream of dragexit events to
2258         occur at the DHTML level when hovering over an element that is
2259         not accepting the drag.
2260
2261         Reviewed by Chris
2262
2263         * WebView.subproj/WebView.m:
2264         (-[WebView _dragOperationForDraggingInfo:]):  If the potential target refuses
2265         the item, don't turn around and immediately tell it to cancel the drag, which is
2266         what leads to the exit event happening at the DOM level.  If the target just
2267         refused the drag, it should not have anything it needs to cancel.
2268
2269 2004-06-03  Trey Matteson  <trey@apple.com>
2270
2271         DHTML dragging uses the Cocoa NSDragOperation on both the source and dest ends.
2272         Most of the real work is in WebCore.
2273
2274         Reviewed by rjw
2275
2276         * WebCoreSupport.subproj/WebBridge.m:
2277         (-[WebBridge startDraggingImage:at:operation:event:]):  Pass the drag op along.
2278         * WebView.subproj/WebHTMLView.m:
2279         (-[WebHTMLView _startDraggingImage:at:operation:event:]):  Remember drag op from WC.
2280         (-[WebHTMLView mouseDragged:]):  Pass None for the op (alternate code path that will
2281         be going away).
2282         (-[WebHTMLView draggingSourceOperationMaskForLocal:]):  Use the WC drag op.
2283         * WebView.subproj/WebHTMLViewInternal.h:
2284         * WebView.subproj/WebHTMLViewPrivate.h:
2285
2286 === Safari-143 ===
2287
2288 2004-06-04  Chris Blumenberg  <cblu@apple.com>
2289
2290         Fixed: <rdar://problem/3674921>: (can't drag an image from Desktop to Blot document)
2291
2292         Reviewed by mjs.
2293
2294         * WebView.subproj/WebDataSource.m:
2295         (-[WebDataSource _imageElementWithImageResource:]): factored out from _documentFragmentWithImageResource:
2296         (-[WebDataSource _documentFragmentWithImageResource:]): call _imageElementWithImageResource:
2297         * WebView.subproj/WebDataSourcePrivate.h:
2298         * WebView.subproj/WebHTMLView.m:
2299         (-[WebHTMLView _imageExistsAtPaths:]): new
2300         (-[WebHTMLView _documentFragmentWithPaths:]): new
2301         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): handle NSFilenamesPboardType
2302         (+[WebHTMLView _insertablePasteboardTypes]): include NSFilenamesPboardType
2303         (-[WebHTMLView _canProcessDragWithDraggingInfo:]): check for NSFilenamesPboardType and check that the files are images
2304
2305 2004-06-04  Richard Williamson   <rjw@apple.com>
2306
2307         Fixed crasher from last checkin.
2308         
2309         Reviewed by Ken.
2310
2311         * Plugins.subproj/WebPluginDatabase.m:
2312         (+[WebPluginDatabase setAdditionalWebPlugInPaths:]):
2313         (pluginLocations):
2314
2315 2004-06-03  Richard Williamson   <rjw@apple.com>
2316
2317         Add SPI to allow setting of plugin load path.
2318                 Reviewed by Hyatt.
2319
2320         * Plugins.subproj/WebPluginDatabase.h:
2321         * Plugins.subproj/WebPluginDatabase.m:
2322         (+[WebPluginDatabase setAdditionalWebPlugInPaths:]):
2323         (pluginLocations):
2324
2325 2004-06-03  Darin Adler  <darin@apple.com>
2326
2327         Reviewed by Chris.
2328
2329         - fixed <rdar://problem/3677038>: (Need SPI to get URL of favicon for a site)
2330
2331         * Misc.subproj/WebIconDatabase.h: Add iconURLForURL: method.
2332         * Misc.subproj/WebIconDatabase.m: (-[WebIconDatabase iconURLForURL:]): Added.
2333
2334 2004-06-02  Ken Kocienda  <kocienda@apple.com>
2335
2336         Reviewed by John
2337
2338         Fix for these bugs:
2339         
2340         <rdar://problem/3675806>: "Make API name change for -webViewShouldBeginEditing:inDOMRange:"
2341         <rdar://problem/3675809>: "Make API name change for -webViewShouldEndEditing:inDOMRange:"
2342
2343         New names are -webView:shouldBeginEditingInDOMRange: and -webView:shouldEndEditingInDOMRange:, 
2344         respectively.
2345
2346         * WebView.subproj/WebDefaultEditingDelegate.m:
2347         (-[WebDefaultEditingDelegate webView:shouldBeginEditingInDOMRange:]):
2348         (-[WebDefaultEditingDelegate webView:shouldEndEditingInDOMRange:]):
2349         * WebView.subproj/WebEditingDelegate.h:
2350         * WebView.subproj/WebView.m:
2351         (-[WebView _shouldBeginEditingInDOMRange:]):
2352         (-[WebView _shouldEndEditingInDOMRange:]):
2353
2354 2004-06-02  Richard Williamson   <rjw@apple.com>
2355
2356         conformsToProtocol:@protocol(WebPlugin) becomes
2357         respondsToSelector:@selector(webPlugInInitialize) because
2358         protocol was changed to an informal protocol.
2359                 Reviewed by Kevin.
2360
2361         * WebCoreSupport.subproj/WebBridge.m:
2362         (-[WebBridge frameRequiredForMIMEType:URL:]):
2363         * WebView.subproj/WebHTMLView.m:
2364         (-[WebHTMLView addSubview:]):
2365
2366 2004-06-01  Trey Matteson  <trey@apple.com>
2367
2368         First cut at source side of DHTML dragging.  Most of the work is in WebCore.
2369
2370         Reviewed by hyatt.
2371
2372         * WebCoreSupport.subproj/WebBridge.m:
2373         (-[WebBridge startDraggingImage:at:event:]):  Added image and loc args for
2374         when WC tells WK to start a drag.
2375         * WebView.subproj/WebHTMLView.m:
2376         (-[WebHTMLView _startDraggingImage:at:event:]):  New image and loc args coming
2377         from WebCore.  Hysteresis is moved to WC.
2378         (-[WebHTMLView mouseDragged:]):  Call new _startDragging method.
2379         (-[WebHTMLView draggedImage:movedTo:]):  Pass event to WC.
2380         (-[WebHTMLView draggedImage:endedAt:operation:]):  Pass event to WC.
2381         (-[WebHTMLView mouseUp:]):  Reset firstMouseDownEvent.  Fixes bug where
2382         we would occasionally short-circuit WC event handling due to aliasing
2383         problem with this event.
2384         * WebView.subproj/WebHTMLViewPrivate.h:
2385
2386 2004-06-02  Darin Adler  <darin@apple.com>
2387
2388         Reviewed by John.
2389
2390         - fixed problem with timing of delegate callbacks in the back/forward cache case
2391
2392         * WebView.subproj/WebFrame.m:
2393         (-[WebFrame _transitionToCommitted:]): Move code to indicate page is done loading in the
2394         back/forward cache case from here ...
2395         (-[WebFrame _opened]): ... to here.
2396
2397 2004-06-01  Richard Williamson   <rjw@apple.com>
2398
2399         Fixed deployment build warning.
2400
2401         * WebCoreSupport.subproj/WebImageRenderer.m:
2402         (-[WebImageRenderer incrementalLoadWithBytes:length:complete:]):
2403
2404 2004-06-01  Richard Williamson   <rjw@apple.com>
2405
2406         Added support for drawImage and drawImageFromRect to <CANVAS>
2407         Added support for composite attribute to <CANVAS>
2408
2409         Reviewed by Trey.
2410
2411         * WebCoreSupport.subproj/WebImageRenderer.h:
2412         * WebCoreSupport.subproj/WebImageRenderer.m:
2413         (-[WebImageContext initWithBounds:context:]):
2414         (-[WebImageContext dealloc]):
2415         (-[WebImageContext saveGraphicsState]):
2416         (-[WebImageContext restoreGraphicsState]):
2417         (-[WebImageContext isDrawingToScreen]):
2418         (-[WebImageContext focusStack]):
2419         (-[WebImageContext setFocusStack:]):
2420         (-[WebImageContext bounds]):
2421         (-[WebImageContext isFlipped]):
2422         (-[WebImageRenderer incrementalLoadWithBytes:length:complete:]):
2423         (-[WebImageRenderer dealloc]):
2424         (-[WebImageRenderer _beginRedirectContext:]):
2425         (-[WebImageRenderer _endRedirectContext:]):
2426         (-[WebImageRenderer _needsRasterFlush]):
2427         (-[WebImageRenderer drawClippedToValidInRect:fromRect:]):
2428         (-[WebImageRenderer drawImageInRect:fromRect:]):
2429         (-[WebImageRenderer flushRasterCache]):
2430         (-[WebImageRenderer drawImageInRect:fromRect:compositeOperator:context:]):
2431         (-[WebImageRenderer tileInRect:fromPoint:context:]):
2432         * WebCoreSupport.subproj/WebImageRendererFactory.m:
2433         (-[WebImageRendererFactory imageRendererWithMIMEType:]):
2434         (-[WebImageRendererFactory imageRendererWithName:]):
2435         (-[WebImageRendererFactory CGCompositeOperationInContext:]):
2436         (-[WebImageRendererFactory setCGCompositeOperation:inContext:]):
2437         (-[WebImageRendererFactory setCGCompositeOperationFromString:inContext:]):
2438
2439 2004-06-01  John Sullivan  <sullivan@apple.com>
2440
2441         Reviewed by Trey.
2442         
2443         Work on text-align API. Marked these bugs fixed:
2444         <rdar://problem/3655380>: (Editing:Ê-alignCenter:ÊmethodÊunimplementedÊ(WebKitÊeditingÊAPI))
2445         <rdar://problem/3655381>: (Editing:Ê-alignJustified:ÊmethodÊunimplementedÊ(WebKitÊeditingÊAPI))
2446         <rdar://problem/3655383>: (Editing:Ê-alignLeft:ÊmethodÊunimplementedÊ(WebKitÊeditingÊAPI))
2447         <rdar://problem/3655384>: (Editing:Ê-alignRight:ÊmethodÊunimplementedÊ(WebKitÊeditingÊAPI))        
2448         
2449         in favor of opening this bug:
2450         <rdar://problem/3675191>: (Editing: -alignLeft: and friends mostly implemented but not 
2451         yet working (WebKit editing API))        
2452
2453         * English.lproj/StringsNotToBeLocalized.txt:
2454         updated for these changes
2455         
2456         * WebView.subproj/WebHTMLView.m:
2457         (-[WebHTMLView changeDocumentBackgroundColor:]):
2458         added a FIXME about why this is still not quite right
2459         (-[WebHTMLView _alignSelectionUsingCSSValue:]):
2460         new method, bottleneck for the various values
2461         (-[WebHTMLView alignCenter:]):
2462         call _alignSelectionUsingCSSValue:@"center"
2463         (-[WebHTMLView alignJustified:]):
2464         call _alignSelectionUsingCSSValue:@"justify"
2465         (-[WebHTMLView alignLeft:]):
2466         call _alignSelectionUsingCSSValue:@"left"
2467         (-[WebHTMLView alignRight:]):
2468         call _alignSelectionUsingCSSValue:@"right"
2469
2470 2004-06-01  John Sullivan  <sullivan@apple.com>
2471
2472         Reviewed by Ken.
2473         
2474         - fixed <rdar://problem/3655378>: (Editing:Ê-changeDocumentBackgroundColor:Ê
2475         methodÊunimplementedÊ(WebKitÊeditingÊAPI))
2476         - made startSpeaking: actually work; previous implementation raised a DOMException
2477
2478         * WebView.subproj/WebHTMLView.m:
2479         (-[WebHTMLView _changeCSSColorUsingSelector:inRange:]):
2480         added range parameter
2481         (-[WebHTMLView _entireDOMRange]):
2482         new convenience method
2483         (-[WebHTMLView changeDocumentBackgroundColor:]):
2484         now affects entire document, not just selected range, a la NSTextView
2485         (-[WebHTMLView changeColor:]):
2486         now passes in a range to _changeCSSColorUsingSelector:inRange:
2487         (-[WebHTMLView startSpeaking:]):
2488         now uses _entireDOMRange
2489
2490 2004-06-01  Chris Blumenberg  <cblu@apple.com>
2491
2492         Fixed: <rdar://problem/3661505>: (REGRESSION (Safari-140) can't drag standalone images more than once)
2493
2494         Reviewed by john.
2495
2496         * WebView.subproj/WebImageView.m:
2497         (-[WebImageView mouseDown:]): set ignoringMouseDraggedEvents to NO. This line of code was deleted somehow.
2498
2499 2004-06-01  John Sullivan  <sullivan@apple.com>
2500
2501         Reviewed by Darin.
2502         
2503         - fixed <rdar://problem/3657003>: (HTML Editing: Color panel doesn't work)
2504
2505         * WebView.subproj/WebHTMLView.m:
2506         (-[WebHTMLView _colorPanelColorAsStyleUsingSelector:]):
2507         new method, returns a DOMCSSStyleDeclaration *
2508         (-[WebHTMLView _changeCSSColorUsingSelector:]):
2509         new method, sets a color-related style attribute on the selection
2510         (-[WebHTMLView changeDocumentBackgroundColor:]):
2511         call _changeCSSColorUsingSelector: with @selector(setBackgroundColor:)
2512         (-[WebHTMLView changeColor:]):
2513         call _changeCSSColorUsingSelector: with @selector(setColor:); also added
2514         comments explaining why changeDocumentBackgroundColor: will never actually
2515         be called until an AppKit code-incest mess is straighted out.
2516
2517 2004-06-01  Ken Kocienda  <kocienda@apple.com>
2518
2519         Reviewed by Darin
2520
2521         * WebView.subproj/WebHTMLView.m:
2522         (-[WebHTMLView _fontManagerOperationAsStyle]): Fixed typo in family-name code that caused
2523         family names to match when they should not.
2524
2525 2004-06-01  Chris Blumenberg  <cblu@apple.com>
2526
2527         Made paste and drop ask the delegate before making any replacements.
2528
2529         Reviewed by kocienda.
2530
2531         * WebView.subproj/WebHTMLView.m:
2532         (-[WebHTMLView _pasteWithPasteboard:allowPlainText:]): renamed, now calls _shouldInsertFragment:replacingDOMRange:givenAction:
2533         (-[WebHTMLView _shouldInsertFragment:replacingDOMRange:givenAction:]): new, asks delegate
2534         (-[WebHTMLView concludeDragForDraggingInfo:]): now calls _shouldInsertFragment:replacingDOMRange:givenAction:
2535         (-[WebHTMLView paste:]): call renamed _pasteWithPasteboard:allowPlainText:
2536         (-[WebHTMLView pasteAsRichText:]): ditto
2537
2538 2004-05-28  Darin Adler  <darin@apple.com>
2539
2540         Reviewed by Maciej.
2541
2542         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView _fontManagerOperationAsStyle]):
2543         First cut at figuring out what operation the font manager is doing without
2544         digging into its private data structures.
2545
2546 2004-05-28  Darin Adler  <darin@apple.com>
2547
2548         Reviewed by Ken.
2549
2550         - various editing-related improvements
2551         - fixed <rdar://problem/3655366>: (Editing: -selectParagraph: method unimplemented (WebKit editing API))
2552         - fixed <rdar://problem/3655367>: (Editing: -selectLine: method unimplemented (WebKit editing API))
2553         - fixed <rdar://problem/3655369>: (Editing: -selectWord: method unimplemented (WebKit editing API))
2554         - fixed <rdar://problem/3655392>: (Editing: -uppercaseWord: method unimplemented (WebKit editing API))
2555         - fixed <rdar://problem/3655393>: (Editing: -lowercaseWord: method unimplemented (WebKit editing API))
2556         - fixed <rdar://problem/3655394>: (Editing: -capitalizeWord: method unimplemented (WebKit editing API))
2557
2558         * WebView.subproj/WebHTMLView.m:
2559         (-[WebHTMLView _alterCurrentSelection:direction:granularity:]): Removed the call to
2560         ensureCaretVisible. This is now handled on the other side of the bridge.
2561         (-[WebHTMLView moveDown:]): Changed to use WebSelectByLine granularity instead of
2562         WebSelectDown direction.
2563         (-[WebHTMLView moveDownAndModifySelection:]): Ditto.
2564         (-[WebHTMLView moveUp:]): Ditto.
2565         (-[WebHTMLView moveUpAndModifySelection:]): Ditto.
2566         (-[WebHTMLView _expandSelectionToGranularity:]): Added.
2567         (-[WebHTMLView selectParagraph:]): Implemented by calling _expandSelectionToGranularity.
2568         (-[WebHTMLView selectLine:]): Ditto.
2569         (-[WebHTMLView selectWord:]): Ditto.
2570         (-[WebHTMLView _fontManagerOperationAsStyle]): Added. Placeholder for the job of figuring
2571         out what style change to make based on NSFontManager.
2572         (-[WebHTMLView changeFont:]): Implemented, but not really tested because guts are still
2573         missing due to lack of above method.
2574         (-[WebHTMLView insertTab:]): Removed the call to ensureCaretVisible.
2575         (-[WebHTMLView insertNewline:]): Removed the call to ensureCaretVisible.
2576         (-[WebHTMLView insertParagraphSeparator:]): Made this insert a newline for now.
2577         (-[WebHTMLView _changeWordCaseWithSelector:]): Added.
2578         (-[WebHTMLView uppercaseWord:]): Implemented by calling _changeWordCaseWithSelector.
2579         (-[WebHTMLView lowercaseWord:]): Ditto.
2580         (-[WebHTMLView capitalizeWord:]): Ditto.
2581         (-[WebHTMLView deleteBackward:]): Removed the call to ensureCaretVisible.
2582         (-[WebHTMLView checkSpelling:]): Put a pile of AppKit code in here as a placeholder.
2583         (-[WebHTMLView startSpeaking:]): Use the new stringForRange: method instead of outerText.
2584         That way we can handle cases where the entire document is selected.
2585         (-[WebHTMLView insertText:]): Removed the call to ensureCaretVisible.
2586
2587 2004-05-28  Chris Blumenberg  <cblu@apple.com>
2588
2589         Fixed: <rdar://problem/3672129>: (selection deselects when clicking editable WebView in background window)
2590
2591         Fixed this problem by using NSTextView's approach of only allowing dragging on first mouse down.
2592
2593         Reviewed by john.
2594
2595         * WebView.subproj/WebHTMLView.m:
2596         (-[WebHTMLView acceptsFirstMouse:]): store the first mouse down
2597         (-[WebHTMLView mouseDown:]): Don't tell WebCore about the first mouse down event since only dragging can occur on the first mouse down.
2598         (-[WebHTMLView mouseDragged:]): Don't tell WebCore about the drags that occur after the first mouse down since only dragging can occur after the first mouse down.
2599         * WebView.subproj/WebHTMLViewInternal.h:
2600
2601 2004-05-28  Darin Adler  <darin@apple.com>
2602
2603         * WebView.subproj/WebView.m: At Ken's suggestion, for better efficiency and safety,
2604         use _cmd rather than explicit selector names in the forwarding methods.
2605
2606 2004-05-28  Darin Adler  <darin@apple.com>
2607
2608         Reviewed by Ken.
2609
2610         - implemented a few more editing operations, moved code from WebView to WebHTMLView
2611
2612         * WebCoreSupport.subproj/WebBridge.m:
2613         (-[WebBridge respondToChangedContents]): Call _updateFontPanel on the WebHTMLView, not the WebView.
2614         (-[WebBridge respondToChangedSelection]): Ditto.
2615
2616         * WebView.subproj/WebHTMLView.m: Moved WebElementOrTextFilter class here from WebView and
2617         gave it a prefix so it won't conflict with developers' class names.
2618         (-[WebHTMLView _updateFontPanel]): Moved here from WebView.
2619
2620         * WebView.subproj/WebView.m:
2621         (-[WebView toggleSmartInsertDelete:]): Added.
2622         (-[WebView toggleContinuousSpellChecking:]): Added.
2623         (-[WebView isContinuousGrammarCheckingEnabled]): Added.
2624         (-[WebView setContinuousGrammarCheckingEnabled:]): Added.
2625         (-[WebView toggleContinuousGrammarChecking:]): Added.
2626         (-[WebView setSmartInsertDeleteEnabled:]): Implemented. We have the flag now, although we
2627         still don't actually have smart insert and delete implemented.
2628         (-[WebView smartInsertDeleteEnabled]): Ditto.
2629         (-[WebView setContinuousSpellCheckingEnabled:]): Implemented.
2630         (-[WebView isContinuousSpellCheckingEnabled]): Implemented.
2631         (-[WebView spellCheckerDocumentTag]): Implemented.
2632         (-[WebView _preflightSpellCheckerNow:]): Added.
2633         (-[WebView _preflightSpellChecker]): Added.
2634         (-[WebView _continuousCheckingAllowed]): Added.
2635
2636         * WebView.subproj/WebHTMLViewInternal.h: Added. We'll things here from WebHTMLViewPrivate so
2637         they are internal to the framework, rather than SPI.
2638         * WebKit.pbproj/project.pbxproj: Added WebHTMLViewInternal.h.
2639         * WebView.subproj/WebHTMLViewPrivate.h: Moved WebHTMLViewPrivate into the internal header.
2640         Despite its name, it's internal, not SPI.
2641
2642         * WebView.subproj/WebViewPrivate.h: Moved WebViewPrivate into the internal header.
2643         Despite its name, it's internal, not SPI. Added a number of new operations which should
2644         be public API. We'll have to figure out what to do about API review and the WWDC deadline.
2645         * WebView.subproj/WebViewInternal.h: Removed _updateFontPanel method.
2646
2647         * English.lproj/StringsNotToBeLocalized.txt: Update.
2648
2649 2004-05-27  Ken Kocienda  <kocienda@apple.com>
2650
2651         Reviewed by John
2652
2653         The font panel now updates correctly, reflecting the current selection. There may
2654         still be some bugs and corner cases to handle, but this will work for a general
2655         implementation of the feature.
2656
2657         * WebView.subproj/WebView.m:
2658         (+[ElementOrTextFilter filter]): Added. This filter will accept DOM elements and
2659         text nodes and skip everything else. This filter is used when walking a selection
2660         to determine the fonts in use.
2661         (-[ElementOrTextFilter acceptNode:]): DOM node filter implementation method.
2662         (-[WebView _fontFromStyle]): Removed, in lieu of new fontForCurrentPosition call on the bridge.
2663         (-[WebView _updateFontPanel]): Reworked to use a TreeWalker instead of a NodeIterator. This
2664         was done since the iterator must be rooted at the document root, but start iterating 
2665         at the start of the selection. TreeWalker's setCurrentNode allows this to be done.
2666
2667
2668 2004-05-27  Kevin Decker  <kdecker@apple.com>
2669
2670         Reviewed by NOBODY (OOPS!).
2671
2672         * Plugins.subproj/WebScriptObject.h:
2673
2674 2004-05-27  Kevin Decker  <kdecker@apple.com>
2675
2676         Reviewed by Ken.
2677         
2678         - error messages (eg. from JavaScriptCore) sent to the bridge now 
2679           get delievered to a new delegate method.  
2680     
2681         * WebCoreSupport.subproj/WebBridge.m:
2682         (-[WebBridge addMessageToConsole:]):
2683         * WebKit.pbproj/project.pbxproj:
2684         * WebView.subproj/WebUIDelegatePrivate.h:
2685
2686 === Safari-142 ===
2687
2688 2004-05-27  Trey Matteson  <trey@apple.com>
2689
2690         First cut at DHTML dragging, destination side.  Dragging text, files and URLs onto
2691         elements works.  Type conversion from NSPasteboard to MIME types is hardwired.
2692         No JS access yet to modifier keys, or operations mask.
2693
2694         Reviewed by Chris.
2695
2696         * WebView.subproj/WebHTMLView.m:
2697         (-[WebHTMLView draggingUpdatedWithDraggingInfo:]): Call DHTML dragging via bridge.
2698         (-[WebHTMLView draggingCancelledWithDraggingInfo:]): Ditto.
2699         (-[WebHTMLView concludeDragForDraggingInfo:]): Ditto.
2700         * WebView.subproj/WebHTMLViewPrivate.h:
2701         * WebView.subproj/WebView.m:
2702         (-[WebView _setWebKitDragRespondsToDragging:]):  New SPI for finer grained control than the
2703         delegate currently has.
2704         (-[WebView _webKitDragRespondsToDragging]): Ditto.
2705         (-[WebView _commonInitializationWithFrameName:groupName:]):  Init new flag.
2706         (-[WebView _dragOperationForDraggingInfo:]):  Comment.
2707         * WebView.subproj/WebViewPrivate.h:
2708
2709 2004-05-27  Darin Adler  <darin@apple.com>
2710
2711         * WebView.subproj/WebFrameView.m: (-[WebFrameView _scrollVerticallyBy:]): Added comment.
2712
2713 2004-05-27  Darin Adler  <darin@apple.com>
2714
2715         Reviewed by Maciej.
2716
2717         - fixed <rdar://problem/3667948>: (REGRESSION: Page Down key goes down two pages when smooth scrolling is on)
2718
2719         * WebView.subproj/WebFrameView.m:
2720         (-[WebFrameView _scrollVerticallyBy:]): Added return value to indicate if any scrolling was done.
2721         This requires using secret AppKit methods; the public methods don't have a return value.
2722         (-[WebFrameView _pageVertically:]): Added return value to indicate if any scrolling was done.
2723         (-[WebFrameView scrollPageUp:]): Base call through to next responder on whether any scrolling was done,
2724         using return value, rather than looking at new scroll position. This was the cause of the bug,
2725         since with smooth scrolling no scrolling has happened yet when the function returns.
2726         (-[WebFrameView scrollPageDown:]): Ditto.
2727
2728         - removed temporary DOMDocument method from WebView
2729
2730         * WebView.subproj/WebView.m:
2731         (-[WebView computedStyleForElement:pseudoElement:]): Call getComputedStyle on the document that owns the
2732         element rather than on the document that currently contains the selection.
2733         (-[WebView _updateFontPanel]): Get the document from the DOM range rather than using the DOMDocument method.
2734         (-[WebView styleDeclarationWithText:]): Change this method to not use the DOMDocument method, but do the same
2735         job with inline code.
2736
2737         * WebView.subproj/WebViewPrivate.h: Moved a recently-added category that is not SPI out of here.
2738         * WebView.subproj/WebViewInternal.h: Moved the category in here. And removed the DOMDocument method from it.
2739
2740         - other changes
2741
2742         * English.lproj/StringsNotToBeLocalized.txt: Updated for recent changes.
2743
2744         * copy-webcore-files-to-webkit: Change tabs to spaces. Quiet the script down by
2745         making it no longer echo each command or print messages about what it's doing by default.
2746
2747 2004-05-27  Darin Adler  <darin@apple.com>
2748
2749         Reviewed by Maciej.
2750
2751         - moved to new symlink technique for embedding frameworks
2752
2753         * WebKit.pbproj/project.pbxproj: Get rid of embed-frameworks build step
2754         because we don't need it any more.
2755
2756 2004-05-27  Darin Adler  <darin@apple.com>
2757
2758         - fixed Deployment build
2759
2760         * WebView.subproj/WebView.m: (-[WebView concludeDragOperation:]):
2761         Got rid of ASSERT-only local variable.
2762
2763 2004-05-26  Maciej Stachowiak  <mjs@apple.com>
2764
2765         Reviewed by Dave.
2766
2767         - fix further problems with Emerson feed: redirection for RSS feeds
2768
2769         This is done by removing removing the calls to defer callbacks
2770         while waiting for [... Maciej stopped typing here ...]
2771
2772         * WebView.subproj/WebMainResourceClient.m:
2773         (-[WebMainResourceClient continueAfterNavigationPolicy:formState:]):
2774         (-[WebMainResourceClient connection:willSendRequest:redirectResponse:]):
2775         (-[WebMainResourceClient continueAfterContentPolicy:response:]):
2776         (-[WebMainResourceClient checkContentPolicyForResponse:]):
2777
2778 2004-05-26  Chris Blumenberg  <cblu@apple.com>
2779
2780         Added and implemented proposed dragging API changes. These changes are necessary to make JS dragging work properly.
2781
2782         Reviewed by trey.
2783
2784         * WebView.subproj/WebDefaultUIDelegate.m:
2785         (-[WebDefaultUIDelegate webView:shouldDetermineDragOperationForDraggingInfo:dragOperation:]): instead of calling back to the WebView to get the default drag operation, return YES. Return NO in order to return a custom drag operation. Removed element parameter since another new API provides a way to get that.
2786         (-[WebDefaultUIDelegate webView:shouldProcessDragWithDraggingInfo:]): Removed element parameter since another new API provides a way to get that.
2787         * WebView.subproj/WebDocumentInternal.h:
2788         * WebView.subproj/WebHTMLView.m:
2789         (-[WebHTMLView draggingUpdatedWithDraggingInfo:]): now returns a drag operation
2790         (-[WebHTMLView draggingCancelledWithDraggingInfo:]): moved
2791         * WebView.subproj/WebUIDelegatePrivate.h:
2792         * WebView.subproj/WebView.m:
2793         (-[WebView elementAtPoint:]): new proposed API
2794         (-[WebView dragOperationForDraggingInfo:]): removed code from this API that should be removed
2795         (-[WebView _dragOperationForDraggingInfo:]): call new API
2796         (-[WebView concludeDragOperation:]): call new API
2797
2798 2004-05-26  Darin Adler  <darin@apple.com>
2799
2800         Reviewed by John.
2801
2802         - moved HTML editing operations from WebView to WebHTMLView, leaving only forwarding machinery
2803           at the WebView level
2804
2805         - fixed <rdar://problem/3655412>: (Editing: -startSpeaking: method unimplemented (WebKit editing API))
2806         - fixed <rdar://problem/3655414>: (Editing: -stopSpeaking: method unimplemented (WebKit editing API))
2807         - fixed <rdar://problem/3655375>: (Editing: -pasteAsRichText: method unimplemented (WebKit editing API))
2808
2809         * WebView.subproj/WebView.h: Added missing declaration of selectionAffinity. I think this
2810         omission was an editorial mistake.
2811
2812         * WebView.subproj/WebView.m:
2813         (-[WebView searchFor:direction:caseSensitive:wrap:]): Use _frameForCurrentSelection directly,
2814         since it no longer returns nil.
2815         (-[WebView pasteboardTypesForSelection]): Use _frameForCurrentSelection instead of going through the bridge.
2816         (-[WebView _frameForCurrentSelection]): Renamed from _currentFrame and changed to return main frame
2817         rather than nil when called on WebView that has no current selection.
2818         (-[WebView _bridgeForCurrentSelection]): Moved in file.
2819         (-[WebView _updateFontPanel]): Removed the one call to _currentSelectionIsEditable here, since
2820         it was the only one left in this file. Eventually this code will move to WebHTMLView.
2821         (-[WebView _performResponderOperation:with:]): Name change.
2822
2823         * WebView.subproj/WebDataSource.m: (-[WebDataSource _documentFragmentWithImageResource:]):
2824         Build document fragment using DOM instead of composing HTML text.
2825
2826         * WebView.subproj/WebHTMLView.m:
2827         (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): Added allowPlainText boolean,
2828         moved method into new location in file so it can be in the right category, changed link pasting
2829         to use DOM instead of composing HTML text.
2830         (-[WebHTMLView _replaceSelectionWithPasteboard:selectReplacement:allowPlainText:]): Added allowPlainText
2831         boolean, moved method into new location in file so it can be in the right category.
2832         (-[WebHTMLView concludeDragForDraggingInfo:]): Pass YES for allowPlainText.
2833         (-[WebHTMLView centerSelectionInVisibleArea:]): Moved here from WebView.
2834         (-[WebHTMLView _alterCurrentSelection:direction:granularity:]): Ditto.
2835         (-[WebHTMLView moveBackward:]): Ditto.
2836         (-[WebHTMLView moveBackwardAndModifySelection:]): Ditto.
2837         (-[WebHTMLView moveDown:]): Ditto.
2838         (-[WebHTMLView moveDownAndModifySelection:]): Ditto.
2839         (-[WebHTMLView moveForward:]): Ditto.
2840         (-[WebHTMLView moveForwardAndModifySelection:]): Ditto.
2841         (-[WebHTMLView moveLeft:]): Ditto.
2842         (-[WebHTMLView moveLeftAndModifySelection:]): Ditto.
2843         (-[WebHTMLView moveRight:]): Ditto.
2844         (-[WebHTMLView moveRightAndModifySelection:]): Ditto.
2845         (-[WebHTMLView moveToBeginningOfDocument:]): Ditto.
2846         (-[WebHTMLView moveToBeginningOfLine:]): Ditto.
2847         (-[WebHTMLView moveToBeginningOfParagraph:]): Ditto.
2848         (-[WebHTMLView moveToEndOfDocument:]): Ditto.
2849         (-[WebHTMLView moveToEndOfLine:]): Ditto.
2850         (-[WebHTMLView moveToEndOfParagraph:]): Ditto.
2851         (-[WebHTMLView moveUp:]): Ditto.
2852         (-[WebHTMLView moveUpAndModifySelection:]): Ditto.
2853         (-[WebHTMLView moveWordBackward:]): Ditto.
2854         (-[WebHTMLView moveWordBackwardAndModifySelection:]): Ditto.
2855         (-[WebHTMLView moveWordForward:]): Ditto.
2856         (-[WebHTMLView moveWordForwardAndModifySelection:]): Ditto.
2857         (-[WebHTMLView moveWordLeft:]): Ditto.
2858         (-[WebHTMLView moveWordLeftAndModifySelection:]): Ditto.
2859         (-[WebHTMLView moveWordRight:]): Ditto.
2860         (-[WebHTMLView moveWordRightAndModifySelection:]): Ditto.
2861         (-[WebHTMLView pageDown:]): Ditto.
2862         (-[WebHTMLView pageUp:]): Ditto.
2863         (-[WebHTMLView selectParagraph:]): Ditto.
2864         (-[WebHTMLView selectLine:]): Ditto.
2865         (-[WebHTMLView selectWord:]): Ditto.
2866         (-[WebHTMLView copy:]): Moved down in file so it's in the right category.
2867         (-[WebHTMLView cut:]): Ditto.
2868         (-[WebHTMLView delete:]): Ditto.
2869         (-[WebHTMLView paste:]): Ditto.
2870         (-[WebHTMLView copyFont:]): Moved here from WebView.
2871         (-[WebHTMLView pasteFont:]): Ditto.
2872         (-[WebHTMLView pasteAsPlainText:]): Ditto.
2873         (-[WebHTMLView pasteAsRichText:]): Implemented this by calling the paste code with
2874         allowPlainText:NO; believe it or not, that's what this means in NSTextView.
2875         (-[WebHTMLView changeFont:]): Moved here from WebView.
2876         (-[WebHTMLView changeAttributes:]): Ditto.
2877         (-[WebHTMLView changeDocumentBackgroundColor:]): Ditto.
2878         (-[WebHTMLView changeColor:]): Ditto.
2879         (-[WebHTMLView alignCenter:]): Ditto.
2880         (-[WebHTMLView alignJustified:]): Ditto.
2881         (-[WebHTMLView alignLeft:]): Ditto.
2882         (-[WebHTMLView alignRight:]): Ditto.
2883         (-[WebHTMLView indent:]): Ditto.
2884         (-[WebHTMLView insertTab:]): Moved here from WebView, also call insertText rather than
2885         replaceSelectionWithText so it's undoable like a typed character.
2886         (-[WebHTMLView insertBacktab:]): Moved here from WebView.
2887         (-[WebHTMLView insertNewline:]): Moved here from WebView, also call insertText rather than
2888         replaceSelectionWithText so it's undoable like a typed character.
2889         (-[WebHTMLView insertParagraphSeparator:]): Moved here from WebView.
2890         (-[WebHTMLView changeCaseOfLetter:]): Ditto.
2891         (-[WebHTMLView uppercaseWord:]): Ditto.
2892         (-[WebHTMLView lowercaseWord:]): Ditto.
2893         (-[WebHTMLView capitalizeWord:]): Ditto.
2894         (-[WebHTMLView deleteForward:]): Ditto.
2895         (-[WebHTMLView deleteBackward:]): Ditto.
2896         (-[WebHTMLView deleteBackwardByDecomposingPreviousCharacter:]): Ditto.
2897         (-[WebHTMLView deleteWordForward:]): Ditto.
2898         (-[WebHTMLView deleteWordBackward:]): Ditto.
2899         (-[WebHTMLView deleteToBeginningOfLine:]): Ditto.
2900         (-[WebHTMLView deleteToEndOfLine:]): Ditto.
2901         (-[WebHTMLView deleteToBeginningOfParagraph:]): Ditto.
2902         (-[WebHTMLView deleteToEndOfParagraph:]): Ditto.
2903         (-[WebHTMLView complete:]): Ditto.
2904         (-[WebHTMLView checkSpelling:]): Ditto.
2905         (-[WebHTMLView showGuessPanel:]): Ditto.
2906         (-[WebHTMLView performFindPanelAction:]): Ditto.
2907         (-[WebHTMLView startSpeaking:]): Implemented this.
2908         (-[WebHTMLView stopSpeaking:]): Implemented this.
2909         (-[WebHTMLView insertText:]): Moved here from WebView.
2910
2911         * WebView.subproj/WebHTMLViewPrivate.h: Removed declarations of methods that are neither
2912         SPI nor needed outside WebHTMLView.m.
2913
2914         * WebView.subproj/WebViewInternal.h: Removed _currentFrame, and added _frameForCurrentSelection and
2915         _bridgeForCurrentSelection.
2916
2917 2004-05-25  Maciej Stachowiak  <mjs@apple.com>
2918
2919         Reviewed by Richard.
2920
2921         <rdar://problem/3652498>: new sniffing support is crashing
2922         
2923         * WebView.subproj/WebMainResourceClient.m:
2924         (-[WebMainResourceClient checkContentPolicyForResponse:]): Retain
2925         listener around call, in case delegate does something that ends up
2926         invalidating it, like navigating to a new URL.
2927
2928 2004-05-25  Chris Blumenberg  <cblu@apple.com>
2929
2930         Fixed regression where undoing typing would undo character-by-character.
2931
2932         Reviewed by kocienda.
2933
2934         * WebView.subproj/WebView.m:
2935         (-[WebView insertText:]): call insertText: rather than replaceSelectionWithText:: since text insertion via insertText: is coalesced and this is the behavior we want here
2936
2937 2004-05-25  Ken Kocienda  <kocienda@apple.com>
2938
2939         Reviewed by John
2940
2941         Change postDidChangeSelectionNotification and postDidChangeNotification tp
2942         respondToChangedSelection and respondToChangedContents, respectively, to
2943         account for the fact that we do work in these calls other than post a
2944         notification. The need to clear the typing style on both kinds of changes
2945         inspired the name change.
2946
2947         Add in support to set and access typing style. We don't do anything with it yet
2948         except store and return it. Using the typing style is still to come.
2949
2950         * WebCoreSupport.subproj/WebBridge.m:
2951         (-[WebBridge respondToChangedContents]): Change name of functions as described.
2952         (-[WebBridge respondToChangedSelection]): Change name of functions as described.
2953         * WebView.subproj/WebView.m:
2954         (-[WebView dealloc:]): Dealloc typing style ivar.
2955         (-[WebView setTypingStyle:]): Change to set typing style ivar.
2956         (-[WebView typingStyle]): Return new typing style ivar.
2957         * WebView.subproj/WebViewPrivate.h: Add ivar for typing style.
2958
2959 2004-05-25  Ken Kocienda  <kocienda@apple.com>
2960
2961         Reviewed by John
2962         
2963         Improve _bridgeForCurrentSelection so that it is frame-savvy.
2964         Fixup setSelectedDOMRange:affinity: so that it uses the right bridge.
2965
2966         * WebView.subproj/WebView.m:
2967         (-[WebView _bridgeForCurrentSelection]): Use _currentFrame not mainFrame. 
2968         (-[WebView _currentFrame]): Moved to WebView (WebInternal) category so _bridgeForCurrentSelection can use it.
2969         (-[WebView setSelectedDOMRange:affinity:]): Derive the bridge to use from the range passed in; _bridgeForCurrentSelection
2970         is not the right way to get at the document for the range.
2971         * WebView.subproj/WebViewInternal.h: Add _currentFrame declaration.
2972
2973 2004-05-24  Darin Adler  <darin@apple.com>
2974
2975         Reviewed by Ken.
2976
2977         - fixed <rdar://problem/3666022>: (REGRESSION: crash from infinite regress in -[WebFrameView(WebPrivate) scrollPageDown:])
2978
2979         * WebView.subproj/WebView.m:
2980         (-[WebView _performResponderOperation:sender:]): Helper method that knows how to pass on operations
2981         to the responder chain, allowing us to implement operations that will get passed to views inside
2982         us as necessary. Moved a few methods to this, and soon will move even more.
2983         (-[WebView scrollLineDown:]): Use the above method.
2984         (-[WebView scrollLineUp:]): Ditto.
2985         (-[WebView scrollPageDown:]): Ditto.
2986         (-[WebView scrollPageUp:]): Ditto.
2987         (-[WebView copy:]): Ditto.
2988         (-[WebView cut:]): Ditto.
2989         (-[WebView paste:]): Ditto.
2990         (-[WebView delete:]): Ditto.
2991         (-[WebView insertBacktab:]): Ditto.
2992
2993 2004-05-24  Chris Blumenberg  <cblu@apple.com>
2994
2995         Improved editing via drag
2996  
2997         Reviewed by kocienda.
2998
2999         * WebView.subproj/WebDataSource.m: 
3000         (-[WebDataSource _documentFragmentWithImageResource:]): made this method return a fragment instead of replace the selection so that the caller do other things with the fragment
3001         (-[WebDataSource _documentFragmentWithArchive:]): ditto
3002         (-[WebDataSource _replaceSelectionWithArchive:selectReplacement:]): call renamed methods
3003         * WebView.subproj/WebDataSourcePrivate.h:
3004         * WebView.subproj/WebDocumentInternal.h:
3005         * WebView.subproj/WebHTMLView.m:
3006         (-[WebHTMLView _documentFragmentFromPasteboard:]): made this method return a fragment instead of replace the selection so that the caller do other things with the fragment
3007         (-[WebHTMLView _replaceSelectionWithPasteboard:selectReplacement:]): new
3008         (-[WebHTMLView paste:]): call _replaceSelectionWithPasteboard:selectReplacement:
3009         (-[WebHTMLView dragOperationForDraggingInfo:]): handle the case where the destination is editable, but the source is not
3010         (-[WebHTMLView draggingCancelledWithDraggingInfo:]): new, removes drag caret
3011         (-[WebHTMLView draggingUpdatedWithDraggingInfo:]): remove drag caret when we can't handle the drag
3012         (-[WebHTMLView concludeDragForDraggingInfo:]): instead of calling paste, move the selection when doing a move and replace the drag caret when doing a copy
3013         * WebView.subproj/WebHTMLViewPrivate.h:
3014         * WebView.subproj/WebView.m:
3015         (-[WebViewPrivate dealloc]):
3016         (-[WebView _setDraggingDocumentView:]): new
3017         (-[WebView _dragOperationForDraggingInfo:]): if the current dragging document view changes, tell the previous dragging document view that dragging cancelled 
3018         (-[WebView draggingExited:]): new, tell the previous dragging document view that dragging cancelled
3019         (-[WebView concludeDragOperation:]): release the dragging document view
3020         (-[WebView replaceSelectionWithNode:]): pass the selectReplacement BOOL to the bridge
3021         (-[WebView replaceSelectionWithText:]): ditto
3022         (-[WebView replaceSelectionWithMarkupString:]): ditto
3023         (-[WebView replaceSelectionWithArchive:]): ditto
3024         (-[WebView pasteAsPlainText:]): ditto
3025         (-[WebView insertTab:]): ditto
3026         (-[WebView insertText:]): ditto
3027         * WebView.subproj/WebViewPrivate.h: 
3028
3029 2004-05-24  John Sullivan  <sullivan@apple.com>
3030
3031         Reviewed by Dave.
3032         
3033         - added private RSSFeedReferrer field to WebHistoryItem so RSS feeds in the
3034         back/forward list can remember what page (if any) they were initiated from.
3035
3036         * History.subproj/WebHistoryItem.m:
3037         new RSSFeedReferrer ivar in private structure
3038         (-[WebHistoryItemPrivate dealloc]):
3039         release RSSFeedReferrer
3040         (-[WebHistoryItem copyWithZone:]):
3041         copy RSSFeedReferrer
3042         (-[WebHistoryItem RSSFeedReferrer]):
3043         return RSSFeedReferrer
3044         (-[WebHistoryItem setRSSFeedReferrer:]):
3045         set RSSFeedReferrer
3046
3047         * History.subproj/WebHistoryItemPrivate.h:
3048         declare -RSSFeedReferrer and -setRSSFeedReferrer:
3049
3050 2004-05-22  Darin Adler  <darin@apple.com>
3051
3052         Reviewed by Ken.
3053
3054         - implemented some of the trivial WebView editing operations; some had bug reports, to wit:
3055         - fixed <rdar://problem/3655342>: (Editing: -centerSelectionInVisibleArea: method unimplemented (WebKit editing API))
3056         - fixed <rdar://problem/3655398>: (Editing: -deleteWordBackward: method unimplemented (WebKit editing API))
3057         - fixed <rdar://problem/3655397>: (Editing: -deleteWordForward: method unimplemented (WebKit editing API))
3058         - fixed <rdar://problem/3655387>: (Editing: -insertBacktab: method unimplemented (WebKit editing API))
3059         - fixed <rdar://problem/3655386>: (Editing: -insertTab: method unimplemented (WebKit editing API))
3060         - fixed <rdar://problem/3655351>: (Editing: -moveWordBackward: method unimplemented (WebKit editing API))
3061         - fixed <rdar://problem/3655352>: (Editing: -moveWordBackwardAndModifySelection: method unimplemented (WebKit editing API))
3062         - fixed <rdar://problem/3655353>: (Editing: -moveWordForward: method unimplemented (WebKit editing API))
3063         - fixed <rdar://problem/3655354>: (Editing: -moveWordForwardAndModifySelection: method unimplemented (WebKit editing API))
3064         - fixed <rdar://problem/3655374>: (Editing: -pasteAsPlainText: method unimplemented (WebKit editing API))
3065         - fixed <rdar://problem/3655362>: (Editing: -scrollLineDown: method unimplemented (WebKit editing API))
3066         - fixed <rdar://problem/3655363>: (Editing: -scrollLineUp: method unimplemented (WebKit editing API))
3067
3068         * WebView.subproj/WebView.m:
3069         (-[WebView centerSelectionInVisibleArea:]): Implemented. The implementation isn't perfect, but it's hooked
3070         up. It simply calls ensureCaretVisible for now.
3071         (-[WebView moveBackward:]): Implemented. The WebCore API already has a way to specify backward as opposed
3072         to left. A separate issue is the fact that these operations don't have bi-di-savvy implementations, but
3073         now this method is hooked up and will work at least for left-to-right text.
3074         (-[WebView moveBackwardAndModifySelection:]): Ditto.
3075         (-[WebView moveForward:]): Ditto.
3076         (-[WebView moveForwardAndModifySelection:]): Ditto.
3077         (-[WebView moveWordBackward:]): Ditto.
3078         (-[WebView moveWordBackwardAndModifySelection:]): Ditto.
3079         (-[WebView moveWordForward:]): Ditto.
3080         (-[WebView moveWordForwardAndModifySelection:]): Ditto.
3081         (-[WebView scrollLineDown:]): Forward to WebFrameView.
3082         (-[WebView scrollLineUp:]): Ditto.
3083         (-[WebView scrollPageDown:]): Ditto.
3084         (-[WebView scrollPageUp:]): Ditto.
3085         (-[WebView delete:]): Implemented. Follows pattern used in cut, copy, and paste.
3086         (-[WebView pasteAsPlainText:]): Implemented. Calls delegate, then replaceSelectionWithText: on the bridge.
3087         (-[WebView insertTab:]): Implemented. Calls delegate, then replaceSelectionWithText: on the bridge.
3088         (-[WebView insertBacktab:]): Implemented. Does nothing. If we ever change so that you can use a WebView as
3089         a field editor, then we might have to add code here.
3090         (-[WebView deleteWordForward:]): Implement by calling moveForwardAndModifySelection: and then delete:.
3091         Might not be a perfect implementation in the presence of delegates who refuse to delete because it will
3092         change the selection even if the delete is disallowed.
3093         (-[WebView deleteWordBackward:]): Implement by calling moveBackwardAndModifySelection: and then delete:.
3094         Same issue about about delegates as deleteWordForward:.
3095
3096         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView _pasteFromPasteboard:]): Added a FIXME.
3097
3098         * DOM.subproj/DOMViews.h: Updated from recent change to WebCore.
3099
3100 2004-05-21  Richard Williamson   <rjw@apple.com>
3101
3102         Removed _bindObject:forFrame: SPI.
3103
3104         Reviewed by Chris.
3105
3106         * WebView.subproj/WebView.m:
3107         * WebView.subproj/WebViewPrivate.h:
3108
3109 === Safari-141 ===
3110
3111 2004-05-21  Darin Adler  <darin@apple.com>
3112
3113         Reviewed by Ken and Chris.
3114
3115         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView _pasteFromPasteboard:]): Call replaceSelectionWithText:
3116         instead of replaceSelectionWithMarkupString: when pasting plain text.
3117
3118         * WebView.subproj/WebDataSource.m: (-[WebDataSource _replaceSelectionWithMarkupString:baseURL:]):
3119         Remove bogus check for empty markup. There's nothing wrong with an empty string, and no reason
3120         that replacing with empty string should be a no-op instead of a delete.
3121
3122 2004-05-20  Darin Adler  <darin@apple.com>
3123
3124         Reviewed by Chris.
3125
3126         - fixed <rdar://problem/3662383>: (REGRESSION: drag slide-back sometimes causes link to load)
3127
3128         * WebCoreSupport.subproj/WebBridge.m: (-[WebBridge handleMouseDragged:]):
3129         Added BOOL result to handleMouseDragged:.
3130         * WebView.subproj/WebHTMLViewPrivate.h: Added BOOL result to _handleMouseDragged:.
3131
3132         * WebView.subproj/WebHTMLView.m: (-[WebHTMLView _handleMouseDragged:]): Added BOOL result,
3133         returning YES when the drag started, and no when the hysteresis has not yet been overcome.
3134
3135 2004-05-20  Ken Kocienda  <kocienda@apple.com>
3136
3137         Reviewed by Hyatt
3138
3139         Provide the methods to glue the WebView's editing delegate so that these methods work:
3140         
3141         <rdar://problem/3655316>: "Editing: -webViewShouldBeginEditing:inDOMRange: method unimplemented (WebKit editing API)"
3142         <rdar://problem/3655317>: "Editing: -webViewShouldEndEditing:inDOMRange: method unimplemented (WebKit editing API)"
3143
3144         * WebCoreSupport.subproj/WebBridge.m:
3145         (-[WebBridge shouldBeginEditing:]): New method used to glue delegate to focus shifts.
3146         (-[WebBridge shouldEndEditing:]): Ditto.
3147         * WebView.subproj/WebView.m:
3148         (-[WebView _shouldBeginEditingInDOMRange:]): Ditto.
3149         (-[WebView _shouldEndEditingInDOMRange:]): Ditto.
3150         * WebView.subproj/WebViewPrivate.h: Ditto.
3151
3152 2004-05-20  Richard Williamson   <rjw@apple.com>
3153
3154         Fixed typo in header comment.
3155
3156         Reviewed by Ken.
3157
3158         * Plugins.subproj/WebScriptObject.h:
3159
3160 2004-05-19  Chris Blumenberg  <cblu@apple.com>
3161
3162         Fixed: <rdar://problem/3462627>: (API: Need a way to disable/customize dragging)
3163
3164         Reviewed by john.
3165
3166         * Misc.subproj/WebNSPasteboardExtras.m:
3167         (-[NSPasteboard _web_writeImage:URL:title:archive:types:]): fixed bug that caused exception
3168         * WebView.subproj/WebDefaultUIDelegate.m:
3169         (-[WebDefaultUIDelegate webView:shouldBeginDragForElement:dragImage:mouseDownEvent:mouseDraggedEvent:]): new, returns YES
3170         (-[WebDefaultUIDelegate webView:dragOperationForDraggingInfo:overElement:]): new, returns [WebView dragOperationForDraggingInfo:]
3171         (-[WebDefaultUIDelegate webView:shouldProcessDragWithDraggingInfo:overElement:]): new, returns YES
3172         * WebView.subproj/WebDocumentInternal.h: added WebDocumentDragging and WebDocumentElement for document dragging
3173         * WebView.subproj/WebDocumentPrivate.h: moved WebDocumentSelection to WebDocumentInternal.h
3174         * WebView.subproj/WebFrame.m:
3175         (-[WebFrame _actionInformationForNavigationType:event:originalURL:]): 
3176         * WebView.subproj/WebHTMLView.h:
3177         * WebView.subproj/WebHTMLView.m:
3178         (+[WebHTMLView _insertablePasteboardTypes]): new 
3179         (-[WebHTMLView _handleMouseDragged:]): call shouldBeginDragForElement:::: delegate API
3180         (-[WebHTMLView _mayStartDragWithMouseDragged:]): call renamed elementAtPoint SPI
3181         (-[WebHTMLView initWithFrame:]): don't register for drag types since this is handled at the WebView
3182         (-[WebHTMLView menuForEvent:]): call renamed elementAtPoint SPI
3183         (-[WebHTMLView _isSelectionEvent:]): call renamed elementAtPoint SPI
3184         (-[WebHTMLView dragImage:at:offset:event:pasteboard:source:slideBack:]): rather than unregistering drag types on the WebView, just tell it that we're dragging
3185         (-[WebHTMLView draggedImage:endedAt:operation:]): ditto
3186         (-[WebHTMLView _canProcessDragWithDraggingInfo:]): new
3187         (-[WebHTMLView dragOperationForDraggingInfo:]): new WebDocumentDragging SPI
3188         (-[WebHTMLView draggingUpdatedWithDraggingInfo:]): ditto
3189         (-[WebHTMLView concludeDragForDraggingInfo:]): ditto
3190         (-[WebHTMLView elementAtPoint:]): renamed from _elementAtPoint since this is part of the WebDocumentElement SPI
3191         * WebView.subproj/WebHTMLViewPrivate.h:
3192         * WebView.subproj/WebImageView.h:
3193         * WebView.subproj/WebImageView.m:
3194         (-[WebImageView elementAtPoint:]): new
3195         (-[WebImageView menuForEvent:]): call elementAtPoint
3196         (-[WebImageView mouseDragged:]): rather than unregistering drag types on the WebView, just tell it that we're dragging
3197         (-[WebImageView draggedImage:endedAt:operation:]): ditto
3198         * WebView.subproj/WebTextView.h:
3199         * WebView.subproj/WebTextView.m:
3200         (-[WebTextView _elementAtWindowPoint:]): new
3201         (-[WebTextView elementAtPoint:]): new
3202         (-[WebTextView menuForEvent:]): call _elementAtWindowPoint
3203         * WebView.subproj/WebView.h:
3204         * WebView.subproj/WebView.m:
3205         (-[WebViewPrivate dealloc]): removed draggedTypes ivar
3206         (+[WebView URLFromPasteboard:]): implemented
3207         (+[WebView URLTitleFromPasteboard:]): implemented
3208         (-[WebView _registerDraggedTypes]): moved
3209         (-[WebView _frameViewAtWindowPoint:]): new
3210         (-[WebView _draggingDocumentViewAtWindowPoint:]): new
3211         (-[WebView _elementAtWindowPoint:]): new
3212         (-[WebView dragOperationForDraggingInfo:]): updated this API to handle subviews that may want to handle drags
3213         (-[WebView _dragOperationForDraggingInfo:]): new, handles UI delegate for drag control
3214         (-[WebView draggingEntered:]): call _dragOperationForDraggingInfo:
3215         (-[WebView draggingUpdated:]): ditto
3216         (-[WebView concludeDragOperation:]): work with the UI delegate and the subview to handle what happens
3217         * WebView.subproj/WebViewPrivate.h:
3218
3219 2004-05-19  Richard Williamson   <rjw@apple.com>
3220
3221         Removed extraneous tabs that were added (by XCode?).
3222         
3223         * DOM.subproj/DOM-compat.h:
3224         * Plugins.subproj/WebScriptObject.h:
3225
3226 2004-05-19  Richard Williamson   <rjw@apple.com>
3227
3228         Updated header copy script to only copy if modified headers
3229         are different.
3230
3231         * copy-webcore-files-to-webkit:
3232
3233 2004-05-19  Ken Kocienda  <kocienda@apple.com>
3234
3235         Reviewed by Hyatt and Darin
3236
3237         Fix for this bug:
3238         
3239         <rdar://problem/3643230>: "can't tab out of contentEditable Elements"
3240
3241         * WebCoreSupport.subproj/WebBridge.m:
3242         (-[WebBridge interceptEditingKeyEvent:]): Renamed from _editingKeyDown. Also now returns
3243         a BOOL to report whether the event was handled or not.
3244         * WebView.subproj/WebView.m:
3245         (-[WebView _interceptEditingKeyEvent:]): Also renamed from _editingKeyDown. Now includes
3246         a check if the web view is editable and whether the event is a tab key event. If the former
3247         is not true and the latter is, the key is not intercepted. This causes the tab to shift once
3248         the key is processed by other non-editing key-handling mechanisms.
3249         * WebView.subproj/WebViewPrivate.h: Changed declaration due to name change.
3250
3251 2004-05-19  Ken Kocienda  <kocienda@apple.com>
3252
3253         Reviewed by Hyatt
3254
3255         * WebCoreSupport.subproj/WebBridge.m:
3256         (-[WebBridge isEditable]): Return the isEditable value for the
3257         WebView which contains this bridge's frame.
3258
3259 2004-05-19  Darin Adler  <darin@apple.com>
3260
3261         - fixed headers with licenses mangled by Xcode auto-indenting
3262
3263         * DOM.subproj/DOMExtensions.h:
3264         * WebCoreSupport.subproj/WebGraphicsBridge.h:
3265         * WebCoreSupport.subproj/WebGraphicsBridge.m:
3266
3267 2004-05-18  David Hyatt  <hyatt@apple.com>
3268
3269         Improve layout scheduling.
3270         Reviewed by kocienda
3271
3272         * WebView.subproj/WebFrame.m:
3273         (-[WebFrame _transitionToLayoutAcceptable]):
3274         (-[WebFrame _checkLoadCompleteForThisFrame]):
3275         * WebView.subproj/WebHTMLRepresentation.m:
3276         (-[WebHTMLRepresentation finishedLoadingWithDataSource:]):
3277
3278 2004-05-18  Richard Williamson   <rjw@apple.com>
3279
3280         Finished implementation of windowScriptObject.
3281                 Reviewed by Maciej.
3282
3283         * WebView.subproj/WebView.m:
3284         (-[WebView windowScriptObject]):
3285
3286 2004-05-18  Richard Williamson   <rjw@apple.com>
3287
3288         Added WebKit portion of webView:windowScriptObjectAvailable:
3289         implementation.  Still need to implement creating the WebScriptObject
3290         wrapper on the WebCore side.
3291         
3292         Reviewed by Maciej.
3293
3294         Removed "_" from _setPageWidthForPrinting:.  This method facilitates
3295         a work-around for carbon printing.  At some point we may make this
3296         method public API.
3297                 Reviewed by Chris.
3298
3299         * WebCoreSupport.subproj/WebBridge.m:
3300         (-[WebBridge overrideMediaType]):
3301         (-[WebBridge windowObjectCleared]):
3302         * WebView.subproj/WebDefaultFrameLoadDelegate.m:
3303         (-[WebDefaultFrameLoadDelegate webView:windowScriptObjectAvailable:]):
3304         * WebView.subproj/WebHTMLView.m:
3305         (-[WebHTMLView setPageWidthForPrinting:]):
3306
3307 2004-05-18  Darin Adler  <darin@apple.com>
3308
3309         Reviewed by John.
3310
3311         - fixed <rdar://problem/3520322>: "can't use <WebKit/HIWebView.h> or <WebKit/CarbonUtils.h> from non-Objective C"
3312
3313         * Carbon.subproj/CarbonUtils.h: Added ifdefs so file compiles when included from non-Objective-C.
3314         Changed style to match other Carbon headers a bit more closely. Also remove unnecessary includes.
3315         * Carbon.subproj/HIWebView.h: Ditto.
3316
3317         - fixed <rdar://problem/3648505>: "this text file scrolls to the second line instead of first when pressing home"
3318
3319         * WebView.subproj/WebFrameView.m:
3320         (-[WebFrameView _scrollToTopLeft]): Scroll to origin.y instead of assuming that top is 0; can be non-0 for text view.
3321         (-[WebFrameView _scrollToBottomLeft]): Use NSMaxY instead of height for the same reason.
3322
3323 2004-05-17  David Hyatt  <hyatt@apple.com>
3324
3325         Fix for performance regression in PLT caused by not setting _timeOfLastCompletedLoad, causing
3326         page cache to release during the benchmark.
3327         * WebView.subproj/WebFrame.m:
3328         (-[WebFrame _setState:]):
3329
3330 2004-05-17  Chris Blumenberg  <cblu@apple.com>
3331
3332         Implemented new WebView pasteboard methods. Made a lot of factoring changes related to pasteboard management.
3333
3334         Reviewed by john.
3335
3336         * Misc.subproj/WebNSPasteboardExtras.h:
3337         * Misc.subproj/WebNSPasteboardExtras.m:
3338         (+[NSPasteboard _web_writableTypesForURL]): renamed to not include "drag" these types are also used for copying
3339         (+[NSPasteboard _web_writableTypesForImage]): new
3340         (-[NSPasteboard _web_bestURL]): tweak
3341         (-[NSPasteboard _web_writeURL:andTitle:types:]): take an array of types that this method should write, don't declare the types since this complicates things for the caller
3342         (-[NSPasteboard _web_writeImage:URL:title:archive:types:]): ditto
3343         * Misc.subproj/WebNSViewExtras.m:
3344         (-[NSView _web_dragImage:archive:rect:URL:title:event:]): call renamed methods
3345         * WebView.subproj/WebDefaultContextMenuDelegate.m:
3346         (-[WebDefaultUIDelegate copyLinkToClipboard:]): call code factored out to WebView
3347         (-[WebDefaultUIDelegate copyImageToClipboard:]): ditto
3348         * WebView.subproj/WebDocumentPrivate.h:
3349         * WebView.subproj/WebHTMLView.h:
3350         * WebView.subproj/WebHTMLView.m:
3351         (-[WebHTMLView _selectedRTFData]): new factored out method
3352         (-[WebHTMLView _writeSelectionToPasteboard:]): factored code out to writeSelectionWithPasteboardTypes:toPasteboard:
3353         (-[WebHTMLView _dragImageForLinkElement:]): tweak
3354         (-[WebHTMLView _handleMouseDragged:]): call renamed methods
3355         (-[WebHTMLView pasteboardTypesForSelection]): new
3356         (-[WebTextView writeSelectionWithPasteboardTypes:toPasteboard:]): new, code moved from _writeSelectionToPasteboard:
3357         * WebView.subproj/WebHTMLViewPrivate.h:
3358         * WebView.subproj/WebImageView.h:
3359         * WebView.subproj/WebImageView.m:
3360         (-[WebImageView writeImageToPasteboard:types:]): call renamed methods
3361         (-[WebImageView copy:]):
3362         (-[WebImageView writeSelectionToPasteboard:types:]): call renamed methods
3363         * WebView.subproj/WebTextView.h:
3364         * WebView.subproj/WebTextView.m:
3365         (-[WebTextView pasteboardTypesForSelection]): new
3366         (-[WebTextView writeSelectionWithPasteboardTypes:toPasteboard:]): new
3367         * WebView.subproj/WebView.m:
3368         (-[WebView _writeImageElement:withPasteboardTypes:toPasteboard:]): new
3369         (-[WebView _writeLinkElement:withPasteboardTypes:toPasteboard:]): mew
3370         (-[WebView dragOperationForDraggingInfo:]): implemented API
3371         (-[WebView draggingEntered:]): call API
3372         (-[WebView draggingUpdated:]): ditto
3373         (-[WebView concludeDragOperation:]): ditto
3374         (-[WebView pasteboardTypesForSelection]): implemented API
3375         (-[WebView writeSelectionWithPasteboardTypes:toPasteboard:]): ditto
3376         (-[WebView pasteboardTypesForElement:]): ditto
3377         (-[WebView writeElement:withPasteboardTypes:toPasteboard:]): ditto
3378         * WebView.subproj/WebViewPrivate.h:
3379
3380 2004-05-17  Ken Kocienda  <kocienda@apple.com>
3381
3382         Reviewed by John
3383
3384         Remove overrides in WebView for scrollPageDown and scrollPageUp.
3385         NSView behavior gives us just what we want, and there is no
3386         special behavior required for editing.
3387
3388         <rdar://problem/3655364>: "Editing: -scrollPageDown: method unimplemented (WebKit editing API)"
3389         <rdar://problem/3655365>: "Editing: -scrollPageUp: method unimplemented (WebKit editing API)"
3390
3391         * WebView.subproj/WebView.h: Comment methods out and add a note about why.
3392         * WebView.subproj/WebView.m: Remove stubbed out implementation.
3393
3394 2004-05-14  Vicki Murley  <vicki@apple.com>
3395
3396         Reviewed by mjs.
3397  
3398         <rdar://problem/3642427>: framework marketing number should be 2.0 for DoubleBarrel release
3399
3400         * WebKit.pbproj/project.pbxproj: change CFBundleShortVersionString to 2.0
3401
3402 2004-05-14  David Hyatt  <hyatt@apple.com>
3403
3404         Eliminate timedLayout.
3405         Reviewed by darin
3406
3407         * WebView.subproj/WebFrame.m:
3408         (-[WebFramePrivate dealloc]):
3409         (-[WebFrame _detachFromParent]):
3410         (-[WebFrame _transitionToLayoutAcceptable]):
3411         (-[WebFrame _setState:]):
3412         (-[WebFrame _checkLoadCompleteForThisFrame]):
3413         (-[WebFrame stopLoading]):
3414         * WebView.subproj/WebFramePrivate.h:
3415
3416 === Safari-140 ===
3417
3418 2004-05-14  Chris Blumenberg  <cblu@apple.com>
3419
3420         Fixed: <rdar://problem/3655495>: (exception loading applets)
3421
3422         Reviewed by kocienda.
3423
3424         * WebCoreSupport.subproj/WebBridge.m:
3425         (-[WebBridge pluginViewWithPackage:attributes:baseURL:]): when calling old cocoa plug-ins, use old keys
3426
3427 2004-05-14  Chris Blumenberg  <cblu@apple.com>
3428
3429         Fixed: <rdar://problem/3655204>: (repro assertion failure and crash loading java applets)
3430
3431         Reviewed by kocienda.
3432
3433         * WebCoreSupport.subproj/WebBridge.m:
3434         (-[WebBridge pluginViewWithPackage:attributes:baseURL:]): call pluginViewWithArguments: for old Cocoa plug-ins
3435
3436 2004-05-14  Chris Blumenberg  <cblu@apple.com>
3437
3438         Copied headers from WebCore.
3439
3440         * DOM.subproj/DOMCore.h:
3441         * DOM.subproj/DOMEvents.h:
3442
3443 2004-05-14  Ken Kocienda  <kocienda@apple.com>
3444
3445         Reviewed by me
3446
3447         * Plugins.subproj/WebPluginController.m:
3448         (-[WebPluginController webPlugInContainerLoadRequest:inFrame:]): Fixed a compile error: undeclared identifier. 
3449         Looked like a typo.
3450
3451 2004-05-13  Richard Williamson   <rjw@apple.com>
3452
3453         Backed out mistaken change that I didn't mean to checkin.
3454
3455         * WebCoreSupport.subproj/WebBridge.m:
3456         (-[WebBridge syncLoadResourceWithURL:customHeaders:postData:finalURL:responseHeaders:statusCode:]):
3457
3458 2004-05-13  Richard Williamson   <rjw@apple.com>
3459
3460         Updated to reflect new API.
3461
3462         Reviewed by Chris.
3463
3464         * WebCoreSupport.subproj/WebBridge.m:
3465         (-[WebBridge setIsSelected:forView:]):
3466         * WebView.subproj/WebFrame.m:
3467         (-[WebFrame _reloadForPluginChanges]):
3468
3469 2004-05-13  Chris Blumenberg  <cblu@apple.com>
3470
3471         Fixed some indenting issues in public headers.
3472
3473         * Plugins.subproj/WebJavaPlugIn.h:
3474         * Plugins.subproj/WebPlugin.h: