Timeline
Apr 29, 2005:
- 3:34 PM Changeset in webkit [9084] by
-
- 2 edits in trunk/WebCore
Reviewed by David Harrison.
- khtml/xml/dom_nodeimpl.cpp: (NodeImpl::displayNode): Turn newlines into \n for better dumping.
- 1:13 PM Changeset in webkit [9083] by
-
- 2 edits2 adds in trunk
Reviewed by Darin.
<rdar://problem/4083333> When deleting link at end of sentence, entire sentence gets deleted
- khtml/editing/htmlediting.cpp: (khtml::DeleteSelectionCommand::initializePositionData): Work around bug #4103339 (whose real fix is somewhat risky), so this fix can get into a software update.
(khtml::DeleteSelectionCommand::handleGeneralDelete):
Add isAncestor check when comparing m_downstreamEnd.node() and m_startNode.
- layout-tests/editing/deleting/delete-4083333-fix-expected.txt: Added.
- layout-tests/editing/deleting/delete-4083333-fix.html: Added.
- 12:10 PM Changeset in webkit [9082] by
-
- 294 edits in trunk
Reviewed by Dave Harrison.
- changed layout tests to dump more minimal information about caret and selection
- kwq/KWQRenderTreeDebug.cpp: (nodePosition): Changed name; now does position relative to document. (writeSelection): Removed upstream/downstream code and changed format slightly.
- layout-tests/editing/*-expected.txt: Regenerated in new format.
- 10:28 AM Changeset in webkit [9081] by
-
- 7 edits4 adds in trunk
Reviewed by Chris Blumenberg.
Added two layout tests for regression testing.
- fixed <rdar://problem/4097849> REGRESSION (162-163): importNode creates non-HTML elements, thus style attributes (and some others) don't work
- khtml/xml/dom_docimpl.cpp: (DocumentImpl::importNode): Reorganized and partly rewrote this. The change that fixes the bug at hand is to explicitly use XHTML_NAMESPACE for HTML elements, since the old way of getting the namespace will return the null string for HTML elements, and createElementNS will not create an HTML element if passed a null string for the namespace. (DocumentImpl::processHttpEquiv): Removed some bogus getDocument() calls -- no need to call getDocument() in a document object. (DocumentImpl::attrName): Ditto. (DocumentImpl::tagName): Ditto. (DocumentImpl::setFocusNode): Ditto.
- khtml/html/html_elementimpl.cpp: (HTMLElementImpl::cloneNode): Moved the actual cloning here from ElementImpl::cloneNode, because XMLElementImpl already had its own version, and in here we can use createHTMLElement, which will work properly even in an XML document, and is also slightly more efficient.
- khtml/xml/dom_nodeimpl.h: Added a namespaceURI method function to go along with localName.
- khtml/xml/dom_nodeimpl.cpp: (NodeImpl::namespaceURI): Added. Returns null string to be consistent with localName (only works on certain types of elements as documented).
- khtml/xml/dom_elementimpl.h: Removed ElementImpl::cloneNode (see above). Added an override of namespaceURI for XMLElementImpl.
- khtml/xml/dom_elementimpl.cpp: (XMLElementImpl::namespaceURI): Added. Returns the namespace (consistent with localName).
- layout-tests/fast/dom/importNodeHTML.html: Added. Tests both importNode and cloneNode (for comparison).
- layout-tests/fast/dom/importNodeHTML-expected.txt: Added.
- layout-tests/fast/dom/importNodeXML.xhtml: Added. XML version of the same test as above. Tests a different code path, so useful to have.
- layout-tests/fast/dom/importNodeXML-expected.txt: Added.
Apr 28, 2005:
- 6:13 PM Changeset in webkit [9080] by
-
- 7 edits in trunk/WebKit
Reviewed by Maciej.
- fixed problems preventing us from compiling with gcc 4.0
- WebKit.pbproj/project.pbxproj: Removed -fobjc-exceptions because I can't figure out an easy way to pass it only when compiling Objective-C/C++. Removed -Wmissing-prototypes from WARNING_CPLUSPLUSFLAGS since it's now a C-only warning.
- History.subproj/WebHistoryItem.m: (-[WebHistoryItem pageCache]): Changed return type to match the declaration.
- WebCoreSupport.subproj/WebBridge.m: (-[WebBridge _retrieveKeyboardUIModeFromPreferences:]): Fixed a BOOL that should have been a Boolean.
- WebCoreSupport.subproj/WebTextRenderer.m: Removed redundant copy of ROUND_TO_INT, also in a WebCore header. (-[WebTextRenderer _computeWidthForSpace]): Had to add cast because of difference in type of ROUND_TO_INT vs. CEIL_TO_INT. (pathFromFont): Added a cast to convert UInt8 * to char *.
- WebView.subproj/WebFrameView.m: (-[WebFrameView _setDocumentView:]): Fixed parameter type to match the declaration. (-[WebFrameView documentView]): Fixed return type to match the declaration.
- WebView.subproj/WebHTMLView.m: (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]): Initialized a variable to quiet an incorrect gcc 4.0 uninitialized variable warning. (-[WebHTMLView deleteToMark:]): Switched from @try style to NS_DURING style of exception handler because we can't pass -fobjc-exceptions just to Objective-C at the moment (see above). (-[WebHTMLView selectToMark:]): Ditto. (-[WebHTMLView swapWithMark:]): Ditto.
- 5:31 PM Changeset in webkit [9079] by
-
- 21 edits in trunk/WebCore
Reviewed by Dave Harrison.
- fixed problems preventing us from compiling with gcc 4.0
- WebCore.pbproj/project.pbxproj: Removed -fobjc-exceptions because I can't figure out an easy way to pass it only when compiling Objective-C/C++. Removed -Wmissing-prototypes from WARNING_CPLUSPLUSFLAGS since it's now a C-only warning.
- khtml/css/parser.y: Changed some rules that were using a float to pass around an enum to use an int instead to avoid a warning.
- khtml/css/parser.cpp: Regenerated.
- khtml/css/parser.h: Regenerated.
- khtml/ecma/kjs_dom.cpp: (DOMTextProtoFunc::tryCall): Rearranged a return statement to avoid an incorrect warning.
- khtml/ecma/kjs_html.cpp: (KJS::Context2DFunction::tryCall): Initialized a couple of variables to avoid an incorrect warning.
- khtml/ecma/kjs_proxy.cpp: (KJSProxyImpl::evaluate): Rearranged how we exit from the function to avoid an incorret warning.
- khtml/editing/selection.cpp: (khtml::Selection::debugPosition): Changed some %d to %ld where the parameters where long ints.
- khtml/editing/visible_position.cpp: (khtml::VisiblePosition::debugPosition): Ditto.
- khtml/xml/dom_position.cpp: (DOM::Position::debugPosition): Ditto.
- kwq/DOMEvents.mm: (-[DOMMouseEvent initMouseEvent:::::::::::::::]): Rearranged code to avoid a cast that was causing an incorrect warning.
- kwq/DOMUtility.mm: (createObjCDOMNode): Broke out as a separate function. (KJS::ScriptInterpreter::createObjcInstanceForValue): Rearranged code to avoid a namespace collision with KJS::DOMNode and the Objective-C DOMNode class.
- kwq/KWQFileButton.mm: Made fields of KWQFileButtonAdapter public to avoid an error, new to gcc 4.0, about accessing protected Objective-C fields.
- kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::matchLabelsAgainstElement): Rearranged a return statement to avoid an incorrect warning. (KWQKHTMLPart::imageFromRect): Rearranged how this function does its exception handling to avoid a "may be clobbered" warning.
- kwq/KWQKJavaAppletWidget.mm: Fixed incorrect import that said "KHTMLView.h" instead of "khtmlview.h".
- kwq/KWQObject.mm: Made fields of KWQObjectTimerTarget public to avoid an error, new to gcc 4.0, about accessing protected Objective-C fields.
- kwq/WebCoreBridge.mm: (partHasSelection): Used [bridge part] instead of getting directly at instance variable to avoid an error, new to gcc 4.0, about accessing protected Objective-C fields.
- WebCore-combined.exp: Regenerated.
- WebCore-tests.exp: Added some additional symbols needed by the tests under gcc 4.0.
- 5:22 PM Changeset in webkit [9078] by
-
- 9 edits in trunk/JavaScriptCore
Reviewed by Dave Harrison.
- fixed problems preventing us from compiling with gcc 4.0
- JavaScriptCore.pbproj/project.pbxproj: Removed -Wmissing-prototypes from WARNING_CPLUSPLUSFLAGS since it's now a C-only warning.
- bindings/jni/jni_jsobject.cpp: (JSObject::getSlot): Changed some %d to %ld where the parameters where long ints. (JSObject::setSlot): Ditto.
- bindings/jni/jni_utility.cpp: (KJS::Bindings::getJavaVM): Ditto. (KJS::Bindings::getJNIEnv): Ditto.
- bindings/objc/objc_utility.mm: Fixed include of <JavascriptCore/internal.h> that needed the letter "S" capitalized.
- kjs/bool_object.cpp: (BooleanProtoFuncImp::call): Rearranged how this function returns to avoid incorrect gcc 4.0 warning.
- kjs/collector.cpp: (KJS::Collector::markStackObjectsConservatively): Changed code to check the alignment of the passed-in pointers to only require pointer-level alignment, not 8-byte alignment. Prevents a crash on garbage collect when compiled with gcc 4.0.
- kjs/nodes.cpp: (WhileNode::execute): Added a redundant return after an infinite loop to work around incorrect gcc 4.0 warning. (ForNode::execute): Ditto. (SwitchNode::execute):Rearranged how this function returns to avoid incorrect gcc 4.0 warning. (LabelNode::execute): Ditto.
- kjs/string_object.cpp: (replace): Ditto.
- 5:07 PM Changeset in webkit [9077] by
-
- 1 edit in trunk/WebCore/ChangeLog-2005-08-23
Fix ChangeLog
- 5:07 PM Changeset in webkit [9076] by
-
- 2 edits in trunk/WebCore
- WebCore.pbproj/project.pbxproj: Add back the main_thread_malloc files, rolled out by accident.
Apr 27, 2005:
- 6:04 PM Changeset in webkit [9075] by
-
- 6 edits in trunk/WebCore
Rolling out fix for <rdar://problem/4097849> because the following layout tests were failing:
fast/css/namespaces/001
fast/css/namespaces/002
fast/css/namespaces/004
fast/css/namespaces/005
fast/css/namespaces/006
fast/overflow/003
- khtml/html/html_documentimpl.cpp:
- khtml/html/html_documentimpl.h:
- khtml/html/html_elementimpl.cpp: (HTMLElementImpl::namespaceURI):
- khtml/xml/dom_docimpl.cpp: (DocumentImpl::importNode): (DocumentImpl::createElementNS): (DocumentImpl::createHTMLElement): (DocumentImpl::attrId): (DocumentImpl::tagId):
- khtml/xml/dom_docimpl.h:
- 6:03 PM Changeset in webkit [9074]
-
- 4 copies2 deletes in tags/WebCore-413~1
This commit was manufactured by cvs2svn to create tag
'WebCore-413~1'.
- 6:03 PM Changeset in webkit [9073] by
-
- 1 edit in trunk/WebCore/WebCore.pbproj/project.pbxproj
one more tweak
- 5:06 PM Changeset in webkit [9072] by
-
- 1 edit in trunk/WebCore/WebCore.pbproj/project.pbxproj
WebCore-413.1
- 4:14 PM Changeset in webkit [9071]
-
- 34 copies2 deletes in tags/WebCore-415~1
This commit was manufactured by cvs2svn to create tag
'WebCore-415~1'.
- 4:14 PM Changeset in webkit [9070] by
-
- 2 edits in branches/Safari-2-0-branch/WebCore
- merge this fix from HEAD
This fix was included in WebCore-413, which shipped in Tiger GM (8A428). However,
Safari-2-0-anchor was mistakenly placed at WebCore-412. Merging this fix to
the branch so that this fix isn't lost in the SU.
2005-03-27 Darin Adler <Darin Adler>
Reviewed by me, fix by Kida-san.
- fixed <rdar://problem/4067474> 8A424: Safari immediately quit by Cmd+Ctrll+'D'
- kwq/WebCoreBridge.mm: (-[WebCoreBridge convertToNSRange:DOM::]): Added nil check.
- 2:53 PM Changeset in webkit [9069] by
-
- 2 edits in branches/experimental-ui-branch/WebCore
Rolled in the resizable textarea change from tip of tree, so all three of my
experimental prototypes are on the same branch, for ease of demoing. For the
experimental-ui-branch, ALLOW_RESIZING_TEXTAREAS is on unless you recompile
it away.
===
This code allows you to press near the bottom-right corner of any textarea and drag
to resize the textarea on the page. It works correctly with textareas in left-aligned
or centered blocks, but is weird in right-aligned blocks. It also does something
sensible if the width is specified as a % (in that case, you can resize vertically only).
The user-created-size survives resizing the window and survives the back/forward cache.
It does not survive reloading the page.
This complete-lack-of-affordance UI is obviously not shippable, but this proof of concept
code could lead to a real user feature.
- kwq/KWQTextArea.mm: (-[KWQTextArea getNumColumns:andNumRows:forSize:]): new method that determines cols and rows for a given textarea frame size (-[KWQTextAreaTextView _trackResizeFromMouseDown:]): new method that tracks a drag and does a live resize-and-relayout (-[KWQTextAreaTextView mouseDown:]): if the mouse down is in the bottom-right corner, call _trackResizeFromMouseDown:
- 2:41 PM Changeset in webkit [9068] by
-
- 3 edits in branches/Safari-2-0-branch/WebCore
Merging John's fix for alt/shift bug in <rdar://problem/3655817> fix from TOT.
2005-04-27 John Sullivan <sullivan@apple.com>
Reviewed by Adele.
- fixed problem with mouse wheel patch where alt and shift keys were switched
- khtml/xml/dom2_eventsimpl.h: use the order cntl, alt, shift, meta in KeyboardEventImpl constructors to match superclass. This isn't necessary to fix the bug, but is better for clarity.
- khtml/xml/dom2_eventsimpl.cpp: (KeyboardEventImpl::KeyboardEventImpl): switch parameter order to match superclass. Then pass the parameters to superclass's constructor in the right order. (KeyboardEventImpl::initKeyboardEvent): Pass parameters to superclass's constructor in the right order.
- 2:34 PM Changeset in webkit [9067] by
-
- 2 edits in trunk/WebCore
Reviewed by Darin.
Experimental prototype of user-resizable textareas. All the new code is guarded by
#if ALLOW_RESIZING_TEXTAREAS, which is false unless you remove a comment and rebuild.
This code allows you to press near the bottom-right corner of any textarea and drag
to resize the textarea on the page. It works correctly with textareas in left-aligned
or centered blocks, but is weird in right-aligned blocks. It also does something
sensible if the width is specified as a % (in that case, you can resize vertically only).
The user-created-size survives resizing the window and survives the back/forward cache.
It does not survive reloading the page.
This complete-lack-of-affordance UI is obviously not shippable, but this proof of concept
code could lead to a real user feature.
- kwq/KWQTextArea.mm: (-[KWQTextArea getNumColumns:andNumRows:forSize:]): new method that determines cols and rows for a given textarea frame size (-[KWQTextAreaTextView _trackResizeFromMouseDown:]): new method that tracks a drag and does a live resize-and-relayout (-[KWQTextAreaTextView mouseDown:]): if the mouse down is in the bottom-right corner, call _trackResizeFromMouseDown:
- 2:32 PM Changeset in webkit [9066] by
-
- 3 edits in trunk/WebCore
Reviewed by Adele.
- fixed problem with mouse wheel patch where alt and shift keys were switched
- khtml/xml/dom2_eventsimpl.h: use the order cntl, alt, shift, meta in KeyboardEventImpl constructors to match superclass. This isn't necessary to fix the bug, but is better for clarity.
- khtml/xml/dom2_eventsimpl.cpp: (KeyboardEventImpl::KeyboardEventImpl): switch parameter order to match superclass. Then pass the parameters to superclass's constructor in the right order. (KeyboardEventImpl::initKeyboardEvent): Pass parameters to superclass's constructor in the right order.
- 11:36 AM Changeset in webkit [9065] by
-
- 2 edits in branches/Safari-2-0-branch/WebCore
Versioning for SUTiAtlanta - WebCore-415.1
- 10:59 AM Changeset in webkit [9064] by
-
- 2 edits in trunk/WebKit
Reviewed by Dave Harrison.
- fixed <rdar://problem/3547489> pop-up window blocking preference and menu item can easily get out of sync.
- WebView.subproj/WebPreferences.m: (-[WebPreferences _setStringValue:forKey:]): save local value before setting value in NSUserDefaults, so clients reacting to NSUserDefaults change notification but calling back on WebPreferences API will see the updated value. (-[WebPreferences _setIntegerValue:forKey:]): ditto (-[WebPreferences _setBoolValue:forKey:]): ditto
- 9:58 AM Changeset in webkit [9063] by
-
- 3 edits in branches/Safari-2-0-branch/WebCore
Removed fix for 4032346 and merged new fix as described in 4097842.
2005-04-18 David Hyatt <hyatt@apple.com>
Back out fix for 4032346, since it is causing garbled image content on many sites.
The bug tracking the fix is 4069093.
(khtml::RenderBlock::matchedEndLine):
2005-04-25 David Hyatt <hyatt@apple.com>
Fix for 4097842, changing 1st line of a url that wraps doesn't update the second line. Make sure to
update line break info for the last dirty line so that when a clean line tries to figure out where it
started, it will get an accurate position taking into account the updated string. By doing this, the code
will be able to properly distinguish between the old and new positions and not accidentally assume they are
the same.
Reviewed by darin
- khtml/rendering/render_text.cpp: (RenderText::setTextWithOffset):
