⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Aug 4, 2007:

10:20 PM Changeset in webkit [24873] by mjs
  • 3 edits in trunk/JavaScriptCore

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
9:21 PM Changeset in webkit [24872] by aroben
  • 2 edits in trunk/WebCore

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.
8:04 PM Changeset in webkit [24871] by aroben
  • 2 edits in trunk/WebCore

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.
7:15 PM Changeset in webkit [24870] by oliver
  • 1 edit
    3 adds in trunk/LayoutTests

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.
7:06 PM Changeset in webkit [24869] by ddkilzer
  • 4 edits in trunk/WebCore

WebCore:

Reviewed by Oliver.

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:
1:58 AM Changeset in webkit [24868] by bdash
  • 4 edits in trunk/JavaScriptCore

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):
12:37 AM Changeset in webkit [24867] by beidson
  • 1 edit in trunk/WebKit/WebView/WebView.mm

Added the radar # to my FIXME text

12:29 AM Changeset in webkit [24866] by beidson
  • 6 edits in trunk

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:]):

Aug 3, 2007:

8:57 PM Changeset in webkit [24865] by aroben
  • 2 edits in trunk/WebKitTools

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.
6:09 PM Changeset in webkit [24864] by adele
  • 2 edits in trunk/WebCore

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):
5:22 PM Changeset in webkit [24863] by weinig
  • 12 edits in trunk

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:
4:53 PM Changeset in webkit [24862] by andersca
  • 7 edits in trunk/WebCore

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.
4:07 PM Changeset in webkit [24861] by andersca
  • 2 edits in trunk/WebCore

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.
3:37 PM Changeset in webkit [24860] by oliver
  • 3 edits in trunk/WebCore

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):
3:17 PM Changeset in webkit [24859] by sfalken
  • 1 copy in tags/Safari-522.16.1b

New tag.

3:17 PM Changeset in webkit [24858] by sfalken
  • 1 edit in trunk/WebKit/win/WebKit.vcproj/VERSION

Bump version for submit

3:15 PM Changeset in webkit [24857] by adele
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Adele.

  • 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.

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

  • html/HTMLOptionElement.cpp: (WebCore::HTMLOptionElement::setSelected): Reordered to allow setSelectedIndex() to call setChanged().
2:42 PM Changeset in webkit [24856] by spadma
  • 7 edits in S60/trunk/WebKit

2007-07-31 vmalaiya, <​vikram.malaiya@nokia.com>

Reviewed by Sachin
DESC: AKUR-74Q7EL - Browser crashes when Vote button selected on second window
​http://bugs.webkit.org/show_bug.cgi?id=14829

  • BrowserView/inc/FormDataLoginStore.h:
  • BrowserView/inc/FormDataManager.h:
  • BrowserView/src/FormDataLoginStore.cpp: (CFormDataLoginStore::CLoginDataElement::~CLoginDataElement): (CFormDataLoginStore::NewL): (CFormDataLoginStore::ConstructL): (CFormDataLoginStore::GetDataL): (CFormDataLoginStore::SaveDataL): (CFormDataLoginStore::SaveDialogL):
  • BrowserView/src/FormDataManager.cpp: (CFormDataManager::NewL): (CFormDataManager::ConstructL): (CFormDataManager::SaveDataL):
  • BrowserView/src/WebKitControl.cpp: (CStaticObjectContainer::FormDataManagerHandlerL):
  • BrowserView/src/webkitbridge.cpp: (CWebKitBridge::SaveFormData):
2:41 PM Changeset in webkit [24855] by spadma
  • 3 edits in S60/trunk/WebKit

2007-07-24 vmalaiya, <​vikram.malaiya@nokia.com>

Reviewed by Sachin
DESC: VMAA-75EP3L - Browser plugin does not implement memory collection on an out-of-memory condition
​http://bugs.webkit.org/show_bug.cgi?id=14753


  • Plugin/inc/PluginSkin.h:
  • Plugin/src/PluginSkin.cpp: (COOMPluginCollector::COOMPluginCollector): (COOMPluginCollector::~COOMPluginCollector): (COOMPluginCollector::Collect): (COOMPluginCollector::Restore): (COOMPluginCollector::Priority): (COOMPluginCollector::IsCollecting): (CPluginSkin::ConstructL): (CPluginSkin::~CPluginSkin): (CPluginSkin::CreatePluginWinL):
2:28 PM Changeset in webkit [24854] by brmorris
  • 2 edits in S60/branches/3.1m/WebKit

brmorris, rs'd by johnshi

DESC: remove generated header from releasables list (merge of r24167 & r24121 to 3.1m)


Turns out some build systems delete the releasables list after the
makmake phase but before the compilation, and hence the generated
header was not found at compile time.



  • group/version.mk: Added.
2:21 PM Changeset in webkit [24853] by spadma
  • 2 edits in S60/trunk/WebKit

yadavall, reviewed by <​sachin.padma@nokia.com>

DESC: Center key press must be sent to Engine
​http://bugs.webkit.org/show_bug.cgi?id=14593

  • BrowserView/src/KeyEventHandler.cpp: (CKeyEventHandler::HandleOfferKeyEventL):
2:04 PM Changeset in webkit [24852] by spadma
  • 2 edits in S60/trunk/WebKit

lebril, reviewed by Sachin

DESC: ELFN-7535HR: Operator manu: Both time stamps in Link-1 and Link-2 are same when opening link is in Expires Date header 1A iteam
​http://bugs.webkit.org/show_bug.cgi?id=

  • ResourceLoader/CacheSrc/HttpCacheUtil.cpp:
1:09 PM Changeset in webkit [24851] by andersca
  • 2 edits in trunk/WebCore

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):
11:46 AM Changeset in webkit [24850] by bdash
  • 2 edits in trunk/WebCore

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

Windows build fix.

  • rendering/RenderTextControl.cpp: Don't use the same name for two arguments.
11:19 AM Changeset in webkit [24849] by bdash
  • 2 edits in trunk/JavaScriptCore

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

Rubber-stamped by Anders.

  • pcre/pcre_compile.c: Remove non-ASCII character from a comment.
11:03 AM Changeset in webkit [24848] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Mark Rowe.

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

  • WebKit.xcodeproj/project.pbxproj:
10:58 AM Changeset in webkit [24847] by bdash
  • 4 edits in trunk

Bump versioning to avoid 4. issues.

10:20 AM Changeset in webkit [24846] by zbujtas
  • 3 edits in S60/trunk/WebKit

2007-08-02 yadavall <​sriram.yadavalli@nokia.com>

Reviewed by Zalan Bujtas (​zbujtas@gmail.com).
DESC: Widget Cascade menu not in FIFO order
​http://bugs.webkit.org/show_bug.cgi?id=14866

10:18 AM Changeset in webkit [24845] by zbujtas
  • 4 edits in S60/branches/3.1m

2007-08-02 yadavall <​sriram.yadavalli@nokia.com>

Reviewed by Zalan Bujtas (​zbujtas@gmail.com).
DESC: Widget Cascade menu not in FIFO order
​http://bugs.webkit.org/show_bug.cgi?id=14866

9:31 AM Changeset in webkit [24844] by yongjzha
  • 2 edits in S60/branches/3.1m/WebCore

w3liu, reviewed by <​yongjun.zhang@nokia.com>

DESC: FPER-74HDN2: Strange character are shown instead of Chinese on www.sina.com.cn
​http://bugs.webkit.org/show_bug.cgi?id=14830

WARNING: NO TEST CASES ADDED OR CHANGED

  • khtml/khtml_part.cpp: (KHTMLPart::write):
9:21 AM Changeset in webkit [24843] by bdash
  • 7 edits
    3 adds in trunk/JavaScriptCore

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.
7:29 AM Changeset in webkit [24842] by bdash
  • 3 edits
    2 adds in trunk

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

Reviewed by Darin.

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.

  • fast/forms/search-click-in-placeholder-expected.txt: Added.
  • fast/forms/search-click-in-placeholder.html: Added.
4:57 AM Changeset in webkit [24841] by bdash
  • 3 edits in tags/Safari-5522.13/WebCore

Merge r24810.

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:
4:33 AM Changeset in webkit [24840] by bdash
  • 4 edits in trunk

Versioning.

4:32 AM Changeset in webkit [24839] by bdash
  • 1 copy in tags/Safari-5522.13

New tag.

12:21 AM Changeset in webkit [24838] by aroben
  • 3 edits in trunk/LayoutTests

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.
12:21 AM Changeset in webkit [24837] by aroben
  • 3 edits in trunk/LayoutTests

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.
12:21 AM Changeset in webkit [24836] by aroben
  • 2 edits in trunk/WebKitTools

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)"
Note: See TracTimeline for information about the timeline view.