Timeline



Feb 5, 2003:

11:39 PM Changeset in webkit [3581] by cblu
  • 1 edit in trunk/WebKit/ChangeLog

WebKit:

Comment should have been "Made WebPluginError constructors private."

  • ChangeLog:

WebBrowser:

Comment should have been "Made WebPluginError constructors private."

  • ChangeLog:
10:34 PM Changeset in webkit [3580] by cblu
  • 9 edits
    1 add in trunk/WebKit

WebKit:

WebKit API clean-up:

  • Added enum to WebContextMenuDelegate that is the menu item tag of the menu items passed in the default menu item array.
  • WebPluginError should return URL strings.
  • Made WebContextMenuDelegate constructors private.

Reviewed by rjw.

  • Plugins.subproj/WebNetscapePluginDocumentView.m: (-[WebNetscapePluginDocumentView setDataSource:]):
  • Plugins.subproj/WebPluginError.h:
  • Plugins.subproj/WebPluginError.m: (-[WebPluginError pluginPageURL]):
  • Plugins.subproj/WebPluginErrorPrivate.h: Added.
  • WebCoreSupport.subproj/WebBridge.m: (-[WebBridge viewForPluginWithURL:attributes:baseURL:MIMEType:]):
  • WebKit.pbproj/project.pbxproj:
  • WebView.subproj/WebContextMenuDelegate.h:
  • WebView.subproj/WebDefaultContextMenuDelegate.h:
  • WebView.subproj/WebDefaultContextMenuDelegate.m: (-[WebDefaultContextMenuDelegate menuItemWithTag:]): (-[WebDefaultContextMenuDelegate contextMenuItemsForElement:defaultMenuItems:]):

WebBrowser:

WebKit API clean-up:

  • Added enum to WebContextMenuDelegate that is the menu item tag of the menu items passed in the default menu item array.
  • WebPluginError should return URL strings.
  • Made WebContextMenuDelegate constructors private.

Reviewed by rjw.

  • ContextMenuHandler.m: (-[ContextMenuHandler addMenuItemWithTag:fromArray:toArray:]): (-[ContextMenuHandler addMenuItemWithTitle:action:toArray:]): (-[ContextMenuHandler contextMenuItemsForElement:defaultMenuItems:]):
  • LoadProgressMonitor.m: (-[LoadProgressMonitor launchApplicationSheetEnded:returnCode:contextInfo:]): (-[LoadProgressMonitor pluginErrorSheetEnded:returnCode:contextInfo:]): (-[LoadProgressMonitor pluginFailedWithError:dataSource:]):
8:56 PM Changeset in webkit [3579] by trey
  • 19 edits in trunk

WebCore:

Pass formValues up through the bridge at submit time.

Reviewed by Maciej

  • khtml/html/html_formimpl.cpp: (HTMLFormElementImpl::submit): Tell KWQPart about the form values, instead of talking to the khtmlview.
  • kwq/KWQKHTMLPart.h: New ivar to store formValues.
  • kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::KWQKHTMLPart): Init ivar (KWQKHTMLPart::~KWQKHTMLPart): Free ivar (KWQKHTMLPart::openURL): Pass nil formValues (KWQKHTMLPart::openURLRequest): Pass nil formValues (KWQKHTMLPart::clearRecordedFormValues): Clear saved formValues (KWQKHTMLPart::recordFormValue): Record a name/value (KWQKHTMLPart::submitForm): Pass along recorded formValues (KWQKHTMLPart::urlSelected): Pass nil formValues
  • kwq/KWQKHTMLPartBrowserExtension.mm: (KHTMLPartBrowserExtension::createNewWindow): Pass nil formValues
  • kwq/WebCoreBridge.h: New params for formValues

WebKit:

WC now tells us the form values being submitted with a get/post.
We forward this info to a new WebFormDelegate method. Most of this
change is just glue passing the formValues through all our layers.

Reviewed by Maciej

  • WebCoreSupport.subproj/WebBridge.m: (-[WebBridge loadURL:reload:triggeringEvent:formValues:]) Pass along formValues. (-[WebBridge postWithURL:data:contentType:triggeringEvent:formValues:]): Pass along formValues.
  • WebKit.pbproj/project.pbxproj: Reorder previously added files.
  • WebView.subproj/WebControllerPolicyDelegate.m: Fix latent copy/paste error where two keys had the same underlying string value!
  • WebView.subproj/WebFormDelegate.h: Declare new delegate method.
  • WebView.subproj/WebFormDelegate.m: (-[WebFormDelegate frame:willSubmitFormWithValues:]): NOP impl of new method.
  • WebView.subproj/WebFrame.m: (-[WebFrame loadRequest:]): Pass nil formValues. (-[WebFrame reload]): Pass nil formValues.
  • WebView.subproj/WebFramePrivate.h: New ivar to hold form values while waiting for the policy delegate response.
  • WebView.subproj/WebFramePrivate.m: (-[WebFramePrivate dealloc]): ASSERT new ivar is nil. (-[WebFrame _loadItem:fromItem:withLoadType:]): Pass nil formValues. (-[WebFrame _loadRequest:triggeringAction:loadType:formValues:]): Pass along formValues. (-[WebFrame _invalidatePendingPolicyDecisionCallingDefaultAction:]): Pass nil formValues to continuation method. (-[WebFrame _checkNavigationPolicyForRequest:dataSource:formValues:andCall:withSelector:]): Stash formValues before calling policy delegate. (-[WebFrame _continueAfterNavigationPolicy:]): Resurrect formValues after calling policy delegate, pass to continuation method. (-[WebFrame _continueFragmentScrollAfterNavigationPolicy:formValues:]): Bail on nil request instead of shouldContinue=NO (-[WebFrame _loadURL:loadType:triggeringEvent:formValues:]): Pass formValues along. (-[WebFrame _loadURL:intoChild:]): Pass nil formValues. (-[WebFrame _postWithURL:data:contentType:triggeringEvent:formValues:]): Pass formValues along. (-[WebFrame _reloadAllowingStaleDataWithOverrideEncoding:]): Pass nil formValues. (-[WebFrame _continueLoadRequestAfterNavigationPolicy:formValues:]): Bail on nil request instead of shouldContinue=NO. Call new FormDelegate method. (-[WebFrame _loadDataSource:withLoadType:formValues:]): Pass formValues along. (-[WebFrame _downloadRequest:toDirectory:]): Pass nil formValues.
  • WebView.subproj/WebMainResourceClient.m: (-[WebMainResourceClient continueAfterNavigationPolicy:formValues:]): Bail on nil request instead of shouldContinue=NO (-[WebMainResourceClient handle:willSendRequest:]): Pass nil formValues.

WebBrowser:

Add new FormCompletionController class, mostly unimplemented.
Add implementation of new WebFormDelegate method, which tells us
when a form is submitted.

Reviewed by Maciej

  • BrowserWindowController.m: (-[FormDelegate dealloc]): Free completion controller. (-[FormDelegate controlTextDidBeginEditing:inFrame:]): Alloc completion controller. (-[FormDelegate frame:willSubmitFormWithValues:]): New FormDelegate method, pass on to completion controller.
  • FormCompletionController.h: Added.
  • FormCompletionController.m: Added. New class, NOP impls. (-[FormCompletionController initWithSourceField:frame:]): (+[FormCompletionController _loadCompletionDB]): (+[FormCompletionController frame:willSubmitFormWithValues:]): (-[FormCompletionController executeQuery]): (-[FormCompletionController reflectSelectedRow]): (-[FormCompletionController reflectFinalSelectedRow]):
  • LocationChangeHandler.m: (-[LocationChangeHandler locationChangeStartedForDataSource:]): Made some code a little clearer.
  • WebBrowser.pbproj/project.pbxproj: New classes.
5:41 PM Changeset in webkit [3578] by rjw
  • 20 edits in trunk/WebKit

WebKit:

WebHistory* API clenaup.

Reviewed by Chris.

  • Bookmarks.subproj/WebBookmarkLeaf.m:
  • History.subproj/WebHistory.h:
  • History.subproj/WebHistory.m:
  • History.subproj/WebHistoryItem.h:
  • History.subproj/WebHistoryItem.m: (-[WebHistoryItem title]): (-[WebHistoryItem setDisplayTitle:]): (-[WebHistoryItem hash]): (-[WebHistoryItem anchor]): (-[WebHistoryItem isEqual:]): (-[WebHistoryItem description]): (-[WebHistoryItem _retainIconInDatabase:]): (+[WebHistoryItem entryWithURL:]): (-[WebHistoryItem initWithURL:title:]): (-[WebHistoryItem initWithURL:target:parent:title:]): (-[WebHistoryItem URL]): (-[WebHistoryItem target]): (-[WebHistoryItem parent]): (-[WebHistoryItem dictionaryRepresentation]): (-[WebHistoryItem initFromDictionaryRepresentation:]):
  • History.subproj/WebHistoryItemPrivate.h:
  • History.subproj/WebHistoryPrivate.h:
  • History.subproj/WebHistoryPrivate.m:
  • WebKit.pbproj/project.pbxproj:
  • WebView.subproj/WebController.h:
  • WebView.subproj/WebController.m: (-[WebController setPreferences:]): (-[WebController preferences]):
  • WebView.subproj/WebControllerPrivate.h:
  • WebView.subproj/WebControllerPrivate.m: (-[WebControllerPrivate dealloc]):
  • WebView.subproj/WebDataSourcePrivate.m:
  • WebView.subproj/WebFramePrivate.m:

WebBrowser:

WebHistory* API clenaup.

Reviewed by Chris.

  • BrowserDocument.m:
  • Debug/DebugUtilities.m:
  • GlobalHistory.m: (-[GlobalHistory updateDisplayTitleForHistoryItem:]):
  • URLCompletionController.m: (-[URLCompletionController tableView:objectValueForTableColumn:row:]):

Tests:

Added (-[DataResourceRequest initWithString:]).

Reviewed by me.

  • SnippetEditor/DataResourceRequest.h:
  • SnippetEditor/DataResourceRequest.m: (-[DataResourceRequest initWithString:]):
  • SnippetEditor/SnippetController.m: (-[SnippetController updateFromText:]):
3:54 PM Changeset in webkit [3577] by darin
  • 3 edits in trunk/WebCore

Reviewed by Dave.

  • fixed 3165682 -- REGRESSION: assertion failure in appendChildNode loading cnn.com
  • khtml/rendering/render_table.cpp: (RenderTable::addChild): Get rid of double-addChild introduced when I carelessly rolled in a change from the KHTML guys.
2:03 PM Changeset in webkit [3576]
  • 19 copies
    1 delete in tags/Safari-59

This commit was manufactured by cvs2svn to create tag 'Safari-59'.

2:03 PM Changeset in webkit [3575] by sheridan
  • 8 edits in branches/Safari-0-8-2-branch

Safari-59 versionstamp

1:54 PM Changeset in webkit [3574] by hyatt
  • 3 edits in trunk/WebCore

Fix for 3164949. My fix to make form controls actually obey
display settings in CSS exposed a bug in the html4.css
stylesheet. SELECT had been set to be display: block, which
is of course just plain wrong.

Reviewed by darin

  • khtml/css/html4.css:
12:28 AM Changeset in webkit [3573] by rjw
  • 10 edits in trunk

WebKit:

Cleanup public WebHistory API, stage 1.

Reviewed by trey.

  • History.subproj/WebHistory.h:
  • History.subproj/WebHistory.m: (-[_WebCoreHistoryProvider initWithHistory:]): (-[_WebCoreHistoryProvider containsEntryForURLString:]): (-[_WebCoreHistoryProvider dealloc]): (+[WebHistory sharedHistory]): (+[WebHistory createSharedHistoryWithFile:]):
  • History.subproj/WebHistoryPrivate.h:
  • History.subproj/WebHistoryPrivate.m:

WebBrowser:

Cleanup public WebHistory API, stage 1.

Reviewed by trey.

  • GlobalHistory.m: (-[GlobalHistory init]):

WebCore:

Cleanup public WebHistory API, stage 1.

Reviewed by trey.

  • kwq/KWQKHistoryProvider.mm:
  • kwq/WebCoreHistory.h:
  • kwq/WebCoreHistory.m: (+[WebCoreHistory setHistoryProvider:]): (+[WebCoreHistory historyProvider]):

Feb 4, 2003:

11:42 PM Changeset in webkit [3572] by darin
  • 3 edits in trunk/WebCore

Reviewed by Maciej.

  • fixed 3164850 -- REGRESSION: query URLs are broken

Mea culpa. The KURL test in the Tests directory failed last time, but for
some reason I didn't notice that when I ran it, perhaps due to the other
failing tests.

  • kwq/KWQKURL.mm: (appendEscapingBadChars): Don't escape '?' even though it is in the "bad character" set, since this is a more-lenient definition of bad character. Also fixed a buffer overrun bug in the '%'-sequence-preserving code, which is a second way this function is lenient. We still get the speed because most characters this routine processes are not bad characters. (KURL::encode_string): Instead of calling appendEscapingBadChars, put in a strict version of the escaping loop that does not tolerate pre-existing '%' sequences or '?' characters.
7:52 PM Changeset in webkit [3571] by rjw
  • 1 add in trunk/WebKit/API-Issues.rtf

WebKit API Notes.
Reviewed by me.

6:38 PM Changeset in webkit [3570] by darin
  • 3 edits in trunk/WebCore

Reviewed by Dave.

  • rolled in table code fix from Dirk on the khtml-dev list
  • khtml/rendering/render_table.cpp: (RenderTable::addChild): Change return back to break.
5:11 PM Changeset in webkit [3569] by mjs
  • 8 edits in branches/Safari-0-8-2-branch

JavaScriptCore:

Merged changes from the trunk:

2003-02-04 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Darin.

  • WebFoundation.pbproj/project.pbxproj: Updated to build the framework standalone instead of embedded when doing a B&I build for Panther.

WebFoundation:

Merged changes from the trunk:

2003-02-04 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Darin.

  • WebFoundation.pbproj/project.pbxproj: Updated to build the framework standalone instead of embedded when doing a B&I build for Panther.

WebCore:

Merged changes from the trunk:

2003-02-04 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Darin.

  • WebFoundation.pbproj/project.pbxproj: Updated to build the framework standalone instead of embedded when doing a B&I build for Panther.

WebKit:

Merged changes from the trunk:

2003-02-04 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Darin.

  • WebKit.pbproj/project.pbxproj: Updated to build the framework standalone instead of embedded when doing a B&I build for Panther.

WebBrowser:

Merged changes from the trunk:

2003-02-04 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Darin.

  • WebBrowser.pbproj/project.pbxproj: Updated to link against standalone frameworks instead of embeddeding them when doing a B&I build for Panther.
4:51 PM Changeset in webkit [3568] by darin
  • 12 edits in trunk/WebCore

Reviewed by Maciej.

  • we say goodbye to "MIB numbers"
  • khtml/misc/decoder.cpp: (Decoder::Decoder): Get the Latin-1 codec using a name. (Decoder::setEncoding): Detect the Hebrew encoding using isISOLatin1Hebrew() instead of MIB number. (Decoder::decode): Get the UCS-2 codec using a name. Redo APPLE_CHANGES so we don't get the MIB-number-based workaround for a bug in QTextCodec that KWQ doesn't have. Get the Latin-1 codec using a name.
  • khtml/misc/loader.cpp: (CachedCSSStyleSheet::CachedCSSStyleSheet): Get the Latin-1 codec using a name. (CachedScript::CachedScript): Get the Latin-1 codec using a name.
  • kwq/KWQCharsets.h: Remove the two MIB-related functions.
  • kwq/KWQCharsets.mm: Remove the MIB number from the character set table struct. Remove the MIB-related dictionary globals and functions. (buildDictionaries): Remove code to set up the MIB-related dictionaries.
  • kwq/KWQTextCodec.h: Remove codecForMib, mibEnum, and encoding member functions. Add isISOLatin1Hebrew member function.
  • kwq/KWQTextCodec.mm: (QTextCodec::makeDecoder): Pass in the encoding when making KWQTextDecoder rather than passing in the QTextCodec object. (QTextCodec::toUnicode): Ditto. (QTextCodec::isISOLatin1Hebrew): Return true for kCFStringEncodingISOLatinHebrew only. (KWQTextDecoder::convertUsingTEC): Get encoding directly instead of from a QTextCodec. (KWQTextDecoder::toUnicode): Ditto.
  • kwq/make-charset-table.pl: Don't write out MIB numbers.
  • kwq/KWQCharsetData.c: Regenerated.
  • khtml/khtml_part.cpp: (KHTMLPart::completeURL): Don't bother passing the MIB number to KURL. Soon, we'll have to do something smart here, but we won't use MIB numbers.
  • kwq/mac-encodings.txt: Added more 16-bit Unicode character set names. Updated some comments.
4:47 PM Changeset in webkit [3567] by hyatt
  • 3 edits in trunk/WebCore

Patch calcInlineMaxWidth to add in left/right margins/borders/padding
separately (since the inline could break across multiple lines).

Reviewed by darin

  • khtml/rendering/render_block.cpp: (RenderBlock::calcInlineMinMaxWidth):
3:58 PM Changeset in webkit [3566] by mjs
  • 8 edits in trunk

JavaScriptCore:

Reviewed by Darin.

  • JavaScriptCore.pbproj/project.pbxproj: Updated to build the framework standalone instead of embedded when doing a B&I build for Panther.

WebFoundation:

Reviewed by Darin.

  • WebFoundation.pbproj/project.pbxproj: Updated to build the framework standalone instead of embedded when doing a B&I build for Panther.

WebCore:

Reviewed by Darin.

  • WebCore.pbproj/project.pbxproj: Updated to build the framework standalone instead of embedded when doing a B&I build for Panther.

WebKit:

Reviewed by Darin.

  • WebKit.pbproj/project.pbxproj: Updated to build the framework standalone instead of embedded when doing a B&I build for Panther.

WebBrowser:

Reviewed by Darin.

  • WebBrowser.pbproj/project.pbxproj: Updated to link against standalone frameworks instead of embeddeding them when doing a B&I build for Panther.
2:51 PM Changeset in webkit [3565] by hyatt
  • 5 edits in trunk/WebCore

Patch #1: Compacts are supposed to use their maximum intrinsic width
when determining if they will fit. I was wrongly using the
minwidth (i.e., being too aggressive about trying to squeeze
the compact in there).

Also yanking the setMaxWidth/MinWidth setters, since I'm not
using them any more.

Reviewed by kocienda

Patch #2: Implement an iterator for min/max width calculations
of blocks with inline children. This iterator will include
inline flows twice (with a bool flag indicating whether you're at the
start or end of the flow).

Reviewed by darin

  • khtml/rendering/render_block.cpp: (RenderBlock::layoutBlockChildren):
  • khtml/rendering/render_box.h:
  • khtml/rendering/render_object.h:
1:34 PM Changeset in webkit [3564] by darin
  • 8 edits in trunk/WebCore

Reviewed by Maciej.

  • fixed 3142813 -- Safari displays a blank page when trying to open html file encoded in little-endian UTF-16
  • kwq/KWQString.h: Remove fromStringWithEncoding and convertToQCString, add append that takes a QChar * and length.
  • kwq/KWQString.mm: (QString::fromUtf8): Use QTextCodec instead of fromStringWithEncoding. (QString::append): Implement by calling insert.
  • kwq/KWQTextCodec.h: Added an encoding() member function to QTextCodec and made QTextDecoder an abstract base class to pave the way for putting more state in it to handle multibyte encodings.
  • kwq/KWQTextCodec.mm: Add KWQTextDecoder, a concrete subclass of QTextDecoder. (codecForCFStringEncoding): Moved the static inside this function for clarity. (QTextCodec::makeDecoder): Make a KWQTextDecoder. (QTextCodec::toUnicode): Make a KWQTextDecoder and use it do the decoding instead of calling fromStringWithEncoding. (KWQTextDecoder::convertUTF16): Added. Decodes Unicode, using the BOM to decide whether it's little-endian or big-endian. (KWQTextDecoder::convertUsingTEC): Added. Here's where the code from QString::fromStringWithEncoding was moved. This needs to be changed to handle state between multiple buffers passed in eventually, but not to fix this bug. (KWQTextDecoder::toUnicode): Added. Calls convertUTF16 or convertUsingTEC.
  • kwq/WebCoreBridge.mm: (+[WebCoreBridge stringWithData:textEncoding:]): Use QTextCodec instead of fromStringWithEncoding.
  • khtml/misc/decoder.cpp: (Decoder::decode): Rearranged the #if to make things easier to read.
12:41 PM Changeset in webkit [3563] by cblu
  • 2 edits in trunk/WebKit

Fixed: 3163879 - receivedPageIcon:forDataSource: always sends nil image

Reviewed by rjw.

  • WebView.subproj/WebDataSourcePrivate.m: (-[WebDataSource _updateIconDatabaseWithURL:]):
11:08 AM Changeset in webkit [3562] by darin
  • 4 edits in trunk/WebCore

Reviewed by Trey.

  • fixed 3150252 -- files with question marks in their names fail to upload as attachments
  • khtml/html/html_formimpl.cpp: (HTMLFormElementImpl::formData): Don't include a Content-Type header at all if we don't have a MIME type to send. Matches other web browsers' behavior. (HTMLInputElementImpl::encoding): Construct the URL using setPath rather than just passing a filename to the KURL constructor. (HTMLSelectElementImpl::state): Tweak #if formatting a bit. (HTMLSelectElementImpl::restoreState): Improve comment a bit.
  • kwq/KWQKURL.mm: Make ? be a "bad character". I checked carefully to be sure this does not adversely affect any calls, including running our KURL tests. (KURL::setPath): Encode the passed-in path. I determined this was correct both by looking at all the call sites in KHTML code to see that they would want this behavior, and by noticing that the path() getter decodes the path.
9:15 AM Changeset in webkit [3561] by kocienda
  • 3 edits in trunk/WebCore

WebFoundation:

Reviewed by Darin

Fix for this bug:

Radar 3142922 (file URLs should never be sent as referrers)

Note that this code is added to this framework to close this security
hole for WebFoundation-only clients. Safari does not depend on this change
to fix the bug.

  • ProtocolHandlers.subproj/WebHTTPResourceRequest.m: (-[WebResourceRequest setReferrer:]): Return if argument passed in returns YES when sent _web_isFileURL message.

WebCore:

Reviewed by Darin

Radar 3142922 (file URLs should never be sent as referrers)

  • kwq/WebCoreBridge.mm: (-[WebCoreBridge referrer]): Check that the referrer does not start with the prefix "file:". If it does, return nil.
8:53 AM Changeset in webkit [3560] by darin
  • 2 edits in trunk/WebKit

Reviewed by Ken.

  • fixed 3135388 -- encoding changes do not affect text in form elements

The problem was that form data was being saved and restored, and in the case of buttons,
the form data was the incorrectly decoded button label.

  • WebView.subproj/WebFramePrivate.m: (-[WebFrame _transitionToLayoutAcceptable]): Remove the default: case so we get a warning if we omit one of the enum values from this switch statement. (-[WebFrame _transitionToCommitted:]): Move the WebFrameLoadTypeReloadAllowingStaleData case up so it's right next to the case it should be merged with. (-[WebFrame _isLoadComplete]): Remove the default: case so we get a warning if we omit one of enum values from this switch statement. (-[WebFrame _itemForRestoringDocState]): Replace the if statement with a switch statement. Include WebFrameLoadTypeReloadAllowingStaleData, which fixes the bug.

Feb 3, 2003:

5:33 PM Changeset in webkit [3559]
  • 19 copies
    1 delete in branches/Safari-58-1-branch

This commit was manufactured by cvs2svn to create branch
'Safari-58-1-branch'.

5:33 PM Changeset in webkit [3558]
  • 19 copies
    1 delete in tags/Safari-58

This commit was manufactured by cvs2svn to create tag 'Safari-58'.

5:33 PM Changeset in webkit [3557]
  • 19 copies
    1 delete in tags/SAFARI_58_1_ANCHOR

This commit was manufactured by cvs2svn to create tag
'SAFARI_58_1_ANCHOR'.

5:33 PM Changeset in webkit [3556] by sheridan
  • 8 edits in branches/Safari-0-8-2-branch

Safari-58 versionstamp

4:51 PM Changeset in webkit [3555] by darin
  • 3 edits in branches/Safari-0-8-2-branch/WebKit

Rolled over changes from the trunk.

2003-02-03 Chris Blumenberg <​cblu@apple.com>

Fixed: 3163073 - SECURITY: Need to check if filename in encoded download is safe

Reviewed by kocienda, darin, mjs.

  • Downloads.subproj/WebDownloadHandler.m: (-[WebDownloadHandler createFileIfNecessary]): call _web_filenameByFixingIllegalCharacters on the enco
4:47 PM Changeset in webkit [3554] by darin
  • 3 edits in branches/Safari-0-8-2-branch/WebCore

Rolled over changes from the main trunk.

2003-02-03 David Hyatt <​hyatt@apple.com>

Fix for 3163603, livepage.apple.com repaints entire page.

Reviewed by darin.

  • khtml/rendering/render_object.cpp: (RenderObject::setStyle):
4:43 PM Changeset in webkit [3553] by hyatt
  • 3 edits in trunk/WebCore

Fix for 3163603, livepage.apple.com repaints entire page.

Reviewed by darin.

  • khtml/rendering/render_object.cpp: (RenderObject::setStyle):
4:32 PM Changeset in webkit [3552] by darin
  • 3 edits in trunk/WebCore

Reviewed by Dave.

  • fixed 3133902 -- leaked render objects from makeChildrenNonInline (4 after running cvs-base)
  • khtml/rendering/render_block.cpp: (RenderBlock::removeChild): Deatch the anonymous block after emptying it out.
4:08 PM Changeset in webkit [3551] by hyatt
  • 3 edits in trunk/WebCore

Fix for livepage.apple.com crash. The bug # is
3163508.

Reviewed by darin

  • khtml/rendering/render_box.cpp: (RenderBox::setPos):
3:41 PM Changeset in webkit [3550] by hyatt
  • 7 edits in trunk/WebCore

Rename TextSlave to TextRun.

Reviewed by darin

  • khtml/khtml_part.cpp: (firstRunAt): (lastRunAt): (startAndEndLineNodesIncludingNode):
  • khtml/rendering/bidi.cpp: (RenderBlock::layoutInlineChildren):
  • khtml/rendering/render_text.cpp: (TextRun::detach): (throw): (TextRun::operator delete): (TextRun::paintSelection): (TextRun::paintDecoration): (TextRun::paintBoxDecorations): (TextRun::checkSelectionPoint): (TextRunArray::TextRunArray): (TextRunArray::compareItems): (TextRunArray::findFirstMatching): (RenderText::detach): (RenderText::deleteRuns): (RenderText::findTextRun): (RenderText::nodeAtPoint): (RenderText::checkSelectionPoint): (RenderText::cursorPos): (RenderText::posOfChar): (RenderText::paintObject): (RenderText::position): (RenderText::width):
  • khtml/rendering/render_text.h:
  • kwq/KWQRenderTreeDebug.cpp: (writeTextRun): (write):
3:19 PM Changeset in webkit [3549] by hyatt
  • 3 edits in trunk/WebCore

Fix for assertion on nasdaq.com. Patch containingBlock
to return 0 when it can't find a containingBlock at all (e.g.,
when the object is orphaned).

Patch setOverhangingContents to deal with a null containingBlock
caused by it being called from RenderImage::setStyle().

The bug # is 3162989.

Reviewed by NOBODY (OOPS!).

  • khtml/rendering/render_object.cpp: (RenderObject::containingBlock): (RenderObject::setOverhangingContents):
2:48 PM Changeset in webkit [3548] by hyatt
  • 15 edits in trunk/WebCore

Enable support for the "run-in" and "compact" display types.

Also implementing support for :before and :after (they were not even
remotely working before these fixes).

Made sure :first-letter drills into inline children to find the
first letter in a block, e.g., so :before content can be used in
conjunction with :first-letter.

Fixed a regression in addOverhangingFloats that broke meyerweb.com's
front page.

Patched findNextLineBreak to correctly handle words that cross
inline boundaries and white-space mixtures. We now pass Hixie's
insanely evil mixed white-space test (we are the only browser
to do so).

Reviewed by kocienda

  • ChangeLog:
  • khtml/css/cssstyleselector.cpp:
  • khtml/rendering/bidi.cpp: (RenderBlock::findNextLineBreak):
  • khtml/rendering/render_block.cpp: (RenderBlock::setStyle): (RenderBlock::addChildToFlow): (getInlineRun): (RenderBlock::layoutBlockChildren): (RenderBlock::paintObject): (RenderBlock::clearFloats):
  • khtml/rendering/render_box.cpp: (RenderBox::calcWidthUsing):
  • khtml/rendering/render_box.h:
  • khtml/rendering/render_container.cpp: (RenderContainer::insertPseudoChild):
  • khtml/rendering/render_container.h:
  • khtml/rendering/render_flow.cpp: (RenderFlow::addChild):
  • khtml/rendering/render_image.cpp: (RenderImage::setStyle):
  • khtml/rendering/render_inline.cpp: (RenderInline::setStyle): (RenderInline::addChildToFlow):
  • khtml/rendering/render_object.cpp: (RenderObject::createObject):
  • khtml/rendering/render_object.h:
  • khtml/rendering/render_style.cpp: (RenderStyle::addPseudoStyle):
12:09 PM Changeset in webkit [3547] by rjw
  • 2 edits in trunk/WebKit

Fixed headerdoc type.

Reviewed by me.

  • WebView.subproj/WebDocument.h:
11:57 AM Changeset in webkit [3546] by rjw
  • 18 edits in trunk/WebKit

WebKit:

Support for new canProvideDocumentSource and documentSource API.
Updated all the representations. Added support to show for RTF source.
Moved WebKitInitializeUnicode to to early class. This was necessary
because WebCore may use the unicode property functions before WebTextRenderer
gets initialized.
Ensured guarantee that identifierForInitialRequest:fromDataSource: is called with
the first initial request.

Reviewed by maciej.

  • Plugins.subproj/WebNetscapePluginRepresentation.m: (-[WebNetscapePluginRepresentation canProvideDocumentSource]): (-[WebNetscapePluginRepresentation documentSource]):
  • WebCoreSupport.subproj/WebTextRenderer.m:
  • WebView.subproj/WebBaseResourceHandleDelegate.h:
  • WebView.subproj/WebBaseResourceHandleDelegate.m: (-[WebBaseResourceHandleDelegate setIdentifier:]):
  • WebView.subproj/WebDataSource.h:
  • WebView.subproj/WebDataSource.m: (-[WebDataSource initWithRequest:]): (-[WebDataSource controller]): (-[WebDataSource initialRequest]):
  • WebView.subproj/WebDataSourcePrivate.h:
  • WebView.subproj/WebDataSourcePrivate.m: (-[WebDataSourcePrivate dealloc]): (-[WebDataSource _startLoading:]): (-[WebDataSource _originalRequest]):
  • WebView.subproj/WebDocument.h:
  • WebView.subproj/WebHTMLRepresentation.h:
  • WebView.subproj/WebHTMLRepresentation.m: (-[WebHTMLRepresentation finishedLoadingWithDataSource:]): (-[WebHTMLRepresentation canProvideDocumentSource]):
  • WebView.subproj/WebHTMLView.m: (+[WebHTMLView initialize]):
  • WebView.subproj/WebImageRepresentation.m: (-[WebImageRepresentation canProvideDocumentSource]): (-[WebImageRepresentation documentSource]):
  • WebView.subproj/WebTextRepresentation.h:
  • WebView.subproj/WebTextRepresentation.m: (-[WebTextRepresentation dealloc]): (-[WebTextRepresentation setDataSource:]): (-[WebTextRepresentation finishedLoadingWithDataSource:]): (-[WebTextRepresentation canProvideDocumentSource]): (-[WebTextRepresentation documentSource]):

WebBrowser:

Make us of new canProvideDocumentSource API, replacing somewhat
klunky usage of isDocumentHTML.

Reviewed by maciej.

  • BrowserDocument.m: (-[BrowserDocument validateUserInterfaceItem:]): (-[BrowserDocument source]):
  • ContextMenuHandler.m: (-[ContextMenuHandler contextMenuItemsForElement:defaultMenuItems:]):
  • HTMLSourceDocument.m: (-[HTMLSourceDocument makeWindowControllers]): (-[HTMLSourceDocument setDataSource:]):
11:28 AM Changeset in webkit [3545] by cblu
  • 3 edits in trunk/WebKit

Fixed: 3163073 - SECURITY: Need to check if filename in encoded download is safe

Reviewed by kocienda, darin, mjs.

  • Downloads.subproj/WebDownloadHandler.m: (-[WebDownloadHandler createFileIfNecessary]): call _web_filenameByFixingIllegalCharacters on the encoded filename
10:41 AM Changeset in webkit [3544]
  • 16 copies
    1 delete in tags/Safari-57

This commit was manufactured by cvs2svn to create tag 'Safari-57'.

10:41 AM Changeset in webkit [3543] by sheridan
  • 8 edits in branches/Safari-0-8-2-branch

Safari-57 versionstamp

9:38 AM Changeset in webkit [3542] by darin
  • 3 edits in branches/Safari-0-8-2-branch/WebCore

Rolled over changes from the main trunk.

2003-01-30 Trey Matteson <​trey@apple.com>

3161486 - REGRESSION: double click on a widget causes crash

Slight enhancement to the earlier fix of this bug. When handling a doubleclick we
don't want to try to pass the current event down the Cocoa widgets, because Cocoa
has no double-click event, and we just end up resending one we already sent.

The upshot is that we do not pass an extra mouseup event to the widgets at the
end of a double click sequence.

Reviewed by Darin.

  • kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::passWidgetMouseDownEventToWidget): Do not pass event down to the widget for doubleclick events.
9:02 AM Changeset in webkit [3541] by darin
  • 9 edits in trunk/WebCore

Reviewed by Maciej.

  • fixed 3146862 -- meta http-equiv refresh with a space before the delay does not work
  • khtml/xml/dom_docimpl.cpp: (DocumentImpl::processHttpEquiv): Fix code that was trying to use positions within the whitespace-stripped string, but extracting substrings from the non-stripped original.
  • tweaks to render tree dumping format
  • khtml/rendering/render_block.h: Make renderName not be inline, since an inline virtual function doesn't really do any good.
  • khtml/rendering/render_block.cpp: (RenderBlock::renderName): Use the string RenderBlock instead of the string Block for consistency with all the other renderName results.
  • khtml/rendering/render_inline.h: Make renderName not be inline, since an inline virtual function doesn't really do any good.
  • khtml/rendering/render_inline.cpp: (RenderInline::renderName): Use the string RenderInline instead of the string Inline for consistency with all the other renderName results.
  • khtml/rendering/render_text.h: Make textSlaves() a const member function.
  • kwq/KWQRenderTreeDebug.cpp: (operator<<): Format rectangles as "x at (1,2) size 3x4" rather than "(1,2,3,4)". (quoteAndEscapeNonPrintables): Added. Makes it easy to turn an arbitrary Unicode string into something that's all ASCII. (writeTextSlave): Change this so it also writes out the text. (write): Call the new writeTextSlave, and also use "layer" instead of "RenderLayer".

Feb 1, 2003:

5:29 PM Changeset in webkit [3540] by darin
  • 3 edits in trunk/WebCore
  • fix mistake that broke build
  • khtml/rendering/render_block.h: Add include of <qsortedlist.h>.
5:02 PM Changeset in webkit [3539] by mjs
  • 3 edits in trunk/WebKit

Reviewed by Darin and Ken.

  • fixed 3162581 - 56 debug: Assertion failure displaying pop-up menu while downloading
  • WebView.subproj/WebMainResourceClient.m: (-[WebMainResourceClient handle:didReceiveResponse:]): No need to assert that the controller is not deferred if this is a handle being used for a download. (-[WebMainResourceClient handle:didReceiveData:]): Likewise. (-[WebMainResourceClient handleDidFinishLoading:]): Likewise. (-[WebMainResourceClient handle:didFailLoadingWithError:]): Likewise.
4:51 PM Changeset in webkit [3538] by darin
  • 12 edits in trunk/WebCore

Reviewed by Dave.

  • reduced the amount of RenderFlow in the world by about 1/2
  • khtml/rendering/bidi.h: Take RenderBlock * instead of RenderFlow *.
  • khtml/rendering/bidi.cpp: (BidiIterator::BidiIterator): Take RenderBlock * instead of RenderFlow *. (RenderBlock::layoutInlineChildren): Fix debug message to say RenderBlock instead of RenderFlow. (RenderBlock::findNextLineBreak): Fix debug message to say RenderBlock instead of RenderFlow.
  • khtml/rendering/render_block.cpp: (RenderBlock::addChildToFlow): Make RenderBlock * instead of RenderFlow *. (RenderBlock::layout): Fix debug message to say RenderBlock instead of RenderFlow. (RenderBlock::layoutBlockChildren): Fix debug message to say RenderBlock instead of RenderFlow. (RenderBlock::clearFloats): Fix debug message to say RenderBlock instead of RenderFlow.
  • khtml/rendering/render_flow.h: Return a RenderFlow * instead of a RenderObject * from createFlow.
  • khtml/rendering/render_flow.cpp: (RenderFlow::createFlow): Use an if statement instead of ? : to get rid of typecasting.
  • khtml/rendering/render_html.cpp: (RenderHtml::containingBlockWidth): Fix parent call to call RenderBlock. Calling RenderFlow was a mistake, but one that didn't matter since the actual method isn't implemented until up in RenderBox anyway.
  • khtml/rendering/render_inline.h: Change some parameters to RenderBlock *.
  • khtml/rendering/render_inline.cpp: (cloneInline): Make this return a RenderInline * instead of a RenderFlow *. (RenderInline::splitInlines): Change some parameters to RenderBlock * and locals to RenderInline *. (RenderInline::splitFlow): Change some parameters and locals to RenderBlock *.
  • khtml/rendering/render_layer.h: Remove forward declaration of class RenderFlow.
  • khtml/rendering/render_object.h: Remove forward declaration and friend declaration of class RenderFlow.
1:39 PM Changeset in webkit [3537] by trey
  • 12 edits
    2 adds in trunk

WebCore:

New plumbing for autofill/autocomplete. WebBrowser receives controlText delegate
messages from our form widgets. Should result in no behavior change.

New SPI is exposed over the bridge and KWQTextField passes the text delegate msgs to it.

Reviewed by Richard

  • kwq/KWQTextField.mm: Pass text delegate msgs through new SPI. (-[KWQTextField controlTextDidBeginEditing:]): (-[KWQTextField controlTextDidEndEditing:]): (-[KWQTextField controlTextDidChange:]): (-[KWQTextField control:textShouldBeginEditing:]): (-[KWQTextField control:textShouldEndEditing:]): (-[KWQTextField control:didFailToFormatString:errorDescription:]): (-[KWQTextField control:didFailToValidatePartialString:errorDescription:]): (-[KWQTextField control:isValidObject:]): (-[KWQTextField control:textView:doCommandBySelector:]):
  • kwq/WebCoreBridge.h: New SPI for passing text delegate msgs.

WebKit:

New plumbing for autofill/autocomplete. WebBrowser receives controlText delegate
messages from our form widgets. Should result in no behavior change.

New private API is exposed for the app to set a WebFormDelegate.

Reviewed by Richard

  • WebCoreSupport.subproj/WebBridge.m: Pass msgs from WC on to the controller's FormDelegate. (formDelegate): Little utility function. (-[WebBridge controlTextDidBeginEditing:]): (-[WebBridge controlTextDidEndEditing:]): (-[WebBridge controlTextDidChange:]): (-[WebBridge control:textShouldBeginEditing:]): (-[WebBridge control:textShouldEndEditing:]): (-[WebBridge control:didFailToFormatString:errorDescription:]): (-[WebBridge control:didFailToValidatePartialString:errorDescription:]): (-[WebBridge control:isValidObject:]): (-[WebBridge control:textView:doCommandBySelector:]):
  • WebKit.exp: New class exported.
  • WebKit.pbproj/project.pbxproj: New WebFormDelegate files.
  • WebView.subproj/WebControllerPrivate.h: API to set/get FormDelegate.
  • WebView.subproj/WebControllerPrivate.m: (-[WebController _setFormDelegate:]): simple setter (-[WebController _formDelegate]): simple getter
  • WebView.subproj/WebFormDelegate.h: Added.
  • WebView.subproj/WebFormDelegate.m: Added. NOP impls for all these delegate methods. (-[WebFormDelegate controlTextDidBeginEditing:inFrame:]): (-[WebFormDelegate controlTextDidEndEditing:inFrame:]): (-[WebFormDelegate controlTextDidChange:inFrame:]): (-[WebFormDelegate control:textShouldBeginEditing:inFrame:]): (-[WebFormDelegate control:textShouldEndEditing:inFrame:]): (-[WebFormDelegate control:didFailToFormatString:errorDescription:inFrame:]): (-[WebFormDelegate control:didFailToValidatePartialString:errorDescription:inFrame:]): (-[WebFormDelegate control:isValidObject:inFrame:]): (-[WebFormDelegate control:textView:doCommandBySelector:inFrame:]):

WebBrowser:

New plumbing for autofill/autocomplete. WebBrowser receives controlText delegate
messages from our form widgets. Should result in no behavior change.

This is mostly using new API, with a couple autocomplete fixes for newly exposed issues.

Reviewed by Richard

  • BrowserWindowController.h: New formDelegate ivar.
  • BrowserWindowController.m: (-[BrowserWindowController windowDidLoad]): Create FormDelegate, an internal helper object to field WebFormDelegate messages. (-[BrowserWindowController dealloc]): Release FormDelegate. (-[BrowserWindowController controlTextDidEndEditing:]): Pass msg on to completion controller. (-[FormDelegate initWithController:]): Create CompletionController to do autocomplete (commented out for now). (-[FormDelegate controlTextDidBeginEditing:inFrame:]): Pass msg on to completion controller. (-[FormDelegate controlTextDidEndEditing:inFrame:]): Pass msg on to completion controller. (-[FormDelegate controlTextDidChange:inFrame:]): Pass msg on to completion controller. (-[FormDelegate control:textView:doCommandBySelector:inFrame:]): Pass msg on to completion controller.
  • CompletionController.h: Added controlTextDidBeginEditing: Should have been receiving this all along.
  • CompletionController.m: (-[CompletionController _buildUI]): Set TableView font to match the textfield. (-[CompletionController _positionWindow]): Position window relative to TextField, not enclosing ScrollView (bogus code that's a NOP for the locationField). (-[CompletionController controlTextDidEndEditing:]): Hide the completion window when we leave the field.
Note: See TracTimeline for information about the timeline view.