Timeline
Jul 25, 2005:
- 9:39 PM Changeset in webkit [9902] by
-
- 2 edits in trunk/WebKitTools
- Scripts/webkitdirs.pm: Check in missing line of code.
- 8:58 PM Changeset in webkit [9901] by
-
- 5 edits in trunk/WebKitTools
- fixed problem that was causing JavaScriptCore test to fail (except for people who had set DYLD_FRAMEWORK_PATH)
- Scripts/run-javascriptcore-tests: Add code to set DYLD_FRAMEWORK_PATH. Add code to parse configuration parameter so you can pass --deployment if you like.
- Scripts/run-webkit-tests: Add code to parse configuration parameter.
- Scripts/update-javascriptcore-test-results: Add license header.
- Scripts/webkitdirs.pm: Change code that reads configuration option to remove it from @ARGV. This lets us use this option in commands that take other options and pass them along to a subsequent tool.
- 5:56 PM Changeset in webkit [9900]
-
- 37 copies3 deletes in tags/JavaScriptCore-412~6
This commit was manufactured by cvs2svn to create tag
'JavaScriptCore-412~6'.
- 5:56 PM Changeset in webkit [9899] by
-
- 2 edits in branches/Safari-2-0-branch/JavaScriptCore
Versioning for Safari-2-0-branch - JavaScriptCore-412.6
- 5:52 PM Changeset in webkit [9898]
-
- 140 copies2 deletes in tags/WebCore-415~15
This commit was manufactured by cvs2svn to create tag
'WebCore-415~15'.
- 5:52 PM Changeset in webkit [9897] by
-
- 2 edits in branches/Safari-2-0-branch/WebCore
Versioning for Safari-2-0-branch - WebCore-415.15
- 5:48 PM Changeset in webkit [9896]
-
- 36 copies3 deletes in tags/WebKit-412~8
This commit was manufactured by cvs2svn to create tag 'WebKit-412~8'.
- 5:48 PM Changeset in webkit [9895] by
-
- 2 edits in branches/Safari-2-0-branch/WebKit
Versioning for Safari-2-0-branch - WebKit-412.8
- 5:40 PM Changeset in webkit [9894] by
-
- 2 edits in branches/Safari-2-0-branch/WebKit
Merge from TOT to Safari-2-0-branch
<rdar://problem/3470523>
2005-07-25 Vicki Murley <vicki@apple.com>
Reviewed by Darin.
- fixed <rdar://problem/3470523> Safari's user agent should be changed to say Intel rather than PPC on Intel machines
- WebView.subproj/WebView.m: add conditional #defines for "PPC" and "Intel" (-[WebView userAgentForURL:]): use this variable when constructing the user agent string
- 5:14 PM Changeset in webkit [9893] by
-
- 2 edits in trunk/JavaScriptCore
- fixed mistake in my last checkin -- the expected results included results from a patch that hasn't landed yet.
- tests/mozilla/expected.html:
- 5:11 PM Changeset in webkit [9892] by
-
- 2 edits in trunk/WebKit
Reviewed by Darin.
- fixed <rdar://problem/3470523> Safari's user agent should be changed to say Intel rather than PPC on Intel machines
- WebView.subproj/WebView.m: add conditional #defines for "PPC" and "Intel" (-[WebView userAgentForURL:]): use this variable when constructing the user agent string
- 4:35 PM Changeset in webkit [9891] by
-
- 2 edits in trunk/JavaScriptCore
- fix mistake in last change that leads to assertion failure in the Development build
- kjs/lookup.h: (KJS::lookupGetOwnValue):
- 4:13 PM Changeset in webkit [9890] by
-
- 2 edits1 add in trunk/WebCore
Reviewed by Beth Dakin.
- fixed <rdar://problem/4011544> selecting an item from a <select> menu with the mouse doesn't place the focus on the menu
Test cases added:
- manual-tests/focus-select-when-clicked.html: Added.
- kwq/KWQComboBox.mm: (-[KWQPopUpButton needsPanelToBecomeKey]): override this NSView method to return YES
- 3:17 PM Changeset in webkit [9889] by
-
- 50 edits in trunk
Reviewed by Darin.
- http://bugzilla.opendarwin.org/show_bug.cgi?id=4124 (change JavaScript property access to avoid double lookup)
- 10% speedup on JavaScript iBench
- 5% speedup on 24fun BenchJS benchmark
Changed all get methods to getOwnProperty - they are no longer
responsible for prototype lookup, and determine if the property
was found as a side efect.
get() is now a nonvirtual ObjectImp method which calls the virtual
getOwnProperty and walks the prototype chain. A few selected
methods were inlined.
Changed ResolveNode::evaluate plus some other places to use
getProperty which does get() and hasProperty() in one lookup.
Also miscellaneous code cleanup.
- bindings/objc/objc_runtime.h:
- bindings/objc/objc_runtime.mm: (ObjcFallbackObjectImp::ObjcFallbackObjectImp): (ObjcFallbackObjectImp::getOwnProperty):
- bindings/runtime_array.cpp: (RuntimeArrayImp::RuntimeArrayImp): (RuntimeArrayImp::getOwnProperty):
- bindings/runtime_array.h:
- bindings/runtime_method.cpp: (RuntimeMethodImp::getOwnProperty):
- bindings/runtime_method.h:
- bindings/runtime_object.cpp: (RuntimeObjectImp::getOwnProperty):
- bindings/runtime_object.h:
- kjs/array_instance.h:
- kjs/array_object.cpp: (ArrayInstanceImp::getOwnProperty): (ArrayPrototypeImp::getOwnProperty): (ArrayProtoFuncImp::call):
- kjs/array_object.h:
- kjs/date_object.cpp: (DatePrototypeImp::getOwnProperty):
- kjs/date_object.h:
- kjs/function.cpp: (KJS::FunctionImp::getOwnProperty): (KJS::ArgumentsImp::getOwnProperty): (KJS::ActivationImp::getOwnProperty):
- kjs/function.h:
- kjs/lookup.h: (KJS::lookupGetOwnProperty): (KJS::lookupGetOwnFunction): (KJS::lookupGetOwnValue):
- kjs/math_object.cpp: (MathObjectImp::getOwnProperty): (MathObjectImp::getValueProperty):
- kjs/math_object.h:
- kjs/nodes.cpp: (ResolveNode::evaluate):
- kjs/number_object.cpp: (NumberObjectImp::getOwnProperty):
- kjs/number_object.h:
- kjs/object.cpp: (KJS::ObjectImp::get): (KJS::ObjectImp::getOwnProperty): (KJS::ObjectImp::getProperty):
- kjs/object.h: (KJS::ObjectImp::getProperty): (KJS::ObjectImp::getOwnProperty):
- kjs/object_object.cpp: (ObjectProtoFuncImp::call):
- kjs/regexp_object.cpp: (RegExpObjectImp::getOwnProperty):
- kjs/regexp_object.h:
- kjs/string_object.cpp: (StringInstanceImp::getOwnProperty): (StringPrototypeImp::getOwnProperty):
- kjs/string_object.h:
WebCore:
Reviewed by Darin.
- http://bugzilla.opendarwin.org/show_bug.cgi?id=4124 (change JavaScript property access to avoid double lookup)
- 10% speedup on JavaScript iBench
- 5% speedup on 24fun BenchJS benchmark
Changed all get methods to getOwnProperty - they are no longer responsible for
prototype lookup, and determine if the property was found as a side efect.
Also miscellaneous code cleanup.
- khtml/ecma/kjs_css.cpp: (KJS::DOMCSSStyleDeclaration::getOwnProperty): (KJS::DOMStyleSheet::getOwnProperty): (KJS::DOMStyleSheetList::getOwnProperty): (KJS::DOMMediaList::getOwnProperty): (KJS::DOMCSSStyleSheet::getOwnProperty): (KJS::DOMCSSRuleList::getOwnProperty): (KJS::DOMCSSRule::getOwnProperty): (KJS::DOMCSSRule::getValueProperty): (KJS::CSSRuleConstructor::getOwnProperty): (KJS::DOMCSSValue::getOwnProperty): (KJS::CSSValueConstructor::getOwnProperty): (KJS::DOMCSSPrimitiveValue::getOwnProperty): (KJS::CSSPrimitiveValueConstructor::getOwnProperty): (KJS::DOMCSSValueList::getOwnProperty): (KJS::DOMRGBColor::getOwnProperty): (KJS::DOMRect::getOwnProperty): (KJS::DOMCounter::getOwnProperty):
- khtml/ecma/kjs_css.h:
- khtml/ecma/kjs_dom.cpp: (KJS::DOMNode::getOwnProperty): (KJS::DOMNodeList::getOwnProperty): (KJS::DOMAttr::getOwnProperty): (KJS::DOMDocument::getOwnProperty): (KJS::DOMElement::getOwnProperty): (KJS::DOMDocumentType::getOwnProperty): (KJS::DOMNamedNodeMap::getOwnProperty): (KJS::DOMProcessingInstruction::getOwnProperty): (KJS::DOMNotation::getOwnProperty): (KJS::DOMEntity::getOwnProperty): (KJS::NodeConstructor::getOwnProperty): (KJS::DOMExceptionConstructor::getOwnProperty): (KJS::DOMNamedNodesCollection::getOwnProperty): (KJS::DOMCharacterData::getOwnProperty):
- khtml/ecma/kjs_dom.h:
- khtml/ecma/kjs_events.cpp: (KJS::EventConstructor::getOwnProperty): (KJS::DOMEvent::getOwnProperty): (KJS::EventExceptionConstructor::getOwnProperty): (KJS::DOMUIEvent::getOwnProperty): (KJS::DOMMouseEvent::getOwnProperty): (KJS::DOMKeyboardEvent::getOwnProperty): (KJS::MutationEventConstructor::getOwnProperty): (KJS::DOMMutationEvent::getOwnProperty): (KJS::DOMWheelEvent::getOwnProperty): (KJS::Clipboard::getOwnProperty):
- khtml/ecma/kjs_events.h:
- khtml/ecma/kjs_html.cpp: (KJS::HTMLDocument::getOwnProperty): (KJS::KJS::HTMLElement::getOwnProperty): (KJS::KJS::HTMLCollection::getOwnProperty): (KJS::KJS::HTMLSelectCollection::getOwnProperty): (KJS::Image::getOwnProperty): (KJS::Context2D::getOwnProperty): (KJS::Gradient::getOwnProperty): (KJS::ImagePattern::getOwnProperty):
- khtml/ecma/kjs_html.h:
- khtml/ecma/kjs_navigator.cpp: (KJS::Plugin::Plugin): (KJS::Navigator::getOwnProperty): (KJS::Plugins::getOwnProperty): (KJS::MimeTypes::getOwnProperty): (KJS::Plugin::getOwnProperty): (KJS::MimeType::getOwnProperty):
- khtml/ecma/kjs_navigator.h:
- khtml/ecma/kjs_range.cpp: (KJS::DOMRange::getOwnProperty): (KJS::RangeConstructor::getOwnProperty):
- khtml/ecma/kjs_range.h:
- khtml/ecma/kjs_traversal.cpp: (KJS::DOMNodeIterator::getOwnProperty): (KJS::NodeFilterConstructor::getOwnProperty): (KJS::DOMTreeWalker::getOwnProperty):
- khtml/ecma/kjs_traversal.h:
- khtml/ecma/kjs_views.cpp: (KJS::DOMAbstractView::getOwnProperty):
- khtml/ecma/kjs_views.h:
- khtml/ecma/kjs_window.cpp: (KJS::Screen::getOwnProperty): (KJS::Window::~Window): (KJS::Window::getOwnProperty): (KJS::Window::put): (KJS::FrameArray::getOwnProperty): (KJS::Location::Location): (KJS::Location::getOwnProperty): (KJS::Location::put): (KJS::Selection::Selection): (KJS::Selection::getOwnProperty): (KJS::BarInfo::getOwnProperty): (KJS::History::getOwnProperty):
- khtml/ecma/kjs_window.h:
- khtml/ecma/xmlhttprequest.cpp: (KJS::XMLHttpRequest::getOwnProperty):
- khtml/ecma/xmlhttprequest.h:
- 2:55 PM Changeset in webkit [9888] by
-
- 4 edits in trunk/WebKitTools
Reviewed by Geoff Garen.
- Fixed run-safari and gdb-safari to use the Safari application in the build results directory, if any, falling back to the one in the Applications directory otherwise. Does no harm for open source contributors who don't build Safari, and helps out the Safari team, since we do build Safari.
- Scripts/webkitdirs.pm: Added safariPath function that uses WEBKIT_SAFARI environment variable, and if that's not present, looks in either the build results directory or /Applications; factors code that was in both scripts before into a shared function. Also removed some Xcode 2.0 support which is no longer relevant since our projects are now in Xcode 2.1 format and incompatible with older versions of Xcode.
- Scripts/gdb-safari: Use safariPath.
- Scripts/run-safari: Use safariPath.
- 2:50 PM Changeset in webkit [9887] by
-
- 2 edits in trunk/WebCore
Reviewed by hyatt
Backing out a change that made the caret invisible in Mail in certain situations.
- khtml/rendering/render_block.cpp: (khtml::RenderBlock::paintObject):
- 1:38 PM Changeset in webkit [9886] by
-
- 4 edits in trunk/JavaScriptCore
- fixed http://bugzilla.opendarwin.org/show_bug.cgi?id=3971 JS test suite depends on JS 1.2 behavior
Reviewed by darin.
- tests/mozilla/js1_2/Array/tostring_1.js: now tests only for JS 1.5 behavior
- tests/mozilla/js1_2/Array/tostring_2.js: ditto
- tests/mozilla/expected.html:
- 9:59 AM Changeset in webkit [9885] by
-
- 1 edit1 add in trunk/WebCore
Added a manual test for a recent change to WebKit
Test cases added:
- manual-tests/deleteToEndOfLine.html: Added.
Jul 24, 2005:
- 5:35 PM Changeset in webkit [9884] by
-
- 2 edits in trunk/WebKit
Reviewed by mjs
- Fixes <rdar://problem/4120535> deleteToEndOfLine: does not delete thew newline when at the end of a line
Fix to match NSTextView. Delete the next character if deleteToEndOfLine fails
- WebView.subproj/WebHTMLView.m: (-[WebHTMLView deleteToEndOfLine:]):
- 3:58 PM Changeset in webkit [9883] by
-
- 5 edits in trunk/WebKit
Patch by Trey Matteson <trey@apple.com>
Reviewed by john
Fixes <http://bugzilla.opendarwin.org/show_bug.cgi?id=3953> back-forward items have wrong titles after sub-frame navigations
This was caused by a mistaken data structure, where WebDataSource tried to keep a list of
b/f items it was responsible for. The problem arose in the case of frames, where a subframe
was loaded with new content. When this happens a fresh tree of b/f items is created,
but the reference in the DataSource still pointed to the old item.
Since the WebFrame does a lot of work to track the current b/f item, the easiest thing is to
get rid of the DataSource's reference, and have it ask the WebFrame to set the title on the
right b/f item.
- WebView.subproj/WebDataSource.m: (-[WebDataSourcePrivate dealloc]): (-[WebDataSource _setTitle:]):
- WebView.subproj/WebDataSourcePrivate.h:
- WebView.subproj/WebFrame.m: (-[WebFrame _createItem:]): (-[WebFrame _transitionToCommitted:]): (-[WebFrame _loadURL:referrer:loadType:target:triggeringEvent:form:formValues:]): (-[WebFrame _setTitle:]):
- WebView.subproj/WebFramePrivate.h:
- 3:44 PM Changeset in webkit [9882] by
-
- 3 edits in trunk/WebCore
Patch by Anders Carlsson <andersca@mac.com>
Reviewed by hyatt
- <http://bugzilla.opendarwin.org/show_bug.cgi?id=3615> canvas transform matrix does not apply to patterns Fill and stroke patterns are created in the respective fill and stroke functions using the current CTM.
- khtml/ecma/kjs_html.cpp: (KJS::KJS::Context2DFunction::call): (KJS::Context2D::updateFillImagePattern): (KJS::Context2D::updateStrokeImagePattern): (KJS::Context2D::putValueProperty): (KJS::Context2D::Context2D): (KJS::_rh): (KJS::ImagePattern::createPattern):
- khtml/ecma/kjs_html.h:
- 3:01 PM Changeset in webkit [9881] by
-
- 4 edits in trunk
Reviewed by kevin.
Fixes make clean problem introduced in xcode2.1 transition
- Makefile.am:
- 2:58 PM Changeset in webkit [9880] by
-
- 2 edits in trunk/WebKit
Reviewed by kevin
Fixed make clean problem
- Makefile.am:
- 2:36 PM Changeset in webkit [9879] by
-
- 1 edit in trunk/WebKit/ChangeLog
Patch by <opendarwin.org@mitzpettel.com>
Reviewed by darin and hyatt
Fixes <http://bugzilla.opendarwin.org/show_bug.cgi?id=3862>
The fix for <http://bugzilla.opendarwin.org/show_bug.cgi?id=3545> enclosed each run of visually ordered
hebrew with LRO and PDF control characters, but adjusted the run's to and from to include those characters,
so that they would be rendered if the font includes a glyph for bidi control characters.
Also adding a manual test
- WebCoreSupport.subproj/WebTextRenderer.m: (reverseCharactersInRun):
- 2:35 PM Changeset in webkit [9878] by
-
- 1 edit in trunk/WebCore/ChangeLog-2005-08-23
Adding a manual test for <http://bugzilla.opendarwin.org/show_bug.cgi?id=3862>
Test cases added:
- manual-tests/bidi-visible-control-characters.html: Added.
Jul 23, 2005:
- 1:49 PM Changeset in webkit [9877] by
-
- 1 edit1 add in trunk/WebCore
Adding a manual test for <http://bugzilla.opendarwin.org/show_bug.cgi?id=3862>, since
our layout test font doesn't include glyphs for bidi control characters
Test cases added:
- manual-tests/bidi-visible-control-characters.html: Added.
- 1:45 PM Changeset in webkit [9876] by
-
- 2 edits in trunk/WebKit
Patch by <opendarwin.org@mitzpettel.com>
Reviewed by darin and hyatt
Fixes <http://bugzilla.opendarwin.org/show_bug.cgi?id=3862>
The fix for <http://bugzilla.opendarwin.org/show_bug.cgi?id=3545> enclosed each run of visually ordered
hebrew with LRO and PDF control characters, but adjusted the run's to and from to include those characters,
so that they would be rendered if the font includes a glyph for bidi control characters.
Also adding a manual test (our default layout test font does not include bidi control characters)
- WebCoreSupport.subproj/WebTextRenderer.m: (reverseCharactersInRun):
Jul 22, 2005:
- 5:50 PM Changeset in webkit [9875] by
-
- 2 edits in trunk/JavaScriptCore
Reviewed by darin.
- kjs/date_object.cpp: DatePrototypeImp now identifies itself as a child class of DateInstanceImp -- this enables calls to Date.ValueOf().
fixes: ecma/Date/15.9.5.js (once we enable the date tests).
- 5:34 PM Changeset in webkit [9874] by
-
- 4 edits2 adds1 delete in trunk
Reviewed by darin.
- tests/mozilla/jsDriver.pl: now takes the path to testkjs as a command-line argument
- tests/mozilla/run-mozilla-tests: Removed.
WebKitTools:
Moved Tools/Scripts/run-mozilla-tests to WebKitTools/Scripts/run-javascriptcore-tests.
run-javascriptcore-tests now passes its command-line arguments to jsDriver.pl
Moved Tools/Scripts/update-mozilla-js-test-results to
WebKitTools/Scripts/update-javascriptcore-test-results.
Reviewed by darin.
- Scripts/run-javascriptcore-tests: Added.
- 2:52 PM Changeset in webkit [9873] by
-
- 2 edits in trunk/WebKit
Reviewed by Justin Garcia.
Mail (running on tip of tree WebKit) was running into an assertion I recently added.
The assertion is actually correct, catching an old bug in this code.
- WebView.subproj/WebView.m: (-[WebView selectedFrame]): if the first responder is a WebFrameView, then we've found the WebFrameView we're looking for, and we shouldn't look at its superviews.
- 2:34 PM Changeset in webkit [9872] by
-
- 2 edits in trunk/WebCore
Put back in the <script src="foo"/> self-closing tag quirk in HTML. Unfortunately
Dashboard widgets are dependent on this quirk.
Reviewed by mjs
- khtml/html/htmltokenizer.cpp: (khtml::HTMLTokenizer::parseTag):
- 11:09 AM Changeset in webkit [9871] by
-
- 9 edits in trunk
WebCore:
Reviewed by Chris Blumenberg.
Test cases added: none, this only affects Safari autocomplete; it doesn't affect any
web pages.
- some changes in the direction of weaning all the form-related SPI from NSView
- kwq/DOMExtensions.h: added -[DOMHTMLInputElement isTextField]
- kwq/DOMHTML.mm: (-[DOMHTMLInputElement isTextField]): new method that returns YES if this element is one of the types that's represented by a text field (as opposed to a button, slider, etc.). I could have put this method in Safari, but it seems useful for other WebKit clients.
- kwq/WebCoreBridge.h:
- kwq/WebCoreBridge.mm: (-[WebCoreBridge viewForElement:]): added viewForElement: as a stopgap measure. This allowed me to convert controlsInForm: to return DOMElements rather than NSViews, while keeping autocomplete working in Safari tip of tree. When I finish the SPI conversion I'll delete this method. Note that from this point on, autocomplete will not work in Tiger Safari with tip of tree WebKit (it will always fail to find anything to autocomplete) (-[WebCoreBridge controlsInForm:]): now returns an array of DOMElement* rather than an array of NSView*
WebKit:
Reviewed by Chris Blumenberg.
- some changes in the direction of weaning all the form-related SPI from NSView
- PublicHeaderChangesFromTiger.txt: noted that the WebCore change to add -[DOMHTMLInputElement isTextField] to DOMExtensions.h is a public header change.
- WebView.subproj/WebHTMLRepresentation.h:
- WebView.subproj/WebHTMLRepresentation.m: (-[WebHTMLRepresentation viewForElement:]): added viewForElement: as a stopgap measure. This allowed me to convert controlsInForm: to return DOMElements rather than NSViews, while keeping autocomplete working in Safari tip of tree. When I finish the SPI conversion I'll delete this method. Note that from this point on, autocomplete will not work in Tiger Safari with tip of tree WebKit (it will always fail to find anything to autocomplete)