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