Timeline



Jul 4, 2006:

11:17 PM Changeset in webkit [15159] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Maciej.

Bug 9731: [Drosera] crash when trying to access the scope chain
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9731

Because of <rdar://problem/4608404> the WebScriptObject, _globalObj, that
WebCoreScriptDebugger holds is unprotected each time the page changes.
This causes Drosera to crash Safari when trying to access the scope chain.
We simply need to detach and re-attach the debugger when the window script
object is cleared until 4608404 is fixed. This change also attaches the
debugger before we call the windowScriptObjectAvailable: delegate method,
so the debugger is ready before anyone might use the window object.

  • WebCoreSupport/WebFrameBridge.m: (-[WebFrameBridge windowObjectCleared]):
9:52 PM Changeset in webkit [15158] by mjs
  • 67 edits in trunk

LayoutTests:

Reviewed by Darin.


​http://bugzilla.opendarwin.org/show_bug.cgi?id=9734

  • change SVG XPath DOM tests to dump as text, and update expected results


This should stop the random failures in Attribute_Nodes.svg

  • dom/svg/level3/xpath/Attribute_Nodes-expected.txt:
  • dom/svg/level3/xpath/Attribute_Nodes_xmlns-expected.txt:
  • dom/svg/level3/xpath/Comment_Nodes-expected.txt:
  • dom/svg/level3/xpath/Conformance_Expressions-expected.txt:
  • dom/svg/level3/xpath/Conformance_hasFeature_3-expected.txt:
  • dom/svg/level3/xpath/Conformance_hasFeature_empty-expected.txt:
  • dom/svg/level3/xpath/Conformance_hasFeature_null-expected.txt:
  • dom/svg/level3/xpath/Conformance_isSupported_3-expected.txt:
  • dom/svg/level3/xpath/Conformance_isSupported_empty-expected.txt:
  • dom/svg/level3/xpath/Conformance_isSupported_null-expected.txt:
  • dom/svg/level3/xpath/Element_Nodes-expected.txt:
  • dom/svg/level3/xpath/Processing_Instruction_Nodes-expected.txt:
  • dom/svg/level3/xpath/Text_Nodes-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluatorCast01-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_createExpression_INVALID_EXPRESSION_ERR-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_createExpression_NAMESPACE_ERR_01-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_createExpression_NAMESPACE_ERR_02-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_createExpression_NS-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_createExpression_no_NS-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_createNSResolver_all-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_createNSResolver_document-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_createNSResolver_documentElement-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_evaluate_INVALID_EXPRESSION_ERR-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_evaluate_NAMESPACE_ERR-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_evaluate_NOT_SUPPORTED_ERR-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_evaluate_TYPE_ERR-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_evaluate_WRONG_DOCUMENT_ERR-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_evaluate_document-expected.txt:
  • dom/svg/level3/xpath/XPathEvaluator_evaluate_documentElement-expected.txt:
  • dom/svg/level3/xpath/XPathExpression_evaluate_NOT_SUPPORTED_ERR-expected.txt:
  • dom/svg/level3/xpath/XPathExpression_evaluate_WRONG_DOCUMENT_ERR-expected.txt:
  • dom/svg/level3/xpath/XPathExpression_evaluate_document-expected.txt:
  • dom/svg/level3/xpath/XPathExpression_evaluate_documentElement-expected.txt:
  • dom/svg/level3/xpath/XPathNSResolver_lookupNamespaceURI_nist_dmstc-expected.txt:
  • dom/svg/level3/xpath/XPathNSResolver_lookupNamespaceURI_null-expected.txt:
  • dom/svg/level3/xpath/XPathNSResolver_lookupNamespaceURI_prefix-expected.txt:
  • dom/svg/level3/xpath/XPathNSResolver_lookupNamespaceURI_xml-expected.txt:
  • dom/svg/level3/xpath/XPathResult_TYPE_ERR-expected.txt:
  • dom/svg/level3/xpath/XPathResult_booleanValue_false-expected.txt:
  • dom/svg/level3/xpath/XPathResult_booleanValue_true-expected.txt:
  • dom/svg/level3/xpath/XPathResult_invalidIteratorState_ANY_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_invalidIteratorState_ANY_UNORDERED_NODE_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_invalidIteratorState_BOOLEAN_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_invalidIteratorState_FIRST_ORDERED_NODE_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_invalidIteratorState_NUMBER_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_invalidIteratorState_ORDERED_NODE_ITERATOR_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_invalidIteratorState_ORDERED_NODE_SNAPSHOT_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_invalidIteratorState_STRING_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_invalidIteratorState_UNORDERED_NODE_ITERATOR_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_invalidIteratorState_UNORDERED_NODE_SNAPSHOT_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_iterateNext_INVALID_STATE_ERR-expected.txt:
  • dom/svg/level3/xpath/XPathResult_iteratorNext_ORDERED_NODE_ITERATOR_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_numberValue-expected.txt:
  • dom/svg/level3/xpath/XPathResult_resultType-expected.txt:
  • dom/svg/level3/xpath/XPathResult_singleNodeValue_ANY_UNORDERED_NODE_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_singleNodeValue_FIRST_ORDERED_NODE_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_snapshotItem_ORDERED_NODE_SNAPSHOT_TYPE_null-expected.txt:
  • dom/svg/level3/xpath/XPathResult_snapshotItem_ORDERED_NODE_SNAPSHOT_TYPE_order-expected.txt:
  • dom/svg/level3/xpath/XPathResult_snapshotItem_UNORDERED_NODE_SNAPSHOT_TYPE_count-expected.txt:
  • dom/svg/level3/xpath/XPathResult_snapshotItem_UNORDERED_NODE_SNAPSHOT_TYPE_null-expected.txt:
  • dom/svg/level3/xpath/XPathResult_snapshotLength_ORDERED_NODE_SNAPSHOT_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_snapshotLength_UNORDERED_NODE_SNAPSHOT_TYPE-expected.txt:
  • dom/svg/level3/xpath/XPathResult_stringValue-expected.txt:
  • dom/svg/level3/xpath/svgunit.js:

WebKitTools:

Reviewed by Darin.


​http://bugzilla.opendarwin.org/show_bug.cgi?id=9734

  • add support for dumping non-HTML as text - in this case use textContent instead of innerText
  • DumpRenderTree/DumpRenderTree.m: (dump):
9:34 PM Changeset in webkit [15157] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Darin.

Bug 9732: [Drosera] calling removeListener to many times will cause

WebKit's listener count to underflow/wraparound

​http://bugzilla.opendarwin.org/show_bug.cgi?id=9732

Adds a check to make sure the listener was in our listeners set before
decrementing the global listener count. Also checks for nil in addListner
to prevent a possible exception when adding the object to the set.

  • DefaultDelegates/WebScriptDebugServer.m: (-[WebScriptDebugServer addListener:]): (-[WebScriptDebugServer removeListener:]):
9:25 PM Changeset in webkit [15156] by darin
  • 2 edits in trunk/JavaScriptCore
  • fixed build
  • wtf/MathExtras.h: Oops. Added missing #endif.
9:13 PM Changeset in webkit [15155] by darin
  • 7 edits in trunk/JavaScriptCore

Reviewed by Maciej.
Tweaked a bit by Darin.

  • wtf/MathExtras.h: Added include of <float.h>. (isinf): Fix to return false for NAN. (wtf_fmod): Added. An inline that works around the bug.
  • kjs/nodes.cpp:
  • kjs/number_object.cpp:
  • kjs/operations.cpp:
  • kjs/value.cpp: Added includes of MathExtras.h to all files using fmod.
  • JavaScriptCore.xcodeproj/project.pbxproj: Let Xcode 2.3 have its way with the project.
9:04 PM Changeset in webkit [15154] by darin
  • 10 edits
    3 adds in trunk/WebCore

Reviewed by Maciej, reworked a bit by Darin.

Compile fixes for wx port / gcc 4.0.2


  • platform/PlatformString.h:
  • platform/StringImpl.h: Include CoreFoundation for CF-specific code.


  • dom/DocPtr.h:
  • doc/EventTargetNode.h: Silence warnings about 'extra tokens' after #endif on gcc 4.0.2. (Not an error but warnings appear frequently.)


  • WebCoreSources.bkl:
  • webcore-base.bkl: Synching ports sources file with TOT. Also, making sure SRCDIR variable overrides any previous values.


  • loader/ImageDocument.cpp: Use PLATFORM(MAC) for the parts that are specific to the Macintosh version, and move the Objective-C parts to another file.
  • loader/mac/ImageDocumentMac.h: Added.
  • loader/mac/ImageDocumentMac.mm: Added.
6:05 PM Changeset in webkit [15153] by darin
  • 2 edits in trunk/LayoutTests
  • updated test result to reflect window without scroll bars

1) this test should not have scroll bars
2) on the buildbot the result is the non-scroll-bar one

  • dom/svg/level3/xpath/Attribute_Nodes-expected.txt: Changed text node width to 800.
4:40 PM Changeset in webkit [15152] by weinig
  • 2 edits in trunk/WebCore

Build fix.

  • platform/Color.cpp: (WebCore::makeRGBAFromHSLA):
3:14 PM Changeset in webkit [15151] by weinig
  • 6 edits
    8 adds in trunk

LayoutTests:

Reviewed by Darin.

  • fast/css/hsl-color-expected.checksum: Added.
  • fast/css/hsl-color-expected.png: Added.
  • fast/css/hsl-color-expected.txt: Added.
  • fast/css/hsl-color.html: Added.
  • fast/css/hsla-color-expected.checksum: Added.
  • fast/css/hsla-color-expected.png: Added.
  • fast/css/hsla-color-expected.txt: Added.
  • fast/css/hsla-color.html: Added.

WebCore:

Reviewed by Darin.


  • css/cssparser.cpp: (WebCore::CSSParser::parseColorParameters): Moved common code to parse RGB/RGBA values into it's own function. This code was duplicated in the parseColorFromValue() method. (WebCore::CSSParser::parseHSLParameters): Common code to parse HSL/HSLA values. (WebCore::CSSParser::parseColorFromValue): Moved RGB/RGBA parsing out to seperate function as the code was duplicated. Added parsing support for hsl( and hsla(
  • css/cssparser.h: Added two new functions parseColorParameters() and parseHSLParameters().
  • platform/Color.cpp: (WebCore::calcHue): Added Hue parameter to RGB color. (WebCore::makeRGBAFromHSLA): New function to convert HSLA to RGBA.
  • platform/Color.h: Added new functions makeRGBAFromHSLA() and calcHue().
9:45 AM Changeset in webkit [15150] by ap
  • 6 edits
    3 adds in trunk

Reviewed by Maciej.

Test: http/tests/xmlhttprequest/cache-override.html

  • Misc/WebNSURLRequestExtras.h: Added _web_isConditionalRequest
  • Misc/WebNSURLRequestExtras.m: (-[NSURLRequest _web_isConditionalRequest]):
  • WebCoreSupport/WebFrameBridge.m: (-[WebFrameBridge syncLoadResourceWithMethod:URL:customHeaders:postData:finalURL:responseHeaders:statusCode:]): Bypass the cache for conditional requests.
  • WebCoreSupport/WebSubresourceLoader.m: (+[WebSubresourceLoader startLoadingResource:withRequest:customHeaders:referrer:forDataSource:]): Ditto.

Jul 3, 2006:

7:35 PM Changeset in webkit [15149] by ggaren
  • 13 edits in trunk/JavaScriptCore

Reviewed by Darin.


  • Refined value conversions in the API:
    • failed toNumber returns NaN
    • failed toObject returns NULL
    • failed toString returns empty string


  • Refined excpetion handling in the API:
    • failed value conversions do not throw exceptions
    • uncaught exceptions in JSEvaluate, JSObjectCallAsFunction, and JSObjectCallAsConstructor are returned through a JSValueRef* exception argument
    • removed JSContextHasException, because JSContextGetException does the same job


  • API/JSBase.h:
  • API/JSCharBufferRef.cpp: (JSValueCopyStringValue):
  • API/JSContextRef.cpp: (JSEvaluate):
  • API/JSContextRef.h:
  • API/JSNodeList.c: Added test code demonstrating how you would use toNumber, and why you probably don't need toUInt32, etc. (JSNodeListPrototype_item): (JSNodeList_getProperty):
  • API/JSObjectRef.cpp: (JSValueToObject): (JSObjectCallAsFunction): (JSObjectCallAsConstructor):
  • API/JSObjectRef.h:
  • API/JSValueRef.cpp: (JSValueToNumber):
  • API/JSValueRef.h:
  • API/minidom.c: (main):
  • API/testapi.c: (main): Added tests for new rules, and call to JSGCProtect to fix Intel crash
  • JavaScriptCore.exp:
6:42 PM Changeset in webkit [15148] by darin
  • 2 edits in trunk/JavaScriptCore
  • Rolled out HashMap implementation of NPRuntime, at least temporarily.

Fixes hang in the bindings section of layout tests seen on the
buildbot.

This code was using HashMap<const char*, PrivateIdentifier*>.
But that hashes based on pointer identity, not string value.
The default hash for any pointer type is to hash based on the pointer.
And WTF doesn't currently have a string hash for char*.
We'll need to fix that before re-landing this patch.

(Formatting was also incorrect -- extra spaces in parentheses.)

  • bindings/npruntime.cpp: Rolled out last change.
11:46 AM Changeset in webkit [15147] by thatcher
  • 1 edit in trunk/WebKit/Misc/WebNSDataExtras.m

Build fix.

10:36 AM Changeset in webkit [15146] by ggaren
  • 3 edits in trunk/JavaScriptCore

Reviewed, tweaked, landed by ggaren.

  • Port NPRuntime from CFDictionary to HashMap.
  • bindings/npruntime.cpp: (getStringIdentifierDictionary): (getIntIdentifierDictionary): (_NPN_GetStringIdentifier): (_NPN_GetIntIdentifier):
  • bindings/npruntime.h:
1:27 AM Changeset in webkit [15145] by thatcher
  • 3 edits in trunk/WebKitTools/Drosera/English.lproj/MainMenu.nib

Forgot the NIB file change for step over and step out.

1:20 AM Changeset in webkit [15144] by thatcher
  • 7 edits
    1 add in trunk/WebKitTools

Reviewed by Eric.

Bug 9631: [Drosera] Add "Step Over" and "Step Out"
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9631


Adds step over and step out. Along with a little code cleanup
that was minor enough to piggyback on this fix.

  • Drosera/DebuggerDocument.h:
  • Drosera/DebuggerDocument.m: (-[DebuggerDocument stepOver:]): (-[DebuggerDocument stepOut:]): (-[DebuggerDocument windowDidLoad]): (-[DebuggerDocument windowWillClose:]): (-[DebuggerDocument toolbar:itemForItemIdentifier:willBeInsertedIntoToolbar:]): (-[DebuggerDocument toolbarDefaultItemIdentifiers:]): (-[DebuggerDocument toolbarAllowedItemIdentifiers:]): (-[DebuggerDocument validateUserInterfaceItem:]):
  • Drosera/Drosera.xcodeproj/project.pbxproj:
  • Drosera/debugger.html:
  • Drosera/debugger.js:
  • Drosera/viewer.css:
1:03 AM Changeset in webkit [15143] by thatcher
  • 2 edits in trunk/WebKitTools

Reviewed by Eric.

Bug 9628: [Drosera] Split Views acting oddly
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9628


Only update the last X or Y coordinate if the new
size was not constrained. Also adds the resize cursor to
the body during the drag incase there is a constrained
over drag off of the resizer element.

  • Drosera/debugger.js:
12:42 AM Changeset in webkit [15142] by thatcher
  • 2 edits in tags/WebKit-521.14.2/WebKit

Build fix and versioning.

12:37 AM Changeset in webkit [15141] by thatcher
  • 1 copy in tags/WebKit-521.14.2

New tag.

Jul 2, 2006:

4:36 PM Changeset in webkit [15140] by ggaren
  • 5 edits in trunk/WebCore

Reviewed and landed by ggaren.

  • platform/gdk/TransferJobManager.cpp Fixed broken dowload logic
  • platform/gdk/ScrollViewGdk.cpp Add comment that scrollbarSize is a temporary variable
  • Projects/gdk/webcore-gdk.bkl Defined NDEBUG build executable is miscompiled otherwise
  • WebCoreSources.bkl added icon/SiteIcon.cpp editing/IndentOutdentCommand.cpp html/HTMLViewSourceDocument.cpp
11:17 AM Changeset in webkit [15139] by ggaren
  • 2 edits in trunk/JavaScriptCore

Reviewed by Adele.


6:05 AM Changeset in webkit [15138] by ddkilzer
  • 2 edits in trunk/WebKitTools

WebKitTools:

Reviewed by Alexy Proskuryakov. Patch by Mark Rowe.

Bug 9692: Warning about Safari extensions on every launch is obnoxious
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9692

  • WebKitLauncher/WebKitNightlyEnabler.m: (myApplicationWillTerminate): Note that we are exiting cleanly. (cleanUpAfterOurselves): Install bundle load tracking only if we failed to exit cleanly on our last invocation. This doesn't play nicely with multiple concurrent instances of WebKit.app so it can be disabled via defaults write com.apple.Safari WKNEShouldMonitorShutdowns -bool NO.
5:23 AM Changeset in webkit [15137] by ddkilzer
  • 2 edits in trunk/WebKitTools

WebKitTools:

Reviewed by Alexy Proskuryakov. Patch by Mark Rowe.

Bug 9654: Refresh Loop when accessing feed URLs
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9654

  • WebKitLauncher/Info.plist: Don't claim to handle feed:// URLs.
5:02 AM Changeset in webkit [15136] by ddkilzer
  • 3 edits in trunk/WebKitTools

WebKitTools:

Reviewed by Timothy Hatcher. Patch by Mark Rowe.

Bug 9689: Nightly builds should warn a user about potential problems when using
"Safari extensions"
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9689

  • WebKitLauncher/WebKitLauncher.xcodeproj/project.pbxproj:
  • WebKitLauncher/WebKitNightlyEnabler.m: (myBundleDidLoad): Keep track of if any bundles that are loaded. (myApplicationWillFinishLaunching): Notify user if any bundles are loaded. (cleanUpAfterOurselves): Register for NSBundleDidLoadNotification and NSApplicationWillFinishLaunchingNotification notifications so that we can track bundle loads and notify the user at launch completion.
4:59 AM Changeset in webkit [15135] by ddkilzer
  • 4 edits in trunk/WebCore

WebCore:

Reviewed by Timothy Hatcher. Build fix for Linux. Patch by Shawn Stricker.

  • platform/gdk/ScrollViewGdk.cpp: (WebCore::ScrollView::setDrawable): Changed SCROLLBAR_SIZE to scrollbarSize.
  • platform/gdk/TemporaryLinkStubs.cpp: (Image::drawTiled): Added missing parameter to argument list.
  • platform/gdk/TransferJobManager.cpp: (WebCore::writeCallback): Added missing parenthesis to static_cast.
4:39 AM Changeset in webkit [15134] by ddkilzer
  • 4 edits
    4 adds in trunk

WebCore:

Reviewed by Darin. Patch by Mitz.

Replaced the requestingScript state bit added in the fix for bug 9317
with a m_requestingScript bool that is set before and restored
after calling ref() on a CachedScript.


Test: fast/tokenizer/nested-cached-scripts.html

  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::HTMLTokenizer): (WebCore::HTMLTokenizer::begin): (WebCore::HTMLTokenizer::scriptHandler): (WebCore::HTMLTokenizer::notifyFinished):
  • html/HTMLTokenizer.h: (WebCore::HTMLTokenizer::State::):

LayoutTests:

Reviewed by Darin. Patch by Mitz.

  • fast/tokenizer/nested-cached-scripts-expected.txt: Added.
  • fast/tokenizer/nested-cached-scripts.html: Added.
  • fast/tokenizer/resources/nested-1.js: Added.
  • fast/tokenizer/resources/nested-2.js: Added.

Jul 1, 2006:

9:06 PM Changeset in webkit [15133] by ggaren
  • 17 edits
    14 adds in trunk/JavaScriptCore

Reviewed by Maciej.


  • Phase 2 in the JS API.


  • Added support for specifying static tables of values -- this should obviate the need for using complicated callbacks for most lookups.


  • API objects are now created with classes (JSClassRef) -- in order to support static values, and in order to prevent API objects from storing their data inline, and thus falling into the oversized (read: slow and prone to giving Maciej the frowny face) heap.


  • Added two specialized JSObject subclasses -- JSCallbackFunction and JSCallbackConstructor -- to allow JSFunctionMake and JSConstructorMake to continue to work with the new class model. Another solution to this problem would be to create a custom class object for each function and constructor you make. This solution is more code but also more efficient.


  • Substantially beefed up the minidom example to demonstrate and test a lot of these techniques. Its output is still pretty haphazard, though.


  • Gave the <kjs/ preface to some includes -- I'm told this matters to building on some versions of Linux.


  • Implemented JSValueIsInstanceOf and JSValueIsObjectOfClass


  • Removed GetDescription callback. Something in the class datastructure should take care of this.
  • API/JSBase.h:
  • API/JSCallbackConstructor.cpp: Added. (KJS::): (KJS::JSCallbackConstructor::JSCallbackConstructor): (KJS::JSCallbackConstructor::implementsConstruct): (KJS::JSCallbackConstructor::construct): (KJS::JSCallbackConstructor::setPrivate): (KJS::JSCallbackConstructor::getPrivate):
  • API/JSCallbackConstructor.h: Added. (KJS::JSCallbackConstructor::classInfo):
  • API/JSCallbackFunction.cpp: Added. (KJS::): (KJS::JSCallbackFunction::JSCallbackFunction): (KJS::JSCallbackFunction::implementsCall): (KJS::JSCallbackFunction::callAsFunction): (KJS::JSCallbackFunction::setPrivate): (KJS::JSCallbackFunction::getPrivate):
  • API/JSCallbackFunction.h: Added. (KJS::JSCallbackFunction::classInfo):
  • API/JSCallbackObject.cpp: (KJS::): (KJS::JSCallbackObject::JSCallbackObject): (KJS::JSCallbackObject::init): (KJS::JSCallbackObject::~JSCallbackObject): (KJS::JSCallbackObject::className): (KJS::JSCallbackObject::getOwnPropertySlot): (KJS::JSCallbackObject::put): (KJS::JSCallbackObject::deleteProperty): (KJS::JSCallbackObject::implementsConstruct): (KJS::JSCallbackObject::construct): (KJS::JSCallbackObject::implementsCall): (KJS::JSCallbackObject::callAsFunction): (KJS::JSCallbackObject::getPropertyList): (KJS::JSCallbackObject::toBoolean): (KJS::JSCallbackObject::toNumber): (KJS::JSCallbackObject::toString): (KJS::JSCallbackObject::inherits): (KJS::JSCallbackObject::staticValueGetter): (KJS::JSCallbackObject::staticFunctionGetter): (KJS::JSCallbackObject::callbackGetter):
  • API/JSCallbackObject.h:
  • API/JSCharBufferRef.cpp:
  • API/JSClassRef.cpp: Added. (JSClassCreate): (JSClassRetain): (JSClassRelease):
  • API/JSClassRef.h: Added. (StaticValueEntry::StaticValueEntry): (StaticFunctionEntry::StaticFunctionEntry): (JSClass::JSClass):
  • API/JSContextRef.cpp: (JSContextCreate): (JSEvaluate):
  • API/JSContextRef.h:
  • API/JSNode.c: Added. (JSNodePrototype_appendChild): (JSNodePrototype_removeChild): (JSNodePrototype_replaceChild): (JSNodePrototype_class): (JSNode_getNodeType): (JSNode_getChildNodes): (JSNode_getFirstChild): (JSNode_finalize): (JSNode_class): (JSNode_prototype): (JSNode_new): (JSNode_construct):
  • API/JSNode.h: Added.
  • API/JSNodeList.c: Added. (JSNodeListPrototype_item): (JSNodeListPrototype_class): (JSNodeList_length): (JSNodeList_getProperty): (JSNodeList_finalize): (JSNodeList_class): (JSNodeList_prototype): (JSNodeList_new):
  • API/JSNodeList.h: Added.
  • API/JSObjectRef.cpp: (JSObjectMake): (JSFunctionMake): (JSConstructorMake): (JSPropertyEnumerator::JSPropertyEnumerator): (JSObjectCreatePropertyEnumerator): (JSPropertyEnumeratorGetNext): (JSPropertyEnumeratorRetain): (JSPropertyEnumeratorRelease):
  • API/JSObjectRef.h: (JSObjectCallbacks::):
  • API/JSValueRef.cpp: (JSValueIsObjectOfClass): (JSValueIsInstanceOf):
  • API/JSValueRef.h:
  • API/Node.c: Added. (Node_new): (Node_appendChild): (Node_removeChild): (Node_replaceChild): (Node_ref): (Node_deref):
  • API/Node.h: Added.
  • API/NodeList.c: Added. (NodeList_new): (NodeList_length): (NodeList_item): (NodeList_ref): (NodeList_deref):
  • API/NodeList.h: Added.
  • API/minidom.c: (main): (print): (createStringWithContentsOfFile):
  • API/minidom.js:
  • API/testapi.c: (assertEqualsAsCharacters): (MyObject_getProperty): (MyObject_class): (myConstructor_callAsConstructor): (main):
  • API/testapi.js:
  • JavaScriptCore.xcodeproj/project.pbxproj:
8:20 PM Changeset in webkit [15132] by ddkilzer
  • 2 edits in trunk/WebCore

WebCore:

Reviewed by NOBODY (build fix).

  • platform/gdk/TransferJobManager.cpp: (WebCore::writeCallback): Changed 'static_cat' to 'static_cast'.
7:50 PM Changeset in webkit [15131] by ddkilzer
  • 2 edits in trunk/WebKit

WebKit:

Reviewed by NOBODY (fixed Tim's build fix).

  • WebView/WebView.m: Added back missing '/' at the beginning of the file.
6:31 PM Changeset in webkit [15130] by tomernic
  • 2 edits in trunk/WebKit

Reviewed by NOBODY (build fix)

  • WebView/WebView.m: (-[WebView _isMIMETypeRegisteredAsPlugin:]): Changed nil to NO (typo).
12:13 PM Changeset in webkit [15129] by thatcher
  • 2 edits in tags/WebKit-521.14.1/WebKit

Build fix for case-sensitive filesystems and versioning.

12:11 PM Changeset in webkit [15128] by thatcher
  • 1 copy in tags/WebKit-521.14.1/WebKit

New tag part 2.

12:10 PM Changeset in webkit [15127] by thatcher
  • 1 add in tags/WebKit-521.14.1

New tag.

12:08 PM Changeset in webkit [15126] by thatcher
  • 1 edit in trunk/WebKit/WebKit.xcodeproj/project.pbxproj

Build fix for case-sensitive filesystems.

Jun 30, 2006:

10:02 PM Changeset in webkit [15125] by darin
  • 4 edits in trunk/JavaScriptCore

Reviewed and tweaked by Darin.

  • Compile fixes for wx port / gcc 4.0.2
  • kjs/array_object.cpp: Added missing headers.
  • kjs/ExecState.h: gcc needs class prototypes before defining those classes as friend classes
9:39 PM Changeset in webkit [15124] by darin
  • 1 edit
    29 adds in trunk/WebCore

Reviewed and worked over a lot by Darin.

  • check in first set of GDK-specific sources
  • platform/gdk/BrowserExtensionGdk.h: Added.
  • platform/gdk/CookieJarGdk.cpp: Added.
  • platform/gdk/CursorGdk.cpp: Added.
  • platform/gdk/FontCacheGdk.cpp: Added.
  • platform/gdk/FontDataGdk.cpp: Added.
  • platform/gdk/FontGdk.cpp: Added.
  • platform/gdk/FontPlatformData.h: Added.
  • platform/gdk/FontPlatformDataGdk.cpp: Added.
  • platform/gdk/FrameGdk.cpp: Added.
  • platform/gdk/FrameGdk.h: Added.
  • platform/gdk/GlyphMapGdk.cpp: Added.
  • platform/gdk/KeyEventGdk.cpp: Added.
  • platform/gdk/KeyboardCodes.h: Added.
  • platform/gdk/MouseEventGdk.cpp: Added.
  • platform/gdk/PageGdk.cpp: Added.
  • platform/gdk/RenderThemeGdk.cpp: Added.
  • platform/gdk/RenderThemeGdk.h: Added.
  • platform/gdk/ScreenGdk.cpp: Added.
  • platform/gdk/ScrollViewGdk.cpp: Added.
  • platform/gdk/SharedTimerLinux.cpp: Added.
  • platform/gdk/SystemTimeLinux.cpp: Added.
  • platform/gdk/TemporaryLinkStubs.cpp: Added.
  • platform/gdk/TransferJobCurl.cpp: Added.
  • platform/gdk/TransferJobManager.cpp: Added.
  • platform/gdk/TransferJobManager.h: Added.
  • platform/gdk/WheelEventGdk.cpp: Added.
  • platform/gdk/WidgetGdk.cpp: Added.
  • platform/gdk/gdk-encodings.txt: Added.
9:24 PM Changeset in webkit [15123] by darin
  • 2 edits in trunk/WebCore

Reviewed by Darin.

  • WebCoreSources.bkl: Updated for recent file changes.
9:23 PM Changeset in webkit [15122] by darin
  • 5 edits in trunk/WebCore

Reviewed and enhanced by Darin.

Compilation fixes for Linux/Gdk.

  • WebCore/platform/FontData.h: Added forward declaration of a class.
  • WebCore/platform/cairo/GraphicsContextCairo.cpp: Fixed case on include.
  • WebCore/bindings/js/kjs_html.cpp: Added <math.h> include.
  • WebCore/css/maketokenizer: Fixed incorrect check for Linux that was already true even on Mac OS X. Added some additional includes and defines to the Linux extra generated code.
7:51 PM Changeset in webkit [15121] by ddkilzer
  • 7 edits
    4 adds in trunk

WebCore:

Reviewed by Hyatt.

Test: fast/repaint/float-move-during-layout.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlockChildren): (WebCore::RenderBlock::repaintOverhangingFloats): Renamed repaintFloatingDescendants() to this and limited it to repainting overhanging floats. Added a boolean parameter that forces all descendant overhanging floats (that don't have their own layer) to paint.
  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::repaintDuringLayoutIfMoved): Changed to paint all descendant floats.
  • rendering/RenderObject.cpp: (WebCore::RenderObject::repaintOverhangingFloats):
  • rendering/RenderObject.h:

LayoutTests:

Reviewed by Hyatt.

  • fast/repaint/float-move-during-layout-expected.checksum: Added.
  • fast/repaint/float-move-during-layout-expected.png: Added.
  • fast/repaint/float-move-during-layout-expected.txt: Added.
  • fast/repaint/float-move-during-layout.html: Added.
6:33 PM Changeset in webkit [15120] by darin
  • 1 edit in trunk/Bakefiles/presets.bkl

Changes by Mike Emmel.

6:32 PM Changeset in webkit [15119] by darin
  • 1 edit
    8 adds in trunk/WebKitTools

Reviewed by Darin.

  • first check-in of a Gdk shell for testing WebKit
  • GdkLauncher: Added.
6:31 PM Changeset in webkit [15118] by darin
  • 5 edits in trunk/JavaScriptCore

Reviewed by Darin.

Compilation fixes for Linux/Gdk.

  • JavaScriptCore/kjs/interpreter.cpp: added include of signal.h
  • JavaScriptCore/kjs/ExecState.h: added missing class declaration
  • JavaScriptCore/kjs/ExecState.cpp: case wrong on include of context.h
  • JavaScriptCore/JavaScriptCoreSources.bkl: added Context.cpp and ExecState.cpp
4:47 PM Changeset in webkit [15117] by lweintraub
  • 5 edits
    4 adds in trunk

LayoutTests:

Reviewed by justin

  • editing/execCommand/format-block-with-trailing-br-expected.checksum: Added.
  • editing/execCommand/format-block-with-trailing-br-expected.png: Added.
  • editing/execCommand/format-block-with-trailing-br-expected.txt: Added.
  • editing/execCommand/format-block-with-trailing-br.html: Added.

WebCore:

Reviewed by justin

  • editing/VisiblePosition.cpp: Removed is[Not]EqualIgnoringAffinity and made operator== use the logic that ignores affinity.
  • editing/VisiblePosition.h: Ditto. (WebCore::operator==):
  • editing/visible_units.cpp: (WebCore::isStartOfParagraph): Changed to compare VisiblePositions using ==. (WebCore::isEndOfParagraph): Ditto. (WebCore::isStartOfBlock): Ditto. (WebCore::isEndOfBlock): Ditto.
4:41 PM Changeset in webkit [15116] by thatcher
  • 9 copies in tags/Safari-521.14

New tag.

4:35 PM Changeset in webkit [15115] by thatcher
  • 8 edits in /

Versioning

3:32 PM Changeset in webkit [15114] by justing
  • 1 edit in trunk/LayoutTests/editing/pasteboard/pasting-object.html

Fixed the test case that I just checked in.

3:17 PM Changeset in webkit [15113] by justing
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by levi

  • editing/pasteboard/pasting-object-expected.checksum: Added.
  • editing/pasteboard/pasting-object-expected.png: Added.
  • editing/pasteboard/pasting-object-expected.txt: Added.
  • editing/pasteboard/pasting-object.html: Added.

WebCore:

Reviewed by levi

  • editing/htmlediting.cpp: (WebCore::canHaveChildrenForEditing): Don't insert children into object tags while editing.
12:43 PM Changeset in webkit [15112] by justing
  • 22 edits
    4 adds in trunk

LayoutTests:

Reviewed by harrison


Added:

  • editing/selection/skip-non-editable-1-expected.txt: Added.
  • editing/selection/skip-non-editable-1.html: Added.
  • editing/selection/skip-non-editable-2-expected.txt: Added.
  • editing/selection/skip-non-editable-2.html: Added.

Equivalent render tree:

  • editing/execCommand/format-block-with-braces-expected.txt:


Fixes:

  • editing/selection/iframe-expected.checksum:
  • editing/selection/iframe-expected.png:
  • editing/selection/iframe-expected.txt:
  • editing/selection/inline-table-expected.checksum:
  • editing/selection/inline-table-expected.png:
  • editing/selection/inline-table-expected.txt:
  • editing/selection/move-between-blocks-no-001-expected.checksum:
  • editing/selection/move-between-blocks-no-001-expected.png:
  • editing/selection/move-between-blocks-no-001-expected.txt:
  • editing/selection/select-box-expected.checksum:
  • editing/selection/select-box-expected.png:
  • editing/selection/select-box-expected.txt:

WebCore:

Reviewed by harrison


<rdar://problem/4598309
Caret gets stuck when it hits non editable content within an editable body
<​http://bugzilla.opendarwin.org/show_bug.cgi?id=9510>
Caret will jump out of an editable region


  • dom/Position.cpp: (WebCore::Position::upstream): Don't move into a new editable region so that candidates that that straddle an editable/non-editable boundary will correspond to different VisiblePositions. (WebCore::Position::downstream): Ditto.
  • editing/SelectionController.cpp: (WebCore::SelectionController::modifyMovingRightForward): Use VisiblePosition::next(true) (WebCore::SelectionController::modifyMovingLeftBackward): Use VisiblePosition::previous(true)
  • editing/VisiblePosition.cpp: (WebCore::VisiblePosition::next): Added code to jump to the next editable region as long as it has the same highest editable root. (WebCore::VisiblePosition::previous): Ditto.
  • editing/VisiblePosition.h:
  • editing/htmlediting.cpp: (WebCore::highestEditableRoot): Added, returns the highest editable root of a node if that node is in editable content, otherwise 0.
  • editing/htmlediting.h:
12:09 PM Changeset in webkit [15111] by ddkilzer
  • 3 edits
    2 adds in trunk

WebCore:

Reviewed by Darin.

Test: editing/deleting/delete-all-text-in-text-field-assertion.html

Bug 9661 filed to fix the follow-up issue:
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9661
textContent returns "\n" for a <br> even if it is collapsed

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::setValueFromRenderer): Work around "\n" value when all characters are deleted from a text field.

LayoutTests:

Reviewed by Darin.

  • editing/deleting/delete-all-text-in-text-field-assertion-expected.txt: Added.
  • editing/deleting/delete-all-text-in-text-field-assertion.html: Added.
11:56 AM Changeset in webkit [15110] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Darin.

Only enable shouldCloseWithWindow when ObjC GC is enabled.
This maintains backwards compatibility with applications
that expect a WebView to be usable after the window closes.

  • WebView/WebView.m: (-[WebViewPrivate init]):
10:53 AM Changeset in webkit [15109] by harrison
  • 2 edits in trunk/WebCore

Reviewed by Tim Omernick

<rdar://problem/4609481> Crash calling -[WebScriptObject valueForKey:] when JS disabled


  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::executionContextForDOM): Early return if JS is not enabled.
10:35 AM Changeset in webkit [15108] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Anders.

Call _close in dealloc to ensure we cleanup for backwards
compatibility. This will safeguard and cleanup even if the
application doesn't use the new close API yet, like Mail.

  • WebView/WebView.m: (-[WebView dealloc]):
7:05 AM Changeset in webkit [15107] by thatcher
  • 2 edits in trunk/WebCore

Reviewed by Tim H.

  • page/FrameView.cpp: (WebCore::FrameView::handleMouseMoveEvent): Added a check that oldSubframe is still a subframe of the view's frame before passing the event to it.
12:29 AM Changeset in webkit [15106] by thatcher
  • 23 edits in trunk

WebCore:

Reviewed by Darin.

<rdar://problem/3694163> -[KWQPageState finalize] is incorrect; design change needed
<rdar://problem/3694165> -[WebCoreFrameBridge finalize] is incorrect; design change needed

Added close methods that need to be called before these
objects can be released. This ensures proper tear-down under GC.

  • WebCore.xcodeproj/project.pbxproj:
  • bridge/mac/WebCoreFrameBridge.h:
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge dealloc]): (-[WebCoreFrameBridge finalize]): (-[WebCoreFrameBridge close]):
  • bridge/mac/WebCorePageBridge.h:
  • bridge/mac/WebCorePageBridge.mm: (-[WebCorePageBridge setMainFrame:]): (-[WebCorePageBridge dealloc]): (-[WebCorePageBridge finalize]): (-[WebCorePageBridge close]): (-[WebCorePageBridge mainFrame]): (-[WebCorePageBridge setGroupName:]): (-[WebCorePageBridge groupName]):
  • kwq/KWQPageState.h:
  • kwq/KWQPageState.mm: (-[KWQPageState dealloc]): (-[KWQPageState finalize]): (-[KWQPageState close]):

WebKit:

Reviewed by Darin.

<rdar://problem/4484405> WebKit leaks, improper tear-down
<rdar://problem/3694059> -[WebBackForwardList finalize] is incorrect; design change needed
<rdar://problem/3694103> -[WebFrame finalize] is incorrect; design change needed
<rdar://problem/3694104> -[WebHTMLView finalize] is incorrect; design change needed

Adds a close method to WebView, this needs to be called when the
WebView is no longer needed. To make this easier for the common cases
there is now an "auto close" on WebView that listens to the view's
parent window. If the parent window closes and the WebView has no
hostWindow then the WebView is automatically closed if autoClose is YES.
To manage WebView closing yourself call setAutoClose: and pass NO.

When a WebView closes it will tear-down and not be usable anymore.
Close will will called on various other internal objects as a part
of this, to ensure proper tear-down in GC without relying on finalize.

  • History/WebBackForwardList.m: (-[WebBackForwardList dealloc]): (-[WebBackForwardList finalize]): (-[WebBackForwardList _close]):
  • History/WebHistoryItem.m: (+[WebHistoryItem _closeObjectsInPendingPageCaches]): (+[WebHistoryItem _releaseAllPendingPageCaches]):
  • History/WebHistoryItemPrivate.h:
  • WebCoreSupport/WebFrameBridge.m: (-[WebFrameBridge close]): (-[WebFrameBridge saveDocumentToPageCache:]): (-[WebFrameBridge canGoBackOrForward:]):
  • WebView/WebFrame.m: (-[WebFrame _detachFromParent]): (-[WebFrame dealloc]): (-[WebFrame finalize]):
  • WebView/WebFrameView.m: (-[WebFrameView _setWebFrame:]): (-[WebFrameView finalize]):
  • WebView/WebHTMLView.m: (-[WebHTMLView close]): (-[NSArray dealloc]): (-[NSArray finalize]):
  • WebView/WebHTMLViewInternal.h:
  • WebView/WebHTMLViewPrivate.h:
  • WebView/WebScriptDebugDelegate.m: (-[WebScriptCallFrame _initWithFrame:initWithWebFrame:]): (-[WebScriptCallFrame parsedSource:fromURL:sourceId:startLine:errorLine:errorMessage:]): (-[WebScriptCallFrame enteredFrame:sourceId:line:]): (-[WebScriptCallFrame hitStatement:sourceId:line:]): (-[WebScriptCallFrame leavingFrame:sourceId:line:]):
  • WebView/WebScriptDebugDelegatePrivate.h:
  • WebView/WebView.m: (-[WebViewPrivate init]): (-[WebView _close]): (-[WebView dealloc]): (-[WebView finalize]): (-[WebView viewWillMoveToWindow:]): (-[WebView _windowWillClose:]): (-[WebView setPreferencesIdentifier:]): (-[WebView mainFrame]): (-[WebView setHostWindow:]): (-[WebView searchFor:direction:caseSensitive:wrap:]): (-[WebView writeSelectionWithPasteboardTypes:toPasteboard:]): (-[WebView close]): (-[WebView setAutoClose:]): (-[WebView autoClose]): (-[WebView _frameViewAtWindowPoint:]):
  • WebView/WebViewPrivate.h:

WebKitTools:

Reviewed by Darin.

Need to call window close so WebView tears-down completly.

  • DumpRenderTree/DumpRenderTree.m: (main): call [window close]
Note: See TracTimeline for information about the timeline view.