WebKit-https.git
15 years ago Reviewed by Darin Adler.
mjs [Sun, 5 Aug 2007 10:52:19 +0000 (10:52 +0000)]
    Reviewed by Darin Adler.

        <rdar://problem/5369110> CrashTracer: [USER] reproducible crash opening particular mail messages

        * platform/network/mac/ResourceHandleMac.mm:
        (-[WebCoreResourceHandleAsDelegate connection:willSendRequest:redirectResponse:]): Make sure to retain
        self for the body of this method. Otherwise, the willSendRequest could trigger events which will
        cancel the connection, and we access ivars after this point.
        (-[WebCoreSynchronousLoader connection:willSendRequest:redirectResponse:]): retain and release
        in the right order.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24875 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Maciej.
darin [Sun, 5 Aug 2007 10:16:41 +0000 (10:16 +0000)]
    Reviewed by Maciej.

        - fix <rdar://problem/5371862> crash in Dashcode due to Quartz Composer JavaScript garbage collector reentrancy

        * API/JSBase.cpp: (JSGarbageCollect): Don't call collector() if isBusy() returns true.

        * kjs/collector.h: Added isBusy(), removed the unused return value from collect()
        * kjs/collector.cpp: Added an "operation in progress" flag to the allocator.
        (KJS::Collector::allocate): Call abort() if an operation is already in progress. Set the new flag instead
        of using the debug-only GCLock.
        (KJS::Collector::collect): Ditto.
        (KJS::Collector::isBusy): Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24874 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Darin and Adam.
mjs [Sun, 5 Aug 2007 05:20:35 +0000 (05:20 +0000)]
    Reviewed by Darin and Adam.

        <rdar://problem/5368990> REGRESSION: newsgator.com sign-on 6x slower than Safari 3 beta due to GC changes (14808)

        * kjs/string_object.cpp:
        (KJS::replace): if the string didn't change (very common in some cases) reuse the original string value.
        (KJS::StringProtoFunc::callAsFunction): Pass in the StringImp* when replacing, not just the UString.
        * kjs/string_object.h:
        (KJS::StringInstance::internalValue): covariant override to return StringImp for convenience

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24873 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Another workaround for <rdar://problem/5386894>
aroben [Sun, 5 Aug 2007 04:21:52 +0000 (04:21 +0000)]
    Another workaround for <rdar://problem/5386894>

        Reviewed by Sam.

        This fixed ~150 failing tests.

        * platform/network/cf/ResourceResponseCFNet.cpp:
        (WebCore::ResourceResponse::doUpdateResourceResponse): Hardcode the
        MIME type for .svg files as well.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24872 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Workaround for <rdar://problem/5386894> CFURLResponseGetMIMEType returns...
aroben [Sun, 5 Aug 2007 03:04:47 +0000 (03:04 +0000)]
    Workaround for <rdar://problem/5386894> CFURLResponseGetMIMEType returns "text/html" for local .xhtml and .xml files

        Reviewed by Sam.

        This fixes ~350 failing tests.

        * platform/network/cf/ResourceResponseCFNet.cpp:
        (WebCore::ResourceResponse::doUpdateResourceResponse): Use a workaround
        identical to the one in ResourceResponseMac.mm, but include .xml files
        as well.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24871 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Sam.
oliver [Sun, 5 Aug 2007 02:15:14 +0000 (02:15 +0000)]
    Reviewed by Sam.

        Layout test for <rdar://problem/5386550> REGRESSION (r24866): text/plain documents are always downloaded

        * fast/loader/plain-text-document-expected.txt: Added.
        * fast/loader/plain-text-document.html: Added.
        * fast/loader/plain-text-document.txt: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24870 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoWebCore:
ddkilzer [Sun, 5 Aug 2007 02:06:22 +0000 (02:06 +0000)]
WebCore:

        Reviewed by Oliver.

        - fix for http://bugs.webkit.org/show_bug.cgi?id=14882
          <rdar://problem/5386550> REGRESSION (r24866): text/plain documents are always downloaded

        The supportedNonImageMimeTypes list is used to determine which MIME types may be viewed
        within the web browser (e.g., plug-ins add their own MIME types to the list during
        initialization), so we must add "text/plain" and "text/" back to the list.  Since
        this change would then break DOMImplementation::isTextMIMEType(), that method was reverted
        to its original form and MIMETypeRegistry::shouldTreatAsText() was removed.

        * dom/DOMImplementation.cpp:
        (WebCore::DOMImplementation::isTextMIMEType): Revert to previous version.
        * platform/MIMETypeRegistry.cpp:
        (WebCore::initialiseSupportedNonImageMimeTypes): Added back "text/plain" and "text/" to the list.
        (WebCore::MIMETypeRegistry::shouldTreatAsText): Removed.
        * platform/MIMETypeRegistry.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24869 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-08-04 Mark Rowe <mrowe@apple.com>
bdash [Sat, 4 Aug 2007 08:58:35 +0000 (08:58 +0000)]
2007-08-04  Mark Rowe  <mrowe@apple.com>

        Reviewed by Oliver Hunt.

        <rdar://problem/5385145> r24843 introduces a crash on calling fork() (14878)
        http://bugs.webkit.org/show_bug.cgi?id=14878

        Provide nooop functions for all members of the malloc_zone_t and malloc_introspection_t structures that we
        register to avoid crashes in system code that assumes they will be non-null.

        * kjs/CollectorHeapIntrospector.cpp:
        (KJS::CollectorHeapIntrospector::CollectorHeapIntrospector):
        (KJS::CollectorHeapIntrospector::forceLock): Grab the lock.
        (KJS::CollectorHeapIntrospector::forceUnlock): Release the lock.
        * kjs/CollectorHeapIntrospector.h:
        (KJS::CollectorHeapIntrospector::goodSize):
        (KJS::CollectorHeapIntrospector::check):
        (KJS::CollectorHeapIntrospector::print):
        (KJS::CollectorHeapIntrospector::log):
        (KJS::CollectorHeapIntrospector::statistics):
        (KJS::CollectorHeapIntrospector::size):
        (KJS::CollectorHeapIntrospector::zoneMalloc):
        (KJS::CollectorHeapIntrospector::zoneCalloc):
        (KJS::CollectorHeapIntrospector::zoneFree):
        * wtf/FastMalloc.cpp:
        (WTF::FastMallocZone::goodSize):
        (WTF::FastMallocZone::check):
        (WTF::FastMallocZone::print):
        (WTF::FastMallocZone::log):
        (WTF::FastMallocZone::forceLock): Grab the TCMalloc locks.
        (WTF::FastMallocZone::forceUnlock): Release the TCMalloc locks.
        (WTF::FastMallocZone::FastMallocZone):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24868 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoAdded the radar # to my FIXME text
beidson [Sat, 4 Aug 2007 07:37:59 +0000 (07:37 +0000)]
Added the radar # to my FIXME text

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24867 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoWebCore:
beidson [Sat, 4 Aug 2007 07:29:55 +0000 (07:29 +0000)]
WebCore:

        Reviewed by Oliver

        Fix for http://bugs.webkit.org/show_bug.cgi?id=14824 and <rdar://problem/5372989>

        Two issues -
        1 - The WebCore MIMEType registry was designed assuming the list of types would never change
            That is false, as WebKit has API and SPI calls which directly mutate the MIMETypeRegistry
        2 - DOMImplementation didn't consult the registry for any MIMEType that started with "text/",
            instead maintaining it's own hard coded rules

        * dom/DOMImplementation.cpp:
        (WebCore::DOMImplementation::isTextMIMEType): For now, call through to the MIMETypeRegistry
          until we decided a different fate for this function

        * platform/MIMETypeRegistry.cpp:
        (WebCore::initialiseSupportedNonImageMimeTypes):  "SupportedNonImageMIMETypes" is really a misnomer for
          "MIMETypes we should show as HTML" but that line has slowly been blurred since Tiger.  In an attempt
          to start to unblur it, remove "text/" and "text/plain"
        (WebCore::MIMETypeRegistry::shouldTreatAsText):  The decision is very close to the old DOMImplementation
          method, except we don't automatically hand off "text/" types as true if they are in the set of supported
          MIMETypes
        (WebCore::MIMETypeRegistry::getSupportedImageMIMETypes): Non-const (can be changed!)
        (WebCore::MIMETypeRegistry::getSupportedImageResourceMIMETypes): Non-const (can be changed!)
        (WebCore::MIMETypeRegistry::getSupportedNonImageMIMETypes): Non-const (can be changed!)
        * platform/MIMETypeRegistry.h:

WebKit:

        Reviewed by Oliver

        Fix for http://bugs.webkit.org/show_bug.cgi?id=14824 and <rdar://problem/5372989>

        When unregistering a MIMEType, remove it from the WebCore registry unconditionally
        When registrying a MIMEType whose view class is WebHTMLView, add it to the WebCore registry

        * WebView/WebView.mm:
        (+[WebView _unregisterViewClassAndRepresentationClassForMIMEType:]):
        (+[WebView _registerViewClass:representationClass:forURLScheme:]):
        (+[WebView registerViewClass:representationClass:forMIMEType:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24866 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Catch SIGPIPE on Windows so that run-webkit-tests doesn't quit when DRT crashes
aroben [Sat, 4 Aug 2007 03:57:19 +0000 (03:57 +0000)]
 Catch SIGPIPE on Windows so that run-webkit-tests doesn't quit when DRT crashes

        Reviewed by Sam.

        * Scripts/run-webkit-tests: Also close ERROR when we finish running the tests.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24865 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Adam.
adele [Sat, 4 Aug 2007 01:09:02 +0000 (01:09 +0000)]
    Reviewed by Adam.

        Fix for <rdar://problem/5345862> CrashTracer: [USER] 4 crashes in Safari at com.apple.WebCore: WebCore::Frame::isContentEditable const + 10

        Couldn't reproduce the problem, but a nil check for the frame should fix this.

        * css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::adjustRenderStyle):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24864 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoLayoutTests:
weinig [Sat, 4 Aug 2007 00:22:45 +0000 (00:22 +0000)]
LayoutTests:

        Reviewed by Darin.

        Dump contents of subframes using new layoutTestController.dumpChildFramesAsText()
        function.

        * http/tests/security/cross-frame-access-port-expected.txt:
        * http/tests/security/cross-frame-access-port-explicit-domain-expected.txt:
        * http/tests/security/cross-frame-access-protocol-expected.txt:
        * http/tests/security/cross-frame-access-protocol-explicit-domain-expected.txt:
        * http/tests/security/resources/cross-frame-access.js:

WebKitTools:

        Reviewed by Darin.

        Recursively dump all frames as text using new
        layoutTestController.dumpChildFramesAsText() function.

        * DumpRenderTree/DumpRenderTree.m:
        (dumpFramesAsText):
        (dump):
        (+[LayoutTestController isSelectorExcludedFromWebScript:]):
        (-[LayoutTestController dumpChildFramesAsText]):
        (runTest):
        * DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.cpp:
        (dumpFramesAsText):
        (dump):
        (runTest):
        * DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.h:
        * DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/LayoutTestController.cpp:
        (dumpChildFramesAsTextCallback):
        * DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24863 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Oliver.
andersca [Fri, 3 Aug 2007 23:53:57 +0000 (23:53 +0000)]
    Reviewed by Oliver.

        <rdar://problem/5383286>
        XMLHTTPRequest does not return 401 when user cancels authentication dialog (affects .Mac)

        * loader/ResourceLoader.h:
        Make receivedCancellation virtual.

        * loader/SubresourceLoader.cpp:
        (WebCore::SubresourceLoader::receivedCancellation):
        Call SubresourceLoaderClient::receivedCancellation.

        * xml/XMLHttpRequest.cpp:
        (WebCore::XMLHttpRequest::receivedCancellation):
        Save the failure response.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24862 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Geoff.
andersca [Fri, 3 Aug 2007 23:07:22 +0000 (23:07 +0000)]
    Reviewed by Geoff.

        <rdar://problem/5374393>
        Image change on disk not noticed by WebView; -[WebCache setDisabled:] used to cause a reload every time

        This is essentially a better fix for the crash in <rdar://problem/5362783>.

        * loader/DocLoader.cpp:
        (WebCore::DocLoader::requestResource):
        If the resource already exists in the m_docResources map, remove it and disassociate it from the doc loader.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24861 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Geoff.
oliver [Fri, 3 Aug 2007 22:37:08 +0000 (22:37 +0000)]
    Reviewed by Geoff.

        <rdar://problem/5375190> Mail crashed in WebCore::CachedImage::imageSize() const when viewing a particular message

        Fix the uncached load path to confirm that the type of resource being
        returned actually matches the type that was requested.

        Also make sure we never create a resource in the Cache for invalid urls.

        * loader/Cache.cpp:
        (WebCore::Cache::requestResource):
        * loader/DocLoader.cpp:
        (WebCore::DocLoader::requestResource):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24860 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoBump version for submit
sfalken [Fri, 3 Aug 2007 22:17:05 +0000 (22:17 +0000)]
Bump version for submit

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24858 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoLayoutTests:
adele [Fri, 3 Aug 2007 22:15:13 +0000 (22:15 +0000)]
LayoutTests:

        Reviewed by Adele.

        - test for http://bugs.webkit.org/show_bug.cgi?id=14879
          REGRESSION: First item in select (pop-up menu) is displayed even if another item was selected via JavaScript

        * fast/forms/menulist-deselect-update-expected.checksum: Added.
        * fast/forms/menulist-deselect-update-expected.png: Added.
        * fast/forms/menulist-deselect-update-expected.txt: Added.
        * fast/forms/menulist-deselect-update.html: Added.

WebCore:

        Reviewed by Adele.

        - fix http://bugs.webkit.org/show_bug.cgi?id=14879
          REGRESSION: First item in select (pop-up menu) is displayed even if another item was selected via JavaScript

        Test: fast/forms/menulist-deselect-update.html

        * html/HTMLOptionElement.cpp:
        (WebCore::HTMLOptionElement::setSelected): Reordered to allow setSelectedIndex() to call setChanged().

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24857 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Darin.
andersca [Fri, 3 Aug 2007 20:09:51 +0000 (20:09 +0000)]
    Reviewed by Darin.

        <rdar://problem/5286444>
        http://bugs.webkit.org/show_bug.cgi?id=14269
        REGRESSION: Gmail links stop working after computer sleep

        Add a PowerNotifier object that takes care of resetting and firing the shared timer when coming
        back from sleep.

        * platform/mac/SharedTimerMac.cpp:
        (-[PowerNotifier init]):
        (-[PowerNotifier didWake:]):
        (WebCore::setSharedTimerFireTime):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24851 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-08-04 Mark Rowe <mrowe@apple.com>
bdash [Fri, 3 Aug 2007 18:46:39 +0000 (18:46 +0000)]
2007-08-04  Mark Rowe  <mrowe@apple.com>

        Windows build fix.

        * rendering/RenderTextControl.cpp: Don't use the same name for two arguments.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24850 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-08-04 Mark Rowe <mrowe@apple.com>
bdash [Fri, 3 Aug 2007 18:19:31 +0000 (18:19 +0000)]
2007-08-04  Mark Rowe  <mrowe@apple.com>

        Rubber-stamped by Anders.

        * pcre/pcre_compile.c: Remove non-ASCII character from a comment.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24849 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Mark Rowe.
thatcher [Fri, 3 Aug 2007 18:03:00 +0000 (18:03 +0000)]
    Reviewed by Mark Rowe.

        Correct the bundle version check to work in Debug and Release builds too.

        * WebKit.xcodeproj/project.pbxproj:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24848 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoBump versioning to avoid 4. issues.
bdash [Fri, 3 Aug 2007 17:58:04 +0000 (17:58 +0000)]
Bump versioning to avoid 4. issues.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24847 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-08-02 Mark Rowe <mrowe@apple.com>
bdash [Fri, 3 Aug 2007 16:21:44 +0000 (16:21 +0000)]
2007-08-02  Mark Rowe  <mrowe@apple.com>

        Reviewed by Geoff Garen.

        <rdar://problem/4212199> 'leaks' reports false leaks in WebKit (because the WTF allocator uses mmap?)

        Implement malloc zone introspection routines to allow leaks, heap, and friends to request information
        about specific memory regions that were allocated by FastMalloc or the JavaScriptCore collector.

        This requires tool-side support before the regions will be displayed.  The addition of that support is
        tracked by <rdar://problems/5353057&5353060>.

        * JavaScriptCore.exp: Export the two variables that are used by leaks to introspect the allocators.
        * JavaScriptCore.xcodeproj/project.pbxproj:
        * kjs/AllInOneFile.cpp:
        * kjs/CollectorZone.cpp: Added.
        (KJS::):
        (KJS::CollectorZone::registerZone):
        (KJS::CollectorZone::CollectorZone): Create and register our zone with the system.
        (KJS::CollectorZone::zoneEnumerator): Iterate over the CollectorBlocks that are in use and report them to the caller as being used.
        * kjs/CollectorZone.h: Added.
        (KJS::CollectorZone::zoneObjectSize): Return zero to indicate the specified pointer does not belong to this zone.
        * kjs/collector.cpp:
        (KJS::Collector::registerThread): Register the CollectorZone with the system when the first thread is registered with the collector.
        * wtf/FastMalloc.cpp:
        (WTF::TCMalloc_PageHeap::GetDescriptorEnsureSafe):
        (WTF::TCMalloc_ThreadCache_FreeList::enumerateFreeObjects): Enumerate the objects on the free list.
        (WTF::TCMalloc_ThreadCache::enumerateFreeObjects): Ditto.
        (WTF::TCMalloc_Central_FreeList::enumerateFreeObjects): Ditto.
        (WTF::TCMalloc_ThreadCache::InitModule): Register the FastMallocZone with the system when initializing TCMalloc.
        (WTF::FreeObjectFinder::FreeObjectFinder):
        (WTF::FreeObjectFinder::visit): Add an object to the free list.
        (WTF::FreeObjectFinder::isFreeObject):
        (WTF::FreeObjectFinder::freeObjectCount):
        (WTF::FreeObjectFinder::findFreeObjects): Find the free objects within a thread cache or free list.
        (WTF::PageMapFreeObjectFinder::PageMapFreeObjectFinder): Find the free objects within a TC_PageMap.
        (WTF::PageMapFreeObjectFinder::visit): Called once per allocated span.  Record whether the span or any subobjects are free.
        (WTF::PageMapMemoryUsageRecorder::PageMapMemoryUsageRecorder):
        (WTF::PageMapMemoryUsageRecorder::visit): Called once per allocated span.  Report the range of memory as being allocated, and the span or
        it's subobjects as being used if they do not appear on the free list.
        (WTF::FastMallocZone::zoneEnumerator): Map the key remote TCMalloc data structures into our address space.  We then locate all free memory ranges
        before reporting the other ranges as being in use.
        (WTF::FastMallocZone::zoneObjectSize): Determine whether the given pointer originates from within our allocation zone.  If so,
        we return its allocation size.
        (WTF::FastMallocZone::zoneMalloc):
        (WTF::FastMallocZone::zoneCalloc):
        (WTF::FastMallocZone::zoneFree):
        (WTF::FastMallocZone::zoneRealloc):
        (WTF::):
        (WTF::FastMallocZone::FastMallocZone): Create and register our zone with the system.
        (WTF::FastMallocZone::registerZone):
        * wtf/MallocZoneSupport.h: Added.
        (WTF::RemoteMemoryReader::RemoteMemoryReader): A helper class to ease the process of mapping memory in a different process into
        our local address space
        (WTF::RemoteMemoryReader::operator()):
        * wtf/TCPageMap.h:
        (TCMalloc_PageMap2::visit): Walk over the heap and visit each allocated span.
        (TCMalloc_PageMap3::visit): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24843 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-08-03 Mitz Pettel <mitz@webkit.org>
bdash [Fri, 3 Aug 2007 14:29:26 +0000 (14:29 +0000)]
2007-08-03  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Darin.

        - fix http://bugs.webkit.org/show_bug.cgi?id=14653
          REGRESSION (r23994): No caret is drawn after clicking a search field's placeholder text
          <rdar://problem/5383841>

        Test: fast/forms/search-click-in-placeholder.html

        Defined a subclass of RenderBlock that never hit-tests children for use in
        text controls. This avoids returning placeholder text as the hit node.
        Since text controls cannot contain inline elements, there is no harm in
        doing that unconditionally, and not just in the case that the field is
        showing placeholder text.

        * rendering/RenderTextControl.cpp:
        (WebCore::RenderTextControlInnerBlock::RenderTextControlInnerBlock):
        (WebCore::RenderTextControlInnerBlock::~RenderTextControlInnerBlock):
        (WebCore::RenderTextControlInnerBlock::nodeAtPoint):
        (WebCore::RenderTextControl::createSubtreeIfNeeded):

2007-08-03  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Darin.

        - test for http://bugs.webkit.org/show_bug.cgi?id=14653
          REGRESSION (r23994): No caret is drawn after clicking a search field's placeholder text
          <rdar://problem/5383841>

        * fast/forms/search-click-in-placeholder-expected.txt: Added.
        * fast/forms/search-click-in-placeholder.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24842 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoVersioning.
bdash [Fri, 3 Aug 2007 11:33:43 +0000 (11:33 +0000)]
Versioning.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24840 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Updated Skipped files for Windows and Leopard.
aroben [Fri, 3 Aug 2007 07:21:49 +0000 (07:21 +0000)]
    Updated Skipped files for Windows and Leopard.

        * mac/leopard/Skipped: Removed a test that has been moved (and only
        fails on Windows), and added new failures.
        * win/Skipped: Added new failures and annotated some old ones.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24838 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Fix a typo in fast/events/no-blur-on-page
aroben [Fri, 3 Aug 2007 07:21:25 +0000 (07:21 +0000)]
 Fix a typo in fast/events/no-blur-on-page

        Reviewed by Sam.

        * fast/events/no-blur-on-page-leave-expected.txt: Updated.
        * fast/events/no-blur-on-page-leave.html: Fixed typo.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24837 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Fix fast/dom/Window/alert-undefined.html
aroben [Fri, 3 Aug 2007 07:21:15 +0000 (07:21 +0000)]
 Fix fast/dom/Window/alert-undefined.html

        Reviewed by Sam.

        * DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/WaitUntilDoneDelegate.cpp:
        (WaitUntilDoneDelegate::runJavaScriptAlertPanelWithMessage): Don't let
        Windows translate a null BSTR into "(null)"

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24836 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Geoff
oliver [Fri, 3 Aug 2007 03:17:33 +0000 (03:17 +0000)]
    Reviewed by Geoff

        Fix for <rdar://problem/5369332> Xcode crashes while selecting a hyperlink within a AppleScript dictionary (WebCore::Font::drawGlyphBuffer)

        There were many places where we were not correctly retaining/releasing the
        NSFont object stored in the C++ PlatformFontData object, this resulted in
        the GC incorrectly collecting the NSFont.

        This patch fixes the problem by prevent direct modification of the PlatformFontData
        font pointer, allowing us to enforce correct CFRetain/Release behaviour.

        * platform/FontData.h:
        (WebCore::FontData::getNSFont):
        * platform/mac/FontCacheMac.mm:
        (WebCore::FontCache::getFontDataForCharacters):
        (WebCore::FontCache::createFontPlatformData):
        * platform/mac/FontDataMac.mm:
        (WebCore::initFontData):
        (WebCore::FontData::platformInit):
        (WebCore::FontData::platformDestroy):
        (WebCore::FontData::smallCapsFontData):
        (WebCore::FontData::containsCharacters):
        (WebCore::FontData::determinePitch):
        (WebCore::FontData::platformWidthForGlyph):
        (WebCore::FontData::checkShapesArabic):
        * platform/mac/FontMac.mm:
        (WebCore::initializeATSUStyle):
        (WebCore::overrideLayoutOperation):
        (WebCore::Font::drawGlyphs):
        * platform/mac/FontPlatformData.h:
        (WebCore::FontPlatformData::FontPlatformData):
        (WebCore::FontPlatformData::~FontPlatformData):
        (WebCore::FontPlatformData::hash):
        (WebCore::FontPlatformData::operator==):
        (WebCore::FontPlatformData::font):
        (WebCore::FontPlatformData::setFont):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24835 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Kevin McCullough.
weinig [Fri, 3 Aug 2007 00:24:05 +0000 (00:24 +0000)]
    Reviewed by Kevin McCullough.

        Change test to use shared shouldBe function to
        show failures more clearly.

        * fast/dom/Window/window-resize-expected.txt:
        * fast/dom/Window/window-resize.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24834 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Oops, this change wasn't supposed to be commited.
antti [Thu, 2 Aug 2007 23:54:15 +0000 (23:54 +0000)]
    Oops, this change wasn't supposed to be commited.

        * page/mac/WebCoreFrameBridge.mm:
        (-[WebCoreFrameBridge setBaseBackgroundColor:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24833 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Darin.
antti [Thu, 2 Aug 2007 23:45:27 +0000 (23:45 +0000)]
    Reviewed by Darin.

        <rdar://problem/5355951>
        plainText() fragments TCMalloc heap badly on large pages

        also likely fixes some cases of
        <rdar://problem/5335382>
        CrashTracer: [REGRESSION] 73 crashes in Safari at com.apple.WebCore: WebCore::DeprecatedStringData::increaseUnicodeSize + 52

        If you load http://dscoder.com/test.txt with WebKit build with TCMalloc and system malloc you see that
        Safari RPRVT with TCMalloc is 118.8MB
        Safari RPRVT with system malloc is 69.7MB

        Difference is almost entirely caused by heap fragmentation from a full document plainText() call (for indexing purposes).

        The patch helps in two ways:
        - construct plainText string in pieces to avoid O(n^2) reallocs
        - allocate buffers using system malloc so they can be returned back to OS and don't fragment and grow TCMalloc heap

        This shrinks http://dscoder.com/test.txt RPRVT to 79.0MB and makes full document plainText() take 50ms instead of 500ms.
        The benefits are not limited to extreme cases, web pages above ~200kB can show substantial improvement in RPRVT.

        * editing/TextIterator.cpp:
        (WebCore::plainTextToMallocAllocatedBuffer):
        (WebCore::plainText):
        * editing/TextIterator.h:
        * page/mac/WebCoreFrameBridge.mm:
        (-[WebCoreFrameBridge selectedString]):
        (-[WebCoreFrameBridge stringForRange:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24832 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Fix for 5374437, allow comment nodes to be the child of a document.
hyatt [Thu, 2 Aug 2007 22:50:26 +0000 (22:50 +0000)]
    Fix for 5374437, allow comment nodes to be the child of a document.
        Refine the check to always make a root element to check documentElement()
        rather than firstChild(), since a comment node could be present as the
        firstChild() now.

        Reviewed by Tim Hatcher

        * html/HTMLDocument.cpp:
        (WebCore::HTMLDocument::childAllowed):
        * html/HTMLParser.cpp:
        (WebCore::HTMLParser::finished):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24831 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Tim.
kmccullo [Thu, 2 Aug 2007 22:39:52 +0000 (22:39 +0000)]
    Reviewed by Tim.

        - It would help if I actually called the right function.

        * Drosera/DebuggerDocument.cpp:
        (DebuggerDocument::showConsole):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24830 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoLayoutTests:
antti [Thu, 2 Aug 2007 22:00:11 +0000 (22:00 +0000)]
LayoutTests:

        Reviewed by Darin.

        <rdar://problem/5228138>
        REGRESSION(Leopard): test failures: tests that test Georgian numbering

        Correct counter test results for Georgian numbers.
        Add leopard specific versions needed due to font changes (actual characters instead of boxes).

        * css2.1/t1202-counter-09-b-expected.txt:
        * css2.1/t1202-counters-09-b-expected.txt:
        * mac/leopard/Skipped:
        * mac/leopard/css2.1: Added.
        * mac/leopard/css2.1/t1202-counter-09-b-expected.txt: Added.
        * mac/leopard/css2.1/t1202-counters-09-b-expected.txt: Added.
        * mac/leopard/fast/lists: Added.
        * mac/leopard/fast/lists/w3-list-styles-expected.txt: Added.

WebCore:

        Reviewed by Darin.

        <rdar://problem/5228138>
        REGRESSION(Leopard): test failures: tests that test Georgian numbering

        Fix to Georgian number tables to get CSS2.1 test results right. Font has relevant characters now
        which revealed that results were actually wrong.

        * rendering/RenderListMarker.cpp:
        (WebCore::toGeorgian):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24829 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Geoff.
antti [Thu, 2 Aug 2007 21:41:26 +0000 (21:41 +0000)]
    Reviewed by Geoff.

        Fix <rdar://problem/5228233>
        REGRESSION(Leopard): layout test failures: fast/table/absolute-table-at-bottom

        Remove an arrow symbol unrelated to what is being tested. It has different metrics on Leopard.

        * fast/table/absolute-table-at-bottom-expected.checksum:
        * fast/table/absolute-table-at-bottom-expected.png:
        * fast/table/absolute-table-at-bottom-expected.txt:
        * fast/table/absolute-table-at-bottom.html:
        * mac/leopard/Skipped:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24828 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Tim
beidson [Thu, 2 Aug 2007 20:08:15 +0000 (20:08 +0000)]
    Reviewed by Tim

        <rdar://problem/5381463> - setMIMETypesShownAsHTML mutates while enumerating

        * WebView/WebView.mm:
        (+[WebView setMIMETypesShownAsHTML:]): Copy the dictionary before we work with it.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24827 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-08-02 Ada Chan <adachan@apple.com>
adachan [Thu, 2 Aug 2007 18:24:05 +0000 (18:24 +0000)]
2007-08-02  Ada Chan  <adachan@apple.com>

        Reviewed by Steve.

WebCore:
        <rdar://problem/5079175> Added parameters headerHeight and footerHeight to
        computePageRectsForFrame() so we can account for the header and footer when
        calculating page heights for this frame.

        * bridge/win/FrameWin.cpp:
        (WebCore::computePageRectsForFrame):
        * bridge/win/FrameWin.h:

WebKit/win:
        <rdar://problem/5079175> Printing header and footer

        * Interfaces/IWebUIDelegate.idl: added methods for header/footer drawing.
        * WebFrame.cpp:
        (WebFrame::headerAndFooterHeights): ask client for the header and
        footer heights via IWebUIDelegate2 methods.
        (WebFrame::computePageRects): pass in header and footer heights when
        calculating page rect heights.
        (WebFrame::spoolPages): ask client to draw header and footer via
        IWebUIDelegate2 methods.
        * WebFrame.h:
        * WebKitGraphics.cpp:
        (DrawTextAtPoint): the code assumes color has 4 components - might as well
        assert it.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24826 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoAdd an interface for the useragent string
staikos [Thu, 2 Aug 2007 11:56:59 +0000 (11:56 +0000)]
Add an interface for the useragent string

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24823 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-08-02 Mark Rowe <mrowe@apple.com>
bdash [Thu, 2 Aug 2007 10:01:46 +0000 (10:01 +0000)]
2007-08-02  Mark Rowe  <mrowe@apple.com>

        Layout test fix, rubber-stamped by Oliver.

        * fast/js/resources/string-concatenate-outofmemory.js:  Don't let an out of memory exception during verification cause a failure.
        The exception is a perfectly valid success case as it shows that access the string did not cause a crash.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24822 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-08-02 Mark Rowe <mrowe@apple.com>
bdash [Thu, 2 Aug 2007 09:49:12 +0000 (09:49 +0000)]
2007-08-02  Mark Rowe  <mrowe@apple.com>

        Build fix.

        * kjs/ustring.cpp:
        (KJS::UString::expandedSize):  Use std::numeric_limits<size_t>::max() rather than the non-portable SIZE_T_MAX.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24821 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-08-02 Mark Rowe <mrowe@apple.com>
bdash [Thu, 2 Aug 2007 09:33:22 +0000 (09:33 +0000)]
2007-08-02  Mark Rowe  <mrowe@apple.com>

        Reviewed by Maciej.

        <rdar://problem/5352887> "Out of memory" error during repeated JS string concatenation leaks hundreds of MBs of RAM

        A call to fastRealloc was failing which lead to UString::expandCapacity leaking the buffer it was trying to reallocate.
        It also resulted in the underlying UString::rep having both a null baseString and buf field, which meant that attempting
        to access the contents of the string after the failed memory reallocation would crash.

        A third issue is that expandedSize size was calculating the new length in a way that led to an integer overflow occurring.
        Attempting to allocate a string more than 190,000,000 characters long would fail a the integer overflow would lead to a
        memory allocation of around 3.6GB being attempted rather than the expected 390MB.  Sizes that would lead to an overflow
        are now  returned as zero and callers are updated to treat this as though the memory allocation has failed.

        * kjs/array_object.cpp:
        (ArrayProtoFunc::callAsFunction): Check whether the append failed and raise an "Out of memory" exception if it did.
        * kjs/ustring.cpp:
        (KJS::allocChars): Wrapper around fastMalloc that takes a length in characters.  It will return 0 when asked to allocate a zero-length buffer.
        (KJS::reallocChars): Wrapper around fastRealloc that takes a length in characters.  It will return 0 when asked to allocate a zero-length buffer.
        (KJS::UString::expandedSize): Split the size calculation in two and guard against overflow during each step.
        (KJS::UString::expandCapacity): Don't leak r->buf if reallocation fails.  Instead free the memory and use the null representation.
        (KJS::UString::expandPreCapacity): If fastMalloc fails then use the null representation rather than crashing in memcpy.
        (KJS::UString::UString): If calls to expandCapacity, expandPreCapacity or fastMalloc fail then use the null representation rather than crashing in memcpy.
        (KJS::UString::append): Ditto.
        (KJS::UString::operator=): Ditto.
        * kjs/ustring.h: Change return type of expandedSize from int to size_t.

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

        Reviewed by Maciej.

        <rdar://problem/5352887> "Out of memory" error during repeated JS string concatenation leaks hundreds of MBs of RAM

        Update test to check that accessing the string after the "Out of memory" exception was raised does not crash.

        * fast/js/resources/string-concatenate-outofmemory.js:
        * fast/js/string-concatenate-outofmemory-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24820 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoforgot to add info about the bug being fixed
aliceli1 [Thu, 2 Aug 2007 07:24:13 +0000 (07:24 +0000)]
forgot to add info about the bug being fixed

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24819 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoWebCore:
aliceli1 [Thu, 2 Aug 2007 07:13:31 +0000 (07:13 +0000)]
WebCore:

        Reviewed by Kevin McCullough.

        fixed <rdar://problem/5310312> REGRESSION: javascript is mis-escaped at http://labs.zarate.org/passwd causing bookmarklet to break

        * WebCore.exp:
        expose some calls for WebKit to call.
        * manual-tests/JavaScript-bookmarklets.html: Added.

WebKit:

        Reviewed by Kevin McCullough.

        * Misc/WebNSURLExtras.mm:
        (+[NSURL _web_URLWithUserTypedString:relativeToURL:]):
        (-[NSURL _web_userVisibleString]):
        (-[NSURL _web_URLWithLowercasedScheme]):
        (-[NSURL _web_dataForURLComponentType:]):
        These 4 changes are just casting changes.

        (-[NSString _webkit_stringByReplacingValidPercentEscapes]):
        This change replaces the call to an NSURL method with a webcore one that doesn't abort the escaping effort once an illegal character is encountered.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24818 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Don't delete the stderr file right after creating it
aroben [Thu, 2 Aug 2007 07:01:42 +0000 (07:01 +0000)]
    Don't delete the stderr file right after creating it

        Reviewed by NOBODY.

        * Scripts/run-webkit-tests:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24817 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago When DRT crashes, record stderr and restart DRT
aroben [Thu, 2 Aug 2007 06:24:50 +0000 (06:24 +0000)]
    When DRT crashes, record stderr and restart DRT

        This prevents a DRT crash from causing the next few hundred tests to
        "fail" because DRT is no longer running.

        I also changed the terminology that run-webkit-tests uses in its
        output a bit, so that crashing tests are referred to as "crashes"
        instead of "failures".

        Reviewed by Mark.

        * Scripts/run-webkit-tests: Detect a crash and record it as a tool
        failure.
        (sub openDumpTool): Use open3 so that we can access stderr.
        (sub dumpToolCrashed): Added.
        (sub printFailureMessageForTest): Added.
        (sub htmlForExpectedAndActualResults): Added.
        (sub deleteExpectedAndActualResults): Added.
        (sub recordActualResultsAndDiff): Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24816 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoAdd an interface to manage global history for clients
treat [Thu, 2 Aug 2007 04:21:28 +0000 (04:21 +0000)]
Add an interface to manage global history for clients

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24815 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoDo not call update or repaint from inside a paint event.
treat [Thu, 2 Aug 2007 02:24:33 +0000 (02:24 +0000)]
Do not call update or repaint from inside a paint event.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24814 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Fix build.
andersca [Thu, 2 Aug 2007 01:43:19 +0000 (01:43 +0000)]
    Fix build.

        * Misc/WebNSURLExtras.mm:
        (+[NSURL _web_URLWithUserTypedString:relativeToURL:]):
        (-[NSURL _web_userVisibleString]):
        (-[NSURL _web_URLWithLowercasedScheme]):
        (-[NSURL _web_dataForURLComponentType:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24813 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by .
aliceli1 [Thu, 2 Aug 2007 01:18:37 +0000 (01:18 +0000)]
    Reviewed by .

        Making WebNSURLExtras objc++

        * Misc/WebNSURLExtras.m: Removed.
        * Misc/WebNSURLExtras.mm: Copied from WebKit/Misc/WebNSURLExtras.m.
        * WebKit.xcodeproj/project.pbxproj:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24812 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Anders Carlsson and Kevin Decker.
darin [Thu, 2 Aug 2007 01:15:23 +0000 (01:15 +0000)]
    Reviewed by Anders Carlsson and Kevin Decker.

        - fix <rdar://problem/5377432> Removal of MakeDataExecutable from 64-bit breaks WebKit build

        The trick was to ifdef out more of the code that's only needed to support CFM, which exists
        only for 32-bit PowerPC.

        * Plugins/WebNetscapePluginPackage.h: Define a SUPPORT_CFM symbol in this internal header when
        we support CFM. We support it only on 32-bit PowerPC. Only define the isBundle, isCFM, and
        connID fields when SUPPORT_CFM is on. Also use ResFileRefNum instead of SInt16.
        * Plugins/WebNetscapePluginPackage.m: Only compile the function pointer and transition vector
        functions when SUPPORT_CFM is on.
        (-[WebNetscapePluginPackage openResourceFile]): Put the non-bundle case inside a SUPPORT_CFM
        ifdef, since all non-CFM plug-ins are bundles.
        (-[WebNetscapePluginPackage closeResourceFile:]): Ditto.
        (-[WebNetscapePluginPackage _initWithPath:]): Use SUPPORT_CFM to compile out the code for
        non-bundle and bundle-based CFM plug-ins, and code that sets isBundle and isCFM.
        (-[WebNetscapePluginPackage executableType]): Put the CFM case inside SUPPORT_CFM.
        (-[WebNetscapePluginPackage load]): Put the non-bundle and CFM cases inside SUPPORT_CFM.
        There was a bit of dead code here.
        (-[WebNetscapePluginPackage _unloadWithShutdown:]): Put the non-bundle case inside
        SUPPORT_CFM.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24811 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Justin.
thatcher [Thu, 2 Aug 2007 00:35:29 +0000 (00:35 +0000)]
    Reviewed by Justin.

        <rdar://problem/5376156> Mail crash in DeleteButtonController::hide() when dropping selected image on DIV's border

        Add the container element back so the selection can not touch the deletion UI nodes. The container
        has style to prevent user selection, user drag and user modification.

        * editing/DeleteButtonController.cpp:
        (WebCore::DeleteButtonController::show): Make the container node, and append the button and outline elements.
        (WebCore::DeleteButtonController::hide): Remove the container elements and null out the other nodes.
        * editing/DeleteButtonController.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24810 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Kevin McCullough.
darin [Thu, 2 Aug 2007 00:30:08 +0000 (00:30 +0000)]
    Reviewed by Kevin McCullough.

        - fix <rdar://problem/5375186> pointers to pieces of class definition passed to JSClassCreate should all be const

        * API/JSObjectRef.h: Added const.

        * API/JSClassRef.cpp:
        (OpaqueJSClass::OpaqueJSClass): Added const.
        (OpaqueJSClass::create): Added const.
        * API/JSObjectRef.cpp:
        (JSClassCreate): Added const.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24809 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Build mod: Fix sln to match configs in vcproj.
sfalken [Wed, 1 Aug 2007 21:14:01 +0000 (21:14 +0000)]
    Build mod: Fix sln to match configs in vcproj.

        Reviewed by Adam.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24808 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago fast/encoding/char-encoding.html no longer needs to be in the Leopard skiplist
oliver [Wed, 1 Aug 2007 04:30:42 +0000 (04:30 +0000)]
    fast/encoding/char-encoding.html no longer needs to be in the Leopard skiplist

        * mac/leopard/Skipped:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24803 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Justin.
harrison [Wed, 1 Aug 2007 00:40:23 +0000 (00:40 +0000)]
    Reviewed by Justin.

        <rdar://problem/5362659> CrashTracer: [USER] 11 crashes in Mail at WebCore::InsertLineBreakCommand::doApply()

        The problem was that deleting with the X control leaves the selection inside the fragment that was deleted.

        * editing/DeleteButtonController.cpp:
        (WebCore::DeleteButtonController::deleteTarget):
        Because the deletion UI only appears when the selection is entirely
        within the target, we unconditionally update the selection to be
        a caret where the target had been.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24802 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoLayoutTests:
adele [Wed, 1 Aug 2007 00:36:49 +0000 (00:36 +0000)]
LayoutTests:

        Reviewed by Hyatt.

        Updated tests for <rdar://problem/5339395> REGRESSION:http://sudokucraving.com does not render grid correctly

        * fast/forms/input-appearance-minWidth.html: Removed. This test is covered by fast/replaced/width100percent-textfield.html.
        * fast/forms/input-appearance-minWidth-expected.checksum: Removed.
        * fast/forms/input-appearance-minWidth-expected.png: Removed.
        * fast/forms/input-appearance-minWidth-expected.txt: Removed.

        * fast/forms/input-table-expected.checksum:
        * fast/forms/input-table-expected.png:
        * fast/forms/input-table-expected.txt:
        * fast/replaced/width100percent-menulist-expected.checksum:
        * fast/replaced/width100percent-menulist-expected.png:
        * fast/replaced/width100percent-menulist-expected.txt:
        * fast/replaced/width100percent-searchfield-expected.checksum: Added.
        * fast/replaced/width100percent-searchfield-expected.png: Added.
        * fast/replaced/width100percent-searchfield-expected.txt: Added.
        * fast/replaced/width100percent-searchfield.html: Added.
        * fast/replaced/width100percent-textarea-expected.checksum:
        * fast/replaced/width100percent-textarea-expected.png:
        * fast/replaced/width100percent-textarea-expected.txt:
        * fast/replaced/width100percent-textfield-expected.checksum:
        * fast/replaced/width100percent-textfield-expected.png:
        * fast/replaced/width100percent-textfield-expected.txt:

WebCore:

        Reviewed by Hyatt.

         Fix for <rdar://problem/5339395> REGRESSION:http://sudokucraving.com does not render grid correctly

        * rendering/RenderTextControl.h: (WebCore::RenderTextControl::hasControlClip): Clip for search fields.
        * rendering/RenderTextControl.cpp:
        (WebCore::RenderTextControl::controlClipRect): Added a control clip so the search field's cancel button and magnifier glass
         never draw outside the control's bounds.
        (WebCore::RenderTextControl::calcPrefWidths): Only include the inner box's padding when calculating the min/max width without using calcContentBoxWidth.
         Our old behavior was causing that inner padding to get counted twice.  Also, no need to add in the border
         for an inner box that can't be controlled from outside this class.
        * rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::calcPrefWidths): ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24801 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Geoff.
andersca [Wed, 1 Aug 2007 00:20:04 +0000 (00:20 +0000)]
    Reviewed by Geoff.

        Speculative fix for <rdar://problem/5359695>
        REGRESSION (Tiger Beta): Multiple crashes in WebCore::Widget::getView() const + 6

        * page/EventHandler.cpp:
        (WebCore::EventHandler::updateDragAndDrop):
        Null check the frame view.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24800 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoWebCore:
thatcher [Tue, 31 Jul 2007 21:17:51 +0000 (21:17 +0000)]
WebCore:

        Reviewed by Oliver and Beth.

        <rdar://problem/5211271> ADOBE Leopard 9A410: At the first Launching InDesign after deactivate, EULA page gets blanked.

        Rename needsAcrobatFrameReloadingQuirk to needsAdobeFrameReloadingQuirk, since this now applies to more Adobe applications.

        * WebCore.exp:
        * page/Settings.cpp:
        (WebCore::Settings::Settings):
        (WebCore::Settings::setNeedsAdobeFrameReloadingQuirk):
        * page/Settings.h:
        (WebCore::Settings::needsAcrobatFrameReloadingQuirk):

WebKit:

        Reviewed by Oliver and Beth.

        <rdar://problem/5211271> ADOBE Leopard 9A410: At the first Launching InDesign after deactivate, EULA page gets blanked.

        Check for more Adobe applications that need the frame reload quirk. Also cache the answer
        so the version check dosen't happen more than once.

        * WebView/WebView.mm:
        (-[WebView _needsAdobeFrameReloadingQuirk]):
        (-[WebView _updateWebCoreSettingsFromPreferences:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24799 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoLayoutTests:
beidson [Tue, 31 Jul 2007 20:29:02 +0000 (20:29 +0000)]
LayoutTests:

        Reviewed by Brady

        Test for http://bugs.webkit.org/show_bug.cgi?id=14757
        HTMLTokenizer::processingData implementation is incorrect

        * http/tests/loading/slow-parsing-subframe-expected.txt: Added.
        * http/tests/loading/slow-parsing-subframe.html: Added.

WebCore:

        Reviewed by Brady and Darin, tweaked by Brady, landed by Brady

        Fix for http://bugs.webkit.org/show_bug.cgi?id=14757 and <rdar://problem/5364692>
        HTMLTokenizer::processingData implementation is incorrect

        * html/HTMLTokenizer.cpp:
        (WebCore::HTMLTokenizer::processingData): Made it also return true if the HTMLTokenizer was inside the write() call.
        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::FrameLoader):
        (WebCore::FrameLoader::checkLoadCompleteTimerFired):
        (WebCore::FrameLoader::scheduleCheckLoadComplete):
        (WebCore::FrameLoader::stopForUserCancel): Changed a call to checkLoadComplete to be asynchronous, since
        stopForUserCancel can be called while parsing.
        * loader/FrameLoader.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24798 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Oliver.
adele [Tue, 31 Jul 2007 19:54:51 +0000 (19:54 +0000)]
    Reviewed by Oliver.

        Fix for <rdar://problem/5308020> REGRESSION: Command-N with Dvorak-Qwerty keyboard layout stopped working inside web page text fields

        * WebView/WebHTMLView.mm: (-[WebHTMLView _handleStyleKeyEquivalent:]):
        The input method may have modified the character we get, so don't use charactersIgnoringModifiers to interpret the character we get.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24797 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Geoff.
andersca [Tue, 31 Jul 2007 19:08:04 +0000 (19:08 +0000)]
    Reviewed by Geoff.

        <rdar://problem/5371582>
        REGRESSION: PLT .5% slower due to r24451 (copying HTMLCollection objects)

        Make the hash maps store CollectionInfo pointers to reduce amount of copying when
        inserting/rehashing etc.

        * dom/Document.cpp:
        (WebCore::Document::~Document):
        (WebCore::Document::nameCollectionInfo):
        * dom/Document.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24796 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Beth.
adele [Tue, 31 Jul 2007 18:38:05 +0000 (18:38 +0000)]
    Reviewed by Beth.

        Updated this test to force display to happen earlier. This works around a bug in DRT that draws the scrollbars in a different way from Safari.

        * fast/replaced/width100percent-textarea-expected.checksum:
        * fast/replaced/width100percent-textarea-expected.png:
        * fast/replaced/width100percent-textarea.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24795 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Sam.
adele [Tue, 31 Jul 2007 18:27:21 +0000 (18:27 +0000)]
    Reviewed by Sam.

        Breaking up fast/replaced/width100percent.html into separate tests for the different elements.

        * fast/replaced/width100percent-button-expected.checksum: Added.
        * fast/replaced/width100percent-button-expected.png: Added.
        * fast/replaced/width100percent-button-expected.txt: Added.
        * fast/replaced/width100percent-button.html: Added.
        * fast/replaced/width100percent-checkbox-expected.checksum: Added.
        * fast/replaced/width100percent-checkbox-expected.png: Added.
        * fast/replaced/width100percent-checkbox-expected.txt: Added.
        * fast/replaced/width100percent-checkbox.html: Added.
        * fast/replaced/width100percent-image-expected.checksum: Added.
        * fast/replaced/width100percent-image-expected.png: Added.
        * fast/replaced/width100percent-image-expected.txt: Added.
        * fast/replaced/width100percent-image.html: Added.
        * fast/replaced/width100percent-menulist-expected.checksum: Added.
        * fast/replaced/width100percent-menulist-expected.png: Added.
        * fast/replaced/width100percent-menulist-expected.txt: Added.
        * fast/replaced/width100percent-menulist.html: Added.
        * fast/replaced/width100percent-radio-expected.checksum: Added.
        * fast/replaced/width100percent-radio-expected.png: Added.
        * fast/replaced/width100percent-radio-expected.txt: Added.
        * fast/replaced/width100percent-radio.html: Added.
        * fast/replaced/width100percent-textarea-expected.checksum: Added.
        * fast/replaced/width100percent-textarea-expected.png: Added.
        * fast/replaced/width100percent-textarea-expected.txt: Added.
        * fast/replaced/width100percent-textarea.html: Added.
        * fast/replaced/width100percent-textfield-expected.checksum: Added.
        * fast/replaced/width100percent-textfield-expected.png: Added.
        * fast/replaced/width100percent-textfield-expected.txt: Added.
        * fast/replaced/width100percent-textfield.html: Added.

        * fast/replaced/width100percent.html: Removed.
        * fast/replaced/width100percent-expected.checksum: Removed.
        * fast/replaced/width100percent-expected.png: Removed.
        * fast/replaced/width100percent-expected.txt: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24794 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoLayoutTests:
weinig [Tue, 31 Jul 2007 17:58:46 +0000 (17:58 +0000)]
LayoutTests:

        Reviewed by Mitz.

        Updated tests for http://bugs.webkit.org/show_bug.cgi?id=14825
        Non-integer hsl() colours are ignored

        * fast/css/hsl-color-expected.checksum:
        * fast/css/hsl-color-expected.png:
        * fast/css/hsl-color-expected.txt:
        * fast/css/hsl-color.html: Add non-integer hue tests
        * fast/css/hsla-color-expected.checksum:
        * fast/css/hsla-color-expected.png:
        * fast/css/hsla-color-expected.txt:
        * fast/css/hsla-color.html: Add non-integer hue tests

WebCore:

        Reviewed by Mitz.

        Fix for http://bugs.webkit.org/show_bug.cgi?id=14825
        Non-integer hsl() colours are ignored

        * css/CSSParser.cpp:
        (WebCore::CSSParser::parseHSLParameters): Parse hue as a Number,
        not an Integer.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24793 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoLayoutTests:
justing [Tue, 31 Jul 2007 03:54:56 +0000 (03:54 +0000)]
LayoutTests:

        Reviewed by Darin.

        <rdar://problem/5369009>
        Crash due to infinite recursion in moveParagraphs deleting line from a Mail message

        * editing/deleting/5369009-expected.checksum: Added.
        * editing/deleting/5369009-expected.png: Added.
        * editing/deleting/5369009-expected.txt: Added.
        * editing/deleting/5369009.html: Added.

WebCore:

        Reviewed by Darin.

        <rdar://problem/5369009> Crash due to infinite recursion in moveParagraphs on delete

        After the delete, we moved content into the previous block, and a style
        rule turned a style span in the moved content into a block, throwing
        moveParagraphs into infinite recursion, as it continually tried and failed
        to get the style span into the same paragraph as the content just before it.

        Added a method to ReplaceSelectionCommand to keep inserted style spans
        from turning into blocks because of style rules.  Will add code to prevent
        other kinds of content from changing appearance because of style rules,
        post-Leopard (5371536). Also added a bool to ReplaceSelectionCommand's
        constructor to guard against infinite recursion.

        * editing/CompositeEditCommand.cpp:
        (WebCore::CompositeEditCommand::moveParagraphs):
        * editing/JSEditor.cpp:
        * editing/ReplaceSelectionCommand.cpp:
        (WebCore::ReplaceSelectionCommand::ReplaceSelectionCommand):
        (WebCore::ReplaceSelectionCommand::negateStyleRulesThatEffectAppearance):
        (WebCore::ReplaceSelectionCommand::doApply):
        * editing/ReplaceSelectionCommand.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24792 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoFix build.
treat [Tue, 31 Jul 2007 03:44:13 +0000 (03:44 +0000)]
Fix build.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24791 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Darin
sullivan [Tue, 31 Jul 2007 03:18:27 +0000 (03:18 +0000)]
    Reviewed by Darin

        - fixed <rdar://problem/5216176> Need WebKit SPI to allow clients using embedded WebViews to avoid
          clipping ends of some printed pages

        This provides Mail, and other clients that print views that embed WebViews, a way to ensure that
        the HTML is laid out for printing before pagination occurs.

        * WebView/WebHTMLViewPrivate.h:
        new SPI method -_layoutForPrinting

        * WebView/WebHTMLView.mm:
        (-[WebHTMLView _web_setPrintingModeRecursiveAndAdjustViewSize]):
        new method, just like existing _web_setPrintingModeRecursive except passes YES for adjustViewSize
        (-[WebHTMLView _layoutForPrinting]):
        new SPI method, sets printing mode temporarily to adjust the view size for printing
        (-[NSView _web_setPrintingModeRecursiveAndAdjustViewSize]):
        new helper method to do the recursion

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24790 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Brady.
andersca [Tue, 31 Jul 2007 00:48:25 +0000 (00:48 +0000)]
    Reviewed by Brady.

        <rdar://problem/5371515>
        BitmapImage::getTIFFRepresentation() bug results in favicons not working at many sites

        Don't bail out if a single frame is invalid.

        * platform/graphics/mac/ImageMac.mm:
        (WebCore::BitmapImage::getTIFFRepresentation):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24789 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoFix a bug number on an old check-in.
darin [Tue, 31 Jul 2007 00:29:59 +0000 (00:29 +0000)]
Fix a bug number on an old check-in.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24788 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Darin.
adele [Tue, 31 Jul 2007 00:09:31 +0000 (00:09 +0000)]
    Reviewed by Darin.

        Fix for <rdar://problem/5367919> A crash occurs at WebCore::Frame::isFrameSet() when attempting to print a iframe before it loads at http://www.monster.com/

        * WebView/WebHTMLView.mm: (-[WebHTMLView knowsPageRange:]): Nil check for frame.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24787 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Darin.
adele [Mon, 30 Jul 2007 23:56:19 +0000 (23:56 +0000)]
    Reviewed by Darin.

        Fix for <rdar://problem/5274937> Safari 3 breaks Missing Sync's WebKit code by crashing

        * loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::continueAfterContentPolicy): Nil check for the FrameLoader.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24786 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoWebCore:
andersca [Mon, 30 Jul 2007 23:24:37 +0000 (23:24 +0000)]
WebCore:

        Reviewed by Darin.

        <rdar://problem/5370710>
        REGRESSION: After switching from Bookmark view, the Find Banner won't appear while displaying a PDF file

        Add a hasHTMLView to FrameLoaderClient. This is only useful for clients who can show different views for different
        MIME types and URL schemes.

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::transitionToCommitted):
        Call delegate methods here if the current view is not a HTML view.

        * loader/FrameLoaderClient.h:
        (WebCore::FrameLoaderClient::hasHTMLView):

WebKit:

        Reviewed by Darin.

        <rdar://problem/5370710>
        REGRESSION: After switching from Bookmark view, the Find Banner won't appear while displaying a PDF file

        Implement hasHTMLView.

        * WebCoreSupport/WebFrameLoaderClient.h:
        * WebCoreSupport/WebFrameLoaderClient.mm:
        (WebFrameLoaderClient::hasHTMLView):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24785 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoHatcher!
darin [Mon, 30 Jul 2007 23:05:37 +0000 (23:05 +0000)]
Hatcher!

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24784 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Tim Matcher.
darin [Mon, 30 Jul 2007 23:05:04 +0000 (23:05 +0000)]
    Reviewed by Tim Matcher.

        * DumpRenderTree/DumpRenderTree.m: (dump): Fix dumping for documents that include null
        characters. This turned out not to be needed for the test case that motivated me to
        do it, but it's nice to have this for the future.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24783 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoFixed a typo in my last checkin.
justing [Mon, 30 Jul 2007 22:29:35 +0000 (22:29 +0000)]
Fixed a typo in my last checkin.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24782 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoLayoutTests:
weinig [Mon, 30 Jul 2007 18:29:29 +0000 (18:29 +0000)]
LayoutTests:

        Reviewed by Maciej.

        Update tests for <rdar://problem/5363896>
        REGRESSION: Setting document.domain does not override port-based cross-frame security checks -- breaks SAP NetWeaver

        - Also changes frame access test to use shared resource iframes.

        * http/tests/security/cross-frame-access-port-expected.txt:
        * http/tests/security/cross-frame-access-port-explicit-domain-expected.txt:
        * http/tests/security/cross-frame-access-port-explicit-domain.html:
        * http/tests/security/cross-frame-access-port.html:
        * http/tests/security/cross-frame-access-protocol-expected.txt:
        * http/tests/security/cross-frame-access-protocol-explicit-domain-expected.txt:
        * http/tests/security/cross-frame-access-protocol-explicit-domain.html:
        * http/tests/security/cross-frame-access-protocol.html:
        * http/tests/security/resources/cross-frame-iframe-for-port-explicit-domain-test.html: Removed.
        * http/tests/security/resources/cross-frame-iframe-for-port-test.html: Removed.
        * http/tests/security/resources/cross-frame-iframe-for-protocol-explicit-domain-test.html: Removed.
        * http/tests/security/resources/cross-frame-iframe-for-protocol-test.html: Removed.
        * http/tests/security/resources/cross-frame-iframe-with-explicit-domain-set.html: Added.
        * http/tests/security/resources/cross-frame-iframe.html: Added.

WebCore:

        Reviewed by Maciej.

        Fix for <rdar://problem/5363896>
        REGRESSION: Setting document.domain does not override port-based cross-frame security checks -- breaks SAP NetWeaver

        Updates the results for:
            http/tests/security/cross-frame-access-port-explicit-domain.html
            http/tests/security/cross-frame-access-protocol-explicit-domain.html

        * bindings/js/kjs_window.cpp:
        (KJS::createWindow): Use the new setDomainInternal method.
        (KJS::Window::isSafeScript): Don't set check the port or protocol
        if both documents have explicitly set document.domain in the DOM
        and the those domains are equal.
        * dom/Document.cpp:
        (WebCore::Document::Document):
        (WebCore::Document::setDomain): Remove force parameter and change
        all calls that relied on it use the new setDomainInternal method
        which does the same thing.
        (WebCore::Document::setDomainInternal):
        * dom/Document.h: Add new m_domainWasSetInDOM variable and accessor.
        (WebCore::Document::domainWasSetInDOM):
        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::checkCallImplicitClose): Use the new setDomainInternal method.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24781 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoWebCore:
justing [Mon, 30 Jul 2007 18:25:19 +0000 (18:25 +0000)]
WebCore:

        Reviewed by Tristan.

        <rdar://problem/5098931> Attachments are lost when they are moved into a ToDo after a delete

        Added an editor client method for asking whether or not it is
        OK to merge content after a delete.

        * bridge/EditorClient.h:
        * editing/DeleteSelectionCommand.cpp:
        (WebCore::DeleteSelectionCommand::mergeParagraphs): Ask if the
        merge is allowed.
        * platform/gdk/EditorClientGdk.cpp:
        (WebCore::EditorClientGdk::shouldMoveRangeAfterDelete): Added a method stub.
        * platform/gdk/EditorClientGdk.h:
        * platform/graphics/svg/SVGImageEmptyClients.h:
        (WebCore::SVGEmptyEditorClient::shouldMoveRangeAfterDelete): Ditto.

WebKit:

        Reviewed by Tristan.

        <rdar://problem/5098931> Attachments are lost when they are moved into a ToDo after a delete

        Mail needs to be asked if it is OK to do the content movement that happens
        after a deleting in a situation like this one:
        <div contenteditable="plaintext-only">foo</div><div>^bar</div>
        so that they can prevent the move or so that they can save content that
        will be stripped by the move. This could have been done with shouldInsertNode
        and a new WebViewInsertAction for "moves", but WebKit clients like Mail and
        DashCode think that a shouldInsert* means that the user pasted something and
        perform actions only appropriate for pastes.  This change is less risky because
        it won't require those clients to change their code.

        * DefaultDelegates/WebDefaultEditingDelegate.m:
        (-[WebDefaultEditingDelegate webView:shouldMoveRangeAfterDelete:replacingRange:]):
        * WebCoreSupport/WebEditorClient.h:
        * WebCoreSupport/WebEditorClient.mm:
        (WebEditorClient::shouldMoveRangeAfterDelete):
        * WebView/WebEditingDelegatePrivate.h:

win:

        Reviewed by Tristan.

        <rdar://problem/5098931> Attachments are lost when they are moved into a ToDo after a delete

        * WebEditorClient.cpp:
        (WebEditorClient::shouldMoveRangeAfterDelete): Method stub.
        * WebEditorClient.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24780 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoLayoutTests:
andersca [Mon, 30 Jul 2007 18:15:32 +0000 (18:15 +0000)]
LayoutTests:

        Reviewed by Anders.

        <rdar://problem/5352526>
        http://bugs.webkit.org/show_bug.cgi?id=14704
        REGRESSION: sync XMLHttpRequest.send() raises an exception for failed authentication

        * http/tests/xmlhttprequest/failed-auth-expected.txt: Added.
        * http/tests/xmlhttprequest/failed-auth.html: Added.
        This test has other cases from bug 13075 commented out, as they still fail.

        * http/tests/xmlhttprequest/xmlhttprequest-missing-file-exception-expected.txt: Added.
        * http/tests/xmlhttprequest/xmlhttprequest-missing-file-exception.html: Added.
        This test already passed, but I think it's useful to cover this case, too.

WebCore:

        Reviewed by Anders.

        <rdar://problem/5352526>
        http://bugs.webkit.org/show_bug.cgi?id=14704
        REGRESSION: sync XMLHttpRequest.send() raises an exception for failed authentication

        Test: http/tests/xmlhttprequest/failed-auth.html

        * platform/network/mac/ResourceHandleMac.mm:
        (WebCore::ResourceHandle::loadResourceSynchronously): Make the fake response better match the real one
        in case of failed authentication. Unfortunately, NSURLConnection doesn't give us the real response.

        * xml/XMLHttpRequest.cpp:
        (WebCore::XMLHttpRequest::send): If the response has an HTTP code in it, then the error wasn't
        a network one, and an exception shouldn't be raised.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24779 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoUpdating layout test results with new expected results.
tristan [Mon, 30 Jul 2007 17:48:51 +0000 (17:48 +0000)]
Updating layout test results with new expected results.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24778 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Darin.
andersca [Mon, 30 Jul 2007 17:28:03 +0000 (17:28 +0000)]
    Reviewed by Darin.

        <rdar://problem/4942372> REGRESSION: Anchor links in Mail don't work anymore

        Make it possible for KURL to handle addig fragments to non-hierarchical URLs
        such as mailto: and cid:.

        * platform/KURL.cpp:
        (WebCore::KURL::init):
        If the base URL is not hierarchical but the relative URL is a fragment, then
        allow parsing it.

        (WebCore::KURL::parse):
        If the URL is not hierarchical, set the fragment start and end positions correctly.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24777 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoOn Windows make sure the dll also ends up in $$OUTPUT_DIR/bin, so that QtLauncher...
hausmann [Mon, 30 Jul 2007 16:10:42 +0000 (16:10 +0000)]
On Windows make sure the dll also ends up in $$OUTPUT_DIR/bin, so that QtLauncher can find it.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24776 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoLink QtLauncher into $$OUTPUT_DIR/bin
hausmann [Mon, 30 Jul 2007 16:10:29 +0000 (16:10 +0000)]
Link QtLauncher into $$OUTPUT_DIR/bin

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24775 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoRemoved the __BUILDING_QT ifdef in JSStringRef.h and changed UChar for the Qt build...
hausmann [Mon, 30 Jul 2007 15:30:37 +0000 (15:30 +0000)]
Removed the __BUILDING_QT ifdef in JSStringRef.h and changed UChar for the Qt build to use wchar_t on Windows.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24774 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoDon't link against libxml/libxslt unless we really need it.
hausmann [Mon, 30 Jul 2007 15:30:21 +0000 (15:30 +0000)]
Don't link against libxml/libxslt unless we really need it.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24773 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Oliver Hunt.
tristan [Mon, 30 Jul 2007 06:33:54 +0000 (06:33 +0000)]
    Reviewed by Oliver Hunt.

        <rdar://problem/5369282> REGRESSION: DOMHTMLInput throwing exception due to missing _isEdited method

        * bindings/objc/DOMHTML.mm:
        (-[DOMHTMLInputElement _isEdited]):
        (-[DOMHTMLTextAreaElement _isEdited]):
        * bindings/objc/DOMPrivate.h:
        Renamed _isUserEdited to _isEdited.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24771 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-07-30 Diego Escalante Urrelo <diegoe@gnome.org>
alp [Mon, 30 Jul 2007 04:46:14 +0000 (04:46 +0000)]
2007-07-30  Diego Escalante Urrelo  <diegoe@gnome.org>

        Reviewed by Adam.

        http://bugs.webkit.org/show_bug.cgi?id=14806
        Implement can_go_backward and can_go_forward in webkitgtkpage.cpp

        * gtk/Api/webkitgtkpage.cpp: Implement webkit_gtk_page_can_go_backward() and
        webkit_gtk_page_can_go_forward() functions.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24770 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-07-30 Diego Escalante Urrelo <diegoe@gnome.org>
alp [Mon, 30 Jul 2007 04:07:14 +0000 (04:07 +0000)]
2007-07-30  Diego Escalante Urrelo  <diegoe@gnome.org>

        Reviewed by Alp Toker.

        http://bugs.webkit.org/show_bug.cgi?id=14810
        Bad n_params on load_finished's g_signal_new in webkitgtkpage.cpp

        This is the proper fix for #14810.

        * gtk/Api/webkitgtkframe.cpp:
        * gtk/Api/webkitgtkpage.cpp: Fix the marshaller type of the "load_finished" signal and
        correct the class_offset parameters.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24769 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-07-29 Diego Escalante Urrelo <diegoe@gnome.org>
alp [Mon, 30 Jul 2007 03:12:21 +0000 (03:12 +0000)]
2007-07-29  Diego Escalante Urrelo  <diegoe@gnome.org>

        Reviewed by Alp Toker.

        http://bugs.webkit.org/show_bug.cgi?id=14810
        Bad n_params on load_finished's g_signal_new in webkitgtkpage.cpp

        * gtk/Api/webkitgtkpage.cpp: Use a correct n_params value and the corresponding types for
        them in the "load_finished" signal.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24768 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoPaint and scrolling performance fixes and improvements.
treat [Mon, 30 Jul 2007 02:40:26 +0000 (02:40 +0000)]
Paint and scrolling performance fixes and improvements.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24767 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoWebCore:
adele [Sun, 29 Jul 2007 23:25:17 +0000 (23:25 +0000)]
WebCore:

        Reviewed by John.

        WebCore part of fix for <rdar://problem/5102522> REGRESSION: Can't tab to webview that doesn't have editable content (affects Safari, preview pane in Mail)

        * WebCore.exp:
        * page/FocusController.cpp:
        (WebCore::FocusController::setInitialFocus): Added.
        (WebCore::FocusController::advanceFocus): Don't give the focus back to the chrome if this is the initial focus.
        * page/FocusController.h:

WebKit:

        Reviewed by John.

        WebKit part of fix for <rdar://problem/5102522> REGRESSION: Can't tab to webview that doesn't have editable content

        * WebView/WebHTMLView.mm: (-[WebHTMLView becomeFirstResponder]): Call new setInitialFocus method instead of advanceFocus.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24766 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-07-29 Brian Mastenbrook <brian@mastenbrook.net>
zecke [Sun, 29 Jul 2007 21:51:41 +0000 (21:51 +0000)]
2007-07-29  Brian Mastenbrook  <brian@mastenbrook.net>

        Build fix for WebKit/Gtk to compile on Fedora 7 as described in
        http://bugs.webkit.org/show_bug.cgi?id=14557

        * WebKit.pri: cairo-ft is used by the WebKit/Gtk port

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24765 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoRespect the margins when creating frames.
treat [Sun, 29 Jul 2007 15:18:26 +0000 (15:18 +0000)]
Respect the margins when creating frames.
Set the scroll mode to always off like we did before the rendered
frames patch and the other ports do now.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24764 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoWebCore::Widget::setParent should be virtual.
treat [Sun, 29 Jul 2007 14:42:01 +0000 (14:42 +0000)]
WebCore::Widget::setParent should be virtual.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24763 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoWe actually pass the mac versions expected results here.
treat [Sun, 29 Jul 2007 14:22:47 +0000 (14:22 +0000)]
We actually pass the mac versions expected results here.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24762 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Mark Rowe.
weinig [Sun, 29 Jul 2007 02:34:12 +0000 (02:34 +0000)]
    Reviewed by Mark Rowe.

        - Rename test for clarity and consistency.
        - Move resource from ssl/resources to security/resources as it is
          only being used by the security tests.

        * http/tests/security/cross-frame-access-port-domain-expected.txt: Removed.
        * http/tests/security/cross-frame-access-port-domain.html: Removed.
        * http/tests/security/cross-frame-access-port-explicit-domain-expected.txt: Copied from http/tests/security/cross-frame-access-port-domain-expected.txt.
        * http/tests/security/cross-frame-access-port-explicit-domain.html: Copied from http/tests/security/cross-frame-access-port-domain.html.
        * http/tests/security/cross-frame-access-protocol-expected.txt:
        * http/tests/security/cross-frame-access-protocol.html:
        * http/tests/security/resources/cross-frame-iframe-for-port-domain-test.html: Removed.
        * http/tests/security/resources/cross-frame-iframe-for-port-explicit-domain-test.html: Copied from http/tests/security/resources/cross-frame-iframe-for-port-domain-test.html.
        * http/tests/security/resources/cross-frame-iframe-for-protocol-test.html: Copied from http/tests/ssl/resources/cross-frame-access-protocol-iframe.html.
        * http/tests/ssl/resources: Removed.
        * http/tests/ssl/resources/cross-frame-access-protocol-iframe.html: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24761 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago Reviewed by Oliver Hunt.
weinig [Sun, 29 Jul 2007 02:15:18 +0000 (02:15 +0000)]
    Reviewed by Oliver Hunt.

        - Factor access testing code used in protocol and port tests to cross-frame-access.js.
        - Add cross-frame-iframe-for-protocol-explicit-domain-test.html which is parallel test to
          cross-frame-iframe-for-port-domain-test.html but for different protocols.

        * http/tests/security/cross-frame-access-port-domain-expected.txt:
        * http/tests/security/cross-frame-access-port-domain.html:
        * http/tests/security/cross-frame-access-port.html:
        * http/tests/security/cross-frame-access-protocol-explicit-domain-expected.txt: Added.
        * http/tests/security/cross-frame-access-protocol-explicit-domain.html: Added.
        * http/tests/security/cross-frame-access-protocol.html:
        * http/tests/security/resources/cross-frame-access.js:
        * http/tests/security/resources/cross-frame-iframe-for-port-domain-test.html:
        * http/tests/security/resources/cross-frame-iframe-for-port-test.html:
        * http/tests/security/resources/cross-frame-iframe-for-protocol-explicit-domain-test.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24760 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years agoFix up reviewer name typo in ChangeLog
alp [Sun, 29 Jul 2007 02:02:38 +0000 (02:02 +0000)]
Fix up reviewer name typo in ChangeLog

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24759 268f45cc-cd09-0410-ab3c-d52691b4dbfc

15 years ago2007-07-29 Alp Toker <alp.toker@collabora.co.uk>
alp [Sun, 29 Jul 2007 01:30:13 +0000 (01:30 +0000)]
2007-07-29  Alp Toker  <alp.toker@collabora.co.uk>

        Reviewed by Brian Dash.

        http://bugs.webkit.org/show_bug.cgi?id=14711 (revisited)
        RenderThemeGdk's buttons are state-agnostic (pressed, hovered)

        Generalize shadow state into a function.
        use a GtkHBox to work around a theme bug (thanks to Nigel Tao).

        * platform/gdk/RenderThemeGdk.cpp:
        (WebCore::RenderThemeGdk::determineShadow):
        (WebCore::RenderThemeGdk::paintCheckbox):
        (WebCore::RenderThemeGdk::paintRadio):
        (WebCore::RenderThemeGdk::paintButton):
        (WebCore::RenderThemeGdk::gtkWindowContainer):
        * platform/gdk/RenderThemeGdk.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24758 268f45cc-cd09-0410-ab3c-d52691b4dbfc