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