Timeline
Jun 5, 2004:
- 9:28 PM Changeset in webkit [6780] by
-
- 2 edits in trunk/WebCore
As agreed with Hyatt and Louch, do not post incoming dragging events
to text nodes.
Reviewed by Maciej
- khtml/khtmlview.cpp: (KHTMLView::updateDragAndDrop): For any drag target that is a text node, use its parent instead.
Jun 4, 2004:
- 8:21 PM Changeset in webkit [6779] by
-
- 16 edits in trunk
WebCore:
DHTML dragging uses the Cocoa NSDragOperation on both the source and dest ends.
In addition, I caught a prime gaffe where I had used "dropAllowed" instead of "effectAllowed"
for that particular WinIE property. That is renamed throughout.
Reviewed by rjw
- khtml/ecma/kjs_events.cpp: (stringOrUndefined): New little helper. (Clipboard::getValueProperty): Return Undefined if dropEffect or effectAllowed are not set. (Clipboard::putValue): Rename.
- khtml/ecma/kjs_events.h: (KJS::Clipboard::): Rename.
- khtml/ecma/kjs_events.lut.h:
- khtml/xml/dom2_eventsimpl.h: Rename.
- kwq/KWQClipboard.h: Rename, add access to Cocoa drag op.
- kwq/KWQClipboard.mm: (KWQClipboard::dropEffect): These 4 just moved in the file. (KWQClipboard::setDropEffect): (KWQClipboard::effectAllowed): (KWQClipboard::setEffectAllowed): (cocoaOpFromIEOp): Convert from an IE operation string to a Cocoa DragOp (IEOpFromCocoaOp): and vice-versa (KWQClipboard::sourceOperation): Return effectAllowed, converted to a NSDragOp (KWQClipboard::destinationOperation): Return dropEffect, converted to a NSDragOp (KWQClipboard::setSourceOperation): Set effectAllowed, using a Cocoa value (KWQClipboard::setDestinationOperation): Set dropEffect, using a Cocoa value
- kwq/KWQKHTMLPart.h:
- kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::dispatchDragSrcEvent): Return the drag op specified by the source element. (KWQKHTMLPart::khtmlMouseMoveEvent): Pass our drag op up to WebKit. (KWQKHTMLPart::dragSourceMovedTo): Pass NULL for new arg. (KWQKHTMLPart::dragSourceEndedAt): Ditto
- kwq/WebCoreBridge.h:
- kwq/WebCoreBridge.mm: (-[WebCoreBridge dragOperationForDraggingInfo:]): Set the incoming dragOp onto the clipboard so DHTML can access it. In addition, validate whatever op DHTML returns so we play well with Cocoa.
WebKit:
DHTML dragging uses the Cocoa NSDragOperation on both the source and dest ends.
Most of the real work is in WebCore.
Reviewed by rjw
- WebCoreSupport.subproj/WebBridge.m: (-[WebBridge startDraggingImage:at:operation:event:]): Pass the drag op along.
- WebView.subproj/WebHTMLView.m: (-[WebHTMLView _startDraggingImage:at:operation:event:]): Remember drag op from WC. (-[WebHTMLView mouseDragged:]): Pass None for the op (alternate code path that will be going away). (-[WebHTMLView draggingSourceOperationMaskForLocal:]): Use the WC drag op.
- WebView.subproj/WebHTMLViewInternal.h:
- WebView.subproj/WebHTMLViewPrivate.h:
- 4:27 PM Changeset in webkit [6778] by
-
- 7 edits in trunk/WebCore
Add support for auto values in flex transitions. Add support for mapping the back end values to
front end values. Next it's time to actually try to use this stuff in render_flexbox.
Reviewed by darin
- khtml/css/css_valueimpl.cpp: (length):
- khtml/css/css_valueimpl.h: (DOM::FlexGroupTransitionValueImpl::isAuto):
- khtml/css/cssparser.cpp: (FlexGroupTransitionParseContext::length): (FlexGroupTransitionParseContext::commitAutoValue): (FlexGroupTransitionParseContext::commitValue): (CSSParser::parseFlexGroupTransition):
- khtml/css/cssstyleselector.cpp: (khtml::CSSStyleSelector::applyProperty):
- khtml/rendering/render_style.cpp: (FlexGroupTransitionData::operator==):
- khtml/rendering/render_style.h: (khtml::FlexGroupTransitionData::next): (khtml::FlexGroupTransitionData::isAuto):
- 4:00 PM Changeset in webkit [6777] by
-
- 1 edit in trunk/JavaScriptCore/JavaScriptCore.pbproj/project.pbxproj
change the JSC version number from 143.1 back to 144u
- 3:47 PM Changeset in webkit [6776]
-
- 1 copy3 deletes in tags/JavaScriptCore-143~1
This commit was manufactured by cvs2svn to create tag
'JavaScriptCore-143~1'.
- 3:47 PM Changeset in webkit [6775] by
-
- 2 edits in trunk/JavaScriptCore
release marker and version change for JavaScriptCore-143.1
- 3:16 PM Changeset in webkit [6774] by
-
- 2 edits in trunk/JavaScriptCore
Reviewed by Darin
- fixed <rdar://problem/3680594>
- kjs/object.cpp: (KJS::Error::create):
- 3:01 PM Changeset in webkit [6773] by
-
- 3 edits in trunk
change version number to '144u'. the tree is open.
- 2:55 PM Changeset in webkit [6772]
-
- 3 copies in tags/Safari-143
This commit was manufactured by cvs2svn to create tag 'Safari-143'.
- 2:55 PM Changeset in webkit [6771] by
-
- 6 edits in trunk
Safari 2.0 (v143) stamp
- 2:12 PM Changeset in webkit [6770] by
-
- 35 edits in branches/AutoZoneGC_Branch_20040430
Turned off zero link in projects. Converted all Objective-C++ objects to use KWQRetain/KWQRelease/KWQRetainNSRelease to manage lifetimes of Objective-C objects for GC compatibility. No longer allocating C++ objects in GC-scanned memory. Removed absolute paths to files on my hard disk in projects.
- 1:19 PM Changeset in webkit [6769] by
-
- 2 edits in trunk/JavaScriptCore
- kjs/testkjs.cpp: (main): Fix build breakage by adding URL and line number parameters.
- 11:49 AM Changeset in webkit [6768] by
-
- 23 edits in trunk
Reviewed by Dave.
- ObjC bindings do not (yet) pass along sourceurl or line numbers
- we don't have a way as of yet to accomidate line numbers and urls for dynamic javascript
- changed the wording of an error message
- the lexer, parser, and interpreter have been made "sourceURL aware"
- stored the url into Error
- bindings/NP_jsobject.cpp: (NPN_Evaluate):
- bindings/jni/jni_jsobject.cpp: (JSObject::eval):
- bindings/objc/WebScriptObject.mm: (-[WebScriptObject evaluateWebScript:]):
- kjs/function.cpp: (GlobalFuncImp::call):
- kjs/function_object.cpp: (FunctionObjectImp::construct):
- kjs/internal.cpp: (Parser::parse): (InterpreterImp::checkSyntax): (InterpreterImp::evaluate):
- kjs/internal.h:
- kjs/interpreter.cpp: (Interpreter::evaluate):
- kjs/interpreter.h:
- kjs/lexer.cpp: (Lexer::setCode):
- kjs/lexer.h: (KJS::Lexer::sourceURL):
- kjs/nodes.cpp: (Node::Node): (Node::throwError): (FunctionCallNode::evaluate):
- kjs/nodes.h:
- kjs/object.cpp: (KJS::Error::create):
- kjs/object.h:
WebCore:
Reviewed by Dave.
- kwq/KWQKHTMLPart.mm: addMessagetoConsole places sourceURL in the dictionary
- khtml/xml/dom_docimpl.cpp: right now, we don't have a way to get a url, so we leave this blank
- khtml/ecma/kjs_window.cpp: still need to get the real line number and sourceURL
- khtml/ecma/kjs_proxy.cpp: now passing the sourceURL to addMessageToConsole
- khtml/ecma/kjs_events.cpp: still need to grab an accurate line number and sourceURL
- khtml/ecma/kjs_events.cpp: (JSEventListener::handleEvent): (JSLazyEventListener::handleEvent):
- khtml/ecma/kjs_proxy.cpp: (KJSProxyImpl::evaluate):
- khtml/ecma/kjs_window.cpp: (Window::isSafeScript): (ScheduledAction::execute):
- khtml/xml/dom_docimpl.cpp: (DocumentImpl::open):
- kwq/KWQKHTMLPart.h:
- kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::addMessageToConsole):
WebBrowser:
Reviewed by Dave.
- model is now source url aware
- refined the ui with custom NSCells that now display error messages, line numbers, and source urls
- increased the row size to 35 pixels to accomimdate for ErrorCells
- Debug/ErrorCell.h: Added.
- Debug/ErrorCell.m: Added. (-[ErrorCell setURL:]): (-[ErrorCell setErrorMessasge:]): (-[ErrorCell setLineNumber:]): (-[ErrorCell drawInteriorWithFrame:inView:]):
- Debug/ErrorConsole.nib:
- Debug/ErrorConsoleController.m: (-[ErrorConsoleController init]): (-[ErrorConsoleController awakeFromNib]):
- Debug/ErrorConsoleModel.h:
- Debug/ErrorConsoleModel.m: (-[ErrorConsoleModel tableView:willDisplayCell:forTableColumn:row:]): (-[ErrorConsoleModel count]): (-[ErrorConsoleModel selectedURL]):
- WebBrowser.pbproj/project.pbxproj:
- 11:35 AM Changeset in webkit [6767] by
-
- 2 edits in trunk/WebCore
Reviewed by Hyatt
Fix a comical little bug where the style-applying code did not take into
account deleting content from the start of a block. After the deletion,
the caret is placed in the node before the deleted content and so
inserting a styled fragment after the deletion position works great.
The problem is that if the selectionis at the start of a block, there
is no "position before the deletion" and the caret is placed in the new
first child of the block. Inserting the styled content after this
node is just plain wrong. Now, this case is handled correctly, and the
styled content is inserted in the proper position.
- khtml/editing/htmlediting_impl.cpp: (khtml::ApplyStyleCommandImpl::insertFragment):
- 11:21 AM Changeset in webkit [6766] by
-
- 2 edits in trunk/WebCore
Reviewed by Hyatt
- khtml/rendering/bidi.cpp: (khtml::RenderBlock::layoutInlineChildren): Only add additional line height in for root editable elements. This helps to keep layout from deviating too much from the intended page layout, but still gives an empty document a line in which it can blink the caret.
- 11:17 AM Changeset in webkit [6765] by
-
- 9 edits2 adds in trunk/WebCore
Reviewed by Hyatt
Made some progress on this set of related bugs:
<rdar://problem/3675867>: "Make execCommand work as specified in the Javascript execCommand Compatibility Plan"
<rdar://problem/3675898>: "Make queryCommandEnabled work as specified in the Javascript execCommand Compatibility Plan"
<rdar://problem/3675899>: "Make queryCommandIndeterm work as specified in the Javascript execCommand Compatibility Plan"
<rdar://problem/3675901>: "Make queryCommandState work as specified in the Javascript execCommand Compatibility Plan"
<rdar://problem/3675903>: "Make queryCommandSupported work as specified in the Javascript execCommand Compatibility Plan"
<rdar://problem/3675904>: "Make queryCommandValue work as specified in the Javascript execCommand Compatibility Plan"
I did several things to move the ahead with the plan:
- The latter five methods did not appear in the tree at all. Now they do, although
- I removed the implementation for these editing methods from dom_docimpl.cpp, and moved them out to new js_editing.cpp/js_editing.h files. The remaining code in dom_docimpl.cpp is glue to call over to the new JSEditor object defined in the new files.
- For a couple of the command implementations in js_editing.cpp, I made a stab at returning sensible values. For others, I just return place-holder values. I have added some comments to describe what has been done and what still needs to be done.
- WebCore.pbproj/project.pbxproj:
- khtml/dom/dom_doc.cpp: Add call-overs for these methods. (DOM::Document::queryCommandEnabled): (DOM::Document::queryCommandIndeterm): (DOM::Document::queryCommandState): (DOM::Document::queryCommandSupported): (DOM::Document::queryCommandValue):
- khtml/dom/dom_doc.h:
- khtml/ecma/kjs_dom.cpp: Add queryCommandXXX terminology. (DOMDocumentProtoFunc::tryCall):
- khtml/ecma/kjs_dom.h: (KJS::DOMDocument::):
- khtml/ecma/kjs_dom.lut.h: (KJS::):
- khtml/editing/jsediting.cpp: Added. This file implements the guts of editing-related JS methods. (DOM::execCommandNotImplemented): (DOM::queryBoolNotImplemented): (DOM::queryValueNotImplemented): (DOM::JSEditor::commandDict): (DOM::JSEditor::JSEditor): (DOM::JSEditor::~JSEditor): (DOM::JSEditor::commandIdentifier): (DOM::JSEditor::addCommand): (DOM::JSEditor::initDict): (DOM::JSEditor::execCommand): (DOM::JSEditor::queryCommandEnabled): (DOM::JSEditor::queryCommandIndeterm): (DOM::JSEditor::queryCommandState): (DOM::JSEditor::queryCommandSupported): (DOM::JSEditor::queryCommandValue): (DOM::JSEditor::execCommandCopy): (DOM::JSEditor::execCommandCut): (DOM::JSEditor::execCommandDelete): (DOM::JSEditor::execCommandInsertText): (DOM::JSEditor::execCommandPaste): (DOM::JSEditor::execCommandRedo): (DOM::JSEditor::execCommandSelectAll): (DOM::JSEditor::execCommandUndo): (DOM::JSEditor::enabledIfPartNotNull): (DOM::JSEditor::enabledIfSelectionNotEmpty): (DOM::JSEditor::enabledIfSelectionIsRange): (DOM::JSEditor::commandSupported):
- khtml/editing/jsediting.h: Added. (DOM::JSEditor::CommandIdentifier::CommandIdentifier): (DOM::JSEditor::document): (DOM::JSEditor::part):
- khtml/xml/dom_docimpl.cpp: Rework JS editing support into glue that calls over into new JSEditor object. (DocumentImpl::DocumentImpl): (DocumentImpl::~DocumentImpl): (DocumentImpl::jsEditor): (DocumentImpl::execCommand): (DocumentImpl::queryCommandEnabled): (DocumentImpl::queryCommandIndeterm): (DocumentImpl::queryCommandState): (DocumentImpl::queryCommandSupported): (DocumentImpl::queryCommandValue):
- khtml/xml/dom_docimpl.h:
- 11:02 AM Changeset in webkit [6764] by
-
- 4 edits in trunk/WebKit
Fixed: <rdar://problem/3674921>: (can't drag an image from Desktop to Blot document)
Reviewed by mjs.
- WebView.subproj/WebDataSource.m: (-[WebDataSource _imageElementWithImageResource:]): factored out from _documentFragmentWithImageResource: (-[WebDataSource _documentFragmentWithImageResource:]): call _imageElementWithImageResource:
- WebView.subproj/WebDataSourcePrivate.h:
- WebView.subproj/WebHTMLView.m: (-[WebHTMLView _imageExistsAtPaths:]): new (-[WebHTMLView _documentFragmentWithPaths:]): new (-[WebHTMLView _documentFragmentFromPasteboard:allowPlainText:]): handle NSFilenamesPboardType (+[WebHTMLView _insertablePasteboardTypes]): include NSFilenamesPboardType (-[WebHTMLView _canProcessDragWithDraggingInfo:]): check for NSFilenamesPboardType and check that the files are images
- 10:54 AM Changeset in webkit [6763] by
-
- 5 edits in trunk/JavaScriptCore
Fixed crash when attempting to access properties on nil
object.
Reviewed by John.
- bindings/objc/objc_instance.mm: (ObjcInstance::getClass):
- bindings/runtime_object.cpp: (RuntimeObjectImp::get):
- bindings/testM.js:
- bindings/testbindings.mm: (-[MyFirstInterface getString]):
- 9:12 AM Changeset in webkit [6762] by
-
- 2 edits in trunk/WebKit
Fixed crasher from last checkin.
Reviewed by Ken.
- Plugins.subproj/WebPluginDatabase.m: (+[WebPluginDatabase setAdditionalWebPlugInPaths:]): (pluginLocations):
Jun 3, 2004:
- 10:04 PM Changeset in webkit [6761] by
-
- 2 adds in branches/AutoZoneGC_Branch_20040430/WebCore/kwq
Inline implementations of KWQRetain, KWQRelease, and KWQRetainNSRelease. Conditional implementation of NSAutoreleasePool category.
- 6:36 PM Changeset in webkit [6760] by
-
- 6 edits in trunk/WebCore
Added setCompositeOperation method to Context2D.
Actually pass composite operation to drawPixmap (instead of 1).
Reviewed by jay lo.
- khtml/ecma/kjs_html.cpp: (KJS::Context2DFunction::tryCall):
- khtml/ecma/kjs_html.h: (KJS::Context2D::):
- khtml/ecma/kjs_html.lut.h: (KJS::):
- kwq/KWQPainter.h:
- kwq/KWQPainter.mm: (QPainter::compositeOperatorFromString): (QPainter::drawPixmap):
- 6:01 PM Changeset in webkit [6759] by
-
- 3 edits in trunk/WebKit
Add SPI to allow setting of plugin load path.
Reviewed by Hyatt.
- Plugins.subproj/WebPluginDatabase.h:
- Plugins.subproj/WebPluginDatabase.m: (+[WebPluginDatabase setAdditionalWebPlugInPaths:]): (pluginLocations):
- 5:39 PM Changeset in webkit [6758] by
-
- 10 edits in trunk/WebCore
Add support for box-flex-group-transition (whew!), a new property that is going to enable some incredibly
complicated flexing layouts in HTML.
This is Stage 1 - wire up the CSS property and make it get parsed into front-end data. Nobody actually looks
at this data yet.
Reviewed by darin
- khtml/css/css_valueimpl.cpp: (CSSInheritedValueImpl::cssText): (ShadowValueImpl::cssText): (length): (FlexGroupTransitionValueImpl::~FlexGroupTransitionValueImpl): (FlexGroupTransitionValueImpl::cssText):
- khtml/css/css_valueimpl.h: (DOM::FlexGroupTransitionValueImpl::cssValueType):
- khtml/css/cssparser.cpp: (CSSParser::parseValue): (CSSParser::parseShadow): (FlexGroupTransitionParseContext::length): (FlexGroupTransitionParseContext::~FlexGroupTransitionParseContext): (FlexGroupTransitionParseContext::failed): (FlexGroupTransitionParseContext::allowGroup): (FlexGroupTransitionParseContext::commitGroup): (FlexGroupTransitionParseContext::commitSlash): (FlexGroupTransitionParseContext::commitLength): (FlexGroupTransitionParseContext::commitValue): (CSSParser::parseFlexGroupTransition):
- khtml/css/cssparser.h:
- khtml/css/cssproperties.c: (hash_prop): (findProp):
- khtml/css/cssproperties.h:
- khtml/css/cssproperties.in:
- khtml/rendering/render_style.cpp: (StyleFlexibleBoxData::StyleFlexibleBoxData): (StyleFlexibleBoxData::operator==): (StyleFlexibleBoxData::transitionDataEquivalent): (StyleCSS3InheritedData::operator==): (StyleCSS3InheritedData::shadowDataEquivalent): (ShadowData::operator==): (RenderStyle::setBoxFlexGroupTransition): (length): (FlexGroupTransitionData::operator==):
- khtml/rendering/render_style.h: (khtml::FlexGroupTransitionData::next): (khtml::FlexGroupTransitionData::~FlexGroupTransitionData): (khtml::FlexGroupTransitionData::operator!=): (khtml::StyleFlexibleBoxData::~StyleFlexibleBoxData): (khtml::RenderStyle::boxFlexGroupTransition):
- 5:21 PM Changeset in webkit [6757] by
-
- 2 edits in trunk/WebCore
Add extra sanity check to Image parameter of drawImage...
Reviewed by Hyatt.
- khtml/ecma/kjs_html.cpp: (KJS::Context2DFunction::tryCall):
- 2:36 PM Changeset in webkit [6756] by
-
- 2 edits2 adds in trunk
Fix for 3673931, negative margins on objects that dodge floats not handled correctly.
Reviewed by darin
- khtml/rendering/render_block.cpp: (khtml::RenderBlock::layoutBlockChildren):
- 2:24 PM Changeset in webkit [6755] by
-
- 3 edits in trunk/WebKit
Reviewed by Chris.
- fixed <rdar://problem/3677038>: (Need SPI to get URL of favicon for a site)
- Misc.subproj/WebIconDatabase.h: Add iconURLForURL: method.
- Misc.subproj/WebIconDatabase.m: (-[WebIconDatabase iconURLForURL:]): Added.
- 2:17 PM Changeset in webkit [6754] by
-
- 1 edit in trunk/WebCore/ChangeLog-2005-08-23
Removed some tabs from the ChangeLog.
- 8:22 AM Changeset in webkit [6753] by
-
- 4 edits in trunk/WebKit
Reviewed by John
Fix for these bugs:
<rdar://problem/3675806>: "Make API name change for -webViewShouldBeginEditing:inDOMRange:"
<rdar://problem/3675809>: "Make API name change for -webViewShouldEndEditing:inDOMRange:"
New names are -webView:shouldBeginEditingInDOMRange: and -webView:shouldEndEditingInDOMRange:,
respectively.
- WebView.subproj/WebDefaultEditingDelegate.m: (-[WebDefaultEditingDelegate webView:shouldBeginEditingInDOMRange:]): (-[WebDefaultEditingDelegate webView:shouldEndEditingInDOMRange:]):
- WebView.subproj/WebEditingDelegate.h:
- WebView.subproj/WebView.m: (-[WebView _shouldBeginEditingInDOMRange:]): (-[WebView _shouldEndEditingInDOMRange:]):
- 7:38 AM Changeset in webkit [6752] by
-
- 11 edits in trunk/WebCore
Reviewed by Hyatt
Fix for layout regression I caused when fixing:
<rdar://problem/3668619>: "REGRESSION: text placed on pasteboard by WebKit is offset by one character"
Extra height is added to empty blocks that are editable, so we can click to place
the caret in them. This extra height was erroneously being added to non-editable
blocks as a result of my earlier change. After looking at this and doing some
code review, there are the following changes:
Change away from "containingBlock" terminology in the NodeImpl class. The
operation we are doing in NodeImpl is not the same as what is done in CSS
when it uses the term "containingBlock" so:
containingBlock changes to enclosingBlockFlowElement, and
rootEditableBlock changes to rootEditableElement
The vast majority of changes here are to make these name changes. The other
significant piece of work, and the fix for the regression, is to change
bidi.cpp to only include this extra line height if a block is empty
and is content-editable.
- khtml/editing/htmlediting_impl.cpp: (khtml::leadingWhitespacePosition): (khtml::trailingWhitespacePosition): (khtml::DeleteSelectionCommandImpl::doApply): (khtml::InputNewlineCommandImpl::doApply): (khtml::RemoveNodeAndPruneCommandImpl::doApply): (khtml::TypingCommandImpl::issueCommandForDeleteKey):
- khtml/rendering/bidi.cpp: (khtml::RenderBlock::layoutInlineChildren):
- khtml/xml/dom_docimpl.cpp: (DocumentImpl::relinquishesEditingFocus): (DocumentImpl::acceptsEditingFocus):
- khtml/xml/dom_nodeimpl.cpp:
- khtml/xml/dom_nodeimpl.h:
- khtml/xml/dom_position.cpp: (DOM::Position::equivalentLeafPosition): (DOM::Position::previousCharacterPosition): (DOM::Position::nextCharacterPosition): (DOM::Position::previousLinePosition): (DOM::Position::nextLinePosition): (DOM::Position::equivalentUpstreamPosition): (DOM::Position::equivalentDownstreamPosition): (DOM::Position::atStartOfRootEditableElement): (DOM::Position::inRenderedContent): (DOM::Position::rendersOnSameLine): (DOM::Position::rendersInDifferentPosition): (DOM::Position::isLastRenderedPositionInEditableBlock): (DOM::Position::inFirstEditableInRootEditableElement): (DOM::Position::inLastEditableInRootEditableElement): (DOM::Position::inFirstEditableInContainingEditableBlock): (DOM::Position::inLastEditableInContainingEditableBlock):
- khtml/xml/dom_position.h:
- khtml/xml/dom_selection.cpp: (DOM::Selection::moveToRenderedContent):
- 12:01 AM Changeset in webkit [6751] by
-
- 1 edit in trunk/WebCore/kwq/KWQClipboard.mm
Ooops, missed one small change from code review in previous checkin.