Timeline

View changes from
going back days
by author

Repository changesets Wiki changes

Oct 29, 2002:

11:58 PM Changeset in webkit [2505] by mjs

• 7 edits in trunk/WebKit

WebKit:

First step towards policy API change. Pass request instead of URL
to content policy delegate method.

• WebView.subproj/WebControllerPolicyDelegate.h:
• WebView.subproj/WebDefaultPolicyDelegate.m:
• WebView.subproj/WebMainResourceClient.m: (-[WebMainResourceClient handle:didReceiveResponse:]):

WebBrowser:

First step towards policy API change. Pass request instead of URL
to content policy delegate method.

• BrowserWebController.m: (-[BrowserWebController contentPolicyForResponse:andRequest:inFrame:withContentPolicy:]):

11:55 PM Changeset in webkit [2504] by mjs

• 6 edits in trunk/WebKit

• fixed 3087548 - REGRESSION: two windows open for netflix confirmation

• WebView.subproj/WebControllerPrivate.h:
• WebView.subproj/WebControllerPrivate.m: (-[WebController _findFrameInThisWindowNamed:]): Added this function back, and added check for top-level frame name. (-[WebController _findFrameNamed:]): Use _findFrameInThisWindowNamed: to make sure top-level frame names get checked.

8:50 PM Changeset in webkit [2503] by cblu

• 3 edits
  6 adds
  2 deletes in trunk/WebKit

Created Downloads.subproj to hold the to-be-implemented download decoders.

• Downloads.subproj/WebDownloadDecoder.h: Added.
• Downloads.subproj/WebDownloadHandler.h: Added.
• Downloads.subproj/WebDownloadHandler.m: Added. (-[WebDownloadHandler initWithDataSource:]): (-[WebDownloadHandler dealloc]): (-[WebDownloadHandler errorWithCode:]): (-[WebDownloadHandler receivedResponse:]): (-[WebDownloadHandler receivedData:]): (-[WebDownloadHandler finishedLoading]): (-[WebDownloadHandler cancel]):
• Misc.subproj/WebDownloadHandler.h: Removed.
• Misc.subproj/WebDownloadHandler.m: Removed.
• WebKit.pbproj/project.pbxproj:

Oct 28, 2002:

11:05 PM Changeset in webkit [2502] by darin

• 4 edits in trunk/WebCore

• khtml/khtml_part.cpp: (KHTMLPart::checkCompleted): Update so it handles nil for the view.

10:51 PM Changeset in webkit [2501] by darin

• 15 edits in trunk/WebCore

• fixed 3070484 -- icon rollovers are very slow on Mac OS X website

The problem was simply that the part didn't know it was complete.
To let it know, I hooked up some loader signals, and now it works.

• khtml/khtml_part.cpp: (KHTMLPart::init): Create the KWQKHTMLPart a little earlier so we have a chance to initialize the cache before it's used. (KHTMLPart::slotLoaderRequestStarted): Take this out of ifdefs. (KHTMLPart::slotLoaderRequestDone): Ditto.

• khtml/misc/loader.h: Add a KWQLoader pointer to the loader.
• khtml/misc/loader.cpp: (Loader::Loader): Create a KWQLoader; (Loader::~Loader): Destroy the KWQLoader. (Loader::load): Take out ifdefs around code that emits requestStarted.

• kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::KWQKHTMLPart): Call Cache::init().

• kwq/KWQLoader.h: Add a class KWQLoader, with three signals in it.
• kwq/KWQLoader.mm: (KWQLoader::KWQLoader): Initialize the three signals.

• kwq/KWQSignalStubs.mm: (Loader::requestDone): Emit the signal using KWQLoader. (Loader::requestFailed): Ditto. (Loader::requestStarted): Ditto.

• kwq/KWQSlot.h: Add an overload for slots with loader and cached object parameters so we can handle the new slots.
• kwq/KWQSlot.mm: (KWQSlot::KWQSlot): Added slotLoaderRequestDone and slotLoaderRequestStarted. (KWQSlot::call): Added the new overload to handle those two signals.

• kwq/KWQSignal.h: Add an overload for signals with loader and cached object parameters so we can handle the new signals.
• kwq/KWQSignal.mm: (KWQSignal::call): Ditto.

• kwq/KWQObject.mm: (QObject::connect): Remove the special casing that prevents complaints about slotLoaderRequestDone and slotLoaderRequestStarted, since we support those now.

9:29 PM Changeset in webkit [2500] by darin

• 3 edits in trunk/WebKit

• fixed crash I just introduced

• Plugins.subproj/WebPluginDatabase.m: (-[WebPluginDatabase init]): Remove the release call here. When I added the autorelease, I didn't realize that the caller was doing a release.

5:49 PM Changeset in webkit [2499] by trey

• 4 edits in trunk/WebKit

Fixed 3041616 - Extra page added in back button history

Client-side redirects would always put an extra item in the backforward list.
According to Darin this is a regression all the way back from functionality moving
from WebBrowser to WebKit.

• WebCoreSupport.subproj/WebBridge.h: Added state to know if we're doing an "internal" type of load
• WebCoreSupport.subproj/WebBridge.m: (-[WebBridge reportClientRedirectTo:delay:fireDate:]): Set internalLoad bit if delay==0 (-[WebBridge reportClientRedirectCancelled]): Clear internalLoad (-[WebBridge loadURL:]): Set frame's loadType if doing an internal load (similar to what we do when loading child frames)

5:18 PM Changeset in webkit [2498] by darin

• 14 edits in trunk

WebFoundation:

• fixed storage leak of WebFileDatabaseOp objects

• Database.subproj/WebFileDatabase.m: (-[WebFileDatabase setObject:forKey:]): Release op so it doesn't leak. (-[WebFileDatabase removeObjectForKey:]): Release op so it doesn't leak. (-[WebFileDatabase lazySync:]): Retain op before removing it from the list, since we want to work with it a bit before releasing it.

WebCore:

• fixed storage leak of items in QListBox

• kwq/KWQListBox.h: Added private deleteItems method.
• kwq/KWQListBox.mm: (QListBox::~QListBox): Call deleteItems(). (QListBox::deleteItems): Move the code to delete all the items here from clear(), since clear() is not called on destruction. (QListBox::clear): Call deleteItems().

WebKit:

• fixed storage leak of WebNetscapePluginPackage objects

• Plugins.subproj/WebBasePluginPackage.m: (+[WebBasePluginPackage pluginWithPath:]): Add missing autorelease.

• Plugins.subproj/WebBasePluginPackage.h:
• Plugins.subproj/WebPluginController.h:
• Plugins.subproj/WebPluginController.m:
• Plugins.subproj/WebPluginPackage.h:
• Plugins.subproj/WebPluginPackage.m: Not MyCompanyName, but rather Apple Computer.

4:48 PM Changeset in webkit [2497] by cblu

• 4 edits in trunk/WebCore

Lowercase the text encoding name.

• kwq/KWQCharsets.mm: (KWQCFStringEncodingFromIANACharsetName):

3:16 PM Changeset in webkit [2496] by darin

• 14 edits in trunk/WebKit

WebFoundation:

• fixed bug that caused us to leak one block sometimes when we called WebCallFromCFRunLoop

• Misc.subproj/WebCFRunLoopExtras.m: (WebCallFromCFRunLoop): Initialize refCount to 0.

WebKit:

• fixed bug that caused us to leak all WebResourceHandles used for subresources

• WebCoreSupport.subproj/WebSubresourceClient.m: (-[WebSubresourceClient dealloc]): Remove bogus "currentURL == nil" assert. (+[WebSubresourceClient startLoadingResource:withURL:referrer:forDataSource:]): Use the new loadWithRequest: call instead of making handle and doing loadWithDelegate:. This is where the leak was, because no one ever released the handle, but releasing it right away would be too soon.

• WebView.subproj/WebBaseResourceHandleDelegate.h: Added defersCallbacks field, loadWithRequest: method, and setDefersCallbacks: method. Removed handle method.
• WebView.subproj/WebBaseResourceHandleDelegate.m: (-[WebBaseResourceHandleDelegate loadWithRequest:]): New method. Sets up the handle, set the defersCallbacks state on it, and calls loadWithDelegate:. (-[WebBaseResourceHandleDelegate setDefersCallbacks:]): Stores defersCallbacks state, and also sets it on the handle, if any. (-[WebBaseResourceHandleDelegate handle:willSendRequest:]): Remove code to set up the handle, since we now do that at loadWithRequest: time.

• WebView.subproj/WebDataSourcePrivate.h: Remove mainHandle field.
• WebView.subproj/WebDataSourcePrivate.m: (-[WebDataSourcePrivate dealloc]): Don't release mainHandle. (-[WebDataSource _setPrimaryLoadComplete:]): Ditto. (-[WebDataSource _startLoading]): Use loadWithRequest: on the client, no need to deal with the handle directly at all. (-[WebDataSource _addSubresourceClient:]): Call setDefersCallbacks: on the client. (-[WebDataSource _defersCallbacksChanged]): Ditto.

• WebView.subproj/WebControllerPrivate.m: Remove unneeded include.
• WebView.subproj/WebMainResourceClient.m: Ditto.

2:22 PM Changeset in webkit [2495] by rjw

• 4 edits in trunk/WebCore

Fixed 3086188 - REGRESSION: Justified text is broken

Bogus special case for DirW(hite)S(pace). This
should be correctly detected by CF, but is not.
Revert to explicit checking for ' ' character. This'll
work most of the time. All the more reason to do
our own unicode attribute database.

• kwq/KWQChar.mm: (QChar::direction):

1:59 PM Changeset in webkit [2494] by cblu

• 12 edits in trunk

WebCore:

Fixed: 3056726 - View Source window always displays the source in current system encoding
Fixed: 3019352 - Text encoding is not handled when viewing plain text

• kwq/WebCoreBridge.h:
• kwq/WebCoreBridge.mm: (+[WebCoreBridge stringWithData:textEncoding:]): added, creates a string with data using the WebCore's decoding logic. (+[WebCoreBridge stringWithData:textEncodingName:]): added, calls stringWithData:textEncoding: after getting the encoding for a text encoding name

WebKit:

Fixed: 3056726 - View Source window always displays the source in current system encoding
Fixed: 3019352 - Text encoding is not handled when viewing plain text

• WebView.subproj/WebDataSource.h:
• WebView.subproj/WebDataSource.m: (-[WebDataSource stringWithData:]): added, creates a string using the specified encoding.
• WebView.subproj/WebHTMLRepresentation.m: (-[WebHTMLRepresentation documentSource]): uses the bridge specified encoding when creating the string
• WebView.subproj/WebTextView.h:
• WebView.subproj/WebTextView.m: (-[WebTextView dataSourceUpdated:]): use stringWithData: (-[WebTextView supportsTextEncoding]): Yup

1:28 PM Changeset in webkit [2493] by gramps

• 5 edits in trunk/WebCore

• khtml/html/htmlparser.cpp:

(KHTMLParser::insertNode):

• khtml/html/htmltokenizer.cpp:

(HTMLTokenizer::write):

Backed our Dave's whitespace hack because it was adding newlines, i.e.
vertical margins, to web pages all over.

12:51 PM Changeset in webkit [2492] by rjw

• 3 edits in trunk/WebKit

Fixed crasher.

• WebCoreSupport.subproj/WebTextRenderer.m: (-[WebTextRenderer _floatWidthForCharacters:stringLength:fromCharacterPosition:numberOfCharacters:withPadding:applyRounding:attemptFontSubstitution:widths:fonts:glyphs:numGlyphs:]):

12:40 PM Changeset in webkit [2491] by sullivan

• 4 edits in trunk/WebKit

WebKit:

• Bookmarks.subproj/WebBookmark.h:
• Bookmarks.subproj/WebBookmark.m: (-[WebBookmark contentMatches:]): new method, returns YES if two bookmarks are the same, ignoring group and parent.

WebBrowser:

• fixed 3083274 -- Add preference for separating/consolidating contents of Bookmarks Toolbar and Menu
• fixed 3083277 -- Add preferences for showing Address Book and Rendezvous items in Bookmarks menu

Also made Bookmarks menu update only when it is about to be pulled
down (like History menu), rather than every time bookmarks change.

• Defaults.plist,
• PreferenceKeys.h: Add new preference keys with default values: BookmarksMenuAndToolbarAreSame, BookmarksMenuIncludesAddressBook, and BookmarksMenuIncludesRendezvous. Also #defined BrowserPreferenceValueChangedNotification, which is sent when certain preference values change.

• Preferences.subproj/GeneralPreferences.h:
• Preferences.subproj/GeneralPreferences.m: (-[GeneralPreferences sendChangeNotificationForPreferenceKey:]): new method, sends BrowserPreferenceValueChangedNotification with a particular preference key's name in the userInfo (-[GeneralPreferences toggleBookmarksMenuAndToolbarAreSame:]), (-[GeneralPreferences toggleBookmarksMenuIncludesRendezvous:]), (-[GeneralPreferences toggleBookmarksMenuIncludesAddressBook:]): Set the appropriate preference, and call sendChangeNotificationForPreferenceKey (-[GeneralPreferences initializeFromDefaults]): Set the initial state for the new checkboxes.

• Preferences.subproj/English.lproj/GeneralPreferences.nib: Added checkboxes for the three new preferences. With Ron Brannan's help, the text is decent, but the General Preferences panel is now much taller than the others; we'll probably need a reorganization soon.

• BookmarkSource.h: added bookmarkSourceImage, setBookmarkSourceMenu:, and bookmarkSourceMenuTitle to the BookmarkSource protocol.

• GlobalHistory.h: removed setHistoryMenu: and historyMenu
• GlobalHistory.m: (-[GlobalHistory dealloc]): change setHistoryMenu to setBookmarkSourceMenu (-[GlobalHistory bookmarkSourceImage]): new method, returns little clock image (-[GlobalHistory setBookmarkSourceMenu:]): new name for setHistoryMenu (-[GlobalHistory bookmarkSourceMenuTitle]): new method, returns nil here since history uses an existing top-level menu

• AppController.m: (-[AppController applicationDidFinishLaunching:]): use -[GlobalHistory setBookmarkSourceMenu:] instead of obsolete -[GlobalHistory setHistoryMenu:]

• ABHomePagesController.h:
• ABHomePagesController.m: (-[ABHomePagesController dealloc]): call setBookmarkSourceMenu:nil (-[ABHomePagesController refreshContents]): set _menuGood to NO (-[ABHomePagesController updateMenu]): new method, make menu contents match address book contents (if !_menuGood) (-[ABHomePagesController setBookmarkSourceMenu:]): new method, set up menuUpdater (-[ABHomePagesController bookmarkSourceMenuTitle]): new method, return "Address Book" (-[ABHomePagesController bookmarkSourceImage]): new method, return little address book image

• RendezvousController.h:
• RendezvousController.m: (-[RendezvousController dealloc]): call setBookmarkSourceMenu:nil (-[RendezvousController sendContentsChangedNotification]): set _menuGood to NO (-[RendezvousController updateMenu]): new method, make menu contents match Rendezvous list (if !_menuGood) (-[RendezvousController setBookmarkSourceMenu:]): new method, set up menuUpdater (-[RendezvousController bookmarkSourceMenuTitle]): new method, return "Rendezvous" (-[RendezvousController bookmarkSourceImage]): new method, return little Rendezvous logo image

• BookmarksController.h: now implements MenuUpdaterDelegate protocol and updates the Bookmarks menu only when it's about to be shown rather than when any bookmarks change.
• BookmarksController.m: (-[BookmarksController imageForBookmark:]): get bookmark source images using BookmarkSource protocol rather than hardwiring here (-[BookmarksController _removeAllBookmarksFromMenu]): renamed from _removeBookmarksFromMenu for some nitpicky reason (-[BookmarksController bookmarkSource:setShouldIncludeSubmenu:]): new method, creates submenu for bookmark source if appropriate (-[BookmarksController addSpecialBookmarkSourcesToMenu]): new method, checks preferences and calls bookmarkSource:setShouldIncludeSubmenu: for Rendezvous and AddressBook (-[BookmarksController updateMenu]): renamed from _updateBookmarksInMenu (now it's the MenuUpdaterDelegate method); do nothing if _menuGood, otherwise remove all bookmarks, then add them back in. (-[BookmarksController _receivedBookmarksChangedNotification:]): set _menuGood to NO rather than updating bookmarks immediately (-[BookmarksController receivedBrowserPreferenceValueChangedNotification:]): new method, do the appropriate updating based on which preference changed (-[BookmarksController addBookmarksMenuUpdater]): new method, broken out only so it could be called after a delay, since awakeFromNib is slightly too early (-[BookmarksController awakeFromNib]): set up menu updater (after delay); observe BrowserPreferenceValueChangedNotification (-[BookmarksController dealloc]): remove menu updater (-[BookmarksController bookmarkSourceForProxyIdentifier:]): new method, moved here from BookmarksViewController; converts a string identifier into an object implementing <BookmarkSource> (-[BookmarksController addUniqueBookmarksFrom:to:]): new method, used to consolidate toolbar and menu contents when the pref is set that way. (-[BookmarksController updateBookmarkSources]): renamed from _blessOrCreateSpecialFolders, now it is also called in response to a change to BookmarksMenuAndToolbarAreSamePreferenceKey; handle making menu initially be a copy of toolbar, or consolidating the two, depending on pref value.

• BookmarksViewController.m: (-[BookmarksViewController bookmarkSourceContentsChanged:]), (-[BookmarksViewController refreshContents]), (-[BookmarksViewController bookmarksFromContentItems:]), (-[BookmarksViewController bookmarkFromContentItem:]), (-[BookmarksViewController canDeleteFromSelectedBookmarkSource]), (-[BookmarksViewController child:ofContentItem:]), (-[BookmarksViewController isContentItemExpandable:]), (-[BookmarksViewController numberOfChildrenOfContentItem:]), (-[BookmarksViewController titleStringForContentItem:]), (-[BookmarksViewController addressStringForContentItem:]), (-[BookmarksViewController imageForContentItem:]), (-[BookmarksViewController deleteContentItems:]): Updated to call bookmarkSourceForProxyIdentifier on [BookmarksController sharedController] rather than self.

• FavoriteButton.m: (-[FavoriteButton determineDragOperation:]): use -[WebBookmark contentMatches:] instead of (deleted) [FavoriteButton bookmarkMatches:]

• English.lproj/Localizable.strings,
• English.lproj/StringsNotToBeLocalized.txt: kept localizations up to date

10:49 AM Changeset in webkit [2490] by trey

• 7 edits in trunk/WebKit

WebKit:

Finished 2998200 - access to most-recent in submenu of back & forward buttons

• WebView.subproj/WebDataSourcePrivate.h: The BF items we are going to and from were moved here from WebFrame because of timing problems among multiple data sources when using the BF buttons during a load. In addition, the datasource has a list of relevant BF items for which it keeps their titles up to date.
• WebView.subproj/WebDataSourcePrivate.m: (-[WebDataSourcePrivate dealloc]): Free new state. (-[WebDataSource _setTitle:]): Update titles of BF items. (-[WebDataSource _provisionalBackForwardItem]): (-[WebDataSource _setProvisionalBackForwardItem:]): (-[WebDataSource _previousBackForwardItem]): (-[WebDataSource _setPreviousBackForwardItem:]): (-[WebDataSource _addBackForwardItem:]): Maintain new state. (-[WebDataSource _commitIfReady]): Free up previous and provisional BF items after commit.
• WebCoreSupport.subproj/WebBridge.m: (-[WebBridge loadURL:]): Add the new BF item to the datasource. (-[WebBridge saveDocumentState:]): Use the right BF item to save the doc state.
• WebView.subproj/WebFramePrivate.h: Previous and provisional BF are moved to datasource.
• WebView.subproj/WebFramePrivate.m: (-[WebFrame _transitionToCommitted]): Use provisional BF from datasource to set position in BF list. Add new BF item to datasource. (-[WebFrame _isLoadComplete]): (-[WebFrame _checkLoadComplete]): Comments, whitespace. (-[WebFrame _goToItem:withFrameLoadType:]): Set item we will use to saveDocState at commit time.

WebBrowser:

Finished 2998200 - access to most-recent in submenu of back & forward buttons

• BrowserDocument.m: (-[BrowserDocument _addItem:toMenu:]): Center-truncate menu items to reasonable width
• Defaults.plist: Set Back-forward max size to 200.

10:48 AM Changeset in webkit [2489] by darin

• 4 edits in trunk/WebCore

Tests:

• WebFoundation-Misc/ifnsurlextensions-test.chk: Update WebNSURLExtras tests for recently-implemented ftp URL canonicalization.

WebFoundation:

• fixed 3015800 -- bad URL error - unescaped # in URL

Actually this has to be fixed both here and in KURL.

• Misc.subproj/WebNSURLExtras.m: (StringByAddingPercentEscapes): Escape "#" characters too. (URLStringByAddingPercentEscapes): Change the code so that it specifically makes an exception for the first "#" character in the string. I guess that it should really be the first "#" in the path, but that would be a lot harder. (+[NSURL _web_URLWithString:relativeToURL:]): Cite the new CFURL bug I filed.

WebCore:

• fixed 3015800 -- bad URL error - unescaped # in URL

Actually this has to be fixed both here and in NSURLExtras.

• kwq/KWQKURL.mm: Set the "BadChar" flag for "#" so we escape any "#" characters we see after the first one.

Oct 27, 2002:

11:54 PM Changeset in webkit [2488] by darin

• 11 edits in trunk

WebCore:

• fixed 3058845 -- <input type=file> uses hard-coded width of 200 pixels for filename

• khtml/rendering/render_form.cpp: (RenderLineEdit::calcMinMaxWidth): Use a default of 20 "x-widths" wide, to match Mozilla, rather than the 17 that is in KHTML by default. (RenderFileButton::calcMinMaxWidth): Pass in a character width here, using the same 20-character rule as above.
• kwq/KWQFileButton.h: Replace sizeHint() with sizeForCharacterWidth(int).
• kwq/KWQFileButton.mm: (KWQFileButton::sizeForCharacterWidth): Pass character count to the WebCoreFileButton. (KWQFileButton::baselinePosition): Fix baseline computation. It was computing some huge negative number before and was obviously broken.
• kwq/KWQLineEdit.mm: (QLineEdit::sizeForCharacterWidth): Add an assertion.
• kwq/WebCoreViewFactory.h: Replace bestVisualFrameSize with bestVisualFrameSizeForCharacterCount:.

WebKit:

• WebKit part of filename width fix

• WebCoreSupport.subproj/WebFileButton.m: (-[WebFileButton drawRect:]): Compute left position by starting at the right side of the button frame rather than by starting at the right side of the overall bounds. Removes dependency on fixed-width filename area. (-[WebFileButton updateLabel]): Added. Computes ellipsized filename. (-[WebFileButton setFilename:]): Use updateLabel instead of code in line here. (-[WebFileButton setFrameSize:]): Call updateLabel. (-[WebFileButton bestVisualFrameSizeForCharacterCount:]): Compute a width based on the count passed in rather than a hardcoded 200-pixel constant.

10:52 PM Changeset in webkit [2487] by darin

• 5 edits in trunk/WebKit

• fixed 3037369 -- Status text not cleared after start of drag from page

• WebView.subproj/WebHTMLView.m: (-[WebHTMLView dragImage:at:offset:event:pasteboard:source:slideBack:]): Find the highest-level enclosing WebHTMLView, and call _updateMouseoverWithEvent: with the mouseUp event after the drag is done. (-[WebHTMLView mouseMovedNotification:]): Just call _updateMouseoverWithEvent:, the code from here was moved into there.

• WebView.subproj/WebHTMLViewPrivate.h: Declare _updateMouseoverWithEvent.
• WebView.subproj/WebHTMLViewPrivate.m: (-[WebHTMLView _updateMouseoverWithEvent:]): Moved all the code from mouseMovedNotification: in here so it could be used twice.

6:27 PM Changeset in webkit [2486] by gramps

• 8 edits in trunk

WebCore:

• khtml/ecma/kjs_navigator.cpp:

(Navigator::getValueProperty):

Tweaked AppName, Product, and Vendor logic.

WebKit:

• WebView.subproj/WebController.m:

(-[WebController userAgentForURL:]):

Changed default user agent to Mozilla 1.1.

WebBrowser:

• Debug/DebugUtilities.m:

(-[DebugUtilities userAgentSubmenuItem]):

Updated all user agents and added new experimental items.

9:08 AM Changeset in webkit [2485] by darin

• 9 edits in trunk

WebCore:

• kwq/KWQLoader.mm: (KWQServeRequest): Add a missing "delete job" for one particular failure case.

WebKit:

• Bookmarks.subproj/WebBookmarkProxy.m: (-[WebBookmarkProxy dealloc]): Added, so we don't leak the title.

• Bookmarks.subproj/WebBookmarkGroup.m: (-[WebBookmarkGroup removeBookmark:]):
• Bookmarks.subproj/WebBookmarkList.m: (-[WebBookmarkList removeChild:]): Added retain/release pairs so we don't use an object after it's deallocated. In practice this doesn't happen the way these are used by Alex since they are in an array, but they have windows of vulnerability depending on exactly how they are used.

WebBrowser:

• BookmarksController.m: (-[BookmarksController _blessOrCreateSpecialFolders]): Fix indenting.

Oct 26, 2002:

11:45 PM Changeset in webkit [2484] by darin

• 8 edits in trunk/WebCore

• fixed a bug in the WebCore cache that caused it to not cache very much after a while, and also caused a crash I saw twice on my machine

Adjust total size of items in the LRU list, when the size of one of the
items that's in the list is changed.

• khtml/misc/loader.h: Added a size parameter (default 0) to the CachedObject constructor. Added a protected CachedObject::setSize() and made m_size private. Added Cache::adjustSize() and Cache::checkLRUAndUncacheableListIntegrity().
• khtml/misc/loader.cpp: (CachedObject::setSize): Added. This is now the only legal way to change the size of a cached object, and it calls Cache::adjustSize. (CachedCSSStyleSheet::CachedCSSStyleSheet): Pass the length into the base class's constructor instead of setting m_size. (CachedCSSStyleSheet::data): Call setSize() instead of setting m_size. Also get the size back with size() rather than m_size. (CachedScript::CachedScript): Pass the length into the base class's constructor instead of setting m_size. (CachedScript::data): Call setSize() instead of setting m_size. Also get the size back with size() rather than m_size. (CachedImage::CachedImage): Don't set m_size to 0, since the base class's constructor does that already. (CachedImage::clear): Use setSize() to set the size to 0 instead of setting m_size directly. (CachedImage::data): Use setSize() instead of setting m_size directly. (Cache::checkLRUAndUncacheableListIntegrity): Useful debugging function. Left in but ifdef'd out. (Cache::adjustSize): Added. Changes the "total LRU list size" by the passed-in delta, but only if the passed-in object is in the LRU list.

• kwq/KWQLineEdit.mm:
• kwq/KWQTextField.h:
• kwq/KWQTextField.mm: Renamed the Objective C class KWQNSTextField to KWQTextField.

4:55 PM Changeset in webkit [2483] by darin

• 8 edits in trunk/JavaScriptCore

• changed to use #if APPLE_CHANGES and #if !APPLE_CHANGES consistently

We no longer do #ifdef APPLE_CHANGES or #ifndef APPLE_CHANGES.

• kjs/collector.cpp:
• kjs/collector.h:
• kjs/grammar.cpp:
• kjs/internal.cpp:
• kjs/ustring.h:

4:49 PM Changeset in webkit [2482] by darin

• 3 edits
  22 deletes in trunk/WebCore

Tools:

• Scripts/check-forwarding-headers: Added. Finds unused forwarding headers in WebCore.

WebCore:

• removed forwarding headers that are not actually needed

• ForwardingHeaders/kcmdlineargs.h: Removed.
• ForwardingHeaders/kinstance.h: Removed.
• ForwardingHeaders/kio/kprinter.h: Removed.
• ForwardingHeaders/kio/ktrader.h: Removed.
• ForwardingHeaders/kio/scheduler.h: Removed.
• ForwardingHeaders/kparts/factory.h: Removed.
• ForwardingHeaders/kparts/historyprovider.h: Removed.
• ForwardingHeaders/kstddirs.h: Removed.
• ForwardingHeaders/qarray.h: Removed.
• ForwardingHeaders/qbrush.h: Removed.
• ForwardingHeaders/qbutton.h: Removed.
• ForwardingHeaders/qcollection.h: Removed.
• ForwardingHeaders/qcombobox.h: Removed.
• ForwardingHeaders/qframe.h: Removed.
• ForwardingHeaders/qglobal.h: Removed.
• ForwardingHeaders/qiodevice.h: Removed.
• ForwardingHeaders/qlineedit.h: Removed.
• ForwardingHeaders/qnamespace.h: Removed.
• ForwardingHeaders/qpen.h: Removed.
• ForwardingHeaders/qprinter.h: Removed.
• ForwardingHeaders/qsize.h: Removed.
• ForwardingHeaders/qstack.h: Removed.

4:41 PM Changeset in webkit [2481] by darin

• 7 edits in trunk/WebCore

• fixed a couple of files accidentally checked in with KWQQ names

• ForwardingHeaders/qasyncio.h: Change names.
• ForwardingHeaders/qstylesheet.h: Ditto.
• WebCore.pbproj/project.pbxproj: Ditto.
• kwq/KWQMovie.h: Ditto.

4:21 PM Changeset in webkit [2480]

• 1 copy
  89 deletes in branches/unlabeled-1.6.56

This commit was manufactured by cvs2svn to create branch
'unlabeled-1.6.56'.

4:21 PM Changeset in webkit [2479] by darin

• 49 edits in trunk/WebCore

• changed to use #if APPLE_CHANGES and #if !APPLE_CHANGES consistently

We no longer do #ifdef APPLE_CHANGES or #ifndef APPLE_CHANGES.

• lots of files

4:08 PM Changeset in webkit [2478] by darin

• 246 edits
  107 adds
  19 deletes in trunk/WebCore

• changed naming scheme in KWQ for files like qstring.h Now use names like KWQString.h and make a forwarding header called qstring.h.

• tons of files

12:27 AM Changeset in webkit [2477] by darin

• 24 edits
  1 add
  2 deletes in trunk

WebCore:

• fixed 3068404 -- eliminate all user-visible messages from WebCore

• khtml/ecma/kjs_window.cpp: (WindowFunc::tryCall): Use runJavaScriptPrompt function instead of the more generic QInputDialog.

• khtml/html/html_formimpl.cpp: (HTMLInputElementImpl::altText): Get submit alt text for input element from the view factory, rather than using i18n, since we don't support that.
• khtml/html/htmlparser.cpp: (KHTMLParser::handleIsindex): Ditto, for the intro that says "This is a searchable index".
• khtml/rendering/render_form.cpp: (RenderSubmitButton::defaultLabel): Ditto, for "Submit". (RenderResetButton::defaultLabel): Ditto, for "Reset". (RenderFileButton::slotClicked): Ifdef'd out this unused code that had i18n text in it.

• khtml/xml/xml_tokenizer.cpp: (XMLHandler::errorString): Put in an ifdef and a FIXME to avoid i18n. (XMLHandler::fatalError): Ditto. (XMLTokenizer::finish): Ditto.

• khtml/misc/decoder.cpp: (Decoder::decode): Ifdef'd the check for the first language being "ja", since that's not how we're going to trigger Japanese auto-detect.

• khtml/rendering/render_applet.cpp: (RenderEmptyApplet::RenderEmptyApplet): Turned off the code that does the empty applet label. I don't think it worked, and it used i18n. We'll have to tackle this eventually.

• kwq/KWQKHTMLPartImpl.h: Added runJavaScriptPrompt.
• kwq/KWQKHTMLPartImpl.mm: (KWQKHTMLPartImpl::runJavaScriptPrompt): Moved the code here that used to be in KWQInputDialog.

• kwq/kdecore/klocale.h: Remove i18n() and KLocale::languageList(), added the functions that are listed below.
• kwq/KWQKLocale.mm: (inputElementAltText): Added specific function for getting one of the very few translated strings that we need in WebCore. (resetButtonDefaultLabel): Ditto. (searchableIndexIntroduction): Ditto. (submitButtonDefaultLabel): Ditto. (KLocale::language): Use the view factory to ask for the language code, since there's no public API to get it that WebCore can use.

• kwq/WebCoreViewFactory.h: Add new API for above.

• ForwardingHeaders/qinputdialog.h: Added empty file.

• kwq/qt/qinputdialog.h: Removed.
• kwq/KWQInputDialog.mm: Removed.
• WebCore.pbproj/project.pbxproj: Removed QInputDialog files.

WebKit:

• WebCoreSupport.subproj/WebViewFactory.m: (-[WebViewFactory inputElementAltText]): Added. (-[WebViewFactory resetButtonDefaultLabel]): Added. (-[WebViewFactory searchableIndexIntroduction]): Added. (-[WebViewFactory submitButtonDefaultLabel]): Added. (-[WebViewFactory defaultLanguageCode]): Added.

• English.lproj/Localizable.strings: Update.
• English.lproj/StringsNotToBeLocalized.txt: Update.

Oct 25, 2002:

11:43 PM Changeset in webkit [2476] by darin

• 18 edits
  1 add
  2 deletes in trunk

WebCore:

Remove some user-visible strings. We can't have any strings that need
localization in WebCore, given our translation strategy.

• khtml/html/html_formimpl.cpp: (HTMLInputElementImpl::encoding): Put APPLE_CHANGES around some i18n dialog code.
• khtml/khtml_part.cpp: (KHTMLPart::submitForm): Ditto.
• khtml/rendering/render_frames.cpp: (RenderPartObject::slotPartLoadingErrorNotify): Ditto.
• khtml/ecma/kjs_window.cpp: (Window::get): Ditto. (Window::scheduleClose): Ditto. (WindowFunc::tryCall): Put APPLE_CHANGES in to use JavaScript-specific methods in the part of calling the generic ones from KMessageBox.

• ForwardingHeaders/kmessagebox.h: Added (empty file).

• kwq/KWQKHTMLPartImpl.h: Added new runJavaScript functions.
• kwq/KWQKHTMLPartImpl.mm: (KWQKHTMLPartImpl::runJavaScriptAlert): Added. (KWQKHTMLPartImpl::runJavaScriptConfirm): Added.

• kwq/WebCoreViewFactory.h: Added runJavaScriptAlertPanelWithMessage and runJavaScriptConfirmPanelWithMessage.

• kwq/kdeui/kmessagebox.h: Removed.
• kwq/KWQKMessageBox.mm: Removed.
• WebCore.pbproj/project.pbxproj: Removed the KMessageBox files.

WebKit:

• WebCoreSupport.subproj/WebViewFactory.m: (-[WebViewFactory runJavaScriptAlertPanelWithMessage:]): Added. (-[WebViewFactory runJavaScriptConfirmPanelWithMessage:]): Added.

• English.lproj/Localizable.strings: Update.
• English.lproj/StringsNotToBeLocalized.txt: Update.

10:49 PM Changeset in webkit [2475] by darin

• 4 edits in trunk/JavaScriptCore

• fixed 3038011 -- drop-down menu hierarchy broken at yahoo new acct page

• kjs/array_object.cpp: (ArrayProtoFuncImp::call): Fix bug calling concat on an empty array. The old code tried to optimize in a way that would prevent appending any arrays until at least one element was in the destination array. So if you were concatenating a non-empty array into an empty array, you got an empty array.

7:14 PM Changeset in webkit [2474] by rjw

• 4 edits in trunk/WebCore

Removed debug logging.

• khtml/rendering/font.cpp: (Font::drawText):

7:11 PM Changeset in webkit [2473] by rjw

• 12 edits in trunk

Implemented support for bidi text layout.

• WebCoreSupport.subproj/WebTextRenderer.m: (_drawGlyphs): (-[WebTextRenderer drawCharacters:stringLength:fromCharacterPosition:toCharacterPosition:atPoint:withPadding:withTextColor:backgroundColor:rightToLeft:]):

Implemented support for bidi text layout. WebCore
temporarily depends on SPI to get unicode character
attribute from CF. This will change shortly.

• khtml/rendering/font.cpp: (Font::drawText):
• kwq/KWQChar.mm: (QChar::direction):
• kwq/KWQPainter.mm: (QPainter::drawText):
• kwq/WebCoreTextRenderer.h:
• kwq/qt/qpainter.h:

Updated to reflect change in WebKit SPI.

• EtchedStringView.m: (-[EtchedStringView drawRect:]):

5:55 PM Changeset in webkit [2472] by cblu

• 11 edits in trunk/WebKit

Made frameNamed private in WebController and renamed it to _findFrameNamed. Got rid of _frameInThisWindowNamed in WebController because it isn't needed anymore.

• WebView.subproj/WebController.h:
• WebView.subproj/WebController.m: (-[WebController _goToItem:withFrameLoadType:]): call _findFrameNamed
• WebView.subproj/WebControllerPrivate.h:
• WebView.subproj/WebControllerPrivate.m: (-[WebController _findFrameNamed:]): renamed, made private, got rid of call to _frameInThisWindowNamed as it is not needed
• WebView.subproj/WebFrame.m: (-[WebFrame findFrameNamed:]): call [WebController _findFrameNamed:]

5:27 PM Changeset in webkit [2471] by cblu

• 8 edits in trunk/WebKit

• Fix case where we would name a new window _blank
• If a data source can't be created for a plug-in request, don't create the frame or open a window.
• cleaned up plug-in stream notifications

• Plugins.subproj/WebBaseNetscapePluginView.h:
• Plugins.subproj/WebBaseNetscapePluginView.m: (-[WebBaseNetscapePluginView initWithFrame:]): create streamNotifications dict (-[WebBaseNetscapePluginView dealloc]): release streamNotifications dict (-[WebBaseNetscapePluginView frameStateChanged:]): cleaned up (-[WebBaseNetscapePluginView loadRequest:inTarget:withNotifyData:]): create data source before frame
• Plugins.subproj/WebPluginController.m: (-[WebPluginController showURL:inFrame:]): create data source before frame
• WebView.subproj/WebControllerPrivate.m: (-[WebController _setTopLevelFrameName:]): don't allow _blank frames
• WebView.subproj/WebFrame.m: (-[WebFrame findOrCreateFramedNamed:]): no need to check for _blank

5:06 PM Changeset in webkit [2470] by darin

• 3 edits in trunk/WebKit

WebKit:

Tighten up the code that observes mouse moved events.

• WebView.subproj/WebHTMLView.m: (-[WebHTMLView initWithFrame:]): Don't set up observers here. (-[WebHTMLView addMouseMovedObserver]): Added some assertions. (-[WebHTMLView viewWillMoveToWindow:]): Stop observing the "become main" and "resign main" notifications here. (-[WebHTMLView viewDidMoveToWindow]): Start observing them here. (-[WebHTMLView windowDidBecomeMain:]): Assert that the notification is for this window, instead of checking with if. (-[WebHTMLView windowDidResignMain:]): Assert that the notification is for this window.

WebBrowser:

• fixed 3078924 -- Rollovers fail on wired.com after selecting a menu

• BrowserApplication.m: (-[BrowserApplication sendEvent:]): Put in a workaround that sets the mouse focus to the key window.

• BrowserWindowController.m: (-[BrowserWindowController windowDidLoad]): Remove unneeded pageTitleChanged. (-[BrowserWindowController frameLoadWithinPage:]): Call updateBackAndForwardButtons instead of doing the "two calls to setActivity:" dance.

4:41 PM Changeset in webkit [2469] by cblu

• 20 edits in trunk

WebCore:

Cleaned up the frame searching shenanigans. Things are much cleaner and clearer now.

• kwq/KWQKHTMLPartBrowserExtension.mm: (KHTMLPartBrowserExtension::createNewWindow): call findFrameNamed
• kwq/KWQKHTMLPartImpl.mm: (KWQKHTMLPartImpl::bridgeForFrameName): call findOrCreateFramedNamed
• kwq/WebCoreBridge.h:

WebKit:

Cleaned up the frame searching shenanigans. Things are much cleaner and clearer now.

• Plugins.subproj/WebBaseNetscapePluginView.m: (-[WebBaseNetscapePluginView loadRequest:inTarget:withNotifyData:]): call findOrCreateFramedNamed
• Plugins.subproj/WebPluginController.m: (-[WebPluginController showURL:inFrame:]): call findOrCreateFramedNamed
• WebCoreSupport.subproj/WebBridge.m: (-[WebBridge childFrames]): moved (-[WebBridge mainFrame]): moved (-[WebBridge findOrCreateFramedNamed:]): call findOrCreateFramedNamed on the frame (-[WebBridge pluginViewWithPackage:attributes:baseURL:]): call _pluginController
• WebView.subproj/WebControllerPrivate.m: (-[WebController _frameInThisWindowNamed:]): call _descendantFrameNamed
• WebView.subproj/WebFrame.h:
• WebView.subproj/WebFrame.m: (-[WebFrame findFrameNamed:]): was frameNamed. First checks special-case frame names, descendant frames, then the whole controller, then other controllers. (-[WebFrame findOrCreateFramedNamed:]): calls findFrameNamed, opens new window if necessary
• WebView.subproj/WebFramePrivate.h:
• WebView.subproj/WebFramePrivate.m: (-[WebFrame _descendantFrameNamed:]): searches children, children's children etc. (-[WebFrame _pluginController]): this method needed an "_"
• WebView.subproj/WebHTMLView.m: (-[WebHTMLView addSubview:]): call _pluginController
• WebView.subproj/WebHTMLViewPrivate.m: (-[WebHTMLView _elementAtPoint:]): no need to special case for "_blank" since findFrameNamed will return nil for that name

3:51 PM Changeset in webkit [2468] by trey

• 14 edits
  2 deletes in trunk/WebKit

WebKit:

2919039 - Implement limit on back/forward history list
2998200 - access to most-recent in submenu of back & forward buttons
3022566 - need WebKit API for accessing and visiting back/forward list items
3084051 - forward button incorrectly enabled after going forward all the way

Remaining issues at this point are getting the title of the pages
into the menu items (instead of the URL), and saving document
state properly when you jump around in the list. Solutions are
known and coming soon.

• History.subproj/WebBackForwardList.h: Added doc comments to API. Made ivars private. Replaced index- based API with item-based API. Added a sizelimit to the overall list. Tweaked API to get back and forward lists.
• History.subproj/WebBackForwardList.m: Straightforward impl of the above, with the note that I dumped WebHistoryList from the API and impl in favor of an NSArray, which is efficient for this sort of use (queue, stack). (-[WebBackForwardList init]): (-[WebBackForwardList dealloc]): (-[WebBackForwardList goBack]): (-[WebBackForwardList goForward]): (-[WebBackForwardList goToEntry:]): (-[WebBackForwardList backEntry]): (-[WebBackForwardList forwardEntry]): (-[WebBackForwardList maximumSize]): (-[WebBackForwardList setMaximumSize:]): (-[WebBackForwardList description]):
• History.subproj/WebHistoryList.h: Removed. Nuked from API, per above.
• History.subproj/WebHistoryList.m: Removed.
• Misc.subproj/WebKit.h: WebHistoryList gone from API.
• WebCoreSupport.subproj/WebBridge.m: (-[WebBridge loadURL:]): Use if/else instead of buried "return;", so next person doesn;t make the same mis-read I did. (-[WebBridge saveDocumentState:]): Site of future work.
• WebKit.pbproj/project.pbxproj: Nuked WebHistoryList.
• WebView.subproj/WebController.h:
• WebView.subproj/WebController.m: (-[WebController goBackOrForwardToItem:]): New method to support random access nav in backforward list.
• WebView.subproj/WebDataSourcePrivate.m: (-[WebDataSource _setTitle:]): Site of future work (fixing title in items)
• WebView.subproj/WebFramePrivate.h: WebFrameLoadTypeIndexedBack/WebFrameLoadTypeIndexedForward are collapsed into WebFrameLoadTypeIndexedBackForward. Vestigial WebFrameLoadTypeIntermediateBack is nuked.
• WebView.subproj/WebFramePrivate.m: (-[WebFramePrivate setBackForwardItem:]): (-[WebFrame _setBackForwardItem:]): (-[WebFrame _backForwardItem]): Must remember the BF item we're going to so we can update the BF list on commit. (-[WebFrame _transitionToCommitted]): Update the BF list on commit. (-[WebFrame _isLoadComplete]): (-[WebFrame _goToItem:withFrameLoadType:]): Update BF list on within-page nav. Remember BF item we are going to for later update on commit.

WebBrowser:

2919039 - Implement limit on back/forward history list
2998200 - access to most-recent in submenu of back & forward buttons
3084051 - forward button incorrectly enabled after going forward all the way

Remaining issues at this point are getting the title of the pages
into the menu items (instead of the URL), and saving document
state properly when you jump around in the list. Solutions are
known and coming soon.

• BrowserDocument.h: Add BrowserDocumentClientNavigatedWithinPageNotification.
• BrowserDocument.m: (-[BrowserDocument init]): Set up BackForwardListSizeLimit preference. (-[BrowserDocument canGoBack]): (-[BrowserDocument canGoForward]): (-[BrowserDocument goBack:]): (-[BrowserDocument goForward:]): Nits adopting new API. (-[BrowserDocument goToItem:]): (-[BrowserDocument _addItem:toMenu:]): (-[BrowserDocument backListMenuForButton:]): (-[BrowserDocument forwardListMenuForButton:]): Code to build BF menus, and respond to clicks.
• BrowserWindowController.m: (-[BrowserWindowController windowDidLoad]): (-[BrowserWindowController frameLoadWithinPage:]): Register for and handle BrowserDocumentClientNavigatedWithinPageNotification. Must update title bar widgets at this point.
• Defaults.plist: New defaults: BackForwardListSizeLimit, BackForwardMenuSizeLimit
• LocationChangeHandler.m: (-[LocationChangeHandler locationChangedWithinPageForDataSource:]): Post BrowserDocumentClientNavigatedWithinPageNotification.
• PreferenceKeys.h: New default keys.

12:51 PM Changeset in webkit [2467] by cblu

• 3 edits in trunk/WebKit

Minor non-mentionable clean-up.

• WebCoreSupport.subproj/WebBridge.m: (-[WebBridge viewForPluginWithURL:attributes:baseURL:MIMEType:]):

10:11 AM Changeset in webkit [2466] by cblu

• 5 edits in trunk/WebKit

Cleaned up the plug-in API headers so they can be sent to other groups.

• Plugins.subproj/WebPlugin.h:
• Plugins.subproj/WebPluginContainer.h:
• Plugins.subproj/WebPluginViewFactory.h:

3:09 AM Changeset in webkit [2465] by mjs

• 4 edits in trunk/WebCore

• fixed 3065638 - "undefined" output to screen on empty document.write()

• khtml/ecma/kjs_html.cpp: (KJS::HTMLDocFunction::tryCall): Don;t assume there is at least one argument.

2:31 AM Changeset in webkit [2464] by rjw

• 2 edits in trunk/WebKit

* empty log message *

2:15 AM Changeset in webkit [2463] by mjs

• 4 edits in trunk/WebCore

• fixed 3048031 - innerHTML is incorrectly null

• kwq/KWQString.mm: (QString::isNull): Check for NULL by comparing to shared_null_handle instead of checking for 0 length, so garden-variety empty strings don't show as NULL. (QString::QString): Make sure that string created with a NULL char
• uses the shared null handle, so it's properly NULL.