WebKit-https.git
14 years agoGdk build fixes using the Qt build's qmake tool from Holger Freyther, reviewed by...
hausmann [Thu, 3 May 2007 22:04:27 +0000 (22:04 +0000)]
Gdk build fixes using the Qt build's qmake tool from Holger Freyther, reviewed by Zack.

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

14 years ago Reviewed by Kevin.
thatcher [Thu, 3 May 2007 21:50:56 +0000 (21:50 +0000)]
    Reviewed by Kevin.

        <rdar://problem/5067707> REGRESSION: "Open Link" contextual menu item appears twice in Mail

        Remove the check for Mail in fixMenusToSendToOldClients and fixMenusReceivedFromOldClients
        when linked on or after Leopard. The isAppleMail() function is still used for Tiger Mail fixups.

        * WebCoreSupport/WebContextMenuClient.mm:
        (fixMenusToSendToOldClients):
        (fixMenusReceivedFromOldClients):

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

14 years ago Use BUILDING_ON_TIGER.
adele [Thu, 3 May 2007 21:37:16 +0000 (21:37 +0000)]
    Use BUILDING_ON_TIGER.

        * page/mac/FrameMac.mm: (WebCore::Frame::setUseSecureKeyboardEntry):

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

14 years ago Reviewed by Darin
beidson [Thu, 3 May 2007 21:35:24 +0000 (21:35 +0000)]
    Reviewed by Darin

        Some more minor cleanup that resolves a FIXME and ditches another pointless method

        No change in functionality == no layout test

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::commitProvisionalLoad):  Updated a comment
        (WebCore::FrameLoader::continueLoadAfterWillSubmitForm): Folded startLoading() into this method
        (WebCore::FrameLoader::continueLoadAfterNavigationPolicy): continueAfterWillSubmitForm becomes continueLoadAfterWillSubmitForm
        * loader/FrameLoader.h: Removed startLoading()

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

14 years ago Reviewed by Tim Hatcher.
adele [Thu, 3 May 2007 21:29:41 +0000 (21:29 +0000)]
    Reviewed by Tim Hatcher.

        Fix for <rdar://problem/4727607> REGRESSION: KeyScript is deprecated, need a new solution for forcing Roman input for password fields on Leopard

        * page/mac/FrameMac.mm: (WebCore::Frame::setUseSecureKeyboardEntry):
        Use TSMSetDocumentProperty and TSMRemoveDocumentProperty on Leopard disable and enable non-ascii input sources.

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

14 years ago Reviewed by Anders
beidson [Thu, 3 May 2007 18:50:41 +0000 (18:50 +0000)]
    Reviewed by Anders

        commitProvisionalLoad() was the name of two methods.
        One of them was private and only called from one other private place.
        This was driving me crazy.  I had to change that.

        No change in functionality == no layout test

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::transitionToCommitted): roll commitProvisionalLoad() into this method
        * loader/FrameLoader.h: Removed commitProvisionalLoad()

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

14 years ago Build fix: Add AllInOneFile.o to the ignore list, since it includes files
ggaren [Thu, 3 May 2007 18:37:57 +0000 (18:37 +0000)]
    Build fix: Add AllInOneFile.o to the ignore list, since it includes files
        in the ignore list.

        * Scripts/check-for-global-initializers:

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

14 years ago Reviewed by Anders.
darin [Thu, 3 May 2007 16:23:53 +0000 (16:23 +0000)]
    Reviewed by Anders.

        - make ICU initialization lazy to speed up application start-up time
          (when it includes an initial webpage load)

        * platform/TextCodecICU.h: Replaced registerEncodingNames and
        registerCodecs with registerBaseEncodingNames, registerBaseCodecs,
        registerExtendedEncodingNames, and registerExtendedCodecs.
        * platform/TextCodecICU.cpp:
        (WebCore::newTextCodecICU): Moved up to the top of the file.
        (WebCore::TextCodecICU::registerBaseEncodingNames): Added. Registers
        just UTF-8.
        (WebCore::TextCodecICU::registerBaseCodecs): Ditto.
        (WebCore::TextCodecICU::registerExtendedEncodingNames): Renamed.
        (WebCore::TextCodecICU::registerExtendedCodecs): Ditto.
        (WebCore::TextCodecICU::createICUConverter): Compare with the name
        "GBK" in a way that does not require extending the maps.

        * platform/TextEncoding.cpp:
        (WebCore::TextEncoding::usesVisualOrdering): Added code so this will
        just return false if we haven't used any extended encoding names yet.
        The result will be correct, and we won't ask for an encoding name that
        will require registering the extended encoding names.
        (WebCore::TextEncoding::isJapanese): Ditto.
        (WebCore::TextEncoding::backslashAsCurrencySymbol): Ditto, but return
        a backslash instead of false.

        * platform/TextEncodingRegistry.h: Added a new function named
         noExtendedTextEncodingNameUsed.
        * platform/TextEncodingRegistry.cpp:
        (WebCore::buildBaseTextCodecMaps): Added. Registers the base encodings,
        including Latin-1 because it's the default, and the UTF encodings.
        (WebCore::extendTextCodecMaps): Added. Registers all the other encodings.
        (WebCore::newTextCodec): Removed the all to buildTextCodecMap because
        we always build the map when making the canonical encoding name.
        (WebCore::atomicCanonicalTextEncodingName): Changed code to build only
        the base maps at first, then extend the maps the first time we run into
        a name we don't know.
        (WebCore::noExtendedTextEncodingNameUsed): Added.

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

14 years ago2007-05-03 Mark Rowe <mrowe@apple.com>
bdash [Thu, 3 May 2007 04:39:37 +0000 (04:39 +0000)]
2007-05-03  Mark Rowe  <mrowe@apple.com>

        Qt build fix.  In r21185 the Qt version of WebCore::scaleDragImage was
        updated to have the incorrect signature.

        * platform/qt/DragImageQt.cpp:
        (WebCore::scaleDragImage):

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

14 years ago * dom/Node.h: Just fixing a variable name because it's driving me
bdakin [Wed, 2 May 2007 22:27:56 +0000 (22:27 +0000)]
    * dom/Node.h: Just fixing a variable name because it's driving me
        mad.

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

14 years ago Reviewed by Hyatt.
bdakin [Wed, 2 May 2007 22:16:37 +0000 (22:16 +0000)]
    Reviewed by Hyatt.

        Fix for <rdar://problem/4513966> Never-ending identical repaint @
        orbitz.com (Safari eating 85% CPU)

        We need to be a little bit more savvy about when we do a FULL style
        recalc since it can be unnecessary and expensive.

        * css/CSSMutableStyleDeclaration.cpp:
        (WebCore::CSSMutableStyleDeclaration::setChanged): Now takes a
        StyleChangeType as a parameter.
        (WebCore::CSSMutableStyleDeclaration::setProperty): Send
        InlineStyleChange as the paramter. This is the optimization!
        * css/CSSMutableStyleDeclaration.h: setChanged takes a
        StyleChangeType as a parameter.
        * dom/Document.cpp: setChanged now takes a StyleChangeType instead
        of a bool.
        (WebCore::Document::recalcStyle): Same.
        * dom/Element.cpp:
        (WebCore::Element::recalcStyle): This is the fix!! Only set change
        to Force is we use descendant rules AND it's a FullStyleChange.
        * dom/Node.cpp: m_hasStyle was an unused bit on Node, so this patch
        gets rid of it and uses that extra bit for m_changed to be a
        StyleChangeType instead of a bool.
        (WebCore::Node::Node):
        (WebCore::Node::setChanged):
        (WebCore::Node::dump):
        * dom/Node.h: Same.
        (WebCore::): Define the StyleChangeType enum.
        (WebCore::Node::hasClass):
        (WebCore::Node::changed):
        (WebCore::Node::styleChangeType):
        (WebCore::Node::setHasClass):
        * dom/StyledElement.cpp:
        (WebCore::StyledElement::parseMappedAttribute): Don't bother
        calling setHasStyle() since m_hasStyle is gone now. It was never
        used, and this was the only place it was set.
        * dom/Text.cpp:
        (WebCore::Text::recalcStyle): setChanged now takes a
        StyleChangeType instead of a bool.
        * html/HTMLFrameSetElement.cpp:
        (WebCore::HTMLFrameSetElement::recalcStyle): Same.
        * html/HTMLSelectElement.cpp:
        (WebCore::HTMLSelectElement::restoreState): Same.
        (WebCore::HTMLSelectElement::reset): Same.
        * html/HTMLTextAreaElement.cpp:
        (WebCore::HTMLTextAreaElement::setValue): Same.
        * ksvg2/misc/SVGTimer.cpp:
        (WebCore::SVGTimer::applyAnimations): Same.
        * ksvg2/svg/SVGUseElement.cpp:
        (WebCore::SVGUseElement::recalcStyle): Same.
        * platform/graphics/svg/SVGResource.cpp:
        (WebCore::SVGResource::repaintClients): Same.

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

14 years ago Reviewed by Brady.
andersca [Wed, 2 May 2007 19:36:22 +0000 (19:36 +0000)]
    Reviewed by Brady.

        <rdar://problem/5151113> Assertion firing in [FrameProgressEntry addChild:forDataSource:] when navigating cnn.com

        The assertion fired because a plug-in was trying to load a subresource when a new load had started but not yet
        committed. The check that would have prevented this was removed in order to fix <rdar://problem/5085897>.

        This puts back the check but changes it to allow loads where the target is the same frame as the plugin's parent frame.

        * Plugins/WebBaseNetscapePluginView.mm:
        (-[WebBaseNetscapePluginView loadRequest:inTarget:withNotifyData:sendNotification:]):

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

14 years ago Reviewed by Antti.
harrison [Wed, 2 May 2007 14:54:20 +0000 (14:54 +0000)]
    Reviewed by Antti.

        <rdar://problem/4859132> Grammar must always be checked in entire-sentence chunks, and shouldn't show markers for current sentence

        Restore the remainder of the original fix now that blocking bug rdar://5174862 is addressed.

        John made the original patch on 2007-04-10, but this fell prey to rdar://5157329.
        Justin backed out the whole patch on 2007-04-30 to avoid the crash.
        Justin then restored most of the original patch, but used word granularity instead, to keep the build working.
        I just addressed the root cause of rdar://5157329 in rdar://5174862.
        This patch switches the temp uses of word granularity back to sentence granularity.

        * editing/mac/EditorMac.mm:
        (WebCore::Editor::markMisspellingsAfterTypingToPosition):
        * page/Frame.cpp:
        (WebCore::Frame::respondToChangedSelection):

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

14 years agoJavaScriptCore:
harrison [Wed, 2 May 2007 13:11:05 +0000 (13:11 +0000)]
JavaScriptCore:

        Reviewed by Antti.

        <rdar://problem/5174862> Crash resulting from DeprecatedString::insert()

        Added insertion support for more than one value.

        * wtf/Vector.h:
        (WTF::::insert):
        Added support for inserting multiple values.

        (WTF::::prepend):
        New. Insert at the start of vectors. Convenient for vectors used as strings.

LayoutTests:

        Reviewed by Antti.

        - test for <rdar://problem/5174862> Crash resulting from DeprecatedString::insert()

        * editing/selection/move-by-sentence-001-expected.checksum: Added.
        * editing/selection/move-by-sentence-001-expected.png: Added.
        * editing/selection/move-by-sentence-001-expected.txt: Added.
        * editing/selection/move-by-sentence-001.html: Added.

WebCore:

        Reviewed by Antti.

        <rdar://problem/5174862> Crash resulting from DeprecatedString::insert()

        Test added: editing/selection/move-by-sentence-001.html

        * editing/visible_units.cpp:
        (WebCore::previousBoundary):
        (WebCore::nextBoundary):
        Use a UChar Vector instead of DeprecatedString.
        Avoid creating an extra string for secure bullet replacement unless it is actually needed.

        * platform/DeprecatedString.cpp:
        (WebCore::DeprecatedString::insert):
        Call forceUnicode() before setLength(), so that only the unicode buffer is resized.

       (WebCore::DeprecatedString::setLength):
        Adjust the unicode buffer even if the ascii buffer is valid.

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

14 years ago2007-05-02 Holger Freyther <freyther@kde.org>
bdash [Wed, 2 May 2007 09:21:34 +0000 (09:21 +0000)]
2007-05-02  Holger Freyther <freyther@kde.org>

        Reviewed by Mark Rowe.

        * Scripts/build-webkit: Remove the CMake call.
        * Scripts/webkitdirs.pm: Remove isQtWithQMake as QMake is now the only
        buildsystem for the Qt port.

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

14 years ago2007-05-02 Mitz Pettel <mitz@webkit.org>
bdash [Wed, 2 May 2007 09:20:00 +0000 (09:20 +0000)]
2007-05-02  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Brady.

        - http://bugs.webkit.org/show_bug.cgi?id=13564
          Remove friend class FrameLoader from Frame

        No test added since there is no change in functionality.

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::didOpenURL):
        (WebCore::FrameLoader::begin):
        (WebCore::FrameLoader::open):
        * loader/FrameLoader.h: Made clear() private.
        * page/Frame.cpp:
        (WebCore::Frame::setDocument): Changed the argument into a PassRefPtr.
        * page/Frame.h:
        * xml/XSLTProcessor.cpp:
        (WebCore::XSLTProcessor::createDocumentFromSource):

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

14 years ago2007-05-02 Mitz Pettel <mitz@webkit.org>
bdash [Wed, 2 May 2007 09:17:52 +0000 (09:17 +0000)]
2007-05-02  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Dave Hyatt.

        - fix http://bugs.webkit.org/show_bug.cgi?id=9276
          Quirksmode: Image swap resize bug

        Test: fast/replaced/image-resize-width.html

        * rendering/RenderImage.cpp:
        (WebCore::RenderImage::imageChanged): Moved the call to dirtyPrefWidths()
        before the call to calcWidth().

2007-05-02  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Dave Hyatt.

        - test for http://bugs.webkit.org/show_bug.cgi?id=9276
          Quirksmode: Image swap resize bug

        * fast/replaced/image-resize-width-expected.checksum: Added.
        * fast/replaced/image-resize-width-expected.png: Added.
        * fast/replaced/image-resize-width-expected.txt: Added.
        * fast/replaced/image-resize-width.html: Added.
        * fast/replaced/resources/rectangle-red-200x100.png: Added.

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

14 years agoAdd layout test for dashboard bug.
hyatt [Wed, 2 May 2007 08:44:02 +0000 (08:44 +0000)]
Add layout test for dashboard bug.

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

14 years ago Fix for bug 13566, broken Dashboard widgets. setPrefWidthsDirty did
hyatt [Wed, 2 May 2007 08:39:41 +0000 (08:39 +0000)]
    Fix for bug 13566, broken Dashboard widgets.  setPrefWidthsDirty did
        the wrong thing for text children of a positioned element.

        Reviewed by mitz

        fast/block/positioning/pref-width-change.html

        * rendering/RenderObject.cpp:
        (WebCore::RenderObject::setPrefWidthsDirty):

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

14 years ago * rendering/RenderReplaced.cpp: Fix typo in a comment.
darin [Wed, 2 May 2007 01:40:31 +0000 (01:40 +0000)]
    * rendering/RenderReplaced.cpp: Fix typo in a comment.

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

14 years agoAdd layout test for 13558.
hyatt [Tue, 1 May 2007 22:31:59 +0000 (22:31 +0000)]
Add layout test for 13558.

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

14 years ago Fix for 13558, crash when using absolutely positioned generated content
hyatt [Tue, 1 May 2007 22:27:03 +0000 (22:27 +0000)]
    Fix for 13558, crash when using absolutely positioned generated content
        inside an inline.

        Reviewed by darin

        fast/css-generated-content/absolute-position-inside-inline.html

        * rendering/RenderContainer.cpp:
        (WebCore::RenderContainer::updateBeforeAfterContentForContainer):

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

14 years agoLayoutTests:
justing [Tue, 1 May 2007 22:20:46 +0000 (22:20 +0000)]
LayoutTests:

        Reviewed by darin

        <rdar://problem/5002441>
        Pressing space key does nothing above quoted content or a signature

        Demonstrates the bug:
        * editing/inserting/5002441-expected.checksum: Added.
        * editing/inserting/5002441-expected.png: Added.
        * editing/inserting/5002441-expected.txt: Added.
        * editing/inserting/5002441.html: Added.

        Fixed.  Spaces passed to execCommand("InsertText", ...)
        are no longer collapsed:
        * editing/inserting/editable-html-element-expected.checksum:
        * editing/inserting/editable-html-element-expected.png:
        * editing/inserting/editable-html-element-expected.txt:
        * editing/pasteboard/4989774-expected.checksum:
        * editing/pasteboard/4989774-expected.png:
        * editing/pasteboard/4989774-expected.txt:
        * editing/selection/4983858-expected.checksum:
        * editing/selection/4983858-expected.png:
        * editing/selection/4983858-expected.txt:

WebCore:

        Reviewed by darin

        <rdar://problem/5002441>
        Pressing space key does nothing above quoted content or a signature

        Inserting a space under these circumstances inserts a single
        text node containing a regular space and then does a layout.
        That space isn't rendered (which is correct).  Whitespace
        rebalancing is supposed to correct it but failed.  It replaces
        the space with a non-breaking space, but that change doesn't
        dirty line boxes (9441) and so the space isn't rendered.

        This workaround turns all incoming spaces into non-breaking
        spaces before they're inserted (they're rebalanced after
        insertion and turned back into regular spaces if possible).

        * editing/InsertTextCommand.cpp:
        (WebCore::InsertTextCommand::prepareForTextInsertion): Removed
        an old irrelevant FIXME.
        (WebCore::InsertTextCommand::input): Turn incoming spaces into
        non breaking spaces before inserting them.

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

14 years agoLayoutTests:
darin [Tue, 1 May 2007 22:10:41 +0000 (22:10 +0000)]
LayoutTests:

        Reviewed by Hyatt.

        - test for http://bugs.webkit.org/show_bug.cgi?id=13559
          REGRESSION: Canvas aspect ratio is incorrect

        * fast/canvas/canvas-size-change-after-layout-expected.checksum: Added.
        * fast/canvas/canvas-size-change-after-layout-expected.png: Added.
        * fast/canvas/canvas-size-change-after-layout-expected.txt: Added.
        * fast/canvas/canvas-size-change-after-layout.html: Added.

WebCore:

        Reviewed by Hyatt.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13559
          REGRESSION: Canvas aspect ratio is incorrect

        - remove intrinsicWidth/Height functions from RenderObject

        Test: fast/canvas/canvas-size-change-after-layout.html

        * html/HTMLCanvasElement.cpp:
        (WebCore::HTMLCanvasElement::createRenderer): Removed code to set
        the intrinsic size, because the renderer now takes care of this
        in its constructor.
        (WebCore::HTMLCanvasElement::reset): Instead of calling the
        setIntrinsicWidth/Height functions, call canvasSizeChanged, which
        takes care of layout, but only if the size actually changed. Also
        call repaint if the contents of the canvas changed.

        * rendering/RenderApplet.h:
        * rendering/RenderApplet.cpp: (WebCore::RenderApplet::intrinsicSize):
        Override the new intrinsicSize function. This override still doesn't
        make sense, so left a FIXME in here.

        * rendering/RenderBox.h: Added an intrinsicSize function here to replace
        the intrinsicWidth/Height functions removed from RenderObject.
        * rendering/RenderBox.cpp:
        (WebCore::RenderBox::calcReplacedWidthUsing): Use intrinsicSize instead
        of intrinsicWidth.
        (WebCore::RenderBox::calcReplacedHeightUsing): Ditto, but for height.

        * rendering/RenderHTMLCanvas.h: Changed the constructor parameter to be
        an HTMLCanvasElement* instead of Node*. Also added a canvasSizeChanged
        function.
        * rendering/RenderHTMLCanvas.cpp:
        (WebCore::RenderHTMLCanvas::RenderHTMLCanvas): Pass an initial intrinsic
        size based on the current size of the canvas element. This is now the
        responsibility of the renderer and the DOM element calls the canvasSizeChanged
        function when the size changes.
        (WebCore::RenderHTMLCanvas::paint): Removed checks of the type of the element,
        and used the node() function because we don't have to worry about the anonymous
        case in this class.
        (WebCore::RenderHTMLCanvas::canvasSizeChanged): Added. Changes the size and
        then calls setNeedsLayout and setPrefWidthsDirty as needed. I based the logic
        here on careful reading of RenderImage::imageChanged.

        * rendering/RenderImage.cpp:
        (WebCore::RenderImage::RenderImage): Pass a default intrinsic size of 0,0
        instead of calling setIntrinsicWidth/Height.
        (WebCore::RenderImage::setImageSizeForAltText): Call the new intrinsic size
        functions instead of the width/height ones. Simplifies the logic since we
        handle both dimensions at once.
        (WebCore::RenderImage::imageChanged): Ditto.
        (WebCore::RenderImage::layout): Ditto.
        (WebCore::RenderImage::calcAspectRatioWidth): Ditto.
        (WebCore::RenderImage::calcAspectRatioHeight): Ditto.

        * rendering/RenderObject.h: Removed intrinsicWidth/Height virtual functions.

        * rendering/RenderReplaced.h: Replaced intrinsicWidth, intrinsicHeight,
        setIntrinsicWidth, setIntrinsicHeight, m_intrinsicWidth, and m_intrinsicHeight
        with intrinsicSize, setIntrinsicSize, and m_intrinsicSize.
        * rendering/RenderReplaced.cpp:
        (WebCore::RenderReplaced::RenderReplaced): Added a second contructor that takes
        an intrinsic size parameter. Also updated constructor since the intrinsic size
        is now an IntSize instead of two integers.
        (WebCore::RenderReplaced::intrinsicSize): Added. No point in making this inline
        because it's a virtual function.
        (WebCore::RenderReplaced::setIntrinsicSize): Added. Didn't make this inline,
        partly because in the future we might want this to be more than just a simple
        setter function, factoring in common code used in both RenderImage and
        RenderHTMLCanvas.

        * rendering/RenderWidget.h: Removed a stray using declaration that wasn't needed.

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

14 years ago Fix for hang when incorrectly trying to add before/after content to
hyatt [Tue, 1 May 2007 20:11:02 +0000 (20:11 +0000)]
    Fix for hang when incorrectly trying to add before/after content to
        a <select>.  Disallow generated content in all selects and also turn it
        off for <input> buttons.

        Reviewed by aroben

        * rendering/RenderBlock.cpp:
        (WebCore::RenderBlock::setStyle):
        * rendering/RenderButton.cpp:
        (WebCore::RenderButton::canHaveChildren):
        * rendering/RenderButton.h:

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

14 years agoLayoutTests:
darin [Tue, 1 May 2007 19:59:53 +0000 (19:59 +0000)]
LayoutTests:

        Reviewed by Hyatt.

        - added a test for <rdar://problem/5172596> HTML-parser-created tbody elements
          are being ignored when processing CSS child rules

        * fast/css/child-selector-implicit-tbody.html: Added.
        * fast/css/child-selector-implicit-tbody-expected.txt: Added.

        - updated test results for <rdar://problem/5172607> XML parser is creating
          implicit tbody elements; the only change is the tag name that's dumped

        * fast/table/037-expected.txt:
        * tables/mozilla/marvin/x_caption_align_bottom-expected.txt:
        * tables/mozilla/marvin/x_caption_align_top-expected.txt:
        * tables/mozilla/marvin/x_caption_class-expected.txt:
        * tables/mozilla/marvin/x_caption_id-expected.txt:
        * tables/mozilla/marvin/x_caption_style-expected.txt:
        * tables/mozilla/marvin/x_col_align_center-expected.txt:
        * tables/mozilla/marvin/x_col_align_char-expected.txt:
        * tables/mozilla/marvin/x_col_align_justify-expected.txt:
        * tables/mozilla/marvin/x_col_align_left-expected.txt:
        * tables/mozilla/marvin/x_col_align_right-expected.txt:
        * tables/mozilla/marvin/x_col_span-expected.txt:
        * tables/mozilla/marvin/x_col_valign_baseline-expected.txt:
        * tables/mozilla/marvin/x_col_valign_bottom-expected.txt:
        * tables/mozilla/marvin/x_col_valign_middle-expected.txt:
        * tables/mozilla/marvin/x_col_valign_top-expected.txt:
        * tables/mozilla/marvin/x_col_width_pct-expected.txt:
        * tables/mozilla/marvin/x_col_width_px-expected.txt:
        * tables/mozilla/marvin/x_col_width_rel-expected.txt:
        * tables/mozilla/marvin/x_colgroup_align_center-expected.txt:
        * tables/mozilla/marvin/x_colgroup_align_char-expected.txt:
        * tables/mozilla/marvin/x_colgroup_align_justify-expected.txt:
        * tables/mozilla/marvin/x_colgroup_align_left-expected.txt:
        * tables/mozilla/marvin/x_colgroup_align_right-expected.txt:
        * tables/mozilla/marvin/x_colgroup_span-expected.txt:
        * tables/mozilla/marvin/x_colgroup_valign_baseline-expected.txt:
        * tables/mozilla/marvin/x_colgroup_valign_bottom-expected.txt:
        * tables/mozilla/marvin/x_colgroup_valign_middle-expected.txt:
        * tables/mozilla/marvin/x_colgroup_valign_top-expected.txt:
        * tables/mozilla/marvin/x_colgroup_width_rel-expected.txt:
        * tables/mozilla/marvin/x_table_align_center-expected.txt:
        * tables/mozilla/marvin/x_table_align_left-expected.txt:
        * tables/mozilla/marvin/x_table_align_right-expected.txt:
        * tables/mozilla/marvin/x_table_bgcolor_name-expected.txt:
        * tables/mozilla/marvin/x_table_bgcolor_rgb-expected.txt:
        * tables/mozilla/marvin/x_table_cellpadding-expected.txt:
        * tables/mozilla/marvin/x_table_cellpadding_pct-expected.txt:
        * tables/mozilla/marvin/x_table_cellspacing-expected.txt:
        * tables/mozilla/marvin/x_table_cellspacing_pct-expected.txt:
        * tables/mozilla/marvin/x_table_class-expected.txt:
        * tables/mozilla/marvin/x_table_id-expected.txt:
        * tables/mozilla/marvin/x_table_style-expected.txt:
        * tables/mozilla/marvin/x_table_width_pct-expected.txt:
        * tables/mozilla/marvin/x_table_width_px-expected.txt:
        * tables/mozilla/marvin/x_td_align_center-expected.txt:
        * tables/mozilla/marvin/x_td_align_char-expected.txt:
        * tables/mozilla/marvin/x_td_align_justify-expected.txt:
        * tables/mozilla/marvin/x_td_align_left-expected.txt:
        * tables/mozilla/marvin/x_td_align_right-expected.txt:
        * tables/mozilla/marvin/x_td_bgcolor_name-expected.txt:
        * tables/mozilla/marvin/x_td_bgcolor_rgb-expected.txt:
        * tables/mozilla/marvin/x_td_class-expected.txt:
        * tables/mozilla/marvin/x_td_colspan-expected.txt:
        * tables/mozilla/marvin/x_td_height-expected.txt:
        * tables/mozilla/marvin/x_td_id-expected.txt:
        * tables/mozilla/marvin/x_td_nowrap-expected.txt:
        * tables/mozilla/marvin/x_td_rowspan-expected.txt:
        * tables/mozilla/marvin/x_td_style-expected.txt:
        * tables/mozilla/marvin/x_td_valign_baseline-expected.txt:
        * tables/mozilla/marvin/x_td_valign_bottom-expected.txt:
        * tables/mozilla/marvin/x_td_valign_middle-expected.txt:
        * tables/mozilla/marvin/x_td_valign_top-expected.txt:
        * tables/mozilla/marvin/x_td_width-expected.txt:
        * tables/mozilla/marvin/x_th_align_center-expected.txt:
        * tables/mozilla/marvin/x_th_align_char-expected.txt:
        * tables/mozilla/marvin/x_th_align_justify-expected.txt:
        * tables/mozilla/marvin/x_th_align_left-expected.txt:
        * tables/mozilla/marvin/x_th_align_right-expected.txt:
        * tables/mozilla/marvin/x_th_bgcolor_name-expected.txt:
        * tables/mozilla/marvin/x_th_bgcolor_rgb-expected.txt:
        * tables/mozilla/marvin/x_th_class-expected.txt:
        * tables/mozilla/marvin/x_th_colspan-expected.txt:
        * tables/mozilla/marvin/x_th_height-expected.txt:
        * tables/mozilla/marvin/x_th_id-expected.txt:
        * tables/mozilla/marvin/x_th_nowrap-expected.txt:
        * tables/mozilla/marvin/x_th_rowspan-expected.txt:
        * tables/mozilla/marvin/x_th_style-expected.txt:
        * tables/mozilla/marvin/x_th_valign_baseline-expected.txt:
        * tables/mozilla/marvin/x_th_valign_bottom-expected.txt:
        * tables/mozilla/marvin/x_th_valign_middle-expected.txt:
        * tables/mozilla/marvin/x_th_valign_top-expected.txt:
        * tables/mozilla/marvin/x_th_width-expected.txt:
        * tables/mozilla/marvin/x_tr_align_center-expected.txt:
        * tables/mozilla/marvin/x_tr_align_char-expected.txt:
        * tables/mozilla/marvin/x_tr_align_justify-expected.txt:
        * tables/mozilla/marvin/x_tr_align_left-expected.txt:
        * tables/mozilla/marvin/x_tr_align_right-expected.txt:
        * tables/mozilla/marvin/x_tr_bgcolor_name-expected.txt:
        * tables/mozilla/marvin/x_tr_bgcolor_rgb-expected.txt:
        * tables/mozilla/marvin/x_tr_class-expected.txt:
        * tables/mozilla/marvin/x_tr_id-expected.txt:
        * tables/mozilla/marvin/x_tr_style-expected.txt:
        * tables/mozilla/marvin/x_tr_valign_baseline-expected.txt:
        * tables/mozilla/marvin/x_tr_valign_bottom-expected.txt:
        * tables/mozilla/marvin/x_tr_valign_middle-expected.txt:
        * tables/mozilla/marvin/x_tr_valign_top-expected.txt:
        * tables/mozilla_expected_failures/bugs/bug178855-expected.txt:
        * tables/mozilla_expected_failures/marvin/x_caption_align_left-expected.txt:
        * tables/mozilla_expected_failures/marvin/x_caption_align_right-expected.txt:
        * tables/mozilla_expected_failures/marvin/x_colgroup_width_pct-expected.txt:
        * tables/mozilla_expected_failures/marvin/x_colgroup_width_px-expected.txt:

WebCore:

        Reviewed by Hyatt.

        - removed the m_implicit bit from Node, freeing up a bit

        - fixed <rdar://problem/5172607> XML parser is creating implicit tbody elements
        - fixed <rdar://problem/5172596> HTML-parser-created tbody elements are being
          ignored when processing CSS child rules

        Test: fast/css/child-selector-implicit-tbody.html

        * css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::checkSelector):
        Removed code that used to look at implicitNode(), which no longer exists.

        * css/html4.css: Added a style rule to handle cases where we have a tr
        inside a table with no intervening table section.

        * dom/Node.h: Removed the bit.
        * dom/Node.cpp:
        (WebCore::Node::Node): Don't initialize the bit.
        (WebCore::Node::dump): Don't dump the bit.

        * dom/XMLTokenizer.cpp:
        (WebCore::XMLTokenizer::startElementNs): Remove the code that made a
        table body element. This is handled in the render tree, the DOM tree
        should not be mangled.
        (WebCore::XMLTokenizer::endElementNs): Removed code that used to look at
        implicitNode(), which no longer exists.

        * html/HTMLElementFactory.cpp: (WebCore::tableSectionConstructor):
        Removed boolean parameter that was passed to the HTMLTableSectionElement
        constructor.
        * html/HTMLParser.cpp: (WebCore::HTMLParser::handleError): Ditto.
        * html/HTMLTableElement.cpp:
        (WebCore::HTMLTableElement::createTHead): Ditto.
        (WebCore::HTMLTableElement::createTFoot): Ditto.
        (WebCore::HTMLTableElement::insertRow): Changed to return a PassRefPtr,
        because it's possible the row could be removed from the table by JavaScript
        code responding to DOM mutation events before the function returns. Just
        something I noticed by code inspection.

        * html/HTMLTableSectionElement.cpp:
        (WebCore::HTMLTableSectionElement::HTMLTableSectionElement): Removed
        boolean "implicit" parameter to the constructor.
        (WebCore::HTMLTableSectionElement::insertRow): Changed to return a
        PassRefPtr for the same reason cited above.

        * html/HTMLTableElement.h: Changed the return value of insertRow to be
        a PassRefPtr.
        * html/HTMLTableSectionElement.h: Ditto.

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

14 years ago2007-05-01 Jungshik Shin <jungshik.shin@gmail.com>
ap [Tue, 1 May 2007 09:03:04 +0000 (09:03 +0000)]
2007-05-01  Jungshik Shin  <jungshik.shin@gmail.com>

        Reviewed by Alexey.

        - get rid of non-ASCII lteral characters : suppress compiler warnings
        http://bugs.webkit.org/show_bug.cgi?id=13551

        * kjs/testkjs.cpp:
        * pcre/pcre_compile.c:

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

14 years agoLayoutTests:
hyatt [Tue, 1 May 2007 04:57:01 +0000 (04:57 +0000)]
LayoutTests:

        Fix for bug 12691, generated content and display: table-cell duplicates
        content on a size change.  Make sure that the code to update before/after
        content is smart enough to drill into anonymous containers to find the
        old child.

        Reviewed by olliej

        * fast/css-generated-content/table-cell-before-content-expected.checksum: Added.
        * fast/css-generated-content/table-cell-before-content-expected.png: Added.
        * fast/css-generated-content/table-cell-before-content-expected.txt: Added.
        * fast/css-generated-content/table-cell-before-content.html: Added.

WebCore:

        Fix for bug 12691, generated content and display: table-cell duplicates
        content on a size change.  Make sure that the code to update before/after
        content is smart enough to drill into anonymous containers to find the
        old child.

        Reviewed by olliej

        Added fast/css-generated-content/table-cell-before-content.html

        * rendering/RenderBlock.cpp:
        (WebCore::RenderBlock::setStyle):
        * rendering/RenderButton.cpp:
        (WebCore::RenderButton::updateBeforeAfterContent):
        * rendering/RenderButton.h:
        * rendering/RenderContainer.cpp:
        (WebCore::RenderContainer::beforeAfterContainer):
        (WebCore::RenderContainer::updateBeforeAfterContent):
        (WebCore::RenderContainer::updateBeforeAfterContentForObject):
        * rendering/RenderContainer.h:
        * rendering/RenderInline.cpp:
        (WebCore::RenderInline::setStyle):
        (WebCore::RenderInline::addChildToFlow):
        (WebCore::RenderInline::splitInlines):

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

14 years ago Reviewed by justin
justing [Tue, 1 May 2007 04:32:03 +0000 (04:32 +0000)]
    Reviewed by justin

        <http://bugs.webkit.org/show_bug.cgi?id=13552>
        comment to Range::comparePoint in dom/Range.cpp has
        '0xD0' in place of '-': leading to a compiler warning

        * dom/Range.cpp:
        (WebCore::Range::comparePoint):

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

14 years agoLayoutTests:
justing [Mon, 30 Apr 2007 23:45:40 +0000 (23:45 +0000)]
LayoutTests:

        Reviewed by darin

        <rdar://problem/4875189> List gets deleted when replacing contents

        Replacing the contents of a list:
        * editing/inserting/4875189-1-expected.checksum: Added.
        * editing/inserting/4875189-1-expected.png: Added.
        * editing/inserting/4875189-1-expected.txt: Added.
        * editing/inserting/4875189-1.html: Added.
        Of a floating div:
        * editing/inserting/4875189-2-expected.checksum: Added.
        * editing/inserting/4875189-2-expected.png: Added.
        * editing/inserting/4875189-2-expected.txt: Added.
        * editing/inserting/4875189-2.html: Added.
        Pasting a list on top of itself now creates a nested list:
        * editing/pasteboard/paste-4039777-fix-expected.checksum:
        * editing/pasteboard/paste-4039777-fix-expected.png:
        * editing/pasteboard/paste-4039777-fix-expected.txt:

WebCore:

        Reviewed by darin

        <rdar://problem/4875189> List gets deleted when replacing contents

        * editing/ReplaceSelectionCommand.cpp:
        (WebCore::ReplaceSelectionCommand::doApply): Don't expand
        to include elements, like lists, when deleting a selection
        to replace it.

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

14 years ago Reviewed by ggaren
justing [Mon, 30 Apr 2007 22:06:27 +0000 (22:06 +0000)]
    Reviewed by ggaren

        Buildfix.  Leave in grammar checking machinery to
        keep WebKit building on other platforms but don't
        operate on sentence sized chunks so that we don't
        re-introduce 5157329.

        * editing/Editor.h:
        * editing/mac/EditorMac.mm:
        (WebCore::Editor::markMisspellingsAfterTypingToPosition):
        (WebCore::markMisspellingsOrBadGrammar):
        (WebCore::Editor::markMisspellings):
        (WebCore::Editor::markBadGrammar):
        * page/Frame.cpp:
        (WebCore::Frame::respondToChangedSelection):

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

14 years agoLayoutTests:
justing [Mon, 30 Apr 2007 21:14:09 +0000 (21:14 +0000)]
LayoutTests:

        Reviewed by darin

        <rdar://problem/5168598> Crash when deleting a link inside an input field

        * editing/deleting/5168598-expected.checksum: Added.
        * editing/deleting/5168598-expected.png: Added.
        * editing/deleting/5168598-expected.txt: Added.
        * editing/deleting/5168598.html: Added.

WebCore:

        Reviewed by darin

        <rdar://problem/5168598>
        Crash when deleting a link inside an input field

        Deletion code tries to update one of its positions to keep
        it valid and nullifies it.  Later in mergeParagraphs we try
        to find out if it's valid by checking to see if it's still in
        the document and crash.

        The fix is just to not bother keeping the position,
        m_upstreamStart, valid, since after the point in the code
        under examination, it doesn't need to be valid anymore.
        Besides, code to update DeleteSelectionCommand's positions
        should go in the functions that handle the content removal
        that might invalid them (there are already FIXMEs about this).

        Also, users shouldn't be able to insert links (or any rich content)
        into plaintext-only regions like text fields, even if those
        text fields are inside richly editable regions.  This is a
        separate bug and is filed as 5171552.

        * editing/DeleteSelectionCommand.cpp:
        (WebCore::DeleteSelectionCommand::handleGeneralDelete): Don't
        bother updating m_upstreamStart.

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

14 years ago<rdar://problem/5108546> REGRESSION: Zoom follows keyboard focus does not work
pyeh [Mon, 30 Apr 2007 21:03:01 +0000 (21:03 +0000)]
<rdar://problem/5108546> REGRESSION: Zoom follows keyboard focus does not work

        * WebCore.xcodeproj/project.pbxproj:
        Add new file SelectionControllerMac.mm
        * dom/Document.cpp:
        (WebCore::Document::updateSelection):
        Move the code to fire off AXSelectedTextChanged notification to a new method notifyAccessibilityForSelectionChange in SelectionControllerMac.mm
        * editing/SelectionController.cpp:
        (WebCore::SelectionController::setSelection):
        call notifyAccessibilityForSelectionChange to handle firing of accessibility notification and zoomFocusChange
        * editing/SelectionController.h:
        Add notifyAccessibilityForSelectionChange method for Mac platform
        * platform/Widget.h:
        * platform/mac/WidgetMac.mm:
        (WebCore::Widget::convertToScreenCoordinate):
        Add a helper routine that takes IntPoint in local coordinate and convert that to screen coordinate

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

14 years ago Reviewed by harrison
justing [Mon, 30 Apr 2007 19:43:27 +0000 (19:43 +0000)]
    Reviewed by harrison

        <rdar://problem/5157329>
        REGRESSION: Crash at WTF::fastFree() when arrowing or deleting backward through editable text (13493)

        Backing out the changes for 4859132.

        * editing/Editor.h:
        * editing/mac/EditorMac.mm:
        (WebCore::Editor::markMisspellingsAfterTypingToPosition):
        (WebCore::Editor::markMisspellings):
        * page/Frame.cpp:
        (WebCore::Frame::respondToChangedSelection):

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

14 years ago - Removed debugging statements and took out the removal of the deletion
kmccullo [Mon, 30 Apr 2007 17:43:13 +0000 (17:43 +0000)]
    - Removed debugging statements and took out the removal of the deletion
          of the symlink.  This is because multiple instances of run-webkit-tests
          can be running at the same time.
        * Scripts/run-webkit-tests:

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

14 years ago Reviewed by Oliver.
antti [Mon, 30 Apr 2007 10:44:25 +0000 (10:44 +0000)]
    Reviewed by Oliver.

        Fix <rdar://problem/5169261>
        REGRESSION: Google Maps zooming too sensitive when using two fingers on trackpad

        Use line based delta values when generating DOM wheel events, use pixel deltas
        for scrolling only.

        No test case, requires user interaction and specific hardware.

        * page/EventHandler.cpp:
        (WebCore::EventHandler::handleWheelEvent):
        * platform/PlatformWheelEvent.h:
        (WebCore::PlatformWheelEvent::continuousDeltaX):
        (WebCore::PlatformWheelEvent::continuousDeltaY):
        * platform/mac/WheelEventMac.mm:
        (WebCore::PlatformWheelEvent::PlatformWheelEvent):

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

14 years ago Reviewed by Brady.
oliver [Mon, 30 Apr 2007 06:27:02 +0000 (06:27 +0000)]
    Reviewed by Brady.

        Build fix for experimental features build

        * rendering/RenderForeignObject.cpp:

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

14 years ago2007-04-29 Mitz Pettel <mitz@webkit.org>
bdash [Mon, 30 Apr 2007 03:29:00 +0000 (03:29 +0000)]
2007-04-29  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Dave Hyatt.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13544
          REGRESSION (r21120): Member profile frame not erased at Yahoo! Answers

        Test: fast/repaint/layer-hide-when-needs-layout.html

        * rendering/RenderObject.cpp:
        (WebCore::RenderObject::setStyle): If we get a layout hint, we count on
        layout() or updateLayerPositions() to repaint. In the case of a layer
        going invisible, however, we need to repaint here since neither of the
        above will do it.

2007-04-29  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Dave Hyatt.

        - repaint test for http://bugs.webkit.org/show_bug.cgi?id=13544
          REGRESSION (r21120): Member profile frame not erased at Yahoo! Answers

        * fast/repaint/layer-hide-when-needs-layout-expected.checksum: Added.
        * fast/repaint/layer-hide-when-needs-layout-expected.png: Added.
        * fast/repaint/layer-hide-when-needs-layout-expected.txt: Added.
        * fast/repaint/layer-hide-when-needs-layout.html: Added.

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

14 years ago Reviewed by Steve.
oliver [Sun, 29 Apr 2007 22:31:40 +0000 (22:31 +0000)]
    Reviewed by Steve.

        Fix for <rdar://problem/5126393> REGRESSION (r19672-19697): Safari drag image drags at original size (13261)

        Added logic to ensure that when we create the drag image
        we're looking at the rendered image size.

        * page/DragController.cpp:
        (WebCore::DragController::doImageDrag):
          Pass the rendered image size to fitDragImageToMaxSize

        * platform/DragImage.cpp:
        (WebCore::fitDragImageToMaxSize):
           Need to do scaling relative to rendered image size

        * platform/DragImage.h:
           scaleDragImage needs x and y scales to allow for them to be
           scaled seperately on a page

        * platform/gdk/DragImageGdk.cpp:
        (WebCore::scaleDragImage):
           Prevent build bustage

        * platform/mac/DragImageMac.mm:
        (WebCore::scaleDragImage):
           scale width and height independently

        * platform/qt/DragImageQt.cpp:
        (WebCore::scaleDragImage):
           Prevent build bustage

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

14 years ago Reviewed by Adele.
darin [Sun, 29 Apr 2007 20:32:51 +0000 (20:32 +0000)]
    Reviewed by Adele.

        - fix the Gmail part of http://bugs.webkit.org/show_bug.cgi?id=12798
          and <rdar://problem/4118559>

        This patch changes things so that resizing can make things larger than their
        initial size, but not smaller. I believe this resolves the problem for Gmail,
        but you can still get strange behavior if you have a small right-aligned <textarea>.

        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::resize): Rewrote to use more IntPoint and IntSize and add
        in a new minimum-size algorithm. This function now has the side effect of storing
        a new minimumSizeForResizing in the element being resized, and respects any existing
        minimum as well.
        (WebCore::RenderLayer::offsetFromResizeCorner): Reversed this so it converts the
        point into coordinates relative to the resize corner, hence negative numbers mean
        the point is above and to the left. This is used only in the algorithm above.

        * dom/Element.h: Removed unneeded include of Attr.h, and added forward declarations
        as needed. Removed all friend declarations; there was no need for any of them.
        Tweaked formatting of various declarations and removed unneeded parameter names.
        Made as much as possible private, and a couple things public to avoid the need for
        friend classes. Added minimumSizeForResizing, setMinimumSizeForResizing, rareData,
        and createRareData functions.

        * dom/Element.cpp:
        (WebCore::rareDataMap): Added. Function to return the single global map.
        (WebCore::rareDataFromMap): Added. For use by inlined functions to get the
        data from the map in the case where it's present.
        (WebCore::ElementRareData::ElementRareData): Added.
        (WebCore::Element::~Element): Added code to delete the element from the rare
        data map when appropriate.
        (WebCore::Element::rareData): Added.
        (WebCore::Element::createRareData): Added.
        (WebCore::Element::setAttributeNodeNS): Got rid of the old inline version of this,
        since it was the sole reason we had to include Attr.h in Element.h.
        (WebCore::Element::minimumSizeForResizing): Added. Accesses rare data.
        (WebCore::Element::setMinimumSizeForResizing): Ditto.

        * dom/Node.h: Renamed the m_specified bit since it's now used for two purposes.
        Its new long name is m_attrWasSpecifiedOrElementHasRareData.

        * dom/Attr.cpp: (WebCore::Attr::Attr): Updated for bit name change.
        * dom/Attr.h:
        (WebCore::Attr::specified): Ditto.
        (WebCore::Attr::setSpecified): Added, since setting the bit directly is pretty ugly now.
        * dom/Document.cpp: (WebCore::Document::adoptNode): Updated for bit name change.
        * dom/Node.cpp:
        (WebCore::Node::Node): Ditto.
        (WebCore::Node::dump): Ditto.

        * dom/NamedAttrMap.cpp: (WebCore::NamedAttrMap::isReadOnlyNode): Moved out of line,
        since this was the sole reason we had to include Element.h in NamedAttrMap.h.

        * html/HTMLElement.cpp:
        (WebCore::HTMLElement::nodeName): Changed to use tagQName() since that's a public
        way to do something that's now private.
        (WebCore::HTMLElement::cloneNode): Ditto.

        * dom/Attribute.cpp:
        * dom/NamedAttrMap.h:
        * dom/NamedMappedAttrMap.cpp:
        * dom/StyledElement.h:
        * html/HTMLTokenizer.h:
        * xml/XPathFunctions.cpp:
        * xml/XPathStep.cpp:
        Updated includes, and other administrivia.

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

14 years agoLayoutTests:
weinig [Sun, 29 Apr 2007 20:09:08 +0000 (20:09 +0000)]
LayoutTests:

        Reviewed by Darin and Hyatt.

        - test and updated results for http://bugs.webkit.org/show_bug.cgi?id=13487
          Implement O(1) absoluteClippedOverflowRect and absoluteOutlineBox during layout for a possible speed gain

        * fast/repaint/bugzilla-3509-expected.checksum:
        * fast/repaint/bugzilla-3509-expected.png:
        * fast/repaint/intermediate-layout-position-clip-expected.checksum: Added.
        * fast/repaint/intermediate-layout-position-clip-expected.png: Added.
        * fast/repaint/intermediate-layout-position-clip-expected.txt: Added.
        * fast/repaint/intermediate-layout-position-clip.html: Added.
        * fast/repaint/intermediate-layout-position-expected.checksum:
        * fast/repaint/intermediate-layout-position-expected.png:

WebCore:

        Reviewed by Darin and Hyatt.

        - http://bugs.webkit.org/show_bug.cgi?id=13487
          Implement O(1) absoluteClippedOverflowRect and absoluteOutlineBox during layout for a possible speed gain

        Test for a bug fixed by this patch: fast/repaint/intermediate-layout-position-clip.html

        By keeping track of the total translation and clip during layout, absolutePosition and
        computeAbsoluteRepaintRect become O(1).

        * WebCore.pro: Added LayoutState.cpp
        * WebCore.vcproj/WebCore/WebCore.vcproj: Added LayoutState.{cpp,h}
        * WebCore.xcodeproj/project.pbxproj: Added LayoutState.{cpp,h}
        * WebCoreSources.bkl: Added LayoutState.cpp
        * page/FrameView.cpp:
        (WebCore::FrameView::layout): When doing subtree layout, push an initial layout state for the
        layout root.
        * rendering/LayoutState.cpp: Added. A LayoutState corresponding to a box consists of the origin of
        its coordinate system in view coordinates and possibly the clip rect, in view coordinates, which
        applies to its children.
        (WebCore::LayoutState::LayoutState):
        (WebCore::LayoutState::destroy):
        (WebCore::throw):
        (WebCore::LayoutState::operator delete):
        * rendering/LayoutState.h: Added.
        (WebCore::LayoutState::LayoutState):
        * rendering/RenderBlock.cpp:
        (WebCore::RenderBlock::layoutBlock): Factored out the "only positioned objects" case into
        layoutOnlyPositionedObjects(). Added code to push/pop LayoutState, unless we have columns, in
        which case we disable/enable LayoutState. Removed the translation by the layout delta
        of the old absolute rects, since that is now integral to absoluteClippedOverflowRect() and
        absoluteOutlineBox().
        (WebCore::RenderBlock::layoutOnlyPositionedObjects): Added this helper function that checks for
        and handles the case where only positioned objects need layout. Returns true in that case.
        (WebCore::RenderBlock::repaintOverhangingFloats): Disabled LayoutState when repainting floats, since
        they may belong to other containers.
        (WebCore::RenderBlock::updateFirstLetter): Disabled LayoutState when adding and removing from the
        first letter container, since it may be different than ourselves.
        * rendering/RenderBlock.h:
        * rendering/RenderBox.cpp:
        (WebCore::RenderBox::setStyle): Minor cleanup: changed element()->document() to the equivalent document().
        (WebCore::RenderBox::offsetForPositionedInContainer): Added. Factored out common code from
        absolutePosition, computeAbsoluteRepaintRect and RenderLayer::updateLayerPosition into this function.
        (WebCore::RenderBox::absolutePosition): Added an O(1) code path when this function is called during
        layout (and LayoutState has not been disabled). Factored out some code into
        offsetForPositionedInContainer.
        (WebCore::RenderBox::absoluteClippedOverflowRect): Added the layout delta here, so callers don't need
        to.
        (WebCore::RenderBox::computeAbsoluteRepaintRect): Added an O(1) code path when this function is called
        during layout (and LayoutState has not been disabled). Factored out some code into
        offsetForPositionedInContainer.
        * rendering/RenderContainer.cpp:
        (WebCore::RenderContainer::layout): Added LayoutState push/pop.
        * rendering/RenderFlexibleBox.cpp:
        (WebCore::RenderFlexibleBox::layoutBlock): Factored out the "only positioned objects" case into
        layoutOnlyPositionedObjects(). Added LayoutState push/pop. Removed the translation by the layout delta
        of the old absolute rects, since that is now integral to absoluteClippedOverflowRect() and
        absoluteOutlineBox().
        * rendering/RenderFlow.cpp:
        (WebCore::RenderFlow::absoluteClippedOverflowRect): Added an ASSERT.
        * rendering/RenderForeignObject.cpp:
        (WebCore::RenderForeignObject::layout): Disabled LayoutState because it is incompatible with arbitrary
        affine transforms.
        * rendering/RenderHTMLCanvas.cpp:
        (WebCore::RenderHTMLCanvas::layout): Removed the translation by the layout delta of the old absolute
        rects, since that is now integral to absoluteClippedOverflowRect() and absoluteOutlineBox().
        * rendering/RenderImage.cpp:
        (WebCore::RenderImage::layout): Ditto.
        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::updateLayerPositions): Added ASSERTs. Factored out some code into
        RenderBox::offsetForPositionedInContainer.
        (WebCore::RenderLayer::updateScrollInfoAfterLayout): Disabled LayoutState around the call to
        scrollToOffset.
        * rendering/RenderListItem.cpp:
        (WebCore::RenderListItem::updateMarkerLocation): Disabled LayoutState when moving the list marker, since
        it can trigger repainting in other containers.
        * rendering/RenderObject.cpp:
        (WebCore::RenderObject::isRoot): Minor cleanup: changed element()->document() to the equivalent
        document().
        (WebCore::RenderObject::computeAbsoluteRepaintRect): Removed 'return' at the end of the function.
        (WebCore::RenderObject::container): Reordered to avoid calling isText() twice.
        (WebCore::RenderObject::absoluteOutlineBox): Added the layout delta here, so callers don't need to.
        * rendering/RenderSVGContainer.cpp:
        (WebCore::RenderSVGContainer::layout): Disabled LayoutState because it is incompatible with arbitrary
        affine transforms.
        * rendering/RenderTable.cpp:
        (WebCore::RenderTable::layout):  Factored out the "only positioned objects" case into
        layoutOnlyPositionedObjects(). Added LayoutState push/pop. Removed the translation by the layout delta
        of the old absolute rects, since that is now integral to absoluteClippedOverflowRect() and
        absoluteOutlineBox().
        * rendering/RenderTableCell.cpp:
        (WebCore::RenderTableCell::absoluteClippedOverflowRect): Added the layout delta here, so callers
        don't need to.
        (WebCore::RenderTableCell::computeAbsoluteRepaintRect): For the O(1) code path, skipped the correction
        for the parent row, because RenderTableRow doesn't push a translation into LayoutState.
        (WebCore::RenderTableCell::absolutePosition): Ditto.
        * rendering/RenderTableRow.cpp:
        (WebCore::RenderTableRow::layout): Added LayoutState push/pop.
        * rendering/RenderTableSection.cpp:
        (WebCore::RenderTableSection::setCellWidths): Added LayoutState push/pop if cells are repainted or
        receive layout.
        (WebCore::RenderTableSection::calcRowHeight): Ditto.
        (WebCore::RenderTableSection::layoutRows): Added LayoutState push/pop.
        * rendering/RenderView.cpp:
        (WebCore::RenderView::RenderView):
        (WebCore::RenderView::layout): Added initial LayoutState setup for the layout.
        (WebCore::RenderView::paintBoxDecorations): Minor cleanup: changed element()->document() to the
        equivalent document().
        (WebCore::RenderView::repaintViewRectangle): Ditto.
        (WebCore::RenderView::pushLayoutState): Added. Pushes initial layout state for subtree layout.
        * rendering/RenderView.h:
        (WebCore::RenderView::pushLayoutState):
        (WebCore::RenderView::popLayoutState):
        (WebCore::RenderView::layoutState):
        (WebCore::RenderView::disableLayoutState):
        (WebCore::RenderView::enableLayoutState):

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

14 years agoAdd layout test for previous checkin.
hyatt [Sun, 29 Apr 2007 10:33:17 +0000 (10:33 +0000)]
Add layout test for previous checkin.

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

14 years ago Fix for bugzilla 13535, regression from positioning cleanup a while
hyatt [Sun, 29 Apr 2007 10:30:48 +0000 (10:30 +0000)]
    Fix for bugzilla 13535, regression from positioning cleanup a while
        back.  Some code got added to compute available width for a relpositioned
        inline, only the code meant to be used for inline flows and not inline blocks.

        Reviewed by mjs

        fast/block/positioning/inline-block-relposition.html

        * rendering/RenderBox.cpp:
        (WebCore::RenderBox::containingBlockWidthForPositioned):

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

14 years ago2007-04-29 Oliver Hunt <oliver@apple.com>
oliver [Sun, 29 Apr 2007 10:17:19 +0000 (10:17 +0000)]
2007-04-29  Oliver Hunt  <oliver@apple.com>
        Drag and drop support for WebKit/Qt

WebCore:

        Reviewed by Zack.

        Implement basic drag and drop support for qt
        this allows content and link dragging but does not
        provide the DOM data access methods.

        * platform/DragData.h:
        * platform/qt/ClipboardQt.cpp:
        (WebCore::ClipboardQt::ClipboardQt):
        (WebCore::ClipboardQt::~ClipboardQt):
        (WebCore::ClipboardQt::clearData):
        (WebCore::ClipboardQt::clearAllData):
        (WebCore::ClipboardQt::setData):
        (WebCore::ClipboardQt::declareAndWriteDragImage):
        (WebCore::ClipboardQt::writeURL):
        (WebCore::ClipboardQt::writeRange):
        (WebCore::ClipboardQt::hasData):
        * platform/qt/ClipboardQt.h:
        (WebCore::ClipboardQt::clipboardData):
        (WebCore::ClipboardQt::invalidateWritableData):
        * platform/qt/DragDataQt.cpp:
        (WebCore::DragData::canSmartReplace):
        (WebCore::DragData::containsPlainText):
        (WebCore::DragData::asPlainText):
        (WebCore::DragData::createClipboard):
        (WebCore::DragData::containsCompatibleContent):
        (WebCore::DragData::containsURL):
        (WebCore::DragData::asURL):
        (WebCore::DragData::asFragment):

WebKitQt:

        Reviewed by Zack.

        Tie QT drag events to the DragController logic
        to allow drag and drop events to be handled by
        webkit.

        * Api/qwebframe.cpp:
        * Api/qwebframe.h:
        * Api/qwebpage.cpp:
        (QWebPagePrivate::QWebPagePrivate):
        (QWebPage::QWebPage):
        (dropActionToDragOp):
        (dragOpToDropAction):
        (QWebPage::dragEnterEvent):
        (QWebPage::dragLeaveEvent):
        (QWebPage::dragMoveEvent):
        (QWebPage::dropEvent):
        * Api/qwebpage.h:
        * WebCoreSupport/DragClientQt.cpp:
        (WebCore::DragClientQt::startDrag):
        * WebCoreSupport/DragClientQt.h:
        (WebCore::DragClientQt::DragClientQt):

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

14 years ago Reviewed by Darin and Oliver.
mjs [Sun, 29 Apr 2007 07:33:46 +0000 (07:33 +0000)]
    Reviewed by Darin and Oliver.

        - change Document to hold onto Frame* instead of FrameView*

        This is an architectural cleanup and a prerequisite to fixing citibank login
        in an architecturally sound way.

        Some fixups for the page cache:

        * history/CachedPage.cpp:
        (WebCore::CachedPage::restore): Add an assert that the document's view
        matches the cached page's view - this needs to be true by the time this
        function is called.
        (WebCore::CachedPage::clear): Change assert to require frame matching, not
        view matching.
        * history/CachedPage.h:
        (WebCore::CachedPage::view): Add a way to get the view out.
        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::begin): pass Frame, not FrameView, when creating document
        (WebCore::FrameLoader::open): restore view based on CachedPage, not Document.

        Remaining changes are just straightforward updates based on which
        pointer is held or passed:

        * dom/DOMImplementation.cpp:
        (WebCore::DOMImplementation::createDocument):
        (WebCore::DOMImplementation::createHTMLDocument):
        * dom/DOMImplementation.h:
        * dom/Document.cpp:
        (WebCore::Document::Document):
        (WebCore::Document::view):
        (WebCore::Document::frame):
        (WebCore::Document::recalcStyle):
        (WebCore::Document::updateLayout):
        (WebCore::Document::attach):
        (WebCore::Document::detach):
        (WebCore::Document::createTokenizer):
        (WebCore::Document::recalcStyleSelector):
        (WebCore::Document::setInPageCache):
        (WebCore::Document::applyXSLTransform):
        * dom/Document.h:
        * html/HTMLDocument.cpp:
        (WebCore::HTMLDocument::HTMLDocument):
        * html/HTMLDocument.h:
        * html/HTMLViewSourceDocument.cpp:
        (WebCore::HTMLViewSourceDocument::HTMLViewSourceDocument):
        * html/HTMLViewSourceDocument.h:
        * ksvg2/svg/SVGDocument.cpp:
        (WebCore::SVGDocument::SVGDocument):
        * ksvg2/svg/SVGDocument.h:
        * loader/ImageDocument.cpp:
        (WebCore::ImageDocument::ImageDocument):
        * loader/ImageDocument.h:
        * loader/PluginDocument.cpp:
        (WebCore::PluginDocument::PluginDocument):
        * loader/PluginDocument.h:
        * loader/TextDocument.cpp:
        (WebCore::TextDocument::TextDocument):
        * loader/TextDocument.h:
        * xml/XSLTProcessor.cpp:
        (WebCore::XSLTProcessor::createDocumentFromSource):
        (WebCore::XSLTProcessor::transformToDocument):
        * xml/XSLTProcessor.h:

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

14 years ago2007-04-28 Jungshik Shin <jungshik.shin@gmail.com>
bdash [Sun, 29 Apr 2007 06:42:32 +0000 (06:42 +0000)]
2007-04-28  Jungshik Shin  <jungshik.shin@gmail.com>

        Reviewed by Sam Weinig.

        - Replace copyright sign in Latin-1 (0xA9) with '(C)'
        http://bugs.webkit.org/show_bug.cgi?id=13531

        * bindings/npruntime.h:

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

14 years agoLayoutTests:
adele [Sun, 29 Apr 2007 05:32:01 +0000 (05:32 +0000)]
LayoutTests:

        Added test to cover updated fix for <rdar://problem/5152299>

        * fast/events/no-blur-on-enter-button-expected.txt: Added.
        * fast/events/no-blur-on-enter-button.html: Added.

WebCore:

        RS by Darin.

        Update to fix for <rdar://problem/5152299>

        Test: fast/events/no-blur-on-enter-button.html

        * html/HTMLButtonElement.cpp: (WebCore::HTMLButtonElement::defaultEventHandler):
        Don't blur the button when handling the enter key.  This matches our new behavior for input elements.

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

14 years agoImproved the ChangeLog comment. Maybe the code needs a comment like this too.
darin [Sun, 29 Apr 2007 04:35:06 +0000 (04:35 +0000)]
Improved the ChangeLog comment. Maybe the code needs a comment like this too.

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

14 years agoJavaScriptCore:
darin [Sun, 29 Apr 2007 04:25:33 +0000 (04:25 +0000)]
JavaScriptCore:

        Reviewed by Maciej.

        - fix <rdar://problem/5154144> Hamachi test fails: assertion failure in ListHashSet

        Test: fast/forms/add-remove-form-elements-stress-test.html

        * wtf/ListHashSet.h:
        (WTF::ListHashSetNodeAllocator::ListHashSetNodeAllocator): Initialize
        m_isDoneWithInitialFreeList to false.
        (WTF::ListHashSetNodeAllocator::allocate): Added assertions based on a debug-only
        m_isAllocated flag that make sure we don't allocate a block that's already allocated.
        These assertions helped pinpoint the bug. Set m_isDoneWithInitialFreeList when we
        allocate the last block of the initial free list. Once we're done with the initial
        free list, turn off the rule that says that the next node in the pool after the last
        node in the free list is also free. This rule works because any free nodes are added
        to the head of the free list, so a node that hasn't been allocated even once is always
        at the tail of the free list. But it doesn't work any longer once the entire pool
        has been used at least once.
        (WTF::ListHashSetNodeAllocator::deallocate): Set the node's m_isAllocated to false.
        (WTF::ListHashSetNodeAllocator::pastPool): Added. Used above.
        (WTF::ListHashSetNodeAllocator::inPool): Changed to use the pastPool function.
        (WTF::ListHashSetNode::ListHashSetNode): Initialize m_isAllocated to true.
        (WTF::ListHashSetNode::operator new): Removed variable name for unused size
        parameter.
        (WTF::ListHashSetNode::destroy): Changed to call the destructor rather than
        delete -- this gets rid of the need to define an operator delete.

LayoutTests:

        Reviewed by Maciej.

        - test for <rdar://problem/5154144> Hamachi test fails: assertion failure in ListHashSet

        * fast/forms/add-remove-form-elements-stress-test-expected.txt: Added.
        * fast/forms/add-remove-form-elements-stress-test.html: Added.

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

14 years agoLayoutTests:
adele [Sun, 29 Apr 2007 04:09:26 +0000 (04:09 +0000)]
LayoutTests:

        Tests for <rdar://problem/5152299> REGRESSION: <button> elements do not respond to space and return key as similar <input> elements do

        * fast/forms/button-enter-click-expected.txt: Added.
        * fast/forms/button-enter-click.html: Added.
        * fast/forms/button-spacebar-click-expected.txt: Added.
        * fast/forms/button-spacebar-click.html: Added.

WebCore:

        Fix by Darin, reviewed by me.

        Fix for <rdar://problem/5152299> REGRESSION: <button> elements do not respond to space and return key as similar <input> elements do

        Tests:
        fast/forms/button-spacebar-click.html
        fast/forms/button-enter-click.html

        * html/HTMLButtonElement.h: (WebCore::HTMLButtonElement::): Made type enum private.
        Removed unnecessary m_value and m_currValue, and unused m_dirty.
        * html/HTMLButtonElement.cpp:
        (WebCore::HTMLButtonElement::HTMLButtonElement): Removed initialization of m_dirty.
        (WebCore::HTMLButtonElement::parseMappedAttribute): Removed case to parse value and accesskey attributes, which don't do anything.
        (WebCore::HTMLButtonElement::defaultEventHandler): Removed unnecessary check for button type.
         Added cases to handle spacebar and enter keys in the same way that they're handled for <input type=button/reset/submit> buttons.
        (WebCore::HTMLButtonElement::isSuccessfulSubmitButton): Shortened comment.
        (WebCore::HTMLButtonElement::appendFormData): Use value() instead of m_currValue.  Renamed encoding argument to formData.

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

14 years agoLayoutTests:
darin [Sun, 29 Apr 2007 01:36:11 +0000 (01:36 +0000)]
LayoutTests:

        Reviewed by Darin.

        - test for http://bugs.webkit.org/show_bug.cgi?id=13501
          REGRESSION (r21093): Pasting into text field fails and makes the caret disappear

        * fast/dynamic/anonymous-block-layer-lost-expected.checksum: Added.
        * fast/dynamic/anonymous-block-layer-lost-expected.png: Added.
        * fast/dynamic/anonymous-block-layer-lost-expected.txt: Added.
        * fast/dynamic/anonymous-block-layer-lost.html: Added.

WebCore:

        Reviewed by Darin.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13501
          REGRESSION (r21093): Pasting into text field fails and makes the caret disappear

        Test: fast/dynamic/anonymous-block-layer-lost.html

        * rendering/RenderBlock.cpp:
        (WebCore::RenderBlock::removeChild): Avoid cutting off layers when removing
        the anonymous block.

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

14 years ago Reviewed by Darin.
darin [Sun, 29 Apr 2007 01:34:03 +0000 (01:34 +0000)]
    Reviewed by Darin.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13471
          REGRESSION (r21045): Secure keyboard entry mode remains in effect after leaving a password field by submitting

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::clear): Replaced direct access to Frame's d->m_doc
        with a call to Frame::setDocument.
        (WebCore::FrameLoader::begin): Ditto.
        (WebCore::FrameLoader::open): Ditto.
        (WebCore::FrameLoader::switchOutLowBandwidthDisplayIfReady): Ditto.
        (WebCore::FrameLoader::canCachePage): Added a comment.
        * manual-tests/secure-keyboard-enabled-after-submit.html: Added.
        * page/Frame.cpp:
        (WebCore::Frame::setDocument): Reset secure keyboard entry, in case the
        old document's focused node has set it.

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

14 years ago Reviewed by Adele.
darin [Sun, 29 Apr 2007 01:30:19 +0000 (01:30 +0000)]
    Reviewed by Adele.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13379
          REGRESSION: Cannot add an option to a selected popup

        * manual-tests/update-empty-popup.html: Added.
        * platform/mac/PopupMenuMac.mm:
        (WebCore::PopupMenu::show): Added a call to the client's hidePopup()
        callback before returning in the case that the menu is empty. Otherwise
        the client thinks that the menu stayed open.

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

14 years agoLayoutTests:
adele [Sat, 28 Apr 2007 19:00:38 +0000 (19:00 +0000)]
LayoutTests:

        Reviewed by Darin.

        Test for <rdar://problem/4806988> REGRESSION: history.go(0) doesn't reload page (affects buttons at facebook.com)

        * fast/history/history_reload-expected.txt: Added.
        * fast/history/history_reload.html: Added.
        * fast/history/resources/history_reload_window.html: Added.

WebCore:

        Reviewed by Darin.

        Fix for <rdar://problem/4806988> REGRESSION: history.go(0) doesn't reload page (affects buttons at facebook.com)

        Test: fast/history/history_reload.html

        * loader/FrameLoader.cpp: (WebCore::FrameLoader::scheduleHistoryNavigation):
        We avoid doing a redirection when going to the same url in a history navigation, but we have to still
        schedule a redirection when the number of steps is zero so we can force a reload.

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

14 years ago Reviewed by Darin.
ap [Sat, 28 Apr 2007 18:29:19 +0000 (18:29 +0000)]
    Reviewed by Darin.

        http://bugs.webkit.org/show_bug.cgi?id=13514
        REGRESSION (r20997-21003): Crash in WebCore::Node::document

        Test: fast/dom/NodeList/childNodes-reset-cache.html

        * dom/Node.cpp:
        (WebCore::Node::notifyLocalNodeListsChildrenChanged): Reset the common caches for child node
        lists. We cannot rely on notifications, since there may be no list objects left around.

        * dom/ChildNodeList.cpp:
        (WebCore::ChildNodeList::rootNodeChildrenChanged):
        * dom/ChildNodeList.h:
        Override rootNodeChildrenChanged() with an empty implementation, now that caches are reset
        in Node::notifyLocalNodeListsChildrenChanged().

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

14 years ago2007-04-28 Mark Rowe <mrowe@apple.com>
bdash [Sat, 28 Apr 2007 11:36:32 +0000 (11:36 +0000)]
2007-04-28  Mark Rowe  <mrowe@apple.com>

        Build fix.  Change use of nil to 0 in C++ source file.

        * editing/Editor.cpp:
        (WebCore::Editor::increaseSelectionListLevel):
        (WebCore::Editor::increaseSelectionListLevelOrdered):
        (WebCore::Editor::increaseSelectionListLevelUnordered):

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

14 years agoLayoutTests:
darin [Sat, 28 Apr 2007 08:11:43 +0000 (08:11 +0000)]
LayoutTests:

        Reviewed by Adele.

        - test for http://bugs.webkit.org/show_bug.cgi?id=6216
          REGRESSION: getElementsByTagNameNS with empty namespace doesn't work

        * fast/dom/namespaces-1-expected.txt: Added.
        * fast/dom/namespaces-1.html: Added.

WebCore:

        Reviewed by Adele.

        - fix http://bugs.webkit.org/show_bug.cgi?id=6216
          REGRESSION: getElementsByTagNameNS with empty namespace doesn't work

        * dom/Node.cpp:
        (WebCore::TagNodeList::TagNodeList): Changed parameter to PassRefPtr, tweaked formatting.
        (WebCore::TagNodeList::item): Tweaked formatting.
        (WebCore::TagNodeList::nodeMatches): Tweaked formatting.
        (WebCore::Node::setPrefix): Tweaked formatting.
        (WebCore::Node::getElementsByTagNameNS): Added special case to turn empty namespaces into
        null namespaces.

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

14 years agoLayoutTests:
darin [Sat, 28 Apr 2007 08:07:08 +0000 (08:07 +0000)]
LayoutTests:

        Reviewed by Kevin Decker.

        - update test result for fix for <rdar://problem/5148271> Chinese numbers
          are incorrect in numbered lists (including pasting from TextEdit to Mail)

        * fast/lists/w3-list-styles-expected.txt: Updated. The new test results match
        the behavior of Firefox.

WebCore:

        Reviewed by Kevin Decker.

        - fix <rdar://problem/5148271> Chinese numbers are incorrect in numbered lists
          (including pasting from TextEdit to Mail)

        Covered by an existing regression test, which had wrong results checked in.

        Hyatt told me not to use the algorithm in the CSS 3 specification, but I didn't listen!
        This corrects mistakes there and makes our results match current Firefox.

        * rendering/RenderListMarker.cpp:
        (WebCore::toCJKIdeographic): Added code to strip off trailing zero digits.
        Added special case for zero, since that's the one case where we don't want
        to strip off trailing zero digits. Fixed mixed-up logic that would put the
        digits in each group backwards.
        (WebCore::listMarkerText): Corrected character values for the CJK_IDEOGRAPHIC case.

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

14 years ago Reviewed by Oliver.
darin [Sat, 28 Apr 2007 08:02:40 +0000 (08:02 +0000)]
    Reviewed by Oliver.

        - fix <rdar://problem/5138250> REGRESSION (r18245): Changing any one aspect
          of a font applies all the aspects to selected text

        Since this bug is only visible with the font panel, I don't
        think there's any way to make a DumpRenderTree-driven test.

        * editing/Editor.cpp: (WebCore::Editor::fontForSelection):
        Set hasMultipleFonts properly. The old code had an early exit
        that was left over from when hasMultipleFonts was a pointer.

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

14 years ago Reviewed by Maciej.
aroben [Sat, 28 Apr 2007 06:57:49 +0000 (06:57 +0000)]
    Reviewed by Maciej.

        Cleaned up prepare-ChangeLog

        * Scripts/prepare-ChangeLog: No code changes, but style now matches
        the prevalent style of our perl scripts.

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

14 years ago2007-04-27 Mitz Pettel <mitz@webkit.org>
bdash [Sat, 28 Apr 2007 06:48:29 +0000 (06:48 +0000)]
2007-04-27  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Antti.

        - http://bugs.webkit.org/show_bug.cgi?id=13521
          Update fast/css/percentage-non-integer for the new empty table cell behavior

        * fast/css/percentage-non-integer-expected.checksum:
        * fast/css/percentage-non-integer-expected.png:
        * fast/css/percentage-non-integer-expected.txt:
        * fast/css/percentage-non-integer.html:

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

14 years ago2007-04-27 Mitz Pettel <mitz@webkit.org>
bdash [Sat, 28 Apr 2007 06:40:41 +0000 (06:40 +0000)]
2007-04-27  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Dave Hyatt.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13525
          Repainting a listbox schedules a relayout

        No test possible as there is no change to functionality.

        - fix the root cause of http://bugs.webkit.org/show_bug.cgi?id=10862
          REGRESSION: Collapsing form input on www.myspace.com

        No test possible since text fields are no longer implemented as flexboxes.

        * rendering/RenderObject.cpp:
        (WebCore::RenderObject::scheduleRelayout): Added a check that the subtree
        root is in the tree and not free-floating.
        * rendering/bidi.cpp:
        (WebCore::RenderBlock::bidiReorderCharacters): Changed addChild and removeChild
        to appendChildNode and removeChildNode to save some unnecessary overhead.
        These calls still mark the tree for relayout, but the change in scheduleRelayout
        ensures that a layout won't be scheduled.

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

14 years agoAdd back in WebInspectorPanel.
thatcher [Sat, 28 Apr 2007 05:04:23 +0000 (05:04 +0000)]
Add back in WebInspectorPanel.

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

14 years agoRevert some unintended changes and fix the build.
thatcher [Sat, 28 Apr 2007 04:54:16 +0000 (04:54 +0000)]
Revert some unintended changes and fix the build.

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

14 years ago2007-04-27 Timothy Hatcher <timothy@apple.com>
thatcher [Sat, 28 Apr 2007 04:48:58 +0000 (04:48 +0000)]
2007-04-27 Timothy Hatcher <timothy@apple.com>

        Reviewed by Darin.

        There were only a couple calls to ResourceLoadDelegate forwarder left,
        this removed the calls and added a new cached method for didFailLoad.

        * WebKit/DefaultDelegates/WebDefaultResourceLoadDelegate.h: Removed.
        * WebKit/DefaultDelegates/WebDefaultResourceLoadDelegate.m: Removed.
        * WebKit/Plugins/WebNullPluginView.mm: Call the resource load delegate directly.
        * WebKit/WebCoreSupport/WebFrameLoaderClient.mm: Call the cached didFailLoad delegate function.
        * WebKit/WebKit.xcodeproj/project.pbxproj: Remove WebDefaultResourceLoadDelegate.
        * WebKit/WebView/WebDataSource.mm: Remove the #import for WebDefaultResourceLoadDelegate.h
        * WebKit/WebView/WebView.mm: Remove the ResourceLoadDelegate forwarder, and remove a method that isn't used.
        * WebKit/WebView/WebViewInternal.h: Ditto.
        * WebKit/WebView/WebViewPrivate.h: Remove a method that is no longer used.

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

14 years ago<rdar://problem/5153017> REPRODUCIBLE ASSERT: range != nil in WebViewFactory.mm:415...
pyeh [Sat, 28 Apr 2007 01:41:56 +0000 (01:41 +0000)]
<rdar://problem/5153017> REPRODUCIBLE ASSERT: range != nil in WebViewFactory.mm:415 -[WebViewFactory startOfTextMarkerRange:] (13354)

        * bridge/mac/WebCoreAXObject.mm:
        (-[WebCoreAXObject textMarkerRangeForRange:]):
        Fix the range length check.  It was incorrectly assuming that the max range requested cannot be equal
        to the length of the string.  Since NSRange location is index based, it's perfectly valid to have the max
        range equal to the the text length.

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

14 years agoWebCore:
harrison [Sat, 28 Apr 2007 01:04:51 +0000 (01:04 +0000)]
WebCore:

        Reviewed by Darin.

        <rdar://problem/5160627> Export JS list creation support as ObjC SPI for Mail

        * WebCore.exp:
        * editing/Editor.h:
        * editing/Editor.cpp:
        (WebCore::Editor::insertOrderedList):
        (WebCore::Editor::insertUnorderedList):
        (WebCore::Editor::canIncreaseSelectionListLevel):
        (WebCore::Editor::canDecreaseSelectionListLevel):
        (WebCore::Editor::increaseSelectionListLevel):
        (WebCore::Editor::increaseSelectionListLevelOrdered):
        (WebCore::Editor::increaseSelectionListLevelUnordered):
        (WebCore::Editor::decreaseSelectionListLevel):
        Add insertOrderedList() and insertUnorderedList().
        Move existing list management functions from WebCoreFrameBridge.
        The level increasing functions now return a PassRefPtr<Node> instead of Node.
        Add check for canEditRichly.

        - in many other functions, condense the check for nil client().

        * editing/InsertListCommand.h:
        * editing/InsertListCommand.cpp:
        (WebCore::InsertListCommand::insertList):
        (WebCore::InsertListCommand::fixOrphanedListChild):
        (WebCore::InsertListCommand::doApply):
        Add insertList(), which returns the new list element.
        Don't set the list element's ID if the specified ID was an empty string.

        * editing/ModifySelectionListLevel.h:
        * editing/ModifySelectionListLevel.cpp:
        (WebCore::IncreaseSelectionListLevelCommand::increaseSelectionListLevelWithType):
        (WebCore::IncreaseSelectionListLevelCommand::increaseSelectionListLevel):
        (WebCore::IncreaseSelectionListLevelCommand::increaseSelectionListLevelOrdered):
        (WebCore::IncreaseSelectionListLevelCommand::increaseSelectionListLevelUnordered):
        Return PassRefPtr<Node> instead of Node.

        * page/mac/WebCoreFrameBridge.h:
        * page/mac/WebCoreFrameBridge.mm:
        Move list management functions to Editor.h and Editor.cpp.

WebKit:

        Reviewed by Darin.

        <rdar://problem/5160627> Export JS list creation support as ObjC SPI for Mail

        * WebKit.xcodeproj/project.pbxproj:
        * WebView/WebHTMLView.mm:
        (-[WebHTMLView _selectedRange]):
        (-[WebHTMLView _shouldDeleteRange:]):
        (-[WebHTMLView _canEdit]):
        (-[WebHTMLView _canEditRichly]):
        (-[WebHTMLView _hasSelection]):
        (-[WebHTMLView _hasSelectionOrInsertionPoint]):
        (-[WebHTMLView _hasInsertionPoint]):
        (-[WebHTMLView _isEditable]):
        Condense the check for nil [self frame].
        Remove canEditRichly checks and rely on the editor to do the check instead.

        (-[WebHTMLView _insertOrderedList]):
        (-[WebHTMLView _insertUnorderedList]):
        New.

        (-[WebHTMLView _canIncreaseSelectionListLevel]):
        (-[WebHTMLView _canDecreaseSelectionListLevel]):
        (-[WebHTMLView _increaseSelectionListLevel]):
        (-[WebHTMLView _increaseSelectionListLevelOrdered]):
        (-[WebHTMLView _increaseSelectionListLevelUnordered]):
        (-[WebHTMLView _decreaseSelectionListLevel]):
        Moved from bridge to frame editor.

        * WebView/WebHTMLViewPrivate.h:
        Add _insertOrderedList and _insertUnorderedList to WebHTMLView(WebPrivate)

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

14 years ago Fix for <rdar://problem/5134254>, toolbar buttons don't work in gmail after...
hyatt [Sat, 28 Apr 2007 00:48:43 +0000 (00:48 +0000)]
    Fix for <rdar://problem/5134254>, toolbar buttons don't work in gmail after double-click.  Make sure
        to clear out the capturing frame, since you don't get a second up when a double-click happens.

        Reviewed by mjs

        * page/EventHandler.cpp:
        (WebCore::EventHandler::handleMouseDoubleClickEvent):

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

14 years agoLayoutTests:
justing [Fri, 27 Apr 2007 21:59:53 +0000 (21:59 +0000)]
LayoutTests:

        Reviewed by oliver

        <rdar://problem/5164796>
        Enable execCommand("CreateLink", ...) for caret selections

        * editing/execCommand/5164796-expected.checksum: Added.
        * editing/execCommand/5164796-expected.png: Added.
        * editing/execCommand/5164796-expected.txt: Added.
        * editing/execCommand/5164796.html: Added.

WebCore:

        Reviewed by oliver

        <rdar://problem/5164796>
        Enable execCommand("CreateLink", ...) for caret selections

        * editing/CreateLinkCommand.cpp:
        (WebCore::CreateLinkCommand::doApply): For a caret
        selection, insert a link at the caret position
        with the url as the link text.
        * editing/JSEditor.cpp: Enable CreateLink for caret
        selections.

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

14 years ago Rubberstamped by Mark
beidson [Fri, 27 Apr 2007 17:02:13 +0000 (17:02 +0000)]
    Rubberstamped by Mark

        Remove default implementation of UIDelegate method that was removed

        I have also been instructed to give Tim a hard time about this one - apparently
        it was his job to clean it out and he failed...  failed miserably.

        :)

        * DefaultDelegates/WebDefaultUIDelegate.m: Removed webViewPrint:

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

14 years ago2007-04-27 Nazar Kulyk <schamane@myeburg.net>
bdash [Fri, 27 Apr 2007 14:55:19 +0000 (14:55 +0000)]
2007-04-27  Nazar Kulyk  <schamane@myeburg.net>

        Reviewed by Mark Rowe.

        Basic auto-correction of user-entered URLs.

        * GdkLauncher/main.cpp:
        (autocorrectURL):
        (goToURLBarText):
        (main):

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

14 years ago Forgot this png from previous commit.
antti [Fri, 27 Apr 2007 13:08:36 +0000 (13:08 +0000)]
    Forgot this png from previous commit.

        * fast/table/empty-cells-expected.png: Added.

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

14 years ago2007-04-28 Mark Rowe <mrowe@apple.com>
bdash [Fri, 27 Apr 2007 12:36:52 +0000 (12:36 +0000)]
2007-04-28  Mark Rowe  <mrowe@apple.com>

        Rubber-stamped by Anders.

        Rename some methods and variables to match the style guidelines.

        * GdkLauncher/main.cpp:
        (stringIsEmpty):
        (stringIsEqual):
        (goToURLBarText):
        (goButtonClickedCallback):
        (urlBarEnterCallback):
        (frameResizeCallback):
        (frameDestroyCallback):
        (menuMainBackCallback):
        (menuMainForwardCallback):
        (menuMainQuitCallback):
        (main):

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

14 years agoLayoutTests:
antti [Fri, 27 Apr 2007 11:32:29 +0000 (11:32 +0000)]
LayoutTests:

        Reviewed by Darin, Hyatt.

        Test for http://bugs.webkit.org/show_bug.cgi?id=13081
        Empty table cells not handled correctly
        <rdar://problem/5067926>

        Added test case and checked in updated results for others. Changes represent
        progressions or are not visible in rendering output.

        * fast/block/positioning/negative-right-pos-expected.txt:
        * fast/css/acid2-expected.txt:
        * fast/css/acid2-pixel-expected.txt:
        * fast/css/percentage-non-integer-expected.checksum:
        * fast/css/percentage-non-integer-expected.png:
        * fast/css/percentage-non-integer-expected.txt:
        * fast/repaint/table-cell-move-expected.checksum:
        * fast/repaint/table-cell-move-expected.png:
        * fast/repaint/table-cell-move-expected.txt:
        * fast/table/012-expected.txt:
        * fast/table/032-expected.txt:
        * fast/table/empty-cells-expected.checksum: Added.
        * fast/table/empty-cells-expected.txt: Added.
        * fast/table/empty-cells.html: Added.
        * fast/table/empty-section-crash-expected.txt:
        * http/tests/misc/acid2-expected.txt:
        * http/tests/misc/acid2-pixel-expected.txt:
        * tables/mozilla/bugs/bug100334-expected.txt:
        * tables/mozilla/bugs/bug1188-expected.txt:
        * tables/mozilla/bugs/bug16012-expected.checksum:
        * tables/mozilla/bugs/bug16012-expected.png:
        * tables/mozilla/bugs/bug16012-expected.txt:
        * tables/mozilla/bugs/bug1818-6-expected.txt:
        * tables/mozilla/bugs/bug222336-expected.txt:
        * tables/mozilla/bugs/bug222467-expected.txt:
        * tables/mozilla/bugs/bug78162-expected.txt:
        * tables/mozilla/core/misc-expected.checksum:
        * tables/mozilla/core/misc-expected.png:
        * tables/mozilla/core/misc-expected.txt:
        * tables/mozilla_expected_failures/bugs/bug14007-1-expected.checksum:
        * tables/mozilla_expected_failures/bugs/bug14007-1-expected.png:
        * tables/mozilla_expected_failures/bugs/bug14007-1-expected.txt:
        * tables/mozilla_expected_failures/bugs/bug72393-expected.checksum:
        * tables/mozilla_expected_failures/bugs/bug72393-expected.png:
        * tables/mozilla_expected_failures/bugs/bug72393-expected.txt:
        * tables/mozilla_expected_failures/other/empty_cells-expected.txt:

WebCore:

        Reviewed by Darin, Hyatt.

        Fix http://bugs.webkit.org/show_bug.cgi?id=13081
        Empty table cells not handled correctly
        <rdar://problem/5067926>

        To match Firefox and IE, empty table cells with variable width need to be
        treated as special case. They generally get zero width.

        * rendering/AutoTableLayout.cpp:
        (WebCore::AutoTableLayout::recalcColumn):
        (WebCore::AutoTableLayout::calcEffectiveWidth):
        (WebCore::AutoTableLayout::layout):
        * rendering/AutoTableLayout.h:
        (WebCore::AutoTableLayout::Layout::Layout):
        * rendering/RenderStyle.h:
        (WebCore::RenderStyle::hasPadding):

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

14 years ago Reviewed by Oliver.
rwlbuis [Fri, 27 Apr 2007 11:20:09 +0000 (11:20 +0000)]
    Reviewed by Oliver.

        http://bugs.webkit.org/show_bug.cgi?id=13052
        Text not rendered in a second inline SVG element

        Correct the rendering of svg text when embedded in non-svg container.

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

14 years ago Reviewed by Mark.
mjs [Fri, 27 Apr 2007 08:43:19 +0000 (08:43 +0000)]
    Reviewed by Mark.

        <rdar://problem/5154113> Repro ASSERT (would be crash) in KJS::GCLock::GCLock (13462)
        http://bugs.webkit.org/show_bug.cgi?id=13462

        * WebInspector/WebInspector.m:
        (-[WebInspectorPrivate dealloc]): Delay release of WebView to avoid GC re-entrancy.

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

14 years ago2007-04-27 Mark Rowe <mrowe@apple.com>
bdash [Fri, 27 Apr 2007 08:40:27 +0000 (08:40 +0000)]
2007-04-27  Mark Rowe  <mrowe@apple.com>

        Reviewed by Maciej.

        * WebCore.pro: Remove non-existent paths from qmake files.

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

14 years ago Reviewed by Mitz.
andersca [Fri, 27 Apr 2007 08:10:54 +0000 (08:10 +0000)]
    Reviewed by Mitz.

        <rdar://problem/5165755>
        View Source is broken; empty window is shown

        Return YES for applewebdata URLs.

        * WebView/WebView.mm:
        (+[WebView _canHandleRequest:]):

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

14 years ago2007-04-25 Lars Naesbye Christensen <lars@naesbye.dk>
bdash [Fri, 27 Apr 2007 07:52:27 +0000 (07:52 +0000)]
2007-04-25  Lars Naesbye Christensen <lars@naesbye.dk>

        Reviewed by Timothy Hatcher.

        http://bugs.webkit.org/show_bug.cgi?id=13477
        Bug 13477: Help and context menu cursors need dropshadows.

        * Resources/cellCursor.png:
        * Resources/contextMenuCursor.png:
        * Resources/helpCursor.png:
        * platform/mac/CursorMac.mm:
        (WebCore::contextMenuCursor):
        (WebCore::copyCursor):
        (WebCore::progressCursor):
        (WebCore::noDropCursor):

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

14 years ago2007-04-27 Christopher Brichford <chrisb@adobe.com>
bdash [Fri, 27 Apr 2007 07:44:03 +0000 (07:44 +0000)]
2007-04-27  Christopher Brichford  <chrisb@adobe.com>

        Reviewed by Timothy Hatcher.

        Fix for: Bug 13211: Move JavaScriptCore mac project files for apollo port
        http://bugs.webkit.org/show_bug.cgi?id=13211

        * JavaScriptCore.apolloproj/mac/JavaScriptCore.Debug.xcconfig: Added.
        * JavaScriptCore.apolloproj/mac/JavaScriptCore.Release.xcconfig: Added.
        * JavaScriptCore.apolloproj/mac/JavaScriptCore.xcconfig: Added.
        * JavaScriptCore.apolloproj/mac/JavaScriptCore.xcodeproj/project.pbxproj: Added.
        * JavaScriptCore.apolloproj/mac/JavaScriptCore/JavaScriptCore.Debug.xcconfig: Removed.
        * JavaScriptCore.apolloproj/mac/JavaScriptCore/JavaScriptCore.Release.xcconfig: Removed.
        * JavaScriptCore.apolloproj/mac/JavaScriptCore/JavaScriptCore.xcconfig: Removed.
        * JavaScriptCore.apolloproj/mac/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj: Removed.

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

14 years ago2007-04-27 Holger Freyther <freyther@kde.org>
bdash [Fri, 27 Apr 2007 07:37:26 +0000 (07:37 +0000)]
2007-04-27  Holger Freyther <freyther@kde.org>

        Reviewed by Maciej.

        Remove unmaintained CMake build system.

        * CMakeLists.txt: Removed.

2007-04-27  Holger Freyther <freyther@kde.org>

        Reviewed by Maciej.

        Remove unmaintained CMake build system.

        * CMakeLists.txt: Removed.
        * pcre/CMakeLists.txt: Removed.

2007-04-27  Holger Freyther <freyther@kde.org>

        Reviewed by Maciej.

        Remove unmaintained CMake build system.

        * CMakeLists.txt: Removed.

2007-04-27  Holger Freyther <freyther@kde.org>

        Reviewed by Maciej.

        Remove unmaintained CMake build system.

        * QtLauncher/CMakeLists.txt: Removed.
        * WebKitPart/CMakeLists.txt: Removed.

2007-04-27  Holger Freyther <freyther@kde.org>

        Reviewed by Maciej.

        Remove unmaintained CMake build system.

        * DumpRenderTree/DumpRenderTree.qtproj/CMakeLists.txt: Removed.
        * Scripts/build-webkit: Remove references to CMake.
        * Scripts/webkitdirs.pm: Remove references to CMake.

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

14 years ago2007-04-27 Mark Rowe <mrowe@apple.com>
bdash [Fri, 27 Apr 2007 06:03:58 +0000 (06:03 +0000)]
2007-04-27  Mark Rowe  <mrowe@apple.com>

        Reviewed by Oliver.

        * JavaScriptCore.xcodeproj/project.pbxproj: Improve dependencies in Xcode project
        by marking dftables as a dependency of Generate Derived Sources rather than of
        JavaScriptCore itself.

2007-04-27  Mark Rowe  <mrowe@apple.com>

        Reviewed by Oliver.

        * JavaScriptGlue.xcodeproj/project.pbxproj: Improve dependencies in Xcode project
        by marking JavaScriptGlue as a dependency of testjsglue.

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

14 years ago2007-04-26 Matt Perry <mpComplete@gmail.com>
bdash [Fri, 27 Apr 2007 05:48:36 +0000 (05:48 +0000)]
2007-04-26  Matt Perry  <mpComplete@gmail.com>

        Reviewed by Maciej.

        Added a version check to hackAroundLibXMLEntityBug, since that bug has
        been fixed in recent versions of libxml2.

        * dom/XMLTokenizer.cpp:
        (WebCore::hackAroundLibXMLEntityBug):

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

14 years ago2007-04-26 Christopher Brichford <chrisb@adobe.com>
bdash [Fri, 27 Apr 2007 05:36:21 +0000 (05:36 +0000)]
2007-04-26  Christopher Brichford  <chrisb@adobe.com>

        Reviewed by Timothy Hatcher.

        - Fix for Bug 13204: Submit initial version of mac project files for apollo
          port of WebCore
          http://bugs.webkit.org/show_bug.cgi?id=13204

        * WebCore.apolloproj/mac/WebCore.Debug.xcconfig: Added.
        * WebCore.apolloproj/mac/WebCore.Release.xcconfig: Added.
        * WebCore.apolloproj/mac/WebCore.xcconfig: Added.
        * WebCore.apolloproj/mac/WebCore.xcodeproj/project.pbxproj: Added.

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

14 years ago2007-04-26 Alp Toker <alp@atoker.com>
bdash [Fri, 27 Apr 2007 05:11:50 +0000 (05:11 +0000)]
2007-04-26  Alp Toker  <alp@atoker.com>

        Reviewed by Maciej.

        http://bugs.webkit.org/show_bug.cgi?id=13505
        Cast loses precision on x86_64

        * platform/graphics/cairo/GraphicsContextCairo.cpp:
        (WebCore::GraphicsContext::beginTransparencyLayer):
        (WebCore::GraphicsContext::endTransparencyLayer):
        Use a stack of opacity values instead of Cairo user data.

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

14 years ago2007-04-26 Alp Toker <alp@atoker.com>
bdash [Fri, 27 Apr 2007 05:09:15 +0000 (05:09 +0000)]
2007-04-26  Alp Toker  <alp@atoker.com>

        Reviewed by Mark Rowe.

        * GdkLauncher/main.cpp:
        (main): Avoid use of deprecated API.

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

14 years ago2007-04-26 Mitz Pettel <mitz@webkit.org>
bdash [Fri, 27 Apr 2007 05:00:58 +0000 (05:00 +0000)]
2007-04-26  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Darin.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13500
          REGRESSION: Listbox scrollbar does not reflect actual scrolled position when selected option is added after layout

        Test: fast/forms/listbox-scrollbar-incremental-load.html

        The scroller position was not updated correctly during insertion, since
        scrollbar proportions are updated only during layout

        * rendering/RenderListBox.cpp:
        (WebCore::RenderListBox::selectionChanged): Delay scrolling to reveal the
        selection until after layout if m_optionsChanged is set. Layout is
        guaranteed to happen soon when the flag is set.

2007-04-26  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Darin.

        - pixel test for http://bugs.webkit.org/show_bug.cgi?id=13500
          REGRESSION: Listbox scrollbar does not reflect actual scrolled position when selected option is added after layout

        * fast/forms/listbox-scrollbar-incremental-load-expected.checksum: Added.
        * fast/forms/listbox-scrollbar-incremental-load-expected.png: Added.
        * fast/forms/listbox-scrollbar-incremental-load-expected.txt: Added.
        * fast/forms/listbox-scrollbar-incremental-load.html: Added.

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

14 years ago Build fix -- added #includes that we used to get implicitly through
ggaren [Fri, 27 Apr 2007 01:41:40 +0000 (01:41 +0000)]
    Build fix -- added #includes that we used to get implicitly through
        JSStringRef.h.

        * API/JSNode.c:
        * API/JSNodeList.c:
        * API/minidom.c:
        * API/testapi.c:

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

14 years ago Reviewed by Maciej Stachowiak, Adam Roben.
ggaren [Fri, 27 Apr 2007 01:15:54 +0000 (01:15 +0000)]
    Reviewed by Maciej Stachowiak, Adam Roben.

        Fixed
        <rdar://problem/4885130> Remove #include of JSStringRefCF.h from JSStringRef.h
        <rdar://problem/4885123> JavaScriptCore is not cross-platform -- JSStringRef.h references CF datatypes

        * API/JSStringRef.h: Removed #include -- no clients need it anymore.

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

14 years agoFix layout test results jnow that we no longer make empty render texts.
hyatt [Fri, 27 Apr 2007 00:36:48 +0000 (00:36 +0000)]
Fix layout test results jnow that we no longer make empty render texts.

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

14 years agoFix layout test results jnow that we no longer make empty render texts.
hyatt [Fri, 27 Apr 2007 00:35:17 +0000 (00:35 +0000)]
Fix layout test results jnow that we no longer make empty render texts.

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

14 years agoFix layout test results jnow that we no longer make empty render texts.
hyatt [Fri, 27 Apr 2007 00:34:17 +0000 (00:34 +0000)]
Fix layout test results jnow that we no longer make empty render texts.

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

14 years agoFix layout test results jnow that we no longer make empty render texts.
hyatt [Fri, 27 Apr 2007 00:33:23 +0000 (00:33 +0000)]
Fix layout test results jnow that we no longer make empty render texts.

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

14 years ago Fix for 13511, crash on a counters page. Make sure that RenderTexts
hyatt [Fri, 27 Apr 2007 00:30:34 +0000 (00:30 +0000)]
    Fix for 13511, crash on a counters page.  Make sure that RenderTexts
        never get a null StringImpl.  I would have added an assert that the length
        had to be nonzero also, but unfortunately editing creates empty RenderTexts
        because of its EditingText DOM subclass that always forces renderers to be
        made.  I suppose there is a good reason for this...

        Reviewed by aroben

        * dom/CharacterData.cpp:
        (WebCore::CharacterData::setData):
        (WebCore::CharacterData::appendData):
        (WebCore::CharacterData::insertData):
        (WebCore::CharacterData::deleteData):
        (WebCore::CharacterData::replaceData):
        * rendering/RenderCounter.cpp:
        (WebCore::RenderCounter::RenderCounter):
        * rendering/RenderText.cpp:
        (WebCore::RenderText::RenderText):
        (WebCore::RenderText::trimmedPrefWidths):
        (WebCore::RenderText::setTextWithOffset):
        (WebCore::RenderText::setTextInternal):
        (WebCore::RenderText::setText):

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

14 years ago Reviewed by Oliver.
aliceli1 [Thu, 26 Apr 2007 23:21:45 +0000 (23:21 +0000)]
    Reviewed by Oliver.

        * editing/SelectionController.cpp:
        (WebCore::SelectionController::selectAll):
        adding null check

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

14 years agoLayoutTests:
justing [Thu, 26 Apr 2007 21:53:59 +0000 (21:53 +0000)]
LayoutTests:

        Reviewed by oliver

        <rdar://problem/5120591> Crash when creating a link in Leopard wiki server

        * editing/execCommand/5120591-expected.checksum: Added.
        * editing/execCommand/5120591-expected.png: Added.
        * editing/execCommand/5120591-expected.txt: Added.
        * editing/execCommand/5120591.html: Added.

WebCore:

        Reviewed by oliver

        <rdar://problem/5120591>
        Crash when creating a link in Leopard wiki server

        * editing/CreateLinkCommand.cpp:
        (WebCore::CreateLinkCommand::doApply): Use a RefPtr,
        setting the anchor's href can destroy it if there is
        an ancestor with a subtree modified event listener.

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

14 years ago2007-04-25 David Kilzer <ddkilzer@apple.com>
ddkilzer [Thu, 26 Apr 2007 21:15:26 +0000 (21:15 +0000)]
2007-04-25  David Kilzer  <ddkilzer@apple.com>

        Reviewed by Maciej.

        Add assertions for debug builds.

        * kjs/JSLock.cpp:
        (KJS::JSLock::lock): Assert the return value of pthread_mutex_lock() in debug builds.
        (KJS::JSLock::unlock): Assert the return value of pthread_mutex_unlock() in debug builds.

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

14 years ago - Changed debugging statements for more information.
kmccullo [Thu, 26 Apr 2007 20:37:04 +0000 (20:37 +0000)]
    - Changed debugging statements for more information.

        * Scripts/run-webkit-tests:

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

14 years ago Reviewed by hyatt
hyatt [Thu, 26 Apr 2007 19:51:11 +0000 (19:51 +0000)]
    Reviewed by hyatt

        - http://bugs.webkit.org/show_bug.cgi?id=13496
          Eliminate repaintObjectsBeforeLayout

        No tests added since there is no change to functionality. Several
        tests in fast/repaint test repainting of objects that move during layout.

        * page/FrameView.cpp:
        (WebCore::FrameView::layout):
        * rendering/RenderBlock.cpp:
        (WebCore::RenderBlock::layoutBlockChildren): Removed an early
        setNeedsLayout(false). Resetting the bit here prevents the full repaint
        that we rely on when we decide not to repaint children that move.
        * rendering/RenderBlock.h:
        * rendering/RenderObject.cpp:
        * rendering/RenderObject.h:
        * rendering/RenderTableSection.cpp:
        (WebCore::RenderTableSection::setCellWidths): Replaced the call to
        repaintObjectsBeforeLayout() if the cell didn't need layout with
        an unconditional repaint(). We don't bother with repainting only the
        delta between the old and new widths, since by marking the cell for
        layout results in a full repaint with the new width anyway.

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

14 years ago Fix for 13509. Don't recur to parent inline flows if they have been
hyatt [Thu, 26 Apr 2007 19:24:39 +0000 (19:24 +0000)]
    Fix for 13509.  Don't recur to parent inline flows if they have been
        marked as needing layout themselves.

        Reviewed by adele

        * rendering/RenderFlow.cpp:
        (WebCore::RenderFlow::dirtyLinesFromChangedChild):

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