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