WebCore:
[WebKit-https.git] / WebKit / ChangeLog
1 2006-09-26  John Sullivan  <sullivan@apple.com>
2
3         Reviewed by Darin
4
5         * WebView/WebHTMLViewPrivate.h:
6         * WebView/WebHTMLView.m:
7         (-[WebHTMLView markAllMatchesForText:caseSensitive:limit:]):
8         Added limit parameter, passed over the bridge. Stop the search if it hits limit.
9
10         * WebView/WebViewPrivate.h:
11         * WebView/WebView.m:
12         (-[WebView markAllMatchesForText:caseSensitive:highlight:limit:]):
13         Added limit parameter, passed to WebHTMLView.
14
15 2006-09-26  David Harrison  <harrison@apple.com>
16
17         Reviewed by John and TimH.
18
19         <rdar://problem/4743256> Seed: Ctrl-Y key binding does nothing when kill ring is empty
20
21         Use deleteBackward: when the killring string is empty. Was always using insertText:,
22         but that ends up early-returning if the string to insert is empty.
23         
24         * WebView/WebHTMLView.m:
25         (-[NSArray yank:]):
26         (-[NSArray yankAndSelect:]):
27
28 2006-09-25  Timothy Hatcher  <timothy@apple.com>
29
30         Reviewed by Brady.
31
32         Use the non-deprecated method names for getComputedStyle, setEnd and setStart.
33
34         * WebInspector/WebInspector.m:
35         (-[WebInspector _highlightNode:]):
36         * WebView/WebHTMLView.m:
37         (unionDOMRanges):
38         (-[WebHTMLView _selectRangeInMarkedText:]):
39         (-[WebTextCompleteController doCompletion]):
40         * WebView/WebView.m:
41         (-[WebView computedStyleForElement:pseudoElement:]):
42
43 2006-09-22  Timothy Hatcher  <timothy@apple.com>
44
45         Reviewed by Darin.
46
47         Pass all headers through unifdef to filter out BUILDING_ON_TIGER blocks
48         if MACOSX_DEPLOYMENT_TARGET is 10.4.
49
50         * MigrateHeaders.make:
51
52 2006-09-20  Justin Garcia  <justin.garcia@apple.com>
53
54         Reviewed by john
55         
56         <http://bugzilla.opendarwin.org/show_bug.cgi?id=7165>
57         TinyMCE: Dragging & dropping content always leaves a copy when editing inside a subframe
58         
59         The top level WebHTMLView is responsible for performing dragging operations, but
60         the inner view, the view that holds the drag caret, should be consulted to determine
61         if the drag is a move drag.
62
63         * WebCoreSupport/WebFrameBridge.m:
64         (-[WebFrameBridge startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
65         * WebKit.xcodeproj/project.pbxproj:
66         * WebView/WebHTMLView.m:
67         (-[NSArray dragImage:at:offset:event:pasteboard:source:slideBack:]): Don't set initatedDrag here, because
68         it's only the top level WebHTMLView that performs this operation.
69         (-[WebHTMLView _setInitiatedDrag:]): Added.
70         (-[WebHTMLView _initiatedDrag]): Ditto.
71         (-[WebHTMLView _canProcessDragWithDraggingInfo:]): Ask the innerView if it initiated the drag,
72         not the top level view.
73         (-[WebHTMLView _isMoveDrag]): The top level view asks the innerView if it should perform
74         a move drag, so don't ASSERT _isTopHTMLView.
75         (-[WebHTMLView draggingUpdatedWithDraggingInfo:actionMask:]): Ask the innerView if _isMoveDrag.
76         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): Ditto.
77         * WebView/WebHTMLViewInternal.h: Added two private SPI so that
78         the top level WebHTMLView can set and get the initiatedDrag BOOL.
79
80 === Safari-521.27 ===
81
82 2006-09-20  Brady Eidson  <beidson@apple.com>
83
84         Reviewed by Tim Omernick
85
86         Fixing part of a crash Tim O showed me.  [WebIconDatabase init] should finish
87         gracefully even if we can't open the database
88
89         * Misc/WebIconDatabase.m:
90         (-[WebIconDatabase init]):
91
92 2006-09-20  Timothy Hatcher  <timothy@apple.com>
93
94         Reviewed by Tim O.
95
96         <rdar://problem/4741392> Develop a system to find what version of WebKit an app was linked with
97
98         * Misc/WebKitVersionChecks.h: Added.
99         * Misc/WebKitVersionChecks.m: Added.
100         (WebKitLinkedOnOrAfter): Added.
101         (WebKitLinkTimeVersion): Added.
102         (WebKitRunTimeVersion): Added.
103         * WebKit.xcodeproj/project.pbxproj:
104
105 2006-09-20  Tim Omernick  <timo@apple.com>
106
107         Reviewed by Darin.
108
109         * Misc/WebIconDatabase.m:
110         (-[WebIconDatabase init]):
111         Fixed a typo.
112
113 2006-09-20  Brady Eidson  <beidson@apple.com>
114
115         Reviewed by Darin
116
117         Preparing to make the WebIconDatabase disabled by default - this patch tells the bridge whether its enabled or not
118
119         * Misc/WebIconDatabase.m:
120         (-[WebIconDatabase init]):
121
122 2006-09-19  Brady Eidson <beidson@apple.com>
123
124         Reviewed by Sarge Decker
125
126         <rdar://problem/4739892> and <rdar://problem/4729797>
127         - WebCore::IconDatabase needs to have and respect an enabled() flag
128         - Mail on ToT WebKit crashes in IconDatabase code when mailing a page from Safari
129
130         * Misc/WebIconDatabase.m:
131         (-[WebIconDatabase init]): If preference says icons are disabled, tell the bridge
132         (-[WebIconDatabase _isEnabled]): Ask the bridge if the database is enabled
133
134 2006-09-19  Alexey Proskuryakov  <ap@nypop.com>
135
136         Reviewed by Tim O.
137
138         http://bugzilla.opendarwin.org/show_bug.cgi?id=10661
139         REGRESSION: CFM plug-ins (Shockwave, SVG) are not loaded
140
141         * Plugins/WebBasePluginPackage.m:
142         (-[WebBasePluginPackage initWithPath:]): On 32-bit PowerPC, don't bail out if 
143         the bundle is nil - it can be a CFM plugin.
144
145 2006-09-18  Brady Eidson  <beidson@apple.com>
146
147         Reviewed by Anders
148
149         Implement a bridge method so WebCore can find the reload type of a frame load
150
151         * WebCoreSupport/WebFrameBridge.m:
152         (-[WebFrameBridge isLoadTypeReload]):
153
154 2006-09-18  Sam Weinig  <sam.weinig@gmail.com>
155
156         Reviewed by Tim H.
157
158         Patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10903
159         Yet Another Objective-C Bindings Patch
160
161         * MigrateHeaders.make:
162
163 2006-09-17  David Harrison  <harrison@apple.com>
164
165         Reviewed by John Sullivan.
166
167         <rdar://problem/4494340> REGRESSION: Making the font size bigger/smaller in an HTML message doesn't affect the body until you reopen it
168
169         Problem was the public API -[WebView setTextSizeMultiplier] did not notify anyone that the value changed.
170         
171         * WebView/WebDocumentInternal.h:
172         Add _textSizeMultiplierChanged to the _WebDocumentTextSizing protocol.
173         
174         * WebView/WebHTMLView.m:
175         (-[WebHTMLView _textSizeMultiplierChanged]):
176         Send [self _updateTextSizeMultiplier].
177         
178         * WebView/WebPDFView.m:
179         (-[WebPDFView _textSizeMultiplierChanged]):
180         ASSERT_NOT_REACHED() because WebPDFView does not track the common multiplier.
181         
182         * WebView/WebView.m:
183         (-[WebView setTextSizeMultiplier:]):
184         Send [self _notifyTextSizeMultiplierChanged];
185         
186         (-[WebView _performTextSizingSelector:withObject:onTrackingDocs:selForNonTrackingDocs:newScaleFactor:]):
187         Alter the _textSizeMultiplier directly so that notification is not sent.
188         Minor formatting.
189         
190         (-[WebView _notifyTextSizeMultiplierChanged]):
191         New. Send _textSizeMultiplierChanged to all document views that track the common multiplier.
192
193 2006-09-16  Brady Eidson <beidson@apple.com>
194
195         Reviewed by Hyatt
196
197         WebIconLoader is dead, long live WebCore::IconLoader
198         (code prune)
199
200         * Loader/WebFrameLoader.h:
201         * Loader/WebFrameLoader.m:
202         (-[WebFrameLoader dealloc]):
203         (-[WebFrameLoader commitProvisionalLoad]):
204         * Loader/WebIconLoader.h: Removed.
205         * Loader/WebIconLoader.m: Removed.
206         * Misc/WebIconDatabase.h:
207         * Misc/WebIconDatabase.m:
208         * Misc/WebIconDatabasePrivate.h:
209         * WebCoreSupport/WebFrameBridge.m:
210         (-[WebFrameBridge notifyIconChanged:]):
211         * WebCoreSupport/WebIconDatabaseBridge.h:
212         * WebCoreSupport/WebIconDatabaseBridge.m:
213         (-[WebIconDatabaseBridge _init]):
214         (-[WebIconDatabaseBridge _setIconData:forIconURL:]):
215         (-[WebIconDatabaseBridge _setHaveNoIconForIconURL:]):
216         * WebKit.exp:
217         * WebKit.xcodeproj/project.pbxproj:
218         * WebView/WebDataSource.m:
219         (-[WebDataSourcePrivate dealloc]):
220         (-[WebDataSource _stopLoading]):
221         (-[WebDataSource _setPrimaryLoadComplete:]):
222         * WebView/WebDataSourceInternal.h:
223
224 2006-09-16  Mark Rowe  <opendarwin.org@bdash.net.nz>
225
226         Reviewed by Anders.
227
228         http://bugzilla.opendarwin.org/show_bug.cgi?id=10887
229         Fix build error
230
231         * MigrateHeaders.make: Remove reference to DOMEventPrivate.h.
232
233 2006-09-15  Timothy Hatcher  <timothy@apple.com>
234
235         Reviewed by Brady.
236
237         Make new style ObjC methods public API.
238
239         * MigrateHeaders.make:
240
241 2006-09-15  Sam Weinig  <sam.weinig@gmail.com>
242
243         Reviewed by Tim H.
244
245         Patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10870
246         Auto-generate DOMNode for the Objective-C bindings
247
248         * MigrateHeaders.make:
249
250 2006-09-15  Timothy Hatcher  <timothy@apple.com>
251
252         Reviewed by Justin.
253
254         Call the bridge directly for alter selection calls. All of this logic is now
255         in WebCore's SelectionController.
256
257         * WebView/WebHTMLView.m:
258         (-[NSArray moveBackward:]):
259         (-[NSArray moveBackwardAndModifySelection:]):
260         (-[NSArray moveDown:]):
261         (-[NSArray moveDownAndModifySelection:]):
262         (-[NSArray moveForward:]):
263         (-[NSArray moveForwardAndModifySelection:]):
264         (-[NSArray moveLeft:]):
265         (-[NSArray moveLeftAndModifySelection:]):
266         (-[NSArray moveRight:]):
267         (-[NSArray moveRightAndModifySelection:]):
268         (-[NSArray moveToBeginningOfDocument:]):
269         (-[NSArray moveToBeginningOfDocumentAndModifySelection:]):
270         (-[NSArray moveToBeginningOfSentence:]):
271         (-[NSArray moveToBeginningOfSentenceAndModifySelection:]):
272         (-[NSArray moveToBeginningOfLine:]):
273         (-[NSArray moveToBeginningOfLineAndModifySelection:]):
274         (-[NSArray moveToBeginningOfParagraph:]):
275         (-[NSArray moveToBeginningOfParagraphAndModifySelection:]):
276         (-[NSArray moveToEndOfDocument:]):
277         (-[NSArray moveToEndOfDocumentAndModifySelection:]):
278         (-[NSArray moveToEndOfSentence:]):
279         (-[NSArray moveToEndOfSentenceAndModifySelection:]):
280         (-[NSArray moveToEndOfLine:]):
281         (-[NSArray moveToEndOfLineAndModifySelection:]):
282         (-[NSArray moveToEndOfParagraph:]):
283         (-[NSArray moveToEndOfParagraphAndModifySelection:]):
284         (-[NSArray moveParagraphBackwardAndModifySelection:]):
285         (-[NSArray moveParagraphForwardAndModifySelection:]):
286         (-[NSArray moveUp:]):
287         (-[NSArray moveUpAndModifySelection:]):
288         (-[NSArray moveWordBackward:]):
289         (-[NSArray moveWordBackwardAndModifySelection:]):
290         (-[NSArray moveWordForward:]):
291         (-[NSArray moveWordForwardAndModifySelection:]):
292         (-[NSArray moveWordLeft:]):
293         (-[NSArray moveWordLeftAndModifySelection:]):
294         (-[NSArray moveWordRight:]):
295         (-[NSArray moveWordRightAndModifySelection:]):
296         (-[NSArray pageUp:]):
297         (-[NSArray pageDown:]):
298         (-[NSArray pageUpAndModifySelection:]):
299         (-[NSArray pageDownAndModifySelection:]):
300
301 2006-09-15  Adam Roben  <aroben@apple.com>
302
303         Reviewed by eseidel.
304
305         Fixes http://bugzilla.opendarwin.org/show_bug.cgi?id=10876
306         containsItemForURLUnicode uses matchLetter instead of
307         matchUnicodeLetter
308
309         Small fixes for _WebCoreHistoryProvider.
310
311         * History/WebHistory.m: Consistently use BUFFER_SIZE #define
312         (-[_WebCoreHistoryProvider containsItemForURLLatin1:length:]):
313         (-[_WebCoreHistoryProvider containsItemForURLUnicode:length:]):
314         Replace incorrect call to matchLetter to matchUnicodeLetter
315
316 2006-09-13  Brady Eidson <beidson@apple.com>
317
318         Reviewed by Maciej
319
320         Add infrastructure to support icon loads taking place in WebCore
321         Will remove WebKit icon loaders in a later patch
322
323         * WebCoreSupport/WebFrameBridge.m:
324         (-[WebFrameBridge notifyIconChanged:]):
325         (-[WebFrameBridge originalRequestURL]):
326         - Nuked two old, obsolete methods
327         - Added bridge for notifying of an icon change
328         - Added bridge for getting the "original request URL" which is still needed
329           until that info is available in the WebCore loaders
330
331         * WebView/WebDataSource.m:
332         (-[WebDataSource _loadIcon]):
333         - Empty body just for now, as it still gets called - next patch will prune all the old impl out
334
335 2006-09-13  Timothy Hatcher  <timothy@apple.com>
336
337         Reviewed by Tim O.
338
339         <rdar://problem/4726416> obsolete confusing "ADVISORY NOTE" comment in WebHistory.h
340
341         Removed the "ADVISORY NOTE: This method may change for the 1.0 SDK" comments.
342
343         * History/WebHistory.h:
344
345 === Safari-521.26 ===
346
347 2006-09-11  MorganL  <morlmor@yahoo.com>
348
349         Reviewed/landed by aroben.
350
351         Fixes http://bugzilla.opendarwin.org/show_bug.cgi?id=10765
352         Windows build busted due to std::copy usage in Vector.h
353
354         * WebKit.vcproj/WebKit.vcproj: Define _SCL_SECURE_NO_DEPRECATE to get
355         rid of deprecation warnings on std::copy
356
357 2006-09-11  Brady Eidson  <beidson@apple.com>
358
359         Despite the fact that some people built okay without this change, it sure was biting me, 
360         probably because I just wiped my build directory for a fresh build - 
361         Today's earlier removal of DOMList.h needed to occur in MigrateHeaders.make, as well
362
363         * MigrateHeaders.make: removed DOMList.h
364
365 2006-09-10  Darin Adler  <darin@apple.com>
366
367         Reviewed by Brady.
368
369         - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=10547
370           REGRESSION: Links that should open in a new window open in the same window,
371           while opening another blank window
372
373         * WebView/WebFrame.m:
374         (-[WebFrame _continueLoadRequestAfterNewWindowPolicy:frameName:formState:]):
375         Changed a mistaken "self" to "frame", which was the cause of the bug. Also
376         fixed the method so that it won't crash if createWebViewWithRequest does
377         something that has a side effect of releasing this WebView or this frame
378         by retaining "self" and "frame" as needed. Also fixed a problem where the
379         code to set "opener" was backwards, and would set the opener of the old
380         frame to point to the new frame instead of vice versa.
381
382 2006-09-09  Sam Weinig  <sam.weinig@gmail.com>
383
384         Reviewed by Eric.
385
386         Patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10795
387         Auto-generate the Objective-C DOM XPath bindings
388
389         * MigrateHeaders.make:
390
391 2006-09-09  Sam Weinig  <sam.weinig@gmail.com>
392
393         Reviewed by Eric.
394
395         Patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10791
396         Even More Objective-C DOM auto-generation cleanup
397
398         * MigrateHeaders.make:
399
400 2006-09-08  Tim Omernick  <timo@apple.com>
401
402         Reviewed by Brady Eidson.
403
404         Rolled out Maciej's code cleanup from 8/22.  It turns out that keeping the "loading" flag
405         is a useful optimization, as it avoids many Objective-C method calls while polling resources
406         for their load state.
407
408         This fixes a 3-4% PLT performance regression (as measured on my MacBook Pro).
409
410         * Loader/WebFrameLoader.m:
411         (-[WebFrameLoader addPlugInStreamLoader:]):
412         (-[WebFrameLoader removePlugInStreamLoader:]):
413         (-[WebFrameLoader addSubresourceLoader:]):
414         (-[WebFrameLoader removeSubresourceLoader:]):
415         * WebView/WebDataSource.m:
416         (-[WebDataSourcePrivate dealloc]):
417         (-[WebDataSource _prepareForLoadStart]):
418         (-[WebDataSource _setLoading:]):
419         (-[WebDataSource _updateLoading]):
420         (-[WebDataSource _startLoading]):
421         (-[WebDataSource _stopLoading]):
422         (-[WebDataSource _setPrimaryLoadComplete:]):
423         (-[WebDataSource isLoading]):
424         * WebView/WebDataSourceInternal.h:
425
426 2006-09-07  Sam Weinig  <sam.weinig@gmail.com>
427
428         Reviewed by Darin and Tim H.
429
430         Patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10774
431         Auto-generate the Objective-C DOM Traversal bindings
432
433         * MigrateHeaders.make:
434
435 2006-09-07  Sam Weinig  <sam.weinig@gmail.com>
436
437         Reviewed by Darin.
438
439         Patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10766
440         Auto-generate the Objective-C DOM Events bindings
441
442         * MigrateHeaders.make:
443         * WebKit.xcodeproj/project.pbxproj:
444
445 2006-09-06  Alexey Proskuryakov  <ap@nypop.com>
446
447         * MigrateHeaders.make: Fixed a double slash in 
448         "$(PRIVATE_HEADERS_DIR)//DOMCharacterDataPrivate.h" (this was reported
449         to cause a build failure under certain circumstances).
450
451 2006-09-05  MorganL  <morlmor@yahoocom>
452
453         Reviewed/landed by aroben.
454
455         Fixes bug 10743: Windows build is busted.
456
457         * COM/WebView.cpp:
458         (WebView::keyPress):
459
460 2006-09-05  Timothy Hatcher  <timothy@apple.com>
461
462         Reviewed by Darin.
463
464         Drastically simplified the makefile for migrating headers from
465         WebCore and JavaScriptCore. The old version was always copying the files.
466
467         * MigrateHeaders.make:
468
469 2006-09-05  Darin Adler  <darin@apple.com>
470
471         Reviewed by Alexey.
472
473         - WebKit side of changes to encoding
474
475         * WebView/WebHTMLRepresentation.m: (-[WebHTMLRepresentation documentSource]):
476         Changed to call new -[WebCoreFrameBridge stringWithData:] instead of the calling
477         the old methods that used a CFStringEncoding: -[WebCoreFrameBridge textEncoding]
478         and +[WebCoreFrameBridge stringWithData:textEncoding:].
479
480         * WebView/WebResource.m: (-[WebResource _stringValue]): Removed special case for
481         nil encoding name. The bridge itself now has the rule that "nil encoding name
482         means Latin-1", so we don't need to check for nil.
483
484         * WebView/WebFrame.m: (-[WebFrame _checkLoadComplete]): Retain the frame until
485         we get the parent frame while walking up parent frames, because it's possible
486         for _checkLoadCompleteForThisFrame to release the last reference to the frame.
487         (Not reviewed; needed to run performance tests successfully.)
488
489 2006-09-05  Timothy Hatcher  <timothy@apple.com>
490
491         Reviewed by Tim O.
492
493         <rdar://problem/4682225> conflicting typedefs in Netscape plug-in headers
494
495         * Plugins/npfunctions.h: fix the return type for NPN_IntFromIdentifierProcPtr to be int32_t
496
497 2006-09-04  Timothy Hatcher  <timothy@apple.com>
498
499         Reviewed by Darin.
500
501         Bug 10714: ObjC autogeneration needs safe-guards against easily modifying the public API
502         http://bugzilla.opendarwin.org/show_bug.cgi?id=10714
503
504         - Added the new private DOM headers.
505         - Factored out the common commands into variables.
506         - Made WebDashboardRegion.h private again.
507         - Rename DOMDOMImplementation.h to DOMImplementation.h when files
508           are migrated. Also fixes up #imports.
509
510         * MigrateHeaders.make:
511
512 2006-09-03  Sam Weinig  <sam.weinig@gmail.com>
513
514         Reviewed by Darin and Tim H.
515
516         - patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10711
517           Auto-generate the Objective-C DOM Stylesheet bindings
518
519         * MigrateHeaders.make:
520
521 2006-09-02  Sam Weinig  <sam.weinig@gmail.com>
522
523         Reviewed by Tim H.
524
525         - patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10684
526           Auto-generate the Objective-C DOM CSS bindings
527
528         * MigrateHeaders.make:
529
530 2006-09-01  MorganL  <morlmor@yahoo.com>
531
532         Reviewed by Darin. Updated/landed by Adam.
533
534         Fixes http://bugzilla.opendarwin.org/show_bug.cgi?id=10553
535         Windows build fixes
536
537         * COM/WebFrame.cpp:
538         (WebFrame::initWithName):
539
540 2006-09-01  Brady Eidson  <beidson@apple.com>
541
542         Reviewed by Darin
543
544         A "never should be reached" method was reached - lets not release the shared database bridge, esp
545         since we never retain it!
546
547         * Misc/WebIconDatabase.m:
548         (-[WebIconDatabase _applicationWillTerminate:]): Don't release the bridge
549
550 2006-09-01  Darin Adler  <darin@apple.com>
551
552         Reviewed by Brady.
553
554         - a few small tweaks to the icon database bridge
555
556         * Misc/WebIconDatabase.m: (-[WebIconDatabase init]): Updated for name change.
557
558         * WebCoreSupport/WebIconDatabaseBridge.h: Removed unneeded declarations.
559         * WebCoreSupport/WebIconDatabaseBridge.m:
560         (-[WebIconDatabaseBridge init]): Added. Always returns nil since you're not supposed
561         to allocate one of these.
562         (-[WebIconDatabaseBridge _init]): Renamed from init. Used internally to make the
563         shared instance. Added the "self = [super init]" idiom even though it's not important
564         in this case just to be consistent.
565         (-[WebIconDatabaseBridge releaseCachedLoaderForIconURL:]): Moved this up in the file
566         so it can be called without declaring it in the header.
567         (+[WebIconDatabaseBridge sharedInstance]): Renamed. Calls the new _init. Also use
568         CFRetain for compatibility.
569         (-[WebIconDatabaseBridge dealloc]): Emptied this out and made it just assert (false).
570         (-[WebIconDatabaseBridge finalize]): Added and made it assert (false) too.
571
572 2006-09-01  Timothy Hatcher  <timothy@apple.com>
573
574         Reviewed by Adele.
575
576         Bug 10677: Omit "-webkit-text-security: none;" from the computed style list
577         http://bugzilla.opendarwin.org/show_bug.cgi?id=10677
578
579         * WebInspector/webInspector/inspector.js:
580
581 2006-08-31  Sam Weinig  <sam.weinig@gmail.com>
582
583         Reviewed by Tim H.
584
585         - patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10669
586           Auto-generate the remaining Objective-C HTML DOM bindings
587
588         * MigrateHeaders.make:
589
590 2006-08-31  Adele Peterson  <adele@apple.com>
591
592         Reviewed by Darin.
593
594         WebKit part of fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=10666
595         Password: Disallow Spelling, Font, Speech, and Writing Direction context menu
596
597         * DefaultDelegates/WebDefaultContextMenuDelegate.m: (-[WebDefaultUIDelegate editingContextMenuItemsForElement:defaultMenuItems:]):
598           Check that the selection isn't in a password field before adding these items to the default editing context menu.
599               Search In Google, Search In Spotlight, Look up in Dictionary, Spelling, Font, Speech, Writing Direction
600         * WebView/WebHTMLView.m: (-[WebHTMLView _isSelectionInPasswordField]): Added.
601         * WebView/WebHTMLViewPrivate.h:
602
603 2006-08-31  Sam Weinig  <sam.weinig@gmail.com>
604
605         Reviewed by Tim H.
606
607         - patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10653
608           Auto-generate another 20 Objective-C DOM HTML bindings
609
610         * MigrateHeaders.make:
611         * WebKit.xcodeproj/project.pbxproj:
612
613 2006-08-31  Adele Peterson  <adele@apple.com>
614
615         Reviewed by John Sullivan.
616
617         Removed wkSecureEventInput and wkSetSecureEventInput, since this can be done with API.
618
619         * WebCoreSupport/WebSystemInterface.m: (InitWebCoreSystemInterface):
620
621 2006-08-31  John Sullivan  <sullivan@apple.com>
622
623         Reviewed by Tim Omernick
624         
625         - fixed <rdar://problem/4711200> Loading history would be faster if it bypassed 
626         NSURL API for local files
627
628         * History/WebHistory.m:
629         (-[WebHistoryPrivate _loadHistoryGutsFromURL:savedItemsCount:collectDiscardedItemsInto:error:]):
630         Load file URLs using [NSDictionary dictionaryWithContentsOfFile:]. I also cleaned up some minor
631         style issues in this method, and I removed the support for old NSArray-style history files (which we
632         stopped using before Safari 1.0).
633
634 2006-08-30  Adele Peterson  <adele@apple.com>
635
636         Reviewed by Hyatt.
637
638         WebKit part of fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=10575
639         Enable secure input mode for new password fields
640
641         * WebCoreSupport/WebSystemInterface.m: (InitWebCoreSystemInterface):
642
643 2006-08-30  Brady Eidson  <beidson@apple.com>
644
645         Reviewed by John
646
647         <rdar://problem/4707718> Change behavior so if the WebCore::IconDatabase can't open, WebKit releases the bridge and 
648         continues on as if the IconDatabase is disabled.
649  
650         * Misc/WebIconDatabase.m:
651         (-[WebIconDatabase init]): Release the bridge on failure to open
652         * WebCoreSupport/WebIconDatabaseBridge.m:
653         (+[WebIconDatabaseBridge sharedBridgeInstance]): Moved static shared instance out as a global
654         (-[WebIconDatabaseBridge dealloc]): Clear pointer to the shared instance
655
656 2006-08-30  Timothy Hatcher  <timothy@apple.com>
657
658         Reviewed by Darin.
659
660         Simplify the header copies from WebCore and JavaScriptCore.
661         Headers that need to be migrated from the other projects need
662         to be added to MigrateHeaders.make.
663
664         * MigrateHeaders.make: Added.
665         * WebKit.xcodeproj/project.pbxproj:
666
667 2006-08-30  Sam Weinig  <sam.weinig@gmail.com>
668
669         Reviewed by Tim H.
670
671         - patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10640
672           Auto-generate 10 more Objective-C DOM HTML bindings
673
674         * WebKit.xcodeproj/project.pbxproj:
675
676 2006-08-30  Adele Peterson  <adele@apple.com>
677
678         Reviewed by Darin.
679
680         WebKit part of fix for:
681         http://bugzilla.opendarwin.org/show_bug.cgi?id=10576
682         Disallow copy from new password fields
683
684         * WebView/WebHTMLView.m:
685         (-[WebHTMLView _canCopy]): Now also calls across the bridge to ask if it mayCopy.
686         (-[WebHTMLView _canCut]): Calls _canCopy now.
687         (-[NSArray validateUserInterfaceItem:]): Calls _canCut when validating the "Cut" menu item.
688          This used to call _canDelete (which used to be the same as _canCut), but now _canCut also checks _canCopy.
689
690 2006-08-30  Karl Adam  <karladam@yahoo-inc.com>
691
692         Reviewed by Eric and Tim H.
693
694         Bug 10634: -webView:dragDestinationActionMaskForDraggingInfo: is ignored
695         http://bugzilla.opendarwin.org/show_bug.cgi?id=10634
696         
697         Remove the check for canShowFile: from _web_bestURL: since
698         it shouldn't be concerned with whether or not the view
699         can show the URL, merely return the most appropriate URL.
700
701         * Misc/WebNSPasteboardExtras.m:
702         (-[NSPasteboard _web_bestURL]):
703         * Misc/WebNSViewExtras.m:
704         (-[NSView _web_dragOperationForDraggingInfo:]):
705
706 2006-08-29  Brady Eidson  <beidson@apple.com>
707
708         Reviewed by Kevin Decker (Sarge)
709
710         <rdar://problem/4678414> - New IconDB needs to delete icons when asked
711
712         * Misc/WebIconDatabase.m:
713         (-[WebIconDatabase removeAllIcons]): Call through to WebCore to remove icons, then send notification
714
715 2006-08-29  Brady Eidson  <beidson@apple.com>
716
717         Reviewed by Alice
718
719         Added a truth value check for to setIconURL:forURL so WebKit can avoid sending a notification
720         This is a win on the iBench
721
722         * Misc/WebIconDatabase.m:
723         (-[WebIconDatabase _setIconURL:forURL:]):
724
725 2006-08-29  Brady Eidson  <beidson@apple.com>
726
727         Reviewed by Tim Hatchers rubber stamp
728
729         Removed some accidentally left-in console spew during the conversion to the new DB
730  
731         * Misc/WebIconDatabase.m:
732         (objectFromPathForKey): Nuked some NSLogs
733
734 2006-08-29  Tim Omernick  <timo@apple.com>
735
736         Reviewed by Darin Adler.
737
738         <rdar://problem/4688618> REGRESSION(10.4.7-9A241): JMol java applet fails in Safari not Firefox
739
740         No layout test for now because Java doesn't work in DumpRenderTree.
741
742         * Plugins/WebPluginDatabase.m:
743         (-[WebPluginDatabase refresh]):
744         Add each plug-in MIME type to registeredMIMETypes, even if we don't register a document view class for the
745         MIME type.  This fixes -[WebPluginDatabase isMIMETypeRegistered:] and thus fallback content for Java applets
746         (we were always rendering fallback content, if any, for Java applets).
747
748 2006-08-29  Sam Weinig  <sam.weinig@gmail.com>
749
750         Reviewed by Tim H.
751
752         - patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10628
753           Auto-generate the remaining Objective-C DOM bindings
754
755           Auto-generates DOMHTMLCollection, DOMHTMLElement, DOMHTMLFormElement,
756           and DOMHTMLOptionsCollection.
757
758         * WebKit.xcodeproj/project.pbxproj:
759
760 2006-08-28  Brady Eidson  <beidson@apple.com>
761
762         Reviewed by Darin
763
764         Short of a few small snippets that still need to be pushed to WebCore, this
765         is a final prune of WebIconDatabase.  WebFileDatabase and WebLRUFileList are gone
766         and the small remaining snippets of WebFileDatabase code that were still important
767         are now in static functions in WebIconDatabase.m
768
769         * Loader/WebIconLoader.m:
770         (-[WebIconLoader didFinishLoading]): Removed the flag
771         * Misc/WebFileDatabase.h: Removed.
772         * Misc/WebFileDatabase.m: Removed.
773         * Misc/WebIconDatabase.m:
774         (+[WebIconDatabase sharedIconDatabase]):
775         (-[WebIconDatabase init]):
776         (-[WebIconDatabase iconForURL:withSize:cache:]):
777         (-[WebIconDatabase iconURLForURL:]):
778         (-[WebIconDatabase defaultIconWithSize:]):
779         (-[WebIconDatabase retainIconForURL:]):
780         (-[WebIconDatabase releaseIconForURL:]):
781         (-[WebIconDatabase _isEnabled]):
782         (-[WebIconDatabase _setIconData:forIconURL:]):
783         (-[WebIconDatabase _setHaveNoIconForIconURL:]):
784         (-[WebIconDatabase _setIconURL:forURL:]):
785         (-[WebIconDatabase _hasEntryForIconURL:]):
786         (-[WebIconDatabase _applicationWillTerminate:]):
787         (-[WebIconDatabase _resetCachedWebPreferences:]):
788         (uniqueFilePathForKey): Added from WebFileDatabase
789         (objectFromPathForKey): Added from WebFileDatabase
790         (iconDataFromPathForIconURL):
791         (-[WebIconDatabase _convertToWebCoreFormat]): Make use of static functions and local variables
792           instead of using WebFileDatabase and WebIconDatabase variables that are now obsolete
793         * Misc/WebIconDatabasePrivate.h: Removed alot of obsoleted members
794         * Misc/WebLRUFileList.h: Removed.
795         * Misc/WebLRUFileList.m: Removed.
796         * WebKit.xcodeproj/project.pbxproj: Deleted 4 files
797         * WebKitPrefix.h: Removed ICONDEBUG
798
799 2006-08-28  Tim Omernick  <timo@apple.com>
800
801         Reviewed by Darin Adler.
802
803         * Plugins/WebBaseNetscapePluginView.m:
804         (-[WebBaseNetscapePluginView createPluginScriptableObject]):
805         Removed a bogus typecast.
806
807 2006-08-28  Tim Omernick  <timo@apple.com>
808
809         Reviewed by John Sullivan.
810
811         Part of <rdar://problem/4481553> NetscapeMoviePlugIn example code scripting doesn't work in Firefox (4319)
812         <http://bugzilla.opendarwin.org/show_bug.cgi?id=4319>: NetscapeMoviePlugIn example code scripting doesn't work
813         in Firefox
814
815         * Plugins/WebBaseNetscapePluginView.h:
816         * Plugins/WebBaseNetscapePluginView.m:
817         (-[WebBaseNetscapePluginView createPluginScriptableObject]):
818         Renamed this method (see corresponding WebCore ChangeLog entry for an explanation).
819         Style changes.
820
821 2006-08-28  Brady Eidson  <beidson@apple.com>
822
823         Reviewed by Tim Hatcher's rubberstamp
824
825         Rolled out my last change (16070 - pruning WebFileDatabase code) as it caused a difficult-to-track down
826         failure in layout tests on a release build.
827
828         * Misc/WebFileDatabase.h:
829         * Misc/WebFileDatabase.m:
830         (+[WebFileDatabaseOp opWithCode:key:object:]):
831         (-[WebFileDatabaseOp initWithCode:key:object:]):
832         (-[WebFileDatabaseOp opcode]):
833         (-[WebFileDatabaseOp key]):
834         (-[WebFileDatabaseOp object]):
835         (-[WebFileDatabaseOp perform:]):
836         (-[WebFileDatabaseOp dealloc]):
837         (SetThreadPriority):
838         (-[WebFileDatabase _createLRUList:]):
839         (-[WebFileDatabase _truncateToSizeLimit:]):
840         (+[WebFileDatabase _syncLoop:]):
841         (databaseInit):
842         (-[WebFileDatabase setTimer]):
843         (-[WebFileDatabase setObject:forKey:]):
844         (-[WebFileDatabase removeObjectForKey:]):
845         (-[WebFileDatabase removeAllObjects]):
846         (-[WebFileDatabase objectForKey:]):
847         (-[WebFileDatabase performSetObject:forKey:]):
848         (-[WebFileDatabase performRemoveObjectForKey:]):
849         (-[WebFileDatabase open]):
850         (-[WebFileDatabase close]):
851         (-[WebFileDatabase lazySync:]):
852         (-[WebFileDatabase sync]):
853         (-[WebFileDatabase sizeLimit]):
854         (-[WebFileDatabase count]):
855         (-[WebFileDatabase usage]):
856         (-[WebFileDatabase setSizeLimit:]):
857         * Misc/WebIconDatabase.m:
858         (-[WebIconDatabase _createFileDatabase]):
859         (-[WebIconDatabase _loadIconDictionaries]):
860         * WebKit.xcodeproj/project.pbxproj:
861
862 2006-08-28  Darin Adler  <darin@apple.com>
863
864         Reviewed by Tim Hatcher.
865
866         * WebInspector/webInspector/inspector.js: Add "resize: none" to the list of
867         default values for CSS properties so it will be omitted from most displays
868         of computed style.
869
870 2006-08-28  Brady Eidson  <beidson@apple.com>
871
872         Reviewed by Maciej
873
874         Major prune of unnecessary WebFileDatabase code.  In the end, what
875         useful code that remains in WebFileDatabase will likely be moved directly
876         into WebIconDatabase
877
878         * Misc/WebFileDatabase.h:
879         * Misc/WebFileDatabase.m:
880         (-[WebFileDatabase initWithPath:]):
881         (-[WebFileDatabase objectForKey:]):
882         (-[WebFileDatabase open]):
883         (-[WebFileDatabase close]):
884         * Misc/WebIconDatabase.m:
885         (-[WebIconDatabase _createFileDatabase]):
886         (-[WebIconDatabase _loadIconDictionaries]):
887         * Misc/WebLRUFileList.h: Removed.
888         * Misc/WebLRUFileList.m: Removed.
889         * WebKit.xcodeproj/project.pbxproj:
890
891 2006-08-27  Sam Weinig  <sam.weinig@gmail.com>
892
893         Reviewed by Tim H.
894
895         - patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=4624
896           WebCore needs autogenerated Obj-C DOM bindings
897
898           First round of auto-generated Objective C DOM bindings, starting
899           with the DOM Core.
900
901         * WebKit.xcodeproj/project.pbxproj:
902
903 2006-08-25  Brady Eidson  <beidson@apple.com>
904
905         Reviewed by Tim Hatcher
906
907         Fixed up some leaks on [WebIconDatabase init]
908
909         * Misc/WebIconDatabase.m:
910         (-[WebIconDatabase init]):
911
912 2006-08-24  Timothy Hatcher  <timothy@apple.com>
913
914         Reviewed by Darin.
915
916         * WebView/WebFrame.m:
917         (-[WebFrame _updateBackground]): reworded the comment about scroll view and setDrawsBackground:YES
918         * WebView/WebView.m:
919         (-[WebViewPrivate dealloc]): release the background color
920
921 2006-08-24  Timothy Hatcher  <timothy@apple.com>
922
923         Reviewed by Hyatt.
924
925         WebView API to allow changing the background color that draws under transparent page backgrounds.
926
927         * WebView/WebFrame.m:
928         (-[WebFrame _makeDocumentView]):
929         (-[WebFrame _updateBackground]):
930         * WebView/WebFrameInternal.h:
931         * WebView/WebFrameView.m:
932         (-[WebFrameView drawRect:]):
933         * WebView/WebView.m:
934         (-[WebView setBackgroundColor:]):
935         (-[WebView backgroundColor]):
936         (-[WebView _commonInitializationWithFrameName:groupName:]):
937         (-[WebView setDrawsBackground:]):
938         * WebView/WebViewPrivate.h:
939
940 2006-08-24  Timothy Hatcher  <timothy@apple.com>
941
942         Reviewed by Darin.
943
944         Fix for Mail stationary. Selecting a stationary item would cause this exception.
945         *** -[WebSubresourceLoader copyWithZone:]: selector not recognized
946         Uncaught exception - *** -[WebSubresourceLoader copyWithZone:]: selector not recognized
947
948         * Loader/WebFrameLoader.m:
949         (-[WebFrameLoader willUseArchiveForRequest:originalURL:loader:]):
950         Use _webkit_setObject:forUncopiedKey: when addign the resource to pendingArchivedResources.
951
952 2006-08-23  Maciej Stachowiak  <mjs@apple.com>
953
954         Reviewed by Adele.
955         
956         - fix assertion which I accidentally changed to one that sometimes fails
957         http://bugzilla.opendarwin.org/show_bug.cgi?id=10531
958         
959         * Loader/WebDocumentLoadState.m:
960         (-[WebDocumentLoadState dealloc]):
961         * WebView/WebDataSource.m:
962         (-[WebDataSourcePrivate dealloc]):
963         (-[WebDataSource dealloc]):
964
965 2006-08-23  Maciej Stachowiak  <mjs@apple.com>
966
967         Reviewed by Brady.
968         
969         - move WebFrame code that creates WebDataSources down to WebFrameLoader, in preparation
970         for WebFrameLoader just holding on to WebDocumentLoadState
971
972         * Loader/WebFrameLoader.h:
973         * Loader/WebFrameLoader.m:
974         (-[WebFrameLoader _loadRequest:archive:]):
975         (-[WebFrameLoader _loadRequest:triggeringAction:loadType:formState:]):
976         (-[WebFrameLoader _reloadAllowingStaleDataWithOverrideEncoding:]):
977         (-[WebFrameLoader reload]):
978         * WebView/WebFrame.m:
979         (-[WebFrame _loadItem:withLoadType:]):
980         (-[WebFrame _continueLoadRequestAfterNewWindowPolicy:frameName:formState:]):
981         (-[WebFrame _loadURL:referrer:loadType:target:triggeringEvent:form:formValues:]):
982         (-[WebFrame _postWithURL:referrer:target:data:contentType:triggeringEvent:form:formValues:]):
983         (-[WebFrame _shouldReloadToHandleUnreachableURLFromRequest:]):
984         (-[WebFrame loadRequest:]):
985         (-[WebFrame loadArchive:]):
986         (-[WebFrame reload]):
987         * WebView/WebFrameInternal.h:
988         * WebView/WebFramePrivate.h:
989         * WebView/WebView.m:
990         (-[WebView setCustomTextEncodingName:]):
991
992 2006-08-23  Brady Eidson  <beidson@apple.com>
993
994         Reviewed by Maciej
995
996         First pass at pruning unused WebIconDatabase code.  Focus on removing methods that simply
997         have no place in the new DB at all.  A few renames and a few important FIXMEs result, but no
998         functionality changes.
999
1000         * Loader/WebIconLoader.m:
1001         (-[WebIconLoader didFinishLoading]): Call to WebIconDatabase instead of directly to the bridge
1002         * Misc/WebIconDatabase.h:
1003         * Misc/WebIconDatabase.m:
1004         (-[WebIconDatabase init]):
1005         (-[WebIconDatabase iconForURL:withSize:cache:]):
1006         (-[WebIconDatabase iconURLForURL:]):
1007         (-[WebIconDatabase defaultIconWithSize:]):
1008         (-[WebIconDatabase retainIconForURL:]):
1009         (-[WebIconDatabase releaseIconForURL:]):
1010         (-[WebIconDatabase removeAllIcons]):
1011         (-[WebIconDatabase _setIconData:forIconURL:]):
1012         (-[WebIconDatabase _setHaveNoIconForIconURL:]):
1013         (-[WebIconDatabase _setIconURL:forURL:]):
1014         (-[WebIconDatabase _hasEntryForIconURL:]):
1015         (-[WebIconDatabase _applicationWillTerminate:]):
1016         (-[WebIconDatabase _resetCachedWebPreferences:]):
1017         * Misc/WebIconDatabasePrivate.h: Changed setIcon: to setIconData:
1018
1019 2006-08-23  Maciej Stachowiak  <mjs@apple.com>
1020
1021         Reviewed by Adele.
1022         
1023         - started factoring most of the guts of WebDataSource into a new class
1024         
1025         WebDocumentLoadState is decoupled from the rest of WebKit and will
1026         be moved down to WebCore. I only moved one of the data fields of
1027         WebDataSource for now.
1028
1029         * Loader/WebDocumentLoadState.h: Added.
1030         * Loader/WebDocumentLoadState.m: Added.
1031         (-[WebDocumentLoadState initWithRequest:]): New class.
1032         (-[WebDocumentLoadState dealloc]):
1033         (-[WebDocumentLoadState setFrameLoader:]):
1034         (-[WebDocumentLoadState setMainResourceData:]):
1035         (-[WebDocumentLoadState mainResourceData]):
1036         * Loader/WebFrameLoader.m:
1037         (-[WebFrameLoader _setDataSource:]): Remove redundant _setWebFrame: call,
1038         it would have been called already by this point.
1039         (-[WebFrameLoader _setProvisionalDataSource:]): ditto.
1040         * WebKit.xcodeproj/project.pbxproj:
1041         * WebView/WebDataSource.m:
1042         (-[WebDataSourcePrivate dealloc]):
1043         (-[WebDataSource _setWebFrame:]):
1044         (-[WebDataSource _setPrimaryLoadComplete:]):
1045         (-[WebDataSource initWithRequest:]):
1046         (-[WebDataSource data]):
1047
1048 2006-08-23  Maciej Stachowiak  <mjs@apple.com>
1049
1050         Reviewed by Adele.
1051         
1052         - refactoring of WebDataSource in preparation for moving a bunch of it to a new class
1053         
1054         - minimize WebDataSourceInternal.h to be only methods called from outside WebDataSource
1055         - reduce Private category implementation to be only the SPI methods
1056         - make new Internal and FileInternal categories which contain the
1057         remainder (depending on whether they are called from outside of
1058         WebDataSource)
1059
1060         * WebView/WebDataSource.m:
1061         (-[WebDataSource _setMainDocumentError:]):
1062         (addTypesFromClass):
1063         (+[WebDataSource _representationClassForMIMEType:]):
1064         (-[WebDataSource _commitIfReady]):
1065         (-[WebDataSource _commitLoadWithData:]):
1066         (-[WebDataSource _doesProgressiveLoadWithMIMEType:]):
1067         (-[WebDataSource _addResponse:]):
1068         (-[WebDataSource _revertToProvisionalState]):
1069         (-[WebDataSource _mainDocumentError]):
1070         (-[WebDataSource _addSubframeArchives:]):
1071         (-[WebDataSource _fileWrapperForURL:]):
1072         (+[WebDataSource _repTypesAllowImageTypeOmission:]):
1073         (-[WebDataSource _decidePolicyForMIMEType:decisionListener:]):
1074         (-[WebDataSource _finishedLoading]):
1075         (-[WebDataSource _setResponse:]):
1076         (-[WebDataSource _setRequest:]):
1077         (-[WebDataSource _setupForReplaceByMIMEType:]):
1078         (-[WebDataSource _receivedMainResourceError:complete:]):
1079         (-[WebDataSource _mainReceivedError:complete:]):
1080         (-[WebDataSource _defersCallbacks]):
1081         (-[WebDataSource _downloadWithLoadingConnection:request:response:proxy:]):
1082         (-[WebDataSource _didFailLoadingWithError:forResource:]):
1083         (-[WebDataSource _didFinishLoadingForResource:]):
1084         (-[WebDataSource _didReceiveData:contentLength:forResource:]):
1085         (-[WebDataSource _didReceiveResponse:forResource:]):
1086         (-[WebDataSource _didCancelAuthenticationChallenge:forResource:]):
1087         (-[WebDataSource _didReceiveAuthenticationChallenge:forResource:]):
1088         (-[WebDataSource _willSendRequest:forResource:redirectResponse:]):
1089         (-[WebDataSource _identifierForInitialRequest:]):
1090         (-[WebDataSource _archivedSubresourceForURL:]):
1091         (-[WebDataSource _startLoading]):
1092         (-[WebDataSource _stopRecordingResponses]):
1093         (-[WebDataSource _loadingStartedTime]):
1094         (-[WebDataSource _replaceSelectionWithArchive:selectReplacement:]):
1095         (-[WebDataSource _documentFragmentWithArchive:]):
1096         (-[WebDataSource _documentFragmentWithImageResource:]):
1097         (-[WebDataSource _imageElementWithImageResource:]):
1098         (-[WebDataSource _title]):
1099         (-[WebDataSource _isStopping]):
1100         (-[WebDataSource _setWebFrame:]):
1101         (-[WebDataSource _URL]):
1102         (-[WebDataSource _loadFromPageCache:]):
1103         (-[WebDataSource _popSubframeArchiveWithName:]):
1104         (-[WebDataSource _setIsClientRedirect:]):
1105         (-[WebDataSource _setURL:]):
1106         (-[WebDataSource _setLastCheckedRequest:]):
1107         (-[WebDataSource _defersCallbacksChanged]):
1108         (-[WebDataSource _lastCheckedRequest]):
1109         (-[WebDataSource _stopLoading]):
1110         (-[WebDataSource _bridge]):
1111         (-[WebDataSource _webView]):
1112         (-[WebDataSource _triggeringAction]):
1113         (-[WebDataSource _setTriggeringAction:]):
1114         (-[WebDataSource __adoptRequest:]):
1115         (-[WebDataSource _isDocumentHTML]):
1116         (-[WebDataSource _stopLoadingWithError:]):
1117         (-[WebDataSource _setPrimaryLoadComplete:]):
1118         (-[WebDataSource _responses]):
1119         (-[WebDataSource _makeRepresentation]):
1120         (-[WebDataSource _isClientRedirect]):
1121         (-[WebDataSource _originalRequest]):
1122         (-[WebDataSource _URLForHistory]):
1123         (-[WebDataSource _addToUnarchiveState:]):
1124         (-[WebDataSource _setOverrideEncoding:]):
1125         (-[WebDataSource _setIconURL:]):
1126         (-[WebDataSource _setIconURL:withType:]):
1127         (-[WebDataSource _overrideEncoding]):
1128         (-[WebDataSource _setTitle:]):
1129         * WebView/WebDataSourceInternal.h:
1130
1131 2006-08-23  Brady Eidson  <beidson@apple.com>
1132
1133         Reviewed by John Sullivan
1134
1135         /me crosses fingers
1136         Flip the switch to the new Icon Database
1137         Massive code pruning is coming up
1138
1139         * WebKitPrefix.h: Flipped the switch
1140
1141 2006-08-23  Brady Eidson  <beidson@apple.com>
1142
1143         Reviewed by Beth
1144         Since I just pushed the default URL icon from WebKit to WebCore, but
1145         WebKit was still using its version of the default icon, this patch makes it
1146         use the WebCore version
1147
1148         * Misc/WebIconDatabase.m:
1149         (-[WebIconDatabase defaultIconWithSize:]):
1150
1151 2006-08-22  Maciej Stachowiak  <mjs@apple.com>
1152
1153         Reviewed by Brady.
1154         
1155         - remove "loading" flag from WebDataSource and code that manages it; it is redundat.
1156
1157         * Loader/WebFrameLoader.m:
1158         (-[WebFrameLoader addPlugInStreamLoader:]):
1159         (-[WebFrameLoader removePlugInStreamLoader:]):
1160         (-[WebFrameLoader addSubresourceLoader:]):
1161         (-[WebFrameLoader removeSubresourceLoader:]):
1162         * WebView/WebDataSource.m:
1163         (-[WebDataSourcePrivate dealloc]):
1164         (-[WebDataSource _setPrimaryLoadComplete:]):
1165         (-[WebDataSource _stopLoading]):
1166         (-[WebDataSource _prepareForLoadStart]):
1167         (-[WebDataSource _startLoading]):
1168         (-[WebDataSource isLoading]):
1169         * WebView/WebDataSourceInternal.h:
1170
1171 2006-08-22  Maciej Stachowiak  <mjs@apple.com>
1172
1173         Reviewed by Eric.
1174
1175         <rdar://problem/4683948> REGRESSION: Assertion failure in [FrameProgressEntry addChild:forDataSource:] (fandango.com)
1176         
1177         * Loader/WebFrameLoader.m:
1178         (-[WebFrameLoader addSubresourceLoader:]): Add a WebKit-level assertion that should fire when this bad
1179         situation occurs.
1180         * Loader/WebSubresourceLoader.m:
1181         (+[WebSubresourceLoader startLoadingResource:withRequest:customHeaders:referrer:forFrameLoader:]): Prevent
1182         the situation from occuring.
1183
1184 2006-08-22  Maciej Stachowiak  <mjs@apple.com>
1185
1186         Reviewed by Anders.
1187         
1188         - pull more WebDataSource code into WebFrameLoader
1189         - make WebMainResourceLoader not depend on WebKit or on SPI
1190
1191         * Loader/WebFrameLoader.h:
1192         * Loader/WebFrameLoader.m:
1193         (-[WebFrameLoader _downloadWithLoadingConnection:request:response:proxy:]):
1194         (-[WebFrameLoader _updateIconDatabaseWithURL:]):
1195         (-[WebFrameLoader _notifyIconChanged:]):
1196         (-[WebFrameLoader _iconLoaderReceivedPageIcon:]):
1197         (-[WebFrameLoader _checkNavigationPolicyForRequest:andCall:withSelector:]):
1198         (-[WebFrameLoader _checkContentPolicyForMIMEType:andCall:withSelector:]):
1199         (-[WebFrameLoader cancelContentPolicy]):
1200         * Loader/WebIconLoader.m:
1201         (-[WebIconLoader didFinishLoading]):
1202         * Loader/WebLoader.m:
1203         (-[NSURLProtocol loadWithRequest:]):
1204         (-[NSURLProtocol setDefersCallbacks:]):
1205         * Loader/WebMainResourceLoader.h:
1206         * Loader/WebMainResourceLoader.m:
1207         (-[WebMainResourceLoader initWithFrameLoader:]):
1208         (-[WebMainResourceLoader dealloc]):
1209         (-[WebMainResourceLoader cancelWithError:]):
1210         (-[WebMainResourceLoader continueAfterNavigationPolicy:formState:]):
1211         (-[WebMainResourceLoader willSendRequest:redirectResponse:]):
1212         (-[WebMainResourceLoader continueAfterContentPolicy:]):
1213         (-[WebMainResourceLoader checkContentPolicy]):
1214         (-[WebMainResourceLoader didReceiveResponse:]):
1215         * Plugins/WebPluginContainerCheck.m:
1216         * WebCoreSupport/WebSystemInterface.m:
1217         (InitWebCoreSystemInterface):
1218         * WebView/WebDataSource.m:
1219         (-[WebDataSource _loadIcon]):
1220         (-[WebDataSource _cancelledError]):
1221         (+[WebDataSource _repTypesAllowImageTypeOmission:]):
1222         (+[WebDataSource _representationClassForMIMEType:]):
1223         (-[WebDataSource _commitLoadWithData:]):
1224         (-[WebDataSource _isDocumentHTML]):
1225         * WebView/WebDataSourceInternal.h:
1226         * WebView/WebFramePrivate.h:
1227         * WebView/WebPolicyDelegate.m:
1228         * WebView/WebPolicyDelegatePrivate.h:
1229
1230 2006-08-21  Maciej Stachowiak  <mjs@apple.com>
1231
1232         Reviewed by Brady.
1233         
1234         - move more code from WebDataSource to WebFrameLoader
1235         
1236         Also marked a few more methods in WebDataSource as likely MOVABLE in a future round, since they do not
1237         use any of WebDataSource's private data.
1238
1239         * Loader/WebFrameLoader.h:
1240         * Loader/WebFrameLoader.m:
1241         (-[WebFrameLoader addPlugInStreamLoader:]):
1242         (-[WebFrameLoader removePlugInStreamLoader:]):
1243         (-[WebFrameLoader addSubresourceLoader:]):
1244         (-[WebFrameLoader removeSubresourceLoader:]):
1245         (-[WebFrameLoader _didReceiveAuthenticationChallenge:forResource:]):
1246         (-[WebFrameLoader _didCancelAuthenticationChallenge:forResource:]):
1247         (-[WebFrameLoader _didReceiveResponse:forResource:]):
1248         (-[WebFrameLoader _didReceiveData:contentLength:forResource:]):
1249         (-[WebFrameLoader _didFinishLoadingForResource:]):
1250         (-[WebFrameLoader _didFailLoadingWithError:forResource:]):
1251         (-[WebFrameLoader _privateBrowsingEnabled]):
1252         (-[WebFrameLoader _finishedLoadingResource]):
1253         (-[WebFrameLoader _receivedError:]):
1254         (-[WebFrameLoader _finishedLoading]):
1255         * Loader/WebMainResourceLoader.m:
1256         (-[WebMainResourceLoader didReceiveData:lengthReceived:allAtOnce:]):
1257         (-[WebMainResourceLoader didFinishLoading]):
1258         * Loader/WebNetscapePlugInStreamLoader.m:
1259         (-[WebNetscapePlugInStreamLoader didFinishLoading]):
1260         (-[WebNetscapePlugInStreamLoader didFailWithError:]):
1261         (-[WebNetscapePlugInStreamLoader cancelWithError:]):
1262         * Loader/WebSubresourceLoader.m:
1263         (+[WebSubresourceLoader startLoadingResource:withRequest:customHeaders:referrer:forFrameLoader:]):
1264         (-[WebSubresourceLoader signalFinish]):
1265         (-[WebSubresourceLoader didFailWithError:]):
1266         (-[WebSubresourceLoader cancel]):
1267         * Plugins/WebNetscapePluginStream.m:
1268         (-[WebNetscapePluginStream start]):
1269         * WebView/WebDataSource.m:
1270         (-[WebDataSource _replaceSelectionWithArchive:selectReplacement:]):
1271         (-[WebDataSource _updateIconDatabaseWithURL:]):
1272         (-[WebDataSource _loadIcon]):
1273         (-[WebDataSource _setPrimaryLoadComplete:]):
1274         (-[WebDataSource _cancelledError]):
1275         (+[WebDataSource _repTypesAllowImageTypeOmission:]):
1276         (+[WebDataSource _representationClassForMIMEType:]):
1277         (-[WebDataSource _commitLoadWithData:]):
1278         (-[WebDataSource _receivedMainResourceError:complete:]):
1279         (-[WebDataSource _iconLoaderReceivedPageIcon:]):
1280         (-[WebDataSource _isDocumentHTML]):
1281         * WebView/WebDataSourceInternal.h:
1282
1283 2006-08-21  Brady Eidson  <beidson@apple.com>
1284
1285         Reviewed by John
1286
1287         Quick ICONDEBUG flag fix
1288
1289         * Misc/WebIconDatabase.m:
1290         (-[WebIconDatabase _applicationWillTerminate:]):
1291
1292 2006-08-21  Brady Eidson  <beidson@apple.com>
1293
1294         Reviewed by Anders
1295
1296         -Renamed an internal only method for clarity
1297         -Tweaked WebDataSource for notification purposes with the new expiring icons
1298         -Fixed a bug with the ICONDEBUG flag
1299
1300         * Misc/WebIconDatabase.m:
1301         (-[WebIconDatabase init]):
1302         (-[WebIconDatabase iconForURL:withSize:cache:]): #ifdef bug fixed
1303         (-[WebIconDatabase _hasEntryForIconURL:]): Renamed for clarity
1304         * Misc/WebIconDatabasePrivate.h:
1305         * WebView/WebDataSource.m:
1306         (-[WebDataSource _updateIconDatabaseWithURL:]):
1307         (-[WebDataSource _notifyIconChanged:]):
1308         (-[WebDataSource _loadIcon]):
1309         (-[WebDataSource _iconLoaderReceivedPageIcon:]):
1310
1311 2006-08-17  Maciej Stachowiak  <mjs@apple.com>
1312
1313         Reviewed by Brady.
1314         
1315         - pull a bit of WebDataSource code into WebFrameLoader
1316
1317         * Loader/WebFrameLoader.m:
1318         (-[WebFrameLoader _receivedError:]):
1319         (-[WebFrameLoader webFrame]):
1320         (-[WebFrameLoader _handleFallbackContent]):
1321         (+[WebFrameLoader _canShowMIMEType:]):
1322         (+[WebFrameLoader _representationExistsForURLScheme:]):
1323         (+[WebFrameLoader _generatedMIMETypeForURLScheme:]):
1324         * WebView/WebDataSource.m:
1325         * WebView/WebDataSourceInternal.h:
1326
1327 2006-08-17  Timothy Hatcher  <timothy@apple.com>
1328
1329         Reviewed by Kevin Decker.
1330
1331         <rdar://problem/4606857> WebKit: WebPreferencesChangedNotification not exported in 64-bit
1332
1333         * WebKit.LP64.exp:
1334
1335 2006-08-17  Timothy Hatcher  <timothy@apple.com>
1336
1337         Reviewed by Kevin Decker.
1338
1339         <rdar://problem/4633896> -[WebView close] should clear all delegates and call setHostWindow:nil
1340         <rdar://problem/4649759> Crash when selecting View Source menu using Chinese (-[WebView _close])
1341
1342         Check to make sure _private is not null. A WebView can be dealloced before _private is setup.
1343         Set the _private->closed flag at the beginning of _close to prevent reentry.
1344         Set the host window and all the delegates to nil in _close.
1345
1346         * WebView/WebView.m:
1347         (-[WebView _close]):
1348
1349 2006-08-16  Brady Eidson  <beidson@apple.com>
1350
1351         Reviewed by Maciej
1352
1353         * Misc/WebIconDatabase.m: Pruned unused method
1354         * Misc/WebIconDatabasePrivate.h: Ditto
1355
1356
1357 2006-08-15  Maciej Stachowiak  <mjs@apple.com>
1358
1359         Reviewed by Adele (preliminary version) and later by Kevin.
1360         
1361         - remove most WebKit dependencies from WebMainResourceLoader.
1362
1363         * Loader/WebFrameLoader.h:
1364         * Loader/WebFrameLoader.m:
1365         (-[WebFrameLoader cannotShowMIMETypeForURL:]):
1366         (-[WebFrameLoader interruptForPolicyChangeErrorWithRequest:]):
1367         (-[WebFrameLoader isHostedByObjectElement]):
1368         (-[WebFrameLoader isLoadingMainFrame]):
1369         (+[WebFrameLoader _canShowMIMEType:]):
1370         (+[WebFrameLoader _representationExistsForURLScheme:]):
1371         (+[WebFrameLoader _generatedMIMETypeForURLScheme:]):
1372         * Loader/WebMainResourceLoader.h:
1373         * Loader/WebMainResourceLoader.m:
1374         (-[WebMainResourceLoader interruptForPolicyChangeError]):
1375         (-[WebMainResourceLoader willSendRequest:redirectResponse:]):
1376         (isCaseInsensitiveEqual):
1377         (shouldLoadAsEmptyDocument):
1378         (-[WebMainResourceLoader continueAfterContentPolicy:response:]):
1379         (-[WebMainResourceLoader didReceiveResponse:]):
1380         (-[WebMainResourceLoader didReceiveData:lengthReceived:allAtOnce:]):
1381         (-[WebMainResourceLoader didFinishLoading]):
1382         (-[WebMainResourceLoader loadWithRequestNow:]):
1383         (-[WebMainResourceLoader loadWithRequest:]):
1384
1385 2006-08-15  Tim Omernick  <timo@apple.com>
1386
1387         Reviewed by Kevin Decker.
1388
1389         * Plugins/WebNetscapePluginPackage.m:
1390         (-[WebNetscapePluginPackage _unloadWithShutdown:]):
1391         Fixed a subtle problem with the 64-bit debug build -- as written, this would LOG() on 64-bit and do nothing on
1392         32-bit!
1393         * Plugins/WebPluginDatabase.m:
1394         (-[WebPluginDatabase _scanForNewPlugins]):
1395         Use +[NSMutableSet set] here.
1396
1397 2006-08-15  Tim Omernick  <timo@apple.com>
1398
1399         Reviewed by Darin Adler.
1400
1401         <http://bugzilla.opendarwin.org/show_bug.cgi?id=8980>
1402         ASSERTION FAILED: !isLoaded (WebKit/WebKit/Plugins/WebBasePluginPackage.m:228 -[WebBasePluginPackage dealloc])
1403
1404         <rdar://problem/4526052> intermittent assertion failure in -[WebBasePluginPackage dealloc] running layout tests (8980)
1405         
1406         * Plugins/WebPluginDatabase.h:
1407         * Plugins/WebPluginDatabase.m:
1408         (+[WebPluginDatabase installedPlugins]):
1409         Observe NSApplicationWillTerminateNotification so we can unload plug-ins on quit.
1410         (-[WebPluginDatabase plugins]):
1411         'plugins' is now a dictionary.
1412         (-[WebPluginDatabase close]):
1413         Call new -_removePlugin: method.
1414         (-[WebPluginDatabase refresh]):
1415         Moved parts of this method out into other methods: -_addPlugin:, -_removePlugin:, and -_scanForNewPlugins.
1416         (-[WebPluginDatabase _plugInPaths]):
1417         No changes; just moved in file.
1418         (-[WebPluginDatabase _addPlugin:]):
1419         New method.  Refactored from -refresh.  Adds a plug-in to the database.
1420         (-[WebPluginDatabase _removePlugin:]):
1421         New method.  Refactored from -refresh.  Remove a plug-in from the database.
1422         (-[WebPluginDatabase _scanForNewPlugins]):
1423         New method.  Refactored from -refresh.  Returns the list of plug-in packages on disk.
1424         (-[WebPluginDatabase _applicationWillTerminate]):
1425         New method.  Called when the application terminates.  Closes the plug-in database so that all plug-ins are
1426         removed from the DB (and unloaded if necessary).
1427
1428         * Plugins/WebBasePluginPackage.h:
1429         * Plugins/WebBasePluginPackage.m:
1430         (-[WebBasePluginPackage initWithPath:]):
1431         Try to create the NSBundle first, so if the file is not a valid bundle we bail out early.  This
1432         avoids some stat()s and allocations during the plug-in refresh process.
1433         (-[WebBasePluginPackage isLoaded]):
1434         Removed.
1435         (-[WebBasePluginPackage load]):
1436         Base class for plug-in packages now always loads "successfully".
1437         (-[WebBasePluginPackage dealloc]):
1438         Removed this assertion.  The base plug-in package class has no concept of
1439         "unloading".
1440         (-[WebBasePluginPackage finalize]):
1441         ditto.
1442         (-[WebBasePluginPackage wasRemovedFromPluginDatabase:]):
1443         Moved code to unload plug-in package to WebNetscapePluginPackage.  Not all plug-in
1444         packages can be "unloaded".
1445
1446         * Plugins/WebNetscapePluginPackage.h:
1447         * Plugins/WebNetscapePluginPackage.m:
1448         (-[WebNetscapePluginPackage _unloadWithShutdown:]):
1449         Combined old -unload and -unloadWithoutShutdown methods into this new one.
1450         (-[WebNetscapePluginPackage initWithPath:]):
1451         Call new unload method.
1452         (-[WebNetscapePluginPackage load]):
1453         ditto
1454         (-[WebNetscapePluginPackage wasRemovedFromPluginDatabase:]):
1455         ditto
1456         (-[WebNetscapePluginPackage open]):
1457         New method.  Called when a plug-in instance starts running.
1458         (-[WebNetscapePluginPackage close]):
1459         New method.  Called when a plug-in instance stops running.  When all plug-in instances
1460         close the plug-in package, and the plug-in package is removed from the database, the plug-in
1461         is unloaded.
1462
1463         * Plugins/WebPluginPackage.m:
1464         (-[WebPluginPackage initWithPath:]):
1465         (-[WebPluginPackage load]):
1466         Made this a bit more efficient by checking if the bundle is already loaded.
1467         (-[WebBasePluginPackage unload]):
1468         Removed.
1469         (-[WebBasePluginPackage isLoaded]):
1470         Removed.
1471
1472         * Plugins/WebBaseNetscapePluginView.m:
1473         (-[WebBaseNetscapePluginView start]):
1474         Open the plug-in package so it remains loaded while this instance uses it.
1475         (-[WebBaseNetscapePluginView stop]):
1476         Close the plug-in package when the plug-in instance is stopped.
1477         
1478         * Plugins/WebBaseNetscapePluginStream.m:
1479         (-[WebBaseNetscapePluginStream startStreamResponseURL:expectedContentLength:lastModifiedDate:MIMEType:]):
1480         This check is not necessary.  Netscape plug-in packages are never unloaded until all their instances have
1481         been stopped, and a Netscape plug-in instance will stop its streams when it is stopped.
1482         (-[WebBaseNetscapePluginStream _destroyStream]):
1483         ditto
1484         (-[WebBaseNetscapePluginStream finishedLoadingWithData:]):
1485         ditto
1486         (-[WebBaseNetscapePluginStream _deliverData]):
1487         ditto
1488
1489 2006-08-15  Mark Rowe  <opendarwin.org@bdash.net.nz>
1490
1491         Reviewed by Tim H.
1492
1493         Build fix:  DWARF and -gfull are incompatible with symbol separation.
1494
1495         * WebKit.xcodeproj/project.pbxproj:
1496
1497 2006-08-15  Mark Rowe  <opendarwin.org@bdash.net.nz>
1498
1499         Reviewed by Tim H.
1500
1501         http://bugzilla.opendarwin.org/show_bug.cgi?id=10394
1502         Bug 10394: WebKit Release and Production configurations should enable dead code stripping
1503
1504         * WebKit.xcodeproj/project.pbxproj:
1505
1506 2006-08-15  Maciej Stachowiak  <mjs@apple.com>
1507
1508         Reviewed by Anders.
1509         
1510         - remove WebKit dependencies from WebPlugInStreamLoader via a protocol veil of ignorance
1511
1512         * Loader/WebNetscapePlugInStreamLoader.h:
1513         * Loader/WebNetscapePlugInStreamLoader.m:
1514         (-[WebNetscapePlugInStreamLoader initWithDelegate:frameLoader:]):
1515         * Loader/WebPlugInStreamLoaderDelegate.h: Added.
1516         * Plugins/WebBaseNetscapePluginStream.h:
1517         * Plugins/WebNetscapePluginStream.m:
1518         (-[WebNetscapePluginStream initWithRequest:pluginPointer:notifyData:sendNotification:]):
1519         * WebKit.xcodeproj/project.pbxproj:
1520
1521 2006-08-15  Mark Rowe  <opendarwin.org@bdash.net.nz>
1522
1523         Reviewed by Tim H.
1524
1525         http://bugzilla.opendarwin.org/show_bug.cgi?id=10384
1526         Bug 10384: Switch to DWARF for Release configuration
1527
1528         * WebKit.xcodeproj/project.pbxproj:
1529
1530 2006-08-15  Graham Dennis  <graham.dennis@gmail.com>
1531
1532         Reviewed by Darin.
1533         
1534         - patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10314
1535           WebUnarchivingState archivedResourceForURL: doesn't work
1536
1537         * WebView/WebUnarchivingState.m:
1538         (-[WebUnarchivingState archivedResourceForURL:]):
1539         Fixed to get objects from the archived resources dictionary using
1540         the URL as a string instead of as the URL itself (as this is how
1541         the data is put into the dictionary).
1542
1543 2006-08-14  Maciej Stachowiak  <mjs@apple.com>
1544
1545         Reviewed by Beth.
1546         
1547         - remove many (but not all) WebKit dependencies from WebNetscapePlugInStreamLoader (it still 
1548         depends on WebNetscapePluginStream).
1549
1550         * Loader/WebFrameLoader.h:
1551         * Loader/WebFrameLoader.m:
1552         (-[WebFrameLoader fileDoesNotExistErrorWithResponse:]):
1553         * Loader/WebNetscapePlugInStreamLoader.h:
1554         * Loader/WebNetscapePlugInStreamLoader.m:
1555         (-[WebNetscapePlugInStreamLoader initWithStream:frameLoader:]):
1556         (-[WebNetscapePlugInStreamLoader releaseResources]):
1557         (-[WebNetscapePlugInStreamLoader didReceiveResponse:]):
1558         * Plugins/WebNetscapePluginStream.m:
1559         (-[WebNetscapePluginStream initWithRequest:pluginPointer:notifyData:sendNotification:]):
1560
1561 2006-08-14  Maciej Stachowiak  <mjs@apple.com>
1562
1563         Reviewed by Tim Omernick.
1564         
1565         - fixed REGRESSION: crash when leaving youtube page while movie is still loading
1566         http://bugzilla.opendarwin.org/show_bug.cgi?id=10398
1567
1568         * Loader/WebNetscapePlugInStreamLoader.m:
1569         (-[WebNetscapePlugInStreamLoader initWithStream:view:]): Set the frame loader for this stream.
1570         (-[WebNetscapePlugInStreamLoader cancelWithError:]): Make sure to destroy the stream as well;
1571         otherwise, when we try to clean up later, we won't have the right context.
1572
1573 2006-08-14  David Hyatt  <hyatt@apple.com>
1574
1575         Fix for Radar bug 4478840, Safari should not reduce null events sent to
1576         plug-ins in windows that are inactive but visible.
1577
1578         With this fix you can view videos in visible background windows on YouTube
1579         (for example) and not see any drop in frame rate.
1580
1581         Reviewed by timo
1582
1583         * Plugins/WebBaseNetscapePluginView.m:
1584         (-[WebBaseNetscapePluginView restartNullEvents]):
1585
1586 2006-08-14  Maciej Stachowiak  <mjs@apple.com>
1587
1588         Reviewed by Brady.
1589         
1590         - removed WebKit-level dependencies from WebFormDataStream. Use WebCore version of system interface
1591
1592         * Loader/WebFormDataStream.m:
1593         (formCanRead):
1594         (formEventCallback):
1595         (webSetHTTPBody):
1596         * WebCoreSupport/WebSystemInterface.m:
1597         (InitWebCoreSystemInterface):
1598
1599 2006-08-14  Maciej Stachowiak  <mjs@apple.com>
1600
1601         Rubber stamped by Brady.
1602         
1603         - move WebFormDataStream from WebView to Loader
1604
1605         * WebKit.xcodeproj/project.pbxproj:
1606         * WebView/WebFormDataStream.h: Removed.
1607         * WebView/WebFormDataStream.m: Removed.
1608
1609 2006-08-13  Maciej Stachowiak  <mjs@apple.com>
1610
1611         Reviewed by Brady.
1612         
1613         - remove WebKit dependencies from WebSubresourceLoader, except WebFormDataStream
1614         
1615         (WebFormDataStream will be moved into the Loader directory soon)
1616
1617         * Loader/WebFrameLoader.h:
1618         * Loader/WebFrameLoader.m:
1619         (-[WebFrameLoader _addExtraFieldsToRequest:mainResource:alwaysFromRequest:]):
1620         * Loader/WebSubresourceLoader.m:
1621         (isConditionalRequest):
1622         (hasCaseInsensitivePrefix):
1623         (isFileURLString):
1624         (setHTTPReferrer):
1625         (+[WebSubresourceLoader startLoadingResource:withRequest:customHeaders:referrer:forFrameLoader:]):
1626
1627 2006-08-13  Brady Eidson  <beidson@apple.com>
1628
1629         Reviewed by Maciej
1630
1631         Relocated the WebIconLoaders
1632
1633         * Misc/WebIconLoader.h: Moved to Loader/
1634         * Misc/WebIconLoader.m: Moved to Loader/
1635         * WebKit.xcodeproj/project.pbxproj:
1636
1637 2006-08-13  Maciej Stachowiak  <mjs@apple.com>
1638
1639         Reviewed by Beth.
1640         
1641         - removed non-Loader WebKit dependencies from WebDataProtocol for
1642         real (whoops) and fix some typos.
1643
1644         * Loader/WebDataProtocol.m:
1645         (isCaseInsensitiveEqual): Added.
1646         (+[WebDataProtocol _webIsDataProtocolURL:]): Avoid WebKit calls.
1647         (-[WebDataProtocol startLoading]): ditto
1648         * Loader/WebFrameLoader.m:
1649         (isCaseInsensitiveEqual): Fixed spelling from isCaseSensitiveEqual.
1650         (-[WebFrameLoader _canUseResourceForRequest:]): Use proper call.
1651
1652 2006-08-13  Brady Eidson  <beidson@apple.com>
1653
1654         Reviewed by Maciej
1655
1656         The way of detecting a failed icon load before was to try and construct
1657         an image from the icon and if that image construction failed, mark the icon
1658         as missing.  
1659         A much more efficient way is to check for an error response.  We'll still
1660         check for invalid image data, but most servers will correctly return an HTTP 
1661         error on a missing icon.
1662         
1663         * Misc/WebIconLoader.m:
1664         (-[WebIconLoader didFinishLoading]): Added check for http error response
1665
1666 2006-08-13  Maciej Stachowiak  <mjs@apple.com>
1667
1668         Reviewed by Darin.
1669         
1670         - remove all non-Loader dependencies from WebLoader
1671         
1672         As part of this I moved WebDataProtocol to the loader directory
1673         and removed dependencies on the rest of WebKit from that too.
1674
1675         * Loader/WebFrameLoader.h:
1676         * Loader/WebFrameLoader.m:
1677         (-[WebFrameLoader setDefersCallbacks:]):
1678         (-[WebFrameLoader stopLoading]):
1679         (-[WebFrameLoader cancelledErrorWithRequest:]):
1680         (-[WebFrameLoader clearArchivedResources]):
1681         (-[WebFrameLoader deliverArchivedResources]):
1682         (-[WebFrameLoader deliverArchivedResourcesAfterDelay]):
1683         (isCaseSensitiveEqual):
1684         (-[WebFrameLoader _canUseResourceForRequest:]):
1685         (-[WebFrameLoader _canUseResourceWithResponse:]):
1686         (-[WebFrameLoader pendingArchivedResources]):
1687         (-[WebFrameLoader willUseArchiveForRequest:originalURL:loader:]):
1688         (-[WebFrameLoader archiveLoadPendingForLoader:]):
1689         (-[WebFrameLoader cancelPendingArchiveLoadForLoader:]):
1690         * Loader/WebLoader.h:
1691         * Loader/WebLoader.m:
1692         (-[NSURLProtocol releaseResources]):
1693         (-[NSURLProtocol loadWithRequest:]):
1694         (-[NSURLProtocol setDefersCallbacks:]):
1695         (-[NSURLProtocol addData:allAtOnce:]):
1696         (-[NSURLProtocol resourceData]):
1697         (-[NSURLProtocol didReceiveData:lengthReceived:allAtOnce:]):
1698         (-[NSURLProtocol connection:didReceiveData:lengthReceived:]):
1699         (-[NSURLProtocol cancelWithError:]):
1700         (-[NSURLProtocol cancelledError]):
1701         * Loader/WebMainResourceLoader.m:
1702         (-[WebMainResourceLoader addData:allAtOnce:]):
1703         (-[WebMainResourceLoader didReceiveData:lengthReceived:allAtOnce:]):
1704         * Loader/WebNetscapePlugInStreamLoader.m:
1705         (-[WebNetscapePlugInStreamLoader didReceiveData:lengthReceived:allAtOnce:]):
1706         * Loader/WebSubresourceLoader.m:
1707         (-[WebSubresourceLoader didReceiveData:lengthReceived:allAtOnce:]):
1708         * WebKit.xcodeproj/project.pbxproj:
1709         * WebView/WebDataProtocol.h: Removed.
1710         * WebView/WebDataProtocol.m: Removed.
1711
1712 2006-08-11  Tim Omernick  <timo@apple.com>
1713
1714         Reviewed by Darin.
1715
1716         <http://bugzilla.opendarwin.org/show_bug.cgi?id=10111> - Menu flickers over Flash content
1717         <rdar://problem/3052546> Plugins don't work with z-index (overlapping elements, etc.)
1718
1719         * Plugins/WebBaseNetscapePluginView.m:
1720         (-[WebBaseNetscapePluginView saveAndSetNewPortStateForUpdate:]):
1721         Don't just clip to the dirty region for "transparent" plug-ins -- do it for all plug-ins.  This is a
1722         generally useful thing to do, as it prevents the plug-in from drawing over parts of the window that
1723         have already been drawn and are not expected to be redrawn in the same update.
1724
1725 2006-08-11  Brady Eidson  <beidson@apple.com>
1726
1727         Reviewed by John, Timo, Adele, and Darin
1728
1729         In addition to a few style/good-practice cleanups, this patch will convert the old icon database
1730         format to the WebCore format if the WebCore db is empty (implying this conversion has yet to take
1731         place).  After the conversion, it will delete all traces of the old format to free the unneeded space
1732
1733         * Misc/WebIconDatabase.m:
1734         (-[WebIconDatabase init]):  
1735         (-[WebIconDatabase _setIconURL:forURL:]):  Changed the bridge's name for this method to be more clear
1736         (-[WebIconDatabase _createFileDatabase]):  
1737         (-[WebIconDatabase _iconDataForIconURL:]):  This grabs the raw data for use in the conversion function
1738         (-[WebIconDatabase _convertToWebCoreFormat]):  This does the actual conversion
1739
1740 2006-08-11  Tim Omernick  <timo@apple.com>
1741
1742         Reviewed by John Sullivan.
1743         
1744         Needed for <rdar://problem/4678070>.
1745
1746         * Plugins/WebBaseNetscapePluginView.m:
1747         (-[WebBaseNetscapePluginView sendEvent:]):
1748         Changed an assertion to an early return.  It should be possible to send events, especially updateEvt (for image capturing
1749         purposes), to off-screen plug-ins.  It just doesn't work right now.  See <rdar://problem/4318269>.
1750
1751 2006-08-11  John Sullivan  <sullivan@apple.com>
1752
1753         Reviewed by Darin
1754         
1755         - fixed <rdar://problem/4522894> Would be nice if Safari shrank pages a little if necessary 
1756         to avoid printing an almost-empty page
1757
1758         * WebView/WebHTMLView.m:
1759         (-[NSArray knowsPageRange:]):
1760         If the last page has a short-enough orphan (< 1/10 of the page height is the number I pulled
1761         out of ... the air), then we adjust the scale factor slightly and check whether this reduces
1762         the page count and thus eliminates the orphan.
1763         
1764 2006-08-07  Brady Eidson  <beidson@apple.com>
1765
1766         Reviewed by Anders and John
1767
1768         * Misc/WebIconDatabase.m:
1769         (-[WebIconDatabase init]):
1770         (-[WebIconDatabase isIconExpiredForIconURL:]): Get if an icon expired
1771         (-[WebIconDatabase isIconExpiredForPageURL:]): Ditto
1772         (-[WebIconDatabase _setIconURL:forURL:]):
1773         (-[WebIconDatabase _sendNotificationForURL:]): Moved to WebKitPendingPublic for use outside of WebIconDatabase
1774         (-[WebIconDatabase loadIconFromURL:]): Allow a load outside the context of a page load
1775         * Misc/WebIconDatabasePrivate.h:
1776
1777         * Misc/WebIconLoader.m:
1778         (-[WebIconLoader didFinishLoading]): fixed up the "flipping the switch" #defs a bit
1779         (-[WebIconLoader willSendRequest:redirectResponse:]): override to allow a load
1780         outside of the context of a page load
1781
1782         * WebCoreSupport/WebIconDatabaseBridge.h: Added.
1783         * WebCoreSupport/WebIconDatabaseBridge.m: Added.
1784         (-[WebIconDatabaseBridge init]):
1785         (-[WebIconDatabaseBridge dealloc]):
1786         (-[WebIconDatabaseBridge loadIconFromURL:]):  Kick off a load on an icon outside
1787         of the context of any page load
1788         (-[WebIconDatabaseBridge _setIconData:forIconURL:]):  WebKit side of bridge method
1789         (-[WebIconDatabaseBridge _setHaveNoIconForIconURL:]):  WebKit side of bridge method
1790         (-[WebIconDatabaseBridge releaseCachedLoaderForIconURL:]): 
1791         (+[WebIconDatabaseBridge sharedBridgeInstance]): Moved this from WebCore to WebKit
1792         so both sides of the bridge get the WebKit version
1793         * WebKit.xcodeproj/project.pbxproj: Added some files
1794         * WebView/WebDataSource.m:
1795         (-[WebDataSource _loadIcon]): Added check for reload/expired icon to force
1796         a load even if we already have it
1797
1798 2006-08-04  Sam Weinig  <sam.weinig@gmail.com>
1799
1800         Reviewed by Darin.
1801
1802         - patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10192
1803           Make WebCore (and friends) compile with -Wshorten-64-to-32
1804
1805           * Adds 'f' to float literals where expecting a float.
1806           * Use ceilf() instead of ceil() when assigning to a float.
1807           * Adds explicit casts where OK.
1808           
1809           NOTE: The -Wshorten-64-to-32 flag was not added for WebKit
1810           because there are still a few places where no error handling 
1811           is in place.  The flag can be added as soon as those are worked
1812           out.
1813
1814         * Misc/WebNSControlExtras.m:
1815         (-[NSControl sizeToFitAndAdjustWindowHeight]):
1816         * Misc/WebNSImageExtras.m:
1817         (-[NSImage _web_scaleToMaxSize:]):
1818         * Misc/WebNSViewExtras.h:
1819         * Misc/WebNSViewExtras.m:
1820         * Plugins/WebBaseNetscapePluginView.m:
1821         (-[WebBaseNetscapePluginView saveAndSetNewPortStateForUpdate:]):
1822         (-[WebBaseNetscapePluginView drawRect:]):
1823         * WebCoreSupport/WebFrameBridge.m:
1824         (-[WebFrameBridge expiresTimeForResponse:]):
1825         * WebInspector/WebInspector.m:
1826         (-[NSWindow window]):
1827         (-[WebInspector treeViewScrollTo:]):
1828         (-[WebInspector _updateSystemColors]):
1829         (-[WebInspector webView:plugInViewWithArguments:]):
1830         (-[WebInspector outlineView:objectValueForTableColumn:byItem:]):
1831         * WebInspector/WebInspectorOutlineView.m:
1832         (-[WebInspectorOutlineView _highlightRow:clipRect:]):
1833         * WebInspector/WebNodeHighlight.m:
1834         (-[WebNodeHighlight initWithBounds:andRects:forView:]):
1835         * WebInspector/WebNodeHighlightView.m:
1836         (-[WebNodeHighlightView roundedRect:withRadius:]):
1837         (-[WebNodeHighlightView initWithHighlight:andRects:forView:]):
1838         (-[WebNodeHighlightView drawRect:]):
1839         * WebView/WebFrame.m:
1840         (-[WebFrame _opened]):
1841         * WebView/WebFrameView.m:
1842         (-[WebFrameView initWithFrame:]):
1843         * WebView/WebHTMLView.m:
1844         (-[WebHTMLView _dragImageForLinkElement:]):
1845         (-[WebHTMLView _web_setPrintingModeRecursive]):
1846         (-[WebHTMLView _web_clearPrintingModeRecursive]):
1847         (-[NSArray layout]):
1848         (-[NSArray _setPrinting:minimumPageWidth:maximumPageWidth:adjustViewSize:]):
1849         (-[NSArray adjustPageHeightNew:top:bottom:limit:]):
1850         (-[NSArray _scaleFactorForPrintOperation:]):
1851         (-[NSArray setPageWidthForPrinting:]):
1852         (-[NSArray _endPrintMode]):
1853         (-[NSArray knowsPageRange:]):
1854         (-[NSArray _originalFontA]):
1855         (-[NSArray _originalFontB]):
1856         (-[WebTextCompleteController _buildUI]):
1857         (-[WebTextCompleteController _placePopupWindow:]):
1858         * WebView/WebPDFView.m:
1859         (-[WebPDFView _makeTextStandardSize:]):
1860         (-[WebPDFView selectionImageForcingWhiteText:]):
1861         (-[PDFPrefUpdatingProxy forwardInvocation:]):
1862         * WebView/WebPreferences.m:
1863         (-[WebPreferences _floatValueForKey:]):
1864         * WebView/WebView.m:
1865         (-[WebView makeTextSmaller:]):
1866         (-[WebView canMakeTextStandardSize]):
1867         (-[WebView makeTextStandardSize:]):
1868
1869 2006-08-04  David Kilzer  <ddkilzer@kilzer.net>
1870
1871         Reviewed by NOBODY (build fix).
1872
1873         * WebCoreSupport/WebSubresourceLoader.m: REALLY moved to Loader/
1874         * WebView/WebFrameLoader.h: REALLY moved to Loader/
1875         * WebView/WebFrameLoader.m: REALLY moved to Loader/
1876         * WebView/WebLoader.h: REALLY moved to Loader/
1877         * WebView/WebLoader.m: REALLY moved to Loader/
1878         * WebView/WebMainResourceLoader.m: REALLY moved to Loader/
1879
1880 2006-08-03  Maciej Stachowiak  <mjs@apple.com>
1881
1882         Reviewed by Darin.
1883         
1884         - moved all loader code that is slated to be moved down to WebCore to a new Loader directory
1885         
1886         (next step is to remove dependencies on the rest of WebKit from this directory)
1887
1888         * Loader/WebNetscapePlugInStreamLoader.h: Added.
1889         * Loader/WebNetscapePlugInStreamLoader.m: Added. Cut out of WebNetscapePluginStream.m
1890         (-[WebNetscapePlugInStreamLoader initWithStream:view:]):
1891         (-[WebNetscapePlugInStreamLoader isDone]):
1892         (-[WebNetscapePlugInStreamLoader releaseResources]):
1893         (-[WebNetscapePlugInStreamLoader didReceiveResponse:]):
1894         (-[WebNetscapePlugInStreamLoader didReceiveData:lengthReceived:]):
1895         (-[WebNetscapePlugInStreamLoader didFinishLoading]):
1896         (-[WebNetscapePlugInStreamLoader didFailWithError:]):
1897         (-[WebNetscapePlugInStreamLoader cancelWithError:]):
1898         * Plugins/WebNetscapePluginStream.m:
1899         * WebKit.xcodeproj/project.pbxproj:
1900         * WebCoreSupport/WebSubresourceLoader.h: Moved to Loader/
1901         * WebCoreSupport/WebSubresourceLoader.m: Moved to Loader/
1902         * WebView/WebFrameLoader.h: Moved to Loader/
1903         * WebView/WebFrameLoader.m: Moved to Loader/
1904         * WebView/WebLoader.h: Moved to Loader/
1905         * WebView/WebLoader.m: Moved to Loader/
1906         * WebView/WebMainResourceLoader.h: Moved to Loader/
1907         * WebView/WebMainResourceLoader.m: Moved to Loader/
1908
1909 2006-08-03  Tim Omernick  <timo@apple.com>
1910
1911         Reviewed by Kevin Decker.
1912
1913         <rdar://problem/4667460> Windowless OpenGL plug-ins render incorrectly on PowerPC
1914
1915         * Plugins/WebBaseNetscapePluginView.m:
1916         (-[WebBaseNetscapePluginView _aglOffscreenImageForDrawingInRect:]):
1917         Fixed color component swapping so that it works on both x86 and PPC.  See comments.
1918
1919 2006-08-03  Brady Eidson  <beidson@apple.com>
1920
1921         Reviewed by Tim Hatcher's rubber stamp
1922         Fixed Intel build break caused by weinig's -W change in r15781
1923
1924         * WebView/WebView.m: wrapped cpu-dependent defs with defined() macro
1925
1926 2006-08-03  Maciej Stachowiak  <mjs@apple.com>
1927
1928         Reviewed by John.
1929
1930         - fixed problem that could cause assertion failures in Safari
1931         
1932         * Plugins/WebBaseNetscapePluginView.m:
1933         (-[WebBaseNetscapePluginView loadRequest:inTarget:withNotifyData:sendNotification:]): Don't
1934         allow a plugin to start new loads once its document is no longer the one actively loading.
1935
1936 2006-08-03  Maciej Stachowiak  <mjs@apple.com>
1937
1938         Reviewed by John.
1939         
1940         - remove use of WebDataSource from WebLoader and subclasses, just have them talk to the
1941         WebFrameLoader instead. 
1942         
1943         For now this is done by forarding all the calls.
1944
1945         * Misc/WebIconLoader.m:
1946         (-[WebIconLoader didFinishLoading]):
1947         * Plugins/WebNetscapePluginStream.m:
1948         (-[WebNetscapePluginStream initWithRequest:pluginPointer:notifyData:sendNotification:]):
1949         (-[WebNetscapePluginStream start]):
1950         (-[WebNetscapePlugInStreamLoader didFinishLoading]):
1951         (-[WebNetscapePlugInStreamLoader didFailWithError:]):
1952         (-[WebNetscapePlugInStreamLoader cancelWithError:]):
1953         * WebCoreSupport/WebFrameBridge.m:
1954         (-[WebFrameBridge startLoadingResource:withMethod:URL:customHeaders:]):
1955         (-[WebFrameBridge startLoadingResource:withMethod:URL:customHeaders:postData:]):
1956         * WebCoreSupport/WebSubresourceLoader.h:
1957         * WebCoreSupport/WebSubresourceLoader.m:
1958         (-[WebSubresourceLoader initWithLoader:frameLoader:]):
1959         (+[WebSubresourceLoader startLoadingResource:withRequest:customHeaders:referrer:forFrameLoader:]):
1960         (+[WebSubresourceLoader startLoadingResource:withMethod:URL:customHeaders:referrer:forFrameLoader:]):
1961         (+[WebSubresourceLoader startLoadingResource:withMethod:URL:customHeaders:postData:referrer:forFrameLoader:]):
1962         (-[WebSubresourceLoader receivedError:]):
1963         (-[WebSubresourceLoader signalFinish]):
1964         (-[WebSubresourceLoader didFailWithError:]):
1965         (-[WebSubresourceLoader cancel]):
1966         * WebKit.xcodeproj/project.pbxproj:
1967         * WebView/WebDataSource.m:
1968         (-[WebDataSource _updateLoading]):
1969         (-[WebDataSource textEncodingName]):
1970         (-[WebDataSource _mainReceivedBytesSoFar:complete:]):
1971         * WebView/WebFrameLoader.h:
1972         * WebView/WebFrameLoader.m:
1973         (-[WebFrameLoader loadIconWithRequest:]):
1974         (-[WebFrameLoader startLoadingMainResourceWithRequest:identifier:]):
1975         (-[WebFrameLoader clearIconLoader]):
1976         (-[WebFrameLoader commitProvisionalLoad]):
1977         (-[WebFrameLoader activeDataSource]):
1978         (-[WebFrameLoader _archivedSubresourceForURL:]):
1979         (-[WebFrameLoader _defersCallbacks]):
1980         (-[WebFrameLoader _identifierForInitialRequest:]):
1981         (-[WebFrameLoader _willSendRequest:forResource:redirectResponse:]):
1982         (-[WebFrameLoader _didReceiveAuthenticationChallenge:forResource:]):
1983         (-[WebFrameLoader _didCancelAuthenticationChallenge:forResource:]):
1984         (-[WebFrameLoader _didReceiveResponse:forResource:]):
1985         (-[WebFrameLoader _didReceiveData:contentLength:forResource:]):
1986         (-[WebFrameLoader _didFinishLoadingForResource:]):
1987         (-[WebFrameLoader _didFailLoadingWithError:forResource:]):
1988         (-[WebFrameLoader _privateBrowsingEnabled]):
1989         (-[WebFrameLoader _addPlugInStreamLoader:]):
1990         (-[WebFrameLoader _removePlugInStreamLoader:]):
1991         (-[WebFrameLoader _finishedLoadingResource]):
1992         (-[WebFrameLoader _receivedError:]):
1993         (-[WebFrameLoader _addSubresourceLoader:]):
1994         (-[WebFrameLoader _removeSubresourceLoader:]):
1995         (-[WebFrameLoader _originalRequest]):
1996         (-[WebFrameLoader webFrame]):
1997         (-[WebFrameLoader _receivedMainResourceError:complete:]):
1998         (-[WebFrameLoader initialRequest]):
1999         (-[WebFrameLoader _receivedData:]):
2000         (-[WebFrameLoader _setRequest:]):
2001         (-[WebFrameLoader _downloadWithLoadingConnection:request:response:proxy:]):
2002         (-[WebFrameLoader _handleFallbackContent]):
2003         (-[WebFrameLoader _isStopping]):
2004         (-[WebFrameLoader _decidePolicyForMIMEType:decisionListener:]):
2005         (-[WebFrameLoader _setupForReplaceByMIMEType:]):
2006         (-[WebFrameLoader _setResponse:]):
2007         (-[WebFrameLoader _mainReceivedError:complete:]):
2008         (-[WebFrameLoader _finishedLoading]):
2009         (-[WebFrameLoader _mainReceivedBytesSoFar:complete:]):
2010         (-[WebFrameLoader _iconLoaderReceivedPageIcon:]):
2011         (-[WebFrameLoader _URL]):
2012         * WebView/WebLoader.h:
2013         * WebView/WebLoader.m:
2014         (-[NSURLProtocol releaseResources]):
2015         (-[NSURLProtocol loadWithRequest:]):
2016         (-[NSURLProtocol setFrameLoader:]):
2017         (-[NSURLProtocol frameLoader]):
2018         (-[NSURLProtocol willSendRequest:redirectResponse:]):
2019         (-[NSURLProtocol didReceiveAuthenticationChallenge:]):
2020         (-[NSURLProtocol didCancelAuthenticationChallenge:]):
2021         (-[NSURLProtocol didReceiveResponse:]):
2022         (-[NSURLProtocol didReceiveData:lengthReceived:]):
2023         (-[NSURLProtocol signalFinish]):
2024         (-[NSURLProtocol didFailWithError:]):
2025         (-[NSURLProtocol willCacheResponse:]):
2026         (-[NSURLProtocol cancelWithError:]):
2027         * WebView/WebMainResourceLoader.h:
2028         * WebView/WebMainResourceLoader.m:
2029         (-[WebMainResourceLoader initWithFrameLoader:]):
2030         (-[WebMainResourceLoader receivedError:]):
2031         (-[WebMainResourceLoader cancelWithError:]):
2032         (-[WebMainResourceLoader _isPostOrRedirectAfterPost:redirectResponse:]):
2033         (-[WebMainResourceLoader addData:]):
2034         (-[WebMainResourceLoader willSendRequest:redirectResponse:]):
2035         (-[WebMainResourceLoader continueAfterContentPolicy:response:]):
2036         (-[WebMainResourceLoader continueAfterContentPolicy:]):
2037         (-[WebMainResourceLoader checkContentPolicyForResponse:]):
2038         (-[WebMainResourceLoader didReceiveResponse:]):
2039         (-[WebMainResourceLoader didReceiveData:lengthReceived:]):
2040         (-[WebMainResourceLoader didFinishLoading]):
2041         (-[WebMainResourceLoader didFailWithError:]):
2042         (-[WebMainResourceLoader loadWithRequestNow:]):
2043
2044 2006-08-03  Sam Weinig  <sam.weinig@gmail.com>
2045
2046         Reviewed by Darin.
2047
2048         - patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10176
2049           Make WebCore compile with -Wundef
2050
2051           * Adds -Wundef flag to Xcode project
2052           * Converts #ifs to #ifdef and #ifndefs where needed.
2053
2054         * Carbon/CarbonUtils.m:
2055         * Carbon/CarbonWindowAdapter.m:
2056         * Carbon/HIViewAdapter.m:
2057         (+[NSView bindHIViewToNSView:nsView:]):
2058         * Carbon/HIWebView.m:
2059         (HIWebViewEventHandler):
2060         * Misc/WebFileDatabase.m:
2061         (UniqueFilePathForKey):
2062         * Misc/WebNSWindowExtras.m:
2063         (swizzleInstanceMethod):
2064         * Misc/WebTypesInternal.h:
2065         * Plugins/WebNetscapeDeprecatedFunctions.c:
2066         * Plugins/WebNetscapeDeprecatedFunctions.h:
2067         * Plugins/WebNetscapePluginPackage.h:
2068         * Plugins/WebNetscapePluginPackage.m:
2069         (-[WebNetscapePluginPackage unloadWithoutShutdown]):
2070         (-[WebNetscapePluginPackage load]):
2071         * WebKit.xcodeproj/project.pbxproj:
2072
2073 2006-08-03  Darin Adler  <darin@apple.com>
2074
2075         Reviewed by Eric Seidel.
2076
2077         - fix storage leak
2078
2079         * WebView/WebFrame.m: (-[WebFramePrivate dealloc]): Release the frame loader.
2080
2081 2006-08-02  Timothy Hatcher  <timothy@apple.com>
2082
2083         Rubber stamped by Maciej.
2084
2085         Adding back resultsWithXpathQuery, removed by Darin's earlier change.
2086         This function is called from ObjC, but not used from JavaScript.
2087
2088         * WebInspector/webInspector/inspector.js:
2089
2090 2006-08-02  Timothy Hatcher  <timothy@apple.com>
2091
2092         Reviewed by Darin.
2093
2094         Bug 10200: [Drosera] Deadlock between Drosera and Safari while loading page
2095         http://bugzilla.opendarwin.org/show_bug.cgi?id=10200
2096
2097         Prevent reentrancy in our debugger callbacks. This was causing a deadlock in Drosera because
2098         suspendProcessIfPaused was being called during a DO call into Safari.
2099
2100         Preventing reentrancy also prevents scripts that Drosera injects and evaluates from showing
2101         up in rare cases (such as a iframe loading about:blank). I thought this would prevent cases
2102         where you call a function from the console and expect it to break on a breakpoint in them, but
2103         this appears to never have worked even without this change. When that is figured out we can
2104         reconsider a better solution to reentrancy. I have filed that as bug 10214.
2105
2106         I also removed the NSRunLoop runMode:beforeDate: calls since DO handles this for us since
2107         we don't use "onway void" as the return type for the callbacks. Note: using onway void for
2108         the listener callbacks causes bad synchronization issues and obscure crashes.
2109
2110         * DefaultDelegates/WebScriptDebugServer.m:
2111         (-[WebScriptDebugServer webView:didLoadMainResourceForDataSource:]):
2112         (-[WebScriptDebugServer webView:didParseSource:baseLineNumber:fromURL:sourceId:forWebFrame:]):
2113         (-[WebScriptDebugServer webView:failedToParseSource:baseLineNumber:fromURL:withError:forWebFrame:]):
2114         (-[WebScriptDebugServer webView:didEnterCallFrame:sourceId:line:forWebFrame:]):
2115         (-[WebScriptDebugServer webView:willExecuteStatement:sourceId:line:forWebFrame:]):
2116         (-[WebScriptDebugServer webView:willLeaveCallFrame:sourceId:line:forWebFrame:]):
2117         (-[WebScriptDebugServer webView:exceptionWasRaised:sourceId:line:forWebFrame:]):
2118         * DefaultDelegates/WebScriptDebugServerPrivate.h:
2119
2120 2006-08-02  Maciej Stachowiak  <mjs@apple.com>
2121
2122         Reviewed by John.
2123         
2124         - fix assertion failure on layout tests by stopping plugins from loading at a clearly defined time
2125         - add more assertions for safety
2126
2127         * WebView/WebDataSource.m:
2128         (-[WebDataSourcePrivate dealloc]): Removed obsolete comment.
2129         (-[WebDataSource _updateLoading]): Add assertion ensuring this method is only called
2130         at a time when this data source is the one that might be loading for a frame.
2131         (-[WebDataSource _stopLoading]): Stop loading plugins as a FIXME suggests we should.
2132         * WebView/WebFrameLoader.m:
2133         (-[WebFrameLoader isLoadingPlugIns]): New helper method.
2134         (-[WebFrameLoader isLoading]): Consider plugin loads too - otherwise we won't stop them
2135         at stopLoading time.
2136
2137 2006-08-02  Adam Roben  <aroben@apple.com>
2138
2139         Reviewed by Brady.
2140
2141         - Rename TransferJob to ResourceLoader (this file was forgotten in an
2142         earlier change by Maciej)
2143
2144         * COM/WebView.cpp:
2145
2146 2006-08-01  Maciej Stachowiak  <mjs@apple.com>
2147
2148         Reviewed by Adele.
2149         
2150         - Change things around so WebFrameLoader tracks the main and provisional data source,
2151         as well as the frame load state, pulling much code out of WebFrame along the way.
2152         
2153         The most significant aspects of this change are:
2154         
2155         - management of WebDataSources and WebFrameState was moved into WebFrameLoader
2156         - there is now just one WebFrameLoader shared between the primary and provisional data source
2157
2158         * WebKit.xcodeproj/project.pbxproj:
2159         * WebView/WebDataSource.m:
2160         (-[WebDataSourcePrivate dealloc]):
2161         (-[WebDataSource _updateLoading]):
2162         (-[WebDataSource _loadIcon]):
2163         (-[WebDataSource _setPrimaryLoadComplete:]):
2164         (-[WebDataSource _stopLoading]):
2165         (-[WebDataSource _startLoading]):
2166         (-[WebDataSource _addSubresourceLoader:]):
2167         (-[WebDataSource _removeSubresourceLoader:]):
2168         (-[WebDataSource _addPlugInStreamLoader:]):
2169         (-[WebDataSource _removePlugInStreamLoader:]):
2170         (-[WebDataSource _defersCallbacksChanged]):
2171         (-[WebDataSource _stopLoadingWithError:]):
2172         (-[WebDataSource _revertToProvisionalState]):
2173         (-[WebDataSource _setupForReplaceByMIMEType:]):
2174         (-[WebDataSource initWithRequest:]):
2175         (-[WebDataSource data]):
2176         (-[WebDataSource isLoading]):
2177         * WebView/WebFrame.m:
2178         (-[WebFramePrivate init]):
2179         (-[WebFramePrivate dealloc]):
2180         (-[WebFrame _closeOldDataSources]):
2181         (-[WebFrame _detachFromParent]):
2182         (-[WebFrame _makeDocumentView]):
2183         (-[WebFrame _receivedMainResourceError:]):
2184         (-[WebFrame _transitionToCommitted:]):
2185         (+[WebFrame _timeOfLastCompletedLoad]):
2186         (-[WebFrame _checkLoadCompleteForThisFrame]):
2187         (-[WebFrame _loadItem:withLoadType:]):
2188         (-[WebFrame _continueAfterWillSubmitForm:]):
2189         (-[WebFrame _continueLoadRequestAfterNavigationPolicy:formState:]):
2190         (-[WebFrame _initWithWebFrameView:webView:bridge:]):
2191         (-[WebFrame _frameLoader]):
2192         (-[WebFrame _provisionalLoadStarted]):
2193         (-[WebFrame _prepareForDataSourceReplacement]):
2194         (-[WebFrame _frameLoadCompleted]):
2195         (-[WebFrame provisionalDataSource]):
2196         (-[WebFrame dataSource]):
2197         (-[WebFrame stopLoading]):
2198         * WebView/WebFrameInternal.h:
2199         * WebView/WebFrameLoader.h:
2200         * WebView/WebFrameLoader.m:
2201         (-[WebFrameLoader initWithWebFrame:]):
2202         (-[WebFrameLoader dealloc]):
2203         (-[WebFrameLoader dataSource]):
2204         (-[WebFrameLoader _setDataSource:]):
2205         (-[WebFrameLoader clearDataSource]):
2206         (-[WebFrameLoader provisionalDataSource]):
2207         (-[WebFrameLoader _setProvisionalDataSource:]):
2208         (-[WebFrameLoader _clearProvisionalDataSource]):
2209         (-[WebFrameLoader state]):
2210         (+[WebFrameLoader timeOfLastCompletedLoad]):
2211         (-[WebFrameLoader _setState:]):
2212         (-[WebFrameLoader clearProvisionalLoad]):
2213         (-[WebFrameLoader markLoadComplete]):
2214         (-[WebFrameLoader commitProvisionalLoad]):
2215         (-[WebFrameLoader stopLoading]):
2216         (-[WebFrameLoader startLoading]):
2217         (-[WebFrameLoader startProvisionalLoad:]):
2218         (-[WebFrameLoader setupForReplace]):
2219         * WebView/WebFramePrivate.h:
2220
2221 2006-08-01  Tim Omernick  <timo@apple.com>
2222
2223         Reviewed by John Sullivan.
2224
2225         <rdar://problem/4480737> Flash crashes after it replaces itself via a document.write()
2226         
2227         I kind of hate to do this, but this is the best way to work around buggy plug-ins like Flash that assume that
2228         NPP_Destroy() cannot be called while the browser is calling one of its other plug-in functions.  The classic
2229         situation is a plug-in that replaces itself via an NPN_Invoke() that executes a document.write().
2230
2231         * Plugins/WebBaseNetscapePluginView.h:
2232         * Plugins/WebBaseNetscapePluginView.m:
2233         (-[WebBaseNetscapePluginView sendEvent:]):
2234         Call -willCallPlugInFunction and -didCallPlugInFunction around calls to the NPP_* functions.
2235         (-[WebBaseNetscapePluginView setWindowIfNecessary]):
2236         ditto
2237         (-[WebBaseNetscapePluginView start]):
2238         It should not be possible to start a plug-in instance while we are calling into it (one of those chicken/egg
2239         problems).  Added a sanity-checking assertion.
2240         (-[WebBaseNetscapePluginView stop]):
2241         If we're already calling a plug-in function, do not call NPP_Destroy().  The plug-in function we are calling
2242         may assume that its instance->pdata, or other memory freed by NPP_Destroy(), is valid and unchanged until said
2243         plugin-function returns.
2244         (-[WebBaseNetscapePluginView pluginScriptableObject]):
2245         Call -willCallPlugInFunction and -didCallPlugInFunction around calls to the NPP_* functions.
2246         (-[WebBaseNetscapePluginView willCallPlugInFunction]):
2247         Increment plug-in function call depth.
2248         (-[WebBaseNetscapePluginView didCallPlugInFunction]):
2249         Decrement plug-in function call depth.  Stop if we're supposed to stop.
2250         (-[WebBaseNetscapePluginView evaluateJavaScriptPluginRequest:]):
2251         Call -willCallPlugInFunction and -didCallPlugInFunction around calls to the NPP_* functions.
2252         (-[WebBaseNetscapePluginView webFrame:didFinishLoadWithReason:]):
2253         ditto
2254         (-[WebBaseNetscapePluginView _printedPluginBitmap]):
2255         ditto
2256
2257         * Plugins/WebBaseNetscapePluginStream.m:
2258         (-[WebBaseNetscapePluginStream startStreamResponseURL:expectedContentLength:lastModifiedDate:MIMEType:]):
2259         Call -willCallPlugInFunction and -didCallPlugInFunction around calls to the NPP_* functions.
2260         (-[WebBaseNetscapePluginStream _destroyStream]):
2261         ditto
2262         (-[WebBaseNetscapePluginStream _deliverData]):
2263         ditto
2264
2265 2006-08-01  Maciej Stachowiak  <mjs@apple.com>
2266
2267         - fix build after last change
2268
2269         * WebView/WebFrame.m:
2270         (-[WebFrame _checkLoadCompleteForThisFrame]):
2271
2272 2006-08-01  Maciej Stachowiak  <mjs@apple.com>
2273
2274         Reviewed by Beth.
2275
2276         - revert part of my last fix that broke the Safari bookmarks view
2277         
2278         * WebView/WebFrame.m:
2279         (-[WebFrame _checkLoadCompleteForThisFrame]): still send layout message for non-HTML views
2280
2281 2006-08-01  Tim Omernick  <timo@apple.com>
2282
2283         Reviewed by Anders.
2284
2285         Fixed an assertion failure I ran into while debugging <rdar://problem/4652683>.
2286
2287         * Plugins/WebNetscapePluginEmbeddedView.m:
2288         (-[WebNetscapePluginEmbeddedView redeliverStream]):
2289         Don't clear the "instance" ivar here.  This code was refactored here from the old WebNetscapePluginRepresentation,
2290         which also had an "instance" ivar.  It is never appropriate to clear a plug-in view's instance.  That is done when
2291         the plug-in is destroyed.
2292
2293 2006-08-01  Maciej Stachowiak  <mjs@apple.com>
2294
2295         Reviewed by Geoff.
2296         
2297         - some refactoring in preparation for moving more stuff to WebFrameLoader.
2298
2299         * WebView/WebFrame.m:
2300         (-[WebFrame _clearDataSource]): 
2301         (-[WebFrame _detachFromParent]):
2302         (-[WebFrame _commitProvisionalLoad]):
2303         (-[WebFrame _transitionToCommitted:]):
2304         (-[WebFrame _clearProvisionalLoad]):
2305         (-[WebFrame _markLoadComplete]):
2306         (-[WebFrame _checkLoadCompleteForThisFrame]):
2307         (-[WebFrame _startProvisionalLoad:]):
2308         (-[WebFrame _continueLoadRequestAfterNavigationPolicy:formState:]):
2309         (-[WebFrame stopLoading]):
2310
2311 2006-07-31  Maciej Stachowiak  <mjs@apple.com>
2312
2313         Reviewed by Tim Hatcher.
2314         
2315         - renamed TransferJob to ResourceLoader in WebCore
2316
2317         * COM/WebFrame.cpp:
2318         (WebFrame::loadDataSource):
2319         (WebFrame::receivedRedirect):
2320         (WebFrame::receivedResponse):
2321         (WebFrame::receivedData):
2322         (WebFrame::receivedAllData):
2323         (WebFrame::setStatusText):
2324         * COM/WebFrame.h:
2325
2326 2006-07-31  Darin Adler  <darin@apple.com>
2327
2328         Reviewed by Maciej.
2329
2330         - omit the margin and padding boxes for display types where they are ignored
2331         - use CSS instead of properties for table spacing and padding as suggested by Tim H.
2332
2333         * WebInspector/webInspector/inspector.css: Added rules for spacing and padding.
2334         Added rules that hide the margin and padding boxes (borders and all but the center cell)
2335         when the hide attribute is present.
2336         * WebInspector/webInspector/inspector.html: Added classes for the rules above.
2337         Removed cellpadding and cellspacing attributes.
2338         * WebInspector/webInspector/inspector.js: Added code to hide/show the margin and
2339         padding boxes based on the display type.
2340
2341 2006-07-31  Duncan Wilcox  <duncan@mclink.it>
2342
2343         Reviewed by Darin.
2344
2345         Fixes <http://bugzilla.opendarwin.org/show_bug.cgi?id=10159>
2346         "REGRESSION: delegate returning no menu elements crashes webkit"
2347
2348         No automated test, because there's no way to programmatically open a context menu,
2349         no manual test because there's no way to customize the context menu delegate.
2350
2351         * WebView/WebView.m:
2352         (-[WebView _menuForElement:defaultItems:]): Make sure the context menu returned
2353         some menu items before accessing the first one.
2354
2355 2006-07-31  Timothy Hatcher  <timothy@apple.com>
2356
2357         Reviewed by John.
2358
2359         <rdar://problem/4658194> REGRESSION: "Search in Google"
2360         and "Search in Spotlight" fail to work on text selected in a frame
2361
2362         Use selectedFrame to get the frame with the text selection.
2363
2364         * WebView/WebView.m:
2365         (-[WebView _searchWithGoogleFromMenu:]):
2366         (-[WebView _searchWithSpotlightFromMenu:]):
2367
2368 2006-07-31  Darin Adler  <darin@apple.com>
2369
2370         Reviewed by Tim Hatcher.
2371
2372         - http://bugzilla.opendarwin.org/show_bug.cgi?id=10168
2373           add a first cut at a Metrics pane to the inspector
2374
2375         * WebInspector/webInspector/inspector.css: Add styles for the new metrics pane.
2376         * WebInspector/webInspector/inspector.html: Add the new metrics pane, starting with
2377         the table to show the box model.
2378         * WebInspector/webInspector/inspector.js: Add the new metrics pane. Add back some
2379         "title" attributes so we have more tooltips. Removed the optional parameter to
2380         getComputedStyle.
2381
2382 2006-07-31  Anders Carlsson  <acarlsson@apple.com>
2383
2384         Reviewed by John.
2385
2386         * Plugins/WebPluginDatabase.m:
2387         (-[WebPluginDatabase refresh]):
2388         Create a mutable set instead of a mutable array.
2389
2390 2006-07-30  Darin Adler  <darin@apple.com>
2391
2392         Reviewed by Tim Hatcher.
2393
2394         * WebInspector/webInspector/inspector.js: Fix bug where a null property value
2395         leads to an empty style pane.
2396
2397 2006-07-30  Darin Adler  <darin@apple.com>
2398
2399         Reviewed by Tim Hatcher.
2400
2401         - http://bugzilla.opendarwin.org/show_bug.cgi?id=10163
2402           some improvements for the inspector
2403
2404         * WebInspector/WebInspector.m:
2405         (+[WebInspector sharedWebInspector:]): Fixed bug that could cause the inspector
2406         to be garbage collected if used in an application with GC enabled.
2407         (-[WebInspector dealloc]): Removed a call to a non-existent close method.
2408         (-[WebInspector window]): Added a custom WebPreferences object and called
2409         setPrivateBrowsingEnabled:YES so the inspector won't appear in the history menu.
2410         Also call setProhibitsMainFrameScrolling:YES to try to get rid of trouble where
2411         the inspector scrolls when dragging.
2412
2413         * WebInspector/webInspector/inspector.css: Added style for the new color swatch,
2414         and JavaScript properties. More of the style should be shared between the panes,
2415         but this should be OK for now.
2416
2417         * WebInspector/webInspector/inspector.html: Added a first cut at a JavaScript
2418         properties pane. Needs work, but better than nothing.
2419
2420         * WebInspector/webInspector/inspector.js: Lots of improvements:
2421         - Omit "typical" property values from computed style display, making it much shorter.
2422         - Use the words "black", "white", and "transparent" when appropriate for color values.
2423         - Refactored the loaded() function to get rid of repetitive scrollbar setup.
2424         - Added a new scrollarea for the JavaScript properties pane.
2425         - Simplified refreshScrollbars() -- we now refresh all scrollbars every time, which does no harm.
2426         - Removed unused resultsWithXpathQuery().
2427         - Use [] instead of "new Array()" and {} instead of "new Object()".
2428         - Removed unused xpathForNode().
2429         - Changed style pane to display the style for a text node's parent instead of saying
2430           it can't display the style for text.
2431         - Fixed regression I caused a while back by checking the length of a computed style
2432           and not trying to display anything if its length is 0. Before this change and the
2433           corresponding change in WebCore, we'd see a complete list of all styles with the
2434           empty string as the value for each one.
2435         - Changed the name of the computedStyle flag on the style rules array to isComputedStyle
2436           to make it easier to understand it's a boolean.
2437         - Fixed an error in the code that does !important scanning where it was trying to
2438           do a special case for computed style, but was checking the computed style flag on
2439           the wrong object.
2440         - Added populateStyleListItem() function to factor out things in common between the
2441           items in the top level list and the expanded tree for shorthand properties.
2442         - Added code to make a color swatch next to the textual representation for any
2443           property that contains a color.
2444         - Implemented a first cut at a simple JavaScript properties pane.
2445
2446 2006-07-29  Darin Adler  <darin@apple.com>
2447
2448         - Removed tabs from these source files that still had them.
2449           We don't use them; that way source files look fine in editors
2450           that have tabs set to 8 spaces or to 4 spaces.
2451         - Removed allow-tabs Subversion property from the files too.
2452
2453         * DefaultDelegates/WebDefaultPolicyDelegate.m:
2454         * History/WebHistory.m:
2455         * Misc/WebDownload.m:
2456         * Misc/WebIconDatabase.m:
2457         * Misc/WebKitErrors.m:
2458         * Misc/WebKitLogging.m:
2459         * Misc/WebNSDataExtras.m:
2460         * Misc/WebNSFileManagerExtras.m:
2461         * Panels/WebPanelAuthenticationHandler.m:
2462         * Plugins/WebBaseNetscapePluginView.m:
2463         * Plugins/npfunctions.h:
2464         * WebCoreSupport/WebSubresourceLoader.m:
2465         * WebView/WebMainResourceLoader.m:
2466         * WebView/WebView.h:
2467         * WebView/WebView.m:
2468
2469 2006-07-29  Sam Weinig  <sam.weinig@gmail.com>
2470
2471         Reviewed by Darin.
2472
2473         - patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10080
2474           Adopt pedantic changes from the Unity project to improve 
2475           cross-compiler compatibility
2476
2477           Changes include:
2478           * Adding missing newline to the end of the file.
2479           * Turning on gcc warning for missing newline at the end of a source file
2480             (GCC_WARN_ABOUT_MISSING_NEWLINE in Xcode, -Wnewline in gcc).
2481
2482         * WebKit.xcodeproj/project.pbxproj:
2483         * WebView/WebResourcePrivate.h:
2484
2485 2006-07-29  Mitz Pettel  <opendarwin.org@mitzpettel.com>
2486
2487         Reviewed by John Sullivan.
2488
2489          - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9984
2490           ASSERTION FAILURE: _private->mouseDownEvent != nil
2491           (WebKit/WebView/WebHTMLView.m:4863 -[WebHTMLView(WebInternal) _delegateDragSourceActionMask])
2492
2493         * WebView/WebHTMLView.m:
2494         (-[WebHTMLView _setMouseDownEvent:]): Moved into the WebHTMLViewFileInternal category and changed
2495         to accept nil.
2496         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
2497         Copy the hit HTMLView's mouse down event to the top HTMLView.
2498         (-[WebHTMLView acceptsFirstMouse:]): Added a call to _setMouseDownEvent:nil before returning.
2499         (-[WebHTMLView shouldDelayWindowOrderingForEvent:]): Added a call to _setMouseDownEvent:nil
2500         before returning.
2501         (-[WebHTMLView mouseUp:]): Added a call to _setMouseDownEvent:nil to clear the event set in
2502         mouseDown: (and used during dragging).
2503         (-[WebHTMLView _delegateDragSourceActionMask]): Copy the hit HTMLView's mouse down event to
2504         the top HTMLView.
2505
2506 2006-07-28  Timothy Hatcher  <timothy@apple.com>
2507
2508         Reviewed by John.
2509
2510         <rdar://problem/4657473> REGRESSION: Spell check not available from contextual menu in Mail
2511
2512         The context menu code should be checking isContentEditable
2513         on DOMNode not just DOMElement. This is needed because DOMText
2514         will be the node class of any text that is clicked.
2515
2516         * DefaultDelegates/WebDefaultContextMenuDelegate.m:
2517         (-[WebDefaultUIDelegate webView:contextMenuItemsForElement:defaultMenuItems:]):
2518
2519 2006-07-25  Geoffrey Garen  <ggaren@apple.com>
2520
2521         Reviewed by Maciej, inspired by John.
2522         
2523         - Fixed <rdar://problem/4651931> 1% REGRESSION on iBench HTML due to 
2524         repeated requests for non-existent favicon
2525         
2526         An optimization to avoid serializing favicon data for missing icons had stomped an
2527         optimization to avoid GETing a missing favicon more than once. The solution
2528         is a happy marriage of optimizations, ensuring that we *retain* the missing
2529         favicon's "i am missing" data without posting a notification or saving it to disk.
2530
2531         * Misc/WebIconDatabase.m:
2532         (-[WebIconDatabase _setIconURL:forURL:]):
2533
2534 2006-07-25  David Harrison  <harrison@apple.com>
2535
2536         Reviewed by timo and Darin.
2537
2538         <rdar://problem/4618584> "Paste and Match Style" is not working in Mail (add SPI)
2539         
2540         * WebKit.xcodeproj/project.pbxproj:
2541         * WebView/WebView.m:
2542         (-[WebView replaceSelectionWithNode:]):
2543         (-[WebView _replaceSelectionWithNode:matchStyle:]):
2544         * WebView/WebViewPrivate.h:
2545         (-[WebView _replaceSelectionWithNode:matchStyle::]):
2546         New SPI that is same as replaceSelectionWithNode: with added parameter whether to match existing style.
2547
2548 2006-07-24  Darin Adler  <darin@apple.com>
2549
2550         Reviewed by Adele and Justin.
2551
2552         - update for change to require context when creating fragments from text
2553           (needed to handle whitespace properly)
2554
2555         * WebView/WebHTMLView.m:
2556         (-[WebHTMLView _documentFragmentFromPasteboard:inContext:allowPlainText:chosePlainText:]):
2557         Added context parameter, pass through to bridge.
2558         (-[WebHTMLView _pasteWithPasteboard:allowPlainText:]): Pass selection range as context
2559         when calling above method.
2560         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): Pass drag caret as context when
2561         calling above method.
2562
2563 2006-07-24  Maciej Stachowiak  <mjs@apple.com>
2564
2565         Reviewed by Geoff.
2566         
2567         - fix <rdar://problem/4609195> Help Viewer loads empty window (not getting didFailLoadingWithError: callback)
2568         (without re-introducing http://bugzilla.opendarwin.org/show_bug.cgi?id=10062 )
2569
2570         * WebView/WebLoader.h:
2571         * WebView/WebMainResourceLoader.m:
2572         (-[WebMainResourceLoader receivedError:]): Copy in some code from the base class to do it in the proper
2573         order, surrounding the call to [ds _receivedMainResourceError:error complete:YES].
2574
2575 2006-07-24  Anders Carlsson  <acarlsson@apple.com>
2576
2577         Reviewed by Tim O.
2578
2579         * Misc/WebIconDatabase.m:
2580         (-[WebIconDatabase removeAllIcons]):
2581         Make an array of the keys and iterate through it to avoid modifying the
2582         dictionary while enumerating it.
2583
2584 2006-07-24  Timothy Hatcher  <timothy@apple.com>
2585
2586         Reviewed by John and Darin.
2587
2588         <rdar://problem/4634290> Cannot selectively install a custom
2589         scroller that differs from the default Aqua frame size.
2590
2591         Adds two new private methods to WebFrameView that allows
2592         an application to set a custom scroll view class. This is needed
2593         if the application wants to install a custom scroller that is wider
2594         than the typical scroller, because NSScrollView does the content
2595         rect calculations in a class method (ignoring custom scrollers.)
2596         The _setScrollViewClass method requires the class to be a subclass
2597         of WebDynamicScrollBarView, or nil can be passed to reset to the default class.
2598         A new scroll view of the specified class will then replace the previous
2599         one without the need to reload content of the frame.
2600
2601         * WebView/WebFrameView.m:
2602         (-[WebFrameView _customScrollViewClass]):
2603         (-[WebFrameView _setCustomScrollViewClass:]):
2604         * WebView/WebFrameViewPrivate.h:
2605
2606 2006-07-24  Alexey Proskuryakov  <ap@nypop.com>
2607
2608         Reviewed by Darin.
2609
2610         Fix http://bugzilla.opendarwin.org/show_bug.cgi?id=10009
2611         REGRESSION: Schubert-IT PDF Plug-in not working for full page (works in frames)
2612
2613         * WebView/WebView.m:
2614         (+[WebView _viewClass:andRepresentationClass:forMIMEType:]): If we've got a type supported by WebPDFView,
2615         make sure to initialize the plugin database, in case a plugin wants to handle it.
2616
2617 2006-07-23  Mark Rowe  <opendarwin.org@bdash.net.nz>
2618
2619         Reviewed by Maciej.
2620
2621         Bug 9686: [Drosera] Need the ability to break into Drosera on Javascript exceptions
2622         http://bugzilla.opendarwin.org/show_bug.cgi?id=9686
2623
2624         WebKit portion of the fix.
2625
2626         * DefaultDelegates/WebDefaultScriptDebugDelegate.m:
2627         (-[WebDefaultScriptDebugDelegate webView:exceptionWasRaised:sourceId:line:forWebFrame:]):
2628         * DefaultDelegates/WebScriptDebugServer.h:
2629         * DefaultDelegates/WebScriptDebugServer.m:
2630         (-[WebScriptDebugServer webView:exceptionWasRaised:sourceId:line:forWebFrame:]): Notify
2631         listeners that an exception has been raised.
2632         * WebView/WebScriptDebugDelegate.h:
2633         * WebView/WebScriptDebugDelegate.m:
2634         (-[WebScriptCallFrame exceptionRaised:sourceId:line:]): Dispatch through to delegate and
2635         WebScriptDebugServer.
2636
2637 2006-07-23  Adele Peterson  <adele@apple.com>
2638
2639         Reviewed by Darin.
2640
2641         - Fix for <rdar://problem/4646276> CrashTracer: 7 crashes in Safari at com.apple.WebCore: WebCore::RenderTableSection::paint + 155
2642
2643         * WebView/WebHTMLView.m: (-[WebHTMLView _web_layoutIfNeededRecursive:testDirtyRect:]):
2644         needsDisplay was returning NO even though the view has a dirty rect (see <rdar://problem/4647062>).  Since we know about
2645         the dirty rect, we don't actually need to check needsDisplay.
2646
2647 2006-07-22  Timothy Hatcher  <timothy@apple.com>
2648
2649         Rolling out r15572.
2650         
2651         Bug 10062: REGRESSION: dom/xhtml/level2/html/HTMLIFrameElement11.xhtml asserts/crashes
2652         http://bugzilla.opendarwin.org/show_bug.cgi?id=10062
2653
2654     2006-07-21  Timothy Hatcher  <timothy@apple.com>
2655
2656         Reviewed by Maciej.
2657
2658         <rdar://problem/4609195> Help Viewer loads empty window
2659         (not getting didFailLoadingWithError: callback)
2660
2661         Call super's didFailWithError before _receivedMainResourceError
2662         because _receivedMainResourceError will cause the datasource's
2663         frame to be set to nil before the didFailLoadingWithError delegate
2664         callback is sent. (This order is needed now that WebDataSource does
2665         not hold on to the WebView; it uses the WebFrame to get to the WebView.
2666         If the WebFrame is nil we can't get to the WebView's resource load delegate.)
2667
2668         * WebView/WebMainResourceLoader.m:
2669         (-[WebMainResourceLoader receivedError:]):
2670
2671
2672 2006-07-22  Timothy Hatcher  <timothy@apple.com>
2673
2674         Reviewed by Adele.
2675
2676         <rdar://problem/4646318> REGRESSION: Ctrl-clicking on a selection containing a word doesn't display a complete contextual menu
2677         
2678         Show the editing context menu if the WebView is editible.
2679         The original change only checked if the DOM element was editable,
2680         and isContentEditable returns NO if entire WebView is editable.
2681
2682         * DefaultDelegates/WebDefaultContextMenuDelegate.m:
2683         (-[WebDefaultUIDelegate webView:contextMenuItemsForElement:defaultMenuItems:]):
2684
2685 2006-07-21  Timothy Hatcher  <timothy@apple.com>
2686
2687         Reviewed by Maciej.
2688
2689         <rdar://problem/4609195> Help Viewer loads empty window
2690         (not getting didFailLoadingWithError: callback)
2691
2692         Call super's didFailWithError before _receivedMainResourceError
2693         because _receivedMainResourceError will cause the datasource's
2694         frame to be set to nil before the didFailLoadingWithError delegate
2695         callback is sent. (This order is needed now that WebDataSource does
2696         not hold on to the WebView; it uses the WebFrame to get to the WebView.
2697         If the WebFrame is nil we can't get to the WebView's resource load delegate.)
2698
2699         * WebView/WebMainResourceLoader.m:
2700         (-[WebMainResourceLoader receivedError:]):
2701
2702 === Safari-521.20 ===
2703
2704 2006-07-21  Timothy Hatcher  <timothy@apple.com>
2705
2706         Reviewed by John.
2707
2708         <rdar://problem/4607572> REGRESSION (521.10.1 - 521.13): most context menu items missing when a form field is focused (common on google.com) (9680)
2709
2710         Do not use _isEditable call since that only checks if the current
2711         selection or frame is editible. We now check if the currently clicked element
2712         is a content editible area, a textarea, an isindex or an input element that
2713         return YES to _isTextField.
2714
2715         * DefaultDelegates/WebDefaultContextMenuDelegate.m:
2716         (-[WebDefaultUIDelegate webView:contextMenuItemsForElement:defaultMenuItems:]):
2717
2718 2006-07-20  John Sullivan  <sullivan@apple.com>
2719
2720         Reviewed by Maciej
2721
2722         - WebKit part of fix for:
2723         <rdar://problem/4557386> REGRESSION (419.3-521.19): repro Safari world leak involving 
2724         closing tabs after clicking in a web page
2725
2726         * WebCoreSupport/WebFrameBridge.m:
2727         (-[WebFrameBridge textViewWasFirstResponderAtMouseDownTime:]):
2728         renamed to be more specific (formerly wasFirstResponderAtMouseDownTime:)        
2729         
2730         * WebView/WebHTMLViewInternal.h:
2731         * WebView/WebHTMLView.m:
2732         (-[WebTextCompleteController dealloc]):
2733         updated for name change
2734         (-[NSArray _setMouseDownEvent:]):
2735         Now only retains the first responder if it's a textView, since that's the only case that the only client
2736         actually cares about. This avoids a reference cycle caused by retaining self. This is the only substantive
2737         part of the patch; all the rest is just renaming for clarity, and comments.
2738         (-[NSArray mouseDown:]):
2739         updated for name change
2740         (-[WebHTMLView _textViewWasFirstResponderAtMouseDownTime:]):
2741         renamed to be more specific (formerly _wasFirstResponderAtMouseDownTime:)        
2742
2743 2006-07-19  Tim Omernick  <timo@apple.com>
2744
2745         Reviewed by Darin.
2746
2747         <rdar://problem/4523432> safari crashed right after disabling "block pop up windows" (or other WebPreferences changes)
2748
2749         * Plugins/WebBaseNetscapePluginView.m:
2750         (-[WebBaseNetscapePluginView viewWillMoveToSuperview:]):
2751         Stop the plug-in when it is removed from its superview.  It is not sufficient to do this in -viewWillMoveToWindow:nil, because
2752         the WebView might still has a hostWindow at that point, which prevents the plug-in from being destroyed.
2753         There is no need to start the plug-in when moving into a superview.  -viewDidMoveToWindow takes care of that.
2754
2755 === Safari-521.19 ===
2756
2757 2006-07-17  Tim Omernick  <timo@apple.com>
2758
2759         Reviewed by Maciej.
2760
2761         <rdar://problem/4612079> need a way to prevent pages from scrolling to reveal elements that are focused
2762         by script
2763
2764         * WebView/WebViewPrivate.h:
2765         * WebView/WebView.m:
2766         (-[WebView setProhibitsMainFrameScrolling:]):
2767         New method.  Prohibits scrolling in the WebView's main frame.  Used to "lock" a WebView to a specific
2768         scroll position.
2769
2770 2006-07-17  Timothy Hatcher  <timothy@apple.com>
2771
2772         Reviewed by Darin.
2773         
2774         <rdar://problem/4635311> REGRESSION: WebKit should call windowScriptObjectAvailable before attaching the script debugger
2775
2776         * WebCoreSupport/WebFrameBridge.m:
2777         (-[WebFrameBridge windowObjectCleared]):
2778
2779 2006-07-17  Timothy Hatcher  <timothy@apple.com>
2780
2781         Reviewed by Maciej.
2782
2783         <rdar://problem/4634874> WebScriptObject and WebUndefined are no longer defined by WebKit
2784
2785         Copy WebScriptObject.h from WebCore's private headers, not JavaScriptCore.
2786
2787         * WebKit.xcodeproj/project.pbxproj:
2788
2789 2006-07-17  John Sullivan  <sullivan@apple.com>
2790
2791         Reviewed by Tim Omernick.
2792         
2793         - fixed <rdar://problem/4604366> Orange Find highlight displays text in wrong size on PDF pages 
2794         if they're not at "actual size"
2795         
2796         To match WebHTMLView, I made the methods that return attributed strings take the view's scale
2797         factor into account.
2798
2799         * WebView/WebPDFView.m:
2800         (-[WebPDFView _scaledAttributedString:]):
2801         new helper method, takes an attributed string and returns one that's scaled by the view's
2802         current scale factor
2803         (-[WebPDFView attributedString]):
2804         pass result through _scaledAttributedString:
2805         (-[WebPDFView selectedAttributedString]):
2806         ditto
2807
2808 2006-07-17  Justin Garcia  <justin.garcia@apple.com>
2809
2810         Reviewed by levi
2811         
2812         Rolled the first fix for:
2813         <http://bugzilla.opendarwin.org/show_bug.cgi?id=9642>
2814         GMail Editor: Operations that use drop down menus blow away the selection
2815         back in and removed the call to _clearSelectionInOtherFrames from 
2816         -[WebHTMLView becomeFirstResponder] to fix the bug.
2817
2818         * WebView/WebHTMLView.m:
2819         (-[NSArray maintainsInactiveSelection]):
2820         (-[NSArray becomeFirstResponder]):
2821         * WebView/WebView.m:
2822         (-[WebView maintainsInactiveSelection]):
2823
2824 2006-07-15  Darin Adler  <darin@apple.com>
2825
2826         Reviewed by John Sullivan.
2827
2828         - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9928
2829           REGRESSION: Text Encoding menu inoperative (after gcc protocol build fix)
2830
2831         * WebView/WebHTMLView.m:
2832         (-[WebHTMLView _documentRange]): Moved into WebHTMLViewFileInternal category.
2833         (-[WebHTMLView selectionRect]): Moved into WebDocumentPrivateProtocols category.
2834         (-[WebHTMLView selectionView]): Ditto.
2835         (-[WebHTMLView selectionImageForcingWhiteText:]): Ditto.
2836         (-[WebHTMLView selectionImageRect]): Ditto.
2837         (-[WebHTMLView pasteboardTypesForSelection]): Ditto.
2838         (-[WebHTMLView selectAll]): Ditto.
2839         (-[WebHTMLView deselectAll]): Ditto.
2840         (-[WebHTMLView string]): Ditto.
2841         (-[WebHTMLView _attributeStringFromDOMRange:]): Ditto.
2842         (-[WebHTMLView attributedString]): Ditto.
2843         (-[WebHTMLView selectedString]): Ditto.
2844         (-[WebHTMLView selectedAttributedString]): Ditto.
2845         (-[WebHTMLView supportsTextEncoding]): Ditto.
2846         (-[WebHTMLView _canProcessDragWithDraggingInfo:]): Moved into WebDocumentInternalProtocols.
2847         (-[WebHTMLView _isMoveDrag]): Ditto.
2848         (-[WebHTMLView _isNSColorDrag:]): Ditto.
2849         (-[WebHTMLView draggingUpdatedWithDraggingInfo:actionMask:]): Ditto.
2850         (-[WebHTMLView draggingCancelledWithDraggingInfo:]): Ditto.
2851         (-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): Ditto.
2852         (-[WebHTMLView elementAtPoint:]): Ditto.
2853         (-[WebHTMLView elementAtPoint:allowShadowContent:]): Ditto.
2854
2855         * WebKit.xcodeproj/project.pbxproj: Let Xcode 2.3 do its thing.
2856
2857 === Safari-521.17 ===
2858
2859 2006-07-14  Timothy Hatcher  <timothy@apple.com>
2860
2861         Rolling out this fix from r15358 since it isn't resolved.
2862
2863     2006-07-11  Justin Garcia  <justin.garcia@apple.com>
2864
2865         Reviewed by levi & thatcher
2866
2867         <http://bugzilla.opendarwin.org/show_bug.cgi?id=9642>
2868         GMail Editor: Operations that use drop down menus blow away the selection
2869
2870         * WebView/WebHTMLView.m:
2871         (-[WebHTMLView maintainsInactiveSelection]): Maintain an inactive selection
2872         when resigning as first responder if the selection is editable
2873         or if the WebView tells us to.
2874         * WebView/WebView.m:
2875         (-[WebView maintainsInactiveSelection]): Just because a WebView is
2876         editable doesn't mean selections inside subframes will be.  Return
2877         NO by default.
2878
2879 2006-07-14  Timothy Hatcher  <timothy@apple.com>
2880
2881         <rdar://problem/4623957> SWB: gcc-5412 (new?) objc warning causes WebCore project failure
2882
2883         Build fix with the new GCC. Removes forward declarations of protocols.
2884
2885         * Misc/WebSearchableTextView.h:
2886         * WebCoreSupport/WebSubresourceLoader.h:
2887         * WebKit.xcodeproj/project.pbxproj:
2888         * WebView/WebDocumentInternal.h:
2889         * WebView/WebDocumentPrivate.h:
2890         * WebView/WebHTMLView.h:
2891         * WebView/WebPDFView.h:
2892         * WebView/WebScriptDebugDelegatePrivate.h:
2893
2894 2006-06-28  Darin Adler  <darin@apple.com>
2895
2896         Reviewed by Adele.
2897
2898         - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9625
2899           <rdar://problem/4604703>
2900           REGRESSION: Focus not removed from password field after ctrl-click in text field
2901
2902         * WebView/WebHTMLView.m: (-[WebHTMLView menuForEvent:]): Set handlingMouseDownEvent to
2903         YES while calling sendContextMenuEvent: on the bridge.
2904
2905 2006-07-14  Timothy Hatcher  <timothy@apple.com>
2906
2907         Reviewed by Maciej.
2908
2909         Moved JavaScriptCore to be a public framework.
2910
2911         * WebKit.xcodeproj/project.pbxproj:
2912
2913 2006-07-13  Mitz Pettel  <opendarwin.org@mitzpettel.com>
2914
2915         Reviewed by Darin.
2916
2917         - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9795
2918           REGRESSION: Crash in [WebHTMLView(WebPrivate) 
2919           _updateMouseoverWithEvent:]
2920           and http://bugzilla.opendarwin.org/show_bug.cgi?id=9850
2921           REGRESSION: Assertion failure (SHOULD NEVER BE REACHED) in -
2922           [WebHTMLView(WebPrivate) removeTrackingRect:]
2923
2924         * WebView/WebHTMLView.m:
2925         (-[WebHTMLView _updateMouseoverWithEvent:]): Return immediately if 
2926         the view has already been closed.
2927
2928 2006-07-13  David Harrison  <harrison@apple.com>
2929
2930         Reviewed by Justin and Levi.
2931
2932         <rdar://problem/4620743> REGRESSION: Option-Delete doesn't delete words during typing
2933
2934         * Tests:
2935         editing/deleting/delete-by-word-001.html
2936         editing/deleting/delete-by-word-002.html
2937
2938         * WebView/WebHTMLView.m:
2939         (-[WebHTMLView _deleteRange:killRing:prepend:smartDeleteOK:deletionAction:granularity:]):
2940
2941 2006-07-13  Timothy Hatcher  <timothy@apple.com>
2942
2943         Rolling out this earlier change (r15378) now that it is fixed on AGL's end.
2944         Fixes <rdar://problem/4624865> Restore 64-bit OpenGL plug-in support once AGL is 64-bit
2945
2946         <rdar://problem/4624858> AGL isn't 64-bit yet; temporarily remove it from WebKit 64-bit build
2947
2948         * Plugins/WebBaseNetscapePluginView.h:
2949         * Plugins/WebBaseNetscapePluginView.m:
2950
2951 2006-07-13  Timothy Hatcher  <timothy@apple.com>
2952
2953         Reviewed by Darin.
2954
2955         <rdar://problem/4616920> REGRESSION: tabbing in mail moves focus
2956         to next control instead of inserting a tab space.
2957
2958         Change editible WebView's tabKeyCyclesThroughElements to NO only
2959         if the setTabKeyCyclesThroughElements SPI wasn't called.
2960
2961         * WebView/WebView.m:
2962         (-[WebView setEditable:]):
2963
2964 2006-07-12  Anders Carlsson  <acarlsson@apple.com>
2965
2966         Reviewed by Darin.
2967
2968         http://bugzilla.opendarwin.org/show_bug.cgi?id=9624
2969         REGRESSION: After ctrl-clicking in a EMPTY input or textarea field, the contextual menu shows "Search in Google" and "Search in Spotlight" as active menu items
2970
2971         * DefaultDelegates/WebDefaultContextMenuDelegate.m:
2972         (-[WebDefaultUIDelegate editingContextMenuItemsForElement:defaultMenuItems:]):
2973         Don't create Dictionary, Spotlight or Google lookup items if there's no selection.
2974
2975 2006-07-12  Mark Rowe  <opendarwin.org@bdash.net.nz>
2976
2977         Reviewed by Timothy.
2978
2979         http://bugzilla.opendarwin.org/show_bug.cgi?id=9868
2980         Applications shown in Drosera's "Attach" window remain after exit
2981
2982         * DefaultDelegates/WebScriptDebugServer.m:
2983         (-[WebScriptDebugServer init]): Register for NSApplicationWillTerminateNotification so we will
2984         know when the application is being exited.
2985         (-[WebScriptDebugServer dealloc]): Unregister notification before we are deallocated.
2986         (-[WebScriptDebugServer applicationTerminating:]): Inform anyone listening that we are going away.
2987
2988 2006-07-12  Tim Omernick  <timo@apple.com>
2989
2990         Reviewed by Tim Hatcher.
2991
2992         <rdar://problem/4624858> AGL isn't 64-bit yet; temporarily remove it from WebKit 64-bit build
2993
2994         Also, fixed a LOG_ERROR() so that it uses the CGL error instead of the AGL error; Tim H missed this in his build fix
2995         from earlier.
2996
2997         * Plugins/WebBaseNetscapePluginView.h:
2998         * Plugins/WebBaseNetscapePluginView.m:
2999
3000 2006-07-11  John Sullivan  <sullivan@apple.com>
3001
3002         Reviewed by Kevin and Tim O
3003
3004         - added support for creating a selection image with white text
3005
3006         * WebView/WebDocumentPrivate.h:
3007         added -selectionImageForcingWhiteText: and -selectionImageRect to the private
3008         <WebDocumentSelection> protocol
3009
3010         * Misc/WebSearchableTextView.m:
3011         (-[NSString selectionImageForcingWhiteText:]):
3012         added stub for this new method to this obsolete class to satisfy the compiler
3013         (-[NSString selectionImageRect]):
3014         ditto
3015         
3016         * WebView/WebHTMLView.m:
3017         (-[WebHTMLView _selectionDraggingImage]):
3018         now calls -selectionImageForcingWhiteText:NO instead of just -selectionImage
3019         (-[WebHTMLView _selectionDraggingRect]):
3020         now calls selectionImageRect, to which the implementation moved
3021         (-[WebHTMLView selectionImageForcingWhiteText:]):
3022         implemented this new method by calling through to new bridge method selectionImageForcingWhiteText:
3023         (-[WebHTMLView selectionImageRect]):
3024         implemented this new method by using existing _selectionDraggingRect implementation
3025         
3026         * WebView/WebPDFView.m:
3027         (-[WebPDFView selectionImageForcingWhiteText:]):
3028         implemented by using code that was formerly in Safari
3029         (-[WebPDFView selectionImageRect]):
3030         implemented by returning selectionRect
3031
3032 2006-07-11  Tim Omernick  <timo@apple.com>
3033
3034         Reviewed by Geoff.
3035
3036         <http://bugzilla.opendarwin.org/show_bug.cgi?id=9843>:
3037         Give Netscape plug-ins access to their own DOM element
3038
3039         * Plugins/WebBaseNetscapePluginView.h:
3040         * Plugins/WebBaseNetscapePluginView.m:
3041         (-[WebBaseNetscapePluginView dealloc]):
3042         Release DOM element.
3043         (-[WebBaseNetscapePluginView getVariable:value:]):
3044         Return NPObject for plugin DOM element.
3045
3046         * Plugins/WebNetscapePluginEmbeddedView.h:
3047         * Plugins/WebNetscapePluginEmbeddedView.m:
3048         (-[WebNetscapePluginEmbeddedView initWithFrame:plugin:URL:baseURL:MIMEType:attributeKeys:attributeValues:loadManually:DOMElement:]):
3049         Now takes a DOMElement, in much the same way that WebKit plug-in views take a DOMElement.
3050
3051         * WebCoreSupport/WebFrameBridge.m:
3052         (-[WebFrameBridge viewForPluginWithURL:attributeNames:attributeValues:MIMEType:DOMElement:loadManually:]):
3053         Pass DOMElement to Netscape plug-ins.
3054         (-[WebFrameBridge viewForJavaAppletWithFrame:attributeNames:attributeValues:baseURL:DOMElement:]):
3055         ditto
3056
3057 2006-07-11  Justin Garcia  <justin.garcia@apple.com>
3058
3059         Reviewed by levi & thatcher
3060         
3061         <http://bugzilla.opendarwin.org/show_bug.cgi?id=9642>
3062         GMail Editor: Operations that use drop down menus blow away the selection
3063         
3064         * WebView/WebHTMLView.m:
3065         (-[WebHTMLView maintainsInactiveSelection]): Maintain an inactive selection
3066         when resigning as first responder if the selection is editable
3067         or if the WebView tells us to.
3068         * WebView/WebView.m:
3069         (-[WebView maintainsInactiveSelection]): Just because a WebView is
3070         editable doesn't mean selections inside subframes will be.  Return
3071         NO by default.
3072
3073 2006-07-11  Tim Omernick  <timo@apple.com>
3074
3075         Reviewed by Tim Hatcher.
3076
3077         <rdar://problem/4622748> WebKit now uses deprecated AGL functions
3078
3079         * Plugins/WebBaseNetscapePluginView.m:
3080         (-[WebBaseNetscapePluginView _createWindowedAGLContext]):
3081         aglSetDrawable() is deprecated in AGL 3.0.  Use aglSetWindowRef() instead.
3082         (-[WebBaseNetscapePluginView _createWindowlessAGLContext]):
3083         aglSetOffScreen() is deprecated in AGL 3.0.  Use CGLSetOffScreen(), which does the same thing.
3084
3085 2006-07-11  Alexey Proskuryakov  <ap@nypop.com>
3086
3087         Reviewed by Tim O.
3088
3089         - http://bugzilla.opendarwin.org/show_bug.cgi?id=7808
3090         Assertion failure in -[WebBaseNetscapePluginStream dealloc] when requesting an invalid URL
3091
3092         * Plugins/WebNetscapePluginStream.m:
3093         (-[WebNetscapePluginStream initWithRequest:pluginPointer:notifyData:sendNotification:]):
3094         Remove the early return when requesting an invalid (unsupported) URL.
3095
3096 === Safari-521.16 ===
3097
3098 2006-07-10  Mitz Pettel  <opendarwin.org@mitzpettel.com>
3099
3100         Reviewed by John Sullivan.
3101
3102         - fix <rdar://problem/4621541>, aka <http://bugzilla.opendarwin.org/show_bug.cgi?id=9838>
3103           REGRESSION (r14968-r14977): View Source doesn't work for pages from the back/forward cache
3104
3105         * WebView/WebDataSource.m:
3106         (-[WebDataSource _setPrimaryLoadComplete:]): Set our data only if the frame loader is
3107         has just loaded it (when coming from the back/forward cache, it hasn't).
3108
3109 2006-07-10  Brady Eidson  <beidson@apple.com>
3110
3111         Reviewed by Alexey
3112
3113         Resolved the console error messages people got from the new DB even if they didn't have it enabled
3114
3115         * Misc/WebIconDatabase.m:
3116         (-[WebIconDatabase init]):
3117         Disabled initializing the IconDatabaseBridge if user is living on the old DB
3118
3119 2006-07-10  Darin Adler  <darin@apple.com>
3120
3121         - try to fix Windows build
3122
3123         * COM/WebFrame.h: Qualify DeprecatedString and KURL with WebCore:: prefixes.
3124
3125 2006-07-09  Darin Adler  <darin@apple.com>
3126
3127         - try to fix Windows build
3128
3129         * COM/WebFrame.cpp: Rename QChar to DeprecatedChar.
3130
3131 2006-07-09  Darin Adler  <darin@apple.com>
3132
3133         - fix newlines to be consistent for all files in the COM directory
3134           (many had mixed style) and set the EOL style to "native" on them.
3135
3136         * COM/*: Set properties and changed files.
3137
3138 2006-07-09  Tim Omernick  <timo@apple.com>
3139
3140         Reviewed by Maciej.
3141
3142         <rdar://problem/4404652> Netscape plug-in mouse events broken in HiDPI
3143
3144         Multiply global mouse coordinates by the window scale factor so that plug-ins can use GlobalToLocal() in HiDPI.
3145         This fixes many bugs involving plug-in mouse event handling in HiDPI.  Most notably, the Flash player will now
3146         correctly respond to clicks.
3147
3148         * Plugins/WebBaseNetscapePluginView.m:
3149         (+[WebBaseNetscapePluginView getCarbonEvent:]):
3150         (-[WebBaseNetscapePluginView getCarbonEvent:withEvent:]):
3151
3152 2006-07-09  Darin Adler  <darin@apple.com>
3153
3154         Reviewed by Tim Hatcher.
3155
3156         - fix assertion firing in plug-in layout tests
3157
3158         * Plugins/WebBaseNetscapePluginView.m:
3159         (-[WebBaseNetscapePluginView restartNullEvents]):
3160         Don't start null events if the plug-in is not in the
3161         started state. This happens when the plug-in moves within
3162         its view hierarchy after it has been stopped.
3163
3164 2006-07-09  Timothy Hatcher  <timothy@apple.com>
3165
3166         Reviewed by Darin.
3167
3168         Bug 9820: Move new DOM API that has been through API review to public headers
3169         http://bugzilla.opendarwin.org/show_bug.cgi?id=9820
3170
3171         * Misc/WebElementDictionary.m: include DOMExtensions.h
3172         * Misc/WebNSViewExtras.m: include DOMExtensions.h
3173         * WebKit.xcodeproj/project.pbxproj: make DOMXPath.h public
3174
3175 2006-07-09  Timothy Hatcher  <timothy@apple.com>
3176
3177         Reviewed by Kevin.
3178
3179         Bug 9818: move new UIDelegate API that has been through API review to public headers
3180         http://bugzilla.opendarwin.org/show_bug.cgi?id=9818
3181
3182         <rdar://problem/4387541> API: Remove webView:setContentRect: & webViewContentRect: delegate methods?
3183         The fix for 4310363 removed the only use of webViewContentRect: in our code. webView:setContentRect:
3184         was never used to begin with. There's no harm in leaving these around in the API, but they'll cruft it up.
3185
3186         Also removes the never used webViewPrint: SPI that was replaced by webView:printFrameView:.
3187
3188         * DefaultDelegates/WebDefaultUIDelegate.m:
3189         * WebCoreSupport/WebFrameBridge.m:
3190         (-[WebFrameBridge print]):
3191         * WebView/WebFrameView.h:
3192         * WebView/WebFrameView.m:
3193         * WebView/WebFrameViewPrivate.h:
3194         * WebView/WebUIDelegate.h:
3195         * WebView/WebUIDelegatePrivate.h:
3196
3197 2006-07-09  Timothy Hatcher  <timothy@apple.com>
3198
3199         Reviewed by Darin.
3200
3201         Bug 9814: Move new WebView API that has been through API review to public headers
3202         http://bugzilla.opendarwin.org/show_bug.cgi?id=9814
3203
3204         * WebView/WebView.h:
3205         * WebView/WebView.m:
3206         (-[WebView close]):
3207         (-[WebView setShouldCloseWithWindow:]):
3208         (-[WebView shouldCloseWithWindow]):
3209         (-[WebView selectedFrame]):
3210         (-[WebView setMainFrameURL:]):
3211         (-[WebView mainFrameURL]):
3212         (-[WebView isLoading]):
3213         (-[WebView mainFrameTitle]):
3214         (-[WebView mainFrameIcon]):
3215         (-[WebView mainFrameDocument]):
3216         (-[WebView setDrawsBackground:]):
3217         (-[WebView drawsBackground]):
3218         (-[WebView toggleSmartInsertDelete:]):
3219         (-[WebView toggleContinuousSpellChecking:]):
3220         (-[WebView canMakeTextStandardSize]):
3221         (-[WebView makeTextStandardSize:]):
3222         (-[WebView maintainsInactiveSelection]):
3223         * WebView/WebViewPrivate.h:
3224
3225 2006-07-09  Timothy Hatcher  <timothy@apple.com>
3226
3227         Reviewed by Maciej.
3228
3229         Bug 9487: The XPath section should be removed and/or moved.
3230         http://bugzilla.opendarwin.org/show_bug.cgi?id=9487
3231
3232         * WebInspector/webInspector/inspector.css:
3233         * WebInspector/webInspector/inspector.html:
3234         * WebInspector/webInspector/inspector.js:
3235
3236 2006-07-09  Anders Carlsson  <acarlsson@apple.com>
3237
3238         Reviewed by Tim O.
3239
3240         * WebCoreSupport/WebSystemInterface.m:
3241         (InitWebCoreSystemInterface):
3242         Initialize wkPathFromFont.
3243
3244 2006-07-09  Darin Adler  <darin@apple.com>
3245
3246         - fix release build
3247
3248         * Plugins/WebBaseNetscapePluginView.m:
3249         (-[WebBaseNetscapePluginView restorePortState:]): Cast inside the assertion so
3250         that we don't have an unused variable in versions with assertions disabled.
3251         The alternative would be to wrap the whole thing in an #if statement.
3252
3253 2006-07-08  Tim Omernick  <timo@apple.com>
3254
3255         Reviewed by John Sullivan.
3256
3257         * Plugins/WebBaseNetscapePluginView.h:
3258         - Added ivars for OpenGL support.  Someday it would be nice to refactor this class so
3259           that each drawing model is encapsulated in a class; this would allow
3260           WebBaseNetscapePluginView to make more efficient use of space, for example by not
3261           keeping OpenGL-related ivars for Quickdraw plug-ins.
3262         
3263         * Plugins/WebBaseNetscapePluginView.m:
3264         - Declared a bunch of internal methods for OpenGL support (see below).
3265         - Removed "forUpdate" from CoreGraphics port state struct; it was always set to "YES",
3266           so I just cleaned up the silly code that used it.
3267         - Declared OpenGL port state struct.
3268         (-[WebBaseNetscapePluginView saveAndSetNewPortStateForUpdate:]):
3269         - Moved a CoreGraphics-related assertion down to the big "switch" statement.
3270         - Don't set window.type here -- according to the Netscape Plug-in API docs, the plug-in
3271           should default to "windowed" mode, and may call NPN_SetValue() during its NPN_New() to
3272           request that the browser use a "windowless" (offscreen) context instead.
3273         - Moved the assertion from the top of this method here; removed a less restrictive
3274           assertion that is now obsolete.
3275         - Removed "forUpdate" flag from CoreGraphics port state struct.
3276         - Fill in OpenGL port state struct.  Set up the viewport appropriately for both windowed
3277           and windowless OpenGL plug-ins.  Windowed plug-ins need to have their GL viewport
3278           transformed by the amount the plug-in is clipped; windowless plug-ins are drawn off-screen
3279           into a surface whose geometry is never changed or clipped, so they may always draw with
3280           a viewport origin of (0, 0).
3281         (-[WebBaseNetscapePluginView restorePortState:]):
3282         - Removed "forUpdate" flag from CoreGraphics port state struct.
3283         - Restore the old OpenGL context saved by -saveAndSetNewPortStateForUpdate:.
3284         (-[WebBaseNetscapePluginView sendEvent:]):
3285         - Updated an assertion to also include OpenGL.  To ensure that attached plug-in window movements
3286           happen atomically with web page redisplays, we assert that the plug-in's window is set only while
3287           the plug-in view is redrawing.
3288         - Same deal as with the assertion; only save/set port state when redrawing the plug-in view.  Plug-ins
3289           that use the new drawing models are only allowed to draw when the web page draws.  I might consider
3290           changing this for windowed OpenGL plug-ins, since they always obscure the page content anyway.
3291         (-[WebBaseNetscapePluginView isNewWindowEqualToOldWindow]):
3292         - Compare new NP_GLContext structs.
3293         (-[WebBaseNetscapePluginView updateAndSetWindow]):
3294         - In OpenGL mode, can only set window when updating plug-in view.
3295         (-[WebBaseNetscapePluginView setWindowIfNecessary]):
3296         - ditto
3297         - Updated logging for OpenGL drawing mode.
3298         (-[WebBaseNetscapePluginView addWindowObservers]):
3299         - No need to observe frame/bounds change notifications for this and all parent views.  See -renewGState
3300           comments below.
3301         (-[WebBaseNetscapePluginView removeWindowObservers]):
3302         - Don't need to remove frame/bounds observers anymore.
3303         (-[WebBaseNetscapePluginView start]):
3304         - Plug-ins are "windowed" by default.  This is not a change from our previous behavior, but this is a
3305           better place to set the default value as it allows the plug-in to override it later.
3306         (-[WebBaseNetscapePluginView stop]):
3307         - Destroy AGL context when the plug-in stops.
3308         (-[WebBaseNetscapePluginView dealloc]):
3309         - Assert that the AGL stuff has been cleaned up.
3310         (-[WebBaseNetscapePluginView drawRect:]):
3311         - If this is a windowless OpenGL plugin, blit its contents back into this view.
3312         (-[WebBaseNetscapePluginView renewGState]):
3313         - This method is called when the view or one of its parents is moved or resized (see comments).
3314         (-[WebBaseNetscapePluginView viewWillMoveToWindow:]):
3315         - Hide the AGL window if the plug-in view is about to be removed from its window.
3316         (-[WebBaseNetscapePluginView viewHasMoved:]):
3317         - Renamed and moved to the "Internal" category.
3318         (-[WebBaseNetscapePluginView invalidateRegion:]):
3319         - Style changes.
3320         - Add support for OpenGL (uses the same region type as CoreGraphics).
3321         (-[WebBaseNetscapePluginView getVariable:value:]):
3322         - Style changes.
3323         - Implemented NPNVsupportsOpenGLBool; returns YES since we now support the OpenGL drawing model.
3324         (-[WebBaseNetscapePluginView setVariable:value:]):
3325         - Implemented NPPVpluginWindowBool, which allows plug-ins to specify whether they should be rendered in
3326           "windowed" or "windowless" mode.  This is an older part of the Netscape Plug-in API that was never
3327           implemented in WebKit.  "Windowed" Quickdraw plug-ins do not actually reside in a separate window, and
3328           can already do many of the same things (such as transparency) that only "windowless" plug-ins can do on
3329           other platforms.  However, we need the "windowed" vs. "windowless" distinction for OpenGL plug-ins so
3330           that they have some way of specifying whether they should be rendered on an accelerated overlay surface,
3331           composited into the browser window.
3332         - Support for setting the drawing model to OpenGL.
3333         (-[WebBaseNetscapePluginView _viewHasMoved]):
3334         - Renamed from -viewHasMoved:, and moved down in the file.
3335         - None of this work is necessary when the plug-in is not in a window; the plug-in's state will be properly
3336           restored when it is moved back into a window.
3337         - Reshape OpenGL surface window here.
3338         (-[WebBaseNetscapePluginView _createAGLContextIfNeeded]):
3339         - Creates the AGL context of the appropriate type (windowed/windowless).
3340         (-[WebBaseNetscapePluginView _createWindowedAGLContext]):
3341         - Creates a windowed AGL context, which is an AGL context attached to a child window.  This is the only way
3342           to get true hardware acceleration.
3343         (-[WebBaseNetscapePluginView _createWindowlessAGLContext]):
3344         - Creates a windowless AGL context, which is an AGL context attached to an offscreen buffer.  This buffer can
3345           then be blitted back into the browser window with a different alpha, or scaled, or whatever.
3346         (-[WebBaseNetscapePluginView _cglContext]):
3347         - Returns the underlying CGL context from the AGL context.  We give the plug-in access to the CGL context because
3348           CGL is the more primitive of the GL drawable APIs and allows for finer control over the context.
3349         (-[WebBaseNetscapePluginView _getAGLOffscreenBuffer:width:height:]):
3350         - Returns the buffer allocated for the offscreen AGL context, if there is one.
3351         (-[WebBaseNetscapePluginView _destroyAGLContext]):
3352         - Destroys the AGL context, as well as the associated offscreen buffer or child window.
3353         (-[WebBaseNetscapePluginView _reshapeAGLWindow]):
3354         - Positions the AGL window over the browser window.
3355         (-[WebBaseNetscapePluginView _hideAGLWindow]):
3356         - Hides the AGL window.
3357         (-[WebBaseNetscapePluginView _aglOffscreenImageForDrawingInRect:]):
3358         - Returns an NSImage representation of the offscreen AGL context's framebuffer.  This is used to draw the offscreen
3359           bits back into the plug-in view.  This is kind of tricky because it has to convert the offscreen buffer in-place
3360           from BGRA to RGBA so that it can be wrapped in an NSBitmapImageRep.  See comments.
3361
3362         * WebKit.xcodeproj/project.pbxproj:
3363         Link OpenGL and AGL.
3364
3365 2006-07-09  Brady Eidson  <beidson@apple.com>
3366
3367         Reviewed by Maciej
3368
3369         The ICONDEBUG flag now chooses either the new icon database or the old one
3370         No longer any need to live side by side to compare results
3371
3372         * Misc/WebIconDatabase.m:
3373         (-[NSMutableDictionary iconURLForURL:]):
3374         (-[NSMutableDictionary retainIconForURL:]):
3375         (-[NSMutableDictionary releaseIconForURL:]):
3376         (-[WebIconDatabase _setHaveNoIconForIconURL:]):
3377         (-[WebIconDatabase _setIconURL:forURL:]):
3378         (-[WebIconDatabase _resetCachedWebPreferences:]):
3379
3380 2006-07-08  Timothy Hatcher  <timothy@apple.com>
3381
3382         Reviewed by Maciej.
3383
3384         Bug 5312: comments aren't available via DOM
3385         http://bugzilla.opendarwin.org/show_bug.cgi?id=5312
3386
3387         Makes the Web Inspector show comment node contents.
3388
3389         * WebInspector/WebInspector.m:
3390         (-[DOMNode _displayName]): return the contents of the comment
3391         * WebInspector/webInspector/inspector.js: check for comment nodes
3392
3393 2006-07-09  Alexey Proskuryakov  <ap@nypop.com>
3394
3395         Reviewed by Darin.
3396
3397         - http://bugzilla.opendarwin.org/show_bug.cgi?id=9572
3398         Add application/xhtml+xml to the Accept header
3399
3400         * WebView/WebFrame.m:
3401         (-[WebFrame _addExtraFieldsToRequest:mainResource:alwaysFromRequest:]): Add an Accept header
3402         to main resource requests.
3403         * English.lproj/StringsNotToBeLocalized.txt: Added new strings.
3404
3405 2006-07-08  Darin Adler  <darin@apple.com>
3406
3407         * DefaultDelegates/WebDefaultContextMenuDelegate.m:
3408         (-[WebDefaultUIDelegate contextMenuItemsForElement:defaultMenuItems:]):
3409         Removed misleading old comment.
3410
3411 === Safari-521.15 ===
3412
3413 2006-07-07  Levi Weintraub  <lweintraub@apple.com>
3414
3415         Reviewed by justin
3416
3417         Finished moving deletion selection expansion across the bridge... say that 3 times fast.
3418
3419         * WebView/WebHTMLView.m:  Pass granularity to WebCore to handle expansion
3420         (-[WebHTMLView _deleteRange:killRing:prepend:smartDeleteOK:deletionAction:granularity:]):
3421         (-[WebHTMLView _deleteSelection]):
3422         (-[WebHTMLView _deleteWithDirection:granularity:killRing:isTypingAction:]):
3423         (-[WebHTMLView deleteToMark:]):
3424
3425 2006-07-07  Brady Eidson  <beidson@apple.com>
3426
3427         Reviewed by John
3428
3429         Changed an ASSERT to a LOG_ERROR for an error that could be handled gracefully, but
3430         whose assertion was reproducibly causing a build bot failure
3431
3432         * Misc/WebIconDatabase.m:
3433         (-[WebIconDatabase _releaseIconForIconURLString:]):
3434
3435 2006-07-06  Levi Weintraub  <lweintraub@apple.com>
3436
3437         Reviewed by justin
3438
3439         Improved table editing
3440
3441         * WebCoreSupport/WebFrameBridge.m: Added method to allow WebCore to trigger
3442         deletion editing delegate
3443         (-[WebFrameBridge shouldDeleteSelectedDOMRange:]):
3444         * WebView/WebHTMLView.m: Moved code that expanded a selection when the delete
3445         key is pressed over to WebCore so we can be more intelligent about how to handle it
3446         (-[WebHTMLView _deleteRange:killRing:prepend:smartDeleteOK:deletionAction:]):
3447         (-[WebHTMLView _deleteWithDirection:granularity:killRing:isTypingAction:]):
3448
3449 2006-07-07  John Sullivan  <sullivan@apple.com>
3450
3451         Reviewed by Tim Hatcher
3452         
3453         - fixed <rdar://problem/4606857> WebKit: WebPreferencesChangedNotification not exported
3454
3455         * WebKit.exp:
3456         added surprisingly missing _WebPreferencesChangedNotification, defined in
3457         WebPreferences.h
3458
3459 2006-07-06  Brady Eidson  <beidson@apple.com>
3460
3461         Reviewed by John.
3462
3463         Small fix to my previous small fix that only lets the ASSERT off the hook if the DB
3464         is closing
3465
3466         * Misc/WebIconDatabase.m:
3467         (-[WebIconDatabase _releaseIconForIconURLString:]):
3468
3469 2006-07-05  Brady Eidson  <beidson@apple.com>
3470
3471         Reviewed by Maciej
3472
3473         Small fix that prevents an assertion from triggering if the DB is being cleaned up 
3474         (ie, the app being shut down)
3475
3476         * Misc/WebIconDatabase.h:
3477         * Misc/WebIconDatabase.m:
3478         (-[NSMutableDictionary init]):
3479         (-[WebIconDatabase _applicationWillTerminate:]):
3480         (-[WebIconDatabase _releaseIconForIconURLString:]):
3481
3482 2006-07-05  Adele Peterson  <adele@apple.com>
3483
3484         Reviewed by Maciej and Hyatt.
3485
3486         WebKit part of initial popup menu implementation.
3487
3488         * WebCoreSupport/WebSystemInterface.m: (InitWebCoreSystemInterface):
3489         Initialize WKPopupMenu.
3490
3491 2006-07-05  Anders Carlsson  <acarlsson@apple.com>
3492
3493         Reviewed by Maciej.
3494
3495         http://bugzilla.opendarwin.org/show_bug.cgi?id=3581
3496         iFrames set to display:none are Missing from frames array
3497         
3498         * WebCoreSupport/WebFrameBridge.m:
3499         (-[WebFrameBridge initSubframeWithOwnerElement:frameName:view:]):
3500         (-[WebFrameBridge createChildFrameNamed:withURL:referrer:ownerElement:allowsScrolling:marginWidth:marginHeight:]):
3501         Modify to pass the owner element instead of the owner renderer.
3502         
3503         * WebView/WebHTMLView.m:
3504         (-[WebHTMLView _topHTMLView]):
3505         Remove assertion, it's not valid anymore.
3506
3507 2006-07-05  Timothy Hatcher  <timothy@apple.com>
3508
3509         Reviewed by Harrison.
3510
3511         <rdar://problem/4608423> HIViewAdapter used but not defined
3512         Adds a new export file to fix the build.
3513
3514         * WebKit.LP64.exp: Added.
3515         * WebKit.xcodeproj/project.pbxproj:
3516
3517 2006-07-04  Timothy Hatcher  <timothy@apple.com>
3518
3519         Reviewed by Maciej.
3520
3521         Bug 9731: [Drosera] crash when trying to access the scope chain
3522         http://bugzilla.opendarwin.org/show_bug.cgi?id=9731
3523
3524         Because of <rdar://problem/4608404> the WebScriptObject, _globalObj, that
3525         WebCoreScriptDebugger holds is unprotected each time the page changes.
3526         This causes Drosera to crash Safari when trying to access the scope chain.
3527         We simply need to detach and re-attach the debugger when the window script
3528         object is cleared until 4608404 is fixed. This change also attaches the
3529         debugger before we call the windowScriptObjectAvailable: delegate method,
3530         so the debugger is ready before anyone might use the window object.
3531
3532         * WebCoreSupport/WebFrameBridge.m:
3533         (-[WebFrameBridge windowObjectCleared]):
3534
3535 2006-07-04  Timothy Hatcher  <timothy@apple.com>
3536
3537         Reviewed by Darin.
3538
3539         Bug 9732: [Drosera] calling removeListener to many times will cause
3540                   WebKit's listener count to underflow/wraparound
3541         http://bugzilla.opendarwin.org/show_bug.cgi?id=9732
3542
3543         Adds a check to make sure the listener was in our listeners set before
3544         decrementing the global listener count. Also checks for nil in addListner
3545         to prevent a possible exception when adding the object to the set.
3546
3547         * DefaultDelegates/WebScriptDebugServer.m:
3548         (-[WebScriptDebugServer addListener:]):