Timeline



Sep 6, 2007:

8:48 PM Changeset in webkit [25410] by tristan
  • 7 edits
    4 adds in trunk

WebCore:

Reviewed by Maciej Stachowiak.


<rdar://problem/5333496> Back button stopped working on sfgate.com (14957)


This fix is specifically targted to address sfgate.com and reuters.com with minimal
impact to other areas of the frameloader. It does not cause any regression tests to fail.
I've added two layout tests: One to detect this particular bug, and one to address
a secondary issue: if an iframe navigation occurs during onload by a timeout an
additional history item is added, similarly to firefox. We tried to match firefox
more than IE with this fix but did gain a little bit of IE compatability.

Tests: http/tests/navigation/onload-navigation-iframe-timeout.html

http/tests/navigation/onload-navigation-iframe.html

  • dom/Document.h: (WebCore::Document::processingLoadEvent): Return the m_processingLoadEvent boolean so FrameLoader knows that the document is in the middle of calling <body onload>


  • html/HTMLFrameOwnerElement.cpp: (WebCore::HTMLFrameOwnerElement::HTMLFrameOwnerElement):
  • html/HTMLFrameOwnerElement.h: (WebCore::HTMLFrameOwnerElement::createdByParser): (WebCore::HTMLFrameOwnerElement::setCreatedByParser): Added support for a new member variable of frame elements: m_createdByParser. This lets us specifically target the fix to only iframes created via JS and not in-document.


  • loader/FrameLoader.cpp: (WebCore::FrameLoader::FrameLoader): Initialize m_navigationDuringLoad to false


(WebCore::FrameLoader::provisionalLoadStarted):
Determine if the load we're about to start is occuring during
an onload.


(WebCore::FrameLoader::updateHistoryForStandardLoad):
If the current frameloader is for a child frame, and the navigation is occuring
during an onload update the current history item rather than adding a new one.


  • loader/FrameLoader.h: Added a new member variable, m_navigationDuringLoad to track the navigation status during the on load, rather than trying to determine the status after (which is nearly impossible due to a new runloop spin).

LayoutTests:

Reviewed by Maciej Stachowiak.

  • http/tests/navigation/onload-navigation-iframe-expected.txt: Added.
  • http/tests/navigation/onload-navigation-iframe-timeout-expected.txt: Added.
  • http/tests/navigation/onload-navigation-iframe-timeout.html: Added.
  • http/tests/navigation/onload-navigation-iframe.html: Added.
7:53 PM Changeset in webkit [25409] by mjs
  • 4 edits
    1 delete in trunk/JavaScriptCore

Reviewed by Darin.


  • Remove single-threaded optimization for FastMalloc.


It does not appear to help anywhere but Mac OS X on PPC, due to
pthread_getspecific being slow there. On Intel, removing the
optimization results in a ~1.5% PLT speedup, a ~1-5% JS iBench
speedup, and a ~1.5% HTML iBench speedup. On PPC this change is a
speedup on some benchmarks, a slight hit on others.

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • kjs/collector.cpp: (KJS::Collector::registerThread):
  • wtf/FastMalloc.cpp: (WTF::TCMalloc_ThreadCache::GetCache): (WTF::TCMalloc_ThreadCache::GetCacheIfPresent): (WTF::TCMalloc_ThreadCache::CreateCacheIfNecessary): (WTF::do_malloc):
  • wtf/FastMallocInternal.h: Removed.
7:02 PM Changeset in webkit [25408] by weinig
  • 2 edits in trunk/LayoutTests

Reviewed by Kevin McCullough.

Update the Windows skiplist with the current failing tests.

  • platform/win/Skipped:
5:14 PM Changeset in webkit [25407] by sfalken
  • 1 copy in tags/Safari-523.4b

New tag.

5:14 PM Changeset in webkit [25406] by sfalken
  • 1 edit in trunk/WebKit/win/WebKit.vcproj/VERSION

Bump version for submit

3:38 PM Changeset in webkit [25405] by ddkilzer
  • 2 edits in trunk/WebCore

WebCore:

Reviewed by Darin.

The implementation of BackForwardList::clearPageCache() was removed in r21793.
It's now safe to remove it from the header.

  • history/BackForwardList.h: Removed clearPageCache().
3:34 PM Changeset in webkit [25404] by thatcher
  • 1 edit in trunk/WebKit/WebCoreSupport/WebDragClient.mm

Even better build fix.

3:17 PM Changeset in webkit [25403] by thatcher
  • 1 edit in trunk/WebKit/WebCoreSupport/WebDragClient.mm

Better build fix.

3:15 PM Changeset in webkit [25402] by thatcher
  • 1 edit in trunk/WebKit/WebCoreSupport/WebDragClient.mm

Build fix.

2:28 PM Changeset in webkit [25401] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Ada.

<rdar://problem/5457844>
Crash when using mouse wheel with plug-ins disabled.

Null check the widget.

  • page/EventHandler.cpp: (WebCore::EventHandler::handleWheelEvent):
2:15 PM Changeset in webkit [25400] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Mitz.

Small tweak.


  • plugins/win/PluginStreamWin.cpp: (WebCore::PluginStreamWin::startStream):
2:12 PM Changeset in webkit [25399] by andersca
  • 2 edits
    1 move in trunk/LayoutTests

Reviewed by Mitz.

Re-enable and the npapi-response-headers.html test so it no longer
relies on the ordering of the requests being made.


  • http/tests/plugins/npapi-response-headers-expected.txt:
  • http/tests/plugins/npapi-response-headers.html: Copied from http/tests/plugins/npapi-response-headers.html.disabled.
  • http/tests/plugins/npapi-response-headers.html.disabled: Removed.
2:04 PM Changeset in webkit [25398] by andersca
  • 3 edits in trunk/WebCore

Reviewed by Adam and Steve.

<rdar://problem/5459321>
ADOBE: Safari 3 on Windows is passing bogus values in stream->headers in NPP_NewStream, causing Flash Player to crash.


Serialize the headers into a stream and pass it to NPP_NewStream in stream->headers.


  • plugins/win/PluginStreamWin.cpp: (WebCore::PluginStreamWin::startStream):
  • plugins/win/PluginStreamWin.h:
1:57 PM Changeset in webkit [25397] by darin
  • 5 edits in trunk/WebCore

Reviewed by Hyatt.

Back out the willRemove() part of the previous patch.
It was incorrect and not needed to fix the bug anyway.

  • dom/Document.cpp: (WebCore::Document::detach): Remove willRemove().
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::clear): Put the willRemove() call back here where it was originally before the previous fix.
  • history/CachedPage.cpp: (WebCore::CachedPage::clear):
  • page/Frame.cpp: (WebCore::Frame::setView): (WebCore::Frame::setDocument): Added FIXMEs about the fact that we call detach() on the document but not willRemove().
1:43 PM Changeset in webkit [25396] by thatcher
  • 18 edits in trunk

WebCore:

Reviewed by Darin.

Changed the blocked exception log message to match AppKit's blocked exception message.

  • platform/mac/BlockExceptions.mm: (ReportBlockedObjCException):

WebKit:

Reviewed by Darin.

<rdar://problem/5443883> Uncaught Objective-C exceptions in WebKit clients lead to hard-to-diagnose crashes

Changed all the direct delegate calls to use helper functions that have direct access to
WebView's delegate objects. These helper methods will catch any ObjC exceptions and call
ReportDiscardedDelegateException to log the discarded exception. WebView's that have
catchesDelegateExceptions set to NO will not pay the cost of a @try/@catch. The delegate
forwarders also have the same behavior.

  • Misc/WebKitLogging.h:
  • Misc/WebKitLogging.m: (ReportDiscardedDelegateException):
  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView loadPluginRequest:]):
  • Plugins/WebNullPluginView.mm: (-[WebNullPluginView viewDidMoveToWindow]):
  • WebCoreSupport/WebChromeClient.mm: (WebChromeClient::createWindow): (WebChromeClient::createModalDialog): (WebChromeClient::runModal): (WebChromeClient::toolbarsVisible): (WebChromeClient::statusbarVisible): (WebChromeClient::addMessageToConsole): (WebChromeClient::canRunBeforeUnloadConfirmPanel): (WebChromeClient::runBeforeUnloadConfirmPanel): (WebChromeClient::runJavaScriptAlert): (WebChromeClient::runJavaScriptConfirm): (WebChromeClient::runJavaScriptPrompt): (WebChromeClient::shouldInterruptJavaScript): (WebChromeClient::setStatusbarText): (WebChromeClient::print):
  • WebCoreSupport/WebContextMenuClient.mm: (WebContextMenuClient::getCustomMenuFromDefaultItems): (WebContextMenuClient::contextMenuItemSelected):
  • WebCoreSupport/WebDragClient.mm: (WebDragClient::startDrag):
  • WebCoreSupport/WebEditorClient.mm: (WebEditorClient::textFieldDidBeginEditing): (WebEditorClient::textFieldDidEndEditing): (WebEditorClient::textDidChangeInTextField): (WebEditorClient::doTextFieldCommandFromEvent): (WebEditorClient::textWillBeDeletedInTextField): (WebEditorClient::textDidChangeInTextArea):
  • WebCoreSupport/WebFrameBridge.mm: (-[WebFrameBridge viewForPluginWithFrame:URL:attributeNames:attributeValues:MIMEType:DOMElement:loadManually:]):
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::dispatchDidLoadResourceFromMemoryCache): (WebFrameLoaderClient::assignIdentifierToInitialRequest): (WebFrameLoaderClient::dispatchWillSendRequest): (WebFrameLoaderClient::dispatchDidReceiveAuthenticationChallenge): (WebFrameLoaderClient::dispatchDidCancelAuthenticationChallenge): (WebFrameLoaderClient::dispatchDidReceiveResponse): (WebFrameLoaderClient::willCacheResponse): (WebFrameLoaderClient::dispatchDidReceiveContentLength): (WebFrameLoaderClient::dispatchDidFinishLoading): (WebFrameLoaderClient::dispatchDidFailLoading): (WebFrameLoaderClient::dispatchDidHandleOnloadEvents): (WebFrameLoaderClient::dispatchDidReceiveServerRedirectForProvisionalLoad): (WebFrameLoaderClient::dispatchDidCancelClientRedirect): (WebFrameLoaderClient::dispatchWillPerformClientRedirect): (WebFrameLoaderClient::dispatchDidChangeLocationWithinPage): (WebFrameLoaderClient::dispatchWillClose): (WebFrameLoaderClient::dispatchDidReceiveIcon): (WebFrameLoaderClient::dispatchDidStartProvisionalLoad): (WebFrameLoaderClient::dispatchDidReceiveTitle): (WebFrameLoaderClient::dispatchDidCommitLoad): (WebFrameLoaderClient::dispatchDidFailProvisionalLoad): (WebFrameLoaderClient::dispatchDidFailLoad): (WebFrameLoaderClient::dispatchDidFinishDocumentLoad): (WebFrameLoaderClient::dispatchDidFinishLoad): (WebFrameLoaderClient::dispatchDidFirstLayout): (WebFrameLoaderClient::dispatchCreatePage): (WebFrameLoaderClient::dispatchUnableToImplementPolicy): (WebFrameLoaderClient::dispatchWillSubmitForm): (WebFrameLoaderClient::dispatchDidLoadMainResource):
  • WebView/WebHTMLView.mm: (-[WebHTMLView callDelegateDoCommandBySelectorIfNeeded:]): (-[WebHTMLView validateUserInterfaceItem:]):
  • WebView/WebPDFView.mm: (-[WebPDFView validateUserInterfaceItem:]): (-[WebPDFView PDFViewSavePDFToDownloadFolder:]):
  • WebView/WebView.mm: (-[WebView _openNewWindowWithRequest:]): (-[WebView _menuForElement:defaultItems:]): (-[WebView _mouseDidMoveOverElement:modifierFlags:]): (-[WebView _cacheResourceLoadDelegateImplementations]): (-[WebView _cacheFrameLoadDelegateImplementations]): (-[WebView _policyDelegateForwarder]): (-[WebView _UIDelegateForwarder]): (-[WebView _editingDelegateForwarder]): (-[WebView _scriptDebugDelegateForwarder]): (-[WebView _setCatchesDelegateExceptions:]): (-[WebView _catchesDelegateExceptions]): (-[_WebSafeForwarder initWithTarget:defaultTarget:]): (-[_WebSafeForwarder forwardInvocation:]): (-[_WebSafeForwarder methodSignatureForSelector:]): (-[WebView _commonInitializationWithFrameName:groupName:]): (-[WebView validateUserInterfaceItem:]): (-[WebView _headerHeight]): (-[WebView _footerHeight]): (-[WebView _drawHeaderInRect:]): (-[WebView _drawFooterInRect:]): (-[WebView _shouldChangeSelectedDOMRange:toDOMRange:affinity:stillSelecting:]): (CallDelegate): (CallDelegateReturningFloat): (CallDelegateReturningBoolean): (CallUIDelegate): (CallUIDelegateReturningFloat): (CallUIDelegateReturningBoolean): (CallFrameLoadDelegate): (CallResourceLoadDelegate): (CallFormDelegate): (CallFormDelegateReturningBoolean):
  • WebView/WebViewInternal.h:
  • WebView/WebViewPrivate.h:
12:39 PM Changeset in webkit [25395] by darin
  • 5 edits in trunk/WebCore

Reviewed by Hyatt.

  • <rdar://problem/5457865> REGRESSION (9A527): Safari crashes when opening a page that immediately redirects to a PDF

I don't know how to make an automated test that trips over this.

The immediate cause of this regression was making a back/forward list entry in
this case. Earlier, the quick redirect would not results in a separate entry.
That's possibly a bug too, but it's better to fix the crash first.

The page cache was putting the document into a strange state: Still attached but
with the renderer set to 0. There was no good reason to do this, so got rid of it.
Moved the responsibility to the caller of not calling detach() when moving into
the page cache. This is more of a frame loader thing than a document thing.

  • dom/Document.cpp: (WebCore::Document::detach): Added assertions that this is only called on a document that's attached and not in the page cache. Also moved the call to willRemove in here, so that callers can't make the mistake of not calling that function. Removed the incorrectly-positioned code that made this function do less if it was called on a document in the page cache.
  • history/CachedPage.cpp: (WebCore::CachedPage::clear): Removed the code to handle a document with a renderer of 0. There was no need to put the document into this state. Any document in the page cache will always be "attached".
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::clear): Added a check to prevent from calling an unnecessary cancelParsing() on a document that's in the page cache and guard the call to detach() with a check of attached(), like all other calls to detach() on DOM objects.
  • page/Frame.cpp: (WebCore::Frame::setView): Added a missing check of attached(), like all other calls to detach() on DOM objects. Also added code to not call detach() on the document when it's in the page cache. (WebCore::Frame::setDocument): This call site already had the attached() check, but was missing the page cache check.
11:48 AM Changeset in webkit [25394] by kmccullo
  • 14 edits
    1 copy
    3 adds in trunk

JavaScriptCore:

Reviewed by Adam, Sam, Darin.

  • Created a JSRetainPtr specifically for JSStringRefs so they can be automatically refed and derefed.
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • kjs/JSRetainPtr.h: Copied from wtf/RetainPtr.h. (KJS::JSRetain): (KJS::JSRelease): (KJS::): (KJS::JSRetainPtr::JSRetainPtr): (KJS::JSRetainPtr::~JSRetainPtr): (KJS::JSRetainPtr::get): (KJS::JSRetainPtr::releaseRef): (KJS::JSRetainPtr::operator->): (KJS::JSRetainPtr::operator UnspecifiedBoolType): (KJS::::operator): (KJS::::adopt): (KJS::::swap): (KJS::swap): (KJS::operator==): (KJS::operator!=):

WebKitTools:

Reviewed by Adam, Sam, Darin.

  • Separated what is platform dependant from what is not. Creating the structure needed for Drosera for Win.
  • Drosera/Debugger.h: Added.
  • Drosera/DebuggerDocument.cpp: (DebuggerDocument::breakpointEditorHTMLCallback): (DebuggerDocument::isPausedCallback): (DebuggerDocument::pauseCallback): (DebuggerDocument::resumeCallback): (DebuggerDocument::stepIntoCallback): (DebuggerDocument::evaluateScriptCallback): (DebuggerDocument::currentFunctionStackCallback): (DebuggerDocument::localScopeVariableNamesForCallFrameCallback): (DebuggerDocument::valueForScopeVariableNamedCallback): (DebuggerDocument::logCallback): (DebuggerDocument::breakpointEditorHTML): (DebuggerDocument::isPaused): (DebuggerDocument::pause): (DebuggerDocument::resume): (DebuggerDocument::stepInto): (DebuggerDocument::evaluateScript): (DebuggerDocument::currentFunctionStack): (DebuggerDocument::localScopeVariableNamesForCallFrame): (DebuggerDocument::valueForScopeVariableNamed): (DebuggerDocument::log): (DebuggerDocument::toolbarPause): (DebuggerDocument::toolbarResume): (DebuggerDocument::toolbarStepInto): (DebuggerDocument::toolbarStepOver): (DebuggerDocument::toolbarStepOut): (DebuggerDocument::toolbarShowConsole): (DebuggerDocument::toolbarCloseCurrentFile): (DebuggerDocument::updateFileSource): (DebuggerDocument::didParseScript): (DebuggerDocument::willExecuteStatement): (DebuggerDocument::didEnterCallFrame): (DebuggerDocument::willLeaveCallFrame): (DebuggerDocument::exceptionWasRaised): (DebuggerDocument::windowScriptObjectAvailable): (DebuggerDocument::toJSArray): (DebuggerDocument::callGlobalFunction): (DebuggerDocument::callFunctionOnObject): (DebuggerDocument::getDroseraJSClass): (DebuggerDocument::staticFunctions): (DebuggerDocument::logException):
  • Drosera/DebuggerDocument.h: (DebuggerDocument::DebuggerDocument):
  • Drosera/ForwardingHeaders: Added.
  • Drosera/ForwardingHeaders/wtf: Added.
  • Drosera/ForwardingHeaders/wtf/Platform.h: Added.
  • Drosera/config.h:
  • Drosera/console.html:
  • Drosera/console.js:
  • Drosera/debugger.js:
  • Drosera/mac/DebuggerApplication.mm: (-[DebuggerApplication attach:]):
  • Drosera/mac/DebuggerDocumentMac.h:
  • Drosera/mac/DebuggerDocumentMac.mm: (+[NSString stringOrNilFromWebScriptResult:]): (+[DebuggerClientMac log:]): (-[DebuggerClientMac initWithServerName:]): (-[DebuggerClientMac dealloc]): (-[DebuggerClientMac pause]): (-[DebuggerClientMac resume]): (-[DebuggerClientMac pause:]): (-[DebuggerClientMac resume:]): (-[DebuggerClientMac stepInto:]): (-[DebuggerClientMac stepOver:]): (-[DebuggerClientMac stepOut:]): (-[DebuggerClientMac showConsole:]): (-[DebuggerClientMac closeCurrentFile:]): (-[DebuggerClientMac validateUserInterfaceItem:]): (-[DebuggerClientMac webView:windowScriptObjectAvailable:]): (-[DebuggerClientMac webView:didEnterCallFrame:sourceId:line:forWebFrame:]): (-[DebuggerClientMac webView:willExecuteStatement:sourceId:line:forWebFrame:]): (-[DebuggerClientMac webView:willLeaveCallFrame:sourceId:line:forWebFrame:]): (-[DebuggerClientMac webView:exceptionWasRaised:sourceId:line:forWebFrame:]): (DebuggerDocument::platformPause): (DebuggerDocument::platformResume): (DebuggerDocument::platformStepInto): (DebuggerDocument::platformEvaluateScript): (DebuggerDocument::platformCurrentFunctionStack): (DebuggerDocument::platformLocalScopeVariableNamesForCallFrame): (DebuggerDocument::platformValueForScopeVariableNamed): (DebuggerDocument::platformLog):
  • Drosera/mac/Drosera.xcodeproj/project.pbxproj:
11:42 AM Changeset in webkit [25393] by oliver
  • 1 edit in trunk/LayoutTests/ChangeLog

Appease Sam[e]

11:42 AM Changeset in webkit [25392] by oliver
  • 1 edit
    2 moves
    3 adds in trunk/LayoutTests

Reviewed by Same and Darin


Split the wrapper test into seperate Obj-C and JS tests so the we can use the JS tests on other platforms.

  • fast/dom/resources/wrapper-identity-base.js: Added -- This provides the basic utility functions and variables needed for the wrapper tests
  • fast/dom/wrapper-identity-expected.txt: Added.
  • fast/dom/wrapper-identity.html: Added.
  • platform/mac/fast/dom/objc-wrapper-identity-expected.txt: Copied from platform/mac/fast/dom/wrapper-identity-expected.txt.
  • platform/mac/fast/dom/objc-wrapper-identity.html: Copied from platform/mac/fast/dom/wrapper-identity.html.
  • platform/mac/fast/dom/wrapper-identity-expected.txt: Removed.
  • platform/mac/fast/dom/wrapper-identity.html: Removed.
11:11 AM Changeset in webkit [25391] by weinig
  • 2 edits in trunk/LayoutTests

Reviewed by Adam Roben.

Add fast/frames/onload-remove-iframe-crash.html to the windows skiplist becuase it currently crashes.

  • platform/win/Skipped:
11:07 AM Changeset in webkit [25390] by weinig
  • 2 edits in trunk/WebKitTools

Reviewed by Adam Roben.

Register the WebKit DLL on initialization of the DumpRenderTree.

  • DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.cpp: (initialize):
10:05 AM Changeset in webkit [25389] by staikos
  • 2 edits in trunk/WebKitQt

Make popup windows work again

9:56 AM Changeset in webkit [25388] by harrison
  • 1 edit in trunk/WebCore/editing/TextIterator.cpp

Check in source change now that I committed the changelog!

9:55 AM Changeset in webkit [25387] by harrison
  • 1 edit in trunk/WebCore/ChangeLog

Reviewed by Darin.

Follow up on my fix for <rdar://problem/5306171>.
My first patch (r25373) broke svg/custom/stroke-width-click.svg.

  • editing/TextIterator.cpp: (WebCore::TextIterator::shouldRepresentNodeOffsetZero): Add currPos.isNotNull() check because positions in non-html content (like svg) do not have visible positions, and we don't want to emit for them either.
9:55 AM BuildingQtOnLinux edited by technomancy@gmail.com
libicu36-dev dependency (diff)
5:20 AM Changeset in webkit [25386] by bdash
  • 6 edits in trunk/WebCore

2007-09-06 Mitz Pettel <​mitz@webkit.org>

Reviewed by Adam Roben.

  • platform/PlatformString.h: (WebCore::String::defaultWritingDirection):
  • platform/StringImpl.cpp: (WebCore::StringImpl::defaultWritingDirection): Moved textDirectionForParagraph() from RenderMenuList.cpp here.
  • platform/StringImpl.h:
  • platform/win/PopupMenuWin.cpp: (WebCore::PopupMenu::itemWritingDirectionIsNatural): Changed to return 'true'. (WebCore::PopupMenu::paint): Changed to use natural directionality for menu items.
  • rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::adjustInnerStyle): Use defaultWritingDirection.
3:31 AM Changeset in webkit [25385] by mjs
  • 1 edit
    1 add in trunk/WebKitSite

Forgot the JS file.

  • misc/WebKitDetect.js: Added.
3:09 AM Changeset in webkit [25384] by mjs
  • 1 edit
    1 add in trunk/WebKitSite

Not reviewed - copy of WebKitDetect script from wiki

  • misc/WebKitDetect.html: Added.
2:25 AM Changeset in webkit [25383] by mjs
  • 2 edits in trunk/WebCore

Reviewed by Oliver Hunt.

  • fixed <rdar://problem/5455457> REGRESSION (9A527-9A535): Colloquy crash on launch in WebCoreScriptDebugger initWithDelegate (with announce.js plug-in installed)
  • page/mac/FrameMac.mm: (WebCore::Frame::windowScriptObject): Ensure that -[WebView windowScriptObject] and -[WebFrame windowObject] return null until the windowScriptObjectAvailable: delegate method is sent.

Sep 5, 2007:

9:34 PM Changeset in webkit [25382] by oliver
  • 2 edits
    42 copies in trunk/LayoutTests

Reviewed by Stephanie.

Moved a number of tests from platform/mac that are not currently cross-platform,
but should eventually be updated to work across all platforms.

  • editing/deleting/5300379-expected.checksum: Copied from platform/mac/editing/deleting/5300379-expected.checksum.
  • editing/deleting/5300379-expected.png: Copied from platform/mac/editing/deleting/5300379-expected.png.
  • editing/deleting/5300379-expected.txt: Copied from platform/mac/editing/deleting/5300379-expected.txt.
  • editing/deleting/5300379.html: Copied from platform/mac/editing/deleting/5300379.html.
  • editing/deleting/delete-to-end-of-paragraph-expected.checksum: Copied from platform/mac/editing/deleting/delete-to-end-of-paragraph-expected.checksum.
  • editing/deleting/delete-to-end-of-paragraph-expected.png: Copied from platform/mac/editing/deleting/delete-to-end-of-paragraph-expected.png.
  • editing/deleting/delete-to-end-of-paragraph-expected.txt: Copied from platform/mac/editing/deleting/delete-to-end-of-paragraph-expected.txt.
  • editing/deleting/delete-to-end-of-paragraph.html: Copied from platform/mac/editing/deleting/delete-to-end-of-paragraph.html.
  • editing/deleting/smart-delete-003-expected.checksum: Copied from platform/mac/editing/deleting/smart-delete-003-expected.checksum.
  • editing/deleting/smart-delete-003-expected.png: Copied from platform/mac/editing/deleting/smart-delete-003-expected.png.
  • editing/deleting/smart-delete-003-expected.txt: Copied from platform/mac/editing/deleting/smart-delete-003-expected.txt.
  • editing/deleting/smart-delete-003.html: Copied from platform/mac/editing/deleting/smart-delete-003.html.
  • editing/deleting/smart-delete-004-expected.checksum: Copied from platform/mac/editing/deleting/smart-delete-004-expected.checksum.
  • editing/deleting/smart-delete-004-expected.png: Copied from platform/mac/editing/deleting/smart-delete-004-expected.png.
  • editing/deleting/smart-delete-004-expected.txt: Copied from platform/mac/editing/deleting/smart-delete-004-expected.txt.
  • editing/deleting/smart-delete-004.html: Copied from platform/mac/editing/deleting/smart-delete-004.html.
  • editing/execCommand/nsresponder-indent-expected.checksum: Copied from platform/mac/editing/execCommand/nsresponder-indent-expected.checksum.
  • editing/execCommand/nsresponder-indent-expected.png: Copied from platform/mac/editing/execCommand/nsresponder-indent-expected.png.
  • editing/execCommand/nsresponder-indent-expected.txt: Copied from platform/mac/editing/execCommand/nsresponder-indent-expected.txt.
  • editing/execCommand/nsresponder-indent.html: Copied from platform/mac/editing/execCommand/nsresponder-indent.html.
  • editing/execCommand/nsresponder-outdent-expected.checksum: Copied from platform/mac/editing/execCommand/nsresponder-outdent-expected.checksum.
  • editing/execCommand/nsresponder-outdent-expected.png: Copied from platform/mac/editing/execCommand/nsresponder-outdent-expected.png.
  • editing/execCommand/nsresponder-outdent-expected.txt: Copied from platform/mac/editing/execCommand/nsresponder-outdent-expected.txt.
  • editing/execCommand/nsresponder-outdent.html: Copied from platform/mac/editing/execCommand/nsresponder-outdent.html.
  • editing/selection/4947387-expected.checksum: Copied from platform/mac/editing/selection/4947387-expected.checksum.
  • editing/selection/4947387-expected.png: Copied from platform/mac/editing/selection/4947387-expected.png.
  • editing/selection/4947387-expected.txt: Copied from platform/mac/editing/selection/4947387-expected.txt.
  • editing/selection/4947387.html: Copied from platform/mac/editing/selection/4947387.html.
  • editing/selection/5195166-1-expected.checksum: Copied from platform/mac/editing/selection/5195166-1-expected.checksum.
  • editing/selection/5195166-1-expected.png: Copied from platform/mac/editing/selection/5195166-1-expected.png.
  • editing/selection/5195166-1-expected.txt: Copied from platform/mac/editing/selection/5195166-1-expected.txt.
  • editing/selection/5195166-1.html: Copied from platform/mac/editing/selection/5195166-1.html.
  • editing/selection/5195166-2-expected.checksum: Copied from platform/mac/editing/selection/5195166-2-expected.checksum.
  • editing/selection/5195166-2-expected.png: Copied from platform/mac/editing/selection/5195166-2-expected.png.
  • editing/selection/5195166-2-expected.txt: Copied from platform/mac/editing/selection/5195166-2-expected.txt.
  • editing/selection/5195166-2.html: Copied from platform/mac/editing/selection/5195166-2.html.
  • editing/selection/select-line-expected.txt: Copied from platform/mac/editing/selection/select-line-expected.txt.
  • editing/selection/select-line.html: Copied from platform/mac/editing/selection/select-line.html.
  • editing/selection/selection-actions-expected.checksum: Copied from platform/mac/editing/selection/selection-actions-expected.checksum.
  • editing/selection/selection-actions-expected.png: Copied from platform/mac/editing/selection/selection-actions-expected.png.
  • editing/selection/selection-actions-expected.txt: Copied from platform/mac/editing/selection/selection-actions-expected.txt.
  • editing/selection/selection-actions.html: Copied from platform/mac/editing/selection/selection-actions.html.
  • platform/win/Skipped:
6:47 PM Changeset in webkit [25381] by harrison
  • 5 edits in trunk/LayoutTests

Updated results for:
<rdar://problem/5306171> Mail: The first return after an attachment in multipart/mixed message is lost

  • http/tests/security/dataURL/xss-DENIED-from-data-url-to-data-url-expected.txt:
  • http/tests/security/javascriptURL/xss-ALLOWED-from-javascript-url-sub-frame-2-level-expected.txt:
  • http/tests/security/javascriptURL/xss-ALLOWED-from-javascript-url-to-javscript-url-expected.txt:
  • http/tests/security/javascriptURL/xss-ALLOWED-to-javascript-url-sub-frame-2-level-expected.txt:
5:32 PM Changeset in webkit [25380] by weinig
  • 3 edits in trunk/LayoutTests

Reviewed by Kevin McCullough.

Use correct part of the test for cross-platform testing.

  • plugins/bindings-test-expected.txt:
  • plugins/bindings-test.html:
4:57 PM Changeset in webkit [25379] by weinig
  • 4 edits
    1 copy
    1 add in trunk/LayoutTests

Reviewed by Geoff Garen.

Split out mac specific tests from plugins/bindings-test.html into its own test.

  • platform/mac/plugins/bindings-test-objc-expected.txt: Added.
  • platform/mac/plugins/bindings-test-objc.html: Copied from plugins/bindings-test.html.
  • platform/win/Skipped:
  • plugins/bindings-test-expected.txt:
  • plugins/bindings-test.html:
4:39 PM Changeset in webkit [25378] by oliver
  • 2 edits in trunk/LayoutTests

Reviewed by Adam and Niko.

Force run-webkit-tests to not run the tests in platform/mac

  • platform/win/Skipped:
4:02 PM Changeset in webkit [25377] by hyatt
  • 3 edits in trunk/WebKit/win

Fix ALT+key combos so that they go into the DOM.

3:39 PM Changeset in webkit [25376] by weinig
  • 4 edits in trunk/LayoutTests

Reviewed by Geoff Garen.

Don't log DumpRenderTree injected properties in fast/dom/Window/window-properties.html. This change makes
it pass on Windows so it can be removed from the Skipped list.

  • fast/dom/Window/window-properties-expected.txt:
  • fast/dom/Window/window-properties.html:
  • platform/win/Skipped:
3:23 PM Changeset in webkit [25375] by zimmermann
  • 18 edits
    4 adds
    1 delete in branches/feature-branch/LayoutTests

Reviewed by Anders.

Fix paths to css style sheets in the last added files. Enable smallFonts.svg as it doesn't crash anymore.
Fix paths to external <use> element reference, to make Batik happy. (We don't support it, yet.)

1:58 PM Changeset in webkit [25374] by weinig
  • 2 edits in trunk/WebKitTools

Reviewed by Oliver.

Fix many layout test failures caused by r25364.
Set text size to standand size at the begining of each test matching the mac.

  • DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.cpp: (runTest):
1:46 PM Changeset in webkit [25373] by harrison
  • 3 edits
    2 adds in trunk

WebCore:

Reviewed by Kevin Decker.

<rdar://problem/5306171> Mail: The first return after an attachment in multipart/mixed message is lost

Test added: editing/selection/toString-1.html


Source changes:

  • editing/TextIterator.cpp: (WebCore::TextIterator::shouldRepresentNodeOffsetZero):
  • Remove outdated and erroneous check for m_lastTextNode.
  • Fix VisiblePosition check to look for line diff. Simple position diff gives the wrong answer when the start is table/0 and the current is the first position inside the content of the table.

LayoutTests:

Reviewed by Kevin Decker.

Testcase for:
<rdar://problem/5306171> Mail: The first return after an attachment in multipart/mixed message is lost

  • editing/selection/toString-1-expected.txt: Added.
  • editing/selection/toString-1.html: Added.
1:36 PM webkit_xl.zip attached to DetectingWebKit by timothy@apple.com
Updated to detect mobile Safari.
1:34 PM webkit_normal.zip attached to DetectingWebKit by timothy@apple.com
Updated to detect mobile Safari.
1:25 PM Changeset in webkit [25372] by zimmermann
  • 3 edits
    5 adds in branches/feature-branch

Reviewed by Oliver.
Fix <mask> in userSpaceOnUse coordinates. Verified by new testcase taken from Batik.

11:45 AM Changeset in webkit [25371] by zimmermann
  • 20 edits in branches/feature-branch/WebCore

(Oops, forgot to commit WebCore/ changes!)
Reviewed by Oliver.

Rewrite text painting code path, to be based on "text chunks" instead of using the InlineBox* hierarchy.
This allows us to reuse paint servers as long as possible - and to actually paint in the way didacted by the spec.

This also implements advanced SVG text decorations (ie. stroked/filled decos with gradient/pattern).
Fixes: ​http://bugs.webkit.org/show_bug.cgi?id=7200 (Gradient appears and disappears in SVG when text is off screen)

Fixes paint server usage on text paths. (ie. styled text paths, with gradients/filters).

11:19 AM Changeset in webkit [25370] by zimmermann
  • 175 edits
    13 adds in branches/feature-branch/LayoutTests

Reviewed by Oliver.

Rewrite text painting code path, to be based on "text chunks" instead of using the InlineBox* hierarchy.
This allows us to reuse paint servers as long as possible - and to actually paint in the way didacted by the spec.

This also implements advanced SVG text decorations (ie. stroked/filled decos with gradient/pattern).
Fixes: ​http://bugs.webkit.org/show_bug.cgi?id=7200 (Gradient appears and disappears in SVG when text is off screen)

Fixes paint server usage on text paths. (ie. styled text paths, with gradients/filters).

7:24 AM Changeset in webkit [25369] by bdash
  • 2 edits in trunk/JavaScriptCore

2007-09-05 Mark Rowe <​mrowe@apple.com>

Unreviewed Qt build fix.

  • wtf/unicode/qt4/UnicodeQt4.h: Fix the constness of the src argument to toUpper to prevent build failures.

Sep 4, 2007:

11:10 PM Changeset in webkit [25368] by bdash
  • 9 edits in trunk/WebCore

2007-09-04 Marvin Decker <​marv.decker@gmail.com>

Reviewed by Maciej Stachowiak.

​http://bugs.webkit.org/show_bug.cgi?id=15072
Bug 15072: Fix ImageDecoder.cpp to not copy incoming data

Change ImageDecoder::setData to take a SharedBuffer*.

  • platform/graphics/cairo/ImageSourceCairo.cpp: (WebCore::ImageSource::setData):
  • platform/image-decoders/ImageDecoder.h: (WebCore::RGBA32Buffer::bytes): (WebCore::RGBA32Buffer::height): (WebCore::ImageDecoder::setData):
  • platform/image-decoders/gif/GIFImageDecoder.cpp: (WebCore::GIFImageDecoder::setData): (WebCore::GIFImageDecoder::frameCount): (WebCore::GIFImageDecoder::decode): (WebCore::GIFImageDecoder::decodingHalted): (WebCore::GIFImageDecoder::initFrameBuffer):
  • platform/image-decoders/gif/GIFImageDecoder.h:
  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp: (WebCore::JPEGImageDecoder::setData): (WebCore::JPEGImageDecoder::decode):
  • platform/image-decoders/jpeg/JPEGImageDecoder.h:
  • platform/image-decoders/png/PNGImageDecoder.cpp: (WebCore::PNGImageDecoder::setData): (WebCore::PNGImageDecoder::decode): (WebCore::PNGImageDecoder::headerAvailable): (WebCore::PNGImageDecoder::rowAvailable):
  • platform/image-decoders/png/PNGImageDecoder.h:
10:16 PM Changeset in webkit [25367] by oliver
  • 2 edits
    76 moves
    6 adds
    3 deletes in trunk/LayoutTests

RS=Adam


Moving a number of mac specific tests into LayoutTests/platform/mac so that they
don't need to be in the windows Skiplist.


This includes most of the tests that used textInputController as it is a test
of the mac-specific NSTextInput API.

  • editing/deleting/5300379-expected.checksum: Removed.
  • editing/deleting/5300379-expected.png: Removed.
  • editing/deleting/5300379-expected.txt: Removed.
  • editing/deleting/5300379.html: Removed.
  • editing/deleting/delete-to-end-of-paragraph-expected.checksum: Removed.
  • editing/deleting/delete-to-end-of-paragraph-expected.png: Removed.
  • editing/deleting/delete-to-end-of-paragraph-expected.txt: Removed.
  • editing/deleting/delete-to-end-of-paragraph.html: Removed.
  • editing/deleting/smart-delete-003-expected.checksum: Removed.
  • editing/deleting/smart-delete-003-expected.png: Removed.
  • editing/deleting/smart-delete-003-expected.txt: Removed.
  • editing/deleting/smart-delete-003.html: Removed.
  • editing/deleting/smart-delete-004-expected.checksum: Removed.
  • editing/deleting/smart-delete-004-expected.png: Removed.
  • editing/deleting/smart-delete-004-expected.txt: Removed.
  • editing/deleting/smart-delete-004.html: Removed.
  • editing/execCommand/nsresponder-indent-expected.checksum: Removed.
  • editing/execCommand/nsresponder-indent-expected.png: Removed.
  • editing/execCommand/nsresponder-indent-expected.txt: Removed.
  • editing/execCommand/nsresponder-indent.html: Removed.
  • editing/execCommand/nsresponder-outdent-expected.checksum: Removed.
  • editing/execCommand/nsresponder-outdent-expected.png: Removed.
  • editing/execCommand/nsresponder-outdent-expected.txt: Removed.
  • editing/execCommand/nsresponder-outdent.html: Removed.
  • editing/input/firstrectforcharacterrange-plain-expected.txt: Removed.
  • editing/input/firstrectforcharacterrange-plain.html: Removed.
  • editing/input/firstrectforcharacterrange-styled-expected.txt: Removed.
  • editing/input/firstrectforcharacterrange-styled.html: Removed.
  • editing/input/mac: Removed.
  • editing/input/mac/hangul-enter-confirms-and-sends-keypress-expected.txt: Removed.
  • editing/input/mac/hangul-enter-confirms-and-sends-keypress.html: Removed.
  • editing/input/mac/hangul.js: Removed.
  • editing/input/mac/kotoeri-enter-to-confirm-and-newline-expected.txt: Removed.
  • editing/input/mac/kotoeri-enter-to-confirm-and-newline.html: Removed.
  • editing/input/mac/kotoeri.js: Removed.
  • editing/input/mac/logger.js: Removed.
  • editing/input/range-for-empty-document-expected.txt: Removed.
  • editing/input/range-for-empty-document.html: Removed.
  • editing/input/replace-invalid-range-expected.txt: Removed.
  • editing/input/replace-invalid-range.html: Removed.
  • editing/input/text-input-controller-expected.txt: Removed.
  • editing/input/text-input-controller.html: Removed.
  • editing/input/wrapped-line-char-rect-expected.txt: Removed.
  • editing/input/wrapped-line-char-rect.html: Removed.
  • editing/selection/4947387-expected.checksum: Removed.
  • editing/selection/4947387-expected.png: Removed.
  • editing/selection/4947387-expected.txt: Removed.
  • editing/selection/4947387.html: Removed.
  • editing/selection/5195166-1-expected.checksum: Removed.
  • editing/selection/5195166-1-expected.png: Removed.
  • editing/selection/5195166-1-expected.txt: Removed.
  • editing/selection/5195166-1.html: Removed.
  • editing/selection/5195166-2-expected.checksum: Removed.
  • editing/selection/5195166-2-expected.png: Removed.
  • editing/selection/5195166-2-expected.txt: Removed.
  • editing/selection/5195166-2.html: Removed.
  • editing/selection/select-line-expected.txt: Removed.
  • editing/selection/select-line.html: Removed.
  • editing/selection/selection-actions-expected.checksum: Removed.
  • editing/selection/selection-actions-expected.png: Removed.
  • editing/selection/selection-actions-expected.txt: Removed.
  • editing/selection/selection-actions.html: Removed.
  • fast/AppleScript: Removed.
  • fast/AppleScript/001-expected.txt: Removed.
  • fast/AppleScript/001.html: Removed.
  • fast/AppleScript/array-expected.txt: Removed.
  • fast/AppleScript/array.html: Removed.
  • fast/AppleScript/date-expected.txt: Removed.
  • fast/AppleScript/date.html: Removed.
  • fast/dom/character-index-for-point-expected.txt: Removed.
  • fast/dom/character-index-for-point.html: Removed.
  • fast/dom/wrapper-identity-expected.txt: Removed.
  • fast/dom/wrapper-identity.html: Removed.
  • fast/dom/wrapper-round-tripping-expected.txt: Removed.
  • fast/dom/wrapper-round-tripping.html: Removed.
  • fast/encoding/char-decoding-mac-expected.txt: Removed.
  • fast/encoding/char-decoding-mac.html: Removed.
  • fast/forms/attributed-strings-expected.txt: Removed.
  • fast/forms/attributed-strings.html: Removed.
  • fast/objc: Removed.
  • fast/objc/longlongTest-expected.txt: Removed.
  • fast/objc/longlongTest.html: Removed.
  • fast/text/attributed-substring-from-range-001-expected.txt: Removed.
  • fast/text/attributed-substring-from-range-001.html: Removed.
  • fast/text/attributed-substring-from-range-expected.txt: Removed.
  • fast/text/attributed-substring-from-range-in-textfield-expected.txt: Removed.
  • fast/text/attributed-substring-from-range-in-textfield.html: Removed.
  • fast/text/attributed-substring-from-range.html: Removed.
  • fast/text/justified-text-rect-expected.checksum: Removed.
  • fast/text/justified-text-rect-expected.png: Removed.
  • fast/text/justified-text-rect-expected.txt: Removed.
  • fast/text/justified-text-rect.html: Removed.
  • platform/mac/editing/deleting/5300379-expected.checksum: Copied from editing/deleting/5300379-expected.checksum.
  • platform/mac/editing/deleting/5300379-expected.png: Copied from editing/deleting/5300379-expected.png.
  • platform/mac/editing/deleting/5300379-expected.txt: Copied from editing/deleting/5300379-expected.txt.
  • platform/mac/editing/deleting/5300379.html: Copied from editing/deleting/5300379.html.
  • platform/mac/editing/deleting/delete-to-end-of-paragraph-expected.checksum: Copied from editing/deleting/delete-to-end-of-paragraph-expected.checksum.
  • platform/mac/editing/deleting/delete-to-end-of-paragraph-expected.png: Copied from editing/deleting/delete-to-end-of-paragraph-expected.png.
  • platform/mac/editing/deleting/delete-to-end-of-paragraph-expected.txt: Copied from editing/deleting/delete-to-end-of-paragraph-expected.txt.
  • platform/mac/editing/deleting/delete-to-end-of-paragraph.html: Copied from editing/deleting/delete-to-end-of-paragraph.html.
  • platform/mac/editing/deleting/smart-delete-003-expected.checksum: Copied from editing/deleting/smart-delete-003-expected.checksum.
  • platform/mac/editing/deleting/smart-delete-003-expected.png: Copied from editing/deleting/smart-delete-003-expected.png.
  • platform/mac/editing/deleting/smart-delete-003-expected.txt: Copied from editing/deleting/smart-delete-003-expected.txt.
  • platform/mac/editing/deleting/smart-delete-003.html: Copied from editing/deleting/smart-delete-003.html.
  • platform/mac/editing/deleting/smart-delete-004-expected.checksum: Copied from editing/deleting/smart-delete-004-expected.checksum.
  • platform/mac/editing/deleting/smart-delete-004-expected.png: Copied from editing/deleting/smart-delete-004-expected.png.
  • platform/mac/editing/deleting/smart-delete-004-expected.txt: Copied from editing/deleting/smart-delete-004-expected.txt.
  • platform/mac/editing/deleting/smart-delete-004.html: Copied from editing/deleting/smart-delete-004.html.
  • platform/mac/editing/execCommand/nsresponder-indent-expected.checksum: Copied from editing/execCommand/nsresponder-indent-expected.checksum.
  • platform/mac/editing/execCommand/nsresponder-indent-expected.png: Copied from editing/execCommand/nsresponder-indent-expected.png.
  • platform/mac/editing/execCommand/nsresponder-indent-expected.txt: Copied from editing/execCommand/nsresponder-indent-expected.txt.
  • platform/mac/editing/execCommand/nsresponder-indent.html: Copied from editing/execCommand/nsresponder-indent.html.
  • platform/mac/editing/execCommand/nsresponder-outdent-expected.checksum: Copied from editing/execCommand/nsresponder-outdent-expected.checksum.
  • platform/mac/editing/execCommand/nsresponder-outdent-expected.png: Copied from editing/execCommand/nsresponder-outdent-expected.png.
  • platform/mac/editing/execCommand/nsresponder-outdent-expected.txt: Copied from editing/execCommand/nsresponder-outdent-expected.txt.
  • platform/mac/editing/execCommand/nsresponder-outdent.html: Copied from editing/execCommand/nsresponder-outdent.html.
  • platform/mac/editing/input: Copied from editing/input/mac.
  • platform/mac/editing/input/firstrectforcharacterrange-plain-expected.txt: Copied from editing/input/firstrectforcharacterrange-plain-expected.txt.
  • platform/mac/editing/input/firstrectforcharacterrange-plain.html: Copied from editing/input/firstrectforcharacterrange-plain.html.
  • platform/mac/editing/input/firstrectforcharacterrange-styled-expected.txt: Copied from editing/input/firstrectforcharacterrange-styled-expected.txt.
  • platform/mac/editing/input/firstrectforcharacterrange-styled.html: Copied from editing/input/firstrectforcharacterrange-styled.html.
  • platform/mac/editing/input/range-for-empty-document-expected.txt: Copied from editing/input/range-for-empty-document-expected.txt.
  • platform/mac/editing/input/range-for-empty-document.html: Copied from editing/input/range-for-empty-document.html.
  • platform/mac/editing/input/replace-invalid-range-expected.txt: Copied from editing/input/replace-invalid-range-expected.txt.
  • platform/mac/editing/input/replace-invalid-range.html: Copied from editing/input/replace-invalid-range.html.
  • platform/mac/editing/input/text-input-controller-expected.txt: Copied from editing/input/text-input-controller-expected.txt.
  • platform/mac/editing/input/text-input-controller.html: Copied from editing/input/text-input-controller.html.
  • platform/mac/editing/input/wrapped-line-char-rect-expected.txt: Copied from editing/input/wrapped-line-char-rect-expected.txt.
  • platform/mac/editing/input/wrapped-line-char-rect.html: Copied from editing/input/wrapped-line-char-rect.html.
  • platform/mac/editing/selection/4947387-expected.checksum: Copied from editing/selection/4947387-expected.checksum.
  • platform/mac/editing/selection/4947387-expected.png: Copied from editing/selection/4947387-expected.png.
  • platform/mac/editing/selection/4947387-expected.txt: Copied from editing/selection/4947387-expected.txt.
  • platform/mac/editing/selection/4947387.html: Copied from editing/selection/4947387.html.
  • platform/mac/editing/selection/5195166-1-expected.checksum: Copied from editing/selection/5195166-1-expected.checksum.
  • platform/mac/editing/selection/5195166-1-expected.png: Copied from editing/selection/5195166-1-expected.png.
  • platform/mac/editing/selection/5195166-1-expected.txt: Copied from editing/selection/5195166-1-expected.txt.
  • platform/mac/editing/selection/5195166-1.html: Copied from editing/selection/5195166-1.html.
  • platform/mac/editing/selection/5195166-2-expected.checksum: Copied from editing/selection/5195166-2-expected.checksum.
  • platform/mac/editing/selection/5195166-2-expected.png: Copied from editing/selection/5195166-2-expected.png.
  • platform/mac/editing/selection/5195166-2-expected.txt: Copied from editing/selection/5195166-2-expected.txt.
  • platform/mac/editing/selection/5195166-2.html: Copied from editing/selection/5195166-2.html.
  • platform/mac/editing/selection/select-line-expected.txt: Copied from editing/selection/select-line-expected.txt.
  • platform/mac/editing/selection/select-line.html: Copied from editing/selection/select-line.html.
  • platform/mac/editing/selection/selection-actions-expected.checksum: Copied from editing/selection/selection-actions-expected.checksum.
  • platform/mac/editing/selection/selection-actions-expected.png: Copied from editing/selection/selection-actions-expected.png.
  • platform/mac/editing/selection/selection-actions-expected.txt: Copied from editing/selection/selection-actions-expected.txt.
  • platform/mac/editing/selection/selection-actions.html: Copied from editing/selection/selection-actions.html.
  • platform/mac/fast/AppleScript: Copied from fast/AppleScript.
  • platform/mac/fast/dom: Added.
  • platform/mac/fast/dom/character-index-for-point-expected.txt: Copied from fast/dom/character-index-for-point-expected.txt.
  • platform/mac/fast/dom/character-index-for-point.html: Copied from fast/dom/character-index-for-point.html.
  • platform/mac/fast/dom/wrapper-identity-expected.txt: Copied from fast/dom/wrapper-identity-expected.txt.
  • platform/mac/fast/dom/wrapper-identity.html: Copied from fast/dom/wrapper-identity.html.
  • platform/mac/fast/dom/wrapper-round-tripping-expected.txt: Copied from fast/dom/wrapper-round-tripping-expected.txt.
  • platform/mac/fast/dom/wrapper-round-tripping.html: Copied from fast/dom/wrapper-round-tripping.html.
  • platform/mac/fast/encoding: Added.
  • platform/mac/fast/encoding/char-decoding-mac-expected.txt: Copied from fast/encoding/char-decoding-mac-expected.txt.
  • platform/mac/fast/encoding/char-decoding-mac.html: Copied from fast/encoding/char-decoding-mac.html.
  • platform/mac/fast/forms: Added.
  • platform/mac/fast/forms/attributed-strings-expected.txt: Copied from fast/forms/attributed-strings-expected.txt.
  • platform/mac/fast/forms/attributed-strings.html: Copied from fast/forms/attributed-strings.html.
  • platform/mac/fast/objc: Copied from fast/objc.
  • platform/mac/fast/text: Added.
  • platform/mac/fast/text/attributed-substring-from-range-001-expected.txt: Copied from fast/text/attributed-substring-from-range-001-expected.txt.
  • platform/mac/fast/text/attributed-substring-from-range-001.html: Copied from fast/text/attributed-substring-from-range-001.html.
  • platform/mac/fast/text/attributed-substring-from-range-expected.txt: Copied from fast/text/attributed-substring-from-range-expected.txt.
  • platform/mac/fast/text/attributed-substring-from-range-in-textfield-expected.txt: Copied from fast/text/attributed-substring-from-range-in-textfield-expected.txt.
  • platform/mac/fast/text/attributed-substring-from-range-in-textfield.html: Copied from fast/text/attributed-substring-from-range-in-textfield.html.
  • platform/mac/fast/text/attributed-substring-from-range.html: Copied from fast/text/attributed-substring-from-range.html.
  • platform/mac/fast/text/justified-text-rect-expected.checksum: Copied from fast/text/justified-text-rect-expected.checksum.
  • platform/mac/fast/text/justified-text-rect-expected.png: Copied from fast/text/justified-text-rect-expected.png.
  • platform/mac/fast/text/justified-text-rect-expected.txt: Copied from fast/text/justified-text-rect-expected.txt.
  • platform/mac/fast/text/justified-text-rect.html: Copied from fast/text/justified-text-rect.html.
  • platform/mac/plugins: Added.
  • platform/mac/plugins/pluginDocumentView-deallocated-dataSource-expected.txt: Copied from plugins/pluginDocumentView-deallocated-dataSource-expected.txt.
  • platform/mac/plugins/pluginDocumentView-deallocated-dataSource.html: Copied from plugins/pluginDocumentView-deallocated-dataSource.html.
  • platform/win/Skipped:
  • plugins/pluginDocumentView-deallocated-dataSource-expected.txt: Removed.
  • plugins/pluginDocumentView-deallocated-dataSource.html: Removed.
10:15 PM Changeset in webkit [25366] by mjs
  • 4 edits
    1 add in trunk/JavaScriptCore

Back out accidentally committed change.

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • kjs/collector.cpp: (KJS::Collector::registerThread):
  • wtf/FastMalloc.cpp: (WTF::fastMallocSetIsMultiThreaded): (WTF::TCMalloc_ThreadCache::GetCache): (WTF::TCMalloc_ThreadCache::GetCacheIfPresent): (WTF::TCMalloc_ThreadCache::CreateCacheIfNecessary): (WTF::do_malloc):
  • wtf/FastMallocInternal.h: Added.
10:01 PM Changeset in webkit [25365] by mjs
  • 8 edits
    1 add
    1 delete in trunk/JavaScriptCore

Reviewed by Darin.


  • Added Vector::appendRange(), which appends to a vector based on a given start and end iterator
  • Added keys() and values() functions to HashMap iterators, which give keys-only and values-only iterators


Together, these allow easy copying of a set, or the keys or values of a map, into a Vector. Examples:


HashMap<int, int> map;
HashSet<int> set;
Vector<int> vec;
...
vec.appendRange(set.begin(), set.end());
vec.appendRange(map.begin().keys(), map.end().keys());
vec.appendRange(map.begin().values(), map.end().values());

This also allows for a slightly nicer syntax when iterating a map. Instead of saying
(*it)->first, you can say *it.values(). Similarly for keys. Example:


HashMap<int, int>::const_iterator end = map.end();
for (HashMap<int, int>::const_iterator it = map.begin(); it != end; ++it)
printf(" [%d => %d]", *it.keys(), *it.values());

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • wtf/HashIterators.h: Added. (WTF::): (WTF::HashTableConstKeysIterator::HashTableConstKeysIterator): (WTF::HashTableConstKeysIterator::get): (WTF::HashTableConstKeysIterator::operator*): (WTF::HashTableConstKeysIterator::operator->): (WTF::HashTableConstKeysIterator::operator++): (WTF::HashTableConstValuesIterator::HashTableConstValuesIterator): (WTF::HashTableConstValuesIterator::get): (WTF::HashTableConstValuesIterator::operator*): (WTF::HashTableConstValuesIterator::operator->): (WTF::HashTableConstValuesIterator::operator++): (WTF::HashTableKeysIterator::HashTableKeysIterator): (WTF::HashTableKeysIterator::get): (WTF::HashTableKeysIterator::operator*): (WTF::HashTableKeysIterator::operator->): (WTF::HashTableKeysIterator::operator++): (WTF::HashTableKeysIterator::operator HashTableConstKeysIterator<HashTableType, KeyType, MappedType>): (WTF::HashTableValuesIterator::HashTableValuesIterator): (WTF::HashTableValuesIterator::get): (WTF::HashTableValuesIterator::operator*): (WTF::HashTableValuesIterator::operator->): (WTF::HashTableValuesIterator::operator++): (WTF::HashTableValuesIterator::operator HashTableConstValuesIterator<HashTableType, KeyType, MappedType>): (WTF::operator==): (WTF::operator!=):
  • wtf/HashTable.h:
  • wtf/Vector.h: (WTF::::appendRange):
8:58 PM Changeset in webkit [25364] by weinig
  • 4 edits in trunk

WebKitTools:

Reviewed by Adam.

Fix for <rdar://problem/5382277>
Implement eventSender.textZoomIn and eventSender.textZoomOut for windows DRT.

  • DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/EventSender.cpp: (textZoomInCallback): (textZoomOutCallback):

LayoutTests:

Reviewed by Adam.

Fix for <rdar://problem/5382277>
Remove fast/css/ignore-text-zoom.html from skip list now that
eventSender.textZoomIn is supported.

  • platform/win/Skipped:
5:25 PM Changeset in webkit [25363] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Darin.

<rdar://problem/5452908> NIBs saved in the Pre-10.2 format while Safari 3 installed do not work without Safari 3

This happened because we encoded a new field that the old WebKit does not know how to read.
And NSCoder throws an exception if initWithCoder finishes while there is still unread data in the archive.

The WebViewVersion is now 4 to distinguish that we do not encode/decode allowsUndo.

  • WebView/WebView.mm: (-[WebView initWithCoder:]): Only try to read allowsUndo if the version is 3. (-[WebView encodeWithCoder:]): No longer encode allowsUndo.
5:16 PM Changeset in webkit [25362] by weinig
  • 2 edits in trunk/WebKit/win

Build fix.

  • WebKit.vcproj/WebKit.vcproj:
4:23 PM Changeset in webkit [25361] by hyatt
  • 1 edit in trunk/WebCore/page/Frame.cpp

Fix build bustage.

3:58 PM Changeset in webkit [25360] by hyatt
  • 12 edits in trunk

WebCore:

Fix for <rdar://problem/5271213>, resizing iChat window is slower than in
Tiger. This patch implements a fast scaling mode that can be used by
WebViews, e.g., during window resizing.

Reviewed by John Sullivan

  • WebCore.exp:
  • WebCore.xcodeproj/project.pbxproj:
  • page/Frame.cpp: (WebCore::FramePrivate::FramePrivate):
  • page/Page.cpp: (WebCore::Page::Page): (WebCore::Page::inLowQualityImageInterpolationMode): (WebCore::Page::setInLowQualityImageInterpolationMode):
  • page/Page.h:
  • platform/graphics/GraphicsContext.cpp: (WebCore::GraphicsContext::drawImage):
  • platform/graphics/GraphicsContext.h: (WebCore::GraphicsContext::setUseLowQualityImageInterpolation): (WebCore::GraphicsContext::useLowQualityImageInterpolation):
  • platform/graphics/cg/GraphicsContextCG.cpp: (WebCore::GraphicsContext::setUseLowQualityImageInterpolation): (WebCore::GraphicsContext::useLowQualityImageInterpolation):
  • rendering/RenderImage.cpp: (WebCore::RenderImage::paint):

WebKit:

Fix for <rdar://problem/5271213>, resizing iChat window is slower than in
Tiger. This patch implements a fast scaling mode that can be used by
WebViews, e.g., during window resizing.

Reviewed by John Sullivan

  • WebView/WebView.mm: (-[WebView _setUseFastImageScalingMode:]): (-[WebView _inFastImageScalingMode]):
  • WebView/WebViewPrivate.h:
3:47 PM Changeset in webkit [25359] by darin
  • 2 edits in trunk/WebKit

Reviewed by Hyatt.

  • WebView/WebView.mm: (-[WebView _loadBackForwardListFromOtherView:]): Added missing null check. (-[WebView _setInitiatedDrag:]): Ditto. (-[WebView _clearUndoRedoOperations]): Ditto. (-[WebView encodeWithCoder:]): Ditto. (-[WebView backForwardList]): Ditto. (-[WebView setMaintainsBackForwardList:]): Ditto.
3:14 PM Changeset in webkit [25358] by adele
  • 3 edits in trunk/LayoutTests

Updated test for <rdar://problem/5279521>. The old test didn't crash before the fix.

  • editing/style/5279521.html:
  • editing/style/5279521-expected.txt:
2:56 PM Changeset in webkit [25357] by beidson
  • 1 edit in trunk/WebCore/loader/icon/SQLDatabase.h

Blind attempt at fixing Windows debug build

1:32 PM Changeset in webkit [25356] by beidson
  • 6 edits in trunk/WebCore

Reviewed by Adam

Some SQLite tweaks for debugging
-In debug builds, track the thread that opened the database and enforce that any statements

that operate on the SQLDatabase do so from the thread that opened it

-Track whether or not a transaction is in progress for the SQLDatabase to find cases where

someone opens a new transaction while there is still one outstanding (a SQLite logic error)

  • loader/icon/SQLDatabase.cpp: (WebCore::SQLDatabase::SQLDatabase): (WebCore::SQLDatabase::open): Track the opening thread. Also, return false if the DB failed to open instead of trying to run a PRAGMA on it! (WebCore::SQLDatabase::close): Reset the opening thread
  • loader/icon/SQLDatabase.h: (WebCore::SQLDatabase::transactionInProgress): (WebCore::SQLDatabase::sqlite3Handle): For access to the raw sqlite3* handle to do a thread safety check
  • loader/icon/SQLStatement.cpp: (WebCore::SQLStatement::prepare): Use the new sqlite3Handle accessor (WebCore::SQLStatement::step): Ditto
  • loader/icon/SQLTransaction.cpp: (WebCore::SQLTransaction::SQLTransaction): Removed the c'tor form that could automatically begin the transaction - never used and shouldn't be in practice! (WebCore::SQLTransaction::begin): Maintain the transaction-in-progress flag on the DB (WebCore::SQLTransaction::commit): Ditto (WebCore::SQLTransaction::rollback): Ditto
  • loader/icon/SQLTransaction.h:
12:50 PM Changeset in webkit [25355] by tristan
  • 2 edits in trunk/WebKit

Reviewed by John Sullivan.


<rdar://problem/5454935> Can't reply to this message in Mail -- -[DOMRange webArchive] is throwing an exception

  • WebView/WebArchiver.mm: (+[WebArchiver _archiveWithMarkupString:fromFrame:nodes:]): Guard the creation of WebResource by ensuring that the passed in responseURL is never nil.
9:55 AM Changeset in webkit [25354] by harrison
  • 3 edits
    4 adds in trunk

WebCore:

Reviewed by Maciej and John.

<rdar://problem/5452675> CrashTracer: [USER] 10 crashes in Mail at com.apple.WebCore: WebCore::Node::nodeIndex const + 6

  • editing/InsertParagraphSeparatorCommand.cpp: (WebCore::InsertParagraphSeparatorCommand::doApply): Adjust pos to pos.downstream() after the refNode is calculated, but before the insertion. Doing it earlier undid the logic of positionAvoidingSpecialElementBoundary(). The downstream is still needed just to set the ending selection.

LayoutTests:

Reviewed by Maciej and John.

Testcase for:
<rdar://problem/5452675> CrashTracer: [USER] 10 crashes in Mail at com.apple.WebCore: WebCore::Node::nodeIndex const + 6

  • editing/inserting/insert-paragraph-05.html: Added.
  • platform/mac/editing/inserting/insert-paragraph-05-expected.checksum: Added.
  • platform/mac/editing/inserting/insert-paragraph-05-expected.png: Added.
  • platform/mac/editing/inserting/insert-paragraph-05-expected.txt: Added.
7:50 AM Changeset in webkit [25353] by antti
  • 3 edits
    2 adds in trunk

WebCore:

Reviewed by Maciej.


Fix <rdar://problem/5452112>
REGRESSION: Initially focused textfield on www.mac.com login page has no insertion point, and doesn't accept typed characters


Use updateLayoutIgnorePendingStylesheets() instead of updateLayout(). We need to have rendering to set input focus.

Test: fast/forms/focus-style-pending.html

  • dom/Element.cpp: (WebCore::Element::focus):

LayoutTests:

Reviewed by Maciej.


Test for <rdar://problem/5452112>
REGRESSION: Initially focused textfield on www.mac.com login page has no insertion point, and doesn't accept typed characters

  • fast/forms/focus-style-pending-expected.txt: Added.
  • fast/forms/focus-style-pending.html: Added.
6:02 AM Changeset in webkit [25352] by hausmann
  • 2 edits in trunk/WebCore

Fixed Qt build on Windows

Sep 3, 2007:

9:31 AM Changeset in webkit [25351] by harrison
  • 2 edits in trunk/WebCore

Reviewed by Kevin Decker.

<rdar://problem/5456785> REGRESSION (r15963-r15970): Heading text not placed in VoiceOver Item Chooser (15132)

  • bridge/mac/WebCoreAXObject.mm: (-[WebCoreAXObject title]): For headings, return the text under the element instead of nil.
6:48 AM Changeset in webkit [25350] by zecke
  • 2 edits in trunk/WebCore

2007-08-20 Oleg Sukhodolsky <​son.two@gmail.com>

Reviewed by Mark.

WebKitQt/WebCoreSupport should not be added to INCLUDEPATH for gdk-port

  • WebCore.pro:
2:56 AM Changeset in webkit [25349] by bdash
  • 12 edits in trunk

2007-09-03 Mark Rowe <​mrowe@apple.com>

Reviewed by Tim Hatcher.

<rdar://problem/5452164> Production build with in symbols directory has no debug info

Enable debug symbol generation on all build configurations. Production builds are stripped
of symbols by Xcode during deployment post-processing.

  • Configurations/Base.xcconfig:
  • JavaScriptCore.xcodeproj/project.pbxproj:

2007-09-03 Mark Rowe <​mrowe@apple.com>

Reviewed by Tim Hatcher.

<rdar://problem/5452164> Production build with in symbols directory has no debug info

Enable debug symbol generation on all build configurations. Production builds are stripped
of symbols by Xcode during deployment post-processing.

  • Configurations/Base.xcconfig:
  • JavaScriptGlue.xcodeproj/project.pbxproj:

2007-09-03 Mark Rowe <​mrowe@apple.com>

Reviewed by Tim Hatcher.

<rdar://problem/5452164> Production build with in symbols directory has no debug info

Enable debug symbol generation on all build configurations. Production builds are stripped
of symbols by Xcode during deployment post-processing.

  • Configurations/Base.xcconfig:
  • WebCore.xcodeproj/project.pbxproj:

2007-09-03 Mark Rowe <​mrowe@apple.com>

Reviewed by Tim Hatcher.

<rdar://problem/5452164> Production build with in symbols directory has no debug info

Enable debug symbol generation on all build configurations. Production builds are stripped
of symbols by Xcode during deployment post-processing.

  • Configurations/Base.xcconfig:
  • WebKit.xcodeproj/project.pbxproj:

Sep 2, 2007:

6:37 PM Changeset in webkit [25348] by beidson
  • 8 edits in trunk

WebCore:

Reviewed by John Sullivan and Mark Rowe

Groundwork for support for monitoring IconDatabase in-memory statistics

  • WebCore.exp:
  • loader/icon/IconDatabase.cpp: (WebCore::IconDatabase::pageURLMappingCount): Stub for now (WebCore::IconDatabase::retainedPageURLCount): Ditto (WebCore::IconDatabase::iconRecordCount): Ditto (WebCore::IconDatabase::iconRecordCountWithData): Ditto
  • loader/icon/IconDatabase.h:

WebKit:

Reviewed by John Sullivan and Mark Rowe

Groundwork for support for monitoring IconDatabase in-memory statistics


  • Misc/WebCoreStatistics.h:
  • Misc/WebCoreStatistics.mm: (+[WebCoreStatistics iconPageURLMappingCount]): (+[WebCoreStatistics iconRetainedPageURLCount]): (+[WebCoreStatistics iconRecordCount]): (+[WebCoreStatistics iconsWithDataCount]):
6:17 PM Changeset in webkit [25347] by bdash
  • 2 edits in trunk/WebCore

2007-09-02 Mark Rowe <​mrowe@apple.com>

Reviewed by Antti.

<rdar://problem/5454704> WebKit seems to get too-narrow widths for "Monotype Corsiva", so lays out incorrectly

  • platform/mac/FontDataMac.mm: (WebCore::FontData::determinePitch): Work around NSFont incorrectly reporting Monotype Corsiva as fixed pitch.
8:19 AM UsingSafari2WithSafari3PublicBetaInstalled created by ddkilzer@webkit.org
7:52 AM WikiStart edited by ddkilzer@webkit.org
(diff)
Note: See TracTimeline for information about the timeline view.