16 years agoadded www.live.com to list and reordered by bug number.
aliceli1 [Wed, 15 Mar 2006 21:59:50 +0000 (21:59 +0000)]
added live.com to list and reordered by bug number.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13310 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by TimH.
ggaren [Wed, 15 Mar 2006 21:29:59 +0000 (21:29 +0000)]
    Reviewed by TimH.

        * projects/compat/hitlist.html: Moved delta.com to the alumni list,
        added description of alumni list.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13309 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoCheck in updated results for fast/box-sizing/percentage-height.html. All results...
hyatt [Wed, 15 Mar 2006 20:56:32 +0000 (20:56 +0000)]
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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13308 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Fix the box-sizing layout test by stopping integer overflowing from
hyatt [Wed, 15 Mar 2006 20:12:18 +0000 (20:12 +0000)]
    Fix the box-sizing layout test by stopping integer overflowing from
        occurring in the flexbox code.

        Reviewed by beth

        * rendering/render_flexbox.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13307 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Eric
bdakin [Wed, 15 Mar 2006 18:43:08 +0000 (18:43 +0000)]
    Reviewed by Eric

        Fix for some leaks and intermittent broken pipes in the layout
        tests after check-in for http://bugzilla.opendarwin.org/

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13306 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoUpdate information about when a bug was fixed.
darin [Wed, 15 Mar 2006 18:26:00 +0000 (18:26 +0000)]
Update information about when a bug was fixed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13305 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Anders.
mjs [Wed, 15 Mar 2006 10:21:48 +0000 (10:21 +0000)]
    Reviewed by Anders.

        - KJS::Node and KJS::StatementNode are bigger than they need to be

        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
        (KJS::GlobalFuncImp::callAsFunction): Pass progNode as body for ContextImp in
        * kjs/function.h: Move body field from DeclaredFunctionImp to
        * 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
        (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:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13304 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Maciej.
adele [Wed, 15 Mar 2006 09:41:41 +0000 (09:41 +0000)]
   Reviewed by Maciej.

       Followup fix for previous checkin.
       <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):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13303 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Maciej.
adele [Wed, 15 Mar 2006 09:05:41 +0000 (09:05 +0000)]
    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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13302 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoLayoutTests:
justing [Wed, 15 Mar 2006 07:01:21 +0000 (07:01 +0000)]

        Reviewed by mjs

        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.


        Reviewed by mjs

        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:
        * editing/ApplyStyleCommand.h:
        * editing/CompositeEditCommand.cpp:
        * editing/CreateLinkCommand.cpp: Added.
        * editing/CreateLinkCommand.h: Added.
        * editing/jsediting.cpp:
        * page/Frame.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13301 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Maciej
bdakin [Wed, 15 Mar 2006 06:40:42 +0000 (06:40 +0000)]
    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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13300 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Rubber stamped by Hyatt.
darin [Wed, 15 Mar 2006 03:41:55 +0000 (03:41 +0000)]
    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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13299 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by adele
justing [Wed, 15 Mar 2006 03:25:50 +0000 (03:25 +0000)]
    Reviewed by adele

        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:
        * khtml/html/HTMLTextFieldInnerElementImpl.cpp:
        * page/Frame.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13298 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago - land new results for tests affected by my setAttribute change
darin [Wed, 15 Mar 2006 02:47:25 +0000 (02:47 +0000)]
    - 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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13297 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Oops, forgot to add this file.
ggaren [Wed, 15 Mar 2006 02:07:09 +0000 (02:07 +0000)]
    Oops, forgot to add this file.

        * fast/js/resources/string-sort.js: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13296 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Eric.
darin [Wed, 15 Mar 2006 01:17:59 +0000 (01:17 +0000)]
    Reviewed by Eric.

        - fix <rdar://problem/4478121> -[WebCoreFrameBridge finalize] crashes under GC

        * bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge finalize]):
        Don't call setBridge(nil).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13295 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoJavaScriptCore:
ggaren [Wed, 15 Mar 2006 01:09:15 +0000 (01:09 +0000)]

        - Fixed <rdar://problem/4478239> string sort puts "closed" before

        Reviewed by Eric.

        * kjs/ustring.cpp:
        (KJS::compare): Inverted a < in order to treat the longer string as >
        the shorter string.


        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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13294 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago - oops, didn't mean to check this in
darin [Tue, 14 Mar 2006 23:54:21 +0000 (23:54 +0000)]
    - 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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13293 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Hyatt
bdakin [Tue, 14 Mar 2006 23:53:50 +0000 (23:53 +0000)]
    Reviewed by Hyatt

        Layout test for http://bugzilla.opendarwin.org/show_bug.cgi?id=6796

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13292 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Hyatt
bdakin [Tue, 14 Mar 2006 23:50:51 +0000 (23:50 +0000)]
    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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13291 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Maciej.
darin [Tue, 14 Mar 2006 23:01:44 +0000 (23:01 +0000)]
    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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13290 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoLayoutTests:
darin [Tue, 14 Mar 2006 22:58:25 +0000 (22:58 +0000)]

        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.


        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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13289 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Maciej.
tomernic [Tue, 14 Mar 2006 20:11:29 +0000 (20:11 +0000)]
    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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13288 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoFix the setpos to move screwup. Didn't realize move was a delta.
hyatt [Tue, 14 Mar 2006 11:21:11 +0000 (11:21 +0000)]
Fix the setpos to move screwup.  Didn't realize move was a delta.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13287 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Fix for bug 7259, opacity performs badly.
hyatt [Tue, 14 Mar 2006 11:03:25 +0000 (11:03 +0000)]
    Fix for bug 7259, opacity performs badly.

        Reviewed by mjs

        Adding some tests in fast/opacity.

        * rendering/RenderTableRow.h:
        * rendering/render_layer.cpp:
        * rendering/render_layer.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13286 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-14 Eric Seidel <eseidel@apple.com>
eseidel [Tue, 14 Mar 2006 10:52:37 +0000 (10:52 +0000)]
2006-03-14  Eric Seidel  <eseidel@apple.com>

        Reviewed by mjs.

        Make the URL bar relinquish focus on page load.

        * Spinneret/Spinneret/Spinneret.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13285 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-14 Eric Seidel <eseidel@apple.com>
eseidel [Tue, 14 Mar 2006 10:33:50 +0000 (10:33 +0000)]
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:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13284 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-14 Eric Seidel <eseidel@apple.com>
eseidel [Tue, 14 Mar 2006 09:49:47 +0000 (09:49 +0000)]
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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13283 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-14 Eric Seidel <eseidel@apple.com>
eseidel [Tue, 14 Mar 2006 09:19:36 +0000 (09:19 +0000)]
2006-03-14  Eric Seidel  <eseidel@apple.com>

        Reviewed by mjs.

        Make MouseEvents aware of current scroll position.

        * platform/win/ScrollViewWin.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13282 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-13 Eric Seidel <eseidel@apple.com>
eseidel [Tue, 14 Mar 2006 07:40:34 +0000 (07:40 +0000)]
2006-03-13  Eric Seidel  <eseidel@apple.com>

        Reviewed by mjs.

        Fix mouse events to be window-relative.

        * platform/win/MouseEventWin.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13281 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-10 Eric Seidel <eseidel@apple.com>
eseidel [Tue, 14 Mar 2006 07:05:29 +0000 (07:05 +0000)]
2006-03-10  Eric Seidel  <eseidel@apple.com>

        Reviewed by hyatt.

        Add scrolling support into Spinneret.

        * Spinneret/Spinneret/WebFrame.cpp:
        * Spinneret/Spinneret/WebView.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13280 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-13 Eric Seidel <eseidel@apple.com>
eseidel [Tue, 14 Mar 2006 06:25:17 +0000 (06:25 +0000)]
2006-03-13  Eric Seidel  <eseidel@apple.com>

        Reviewed by mjs.

        Close DeviceContext leaks in Font code.
        Also add moveCursor stub to make maps.google.com not crash.

        * platform/win/FontWin.cpp:
        * platform/win/TemporaryLinkStubs.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13279 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-13 Eric Seidel <eseidel@apple.com>
eseidel [Tue, 14 Mar 2006 04:50:09 +0000 (04:50 +0000)]
2006-03-13  Eric Seidel  <eseidel@apple.com>

        Reviewed by mjs.

        Fix checksum generation on Intel machines.
        Also moved to CGImage APIs instead of NSBitmapImageRep, may possibly
        give a small speed boost now that it uses a shared buffer.

        * DumpRenderTree/DumpRenderTree.m:
        * DumpRenderTree/ImageDiff.m:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13278 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-13 Eric Seidel <eseidel@apple.com>
eseidel [Tue, 14 Mar 2006 03:30:36 +0000 (03:30 +0000)]
2006-03-13  Eric Seidel  <eseidel@apple.com>

        Reviewed by ggaren & TimH.

        <rdar://problem/4473288> XSL file containing <xsl:include href="../Styles/EscapeString.xsl" /> fails to load file
        XML_DTD_NODE objects in the tree were causing loadChildSheets to
        fail to preload child stylesheets, this patch causes us to
        correctly ignore all nodes besides the first XML_ELEMENT_NODE at
        the root level when searching the tree for xsl:include statements
        to pre-load.

        Test: fast/xsl/xslt-second-level-import.xml

        * khtml/xsl/xsl_stylesheetimpl.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13277 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-10 Rob Buis <buis@kde.org>
eseidel [Tue, 14 Mar 2006 02:26:07 +0000 (02:26 +0000)]
2006-03-10  Rob Buis  <buis@kde.org>

        Reviewed by darin.  Landed by eseidel.

        Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=7666:
        dasharray on rounded rectangles is broken

        Make sure the rounded rectangle is drawn clockwise, as outlined
        in the specification.

        Test: svg/custom/dashArrayOrigin.svg

        * kcanvas/KCanvasCreator.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13276 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by timo.
ggaren [Mon, 13 Mar 2006 22:16:12 +0000 (22:16 +0000)]
    Reviewed by timo.

        - Fixed the load progress indicator to give more incremental feedback,
        and to stop spending so much time near 100%.

        I did two things:
        (1) Fixed some bugs and a misspelling in the previous heuristic's
        (2) Added two new rules to the heuristic:
            (a) Treat the first layout as the half-way point.
            (b) Just like we jump the first 10% to indicate that a load has
            started, jump the last 10% to indicate that a load has finished.

        Rule 2a is good for two reasons. First, it seems unnatural for loading
        to be "more than half done" when you can't even see anything. Second,
        in the early stages of laading our estimate of how much we'll need to
        load is often off by as much as 6000% (e.g., cnn.com). So anything that
        makes the progress indicator more conservative in the early stages of
        loading is helpful.

        Rule 2b is good because it's confusing for loading to be "100% done"
        but still ongoing.

        FIXME: The indicator still isn't perfect. For example, the old behavior
        shows up @ moviefone.com. Two areas for future work:
        (1) Estimate number of linked resources. Our code estimates the size
        of a single resource, but does nothing to estimate the number of
        resources that resource might link to. This is the key to why we're
        so wrong at the beginning.
        (2) Improve "when to do first layout" heuristic. A JavaScript query
        for a style property forces layout, creating a phantom first layout
        with no content, essentially nullifying 2a for certain pages.

        Filed <rdar://problem/4475834> to track estimating the number of
        linked resources. Phantom layouts are already on Hyatt's radar.

        * WebView/WebFrame.m:
        (-[WebFrame _setState:]): Update firstLayoutDone
        (-[WebFrame _numPendingOrLoadingRequests:]): Bug fix: In the recurisve
        case, query 'frame' instead of 'self', so that we actually recurse.
        (-[WebFrame _firstLayoutDone]): New method
        (-[WebFrame _didFirstLayout]): Update firstLayoutDone
        * WebView/WebFramePrivate.h: Added firstLayoutDone ivar
        * WebView/WebView.m:
        (-[WebView _incrementProgressForConnectionDelegate:data:]):
        (1) Implemented 2a and 2b
        (2) Bug fix: only update the 'last time I sent a notification' time if
        we actually send a notification.
        (3) Don't test for progress < 0 because ensuring progress < max
        also ensures max - progress > 0. (Do still test for progress > max
        because rounding errors make that a possibility -- although a very
        minor one.)
        (4) Query only the loading frame and its subframes for pending
        requests instead of defaulting to the main frame. This is a slight
        optimization in cases where the main frame did not begin the load,
        and it makes the code more consistent.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13269 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Tim Hatcher.
darin [Mon, 13 Mar 2006 20:00:54 +0000 (20:00 +0000)]
    Reviewed by Tim Hatcher.

        * Scripts/run-webkit-tests: Fix httpd handling to work on systems that
        have it named httpd-1.3 instead.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13268 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Fix proposed by Mitz Pettel, reviewed by Darin.
ap [Mon, 13 Mar 2006 19:55:58 +0000 (19:55 +0000)]
    Fix proposed by Mitz Pettel, reviewed by Darin.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7718
          run-webkit-tests fast/dom/HTMLObjectElement/ hangs

        * Scripts/run-webkit-tests: Make test paths canonical, to form proper URLs
        (sequences of slashes are equivalent to a single slash in POSIX paths, but not
        in URLs).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13267 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Tim Omernick.
sullivan [Mon, 13 Mar 2006 19:29:24 +0000 (19:29 +0000)]
    Reviewed by Tim Omernick.

        - fixed <rdar://problem/4475857> API: Setting a history item limit programmatically doesn't work

        * History/WebHistory.m:
        (-[WebHistoryPrivate _loadHistoryGuts:URL:error:]):
        use [self historyItemLimit], which prefers the explicitly-set value and falls back to the
        NSUserDefaults value, instead of using the NSUserDefaults value explicitly.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13266 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Fix by Mitz Pettel, reviewed by Maciej.
ap [Sun, 12 Mar 2006 22:17:38 +0000 (22:17 +0000)]
    Fix by Mitz Pettel, reviewed by Maciej.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=5923
          REGRESSION: scrolling on Google maps continues after the mouse is
          released outside the window

        Test: fast/events/drag-outside-window.html

        * rendering/render_layer.cpp:
        (WebCore::RenderLayer::containsPoint): Always return true for the root
        object to ensure that mouse events occurring outside the window target
        the document.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13265 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago - Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=7735
ap [Sun, 12 Mar 2006 22:09:08 +0000 (22:09 +0000)]
    - Fix for bugzilla.opendarwin.org/show_bug.cgi?id=7735
        REGRESSION: crash on importing bookmarks

        No test - only reproducible when Safari imports the bookmarks.

        * dom/EventNames.cpp:
        (WebCore::EventNames::init): Initialize AtomicString before using it.
        * dom/QualifiedName.cpp:
        (WebCore::QualifiedName::init): Ditto.
        * khtml/html/htmlnames.cpp:
        (WebCore::HTMLNames::init): Ditto.
        * ksvg2/scripts/make_names.pl: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13264 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Maciej.
ap [Sun, 12 Mar 2006 20:30:58 +0000 (20:30 +0000)]
    Reviewed by Maciej.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7708
        REGRESSION: Flash callback to JavaScript function not working.

        Test: plugins/invoke.html

        * bindings/c/c_utility.cpp:
        (KJS::Bindings::convertUTF8ToUTF16): Return a correct string length.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13263 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Patch by David Kilzer, reviewed by Darin.
ap [Sun, 12 Mar 2006 08:29:20 +0000 (08:29 +0000)]
    Patch by David Kilzer, reviewed by Darin.

        - Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=7724
        Minor refactoring to HTMLParser::handleError() to remove duplicate code

        * khtml/html/htmlparser.cpp:
        (WebCore::HTMLParser::handleError): Refactored code.

        These tests cover the changed code.  No change in behavior.

        * fast/invalid/missing-address-end-tag-expected.checksum: Added.
        * fast/invalid/missing-address-end-tag-expected.png: Added.
        * fast/invalid/missing-address-end-tag-expected.txt: Added.
        * fast/invalid/missing-address-end-tag.html: Added.
        * fast/invalid/missing-dl-end-tag-expected.checksum: Added.
        * fast/invalid/missing-dl-end-tag-expected.png: Added.
        * fast/invalid/missing-dl-end-tag-expected.txt: Added.
        * fast/invalid/missing-dl-end-tag.html: Added.
        * fast/invalid/missing-dt-end-tag-expected.checksum: Added.
        * fast/invalid/missing-dt-end-tag-expected.png: Added.
        * fast/invalid/missing-dt-end-tag-expected.txt: Added.
        * fast/invalid/missing-dt-end-tag.html: Added.
        * fast/invalid/missing-font-end-tag-expected.checksum: Added.
        * fast/invalid/missing-font-end-tag-expected.png: Added.
        * fast/invalid/missing-font-end-tag-expected.txt: Added.
        * fast/invalid/missing-font-end-tag.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13262 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago - fix Windows build
darin [Sat, 11 Mar 2006 03:32:35 +0000 (03:32 +0000)]
    - fix Windows build

        * bridge/win/FrameWin.h: Removed declaration of generateFrameName.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13261 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoWebCore:
darin [Sat, 11 Mar 2006 03:30:51 +0000 (03:30 +0000)]

        Reviewed by Adele.

        - removed some unused stuff from Frame and MacFrame

        * page/Frame.h: Removed autoloadImages (the setter), enableMetaRefresh,
        setCharset, backgroundURL, findTextBegin, findTextNext, preloadStyleSheet,
        preloadScript, restored, onURL, selectionChanged, htmlError, openFile,
        updateActions, openURLInFrame, overURL, checkLinkSecurity, and cacheId.
        * page/FramePrivate.h: Removed m_restored, m_frameNameId, m_strSelectedURL,
        m_strSelectedURLTarget, m_bDnd, m_bClearing, m_bSecurityInQuestion,
        m_focusNodeRestored, m_focusNodeNumber, and m_newJSInterpreterExists.
        * page/Frame.cpp:
        (WebCore::Frame::didOpenURL): Removed use of m_restored, code to set
        (WebCore::Frame::clear): Removed code to set m_bClearing and m_frameNameId.
        (WebCore::Frame::khtmlMousePressEvent): Removed code to set m_strSelectedURL
        and m_strSelectedURLTarget.

        * bridge/mac/MacFrame.h: Removed scheduleHistoryNavigation and
        * bridge/mac/MacFrame.mm:
        (WebCore::MacFrame::openURLFromPageCache): Removed use of m_restored.
        Removed unused requestedURLString.

        * bridge/mac/WebCoreFrameBridge.h: Removed requestedURLString.


        Reviewed by Adele.

        * WebCoreSupport/WebFrameBridge.m: Remove unused requestedURLString method.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13260 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago - fix Windows build
darin [Sat, 11 Mar 2006 03:25:35 +0000 (03:25 +0000)]
    - fix Windows build

        * page/FrameTree.cpp: Add include of <stdarg.h>.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13259 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Geoff.
darin [Sat, 11 Mar 2006 03:21:52 +0000 (03:21 +0000)]
    Reviewed by Geoff.

        - change how frame namespacing works to be more completely on the WebCore side

        * WebView/WebFrameInternal.h: Remove _setFrameNamespace: and _frameNamespace.
        * WebView/WebFrame.m: Ditto.

        * WebView/WebView.m:
        (-[WebView setGroupName:]): Call -[WebCorePageBridge setGroupName:].
        (-[WebView groupName]): Call -[WebCorePageBridge groupName].

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13258 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoLayoutTests:
darin [Sat, 11 Mar 2006 03:17:11 +0000 (03:17 +0000)]

        Reviewed by Geoff, except for frame-name-reset, which was done by Alexey.

        - test for http://bugzilla.opendarwin.org/show_bug.cgi?id=3308
          Pop-up blocking blocks window.open for already open windows

        * fast/dom/Window/open-existing-pop-up-blocking-expected.checksum: Added.
        * fast/dom/Window/open-existing-pop-up-blocking-expected.png: Added.
        * fast/dom/Window/open-existing-pop-up-blocking-expected.txt: Added.
        * fast/dom/Window/open-existing-pop-up-blocking.html: Added.
        * fast/dom/Window/resources/open-sibling-subframe.html: Added.
        * fast/dom/Window/resources/sibling-subframe-content.html: Added.

        - test for http://bugzilla.opendarwin.org/show_bug.cgi?id=7422
          Setting a frame name to the same value resets it to a generated one

        * fast/frames/frame-name-reset.html: Added.
        * fast/frames/frame-name-reset-expected.txt: Added.


        Reviewed by Geoff.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=3308
          Pop-up blocking blocks window.open for already open windows

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7422
          Setting a frame name to the same value resets it to a generated one

        - refactor frame-name-related functions into the FrameTree object

        * WebCore.xcodeproj/project.pbxproj: Update for rename.
        * bridge/mac/MacFrame.h: Remove generateFrameName.
        * bridge/mac/MacFrame.mm: Ditto.
        * bridge/mac/PageMac.h: Added a declaration for WebCorePageBridge.

        * bridge/mac/WebCoreFrameBridge.h: Remove _frameNamespace, generateFrameName,
        setFrameNamespace, frameNamespace.
        * bridge/mac/WebCoreFrameBridge.mm:
        (-[WebCoreFrameBridge childFrameNamed:]): Change to call the new child
        function on the FrameTree.
        (-[WebCoreFrameBridge findFrameNamed:]): Change to call the new find
        function on the FrameTree.
        (-[WebCoreFrameBridge RenderObject::nodeInfoAtPoint:]): Change since the
        contentPart function has been renamed to contentFrame.

        * bridge/mac/WebCoreFrameNamespaces.h: Removed everything except for the
        one method still used on the WebKit side, framesInNamespace:.
        * bridge/mac/WebCoreFrameNamespaces.m: Renamed.
        * bridge/mac/WebCoreFrameNamespaces.mm: Added. Reimplemented the
        framesInNamespace method to use the namespace in WebCore::Page.

        * bridge/mac/WebCorePageBridge.h: Added setGroupName and groupName.
        * bridge/mac/WebCorePageBridge.mm:
        (-[WebCorePageBridge setGroupName:]): Added. Calls through to Page.
        (-[WebCorePageBridge groupName]): Ditto.

        * khtml/ecma/kjs_html.cpp:
        (KJS::HTMLDocument::namedItemGetter): Changed to call contentFrame by its new name.
        (KJS::HTMLElement::frameGetter): Ditto.
        (KJS::HTMLElement::iFrameGetter): Ditto.

        * khtml/ecma/kjs_window.cpp:
        (KJS::Window::getValueProperty): Changed to call FrameTree::childCount
        instead of Frame::frames.
        (KJS::Window::childFrameGetter): Changed to call FrameTree::child
        instead of Frame::childFrameNamed.
        (KJS::Window::namedFrameGetter): Changed to call FrameTree::find
        instead of Frame::findFrame.
        (KJS::Window::indexGetter): Changed to call FrameTree::child
        instead of Frame::frames.
        (KJS::Window::getOwnPropertySlot): Changed to call FrameTree::child,
        FrameTree::find, and FrameTree::childCount instead of Frame::childFrameNamed,
        and Frame::findFrame, and Frame::frames.
        (KJS::WindowFunc::callAsFunction): Call FrameTree::find to check if the window
        is already open when considering whether to block a pop-up.
        (KJS::FrameArray::getValueProperty): Changed to call FrameTree::childCount
        instead of Frame::frames.
        (KJS::FrameArray::indexGetter): Changed to call FrameTree::child
        instead of Frame::frames.
        (KJS::FrameArray::nameGetter): Changed to call FrameTree::child
        instead of Frame::findFrame.
        (KJS::FrameArray::getOwnPropertySlot): Changed to call FrameTree::child,
        and FrameTree::childCount instead of Frame::findFrame and Frame::frames.

        * khtml/html/html_baseimpl.h:
        * khtml/html/html_baseimpl.cpp:
        (WebCore::HTMLFrameElementImpl::isURLAllowed): Changed to call Page::frameCount
        instead of Frame::topLevelFrameCount.
        (WebCore::HTMLFrameElementImpl::openURL): Changed to call FrameTree::child
        instead of Frame::findFrame.
        (WebCore::HTMLFrameElementImpl::attach): Changed to call Page::incrementFrameCount
        instead of Frame::incrementFrameCount. Changed to call FrameTree::uniqueChildName
        instead of Frame::requestFrameName.
        (WebCore::HTMLFrameElementImpl::close): Changed to call Page::decrementFrameCount
        instead of Frame::decrementFrameCount. Changed to call FrameTree::child
        instead of Frame::findFrame.
        (WebCore::HTMLFrameElementImpl::contentFrame): Renamed from contentPart. Also
        changed to call FrameTree::child instead of Frame::findFrame.
        (WebCore::HTMLFrameElementImpl::contentDocument): Updated for name change of
        contentFrame from contentPart.
        (WebCore::HTMLIFrameElementImpl::attach): Changed to call Page::incrementFrameCount
        instead of Frame::incrementFrameCount. Changed to call FrameTree::uniqueChildName
        instead of Frame::requestFrameName.

        * rendering/render_frames.cpp: (WebCore::isURLAllowed): Changed to call Page::frameCount
        instead of Frame::topLevelFrameCount.

        * page/Frame.h: Removed frameNames, frames, childFrameNamed, findFrame, currentFrame,
        frameExists, incrementFrameCount, decrementFrameCount, topLevelFrameCount,
        generateFrameName, and requestFrameName functions.
        * page/Frame.cpp: (WebCore::Frame::requestFrame): Changed to use FrameTree::child
        instead of Frame::childFrameNamed.

        * page/FrameTree.h: Changed name to an atomic string. Changed childCount to unsigned.
        Added isDescendantOf, traverseNext, child, find, uniqueChildName.
        * page/FrameTree.cpp:
        (WebCore::FrameTree::setName): Changed to call uniqueChildName to handle name
        duplication logic.
        (WebCore::FrameTree::uniqueChildName): Added. Checks for duplication and generates
        an appropriate frame name if there is a duplicate.
        (WebCore::FrameTree::child): Added.
        (WebCore::FrameTree::find): Added.
        (WebCore::FrameTree::isDescendantOf): Added.
        (WebCore::FrameTree::traverseNext): Added.

        * page/Page.h: Added setGroupName, groupName, frameNamespace, incrementFrameCount,
        decrementFrameCount, and frameCount.
        * page/Page.cpp:
        (WebCore::Page::Page): Initialize m_frameCount to 0.
        (WebCore::Page::~Page): Call setGroupName to remove the page from any group it's in.
        (WebCore::Page::setGroupName): Added.
        (WebCore::Page::frameNamespace): Added.

        * platform/win/TemporaryLinkStubs.cpp: Removed FrameWin::generateFrameName.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13257 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by darin, harrison
justing [Sat, 11 Mar 2006 01:35:55 +0000 (01:35 +0000)]
    Reviewed by darin, harrison

        ~3x speedup pasting plain text from the console

        * dom/dom_position.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13256 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoFont refactoring on win32.
hyatt [Sat, 11 Mar 2006 00:19:56 +0000 (00:19 +0000)]
Font refactoring on win32.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13255 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Eric.
mjs [Fri, 10 Mar 2006 23:20:21 +0000 (23:20 +0000)]
    Reviewed by Eric.

        - fix win32 networking to properly use async load API

        * platform/win/TransferJobWin.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13252 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-10 Eric Seidel <eseidel@apple.com>
eseidel [Fri, 10 Mar 2006 21:15:21 +0000 (21:15 +0000)]
2006-03-10  Eric Seidel  <eseidel@apple.com>

        Fix build.

        * platform/win/FontWin.cpp:
        * platform/win/TemporaryLinkStubs.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13251 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoLayoutTests:
adele [Fri, 10 Mar 2006 21:13:35 +0000 (21:13 +0000)]

        - Test for <rdar://problem/4469419>
        REGRESSION: RadioButtons are not checked in Safari when name attribute comes after checked attribute

        * fast/forms/radio-attr-order-expected.checksum: Added.
        * fast/forms/radio-attr-order-expected.png: Added.
        * fast/forms/radio-attr-order-expected.txt: Added.
        * fast/forms/radio-attr-order.html: Added.


        Reviewed by Tim Hatcher.

        - fix for <rdar://problem/4469419>
        REGRESSION: RadioButtons are not checked in Safari when name attribute comes after checked attribute

        Test: fast/forms/radio-attr-order.html

        * khtml/html/HTMLInputElementImpl.cpp: (WebCore::HTMLInputElementImpl::parseMappedAttribute):
        When the name attribute gets parsed for radio buttons, if its supposed to be checked by default, call setChecked.
        Also, when the name changes on a checked button, we don't need to check that
        there's a form before removing it as the checked button for that group.
        That was an old check from when we required a form to keep track of checked buttons.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13250 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-10 Eric Seidel <eseidel@apple.com>
eseidel [Fri, 10 Mar 2006 21:05:05 +0000 (21:05 +0000)]
2006-03-10  Eric Seidel  <eseidel@apple.com>

        Reviewed by darin.

        Add real ScrollViewWin implementation (fixes redraw bugs, adds ScrollBar support.)

        * platform/ScrollView.h:
        * platform/Widget.h:
        * platform/win/FontWin.cpp:
        (WebCore::getCairoFont): Fix crasher.
        * platform/win/ScrollViewWin.cpp:
        * platform/win/TemporaryLinkStubs.cpp:
        * platform/win/WidgetWin.cpp:
        (WebCore::Widget::windowHandle): mark as const

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13249 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Rename the m_renderer variable to m_dataSet now that the class name
hyatt [Fri, 10 Mar 2006 19:28:14 +0000 (19:28 +0000)]
    Rename the m_renderer variable to m_dataSet now that the class name
        has changed.

        Reviewed by adele

        * platform/Font.cpp:
        * platform/Font.h:
        * platform/FontDataSet.h:
        * platform/mac/FontMac.mm:
        * platform/win/FontWin.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13248 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Rename FontRenderer to FontDataSet.
hyatt [Fri, 10 Mar 2006 18:58:31 +0000 (18:58 +0000)]
    Rename FontRenderer to FontDataSet.

        Reviewed by adele

        * WebCore.xcodeproj/project.pbxproj:
        * platform/Font.cpp:
        * platform/Font.h:
        * platform/FontDataSet.h: Added.
        * platform/FontRenderer.h: Removed.
        * platform/mac/FontMac.mm:
        * platform/win/FontWin.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13247 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoFIx a couple of font crashers.
hyatt [Fri, 10 Mar 2006 18:40:58 +0000 (18:40 +0000)]
FIx a couple of font crashers.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13246 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by adele
justing [Fri, 10 Mar 2006 07:37:36 +0000 (07:37 +0000)]
    Reviewed by adele

        Should use an iterator in CompositeEditCommand::doUnapply()

        ~30% speedup of a big undo

        * editing/CompositeEditCommand.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13245 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Tim O.
adele [Fri, 10 Mar 2006 02:20:26 +0000 (02:20 +0000)]
    Reviewed by Tim O.

        Removed unnecessary alerts in TextIterator advance methods.
        We want to be able to use advance() even if the iterator is already at the end.

        * editing/visible_text.cpp:
        (khtml::TextIterator::advance): remove assert since TextIterator will just do nothing if its already at the end.
        (khtml::CharacterIterator::advance): ditto.
        * rendering/RenderTextField.cpp: (WebCore::RenderTextField::visiblePositionForIndex):
        Since the asserts in the iterator are removed, we don't have to check for it.atEnd() before calling advance.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13244 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoMake text paint with the pen color on Win32. Force ClearType quality. Make the...
hyatt [Thu, 9 Mar 2006 23:39:06 +0000 (23:39 +0000)]
Make text paint with the pen color on Win32.  Force ClearType quality.  Make the face validation case-insensitive.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13241 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoLayoutTests:
adele [Thu, 9 Mar 2006 23:30:15 +0000 (23:30 +0000)]

        - Test for http://bugzilla.opendarwin.org/show_bug.cgi?id=7675
        When new text fields change from visibility:hidden to visibility:visible, value doesn't display

        * fast/forms/input-appearance-visibility-expected.checksum: Added.
        * fast/forms/input-appearance-visibility-expected.png: Added.
        * fast/forms/input-appearance-visibility-expected.txt: Added.
        * fast/forms/input-appearance-visibility.html: Added.


        Reviewed by Hyatt.

        - Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=7675
        When new text fields change from visibility:hidden to visibility:visible, value doesn't display

        Test: fast/forms/input-appearance-visibility.html

        * rendering/RenderTextField.cpp: (WebCore::RenderTextField::setStyle):
        Sets the style directly on the children of the inner div.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13240 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoSetBkMode to transparent so that TextOut doesn't erase the background when drawing...
hyatt [Thu, 9 Mar 2006 22:48:30 +0000 (22:48 +0000)]
SetBkMode to transparent so that TextOut doesn't erase the background when drawing text.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13237 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoCheck in basic text rendering on Win32.
hyatt [Thu, 9 Mar 2006 21:24:54 +0000 (21:24 +0000)]
Check in basic text rendering on Win32.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13236 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by harrison
justing [Thu, 9 Mar 2006 20:02:21 +0000 (20:02 +0000)]
    Reviewed by harrison

        Fixed formatting and changed functions to take in
        a const Selection& instead of a Selection.

        * editing/ModifySelectionListLevelCommand.cpp:
        * editing/ModifySelectionListLevelCommand.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13235 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoLayoutTests:
adele [Thu, 9 Mar 2006 19:43:30 +0000 (19:43 +0000)]

        - Tests for http://bugzilla.opendarwin.org/show_bug.cgi?id=6814
        Implement selection methods for RenderTextField

        * fast/forms/input-appearance-selection-expected.checksum: Added.
        * fast/forms/input-appearance-selection-expected.png: Added.
        * fast/forms/input-appearance-selection-expected.txt: Added.
        * fast/forms/input-appearance-selection.html: Added.


        Reviewed by Darin.

        - Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=6814
        Implement selection methods for RenderTextField

        Test: fast/forms/input-appearance-selection.html

        * rendering/RenderTextField.cpp:
        (WebCore::RenderTextField::createDivStyle): removed extra space.
        (WebCore::RenderTextField::select): Removed checks for document and m_div since there should always be a document and
         m_div will be initialized as the renderer gets attached.
        (WebCore::RenderTextField::selectionStart): Calls indexForVisiblePosition.
        (WebCore::RenderTextField::selectionEnd): ditto.
        (WebCore::RenderTextField::setSelectionStart): calls setSelectionRange.
        (WebCore::RenderTextField::setSelectionEnd): ditto.
        (WebCore::RenderTextField::setSelectionRange): gets VisiblePositions for both indices, and sets the selection.
        (WebCore::RenderTextField::visiblePositionForIndex): Added helper method.
        (WebCore::RenderTextField::indexForVisiblePosition): Added helper method.
        * rendering/RenderTextField.h: Added new helper methods for selection.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13234 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by John Sullivan.
darin [Thu, 9 Mar 2006 17:19:48 +0000 (17:19 +0000)]
    Reviewed by John Sullivan.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7681
          memory leak in the plug-in tests

        * DumpRenderTree/TestNetscapePlugIn.subproj/main.c:
        (NPP_Destroy): Added code to release the plug-in object. This is the leak fix.
        (NPP_SetWindow): Remove unneeded code to store the window pointer.

        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.c:
        Moved the browser global in here since it's declared in this file's header.
        Changed the code to set up the pluginClass structure to not use function
        pointer casts. Those are dangerous because they can hide many types of mismatch.
        And indeed when I did this I discovered that many functions were missing their
        boolean return values or had parameter declarations with the wrong types.
        (pluginGetProperty): Use STRINGZ_TO_NPVARIANT macro for greater simplicity and
        clarity. Added boolean return value: return true when successful and false when not.
        (pluginSetProperty): Added boolean return value, return false since we have no
        properties we can set.
        (pluginInvoke): Added boolean return value. Return true when successful and false
        when not. Use NPVARIANT macros where appropriate. Added a missing release for the
        return value from calling the browser. Changed code to put the strings in malloc
        buffers instead of relying on GCC's extension that allows variable-sized arrays
        on the stack.
        (pluginInvokeDefault): Added boolean return value, return false since we have no
        default function to call.
        (pluginInvalidate): Added missing parameter. Removed comment.
        (pluginAllocate): Removed unneeded cast. This is C code, not C++, so you don't have
        to cast the result of malloc.
        (pluginDeallocate): Removed uneeded cast.

        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h: Removed some unneeded
        includes. Changed our PluginObject to use NPObject instead of re-declaring fields
        that match NPObject's fields. Removed unused NPWindow pointer.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13233 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoLayoutTests:
darin [Thu, 9 Mar 2006 16:35:57 +0000 (16:35 +0000)]

        Reviewed by Darin.

        - test for http://bugzilla.opendarwin.org/show_bug.cgi?id=7583
          DRT hangs when doing eventSender.mouseDown on native widgets

        * fast/events/event-sender-mouse-click-expected.txt: Added.
        * fast/events/event-sender-mouse-click.html: Added.


        Test: fast/events/event-sender-mouse-click.html

        Reviewed by Darin.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7583
          DRT hangs when doing eventSender.mouseDown on native widgets

        * DumpRenderTree/DumpRenderTree.m:
        (+[EventSendingController isSelectorExcludedFromWebScript:]):
        (-[EventSendingController mouseClick]): Simulates a click in a native
        widget by queueing a mouseUp before sending the mouseDown, so that
        the widget's mouse tracking event loop doesn't hang indefinitely.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13232 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoLayoutTests:
darin [Thu, 9 Mar 2006 16:32:31 +0000 (16:32 +0000)]

        Reviewed by Darin.

        * fast/canvas: Added.

        - test for http://bugzilla.opendarwin.org/show_bug.cgi?id=6103
          Rectangles are not filled according to the Web Applications Working Draft

        * fast/canvas/fillrect_gradient-expected.checksum: Added.
        * fast/canvas/fillrect_gradient-expected.png: Added.
        * fast/canvas/fillrect_gradient-expected.txt: Added.
        * fast/canvas/fillrect_gradient.html: Added.

        - moved other canvas tests here

        * fast/canvas/image-object-in-canvas-expected.checksum: Added.
        * fast/canvas/image-object-in-canvas-expected.png: Added.
        * fast/canvas/image-object-in-canvas-expected.txt: Added.
        * fast/canvas/image-object-in-canvas.html: Added.
        * fast/canvas/quadraticCurveTo-expected.checksum: Added.
        * fast/canvas/quadraticCurveTo-expected.png: Added.
        * fast/canvas/quadraticCurveTo-expected.txt: Added.
        * fast/canvas/quadraticCurveTo.xml: Added.
        * fast/canvas/resources: Added.
        * fast/canvas/resources/apple.gif: Added.

        * fast/dom/image-object-in-canvas-expected.checksum: Removed.
        * fast/dom/image-object-in-canvas-expected.png: Removed.
        * fast/dom/image-object-in-canvas-expected.txt: Removed.
        * fast/dom/image-object-in-canvas.html: Removed.
        * fast/dom/quadraticCurveTo-expected.checksum: Removed.
        * fast/dom/quadraticCurveTo-expected.png: Removed.
        * fast/dom/quadraticCurveTo-expected.txt: Removed.
        * fast/dom/quadraticCurveTo.xml: Removed.


        Test: fast/canvas/fillrect_gradient.html

        Reviewed by Darin.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=6103
          Rectangles are not filled according to the Web Applications Working Draft

        * khtml/ecma/kjs_html.cpp: (KJS::Context2DFunction::callAsFunction):
        Modified fillrect function to check and use a gradient if set.
        Builds a path to fill rects with a gradient.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13231 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago No test because this fixes an existing pixel test (and does not affect
darin [Thu, 9 Mar 2006 16:10:09 +0000 (16:10 +0000)]
    No test because this fixes an existing pixel test (and does not affect
        the render tree).

        Reviewed by Maciej.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7664
          REGRESSION: tests that include numeric list markers fail in pixel mode

        * platform/GraphicsContext.cpp:
        (WebCore::GraphicsContext::drawText): Restored AlignRight behavior.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13230 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Darin.
darin [Thu, 9 Mar 2006 16:05:03 +0000 (16:05 +0000)]
    Reviewed by Darin.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7680
          REGRESSION: Ellipsis painted over the chrome and in invalid contexts

        * rendering/render_line.cpp:
        (WebCore::EllipsisBox::paint): Replaced call to Font::drawText with
        GraphicsContext::drawText, which checks the paintingDisabled flag.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13229 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoWebCore:
mjs [Thu, 9 Mar 2006 10:57:48 +0000 (10:57 +0000)]

        Reviewed by Eric.

        - make link clicks work by propagating them up to a client

        * Viewer/ImageView.cpp:
        * bridge/win/FrameWin.cpp:
        * bridge/win/FrameWin.h:
        * platform/win/TemporaryLinkStubs.cpp:


        Reviewed by Eric.

        - make link clicks work by handling link click requests
        from WebCore

        * Spinneret/Spinneret/Spinneret.cpp:
        * Spinneret/Spinneret/Spinneret.h:
        * Spinneret/Spinneret/WebFrame.cpp:
        * Spinneret/Spinneret/WebFrame.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13228 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-08 Eric Seidel <eseidel@apple.com>
eseidel [Thu, 9 Mar 2006 09:52:26 +0000 (09:52 +0000)]
2006-03-08  Eric Seidel  <eseidel@apple.com>

        Reviewed by mjs.

        Focus URL bar on Spinneret launch, remove border from WebView.
        Stop WebFrame from deleting the job (prevent crash).

        * Spinneret/Spinneret/Spinneret.cpp:
        * Spinneret/Spinneret/WebFrame.cpp:
        * Spinneret/Spinneret/WebView.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13227 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-09 Eric Seidel <eseidel@apple.com>
eseidel [Thu, 9 Mar 2006 09:49:54 +0000 (09:49 +0000)]
2006-03-09  Eric Seidel  <eseidel@apple.com>

        Reviewed by mjs.

        Add support for Timers under 10ms.
        Fix random crashers due to multi-threaded loading.

        * platform/TransferJob.h:
        * platform/TransferJobInternal.h:
        * platform/Widget.h:
        * platform/win/SharedTimerWin.cpp:
        * platform/win/TemporaryLinkStubs.cpp:
        * platform/win/TransferJobWin.cpp:
        * platform/win/WidgetWin.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13226 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Darin.
ap [Thu, 9 Mar 2006 05:49:42 +0000 (05:49 +0000)]
    Reviewed by Darin.

        - http://bugzilla.opendarwin.org/show_bug.cgi?id=7656
         Query string always appended to Flash URLs, instead of being replaced.

        * Misc/WebNSURLExtras.h: Added _webkit_URLByRemovingResourceSpecifier.
        * Misc/WebNSURLExtras.m:
        (+[NSURL _web_URLWithData:relativeToURL:]): Call _webkit_URLByRemovingResourceSpecifier to
        work around CFURL not removing non-path components from base URLs in some cases.
        (-[NSURL _webkit_URLByRemovingComponent:]): New generic function for removing URL components.
        (-[NSURL _webkit_URLByRemovingFragment]): Moved implementation to the above method.
        (-[NSURL _webkit_URLByRemovingResourceSpecifier]): Added.

        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.c:
        (pluginInvoke): Added a method to test getURL.

        * plugins/geturl-replace-query-expected.txt: Added.
        * plugins/geturl-replace-query.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13225 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoAdded log entry that somehow didn't get checked in last night.
darin [Thu, 9 Mar 2006 02:28:24 +0000 (02:28 +0000)]
Added log entry that somehow didn't get checked in last night.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13224 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Beth.
ggaren [Thu, 9 Mar 2006 00:20:36 +0000 (00:20 +0000)]
    Reviewed by Beth.

        Clarified that sites on the list aren't necessarily "broken."

        * projects/compat/hitlist.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13223 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agomerge cairo to the mozilla trunk.
hyatt [Wed, 8 Mar 2006 23:53:07 +0000 (23:53 +0000)]
merge cairo to the mozilla trunk.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13222 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoLayoutTests:
justing [Wed, 8 Mar 2006 23:07:07 +0000 (23:07 +0000)]

        Reviewed by darin

        TinyMCE: Implement execCommand(insertImage, ...)

        * editing/editing.js:
        * editing/execCommand/insertImage-expected.checksum: Added.
        * editing/execCommand/insertImage-expected.png: Added.
        * editing/execCommand/insertImage-expected.txt: Added.
        * editing/execCommand/insertImage.html: Added.
        * editing/pasteboard/3976872-expected.txt:


        Reviewed by darin

        TinyMCE: Implement execCommand(insertImage, ...)

        Implemented InsertImage and also changed paste to use a single
        rule for when to remove junk that remains after node removal.

        * editing/ReplaceSelectionCommand.cpp:
        * editing/ReplaceSelectionCommand.h:
        * editing/htmlediting.cpp:
        * editing/jsediting.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13221 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Beth.
ggaren [Wed, 8 Mar 2006 22:51:09 +0000 (22:51 +0000)]
    Reviewed by Beth.

        Updated links to gap and bananarepublic bugs.

        * projects/compat/hitlist.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13220 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-08 Eric Seidel <eseidel@apple.com>
eseidel [Wed, 8 Mar 2006 22:20:55 +0000 (22:20 +0000)]
2006-03-08  Eric Seidel  <eseidel@apple.com>

        Reviewed by hyatt.

        Fix build script to properly copy dll files.

        * WebCore.vcproj/WebCore/build-generated-files.sh:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13216 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-08 Rob Buis <buis@kde.org>
eseidel [Wed, 8 Mar 2006 21:32:29 +0000 (21:32 +0000)]
2006-03-08  Rob Buis  <buis@kde.org>

        Reviewed by eseidel.

        Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=6899
        SVG <rect> does not respect display: none

        For svg elements that can render, call StyledElementImpl::rendererIsNeeded
        to decide whether the element should render. In particular this way
        any setting of display="none" is handled.

        * ksvg2/svg/SVGAElementImpl.h:
        * ksvg2/svg/SVGCircleElementImpl.h:
        * ksvg2/svg/SVGEllipseElementImpl.h:
        * ksvg2/svg/SVGForeignObjectElementImpl.h:
        * ksvg2/svg/SVGGElementImpl.h:
        * ksvg2/svg/SVGImageElementImpl.h:
        * ksvg2/svg/SVGLineElementImpl.h:
        * ksvg2/svg/SVGMarkerElementImpl.h:
        * ksvg2/svg/SVGMaskElementImpl.h:
        * ksvg2/svg/SVGPathElementImpl.h:
        * ksvg2/svg/SVGPatternElementImpl.h:
        * ksvg2/svg/SVGPolyElementImpl.h:
        * ksvg2/svg/SVGRectElementImpl.h:
        * ksvg2/svg/SVGSVGElementImpl.h:
        * ksvg2/svg/SVGSwitchElementImpl.h:
        * ksvg2/svg/SVGTSpanElementImpl.h:
        * ksvg2/svg/SVGTextElementImpl.h:
        * ksvg2/svg/SVGUseElementImpl.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13215 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Eric.
mjs [Wed, 8 Mar 2006 11:59:43 +0000 (11:59 +0000)]
    Reviewed by Eric.

        - load URLs, not file paths, in Spinneret

        * Spinneret/Spinneret/Spinneret.cpp:
        * Spinneret/Spinneret/Spinneret.vcproj:
        * Spinneret/Spinneret/WebFrame.cpp:
        * Spinneret/Spinneret/WebFrame.h:
        * Spinneret/Spinneret/WebView.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13214 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by mjs.
mjs [Wed, 8 Mar 2006 11:56:08 +0000 (11:56 +0000)]
    Reviewed by mjs.

        Add stubs to allow JavaScript-enabled WebCore to not crash.
        Add ScreenWin implementation.
        Add alert() support.

        * WebCore.vcproj/WebCore/WebCore.vcproj:
        * bridge/win/FrameWin.cpp:
        * platform/win/ScreenWin.cpp: Added.
        * platform/win/TemporaryLinkStubs.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13213 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-08 Eric Seidel <eseidel@apple.com>
eseidel [Wed, 8 Mar 2006 11:40:39 +0000 (11:40 +0000)]
2006-03-08  Eric Seidel  <eseidel@apple.com>

        Reviewed by mjs.

        Add stubs to allow JavaScript-enabled WebCore to not crash.
        Add ScreenWin implementation.
        Add alert() support.

        * WebCore.vcproj/WebCore/WebCore.vcproj:
        * bridge/win/FrameWin.cpp:
        * platform/win/ScreenWin.cpp: Added.
        * platform/win/TemporaryLinkStubs.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13212 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-08 Eric Seidel <eseidel@apple.com>
eseidel [Wed, 8 Mar 2006 10:59:34 +0000 (10:59 +0000)]
2006-03-08  Eric Seidel  <eseidel@apple.com>

        Reviewed by darin.

        Partially fix JS on win32 by fixing hash table generation.

        * kjs/create_hash_table: limit << results to 32 bits.
        * kjs/testkjs.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13211 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Maciej.
darin [Wed, 8 Mar 2006 09:55:30 +0000 (09:55 +0000)]
    Reviewed by Maciej.

        - fix another instance of the bug I just fixed in the Macintosh-specific code path

        * platform/image-decoders/gif/GIFImageReader.h: Remove not-so-useful 100ms constant.
        * platform/image-decoders/gif/GIFImageReader.cpp: (GIFImageReader::read):
        Use the correct rule for minimum frame durations.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13210 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Darin.
mjs [Wed, 8 Mar 2006 09:43:04 +0000 (09:43 +0000)]
    Reviewed by Darin.

        - fix win32 build

        * platform/win/FontWin.cpp:
        * platform/win/TemporaryLinkStubs.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13209 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Maciej.
darin [Wed, 8 Mar 2006 09:40:47 +0000 (09:40 +0000)]
    Reviewed by Maciej.

        - fix one more thing from http://bugzilla.opendarwin.org/show_bug.cgi?id=7528
          REGRESSION: GIF animation speeds are wrong on a page with many animated GIFs

        No layout test because I don't know how to test animated GIF speed.

        * platform/mac/ImageSourceMac.cpp: (WebCore::ImageSource::frameDurationAtIndex):
        Use the correct rule for minimum frame durations. It's a bit non-intuitive, so
        I made the comment more explicit. The old code in WebKit had this rule.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13208 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Eric.
darin [Wed, 8 Mar 2006 09:26:02 +0000 (09:26 +0000)]
    Reviewed by Eric.

        - fixed AppleScript layout test results to not be endian-dependent
          (Hyatt complained to me about this one)

        * DumpRenderTree/AppleScriptController.m: (-[AppleScriptController doJavaScript:]):
        Added specific code for dumping LongDateTime, instead of dumping the raw bytes
        (which are endian-dependent).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13207 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Make GraphicsContext call into Font to draw all text and eliminate
hyatt [Wed, 8 Mar 2006 09:12:00 +0000 (09:12 +0000)]
    Make GraphicsContext call into Font to draw all text and eliminate
        all knowledge of WebTextRenderers from it.

        Reviewed by darin

        * platform/Font.h:
        * platform/GraphicsContext.cpp:
        * platform/GraphicsContext.h:
        * platform/cairo/GraphicsContextCairo.cpp:
        * platform/mac/FontMac.mm:
        * platform/mac/GraphicsContextMac.mm:
        * platform/win/FontWin.cpp:
        * rendering/InlineTextBox.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13206 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago * kxmlcore/Vector.h: Quick fix to try to get Windows compiling again.
darin [Wed, 8 Mar 2006 09:08:43 +0000 (09:08 +0000)]
    * kxmlcore/Vector.h: Quick fix to try to get Windows compiling again.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13205 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Hyatt.
darin [Wed, 8 Mar 2006 08:59:45 +0000 (08:59 +0000)]
    Reviewed by Hyatt.

        - fixed super-fast animation of many animated GIFs

        No layout test because I don't know how to test animated GIF speed.

        * platform/mac/ImageSourceMac.cpp: (WebCore::ImageSource::frameDurationAtIndex):
        Always take the minimum duration into account, even if there's some kind of failure
        getting the duration from the GIF.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13204 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoJavaScriptCore:
darin [Wed, 8 Mar 2006 08:30:57 +0000 (08:30 +0000)]

        Reviewed by Anders.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7655
          unwanted output while running layout tests

        * kjs/lexer.cpp: (Lexer::lex): Turn off the "yylex: ERROR" message.
        * kjs/regexp.cpp: (KJS::RegExp::RegExp): Remove the code to log errors from PCRE
        to standard output. I think we should arrange for the error text to be in JavaScript
        exceptions instead at some point.
        * kxmlcore/Vector.h: Add a check for overflow so that we'll abort if we pass a
        too-large size rather than allocating a buffer smaller than requested.


        Reviewed by Anders.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7655
          unwanted output while running layout tests

        * khtml/xsl/xsl_stylesheetimpl.cpp: (WebCore::XSLStyleSheetImpl::parseString):
        Pass XML_PARSE_NOERROR and XML_PARSE_NOWARNING. We don't want errors and warnings
        to be logged to stdout or stderr. If we later decide we want the error messages,
        then we should do the additional work to put them into the web page or the
        console (along with the JavaScript errors).

        * platform/ArrayImpl.cpp: (WebCore::ArrayImpl::resize): Add a preflight to protect
        against integer overflow due to large array size. Noticed this while looking into
        the malloc error message.


        Reviewed by Anders.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7655
          unwanted output while running layout tests

        * WebView/WebDataSourcePrivate.h:
        * WebView/WebDataSource.m:
        (-[WebDataSource _setRepresentation:]): Clear the flag that records whether we've sent
        all the data to the representation or not; need this to prevent telling the same representation
        both that we've succeeded and then later that we've failed.
        (-[WebDataSource _setMainDocumentError:]): Don't send an error if representationFinishedLoading
        is already YES. Set representationFinishedLoading.
        (-[WebDataSource _finishedLoading]): Set representationFinishedLoading.
        (-[WebDataSource _setupForReplaceByMIMEType:]): Ditto.


        Reviewed by Anders.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7655
          unwanted output while running layout tests

        * DumpRenderTree/DumpRenderTree.m:
        (checkedMalloc): Added.
        (checkedRealloc): Added.
        (makeLargeMallocFailSilently): Added.
        (main): Call makeLargeMallocFailSilently.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13203 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by harrison
justing [Wed, 8 Mar 2006 01:37:44 +0000 (01:37 +0000)]
    Reviewed by harrison

        I think this layout test is failing b/c the wait times are
        right on the threshold of what is necessary to register a drag.

        * fast/lists/drag-into-marker.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13202 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago Reviewed by Adele.
darin [Wed, 8 Mar 2006 00:08:19 +0000 (00:08 +0000)]
    Reviewed by Adele.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7647
          Crash when typing into text contols

        I missed a couple places that were still saying KWQWidgetHolder
        in the change I landed late last night.

        * kwq/KWQTextArea.h:
        * kwq/KWQTextArea.mm:
        * kwq/KWQTextField.h:
        Changed them all to say WebCoreWidgetHolder.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13201 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoAdd a prerequisite named 'force' to the debug and release rules to prevent the "debug...
thatcher [Tue, 7 Mar 2006 23:35:12 +0000 (23:35 +0000)]
Add a prerequisite named 'force' to the debug and release rules to prevent the "debug is up-to-date" error. Reviewed by Eric.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13200 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-07 Eric Seidel <eseidel@apple.com>
eseidel [Tue, 7 Mar 2006 22:26:55 +0000 (22:26 +0000)]
2006-03-07  Eric Seidel  <eseidel@apple.com>

        Reviewed by hyatt.

        Fix cursor support on win32.

        * platform/win/WidgetWin.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13199 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoEliminate two more unneeded arguments from drawText
hyatt [Tue, 7 Mar 2006 22:03:03 +0000 (22:03 +0000)]
Eliminate two more unneeded arguments from drawText

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13198 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years ago2006-03-07 Eric Seidel <eseidel@apple.com>
eseidel [Tue, 7 Mar 2006 21:49:45 +0000 (21:49 +0000)]
2006-03-07  Eric Seidel  <eseidel@apple.com>

        Reviewed by hyatt.

        Fix basic text support.

        * WebCore.vcproj/WebCore/WebCore.vcproj:
        * platform/cairo/GraphicsContextCairo.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13197 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 years agoRemove unused parameters from drawText.
hyatt [Tue, 7 Mar 2006 21:43:31 +0000 (21:43 +0000)]
Remove unused parameters from drawText.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13196 268f45cc-cd09-0410-ab3c-d52691b4dbfc