LayoutTests:
[WebKit-https.git] / WebCore / ChangeLog
index 3d73932..72b9d44 100644 (file)
+2007-01-30  Graham Dennis  <graham.dennis@gmail.com>
+
+        Reviewed by Maciej.
+
+        Part of fix for http://bugs.webkit.org/show_bug.cgi?id=10725
+        Image data in from RTFD clipboard data thrown away
+        
+        ResourceHandles no longer need to be able to defer loading as that is
+        now handled by the ResourceLoader.
+
+        * loader/MainResourceLoader.cpp:
+        (WebCore::MainResourceLoader::loadNow): updated to remove defersLoading argument.
+        * loader/ResourceLoader.cpp:
+        (WebCore::ResourceLoader::releaseResources): release the deferred request.
+        (WebCore::ResourceLoader::load): defer creating the ResourceHandle instead of
+        asking the ResourceHandle to defer loading.
+        (WebCore::ResourceLoader::setDefersLoading): create the ResourceHandle if we have deferred creating it.
+        * loader/ResourceLoader.h:
+        * platform/network/ResourceHandle.cpp: ResourceHandle's don't need to know about deferLoading.
+        (WebCore::ResourceHandle::ResourceHandle):
+        (WebCore::ResourceHandle::create):
+        * platform/network/ResourceHandle.h: ditto.
+        * platform/network/ResourceHandleInternal.h: ditto.
+        (WebCore::ResourceHandleInternal::ResourceHandleInternal):
+        * platform/network/mac/ResourceHandleMac.mm: ditto.
+        (WebCore::ResourceHandle::start):
+
+2007-01-29  Brady Eidson  <beidson@apple.com>
+
+        Reviewed by Alice
+
+        Lets use smart pointers, shall we?
+
+        * platform/network/ResourceHandleInternal.h:
+
+2007-01-29  Justin Garcia  <justin.garcia@apple.com>
+
+        Reviewed by harrison
+        
+        <rdar://problem/4960137>
+        ASSERTION FAILURE calling SelectionController::setBaseAndExtent (11833)
+
+        * editing/SelectionController.cpp:
+        (WebCore::SelectionController::setSelection): Hand off the selection
+        to the selectionController responsible for it, instead of asserting.
+        A selection object can't be constructed with endpoints in different 
+        documents, so this code is only necessary for one endpoint.
+
+2007-01-29  Ada Chan  <adachan@apple.com>
+
+        Reviewed by Brady.
+        
+        Moved the update of the title of the current entry in the backforward list to WebCore.
+
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::didChangeTitle):
+
+2007-01-29  Adele Peterson  <adele@apple.com>
+
+        Reviewed by Darin.
+
+        More preparation for event handling fixes.
+
+        * bridge/EditorClient.h: Removed EventTargetNode parameter from handleKeyPress since we can just get that from the event.
+        * dom/EventTargetNode.cpp: (WebCore::EventTargetNode::defaultEventHandler): ditto.
+        * page/EventHandler.cpp: (WebCore::EventHandler::defaultKeyboardEventHandler): ditto.
+        * page/EventHandler.h: ditto.
+        * platform/graphics/svg/SVGImageEmptyClients.h: (WebCore::SVGEmptyEditorClient::handleKeyPress): ditto.
+
+        * editing/Editor.h:
+        * editing/Editor.cpp:
+        (WebCore::Editor::handleKeyPress): Removed EventTargetNode parameter.
+        (WebCore::Editor::execCommand): Made commandMap use AtomicStringImpls instead of Strings.
+        (WebCore::Editor::insertText): Moved from the bridge.
+
+        * page/mac/WebCoreFrameBridge.h: Moved insertText method to Editor.
+        * page/mac/WebCoreFrameBridge.mm:
+
+        * WebCore.exp: Updated exports.
+
+2007-01-29  Anders Carlsson  <acarlsson@apple.com>
+
+        Reviewed by Geoff.
+
+        <rdar://problem/4960097>
+        http://bugs.webkit.org/show_bug.cgi?id=12444
+        Assertion failure in -[WebCoreFrameBridge installInFrame:]
+        
+        The problem was that the XML tokenizer didn't create elements with 
+        createdByParser set to true.
+        
+        * dom/Document.cpp:
+        (WebCore::Document::createElement):
+        New method which takes a QName. This is used by both the XML tokenizer and createElementNS.
+        
+        (WebCore::Document::createElementNS):
+        Call new createElement method.
+        
+        * dom/XMLTokenizer.h:        
+        * dom/XMLTokenizer.cpp:
+        (WebCore::XMLTokenizer::XMLTokenizer):
+        (WebCore::XMLTokenizer::~XMLTokenizer):
+        Use an OwnPtr for the pending callbacks object. Also, use String instead of StringImpl in the hash map.
+        
+        (WebCore::toString):
+        (WebCore::XMLTokenizer::startElementNs):
+        Call createElement.
+        
+2007-01-29  Brady Eidson  <beidson@apple.com>
+
+        Reviewed by Oliver
+
+        http://bugs.webkit.org/show_bug.cgi?id=12465
+        Only set the iconURL port for non-zero ports
+
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::iconURL):
+
+2007-01-29  John Sullivan  <sullivan@apple.com>
+
+        * platform/network/mac/ResourceRequestMac.mm:
+        Build fix -- removed #import of no-longer-extant WebDataProtocol.h (and also one of the
+        two #imports for WebCoreSystemInterface.h, while I was at it)
+
+2007-01-29  Anders Carlsson  <acarlsson@apple.com>
+
+        Reviewed by Oliver.
+
+        * page/DragController.cpp:
+        (WebCore::DragController::dragEnteredOrUpdated):
+        Only try a drag operation if the drag client explicitly specified that it is OK
+
+2007-01-29  Oliver Hunt  <oliver@apple.com>
+
+        Reviewed by Adam.
+
+        Migrated more drag logic and state to WebCore
+                                              
+        * page/DragClient.h:       
+
+        * page/DragController.cpp:
+        (WebCore::DragController::delegateDragSourceAction):
+          New forwarding method for the UI delegate
+        * page/DragController.h:
+        (WebCore::DragController::dragOffset):
+          Now store the drag offset -- the offset of the cursor relative to the drag image)
+          in DragController
+        * page/EventHandler.cpp:
+        (WebCore::EventHandler::allowDHTMLDrag):
+          Moved from WebFrameBridge
+        * page/EventHandler.h:
+        * page/mac/EventHandlerMac.mm:
+        (WebCore::EventHandler::eventMayStartDrag):
+        (WebCore::EventHandler::handleDrag):                  
+          EventHandlerMac now uses EventHandler::allowDHTMLDrag rather than the 
+          implementation in WebFrameBridge
+        * page/mac/WebCoreFrameBridge.h:
+          removed definition of allowDHTMLDrag from WebFrameBridge
+        * platform/graphics/svg/SVGImageEmptyClients.h:
+        (WebCore::SVGEmptyDragClient::dragSourceActionMaskForPoint):
+          Updated empty DragClient
+
+2007-01-29  Alexey Proskuryakov  <ap@webkit.org>
+
+        Reviewed by Darin.
+
+        http://bugs.webkit.org/show_bug.cgi?id=12359
+        XPathEvaluator may return some nodes more than once in a result set
+
+        Test: fast/xpath/nodeset-duplicates.html
+
+        * xml/XPathPath.cpp:
+        (WebCore::XPath::LocationPath::doEvaluate): Ensure uniqueness of elements
+        in the node-set.
+
+        * xml/XPathPredicate.cpp:
+        (WebCore::XPath::Union::doEvaluate): Fixed a uniqueness algorithm that was
+        already present here. Added a FIXME about incorrect result ordering.
+
+2007-01-29  Alexey Proskuryakov  <ap@webkit.org>
+
+        Reviewed by Darin.
+
+        http://bugs.webkit.org/show_bug.cgi?id=12340
+        XPathEvaluator behavior does not match Firefox - name() and attribute nodes
+
+        * xml/XPathFunctions.cpp:
+        (WebCore::XPath::FunName::doEvaluate): Do not try to append a prefix if it doesn't exist.
+
+2007-01-29  Alexey Proskuryakov  <ap@webkit.org>
+
+        Reviewed by Darin.
+
+        http://bugs.webkit.org/show_bug.cgi?id=11815
+        XPathEvaluator behavior does not match Firefox - substring() and empty element
+
+        Test: fast/xpath/empty-string-substring.html
+
+        * xml/XPathValue.cpp:
+        (WebCore::XPath::Value::Value):
+        * xml/XPathValue.h:
+        Added a constructor taking const char*, so that string literals and arrays don't 
+        get converted to boolean values.
+
+2007-01-29  Zack Rusin  <zrusin@trolltech.com>
+
+        Cosmetic fixes - don't resize the widget in the constructor
+        (it was a hack from the initial port) and fix the paint
+        signature.
+
+        * platform/qt/ScrollViewCanvasQt.cpp:
+        (WebCore::ScrollViewCanvasQt::ScrollViewCanvasQt):
+        * platform/qt/WidgetQt.cpp:
+        (WebCore::Widget::paint):
+
+2007-01-29  Maciej Stachowiak  <mjs@apple.com>
+
+        - change from struct to class to make picky compilers happy
+
+        * loader/SubstituteData.h:
+
+2007-01-30  Mark Rowe  <mrowe@apple.com>
+
+        Reviewed by Maciej.
+
+        http://bugs.webkit.org/show_bug.cgi?id=12458
+        Bug 12458: Crash in W3C-SVG-1.1/animate-elem-09-t.svg running layout tests under guard malloc
+
+        * ksvg2/svg/SVGAnimationElement.cpp: 
+        (WebCore::parseValues): ptr either points to a ';' or off the end of the buffer.  We're
+        interested in what comes before the ';', so start there instead.
+
+2007-01-29  Zack Rusin  <zrusin@trolltech.com>
+
+        Reviewed by Lars.
+
+        Take the scrollbars into considation when computing
+        the contents size of the main frame.
+
+        * platform/qt/PlatformScrollBarQt.cpp:
+        (WebCore::PlatformScrollbar::paint):
+        * platform/qt/ScrollViewQt.cpp:
+        (WebCore::ScrollView::visibleWidth):
+        (WebCore::ScrollView::visibleHeight):
+        (WebCore::ScrollView::visibleContentRect):
+        * platform/qt/WidgetQt.cpp:
+        (WebCore::Widget::invalidate):
+        (WebCore::Widget::invalidateRect):
+
+2007-01-29  Maciej Stachowiak  <mjs@apple.com>
+
+        - fix includes
+
+        * loader/SubstituteData.h:
+
+2007-01-29  Maciej Stachowiak  <mjs@apple.com>
+
+        - added new file that I forgot in the last commit
+
+        * loader/SubstituteData.h: Added.
+        (WebCore::SubstituteData::SubstituteData):
+        (WebCore::SubstituteData::isValid):
+        (WebCore::SubstituteData::content):
+        (WebCore::SubstituteData::mimeType):
+        (WebCore::SubstituteData::textEncoding):
+        (WebCore::SubstituteData::failingURL):
+
+2007-01-29  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Mark.
+
+        - made data loading cross-platform
+        - fixed <rdar://problem/4910106>
+
+        * WebCore.exp:
+        * WebCore.xcodeproj/project.pbxproj:
+        * loader/CachedResource.cpp:
+        * loader/CachedResource.h:
+        * loader/DocLoader.cpp:
+        (WebCore::DocLoader::requestResource):
+        * loader/DocumentLoader.cpp:
+        (WebCore::DocumentLoader::DocumentLoader):
+        (WebCore::DocumentLoader::setRequest):
+        (WebCore::DocumentLoader::urlForHistory):
+        (WebCore::DocumentLoader::unreachableURL):
+        * loader/DocumentLoader.h:
+        (WebCore::DocumentLoader::substituteData):
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::load):
+        (WebCore::FrameLoader::shouldReloadToHandleUnreachableURL):
+        (WebCore::FrameLoader::reloadAllowingStaleData):
+        (WebCore::FrameLoader::reload):
+        (WebCore::FrameLoader::commitProvisionalLoad):
+        (WebCore::FrameLoader::startLoadingMainResource):
+        (WebCore::FrameLoader::startLoading):
+        (WebCore::FrameLoader::checkNavigationPolicy):
+        * loader/FrameLoader.h:
+        * loader/FrameLoaderClient.h:
+        * loader/MainResourceLoader.cpp:
+        (WebCore::MainResourceLoader::willSendRequest):
+        (WebCore::MainResourceLoader::continueAfterContentPolicy):
+        (WebCore::MainResourceLoader::handleEmptyLoad):
+        (WebCore::MainResourceLoader::handleDataLoad):
+        (WebCore::MainResourceLoader::loadNow):
+        (WebCore::MainResourceLoader::load):
+        * loader/MainResourceLoader.h:
+        * loader/mac/DocumentLoaderMac.mm:
+        * loader/mac/WebDataProtocol.h: Removed.
+        * loader/mac/WebDataProtocol.mm: Removed.
+        * loader/qt/DocumentLoaderQt.cpp:
+        * platform/graphics/svg/SVGImageEmptyClients.h:
+        (WebCore::SVGEmptyFrameLoaderClient::createDocumentLoader):
+
+2007-01-29  Zack Rusin  <zrusin@trolltech.com>
+
+        Order the build files alphabetically instead of
+        the random mess we had before.
+
+        * WebCore.pro:
+
+2007-01-29  Zack Rusin  <zack@kde.org>
+
+        Reviewed by Lars
+
+        Use the platform independent scrollbar code.
+        Remove the redundant empty stubs and forward
+        few missing platform scrollbar calls.
+
+        * WebCore.pro:
+        * platform/qt/PlatformScrollBarQt.cpp:
+        (WebCore::PlatformScrollbar::paint):
+        (WebCore::PlatformScrollbar::setRect):
+
+2007-01-29  Lars Knoll <lars@trolltech.com>
+
+        Reviewed by Zack
+
+        Fix the way we handle native widgets (QWidget) inside
+        WebCore. Now WebCore::Widget owns the QWidget in all
+        cases. This is the only way to get well defined semantics
+        for all teh native widgets we have (currently ScrollView
+        and ScrollBar, but plugins will follow).
+
+        This has the side effect that one cannot rely on a defined 
+        lifetime of a QWebFrame when using the API. 
+
+        * platform/Widget.h:
+        * platform/qt/PlatformScrollBarQt.cpp:
+        (WebCore::PlatformScrollbar::PlatformScrollbar):
+        * platform/qt/ScrollViewQt.cpp:
+        (WebCore::ScrollView::addChild):
+        (WebCore::ScrollView::removeChild):
+        * platform/qt/WidgetQt.cpp:
+        (WebCore::WidgetPrivate::WidgetPrivate):
+        (WebCore::WidgetPrivate::~WidgetPrivate):
+        (WebCore::Widget::qwidget):
+
+2007-01-28  George Staikos  <staikos@kde.org>
+
+        Reviewed by Maciej.
+
+        Don't send data during a redirect -> causes assert failure.
+
+        * platform/network/qt/ResourceHandleManagerQt.cpp:
+        (WebCore::ResourceHandleManager::receivedData):
+
+2007-01-29  Mark Rowe  <mrowe@apple.com>
+
+        Reviewed by Brady.
+
+        http://bugs.webkit.org/show_bug.cgi?id=11085
+        Bug 11085: REGRESSION: favicon.ico always looked for on port 80
+
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::iconURL): Copy the port over to the favicon URL.
+
+2007-01-28  Adam Roben  <aroben@apple.com>
+
+        Reviewed by Adele.
+
+        Fix http://bugs.webkit.org/show_bug.cgi?id=12453
+        REGRESSION(r19197): Tabbing into web page requires two key presses
+
+        All layout tests pass.
+
+        * page/FocusController.cpp:
+        (WebCore::FocusController::advanceFocus): If there's no focused frame,
+        actually go ahead and find a focusable node instead of just focusing
+        the main frame and quitting. This behavior was a holdover from a
+        previous desired tabbing behavior with respect to frames.
+
+2007-01-28  David Kilzer  <ddkilzer@webkit.org>
+
+        Reviewed by Darin and David Hyatt.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=12423
+          Mixing white-space:pre text with non white-space:pre text does not wrap properly
+
+        Test: fast/block/basic/white-space-pre-wraps.html
+
+        * rendering/bidi.cpp:
+        (WebCore::RenderBlock::findNextLineBreak): Wrap a non-wrapping text run if it
+        appears on a line with previous text runs that do wrap.
+
+2007-01-28  Mitz Pettel  <mitz@webkit.org>
+
+        Reviewed by Darin.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=8360
+          Repro crash when onscroll handler deletes the scrolled object
+
+        Test: fast/layers/removed-by-scroll-handler.html
+
+        * page/FrameView.cpp:
+        (WebCore::FrameViewPrivate::FrameViewPrivate): Added a m_enqueueEvents count.
+        (WebCore::FrameView::layout): Raise the enqueue count before starting layout and
+        until after enqueued events are dispatched.
+        (WebCore::FrameView::scheduleEvent): Made this function dispatch the event
+        immediately unless the enqueue count is positive.
+        * rendering/RenderLayer.cpp:
+        (WebCore::RenderLayer::scrollToOffset): Schedule the scroll event through the
+        FrameView.
+        (WebCore::RenderLayer::scrollRectToVisible):
+        (WebCore::RenderLayer::updateScrollInfoAfterLayout):
+        (WebCore::Marquee::start):
+
+2007-01-28  Charles Ying  <charles_ying@yahoo.com>
+
+        Reviewed by eseidel.  Landed by eseidel.
+
+        Fix http://bugs.webkit.org/show_bug.cgi?id=12429
+            animateTransform rotate, with cx/cy is broken
+            
+        SVGTransform now contains a float point to store the rotation center, needed
+            to track rotation center for interpolation by animateTransform.
+            
+        SVGTransform now has an additional constructor:
+            SVGTransform(type) to make empty base value transforms that match their
+            distanced values.
+            
+        Removed m_colorDistance and m_transformDistance from their respective classes
+            as they are now computed at update time.
+
+        Fix a case where animateTransform did not parse transform values correctly
+            according to BNF in W3C spec:
+
+        Test case in:
+        * manual-tests/animation/animateTransform-toanimation.svg: Added.
+
+        Fix a case where animateTransform only specified a to animation only.
+
+        Test case in:
+        * manual-tests/animation/animateTransform-parser.svg: Added.
+
+        * ksvg2/svg/SVGAnimateColorElement.cpp:
+        (WebCore::SVGAnimateColorElement::updateAnimationBaseValueFromElement):
+        (WebCore::SVGAnimateColorElement::updateAnimatedValue):
+        (WebCore::SVGAnimateColorElement::calculateFromAndToValues):
+        * ksvg2/svg/SVGAnimateColorElement.h:
+        * ksvg2/svg/SVGAnimateTransformElement.cpp:
+        (WebCore::SVGAnimateTransformElement::updateAnimatedValue):
+        (WebCore::SVGAnimateTransformElement::updateAnimationBaseValueFromElement):
+        (WebCore::SVGAnimateTransformElement::calculateFromAndToValues):
+        (WebCore::SVGAnimateTransformElement::parseTransformValue):
+        * ksvg2/svg/SVGAnimateTransformElement.h:
+        * ksvg2/svg/SVGTransform.cpp:
+        (SVGTransform::SVGTransform):
+        (SVGTransform::angle):
+        (SVGTransform::rotationCenter):
+        (SVGTransform::setTranslate):
+        (SVGTransform::setScale):
+        (SVGTransform::setRotate):
+        (SVGTransform::setSkewX):
+        (SVGTransform::setSkewY):
+        * ksvg2/svg/SVGTransform.h:
+        * ksvg2/svg/SVGTransformDistance.cpp:
+        (WebCore::SVGTransformDistance::SVGTransformDistance):
+        (WebCore::SVGTransformDistance::scaledDistance):
+        (WebCore::SVGTransformDistance::addSVGTransforms):
+        (WebCore::SVGTransformDistance::addSVGTransform):
+        (WebCore::SVGTransformDistance::addToSVGTransform):
+        (WebCore::SVGTransformDistance::distance):
+        * ksvg2/svg/SVGTransformDistance.h:
+        * manual-tests/animation/animateTransform-parser.svg: Added.
+        * manual-tests/animation/animateTransform-toanimation.svg: Added.
+
+2007-01-27  Adam Roben  <aroben@apple.com>
+
+        Reviewed by Ada.
+
+        Fix <rdar://problem/4957184> REGRESSION: After creating a new mail
+        message, the caret is automatically in message body
+
+        -[WebHTMLView _updateActiveState] was calling
+        FocusController::focusedOrMainFrame, which had the surprising
+        side-effect of setting the focused frame if there was none. The fix
+        was to remove this side-effect.
+
+        No tests possible, as this bug involves focus outside the WebView,
+        which we have no support for in DRT.
+
+        Change inspired by Dave Harrison.
+
+        * page/FocusController.cpp:
+        (WebCore::FocusController::focusedOrMainFrame): Removed this method's
+        non-obvious side-effect of setting the focused frame.
+        * WebCore.xcodeproj/project.pbxproj: Version wars.
+
+2007-01-27  Don Gibson  <dgibson77@gmail.com>
+
+        Reviewed by Maciej.
+
+        http://bugs.webkit.org/show_bug.cgi?id=11880: Improve some egregious
+        Windows form control drawing issues:
+        Provide Classic mode fallbacks for the button/textfield drawing
+        routines.
+        Provide some basic drawing of menulists (still relatively broken).
+        Some cleanup.
+
+        * platform/win/TemporaryLinkStubs.cpp:
+        (WebCore::RenderThemeWin::systemFont):
+        * rendering/RenderThemeWin.cpp:
+        (WebCore::m_menuListTheme):
+        (WebCore::RenderThemeWin::close):
+        (WebCore::RenderThemeWin::determineClassicState):
+        (WebCore::RenderThemeWin::getThemeData):
+        (WebCore::RenderThemeWin::paintButton):
+        (WebCore::RenderThemeWin::paintTextField):
+        (WebCore::RenderThemeWin::paintMenuList):
+        (WebCore::RenderThemeWin::paintMenuListButton):
+        * rendering/RenderThemeWin.h:
+        (WebCore::ThemeData::m_classicState):
+        (WebCore::RenderThemeWin::setRadioSize):
+        (WebCore::RenderThemeWin::paintTextArea):
+
+2007-01-27  David Kilzer  <ddkilzer@webkit.org>
+
+        Reviewed by Adam.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=12260
+          Windows platform build is not maintained
+
+        * loader/win/FrameLoaderWin.cpp:
+        * platform/network/win/ResourceHandleWin.cpp:
+        * platform/win/TemporaryLinkStubs.cpp:
+        (WebCore::contextMenuItemTagOpenLinkInNewWindow):
+        (WebCore::contextMenuItemTagDownloadLinkToDisk):
+        (WebCore::contextMenuItemTagCopyLinkToClipboard):
+        (WebCore::contextMenuItemTagOpenImageInNewWindow):
+        (WebCore::contextMenuItemTagDownloadImageToDisk):
+        (WebCore::contextMenuItemTagCopyImageToClipboard):
+        (WebCore::contextMenuItemTagOpenFrameInNewWindow):
+        (WebCore::contextMenuItemTagCopy):
+        (WebCore::contextMenuItemTagGoBack):
+        (WebCore::contextMenuItemTagGoForward):
+        (WebCore::contextMenuItemTagStop):
+        (WebCore::contextMenuItemTagReload):
+        (WebCore::contextMenuItemTagCut):
+        (WebCore::contextMenuItemTagPaste):
+        (WebCore::contextMenuItemTagNoGuessesFound):
+        (WebCore::contextMenuItemTagIgnoreSpelling):
+        (WebCore::contextMenuItemTagLearnSpelling):
+        (WebCore::contextMenuItemTagSearchWeb):
+        (WebCore::contextMenuItemTagLookUpInDictionary):
+        (WebCore::contextMenuItemTagOpenLink):
+        (WebCore::contextMenuItemTagIgnoreGrammar):
+        (WebCore::contextMenuItemTagSpellingMenu):
+        (WebCore::contextMenuItemTagShowSpellingPanel):
+        (WebCore::contextMenuItemTagCheckSpelling):
+        (WebCore::contextMenuItemTagCheckSpellingWhileTyping):
+        (WebCore::contextMenuItemTagCheckGrammarWithSpelling):
+        (WebCore::contextMenuItemTagFontMenu):
+        (WebCore::contextMenuItemTagBold):
+        (WebCore::contextMenuItemTagItalic):
+        (WebCore::contextMenuItemTagUnderline):
+        (WebCore::contextMenuItemTagOutline):
+        (WebCore::contextMenuItemTagWritingDirectionMenu):
+        (WebCore::contextMenuItemTagDefaultDirection):
+        (WebCore::contextMenuItemTagLeftToRight):
+        (WebCore::contextMenuItemTagRightToLeft):
+        (WebCore::DocumentLoader::unreachableURL):
+        (WebCore::FrameLoader::redirectDataToPlugin):
+        (WebCore::FrameLoader::partClearedInBegin):
+        (WebCore::FrameLoader::createPlugin):
+
+2007-01-27  Sam Weinig  <sam@webkit.org>
+
+        Reviewed by Mitz.
+
+        - Patch for http://bugs.webkit.org/show_bug.cgi?id=11957
+          REGRESSION: Unordered List Bullets too big and changing from size
+
+        * rendering/RenderListMarker.cpp:
+        (WebCore::RenderListMarker::paint): Explicitly set the stroke size to 1.0f.
+
+2007-01-27  Lars Knoll <lars@trolltech.com>
+
+        Fix the Qt build and disable the usage
+        of QScrollbars for PlatformScrollBar for 
+        now as it causes crashes in the layout tests.
+
+        * WebCore.pro:
+        * ksvg2/svg/SVGTransformDistance.cpp:
+        * page/qt/FrameQt.cpp:
+        (WebCore::FrameQt::FrameQt):
+        (WebCore::FrameQt::bindingRootObject):
+        (WebCore::FrameQt::createRootObject):
+        * page/qt/FrameQt.h:
+        * platform/qt/PlatformScrollBarQt.cpp:
+        (WebCore::PlatformScrollbar::PlatformScrollbar):
+        * platform/qt/ScrollViewQt.cpp:
+        (WebCore::ScrollView::addChild):
+
+2007-01-27  David Kilzer  <ddkilzer@kilzer.net>
+
+        Reviewed by NOBODY (buildbot build fix).
+
+        * ksvg2/svg/ColorDistance.cpp:
+        (WebCore::ColorDistance::scaledDistance): Added static cast of arguments to int.
+
+2007-01-27  Rob Buis  <buis@kde.org>
+
+        Reviewed by NOBODY (typo in method name fix).
+
+        * ksvg2/svg/SVGSVGElement.cpp:
+        (WebCore::SVGSVGElement::addSVGWindowEventListener):
+        (WebCore::SVGSVGElement::parseMappedAttribute):
+        * ksvg2/svg/SVGSVGElement.h:
+
+2007-01-27  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by olliej.
+
+        Fix support for SVG's <set> element.
+
+        Manual test added.
+
+        * ksvg2/svg/SVGSetElement.cpp:
+        (WebCore::SVGSetElement::updateAnimatedValue):
+        (WebCore::SVGSetElement::calculateFromAndToValues):
+        * ksvg2/svg/SVGSetElement.h:
+        * manual-tests/animation/set-to.svg: Added.
+
+2007-01-27  Alexey Proskuryakov  <ap@webkit.org>
+
+        Release build fix.
+
+        * loader/TextResourceDecoder.cpp:
+        (WebCore::TextResourceDecoder::checkForHeadCharset):
+
+2007-01-27  Alexey Proskuryakov  <ap@webkit.org>
+
+        Reviewed by Darin.
+
+        http://bugs.webkit.org/show_bug.cgi?id=12389
+        Chinese decoding error at hk.antispam.yahoo.com
+
+        Test: fast/encoding/noscript-in-head.html
+
+        * loader/TextResourceDecoder.cpp:
+        (WebCore::TextResourceDecoder::checkForHeadCharset): We were already allowing 
+        NOSCRIPT tags in head, let's actually skip over their content.
+
+2007-01-26  Geoffrey Garen  <ggaren@apple.com>
+
+        Reviewed by Maciej Stachowiak.
+        
+        Fixed <rdar://problem/4608404> WebScriptObject's _executionContext lack 
+        of ownership policy causes crashes (e.g., in Dashcode)
+
+        Added RootObject ref-counting goodness.
+
+        * page/mac/FrameMac.h:
+        * page/mac/FrameMac.mm:
+        (WebCore::FrameMac::cleanupPluginObjects): Invalidate our RootObjects 
+        instead of detroying them. Track _bindingRootObject separately from the
+        rest of our RootObjects, since it has its own variable.
+
+        * page/mac/WebCoreFrameBridge.mm:
+        (createRootObject): Use the Frame's new, more encapsulated function to
+        create a RootObject.
+
+        * bindings/objc/WebScriptObject.mm: Nixed rootObject setters, since they
+        were unused and they complicated reference-counting.
+
+2007-01-26  David Kilzer  <ddkilzer@webkit.org>
+
+        Reviewed by Eric.
+
+        Release build fix.
+
+        * ksvg2/svg/SVGAnimationElement.cpp:
+        (WebCore::SVGAnimationElement::calculateValueIndexAndPercentagePast): Added static cast
+        to unsigned when calculating flooredValueIndex.
+
+2007-01-26  Darin Adler  <darin@apple.com>
+
+        - checked in slightly newer revision of the patch below
+
+        * css/CSSComputedStyleDeclaration.cpp:
+        (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
+        Grouped all the unimplemented values at the bottom. Fixed the ifdef for
+        CSS_PROP__WEBKIT_DASHBOARD_REGION so that it compiles on all platforms.
+
+        * css/cssparser.cpp: (WebCore::CSSParser::parseValue): Removed default
+        case and added type so that the compiler warns about missing properties.
+        * css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::applyProperty): Ditto.
+
+2007-01-26  Darin Adler  <darin@apple.com>
+
+        Reviewed by Beth.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=12412
+          "unimplemented propertyID: 46" seen at dojotoolkit.org
+
+        Covered by fast/css/computed-style.html
+
+        * css/CSSPropertyNames.in: Sorted the file into two separate sections, one
+        for properties without the -webkit- prefix and another for properties with.
+
+        * css/makeprop: Generate a CSSPropertyID enum instead of a set of #defines.
+        Removed unused CSS_PROP_MIN and CSS_PROP_MAX. Renamed CSS_PROP_TOTAL to
+        numCSSProperties and changed it to a const int instead of #define.
+        Added a declaration of getPropertyName in the header. Changed the return
+        type of getPropertyName to const char* and the parameter type to CSSPropertyID.
+
+        * css/CSSComputedStyleDeclaration.cpp: Sort the lists of properties in
+        an easier to maintain order -- properties without a WebKit prefix first.
+        (WebCore::valueForLength): Changed return value to PassRefPtr.
+        (WebCore::valueForMaxLength): Ditto.
+        (WebCore::valueForBorderStyle): Ditto.
+        (WebCore::valueForTextAlign): Ditto.
+        (WebCore::valueForAppearance): Added.
+        (WebCore::valueForMarginCollapse): Added.
+        (WebCore::valueForShadow): Changed return value to PassRefPtr.
+        (WebCore::getPositionOffsetValue): Ditto.
+        (WebCore::currentColorOrValidColor): Ditto.
+        (WebCore::CSSComputedStyleDeclaration::cssText): Added cast since
+        getPropertyName now takes an enum value.
+        (WebCore::primitiveValueFromLength): Changed return value to PassRefPtr.
+        (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
+        Added cast to CSSPropertyID so that gcc will detect missing enums in the
+        switch statement. Simplified the background-size and border-spacing
+        implementations so they don't have local variables. Added lots of cases
+        to the switch statement for unimplemented properties. Implemented the
+        outline-width, box-sizing, -webkit-appearance, -webkit-rtl-ordering,
+        -webkit-user-drag, and -webkit-user-select properties.
+        (WebCore::CSSComputedStyleDeclaration::item): Added cast since
+        getPropertyName now takes an enum value.
+
+        * css/CSSMutableStyleDeclaration.cpp: Removed declaration of getPropertyName.
+        (WebCore::CSSMutableStyleDeclaration::item): Added cast since
+        getPropertyName now takes an enum value.
+
+        * css/CSSProperty.cpp: Removed declaration of getPropertyName.
+        (WebCore::CSSProperty::cssText): Added cast since
+        getPropertyName now takes an enum value and returns a const char*.
+
+        * css/CSSStyleDeclaration.cpp: Removed declaration of getPropertyName.
+        (WebCore::CSSStyleDeclaration::getPropertyShorthand): Added cast since
+        getPropertyName now takes an enum value and returns a const char*.
+
+2007-01-26  Alice Liu  <alice.liu@apple.com>
+
+        Reviewed by Brady.
+
+        Fixed <rdar://problem/4838076> CrashTracer: 10 crashes in Mail at com.apple.WebCore: WTF::HashMap<WebCore::String
+
+        * loader/DocLoader.cpp:
+        (WebCore::DocLoader::requestResource):
+        Added nil check around resource from cache.
+
+2007-01-26  Adam Roben  <aroben@apple.com>
+
+        Reviewed by Oliver.
+
+        Build fix.
+
+        * ksvg2/svg/SVGAnimationElement.cpp:
+        (WebCore::SVGAnimationElement::calculateValueIndexAndPercentagePast):
+        Keep index variables as unsigned until we need to convert to float.
+
+2007-01-26  Alice Liu  <alice.liu@apple.com>
+
+        Reviewed by Ada.
+
+        Fixed <rdar://problem/4919097> REGRESSION: Ctrl-Y fails to yank (11076)
+
+        * editing/Editor.cpp:
+        (WebCore::Editor::deleteWithDirection):
+        If there's no selection, locally create a range that is to the end of the granularity unit and pass that to deleteRange(). 
+
+2007-01-26  Sam Weinig  <sam@webkit.org>
+
+        Reviewed by Maciej.
+
+        Qt build fix.
+
+        * page/qt/EventHandlerQt.cpp:
+
+2007-01-26  Dex Deacon  <occupant4@gmail.com>
+
+        Reviewed by Hyatt.
+
+        Fix so that we only relayout frameset children on calls to layout.
+
+        * page/FrameView.cpp:
+        (WebCore::FrameView::layout):
+
+2007-01-26  Oliver Hunt  <oliver@apple.com>
+
+        Build Fix
+
+        * ksvg2/svg/ColorDistance.cpp:
+        * ksvg2/svg/SVGAnimationElement.cpp:
+        (WebCore::SVGAnimationElement::handleTimerEvent):
+
+2007-01-26  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by olliej.
+
+        Add support for calcMode, keyTimes and (nearly) keySplines.
+        http://bugs.webkit.org/show_bug.cgi?id=12350
+        Re-architect much of the animation system to remove huge sections of code.
+        Added new ColorDistance, SVGTransformDistance to simplify animateColor, animateTransform calculations.
+        I also added basic <animateMotion> support since it was so easy w/ the new architecture.
+        
+        Adding manual animation tests until bug 12074 is fixed.
+
+        * WebCore.xcodeproj/project.pbxproj:
+        * ksvg2/misc/SVGTimer.cpp:
+        (WebCore::SVGTimer::applyAnimations):
+        * ksvg2/svg/ColorDistance.cpp: Added.
+        (WebCore::ColorDistance::ColorDistance):
+        (WebCore::clampColorValue):
+        (WebCore::ColorDistance::scaledDistance):
+        (WebCore::ColorDistance::addColorsAndClamp):
+        (WebCore::ColorDistance::addToColorAndClamp):
+        (WebCore::ColorDistance::isZero):
+        (WebCore::ColorDistance::distance):
+        * ksvg2/svg/ColorDistance.h: Added.
+        * ksvg2/svg/SVGAnimateColorElement.cpp:
+        (WebCore::SVGAnimateColorElement::SVGAnimateColorElement):
+        (WebCore::SVGAnimateColorElement::updateAnimationBaseValueFromElement):
+        (WebCore::SVGAnimateColorElement::applyAnimatedValueToElement):
+        (WebCore::SVGAnimateColorElement::updateAnimatedValue):
+        (WebCore::SVGAnimateColorElement::calculateFromAndToValues):
+        * ksvg2/svg/SVGAnimateColorElement.h:
+        (WebCore::SVGAnimateColorElement::contextElement):
+        * ksvg2/svg/SVGAnimateElement.h:
+        (WebCore::SVGAnimateElement::updateAnimatedValue):
+        (WebCore::SVGAnimateElement::calculateFromAndToValues):
+        * ksvg2/svg/SVGAnimateMotionElement.cpp:
+        (WebCore::SVGAnimateMotionElement::hasValidTarget):
+        (WebCore::SVGAnimateMotionElement::updateAnimatedValue):
+        (WebCore::parsePoint):
+        (WebCore::SVGAnimateMotionElement::calculateFromAndToValues):
+        (WebCore::SVGAnimateMotionElement::updateAnimationBaseValueFromElement):
+        (WebCore::SVGAnimateMotionElement::applyAnimatedValueToElement):
+        * ksvg2/svg/SVGAnimateMotionElement.h:
+        * ksvg2/svg/SVGAnimateTransformElement.cpp:
+        (WebCore::SVGAnimateTransformElement::SVGAnimateTransformElement):
+        (WebCore::SVGAnimateTransformElement::updateAnimatedValue):
+        (WebCore::SVGAnimateTransformElement::updateAnimationBaseValueFromElement):
+        (WebCore::SVGAnimateTransformElement::applyAnimatedValueToElement):
+        (WebCore::SVGAnimateTransformElement::calculateFromAndToValues):
+        (WebCore::SVGAnimateTransformElement::parseTransformValue):
+        * ksvg2/svg/SVGAnimateTransformElement.h:
+        (WebCore::SVGAnimateTransformElement::contextElement):
+        * ksvg2/svg/SVGAnimationElement.cpp:
+        (WebCore::SVGAnimationElement::parseKeyNumbers):
+        (WebCore::parseValues):
+        (WebCore::parseKeySplines):
+        (WebCore::SVGAnimationElement::parseMappedAttribute):
+        (WebCore::SVGAnimationElement::targetAttributeAnimatedValue):
+        (WebCore::SVGAnimationElement::setTargetAttributeAnimatedValue):
+        (WebCore::SVGAnimationElement::detectAnimationMode):
+        (WebCore::adjustPercentagePastForKeySplines):
+        (WebCore::SVGAnimationElement::valueIndexAndPercentagePastForDistance):
+        (WebCore::SVGAnimationElement::calculateTotalDistance):
+        (WebCore::caculateValueIndexForKeyTimes):
+        (WebCore::SVGAnimationElement::isValidAnimation):
+        (WebCore::SVGAnimationElement::calculateValueIndexAndPercentagePast):
+        (WebCore::SVGAnimationElement::updateAnimationBaseValueFromElement):
+        (WebCore::SVGAnimationElement::applyAnimatedValueToElement):
+        (WebCore::SVGAnimationElement::handleTimerEvent):
+        (WebCore::SVGAnimationElement::updateAnimatedValueForElapsedSeconds):
+        * ksvg2/svg/SVGAnimationElement.h:
+        * ksvg2/svg/SVGParserUtilities.cpp:
+        (WebCore::SVGPolyParser::parsePoints):
+        * ksvg2/svg/SVGParserUtilities.h:
+        * ksvg2/svg/SVGSetElement.cpp:
+        * ksvg2/svg/SVGSetElement.h:
+        (WebCore::SVGSetElement::updateAnimatedValue):
+        (WebCore::SVGSetElement::calculateFromAndToValues):
+        * ksvg2/svg/SVGTransform.cpp:
+        (SVGTransform::type):
+        (SVGTransform::translate):
+        (SVGTransform::scale):
+        (SVGTransform::setSkewX):
+        (SVGTransform::setSkewY):
+        * ksvg2/svg/SVGTransform.h:
+        (WebCore::operator==):
+        (WebCore::operator!=):
+        * ksvg2/svg/SVGTransformDistance.cpp: Added.
+        (WebCore::SVGTransformDistance::SVGTransformDistance):
+        (WebCore::SVGTransformDistance::scaledDistance):
+        (WebCore::SVGTransformDistance::addSVGTransforms):
+        (WebCore::SVGTransformDistance::addSVGTransform):
+        (WebCore::SVGTransformDistance::addToSVGTransform):
+        (WebCore::SVGTransformDistance::isZero):
+        (WebCore::SVGTransformDistance::distance):
+        * ksvg2/svg/SVGTransformDistance.h: Added.
+        * ksvg2/svg/SVGTransformList.cpp:
+        (SVGTransformList::concatenateForType):
+        * ksvg2/svg/SVGTransformList.h:
+        * manual-tests/animation/animateColor-by.svg: Added.
+        * manual-tests/animation/animateColor-from-by.svg: Added.
+        * manual-tests/animation/animateColor-repeat-indefinite.svg: Added.
+        * manual-tests/animation/animateColor-to-from.svg: Added.
+        * manual-tests/animation/animateColor-to.svg: Added.
+        * manual-tests/animation/animateColor-values-simple.svg: Added.
+        * manual-tests/animation/animateMotion-to.svg: Added.
+        * manual-tests/animation/animateTransform-keyTimes.svg: Added.
+        * manual-tests/animation/animateTransform-repeat-once.svg: Added.
+
+2007-01-26  Beth Dakin  <bdakin@apple.com>
+
+        Reviewed by Darin.
+
+        Fix for <rdar://problem/4956565> REGRESSION: After scrolling frame, 
+        hovering over link in this frame doesn't change cursor to pointing 
+        hand
+
+        The mouseMove event was not being propagated correctly after using 
+        the mouse to scroll the frame because m_mousePressed was never 
+        getting set to false.
+
+        * page/EventHandler.cpp:
+        (WebCore::EventHandler::handleMousePressEvent): This code does not 
+        belong here.
+        * page/EventHandler.h: lastEventIsMouseUp() is only ever relevant 
+        in EventHandlerMac, so it can just be a static function there.
+        * page/mac/EventHandlerMac.mm:
+        (WebCore::lastEventIsMouseUp): Make this static.
+        (WebCore::EventHandler::passMouseDownEventToWidget): Here is where 
+        we need to set m_mousePressed to false if lastEventIsMouseUp() is 
+        true.
+
+2007-01-26  David Hyatt  <hyatt@apple.com>
+
+        Fix for style regression caused by strictness checking of the number of properties.  This caused code like:
+        "width: 20 px" to fail, because we interpreted it as two values and rejected it.  Our old code allowed 20 to be
+        used like a pixel value and then ignored the orphaned unit.
+
+        This patch actually scans the list of values looking for orphaned units and reattaches them to the numeric
+        values that they should correspond to.  This means rules like "width: 5 em" will now work in quirks mdoe and
+        the "em" unit type will be honored.
+
+        Reviewed by beth
+
+        * WebCore.xcodeproj/project.pbxproj:
+        * css/cssparser.cpp:
+        (WebCore::unitFromString):
+        (WebCore::CSSParser::checkForOrphanedUnits):
+        (WebCore::CSSParser::parseValue):
+        * css/cssparser.h:
+        (WebCore::ValueList::valueAt):
+        (WebCore::ValueList::deleteValueAt):
+
+2007-01-26  George Staikos  <staikos@kde.org>
+
+        Reviewed by Zack.
+
+        Qt implementation.
+
+        * WebCore.pro:
+        * platform/qt/FontPlatformDataQt.cpp:
+        (WebCore::FontPlatformData::FontPlatformData):
+        * platform/qt/ScrollViewQt.cpp:
+        (WebCore::ScrollView::scrollbarUnderMouse):
+
+2007-01-26  Brady Eidson  <beidson@apple.com>
+
+        Reviewed by Anders' rubberstamp
+        
+        Warning about unreferenced formal parameter caused compile to fail
+
+        * platform/cf/RetainPtr.h:
+        (WebCore::adoptCFReference):
+
+2007-01-26  Darin Adler  <darin@apple.com>
+
+        Reviewed by Brady.
+
+        - fix <rdar://problem/4956688> move Mac-specific part of icon code into IconMac
+        - fix a GC-compatibility problem in the icon code I noticed by code inspection
+
+        * platform/FileChooser.cpp: (WebCore::FileChooser::chooseIcon): Removed Mac-specific
+        path check -- that's in newIconForFile now.
+
+        * platform/graphics/Icon.h: Changed the image to be a RetainPtr. The old code was
+        using -[NSObject retain] which is not good enough for a pointer in a C++ object.
+        * platform/graphics/mac/IconMac.mm:
+        (WebCore::Icon::Icon): Got rid of unneeded code to do what RetainPtr does for us.
+        (WebCore::Icon::~Icon): Ditto.
+        (WebCore::Icon::newIconForFile): Added check for non-absolute file names. Updated
+        for use of RetainPtr. Use constructor that takes NSImage.
+        (WebCore::Icon::paint): Add get() calls needed now that we are using a RetainPtr.
+
+        * WebCore.xcodeproj/project.pbxproj: Let Xcode do its thing.
+
+2007-01-26  Alexey Proskuryakov  <ap@webkit.org>
+
+        Reviewed by Maciej.
+
+        http://bugs.webkit.org/show_bug.cgi?id=12385
+        Assertion failure when loading subresources from "data:" frames
+
+        Test: fast/loader/opaque-base-url.html
+
+        * platform/KURL.cpp:
+        (WebCore::KURL::init): Make the URL invalid if it's relative, but has an opaque or empty base.
+
+2007-01-26  Dex Deacon  <occupant4@gmail.com>
+
+        Reviewed by Darin.
+
+        * page/EventHandler.cpp:
+        (WebCore::EventHandler::handleMouseMoveEvent):
+        Removed a call to invalidateClick(), since we don't want to cancel a
+        click if the mouse moves.
+
+2007-01-26  Darin Adler  <darin@apple.com>
+
+        * css/MediaQueryEvaluator.cpp:
+        (WebCore::MediaQueryEvaluator::mediaTypeMatchSpecific):
+        Oops, forgot a !.
+
+2007-01-26  Darin Adler  <darin@apple.com>
+
+        Patch by Nate Begeman.
+        Reviewed by Darin and Maciej.
+
+        - <rdar://problem/4946877> Avoid unnecessary allocations in CSS, patch attached (12401)
+          http://bugs.webkit.org/show_bug.cgi?id=12401
+
+        * css/MediaQueryEvaluator.h: Add mediaTypeMatchSpecific, a function that
+        takes a char* to avoid creating and destroying a String for simple media
+        type queries.
+        * css/MediaQueryEvaluator.cpp:
+        (WebCore::MediaQueryEvaluator::mediaTypeMatchSpecific): Added. Besides
+        avoiding the conversion to String, this also doesn't have the special
+        case code to handle "" and "all".
+
+        * css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::matchUARules):
+        Call mediaTypeMatchSpecific here when deciding whether to use the
+        print version of the user agent style sheet.
+
+        * loader/Cache.cpp: (WebCore::Cache::requestResource): Don't convert to
+        a string and back to a KURL when calling createResource. Avoids reparsing
+        the URL string.
+
+2007-01-26  Zack Rusin  <zrusin@trolltech.com>
+
+        Reviewed by Lars
+
+        Add history related classes to the project file.
+
+        * WebCore.pro:
+
+2007-01-26  Zack Rusin  <zack@kde.org>
+
+        Reviewed by Lars
+
+        Introduce PlatformScrollBar and try to make 
+        the scrollbar behavior a little more sane.
+
+        * WebCore.pro:
+        * platform/qt/PlatformScrollBarQt.cpp: Added.
+        (WebCore::Scrollbar::Scrollbar):
+        (WebCore::Scrollbar::setSteps):
+        (WebCore::Scrollbar::scroll):
+        (WebCore::Scrollbar::setValue):
+        (WebCore::Scrollbar::setProportion):
+        (WebCore::PlatformScrollbar::PlatformScrollbar):
+        (WebCore::PlatformScrollbar::~PlatformScrollbar):
+        (WebCore::PlatformScrollbar::width):
+        (WebCore::PlatformScrollbar::height):
+        (WebCore::PlatformScrollbar::setEnabled):
+        (WebCore::PlatformScrollbar::paint):
+        (WebCore::PlatformScrollbar::updateThumbPosition):
+        (WebCore::PlatformScrollbar::updateThumbProportion):
+        (WebCore::PlatformScrollbar::setRect):
+        * platform/qt/ScrollViewQt.cpp:
+        (WebCore::ScrollView::addChild):
+        * platform/qt/TemporaryLinkStubs.cpp:
+        (WebCore::PlatformMouseEvent::PlatformMouseEvent):
+
+2007-01-26  Mitz Pettel  <mitz@webkit.org>
+
+        Reviewed by Darin.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=11109
+          REGRESSION (r15471): menus are offset on eink.com
+          <rdar://problem/4824760>
+
+        Changed offset{Left,Top} to match Firefox. If the body element is statically
+        positioned, it is still returned as the offsetParent of elements whose containing
+        block is the initial containing block (the root), but their offset{Left,Top} are
+        relative to the root. If the body element is positioned, then offsets are relative
+        to it. The body's own offset{Left,Top} are always 0. 
+
+        Test: fast/dom/Element/offsetLeft-offsetTop-body-quirk.html
+
+        * rendering/RenderObject.cpp:
+        (WebCore::RenderObject::offsetLeft):
+        (WebCore::RenderObject::offsetTop):
+
+2007-01-26  Mitz Pettel  <mitz@webkit.org>
+
+        Reviewed by Darin and Maciej.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=10983
+          REGRESSION (r12290): Drop shadow of flickr photo note is positioned incorrectly the second time it's shown
+
+        Test: fast/dynamic/staticY.html
+
+        * rendering/RenderBlock.cpp:
+        (WebCore::RenderBlock::layoutPositionedObjects): Avoid marking the object's
+        parents, since they have already been laid out.
+        * rendering/RenderBox.cpp:
+        (WebCore::RenderBox::setStaticY): Added a call to setChildNeedsLayout() to
+        ensure relayout by this box's container (third and fourth cases in the
+        layout test).
+        * rendering/RenderBox.h:
+        * rendering/RenderObject.cpp:
+        (WebCore::RenderObject::markContainingBlocksForLayout): For positioned objects
+        with static y, also mark the object's parent to ensure that it recomputes
+        the static y value (first case in the layout test).
+
+2007-01-26  Lars Knoll <lars@trolltech.com>
+
+        Reviewed by Zack
+
+        Get rid of FrameQtClient. It's unused since we moved over
+        to the loader.
+
+        * WebCore.pro:
+        * page/qt/FrameQt.cpp:
+        (WebCore::FrameQt::FrameQt):
+        * page/qt/FrameQt.h:
+        * page/qt/FrameQtClient.cpp: Removed.
+        * page/qt/FrameQtClient.h: Removed.
+        * platform/network/ResourceHandleInternal.h:
+        * platform/network/qt/ResourceHandleManagerQt.cpp:
+        (WebCore::RequestQt::RequestQt):
+        (WebCore::ResourceHandleManager::add):
+        * platform/network/qt/ResourceHandleManagerQt.h:
+        * platform/network/qt/ResourceHandleQt.cpp:
+        (WebCore::ResourceHandle::start):
+
+2007-01-26  Zack Rusin  <zack@kde.org>
+
+        Fix the compile
+
+        * WebCore.pro:
+
+2007-01-26  Adam Roben  <aroben@apple.com>
+
+        Reviewed by Maciej.
+
+        Fixed <rdar://problem/4946499>, which made it impossible to tab
+        forward into the left frame of tivofaq.com after tabbing out of the
+        document.
+
+        * page/FocusController.cpp:
+        (WebCore::FocusController::advanceFocus): Clear the focused frame
+        before passing off focus to Chrome so that when we re-enter the
+        document we start at the main frame instead of the last focused frame.
+
+2007-01-25  Rob Buis  <buis@kde.org>
+
+        Reviewed by olliej.
+
+        http://bugs.webkit.org/show_bug.cgi?id=11911
+        SVG exhibits double-text-selection behavior
+
+        Fix selection height of the inline text boxes for svg.
+
+        * WebCore.xcodeproj/project.pbxproj:
+        * rendering/InlineTextBox.cpp:
+        (WebCore::InlineTextBox::selectionTop):
+        (WebCore::InlineTextBox::selectionHeight):
+        (WebCore::InlineTextBox::selectionRect):
+        (WebCore::InlineTextBox::paintSelection):
+        (WebCore::InlineTextBox::paintMarkedTextBackground):
+        (WebCore::InlineTextBox::paintCustomHighlight):
+        (WebCore::InlineTextBox::paintSpellingOrGrammarMarker):
+        (WebCore::InlineTextBox::paintTextMatchMarker):
+        * rendering/InlineTextBox.h:
+        * rendering/RenderSVGInlineText.cpp:
+        (WebCore::RenderSVGInlineText::createInlineTextBox):
+        * rendering/RenderSVGInlineText.h:
+        * rendering/RenderText.cpp:
+        (WebCore::RenderText::createInlineTextBox):
+        (WebCore::RenderText::createInlineBox):
+        * rendering/RenderText.h:
+        * rendering/SVGInlineTextBox.cpp: Added.
+        (WebCore::SVGInlineTextBox::selectionTop):
+        (WebCore::SVGInlineTextBox::selectionHeight):
+        * rendering/SVGInlineTextBox.h: Added.
+        (WebCore::SVGInlineTextBox::SVGInlineTextBox):
+
+2007-01-26  Zack Rusin  <zack@kde.org>
+
+        Reviewed by Lars
+
+        Fix drawing of tiled images.
+
+        * WebCore.pro:
+        * platform/graphics/qt/ImageQt.cpp:
+        (WebCore::BitmapImage::drawPattern):
+
+2007-01-26  Zack Rusin  <zack@kde.org>
+
+        Reviewed by Lars
+
+        Fixing image loading - we decode QImages
+        but render QPixmaps now. Implement Icon
+        class
+
+        * platform/graphics/BitmapImage.h:
+        * platform/graphics/Icon.h:
+        * platform/graphics/Image.h:
+        (WebCore::Image::getPixmap):
+        * platform/graphics/qt/IconQt.cpp:
+        (WebCore::Icon::Icon):
+        (WebCore::Icon::~Icon):
+        (WebCore::Icon::newIconForFile):
+        (WebCore::Icon::paint):
+        * platform/graphics/qt/ImageDecoderQt.cpp:
+        (WebCore::ImageDecoderQt::ImageData::ImageData):
+        (WebCore::ImageDecoderQt::ReadContext::read):
+        (WebCore::ImageDecoderQt::ReadContext::readImageLines):
+        (WebCore::ImageDecoderQt::reset):
+        (WebCore::ImageDecoderQt::imageAtIndex):
+        * platform/graphics/qt/ImageDecoderQt.h:
+        * platform/graphics/qt/ImageQt.cpp:
+        (WebCore::BitmapImage::drawPattern):
+        (WebCore::BitmapImage::getPixmap):
+        * platform/graphics/qt/ImageSourceQt.cpp:
+        (WebCore::ImageSource::createFrameAtIndex):
+
+2007-01-25  Justin Garcia  <justin.garcia@apple.com>
+
+        Reviewed by oliver
+        
+        <http://bugs.webkit.org/show_bug.cgi?id=12409>
+        REGRESSION (r19039): Drag caret isn't cleared after a drop
+
+        No new layout tests because the regression was
+        caught by current layout tests.
+
+        * page/DragController.cpp:
+        (WebCore::DragController::concludeDrag): Clear the drag caret.
+
+2007-01-25  Darin Adler  <darin@apple.com>
+
+        Reviewed and landed by Geoffrey Garen.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=12342
+          REGRESSION: destroying a frame from its own script causes various crashes
+
+        * bindings/js/kjs_window.cpp: (KJS::ScheduledAction::execute):
+        Use RefPtr for the frame and the interpreter object so they don't get deleted
+        out from underneath us.
+
+        * bindings/js/kjs_proxy.cpp: (WebCore::KJSProxy::evaluate):
+        Add a check for a page of 0. This was the only call site for the
+        addMessageToConsole function that did not have a check for a page of 0.
+
+2007-01-25 Dirk Mueller  <mueller@kde.org>
+
+        Reviewed and merged by Beth.
+
+        Fix for <rdar://problem/4921692> not processing comments inside a 
+        title element leads to XSS (for at least BlogSpot)
+
+        This is equivalent to commit 626791 in the KDE tree. Basically, the 
+        patch allows comment parsing in titles to avoid capturing a 
+        <script> tag by accident.
+
+        * html/HTMLTokenizer.cpp:
+        (WebCore::HTMLTokenizer::parseSpecial):
+        (WebCore::HTMLTokenizer::parseComment):
+
+2007-01-25  Darin Adler  <darin@apple.com>
+
+        * ksvg2/svg/SVGPreserveAspectRatio.cpp: Add a couple of comments.
+
+2007-01-25  Darin Adler  <darin@apple.com>
+
+        Reviewed by Maciej.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=12365
+          Reproducible crash in WebCore::SVGPreserveAspectRatio::parsePreserveAspectRatio
+          in svg/W3C-SVG-1.1/animate-elem-40-t.svg under guard malloc
+        - fix bug where the parser accepts "meet" or "slice" as an entire string
+        - fix bug where "slice" is ignored if the old alignment was "none"
+          even if the newly-parsed alignment is something other than "none", and
+          "slice" is not ignored if the old alignment was not "none", but the
+          newly-parsed alignment is "none"
+        - optimize to only call notifyAttributeChange if the attribute actually changed
+
+        Test: svg/dom/preserve-aspect-ratio-parser-test.html
+
+        * ksvg2/svg/SVGPreserveAspectRatio.cpp:
+        (WebCore::checkString): Added an overload for using char* constants which is
+        much simpler than the old one, and almost as fast.
+        (WebCore::SVGPreserveAspectRatio::parsePreserveAspectRatio): Fail if there's no
+        more text after "defer". Fail if there's neither "none" nor an x/y specified.
+        Check that there's more text before looking at the "m" and "s" for "meet" and
+        "slice". Merged the failure and success cases. Only call notifyAttributeChange
+        if something changed.
+
+        I think all the "optimized" code where we check the first character before
+        calling checkString isn't really helpful, but I decided to leave it as-is
+        for now.
+
+2007-01-25  Lars Knoll <lars@trolltech.com>
+
+        Reviewed by Zack
+
+        Trivial implementation of getMIMETypeForExtension.
+        Use the above method in the ResourceHandleManager to set
+        a mimetype on Resources that don't have one set (which
+        includes everything loaded from local disk).
+
+        Should fix quite a few test cases :)
+
+        * platform/network/qt/ResourceHandleManagerQt.cpp:
+        (WebCore::ResourceHandleManager::receivedResponse):
+        * platform/qt/MimeTypeRegistryQt.cpp:
+        (WebCore::):
+        (WebCore::MimeTypeRegistry::getMIMETypeForExtension):
+
+2007-01-25 Harald Fernengel <harald@trolltech.com>
+
+        Reviewed by zack
+
+        Compile on platforms where qreal is not a double.
+
+        * platform/graphics/qt/AffineTransformQt.cpp:
+        (WebCore::AffineTransform::map):
+
+2007-01-25  Zack Rusin  <zack@kde.org>
+
+        Reviewed by Lars.
+
+        Add wheel event handling and make explicit usage
+        of the EventHandler.
+
+        * WebCore.pro:
+        * page/qt/EventHandlerQt.cpp:
+        (WebCore::EventHandler::focusDocumentView):
+        (WebCore::EventHandler::passWidgetMouseDownEventToWidget):
+        (WebCore::EventHandler::passMouseDownEventToWidget):
+        (WebCore::EventHandler::lastEventIsMouseUp):
+        (WebCore::EventHandler::handleMouseUp):
+        * platform/PlatformWheelEvent.h:
+        * platform/qt/ScrollViewCanvasQt.cpp:
+        (WebCore::ScrollViewCanvasQt::ScrollViewCanvasQt):
+        (WebCore::ScrollViewCanvasQt::mousePressEvent):
+        (WebCore::ScrollViewCanvasQt::wheelEvent):
+        (WebCore::ScrollViewCanvasQt::dragEnterEvent):
+        (WebCore::ScrollViewCanvasQt::dragLeaveEvent):
+        (WebCore::ScrollViewCanvasQt::dragMoveEvent):
+        * platform/qt/ScrollViewCanvasQt.h:
+        * platform/qt/ScrollViewQt.cpp:
+        (WebCore::ScrollView::wheelEvent):
+        * platform/qt/WheelEventQt.cpp: Added.
+        (WebCore::PlatformWheelEvent::PlatformWheelEvent):
+
+2007-01-25  Rob Buis  <buis@kde.org>
+
+        Reviewed by Maciej.
+
+        http://bugs.webkit.org/show_bug.cgi?id=12391
+        SVG containers with viewports should respect overflow when hit testing
+
+        Take into account the overflow property when hit testing <svg>.
+
+        * rendering/RenderSVGContainer.cpp:
+        (WebCore::RenderSVGContainer::nodeAtPoint):
+
+2007-01-23  Alice Liu  <alice.liu@apple.com>
+
+        Reviewed by Maciej.
+
+        fixed <rdar://problem/4808039> REGRESSION: eBay ID check never completes
+        Also known as http://bugs.webkit.org/show_bug.cgi?id=11230
+
+        * bindings/js/kjs_window.cpp:
+        It's not necessary to check safeScript when doing a location.replace, .reload, or .assign
+        (KJS::Location::getOwnPropertySlot):
+        (KJS::LocationFunc::callAsFunction):
+
+2007-01-25  Alice Liu  <alice.liu@apple.com>
+
+        Reviewed by Geoff and Adele.
+
+        fixed <rdar://problem/4866655> REGRESSION: Copying out of Calculator & CIFIlterWidget is broken on 9A320
+
+        * editing/Editor.cpp:
+        (WebCore::Editor::canDHTMLCut):
+        (WebCore::Editor::canDHTMLCopy):
+        (WebCore::Editor::tryDHTMLCopy):
+        (WebCore::Editor::tryDHTMLCut):
+        Reverting some mistakes made in r17504:
+        Anything in r17503 that used to call mayDHTML* still calls canDHTML* as it was changed in r17504, where canDHTML* now calls !selectionController::isInPasswordField instead of editor::canCopy
+        Anything else in r17503 that used to call mayCopy now calls (!)selectionController::isInPasswordField.
+
+        * editing/JSEditor.cpp:
+        copy via JS should be enabled as long as selection is not in a password field.  a range selection shouldn't be required. 
+
+2007-01-24  Adele Peterson  <adele@apple.com>
+
+        Reviewed by Brady and Oliver.
+
+        Fix for <rdar://problem/4953504> When the keydown event changes focus, the keypress event should go to the new focused node
+
+        Test: fast/events/keydown-keypress-focus-change.html
+
+        * page/EventHandler.cpp:
+        (WebCore::eventTargetNodeForDocument): Added for convenience.
+        (WebCore::EventHandler::keyEvent): Moved from EventHandlerMac.mm.  After dispatching the keyDown event,
+         recheck the target node for the keyPress event in case its changed.
+        * page/EventHandler.h:
+        * page/mac/EventHandlerMac.mm: (WebCore::EventHandler::keyEvent):
+          Moved the bulk of this code to EventHandler.cpp.
+
+2007-01-24  Geoffrey Garen  <ggaren@apple.com>
+
+        New forwarding header. No review necessary.
+
+        * ForwardingHeaders/kjs/ustring.h: Added.
+
+2007-01-24  Oliver Hunt  <oliver@apple.com>
+
+        Reviewed by Maciej.
+
+        Migrating more drag state information to WebCore
+
+        * page/DragController.cpp:
+        * page/DragController.h:
+        (WebCore::DragController::dragOperation):
+        (WebCore::DragController::setDraggingImageURL):
+        (WebCore::DragController::draggingImageURL):
+        (WebCore::DragController::setDragOffset):
+        (WebCore::DragController::dragOffset):
+
+2007-01-24  Darin Adler  <darin@apple.com>
+
+        Reviewed by Adele.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=12331
+          REGRESSION(r18951): Selecting text in a textarea makes the page scroll
+
+        Test: fast/events/autoscroll.html
+
+        * rendering/RenderLayer.cpp: (WebCore::RenderLayer::autoscroll):
+        Adjust position to be relative to the object's corner for the hit test, but not for the
+        scrollRectToVisible call. The previous version accidentally re-used the version that was
+        adjusted for the hit test.
+
+2007-01-24  Beth Dakin  <bdakin@apple.com>
+
+        Reviewed by Geoff.
+
+        Fix for <rdar://problem/4948128> REGRESSION: A crash occurs at 
+        WebCore::Frame::view() after completing a query at mapquest.com 
+        then switching to Bookmarks view
+
+        This is just a null-check. If you are in strict mode and ask for 
+        the clientWidth or clientHeight after the document was removed from 
+        it's parent, we no longer have a frameView, so we have to null-
+        check it. I added null checks for the FrameView in some other 
+        places where it seems like we could potentially run into this 
+        issue.
+
+        * bindings/objc/DOMHTML.mm:
+        (-[DOMHTMLInputElement _rectOnScreen]):
+        * bridge/mac/WebCoreAXObject.mm:
+        (-[WebCoreAXObject accessibilityAttributeValue:]):
+        * dom/ContainerNode.cpp:
+        (WebCore::ContainerNode::getUpperLeftCorner):
+        * dom/Element.cpp:
+        (WebCore::Element::clientWidth): Actual bug fix here!
+        (WebCore::Element::clientHeight): And here!
+        * ksvg2/svg/SVGLength.cpp:
+        (WebCore::SVGLength::PercentageOfViewport):
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::createJavaAppletWidget):
+        * page/mac/WebCoreFrameBridge.mm:
+        (-[WebCoreFrameBridge computePageRectsWithPrintWidthScaleFactor:printHeight:]):
+        * rendering/RenderLayer.cpp:
+        (WebCore::RenderLayer::windowClipRect):
+        (WebCore::RenderLayer::updateOverflowStatus):
+        (WebCore::frameVisibleRect):
+        * rendering/RenderListBox.cpp:
+        (WebCore::RenderListBox::windowClipRect):
+        * rendering/RenderTable.cpp:
+        (WebCore::RenderTable::layout):
+
+2007-01-24  Maciej Stachowiak  <mjs@apple.com>
+
+        Not reviewed, just removing accidentally committed file.
+
+        - removed file that shouldn't be there
+
+        * Image Viewer/Image Viewer.aps: Removed.
+
+2007-01-24  Anders Carlsson  <acarlsson@apple.com>
+
+        Reviewed by Geoff and Adam.
+
+        <rdar://problem/4922655>
+        http://galeri.milliyet.com.tr/ reproducibly crashes Safari
+        
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::requestFrame):
+        If the frame already exists, defer the navigation so that we won't end up replacing the document
+        while we're parsing it. This is similar to what setting window.location ends up doing.
+
+2007-01-25  Mark Rowe  <mrowe@apple.com>
+
+        Reviewed by Darin.
+
+        * Info.plist: Update copyright string.
+
+2007-01-24  Justin Garcia  <justin.garcia@apple.com>
+
+        Reviewed by harrison
+
+        <rdar://problem/4916235>
+        eBay Editor: Crash occurs at WebCore::Range::startPosition() after applying ordered list styling to selected text
+        <rdar://problem/4857269>
+        REGRESSION: editing/execCommand/create-list-from-range-selection.html no longer works as intended
+        <rdar://problem/4916583>
+        eBay Editor: Removing a indent from a list item actually deletes the list item instead
+        <rdar://problem/4928635>
+        designMode outdent causes crash (11750)
+
+        * editing/CompositeEditCommand.cpp:
+        (WebCore::CompositeEditCommand::moveParagraphs): Fixed br insertion
+        after the pruning of empty containers left over after the paragraph
+        to move is removed.
+        * editing/DeleteSelectionCommand.cpp:
+        (WebCore::DeleteSelectionCommand::DeleteSelectionCommand): There are
+        bugs that leave the wrong number of line breaks after a deletion 
+        that expands to include special elements (lists/tables).  Added 
+        a workaround so that moveParagraphs can avoid these until they 
+        are fixed.
+        (WebCore::DeleteSelectionCommand::initializeStartEnd): Ditto.
+        (WebCore::DeleteSelectionCommand::initializePositionData): Ditto.
+        * editing/DeleteSelectionCommand.h:
+        * editing/TextIterator.cpp:
+        (WebCore::TextIterator::advance): We used to not call exitNode for
+        ancestors of a node where the next node in a pre-order traversal is 
+        the ranges's pastEndNode().  This is only correct for ancestors that
+        contain the end of the range.
+        (WebCore::shouldEmitNewlineAfterNode): Added, we should only emit newlines
+        after blocks that have VisiblePositions after them.  VisiblePosition creation
+        though is too expensive to do here, what I added here is an OK substitute.
+        (WebCore::shouldEmitNewlineBeforeNode): Added.  No special logic yet.
+        (WebCore::TextIterator::handleNonTextNode): Call shouldEmitNewlineBeforeNode.
+        (WebCore::TextIterator::exitNode): Call shouldEmitNewlineAfterNode.
+        (WebCore::SimplifiedBackwardsTextIterator::handleNonTextNode): Call
+        shouldEmitNewlineAfterNode.
+
+2007-01-24  Darin Adler  <darin@apple.com>
+
+        Reviewed by Mark Rowe.
+
+        * WebCore.xcodeproj/project.pbxproj: Changed to /usr/sbin/sysctl
+        so we don't rely on people's paths.
+
+2007-01-24  Darin Adler  <darin@apple.com>
+
+        Reviewed by Adele.
+
+        - a couple minor tweaks to recently-changed things
+
+        * dom/Document.h: Use HTMLHeadElement as type for the head function.
+        * dom/Document.cpp: (WebCore::Document::head):
+
+        * ksvg2/svg/SVGElementInstance.cpp:
+        (WebCore::SVGElementInstance::~SVGElementInstance): Change loop to
+        a for loop and use a local variable for document.
+
 2007-01-24  Oliver Hunt  <oliver@apple.com>
 
         Reviewed by Adele.
 
         Fixes DragController to allow a null target Document.
         This is encountered in new tabs/windows that have not 
-        loaded any page (including about:blank)
+        loaded any page
 
         * page/DragController.cpp:
         (WebCore::documentAtPoint):