Timeline
Mar 15, 2006:
- 11:15 PM Changeset in webkit [13326] by
-
- 5 edits in trunk/WebCore
2006-03-15 Eric Seidel <eseidel@apple.com>
Reviewed by darin.
Remove unused code and clean-up style.
- bridge/mac/MacFrame.mm: (WebCore::MacFrame::advanceToNextMisspelling): (WebCore::MacFrame::wheelEvent): (WebCore::MacFrame::nextKeyViewInFrame): (WebCore::MacFrame::nextKeyViewInFrameHierarchy): (WebCore::MacFrame::nextKeyView):
- page/Frame.cpp:
- page/Frame.h:
- page/FrameView.cpp: (WebCore::FrameView::dispatchMouseEvent):
- 10:21 PM Changeset in webkit [13325] by
-
- 15 edits2 adds in trunk
2006-03-15 Eric Seidel <eseidel@apple.com>
Reviewed by mjs.
http://bugzilla.opendarwin.org/show_bug.cgi?id=7790
addEventListener does not work with "dblclick" event.
Renamed khtmlDblclickEvent to dblclickEvent to fix.
Test: fast/events/dblclick-addEventListener.html
- WebCore.xcodeproj/project.pbxproj:
- bridge/mac/MacFrame.mm: (WebCore::MacFrame::mouseUp): (WebCore::MacFrame::mouseMoved): (WebCore::MacFrame::shouldDragAutoNode): (WebCore::MacFrame::sendContextMenuEvent):
- dom/EventNames.h:
- dom/NodeImpl.cpp: (WebCore::NodeImpl::dispatchMouseEvent):
- dom/NodeImpl.h:
- khtml/ecma/kjs_binding.cpp: (KJS::ScriptInterpreter::wasRunByUserGesture):
- khtml/ecma/kjs_dom.cpp: (KJS::DOMNode::getValueProperty): (KJS::DOMNode::putValueProperty):
- khtml/ecma/kjs_window.cpp: (KJS::Window::getValueProperty): (KJS::Window::put):
- khtml/html/HTMLElementImpl.cpp: (WebCore::HTMLElementImpl::parseMappedAttribute):
- page/FrameView.cpp: (WebCore::FrameView::dispatchMouseEvent):
- page/FrameView.h:
- 10:12 PM Changeset in webkit [13324] by
-
- 5 edits4 adds in trunk
Reviewed by Maciej.
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=4395 REGRESSION: document.open doesn't clear the document
Test: fast/dom/Document/document-reopen.html
Test: fast/dom/Document/doc-open-while-parsing.html
- dom/DocumentImpl.cpp: (WebCore::DocumentImpl::open): Only ignore open() while the frame is still loading the main resource.
- page/Frame.cpp: (WebCore::Frame::isComplete): Added const specifier. (WebCore::Frame::isLoadingMainResource): New function.
- page/Frame.h:
- 8:53 PM Changeset in webkit [13323] by
-
- 2 edits in trunk/WebKitTools
Reviewed by Maciej.
- DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj: Fix path that had a lowercase "i" in it so this builds on case-sensitive file systems.
- 7:44 PM Changeset in webkit [13322] by
-
- 5 deletes in trunk
Removed a few stray .cvsignore files that crept in.
- 6:00 PM Changeset in webkit [13321] by
-
- 1 edit182 adds in trunk/LayoutTests
Rubber stamped by Geoff.
even more mozilla table tests.
- 5:44 PM Changeset in webkit [13320] by
-
- 1 edit2 adds in trunk/WebKitTools
Reviewed by Eric.
Added run-testkjs and compare-timing-files scripts, to support super
accurate JS iBench.
- Scripts/compare-timing-files: Added.
- Scripts/run-testkjs: Added.
- 5:15 PM Changeset in webkit [13319] by
-
- 2 edits in trunk/WebKit
Fix by Patrick Beard, reviewed by me.
- fix <rdar://problem/4478181> WebPluginController leaks NSArray and NSMutableSet objects
- Plugins/WebPluginController.m: (-[WebPluginController initWithDocumentView:]): Call CFMakeCollectable on the CFSet so we can act like it's an NSSet. (-[WebPluginController dealloc]): Release _views and _checksInProgress.
- 4:44 PM Changeset in webkit [13318] by
-
- 2 edits in trunk/JavaScriptCore
2006-03-15 Eric Seidel <eseidel@apple.com>
Reviewed by adele.
Build fix.
- kjs/date_object.cpp: (KJS::DateProtoFunc::callAsFunction): use size() not "len()"
- 4:40 PM Changeset in webkit [13317] by
-
- 5 edits in trunk/WebCore
Reviewed by Eric.
- Fixes win32 build.
- page/Frame.cpp: (WebCore::Frame::autoscrollTimerFired): Calls isMouseButtonDown instead of CG function.
- platform/MouseEvent.h: Added isMouseButtonDown
- platform/mac/MouseEventMac.mm: (WebCore::MouseEvent::isMouseButtonDown): Calls CG function to get state of mouse button.
- platform/win/TemporaryLinkStubs.cpp: (MouseEvent::isMouseButtonDown): Added stub.
- 4:35 PM Changeset in webkit [13316] by
-
- 1 edit848 adds in trunk/LayoutTests
Rubber stamped by Geoff.
Even more Mozilla tables tests.
- 4:03 PM Changeset in webkit [13315] by
-
- 1 edit849 adds in trunk/LayoutTests
Rubber stamped by Geoff
More Mozilla table tests.
- 3:01 PM Changeset in webkit [13314] by
-
- 1 edit617 adds in trunk/LayoutTests
Rubber stamped by Geoff.
First round of committing Mozilla table tests to our tree.
- 2:46 PM Changeset in webkit [13313] by
-
- 1 edit in trunk/WebKitSite/projects/compat/hitlist.html
forgot to change the date
- 2:28 PM Changeset in webkit [13312] by
-
- 2 edits in trunk/WebCore
2006-03-15 Eric Seidel <eseidel@apple.com>
Reviewed by geoff.
- platform/KeyEvent.h: Fix #define from QEvent_H to KeyEvent_H
- 2:26 PM Changeset in webkit [13311] by
-
- 3 edits in trunk/JavaScriptCore
2006-03-15 Eric Seidel <eseidel@apple.com>
Reviewed by mjs.
Fix CString copy constructor, fixes Date.parse("") on Win32.
- kjs/date_object.cpp: (KJS::DateProtoFunc::callAsFunction):
- kjs/ustring.cpp: (KJS::CString::CString): (KJS::CString::operator=):
- 1:59 PM Changeset in webkit [13310] by
-
- 1 edit in trunk/WebKitSite/projects/compat/hitlist.html
added www.live.com to list and reordered by bug number.
- 1:29 PM Changeset in webkit [13309] by
-
- 2 edits in trunk/WebKitSite
Reviewed by TimH.
- projects/compat/hitlist.html: Moved delta.com to the alumni list, added description of alumni list.
- 12:56 PM Changeset in webkit [13308] by
-
- 1 edit in trunk/LayoutTests/fast/box-sizing/percentage-height-expected.txt
Check in updated results for fast/box-sizing/percentage-height.html. All results for this test are wrong, but only in DumpRenderTree. Safari renders it correctly.
- 12:12 PM Changeset in webkit [13307] by
-
- 2 edits in trunk/WebCore
Fix the box-sizing layout test by stopping integer overflowing from
occurring in the flexbox code.
Reviewed by beth
- rendering/render_flexbox.cpp: (khtml::RenderFlexibleBox::allowedChildFlex):
- 10:43 AM Changeset in webkit [13306] by
-
- 2 edits in trunk/WebCore
Reviewed by Eric
Fix for some leaks and intermittent broken pipes in the layout
tests after check-in for http://bugzilla.opendarwin.org/
show_bug.cgi?id=4171.
- 10:26 AM Changeset in webkit [13305] by
-
- 1 edit in trunk/LayoutTests/ChangeLog
Update information about when a bug was fixed.
- 2:21 AM Changeset in webkit [13304] by
-
- 13 edits in trunk/JavaScriptCore
Reviewed by Anders.
- KJS::Node and KJS::StatementNode are bigger than they need to be http://bugzilla.opendarwin.org/show_bug.cgi?id=7775
The memory usage of Node was reduced by 2 machine words per node:
- sourceURL was removed and only kept on FunctionBodyNode. The source URL can only be distinct per function or top-level program node, and you always have one.
- refcount was removed and kept in a separate hashtable when greater than 1. newNodes set represents floating nodes with refcount of 0. This helps because almost all nodes have a refcount of 1 for almost all of their lifetime.
- bindings/runtime_method.cpp: (RuntimeMethod::RuntimeMethod): Pass null body, added FIXME.
- kjs/Parser.cpp: (KJS::clearNewNodes): New nodes are tracked in nodes.cpp now, but still clear them at the appropriate time.
- kjs/context.h: (KJS::ContextImp::currentBody): added; used to retrieve source URL and sid for current code. (KJS::ContextImp::pushIteration): moved here from LabelStack (KJS::ContextImp::popIteration): ditto (KJS::ContextImp::inIteration): ditto (KJS::ContextImp::pushSwitch): ditto (KJS::ContextImp::popSwitch): ditto (KJS::ContextImp::inSwitch): ditto
- kjs/function.cpp: (KJS::FunctionImp::FunctionImp): Add FunctionBodyNode* parameter. (KJS::FunctionImp::callAsFunction): Pass body to ContextImp. (KJS::FunctionImp::argumentsGetter): _context renamed to m_context. (KJS::DeclaredFunctionImp::DeclaredFunctionImp): Pass body to superclass constructor. (KJS::GlobalFuncImp::callAsFunction): Pass progNode as body for ContextImp in eval.
- kjs/function.h: Move body field from DeclaredFunctionImp to FunctionImp.
- kjs/grammar.y: Change DBG; statements no longer have a sourceid.
- kjs/internal.cpp: (KJS::ContextImp::ContextImp): Initialize new m_currentBody, m_iterationDepth and m_switchDepth data members. New FunctionBodyNode* parameter - the function body provides source URL and SourceId. (KJS::InterpreterImp::mark): Use exception() function, not _exception directly. (KJS::InterpreterImp::evaluate): Pass progNode to ContextImp constructor to use as the body.
- kjs/internal.h: (KJS::LabelStack::LabelStack): Remove iteration depth and switch depth; statement label stacks don't need these and it bloats their size. Put them in the ContextImp instead.
- kjs/interpreter.cpp: (KJS::ExecState::lexicalInterpreter): Renamed _context to m_context.
- kjs/interpreter.h: (KJS::ExecState::dynamicInterpreter): Renamed _context to m_context. (KJS::ExecState::context): ditto (KJS::ExecState::setException): Renamed _exception to m_exception (KJS::ExecState::clearException): ditto (KJS::ExecState::exception): ditto (KJS::ExecState::hadException): ditto (KJS::ExecState::ExecState): ditto both above renames
- kjs/nodes.cpp: (Node::Node): Removed initialization of line, source URL and refcount. Add to local newNodes set instead of involving parser. (Node::ref): Instead of managing refcount directly, story refcount over 1 in a HashCountedSet, and keep a separate HashSet of "floating" nodes with refcount 0. (Node::deref): ditto (Node::refcount): ditto (Node::clearNewNodes): Destroy anything left in the new nodes set. (currentSourceId): Inline helper to get sourceId from function body via context. (currentSourceURL): ditto for sourceURL. (Node::createErrorCompletion): use new helper (Node::throwError): ditto (Node::setExceptionDetailsIfNeeded): ditto (StatementNode::StatementNode): remove initialization of l0 and sid, rename l1 to m_lastLine. (StatementNode::setLoc): Set own m_lastLine and Node's m_line. (StatementNode::hitStatement): Get sid, first line, last line in the proper new ways. (StatListNode::StatListNode): updated for setLoc changes (BlockNode::BlockNode): ditto (DoWhileNode::execute): excpect iteraton counts on ContextImp, not LabelStack (WhileNode::execute): ditto (ForNode::execute): ditto (ForInNode::execute): ditto (ContinueNode::execute): excpect inIteration on ContextImp, not LabelStack (BreakNode::execute): excpect inIteration and inSwitch on ContextImp, not LabelStack (SwitchNode::execute): expect switch counts on ContextImp, not LabelStack (FunctionBodyNode::FunctionBodyNode): update for new setLoc (FunctionBodyNode::processFuncDecl): reindent (SourceElementsNode::SourceElementsNode): update for new setLoc
- kjs/nodes.h: (KJS::Node::lineNo): Renamed _line to m_line (KJS::StatementNode::firstLine): Use lineNo() (KJS::StatementNode::lastLine): Renamed l1 to m_lastLine (KJS::FunctionBodyNode::sourceId): added (KJS::FunctionBodyNode::sourceURL): added
- kjs/testkjs.cpp:
- 1:41 AM Changeset in webkit [13303] by
-
- 2 edits in trunk/WebCore
Reviewed by Maciej.
Followup fix for previous checkin.
http://bugzilla.opendarwin.org/show_bug.cgi?id=7114
<rdar://problem/3695120> dragging to scroll doesn't work with overflow:auto/scroll/overlay areas
Many of the bugs related to the Radar bug have to do with autoscrolling in the RSS View.
This fix walks the layer tree to find the first scrollable layer, so that autoscrolling will
also work with nested layers, like those in the RSS View.
- bridge/mac/MacFrame.mm: (WebCore::MacFrame::khtmlMouseMoveEvent):
- 1:05 AM Changeset in webkit [13302] by
-
- 7 edits in trunk/WebCore
Reviewed by Maciej.
- Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=7114 Dragging to scroll doesn't work for overflow areas
No test case. There doesn't appear to be a way to trigger the autoscroll with the EventSender.
- bridge/mac/MacFrame.mm: (WebCore::MacFrame::khtmlMouseMoveEvent): If we have a layer that can scroll, let the layer handle its autoscroll. Otherwise, call over the bridge to let AppKit scroll the view. (WebCore::MacFrame::khtmlMouseReleaseEvent): Stops the autoscroll timer.
- page/Frame.cpp: (WebCore::Frame::khtmlMouseReleaseEvent): Stops the autoscroll timer. (WebCore::Frame::handleAutoscroll): Added. Saves the layer, and starts the timer. (WebCore::Frame::autoscrollTimerFired): Added. Calls autoscroll() on the layer. (WebCore::Frame::startAutoscrollTimer): Added. (WebCore::Frame::stopAutoscrollTimer): Added.
- page/Frame.h: Added new autoscroll methods.
- page/FramePrivate.h: (WebCore::FramePrivate::FramePrivate): Initialize m_autoscrollTimer and m_autoscrollLayer.
- rendering/render_layer.cpp: (WebCore::RenderLayer::autoscroll): Calculates how much the layer should scroll, and actually scrolls. (WebCore::RenderLayer::shouldAutoscroll): If the layer has overflow then it should be able to scroll. Except for overflow:hidden areas that aren't editable. In the future, we may want to have a css property that indicates an overflow:hidden style that should also scroll.
- rendering/render_layer.h: Added new autoscroll methods.
Mar 14, 2006:
- 11:01 PM Changeset in webkit [13301] by
-
- 9 edits6 adds in trunk
LayoutTests:
Reviewed by mjs
<http://bugzilla.opendarwin.org/show_bug.cgi?id=7578>
TinyMCE: Implement execCommand CreateLink
- editing/editing.js:
- editing/execCommand/createLink-expected.checksum: Added.
- editing/execCommand/createLink-expected.png: Added.
- editing/execCommand/createLink-expected.txt: Added.
- editing/execCommand/createLink.html: Added.
WebCore:
Reviewed by mjs
<http://bugzilla.opendarwin.org/show_bug.cgi?id=7578>
TinyMCE: Implement execCommand CreateLink
Used ApplyStyleCommand so that creating a link from
a selection doesn't change document structure.
- WebCore.xcodeproj/project.pbxproj:
- editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::ApplyStyleCommand): (WebCore::ApplyStyleCommand::doApply): (WebCore::ApplyStyleCommand::removeInlineStyle): (WebCore::ApplyStyleCommand::addInlineStyleIfNeeded):
- editing/ApplyStyleCommand.h:
- editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::applyStyle):
- editing/CreateLinkCommand.cpp: Added. (WebCore::CreateLinkCommand::CreateLinkCommand): (WebCore::CreateLinkCommand::doApply):
- editing/CreateLinkCommand.h: Added.
- editing/jsediting.cpp:
- page/Frame.cpp: (WebCore::Frame::computeAndSetTypingStyle): (WebCore::Frame::applyStyle): (WebCore::Frame::applyParagraphStyle):
- 10:40 PM Changeset in webkit [13300] by
-
- 12 edits in trunk
Reviewed by Maciej
Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=4171
This patch changes the word-break algorithm used to apply text-
transform:capitalize to use a UBreakIterator. This fixes some
existing edge cases we have in our text-transform:capitalize
support and generally makes our results more consistent.
- 7:41 PM Changeset in webkit [13299] by
-
- 7 edits in trunk/LayoutTests
Rubber stamped by Hyatt.
- land new results for two tests affected by Hyatt's layer change
- fast/encoding/utf-16-big-endian-expected.checksum: Updated.
- fast/encoding/utf-16-big-endian-expected.png: Ditto.
- fast/encoding/utf-16-big-endian-expected.txt: Ditto.
- fast/encoding/utf-16-little-endian-expected.checksum: Ditto.
- fast/encoding/utf-16-little-endian-expected.png: Ditto.
- fast/encoding/utf-16-little-endian-expected.txt: Ditto.
- 7:25 PM Changeset in webkit [13298] by
-
- 5 edits in trunk/WebCore
Reviewed by adele
<http://bugzilla.opendarwin.org/show_bug.cgi?id=7770>
Should only dispatch the khtmlTextInsertedEvent for high level editing operations
Dispatching this event for low level editing operations
isn't necessary and causes a big performance regression.
Also renamed the event because it's dispatched for changes
other than just text insertion.
- dom/EventNames.h:
- editing/EditCommand.cpp: (WebCore::EditCommand::apply): (WebCore::EditCommand::unapply): (WebCore::EditCommand::reapply):
- khtml/html/HTMLTextFieldInnerElementImpl.cpp: (WebCore::HTMLTextFieldInnerElementImpl::defaultEventHandler):
- page/Frame.cpp: (WebCore::dispatchKHTMLEditableContentChanged): (WebCore::Frame::appliedEditing): (WebCore::Frame::unappliedEditing): (WebCore::Frame::reappliedEditing):
- 6:47 PM Changeset in webkit [13297] by
-
- 3 edits in trunk/LayoutTests
- land new results for tests affected by my setAttribute change
- dom/xhtml/level3/core/nodeisequalnode15-expected.txt: Updated to reflect a new kind of failure.
- dom/xhtml/level3/core/nodeisequalnode16-expected.txt: Updated to reflect success.
- 6:07 PM Changeset in webkit [13296] by
-
- 1 edit1 add in trunk/LayoutTests
Oops, forgot to add this file.
- fast/js/resources/string-sort.js: Added.
- 5:17 PM Changeset in webkit [13295] by
-
- 2 edits in trunk/WebCore
Reviewed by Eric.
- fix <rdar://problem/4478121> -[WebCoreFrameBridge finalize] crashes under GC
- bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge finalize]): Don't call setBridge(nil).
- 5:09 PM Changeset in webkit [13294] by
-
- 7 edits2 adds in trunk
- Fixed <rdar://problem/4478239> string sort puts "closed" before "close"
Reviewed by Eric.
- kjs/ustring.cpp: (KJS::compare): Inverted a < in order to treat the longer string as > the shorter string.
LayoutTests:
Reviewed by Eric.
Layout test for <rdar://problem/4478239> string sort puts "closed"
before "close"
Also changed the engine to report data types when tests fail, so that
you don't get messages like, "should be A, was A."
Updated results for these files:
- fast/js/kde/Array-expected.txt:
- fast/js/kde/RegExp-expected.txt:
- fast/js/kde/encode_decode_uri-expected.txt:
- fast/js/resources/js-test-pre.js:
Added these files:
- fast/js/string-sort-expected.txt: Added.
- fast/js/string-sort.html: Added.
- 3:54 PM Changeset in webkit [13293] by
-
- 2 edits in trunk/WebCore
- oops, didn't mean to check this in
- dom/dom_elementimpl.cpp: (WebCore::StyledElementImpl::parseMappedAttribute): Reverted nil-document check that I didn't mean to remove.
- 3:53 PM Changeset in webkit [13292] by
-
- 1 edit4 adds in trunk/LayoutTests
Reviewed by Hyatt
Layout test for http://bugzilla.opendarwin.org/show_bug.cgi?id=6796
- 3:50 PM Changeset in webkit [13291] by
-
- 3 edits in trunk/WebCore
Reviewed by Hyatt
Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=6796
This fix prevents infinite recursion when a block with overflow:
auto can't decide if it needs scrollbars or not. This fix puts a
guard in place in updateScrollInforAfterLayout that only allows you
to re-layout if you are not currently laying out.
- 3:01 PM Changeset in webkit [13290] by
-
- 1 edit2 adds in trunk/LayoutTests
Reviewed by Maciej.
- Test for http://bugzilla.opendarwin.org/show_bug.cgi?id=4059 Some values used with setAttribute() cause the attribute to be removed (null strings vs. empty strings)
The fix occurred somewhere between r9638 (2005-07-09) and the first CVS
nightly build (WebKit-CVS-2005-10-01 03-27-01 GMT.dmg).
- fast/dom/setAttribute-using-initial-input-value-expected.txt: Added.
- fast/dom/setAttribute-using-initial-input-value.html: Added.
- 2:58 PM Changeset in webkit [13289] by
-
- 8 edits2 adds in trunk
LayoutTests:
Reviewed by Geoff and Eric.
- test for <http://bugzilla.opendarwin.org/show_bug.cgi?id=7765>, <rdar://problem/4474884> hasAttribute always returns false for uppercase attribute names like "STYLE"
- also incidentally tests getAttributeNS and handling of null namespace parameters
- fast/dom/Element/attribute-uppercase.html: Added.
- fast/dom/Element/attribute-uppercase-expected.txt: Added.
WebCore:
Reviewed by Geoff and Eric.
- fix <http://bugzilla.opendarwin.org/show_bug.cgi?id=7765>, <rdar://problem/4474884> hasAttribute always returns false for uppercase attribute names like "STYLE"
- re-add missing JavaScript binding for getAttributeNS
- fixed handling of null namespace parameters in getAttributeNS and friends
Test: fast/dom/Element/attribute-uppercase.html
To match Gecko's behavior, the magic "lowercasing" of all attribute names
is only done when calling DOM Level 1 functions that lack an NS suffix.
The functions with an NS suffix are case sensitive, even in HTML documents.
- dom/NamedNodeMapImpl.h: Changed getNamedItem and removeNamedItem to be pure virtual functions instead of calling through to getNamedItemNS and removeNamedItemNS.
- dom/dom_elementimpl.h: Eliminated inline versions of non-NS functions that called the NS functions. They can't work that way any more since the lowercasing behavior is different.
- dom/dom_elementimpl.cpp: (WebCore::ElementImpl::getAttribute): Added. Lowercases the name if the document is an HTML document, and then calls through to the qualified name version. Removed the null check since getDocument is guaranteed to never return 0 now that nodes hold a reference to their document. (WebCore::ElementImpl::getAttributeNS): Removed the lowercasing code. (WebCore::ElementImpl::setAttribute): More of the same. (WebCore::ElementImpl::setAttributeNS): Ditto. (WebCore::ElementImpl::removeAttribute): Ditto. (WebCore::ElementImpl::removeAttributeNS): Ditto. (WebCore::ElementImpl::getAttributeNode): Ditto. (WebCore::ElementImpl::getAttributeNodeNS): Ditto. (WebCore::ElementImpl::hasAttribute): Ditto. (WebCore::ElementImpl::hasAttributeNS): Ditto. (WebCore::NamedAttrMapImpl::getNamedItem): Ditto. (WebCore::NamedAttrMapImpl::getNamedItemNS): Ditto. (WebCore::NamedAttrMapImpl::removeNamedItem): Ditto. (WebCore::NamedAttrMapImpl::removeNamedItemNS): Ditto. (WebCore::StyledElementImpl::parseMappedAttribute): Removed another unneeded null check like the one mentioned above.
- dom/Element.idl: Set ConvertNullToNullString for all the namespaceURI parameters, since we don't want null to turn into "null". Added a semicolon to the end of the hasAttributeNS line -- was causing the next declaration to be eaten up (so getAttributeNS was completely broken).
- khtml/ecma/kjs_dom.cpp: (KJS::DOMNamedNodeMapProtoFunc::callAsFunction): Call valueToStringWithNullCheck to do the equivalent of ConvertNullToNullString for both getNamedItemNS and removeNamedItemNS.
- bindings/js/JSDOMCore.cpp: Touched so that the changes above will take effect.
- 12:11 PM Changeset in webkit [13288] by
-
- 4 edits in trunk/WebKit
Reviewed by Maciej.
<rdar://problem/4476873> Support printing for plugin documents (not embedded plugins)
- WebView/WebFrameViewPrivate.h:
- WebView/WebFrameView.m: (-[WebFrameView documentViewShouldHandlePrint]): Called by the host application before it initializes and runs a print operation. If NO is returned, the host application will abort its print operation and call -printDocumentView on the WebFrameView. The document view is then expected to run its own print operation. If YES is returned, the host application's print operation will continue as normal. (-[WebFrameView printDocumentView]): Called by the host application when the WebFrameView returns YES from -documentViewShouldHandlePrint.
- Plugins/WebNetscapePluginDocumentView.m: (-[WebNetscapePluginDocumentView documentViewShouldHandlePrint]): Allow the plugin to take over printing if it defines an NPP_Print function (-[WebNetscapePluginDocumentView printDocumentView]): Print the plugin document.
- 3:21 AM Changeset in webkit [13287] by
-
- 1 edit in trunk/WebCore/rendering/render_layer.cpp
Fix the setpos to move screwup. Didn't realize move was a delta.
- 3:03 AM Changeset in webkit [13286] by
-
- 4 edits in trunk/WebCore
Fix for bug 7259, opacity performs badly.
Reviewed by mjs
Adding some tests in fast/opacity.
- rendering/RenderTableRow.h: (WebCore::RenderTableRow::requiresLayer):
- rendering/render_layer.cpp: (WebCore::transparencyClipBox): (WebCore::RenderLayer::beginTransparencyLayers): (WebCore::RenderLayer::paintLayer): (WebCore::RenderLayer::intersectsDamageRect): (WebCore::RenderLayer::absoluteBoundingBox):
- rendering/render_layer.h: (WebCore::RenderLayer::root):
- 2:52 AM Changeset in webkit [13285] by
-
- 2 edits in trunk/WebKitTools
2006-03-14 Eric Seidel <eseidel@apple.com>
Reviewed by mjs.
Make the URL bar relinquish focus on page load.
- Spinneret/Spinneret/Spinneret.cpp: (MyEditProc):
- 2:33 AM Changeset in webkit [13284] by
-
- 2 edits in trunk/WebKitTools
2006-03-14 Eric Seidel <eseidel@apple.com>
Reviewed by mjs.
Fix WebView to allow KeyFocus.
Add handling of space and shift-space for scrolling.
- Spinneret/Spinneret/WebView.cpp: (WebKit::scrollMessageForKey): (WebKit::WebViewWndProc):
- 1:49 AM Changeset in webkit [13283] by
-
- 2 edits in trunk/WebCore
2006-03-14 Eric Seidel <eseidel@apple.com>
Reviewed by mjs.
Fix Gif crasher.
- platform/image-decoders/gif/GIFImageReader.cpp: (GIFImageReader::do_lzw): Initialize array after new.
- 1:19 AM Changeset in webkit [13282] by
-
- 2 edits in trunk/WebCore
2006-03-14 Eric Seidel <eseidel@apple.com>
Reviewed by mjs.
Make MouseEvents aware of current scroll position.
- platform/win/ScrollViewWin.cpp: (WebCore::ScrollView::viewportToContents): (WebCore::ScrollView::contentsToViewport):