WebKit-https.git
11 years ago2008-10-12 Pamela Greene <pam@chromium.org>
abarth@webkit.org [Mon, 13 Oct 2008 01:15:01 +0000 (01:15 +0000)]
2008-10-12  Pamela Greene  <pam@chromium.org>

        Fix a race condition by waiting for all create functions to be called
        before terminating.
        See https://bugs.webkit.org/show_bug.cgi?id=21531

        Reviewed by Darin Adler.
        Landed by Adam Barth.

        * platform/mac/svg/carto.net/window-expected.checksum:
        * platform/mac/svg/carto.net/window-expected.png:
        * platform/mac/svg/carto.net/window-expected.txt:
        * svg/carto.net/window.svg:

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

11 years ago2008-10-12 Pamela Greene <pam@chromium.org>
abarth@webkit.org [Mon, 13 Oct 2008 01:14:51 +0000 (01:14 +0000)]
2008-10-12  Pamela Greene  <pam@chromium.org>

        Remove inadvertent const redefinition, and add specific test for it.
        See https://bugs.webkit.org/show_bug.cgi?id=21532

        Reviewed by Darin Adler.
        Landed by Adam Barth.

        * fast/js/const-expected.txt:
        * fast/js/resources/const.js:

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

11 years ago2008-10-12 Pamela Greene <pam@chromium.org>
abarth@webkit.org [Mon, 13 Oct 2008 01:14:27 +0000 (01:14 +0000)]
2008-10-12  Pamela Greene  <pam@chromium.org>

        Add a test for null character data.
        See https://bugs.webkit.org/show_bug.cgi?id=21535

        Reviewed by Darin Adler.
        Landed by Adam Barth.

        * fast/dom/null-chardata-crash-expected.txt: Added.
        * fast/dom/null-chardata-crash.html: Added.

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

11 years ago2008-10-12 Tony Chang <tony@chromium.org>
darin@apple.com [Mon, 13 Oct 2008 01:13:29 +0000 (01:13 +0000)]
2008-10-12  Tony Chang  <tony@chromium.org>

        Reviewed by Sam Weinig.

        - https://bugs.webkit.org/show_bug.cgi?id=21480
          Remove duplicate definitions of MimeClassInfo and PluginInfo.

        * plugins/PluginInfoStore.cpp: Added an include.
        * plugins/PluginInfoStore.h: Removed struct declarations.

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

11 years ago2008-10-12 Darin Adler <darin@apple.com>
darin@apple.com [Mon, 13 Oct 2008 01:07:04 +0000 (01:07 +0000)]
2008-10-12  Darin Adler  <darin@apple.com>

        Reviewed by Dan Bernstein.

        - https://bugs.webkit.org/show_bug.cgi?id=21563
          Bug 21563: platform Gradient class should not use the CSS parser

        * platform/graphics/Gradient.cpp: Removed the overload of addColorStop
        that accepts a CSS color string. That's handled in the DOM now, not here.
        (WebCore::Gradient::addColorStop): Use getRGBA instead of having our own
        copy of it here.
        * platform/graphics/Gradient.h: More of the same. Tweak formatting a bit.

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

11 years agoWebCore:
darin@apple.com [Mon, 13 Oct 2008 00:58:00 +0000 (00:58 +0000)]
WebCore:

2008-10-12  Glenn Wilson  <gwilson@google.com>

        Reviewed by Darin Adler.

        - https://bugs.webkit.org/show_bug.cgi?id=20461

        Added feature to allow multi-line inputs into text fields.
        HTMLInputElement would cut the set text at the first 'system' character,
        so it would truncate at the first carriage return or newline.
        This fix modifies that behavior to truncate at any non-space system character
        but changes \r, \n, and \r\n to a single space.

        Tests: editing/pasteboard/paste-multiline-text-input.html

        * html/HTMLInputElement.cpp:
        (WebCore::HTMLInputElement::constrainValue):

LayoutTests:

2008-10-12  Glenn Wilson  <gwilson@google.com>

        Reviewed by Darin Adler.

        - https://bugs.webkit.org/show_bug.cgi?id=20461

        Added new test to verify that the user can cut and paste multiple lines of text
        into a single line text field.

        * fast/forms/paste-multiline-text-input-expected.txt: Added.
        * fast/forms/paste-multiline-text-input.html: Added.
        * fast/forms/input-truncate-newline-expected.txt: Removed.
        * fast/forms/input-truncate-newline.html: Removed.

        * fast/forms/8250-expected.txt: Added.
        * fast/forms/8250.html: Updated test to be text-only and expect new results.
        * platform/mac/fast/forms/8250-expected.checksum: Removed.
        * platform/mac/fast/forms/8250-expected.png: Removed.
        * platform/mac/fast/forms/8250-expected.txt: Removed.
        * platform/qt/fast/forms/8250-expected.txt: Removed.

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

11 years agoWebCore:
darin@apple.com [Mon, 13 Oct 2008 00:25:00 +0000 (00:25 +0000)]
WebCore:

2008-10-12  Dirk Schulze  <vbs85@gmx.de>

        Reviewed by Eric Seidel.

        - https://bugs.webkit.org/show_bug.cgi?id=20435

        Added exception codes to canvas gradients to match current specification.

        Tests: fast/canvas/gradient-addColorStop-with-invalid-color.html
               fast/canvas/linearGradient-infinite-values.html
               fast/canvas/radialGradient-infinite-values.html

        * html/CanvasGradient.cpp:
        (WebCore::CanvasGradient::addColorStop):
        * html/CanvasGradient.h:
        * html/CanvasGradient.idl:
        * html/CanvasRenderingContext2D.cpp:
        (WebCore::CanvasRenderingContext2D::createLinearGradient):
        (WebCore::CanvasRenderingContext2D::createRadialGradient):
        * html/CanvasRenderingContext2D.h:
        * html/CanvasRenderingContext2D.idl:

LayoutTests:

2008-10-12  Dirk Schulze  <vbs85@gmx.de>

        Reviewed by Eric Seidel.

        - test for https://bugs.webkit.org/show_bug.cgi?id=20435

        Added new tests to check behavior of canvas gradients to infinite values
        or invalid colors.

        * fast/canvas/gradient-addColorStop-with-invalid-color-expected.txt: Added.
        * fast/canvas/gradient-addColorStop-with-invalid-color.html: Added.
        * fast/canvas/linearGradient-infinite-values-expected.txt: Added.
        * fast/canvas/linearGradient-infinite-values.html: Added.
        * fast/canvas/radialGradient-infinite-values-expected.txt: Added.
        * fast/canvas/radialGradient-infinite-values.html: Added.
        * fast/canvas/resources/gradient-addColorStop-with-invalid-color.js: Added.
        * fast/canvas/resources/linearGradient-infinite-values.js: Added.
        * fast/canvas/resources/radialGradient-infinite-values.js: Added.

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

11 years ago2008-10-12 Eric Roman <eroman@chromium.org>
darin@apple.com [Sun, 12 Oct 2008 23:59:09 +0000 (23:59 +0000)]
2008-10-12  Eric Roman  <eroman@chromium.org>

        - https://bugs.webkit.org/show_bug.cgi?id=20246
          Implement RenderThemeWin::systemColor() to match Windows colors.

        Reviewed by David Hyatt.

        * rendering/RenderThemeWin.cpp:
        (WebCore::cssValueIdToSysColorIndex):
        (WebCore::RenderThemeWin::systemColor):
        * rendering/RenderThemeWin.h:

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

11 years ago2008-10-12 Aaron Bockover <abockover@novell.com>
darin@apple.com [Sun, 12 Oct 2008 23:50:45 +0000 (23:50 +0000)]
2008-10-12  Aaron Bockover  <abockover@novell.com>

        Reviewed by Alp Toker.

        Add version parsing for Flash, and the PluginQuirkDontSetNullWindowHandleOnDestroy
        plugin quirk if Flash 10 or newer since at least in b218, setting a NULL window
        handler on destroy crashes WebKit <https://bugs.webkit.org/show_bug.cgi?id=19859>

        * plugins/gtk/PluginPackageGtk.cpp:

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

11 years ago2008-10-12 Jonathon Jongsma <jonathon.jongsma@collabora.co.uk>
darin@apple.com [Sun, 12 Oct 2008 23:01:25 +0000 (23:01 +0000)]
2008-10-12  Jonathon Jongsma  <jonathon.jongsma@collabora.co.uk>

        Reviewed by Oliver Hunt.

        - https://bugs.webkit.org/show_bug.cgi?id=19331
          <rdar://problem/5984433>: Drag and drop of links in edit mode violates assert in MoveSelectionCommand::doApply()

        * page/DragController.cpp:
        (WebCore::DragController::startDrag): When a link is dragged without any
        text selected (note: for this to work, the EditableLinkBehavior setting must be
        set to 'live' or 'default') and then dropped into an editable html document it
        will eventually violate an ASSERT in MoveSelectionCommand that requires that there
        is a selection.  To prevent this, expand the selection to the enclosing anchor tag
        when the drag begins.

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

11 years ago2008-10-12 Darin Adler <darin@apple.com>
darin@apple.com [Sun, 12 Oct 2008 22:53:15 +0000 (22:53 +0000)]
2008-10-12  Darin Adler  <darin@apple.com>

        Reviewed by Mark Rowe.

        - fix https://bugs.webkit.org/show_bug.cgi?id=21557
          Bug 21557: REGRESSION: generated-layer-scrollbar-crash.html is crashing

        * rendering/RenderObject.cpp:
        (WebCore::RenderObject::getPseudoStyle): Keep walking up the parent chain
        until we hit an Element. Nowadays, Node::isElementNode() is just as efficient
        as RenderObject::isText(), and this guarantees that the node can be cast to
        Element*, which is done just below. The reason this is hit is that RenderLayer
        can call getPseudoStyle on a RenderView, which has the document as its
        element() -- and a document is neither a Text node nor an Element. A loose
        end is that this seems to mean you can't style the scrollbar corner or the
        resizer on a RenderView. Hyatt may want to look into that later.

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

11 years ago2008-10-12 Darin Adler <darin@apple.com>
darin@apple.com [Sun, 12 Oct 2008 22:51:13 +0000 (22:51 +0000)]
2008-10-12  Darin Adler  <darin@apple.com>

        Reviewed by Sam Weinig.

        * wtf/unicode/icu/UnicodeIcu.h: Fixed indentation to match WebKit coding style.
        * wtf/unicode/qt4/UnicodeQt4.h: Ditto.

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

11 years agoJavaScriptCore:
darin@apple.com [Sun, 12 Oct 2008 22:49:46 +0000 (22:49 +0000)]
JavaScriptCore:

2008-10-12  Darin Adler  <darin@apple.com>

        Reviewed by Sam Weinig.

        - https://bugs.webkit.org/show_bug.cgi?id=21556
          Bug 21556: non-ASCII digits are allowed in places where only ASCII should be

        * wtf/unicode/icu/UnicodeIcu.h: Removed isDigit, digitValue, and isFormatChar.
        * wtf/unicode/qt4/UnicodeQt4.h: Ditto.

WebCore:

2008-10-12  Darin Adler  <darin@apple.com>

        Reviewed by Sam Weinig.

        - https://bugs.webkit.org/show_bug.cgi?id=21556
          Bug 21556: non-ASCII digits are allowed in places where only ASCII should be

        Tests: fast/dom/HTMLFontElement/size-attribute.html
               fast/dom/HTMLInputElement/size-attribute.html
               fast/dom/HTMLTableElement/cellpadding-attribute.html

        * html/HTMLFontElement.cpp:
        (WebCore::parseFontSizeNumber): Use isASCIIDigit instead of
        Unicode::isDigit, since non-ASCII digits are not valid here.
        * html/HTMLFontElement.h: Tweaked formatting a bit and removed
        the unneeded destructor declaration.
        * platform/text/String.cpp:
        (WebCore::lengthOfCharactersAsInteger): Use isASCIIDigit instead
        of Unicode::isDigit, since non-ASCII digits are not valid.

LayoutTests:

2008-10-12  Darin Adler  <darin@apple.com>

        Reviewed by Sam Weinig.

        - tests for https://bugs.webkit.org/show_bug.cgi?id=21556
          Bug 21556: non-ASCII digits are allowed in places where only ASCII should be

        The fix was in a function used all over the place. I set a breakpoint in the
        function and made some regression tests for the first few places it was used.

        * fast/dom/HTMLFontElement: Added.
        * fast/dom/HTMLFontElement/resources: Added.
        * fast/dom/HTMLFontElement/resources/TEMPLATE.html: Added.
        * fast/dom/HTMLFontElement/resources/size-attribute.js: Added.
        * fast/dom/HTMLFontElement/size-attribute-expected.txt: Added.
        * fast/dom/HTMLFontElement/size-attribute.html: Added.
        * fast/dom/HTMLInputElement/resources/size-attribute.js: Added.
        * fast/dom/HTMLInputElement/size-attribute-expected.txt: Added.
        * fast/dom/HTMLInputElement/size-attribute.html: Added.
        * fast/dom/HTMLTableElement/cellpadding-attribute-expected.txt: Added.
        * fast/dom/HTMLTableElement/cellpadding-attribute.html: Added.
        * fast/dom/HTMLTableElement/resources/cellpadding-attribute.js: Added.

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

11 years agoWebCore:
darin@apple.com [Sun, 12 Oct 2008 22:38:23 +0000 (22:38 +0000)]
WebCore:

2008-10-12  Jungshik Shin  <jshin@chromium.org>

        Reviewed by Darin Adler.

        Treat x-user-defined as windows-1252 for html files with meta charset
        declaration to be compatible with Indian web sites.
        ( https://bugs.webkit.org/show_bug.cgi?id=18270 )

        Test: fast/encoding/charset-xuser-defined.html

        * loader/TextResourceDecoder.cpp:
        (WebCore::TextResourceDecoder::setEncoding):

LayoutTests:

2008-10-12  Jungshik Shin  <jshin@chromium.org>

        Reviewed by Darin Adler.

        Treat x-user-defined as windows-1252 for html files with meta charset
        declaration to be compatible with Indian web sites.
        ( https://bugs.webkit.org/show_bug.cgi?id=18270 )

        * fast/encoding/charset-xuser-defined-expected.txt: Added.
        * fast/encoding/charset-xuser-defined.html: Added.

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

11 years ago2008-10-12 Darin Adler <darin@apple.com>
darin@apple.com [Sun, 12 Oct 2008 22:24:25 +0000 (22:24 +0000)]
2008-10-12  Darin Adler  <darin@apple.com>

        Reviewed by Dan Bernstein.

        - https://bugs.webkit.org/show_bug.cgi?id=21559
          Bug 21559: many SVG tests are slightly different from what make-js-test-wrappers would make

        * svg/custom/immutable-properties.html: Updated path to test resources so it
        does not go up out of and back into LayoutTests.
        * svg/custom/svg-modify-currentTranslate.html: Ditto.
        * svg/custom/svg-xml-dom-sync.html: Ditto.
        * svg/custom/tearoffs-with-tearoffs.html: Ditto.

        * svg/dynamic-updates/resources/*.js: Allow the make-js-test-wrappers script to
        add var successfullyParsed = true; to these files. It's not helpful for the tests,
        but it's not harmful either.

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

11 years ago2008-10-12 Sam Weinig <sam@webkit.org>
weinig@apple.com [Sun, 12 Oct 2008 21:32:38 +0000 (21:32 +0000)]
2008-10-12  Sam Weinig  <sam@webkit.org>

        Reviewed by Dan Bernstein.

        Remove unneeded forward declaration.

        * platform/graphics/ImageBuffer.h:

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

11 years ago2008-10-12 Sam Weinig <sam@webkit.org>
weinig@apple.com [Sun, 12 Oct 2008 20:43:15 +0000 (20:43 +0000)]
2008-10-12  Sam Weinig  <sam@webkit.org>

        Reviewed by Darin Adler.

        Fix for https://bugs.webkit.org/show_bug.cgi?id=21560
        Layering violation: String should not be responsible for creating Lengths

        It was a layering violation for String to know haw to parse
        into Lengths, LengthArrays, and CoordsArrays.

        * GNUmakefile.am:
        * WebCore.pro:
        * WebCore.vcproj/WebCore.vcproj:
        * WebCore.xcodeproj/project.pbxproj:
        * WebCoreSources.bkl:
        * html/HTMLAreaElement.cpp:
        (WebCore::HTMLAreaElement::parseMappedAttribute):
        * html/HTMLFrameSetElement.cpp:
        (WebCore::HTMLFrameSetElement::parseMappedAttribute):
        * platform/text/AtomicString.h:
        (WebCore::AtomicString::percentage):
        * platform/text/PlatformString.h:
        * platform/text/String.cpp:
        * platform/text/StringImpl.cpp:
        * platform/text/StringImpl.h:
        * rendering/Length.cpp: Added.
        (WebCore::parseLength):
        (WebCore::countCharacter):
        (WebCore::newCoordsArray):
        (WebCore::newLengthArray):
        * rendering/Length.h:
        (WebCore::LengthSize::LengthSize):

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

11 years ago2008-10-12 Anders Carlsson <andersca@apple.com>
andersca@apple.com [Sun, 12 Oct 2008 19:51:33 +0000 (19:51 +0000)]
2008-10-12  Anders Carlsson  <andersca@apple.com>

        Reviewed by Darin Adler.

        Make the append method that takes a Vector more strict - it now requires the elements
        of the vector to be appended same type as the elements of the Vector they're being appended to.

        This would cause problems when dealing with Vectors containing other Vectors.

        * wtf/Vector.h:
        (WTF::::append):

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

11 years ago2008-10-12 Darin Adler <darin@apple.com>
darin@apple.com [Sun, 12 Oct 2008 19:34:42 +0000 (19:34 +0000)]
2008-10-12  Darin Adler  <darin@apple.com>

        * Scripts/make-js-test-wrappers: Update exceptions list so the script
        won't overwrite files and mangle tests.

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

11 years agoWebCore:
darin@apple.com [Sun, 12 Oct 2008 18:23:56 +0000 (18:23 +0000)]
WebCore:

2008-10-12  Brad Garcia  <bgarcia@google.com>

        Reviewed by Darin Adler.

        Interpret fractional percentage values for %MultiLength values.
        Fix for https://bugs.webkit.org/show_bug.cgi?id=3591
        Test: fast/frames/frame-length-fractional-percentage.html

        * platform/text/StringImpl.cpp:
        (WebCore::parseLength):

LayoutTests:

2008-10-12  Brad Garcia  <bgarcia@google.com>

        Reviewed by Darin Adler.

        Testcases for fractional frame lengths.
        https://bugs.webkit.org/show_bug.cgi?id=3591

        * fast/frames/frame-length-fractional.html: Added.
        * fast/frames/frame-length-fractional-expected.txt: Added.

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

11 years ago Remember the expanded state of the rules in the Styles pane. So
timothy@apple.com [Sun, 12 Oct 2008 17:04:51 +0000 (17:04 +0000)]
    Remember the expanded state of the rules in the Styles pane. So
        collapsing a rule will persist across all the elements that have
        that same rule. The same applies to Computed Style, so the Computed
        Style rule is now collapsed by default. This should help lessen the
        confusion on why Computed Style can't be edited (a common confusion.)

        https://bugs.webkit.org/show_bug.cgi?id=21553

        Reviewed by Darin Adler.

        * inspector/front-end/PropertiesSection.js:
        (WebInspector.PropertiesSection): Set the _expanded property directly,
        instead of using the setter. Expanded is the default, so no work is needed.
        This prevents calling StylesSidebarPane.collapse before object initialization
        is complete.
        * inspector/front-end/StylesSidebarPane.js:
        (WebInspector.StylesSidebarPane.prototype.update): Check the Preferences object
        for the expanded state of the StylePropertiesSection. Collapse computed style
        by default and expand everything else.
        (WebInspector.StylePropertiesSection): Make a semi unique identifier to be used
        when remembering expanded state.
        (WebInspector.StylePropertiesSection.prototype.expand): Set the expanded state
        to true for this identifier in Preferences.styleRulesExpandedState. Don't remember
        the state if the dontRememberState property is true.
        (WebInspector.StylePropertiesSection.prototype.collapse): Set the expanded state
        to false for this identifier in Preferences.styleRulesExpandedState.
        * inspector/front-end/inspector.js:
        (Preferences.styleRulesExpandedState): Initialize to an empty object.

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

11 years ago2008-10-12 Jan Michael Alonzo <jmalonzo@webkit.org>
jmalonzo@webkit.org [Sun, 12 Oct 2008 11:47:04 +0000 (11:47 +0000)]
2008-10-12  Jan Michael Alonzo  <jmalonzo@webkit.org>

        Reviewed by Alp Toker.

        [Gtk] Remove libWebCoreJS.la when doing make clean
        https://bugs.webkit.org/show_bug.cgi?id=21544

        Remove libWebCoreJS.la when doing a clean target

        * GNUmakefile.am: add libWebCoreJS.la to CLEANFILES

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

11 years ago2008-10-11 Alp Toker <alp@nuanti.com>
alp@webkit.org [Sun, 12 Oct 2008 06:50:21 +0000 (06:50 +0000)]
2008-10-11  Alp Toker  <alp@nuanti.com>

        Reviewed by Darin Adler.

        https://bugs.webkit.org/show_bug.cgi?id=20592
        The focus ring is not shown while navigating on some pages

        Fix drawFocusRing() so it doesn't depend on the current graphics
        state, thus matching other platforms which already work this way.

        This patch provides two code paths: one using straight Cairo and
        one specialised for the GTK+ port with a more conventional appearance.

        Also change focusRingColor() to return a null color, allowing the
        focus ring to pick up the style's current color if it isn't overridden
        with CSS.

        * platform/graphics/cairo/GraphicsContextCairo.cpp:
        (WebCore::GraphicsContext::drawFocusRing):
        * platform/gtk/TemporaryLinkStubs.cpp:
        (WebCore::focusRingColor):

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

11 years ago2008-10-11 Cameron Zwarich <zwarich@apple.com>
cwzwarich@webkit.org [Sun, 12 Oct 2008 05:00:28 +0000 (05:00 +0000)]
2008-10-11  Cameron Zwarich  <zwarich@apple.com>

        Reviewed by Sam Weinig.

        Clean up RegExpMatchesArray.h to match our coding style.

        * kjs/RegExpMatchesArray.h:
        (JSC::RegExpMatchesArray::getOwnPropertySlot):
        (JSC::RegExpMatchesArray::put):
        (JSC::RegExpMatchesArray::deleteProperty):
        (JSC::RegExpMatchesArray::getPropertyNames):

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

11 years ago Rubber-stamped by Sam Weinig.
mitz@apple.com [Sun, 12 Oct 2008 04:56:20 +0000 (04:56 +0000)]
    Rubber-stamped by Sam Weinig.

        - change all internal callers to use Cache::evict() instead of remove()

        * loader/Cache.cpp:
        (WebCore::Cache::revalidateResource):
        (WebCore::Cache::revalidationSucceeded):
        (WebCore::Cache::pruneDeadResources):
        (WebCore::Cache::setDisabled):

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

11 years ago2008-10-11 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Sun, 12 Oct 2008 04:26:47 +0000 (04:26 +0000)]
2008-10-11  David Hyatt  <hyatt@apple.com>

        Make sure invalidatePart knows how to invalidate the track and scrollbar backgrounds.

        Reviewed by Tim Hatcher

        * platform/ScrollbarThemeComposite.cpp:
        (WebCore::ScrollbarThemeComposite::invalidatePart):

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

11 years ago2008-10-11 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Sun, 12 Oct 2008 04:20:26 +0000 (04:20 +0000)]
2008-10-11  David Hyatt  <hyatt@apple.com>

        https://bugs.webkit.org/show_bug.cgi?id=21549 (regression in before/after track pieces).

        Also fixed resizer positioning problems.

        Reviewed by Tim Hatcher

        * platform/ScrollbarThemeComposite.cpp:
        (WebCore::ScrollbarThemeComposite::splitTrack):
        (WebCore::ScrollbarThemeComposite::trackPosition):
        (WebCore::ScrollbarThemeComposite::trackLength):
        * platform/ScrollbarThemeComposite.h:
        (WebCore::ScrollbarThemeComposite::constrainTrackRectToTrackPieces):
        * rendering/RenderLayer.cpp:
        (WebCore::scrollCornerRect):
        (WebCore::RenderLayer::positionOverflowControls):
        (WebCore::RenderLayer::paintOverflowControls):
        (WebCore::RenderLayer::isPointInResizeControl):
        (WebCore::RenderLayer::hitTestOverflowControls):
        * rendering/RenderScrollbarTheme.cpp:
        (WebCore::RenderScrollbarTheme::constrainTrackRectToTrackPieces):
        * rendering/RenderScrollbarTheme.h:

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

11 years ago2008-10-11 Alp Toker <alp@nuanti.com>
alp@webkit.org [Sun, 12 Oct 2008 03:33:28 +0000 (03:33 +0000)]
2008-10-11  Alp Toker  <alp@nuanti.com>

        Reviewed by David Hyatt.

        Implement release/retain for Widget GTK+.

        * platform/Widget.cpp:
        * platform/gtk/WidgetGtk.cpp:
        (WebCore::Widget::~Widget):
        (WebCore::Widget::releasePlatformWidget):
        (WebCore::Widget::retainPlatformWidget):

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

11 years agoFix ChangeLog header that was mangled in r37484.
mrowe@apple.com [Sun, 12 Oct 2008 01:38:08 +0000 (01:38 +0000)]
Fix ChangeLog header that was mangled in r37484.

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

11 years agoDisable animations/font-size-using-ems.html as it has been causing the following...
mrowe@apple.com [Sun, 12 Oct 2008 01:31:19 +0000 (01:31 +0000)]
Disable animations/font-size-using-ems.html as it has been causing the following test, animatinos/generic-from-to.html, to hang since it was added.

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

11 years ago Reviewed by Sam Weinig and Anders Carlsson.
mitz@apple.com [Sat, 11 Oct 2008 23:10:43 +0000 (23:10 +0000)]
    Reviewed by Sam Weinig and Anders Carlsson.

        - update Mac DumpRenderTree to use the new WebView SPI for forcing the
          complex text code path
        - add --complex-text support to Windows DumpRenderTree

        * DumpRenderTree/mac/DumpRenderTree.mm:
        (dumpRenderTree):
        * DumpRenderTree/win/DumpRenderTree.cpp:
        (createWebViewAndOffscreenWindow):
        (main):
        * Scripts/run-webkit-tests:

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

11 years agoWebCore:
mitz@apple.com [Sat, 11 Oct 2008 22:53:28 +0000 (22:53 +0000)]
WebCore:

        Reviewed by Sam Weinig.

        - rename WebCoreSetAlwaysUseATSU to WebCoreSetAlwaysUsesComplexTextCodePath
          and add a corresponding getter; add the same functions on Windows

        * WebCore.base.exp:
        * platform/graphics/Font.cpp:
        (WebCore::Font::setCodePath):
        (WebCore::codePath):
        (WebCore::Font::canUseGlyphCache):
        * platform/graphics/Font.h:
        * platform/mac/WebCoreTextRenderer.h:
        * platform/mac/WebCoreTextRenderer.mm:
        (WebCoreSetAlwaysUsesComplexTextCodePath):
        (WebCoreAlwaysUsesComplexTextCodePath):
        * platform/win/WebCoreTextRenderer.cpp:
        (WebCore::WebCoreSetAlwaysUsesComplexTextCodePath):
        (WebCore::WebCoreAlwaysUsesComplexTextCodePath):
        * platform/win/WebCoreTextRenderer.h:

WebKit/mac:

        Reviewed by Sam Weinig.

        - rename _setAlwaysUseATSU to _setAlwaysUsesComplexTextCodePath and
          update it for the renamed WebCoreTextRenderer method; keep the old
          method around for clients that use it

        * WebView/WebView.mm:
        (+[WebView _setAlwaysUseATSU:]):
        (+[WebView _setAlwaysUsesComplexTextCodePath:]):
        * WebView/WebViewPrivate.h:

WebKit/win:

        Reviewed by Sam Weinig.

        - add SPI for forcing the complex text code path to be used for all text

        * Interfaces/IWebViewPrivate.idl:
        * WebView.cpp:
        (WebView::setAlwaysUsesComplexTextCodePath):
        (WebView::alwaysUsesComplexTextCodePath):
        * WebView.h:

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

11 years ago2008-10-11 Sam Weinig <sam@webkit.org>
weinig@apple.com [Sat, 11 Oct 2008 22:20:09 +0000 (22:20 +0000)]
2008-10-11  Sam Weinig  <sam@webkit.org>

        Reviewed by Cameron Zwarich.

        Stop leaking the event parameter names.  The protected strings were showing up
        in the caches window after one use.  No measurable performance loss.

        * bindings/js/JSEventListener.cpp:
        (WebCore::eventParameterName):

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

11 years ago2008-10-11 Sam Weinig <sam@webkit.org>
weinig@apple.com [Sat, 11 Oct 2008 22:10:38 +0000 (22:10 +0000)]
2008-10-11  Sam Weinig  <sam@webkit.org>

        Reviewed by Cameron Zwarich.

        Remove unused function getNodeEventListener.

        * bindings/js/JSEventListener.cpp:
        * bindings/js/JSEventListener.h:

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

11 years ago2008-10-11 Cameron Zwarich <zwarich@apple.com>
cwzwarich@webkit.org [Sat, 11 Oct 2008 20:58:35 +0000 (20:58 +0000)]
2008-10-11  Cameron Zwarich  <zwarich@apple.com>

        Reviewed by Sam Weinig.

        Bug 21525: 55 StructureID leaks on Wikitravel's main page
        <https://bugs.webkit.org/show_bug.cgi?id=21525>

        Bug 21533: Simple JavaScript code leaks StructureIDs
        <https://bugs.webkit.org/show_bug.cgi?id=21533>

        StructureID::getEnumerablePropertyNames() ends up calling back to itself
        via JSObject::getPropertyNames(), which causes the PropertyNameArray to
        be cached twice. This leads to a memory leak in almost every use of
        JSObject::getPropertyNames() on an object. The fix here is based on a
        suggestion of Sam Weinig.

        This patch also fixes every StructureID leaks that occurs while running
        the Mozilla MemBuster test.

        * kjs/PropertyNameArray.h:
        (JSC::PropertyNameArray::PropertyNameArray):
        (JSC::PropertyNameArray::setCacheable):
        (JSC::PropertyNameArray::cacheable):
        * kjs/StructureID.cpp:
        (JSC::StructureID::getEnumerablePropertyNames):

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

11 years ago2008-10-11 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Sat, 11 Oct 2008 07:50:03 +0000 (07:50 +0000)]
2008-10-11  David Hyatt  <hyatt@apple.com>

        Add support for both positive and negative margins to tracks and track pieces.

        Reviewed by Tim Hatcher

        * platform/ScrollbarThemeComposite.cpp:
        (WebCore::ScrollbarThemeComposite::paint):
        (WebCore::ScrollbarThemeComposite::hitTest):
        (WebCore::ScrollbarThemeComposite::splitTrack):
        (WebCore::ScrollbarThemeComposite::trackPieceRects):
        (WebCore::ScrollbarThemeComposite::trackPosition):
        (WebCore::ScrollbarThemeComposite::trackLength):
        * platform/ScrollbarThemeComposite.h:
        (WebCore::ScrollbarThemeComposite::adjustTrackPieceRect):
        * rendering/RenderScrollbar.cpp:
        (WebCore::RenderScrollbar::trackRect):
        (WebCore::RenderScrollbar::trackPieceRectWithMargins):
        * rendering/RenderScrollbar.h:
        * rendering/RenderScrollbarPart.cpp:
        (WebCore::RenderScrollbarPart::computeScrollbarWidth):
        (WebCore::RenderScrollbarPart::computeScrollbarHeight):
        * rendering/RenderScrollbarTheme.cpp:
        (WebCore::RenderScrollbarTheme::adjustTrackPieceRect):
        * rendering/RenderScrollbarTheme.h:

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

11 years agoUpdate scrollbars layout tests to new syntax.
hyatt@apple.com [Sat, 11 Oct 2008 06:33:35 +0000 (06:33 +0000)]
Update scrollbars layout tests to new syntax.

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

11 years ago2008-10-10 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Sat, 11 Oct 2008 06:30:47 +0000 (06:30 +0000)]
2008-10-10  David Hyatt  <hyatt@apple.com>

        Drop -webkit-scrollbar- from all the scrollbar pseudo-classes, since they are scoped to
        the scrollbar pseudo-elements anyway.

        Reviewed by Tim Hatcher

        * css/CSSSelector.cpp:
        (WebCore::CSSSelector::extractPseudoType):
        * css/CSSSelector.h:
        (WebCore::CSSSelector::):
        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::SelectorChecker::checkScrollbarPseudoClass):

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

11 years ago Reviewed by Sam Weinig.
mitz@apple.com [Sat, 11 Oct 2008 06:26:59 +0000 (06:26 +0000)]
    Reviewed by Sam Weinig.

        - separate Cache::remove() into a private method and a public interface,
          then make Cache::pruneDeadResources() use the private method.

        * loader/Cache.cpp:
        (WebCore::Cache::pruneDeadResources):
        (WebCore::Cache::evict):
        * loader/Cache.h:
        (WebCore::Cache::remove):

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

11 years ago2008-10-10 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Sat, 11 Oct 2008 06:12:14 +0000 (06:12 +0000)]
2008-10-10  David Hyatt  <hyatt@apple.com>

        Fix a math error when splitting a vertical track.  Clean up more of ScrollbarThemeComposite
        now that the track pieces meet under the thumb.

        Reviewed by Tim Hatcher

        * platform/ScrollbarThemeComposite.cpp:
        (WebCore::ScrollbarThemeComposite::paint):
        (WebCore::ScrollbarThemeComposite::splitTrack):

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

11 years ago2008-10-10 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Sat, 11 Oct 2008 06:04:02 +0000 (06:04 +0000)]
2008-10-10  David Hyatt  <hyatt@apple.com>

        Make the track pieces of a scrollbar extend under the thumb (each gets half the thumb).

        Reviewed by Tim Hatcher

        * platform/ScrollbarThemeComposite.cpp:
        (WebCore::ScrollbarThemeComposite::hitTest):
        (WebCore::ScrollbarThemeComposite::splitTrack):

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

11 years ago2008-10-10 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Sat, 11 Oct 2008 05:54:35 +0000 (05:54 +0000)]
2008-10-10  David Hyatt  <hyatt@apple.com>

        Make sure specialized ::scrollbar rules are still triggering custom scrollbar creation.

        Reviewed by Tim Hatcher

        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::createScrollbar):
        * rendering/RenderScrollbar.cpp:
        (WebCore::RenderScrollbar::createCustomScrollbar):
        (WebCore::RenderScrollbar::RenderScrollbar):
        (WebCore::RenderScrollbar::setParent):
        (WebCore::RenderScrollbar::updateScrollbarParts):
        (WebCore::RenderScrollbar::updateScrollbarPart):
        * rendering/RenderScrollbar.h:
        * rendering/style/RenderStyle.h:
        (WebCore::RenderStyle::):

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

11 years agoUse fastcall calling convention on GCC > 4.0
oliver@apple.com [Sat, 11 Oct 2008 05:49:39 +0000 (05:49 +0000)]
Use fastcall calling convention on GCC > 4.0

Reviewed by Cameron Zwarich

Results in a 2-3% improvement in GCC 4.2 performance, so
that it is no longer a regression vs. GCC 4.0

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

11 years ago2008-10-10 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Sat, 11 Oct 2008 05:30:39 +0000 (05:30 +0000)]
2008-10-10  David Hyatt  <hyatt@apple.com>

        Make sure scrollbar background images cause proper part invalidation to occur.

        Reviewed by Tim Hatcher

        * rendering/RenderScrollbar.cpp:
        (WebCore::RenderScrollbar::setHoveredPart):
        (WebCore::RenderScrollbar::setPressedPart):
        * rendering/RenderScrollbarPart.cpp:
        (WebCore::RenderScrollbarPart::styleDidChange):
        (WebCore::RenderScrollbarPart::imageChanged):
        * rendering/RenderScrollbarPart.h:

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

11 years agoJavaScriptCore:
weinig@apple.com [Sat, 11 Oct 2008 02:53:03 +0000 (02:53 +0000)]
JavaScriptCore:

2008-10-10  Sam Weinig  <sam@webkit.org>

        Reviewed by Darin Adler.

        - Add a workaround for a bug in ceil in Darwin libc.
        - Remove old workarounds for JS math functions that are not needed
          anymore.

        The math functions are heavily tested by fast/js/math.html.

        * kjs/MathObject.cpp:
        (JSC::mathProtoFuncAbs): Remove workaround.
        (JSC::mathProtoFuncCeil): Ditto.
        (JSC::mathProtoFuncFloor): Ditto.
        * wtf/MathExtras.h:
        (wtf_ceil): Add ceil workaround for darwin.

LayoutTests:

2008-10-10  Sam Weinig  <sam@webkit.org>

        Reviewed by Darin Adler.

        - Update tests after fixing ceil(-0.1) behavior on darwin.

        * fast/js/math-expected.txt:
        * fast/js/resources/math.js: Remove code to detect sign of 0 now that the harness can handle it.
        * fast/xpath/4XPath/Core/test_core_functions-expected.txt:
        * fast/xpath/4XPath/Core/test_core_functions.html: Update test expect correct result of -0.

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

11 years ago2008-10-10 Sam Weinig <sam@webkit.org>
weinig@apple.com [Sat, 11 Oct 2008 02:05:08 +0000 (02:05 +0000)]
2008-10-10  Sam Weinig  <sam@webkit.org>

        Reviewed by Darin Adler

        Add Assertions to JSObject constructor.

        * kjs/JSObject.h:
        (JSC::JSObject::JSObject):

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

11 years ago2008-10-10 Simon Fraser <simon.fraser@apple.com>
simon.fraser@apple.com [Sat, 11 Oct 2008 00:35:23 +0000 (00:35 +0000)]
2008-10-10  Simon Fraser  <simon.fraser@apple.com>

        Reviewed by Darin Adler

        https://bugs.webkit.org/show_bug.cgi?id=21538

        The blendFunc() for TransformOperations should never mess with the
        operation lists by replacing some ops with identity, otherwise the lists
        no longer match and transitions break.

        Test: transitions/matched-transform-functions.html

        * page/animation/AnimationBase.cpp:
        (WebCore::blendFunc):

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

11 years ago2008-10-10 Simon Fraser <simon.fraser@apple.com>
simon.fraser@apple.com [Sat, 11 Oct 2008 00:32:39 +0000 (00:32 +0000)]
2008-10-10  Simon Fraser  <simon.fraser@apple.com>

        Reviewed by Darin Adler

        https://bugs.webkit.org/show_bug.cgi?id=21537

        Fix re-targetting a running transition; if the target property
        changes, we do need to make a new ImplicitAnimation.

        Test: transitions/retargetted-transition.html

        * page/animation/CompositeAnimation.cpp:
        (WebCore::CompositeAnimationPrivate::updateTransitions):

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

11 years agoPart of <rdar://problem/6278147> Cannot stop a slow script -- Fix assertion
oliver@apple.com [Fri, 10 Oct 2008 23:21:39 +0000 (23:21 +0000)]
Part of <rdar://problem/6278147> Cannot stop a slow script -- Fix assertion

Reviewed by Cameron Zwarich

When JS is terminated by the user choosing to stop a slow script
the exception toString method throws, yet the console logger does
not perform exception checks after logging.  This leaves an
exception on the Machine, which in turn results in an assertion
failure next time we attempt to enter JS.  The solution is just to
clear the exception state after attempting to record the exception.

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

11 years ago2008-10-10 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Fri, 10 Oct 2008 22:24:15 +0000 (22:24 +0000)]
2008-10-10  David Hyatt  <hyatt@apple.com>

        Make textareas work with custom CSS scrollbars and resizers.  Need to access the correct renderer
        when looking for pseudo styles.

        Reviewed by Tim Hatcher

        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::createScrollbar):
        (WebCore::RenderLayer::styleChanged):

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

11 years ago2008-10-10 Dean Jackson <dino@apple.com>
dino@apple.com [Fri, 10 Oct 2008 22:19:16 +0000 (22:19 +0000)]
2008-10-10  Dean Jackson  <dino@apple.com>

        Keyframe example had "0" rather than "0%".

        * WebKitSite/specs/CSSVisualEffects/CSSAnimations.html:

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

11 years ago2008-10-10 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Fri, 10 Oct 2008 22:16:24 +0000 (22:16 +0000)]
2008-10-10  David Hyatt  <hyatt@apple.com>

        Add the ability to style the scroll corner and the resizer from CSS.

        Reviewed by Tim Hatcher

        * css/CSSSelector.cpp:
        (WebCore::CSSSelector::extractPseudoType):
        * css/CSSSelector.h:
        (WebCore::CSSSelector::):
        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):
        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::RenderLayer):
        (WebCore::RenderLayer::~RenderLayer):
        (WebCore::RenderLayer::paintOverflowControls):
        (WebCore::RenderLayer::paintScrollCorner):
        (WebCore::RenderLayer::paintResizer):
        (WebCore::RenderLayer::styleChanged):
        * rendering/RenderLayer.h:
        * rendering/RenderScrollbar.cpp:
        (WebCore::RenderScrollbar::updateScrollbarPart):
        (WebCore::RenderScrollbar::paintPart):
        * rendering/RenderScrollbarPart.cpp:
        (WebCore::RenderScrollbarPart::RenderScrollbarPart):
        (WebCore::RenderScrollbarPart::paintIntoRect):
        * rendering/RenderScrollbarPart.h:
        * rendering/style/RenderStyle.h:
        (WebCore::RenderStyle::):

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

11 years ago2008-10-10 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Fri, 10 Oct 2008 21:05:07 +0000 (21:05 +0000)]
2008-10-10  David Hyatt  <hyatt@apple.com>

        Make CSS scrollbars respect the OS setting regarding where buttons should be placed.

        Reviewed by Sam Weinig

        No test case possible, since the OS setting can vary.

        * css/CSSSelector.cpp:
        (WebCore::CSSSelector::extractPseudoType):
        * css/CSSSelector.h:
        (WebCore::CSSSelector::):
        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::SelectorChecker::checkScrollbarPseudoClass):
        * platform/Scrollbar.h:
        (WebCore::Scrollbar::styleChanged):
        * platform/mac/ScrollbarThemeMac.mm:
        (+[ScrollbarPrefsObserver appearancePrefsChanged:]):
        (WebCore::ScrollbarThemeMac::preferencesChanged):
        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::invalidateScrollbarRect):
        * rendering/RenderScrollbar.cpp:
        (WebCore::RenderScrollbar::styleChanged):
        (WebCore::RenderScrollbar::updateScrollbarPart):
        * rendering/RenderScrollbar.h:
        * rendering/RenderScrollbarTheme.h:
        (WebCore::RenderScrollbarTheme::buttonsPlacement):

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

11 years ago Bug 21526: The hashmap webPreferencesInstances in WebPreferences.cpp can
adachan@apple.com [Fri, 10 Oct 2008 20:43:01 +0000 (20:43 +0000)]
    Bug 21526: The hashmap webPreferencesInstances in WebPreferences.cpp can
        potentially hold onto stale instances of WebPreferences
        https://bugs.webkit.org/show_bug.cgi?id=21526

        webPreferencesInstances now keeps a reference to all the instances it holds.
        Fix up callers of WebPreferences::removeReferenceForIdentifier() to call
        it after they have released their references to the WebPreferences objects.

        Reviewed by Adam Roben.

        * WebPreferences.cpp:
        (WebPreferences::getInstanceForIdentifier):
        (WebPreferences::removeReferenceForIdentifier):
        * WebView.cpp:
        (WebView::close):
        (WebView::setPreferences):

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

11 years ago Fixes a regression with input search fields, where the placeholder text
timothy@apple.com [Fri, 10 Oct 2008 20:27:51 +0000 (20:27 +0000)]
    Fixes a regression with input search fields, where the placeholder text
        would not be updated when the value of the placeholder attribute changed.

        https://bugs.webkit.org/show_bug.cgi?id=21521

        Reviewed by Adele Peterson.

        Test: fast/forms/search-placeholder-value-changed.html

        * html/HTMLInputElement.cpp:
        (WebCore::HTMLInputElement::parseMappedAttribute): Pass true for the
        placeholderValueChanged parameter of updatePlaceholderVisibility.
        (WebCore::HTMLInputElement::updatePlaceholderVisibility): Add parameter
        for placeholderValueChanged that informs the method that the value changed.
        * html/HTMLInputElement.h: Add the placeholderValueChanged to the
        updatePlaceholderVisibility method. Made it default to false.

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

11 years ago2008-10-10 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Fri, 10 Oct 2008 20:12:14 +0000 (20:12 +0000)]
2008-10-10  David Hyatt  <hyatt@apple.com>

        Allow the track part to have negative margins so that it can slightly overlap buttons.  This
        is necessary to emulate the Aqua look in CSS.

        Reviewed by Sam Weinig

        * platform/ScrollbarThemeComposite.cpp:
        (WebCore::ScrollbarThemeComposite::paint):
        (WebCore::ScrollbarThemeComposite::hitTest):
        * rendering/RenderScrollbar.cpp:
        (WebCore::RenderScrollbar::setHoveredPart):
        (WebCore::RenderScrollbar::trackRect):
        * rendering/RenderScrollbar.h:
        * rendering/RenderScrollbarPart.cpp:
        (WebCore::RenderScrollbarPart::computeScrollbarWidth):
        (WebCore::RenderScrollbarPart::computeScrollbarHeight):
        * rendering/RenderScrollbarTheme.cpp:
        (WebCore::RenderScrollbarTheme::backButtonRect):
        (WebCore::RenderScrollbarTheme::forwardButtonRect):
        (WebCore::RenderScrollbarTheme::trackRect):

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

11 years ago2008-10-10 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Fri, 10 Oct 2008 19:32:52 +0000 (19:32 +0000)]
2008-10-10  David Hyatt  <hyatt@apple.com>

        https://bugs.webkit.org/show_bug.cgi?id=21527

        Make CSS scrollbars support :not, :hover and :active.

        Reviewed by Adam Roben

        Added WebCore/manual-tests/scrollbar-hover-active.html

        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):
        (WebCore::CSSStyleSelector::SelectorChecker::checkScrollbarPseudoClass):
        * platform/Scrollbar.cpp:
        (WebCore::Scrollbar::autoscrollPressedPart):
        (WebCore::Scrollbar::startTimerIfNeeded):
        (WebCore::Scrollbar::setHoveredPart):
        (WebCore::Scrollbar::setPressedPart):
        (WebCore::Scrollbar::mouseMoved):
        (WebCore::Scrollbar::mouseExited):
        (WebCore::Scrollbar::mouseUp):
        (WebCore::Scrollbar::mouseDown):
        * platform/Scrollbar.h:
        * rendering/RenderScrollbar.cpp:
        (WebCore::RenderScrollbar::setHoveredPart):
        (WebCore::RenderScrollbar::setPressedPart):
        (WebCore::RenderScrollbar::updateScrollbarParts):
        (WebCore::pseudoForScrollbarPart):
        (WebCore::RenderScrollbar::updateScrollbarPart):
        * rendering/RenderScrollbar.h:

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

11 years ago2008-10-10 Chris Marrin <cmarrin@apple.com>
cmarrin@apple.com [Fri, 10 Oct 2008 18:43:47 +0000 (18:43 +0000)]
2008-10-10  Chris Marrin  <cmarrin@apple.com>

        Reviewed by Dan Bernstein.

        Fix for https://bugs.webkit.org/show_bug.cgi?id=21025
        CSS transition with duration=0 and delay=0 doesn't override ongoing transition

        Test: transitions/interrupt-zero-duration.html

        * page/animation/CompositeAnimation.cpp:
        (WebCore::CompositeAnimation::updateTransitions):

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

11 years ago2008-10-09 Chris Marrin <cmarrin@apple.com>
cmarrin@apple.com [Fri, 10 Oct 2008 18:04:20 +0000 (18:04 +0000)]
2008-10-09  Chris Marrin  <cmarrin@apple.com>

        Reviewed by Dan Bernstein.

        https://bugs.webkit.org/show_bug.cgi?id=21310

        Redesigned how animation events are sent in order to get rid of
        AnimationEventDispatcher. ImplicitAnimation and KeyframeAnimation
        are now ref counted. While calling the event handler, I keep a
        reference to this class to avoid it getting destroyed out from under me.

        I also moved most of the functionality of CompositeAnimation to
        and internal class (CompositeAnimationPrivate) to reduce the exposure
        of the internals of the various animation classes.

        We have several transition and animation LayoutTests which verify that
        this fix works as expected and continues to avoid crashes.

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

11 years agowx build fix. Add new scrollbar-related sources.
kevino@webkit.org [Fri, 10 Oct 2008 17:13:46 +0000 (17:13 +0000)]
wx build fix. Add new scrollbar-related sources.

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

11 years agoWebCore:
hyatt@apple.com [Fri, 10 Oct 2008 16:59:20 +0000 (16:59 +0000)]
WebCore:

2008-10-10  David Hyatt  <hyatt@apple.com>

        https://bugs.webkit.org/show_bug.cgi?id=21522

        Support increment/decrement/start/end on scrollbars.  Allows a fully functional scrollbar to be
        created.  (Just :hover/:active left to finish it up.)

        Reviewed by Adam Roben

        Added scrollbars/scrollbar-buttons.html

        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::SelectorChecker::checkScrollbarPseudoClass):

LayoutTests:

2008-10-10  David Hyatt  <hyatt@apple.com>

        Add test for scrollbar buttons and track supporting increment/decrement/start/end.

        Reviewed by Adam Roben

        * platform/mac/scrollbars/scrollbar-buttons-expected.checksum: Added.
        * platform/mac/scrollbars/scrollbar-buttons-expected.png: Added.
        * platform/mac/scrollbars/scrollbar-buttons-expected.txt: Added.
        * scrollbars/scrollbar-buttons.html: Added.

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

11 years agoLand updated layout test after pseudoclass rename.
hyatt@apple.com [Fri, 10 Oct 2008 16:40:51 +0000 (16:40 +0000)]
Land updated layout test after pseudoclass rename.

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

11 years ago2008-10-10 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Fri, 10 Oct 2008 16:40:26 +0000 (16:40 +0000)]
2008-10-10  David Hyatt  <hyatt@apple.com>

        Rename some pseudoclasses.  Removing "-scrollbar-" from a bunch of the pseudoclasses.  Renaming
        back/forward to decrement/increment.  Adding start/end pseudoclasses.

        Reviewed by Eric Seidel

        * css/CSSSelector.cpp:
        (WebCore::CSSSelector::extractPseudoType):
        * css/CSSSelector.h:
        (WebCore::CSSSelector::):
        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::SelectorChecker::checkScrollbarPseudoClass):

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

11 years ago2008-10-10 Sam Weinig <sam@webkit.org>
weinig@apple.com [Fri, 10 Oct 2008 07:18:48 +0000 (07:18 +0000)]
2008-10-10  Sam Weinig  <sam@webkit.org>

        Reviewed by Cameron Zwarich.

        Remove now unused m_getterSetterFlag variable from PropertyMap.

        * kjs/PropertyMap.cpp:
        (JSC::PropertyMap::operator=):
        * kjs/PropertyMap.h:
        (JSC::PropertyMap::PropertyMap):

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

11 years agoFix build.
hyatt@apple.com [Fri, 10 Oct 2008 07:15:52 +0000 (07:15 +0000)]
Fix build.

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

11 years ago2008-10-10 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Fri, 10 Oct 2008 07:15:04 +0000 (07:15 +0000)]
2008-10-10  David Hyatt  <hyatt@apple.com>

        Switch window-active to window-inactive, since that will work better as far as degrading in other
        browsers.

        Reviewed by Tim Hatcher

        * css/CSSSelector.cpp:
        (WebCore::CSSSelector::extractPseudoType):
        * css/CSSSelector.h:
        (WebCore::CSSSelector::):
        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::SelectorChecker::checkScrollbarPseudoClass):

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

11 years agoYet another Tiger build fix.
timothy@apple.com [Fri, 10 Oct 2008 07:06:08 +0000 (07:06 +0000)]
Yet another Tiger build fix.

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

11 years agoWebCore:
hyatt@apple.com [Fri, 10 Oct 2008 07:04:38 +0000 (07:04 +0000)]
WebCore:

2008-10-09  David Hyatt  <hyatt@apple.com>

        Add support for scrollbar orientation as a pseudoclass.  Add support for whether or not the scrollbar
        is active as well.

        Reviewed by Tim Hatcher

        Added scrollbars/scrollbar-orientation.html

        * css/CSSSelector.cpp:
        (WebCore::CSSSelector::extractPseudoType):
        * css/CSSSelector.h:
        (WebCore::CSSSelector::):
        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::SelectorChecker::checkScrollbarPseudoClass):
        * platform/Scrollbar.cpp:
        (WebCore::Scrollbar::isWindowActive):
        * platform/Scrollbar.h:
        * rendering/RenderScrollbar.cpp:
        (WebCore::RenderScrollbar::paint):
        (WebCore::RenderScrollbar::partForStyleResolve):
        * rendering/RenderScrollbar.h:

LayoutTests:

2008-10-09  David Hyatt  <hyatt@apple.com>

        Add layout test for scrollbars honoring orientation and window active state.

        Reviewed by Tim Hatcher

        * platform/mac/scrollbars/scrollbar-orientation-expected.checksum: Added.
        * platform/mac/scrollbars/scrollbar-orientation-expected.png: Added.
        * platform/mac/scrollbars/scrollbar-orientation-expected.txt: Added.
        * scrollbars/scrollbar-orientation.html: Added.

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

11 years agoAnother build fix.
timothy@apple.com [Fri, 10 Oct 2008 06:39:45 +0000 (06:39 +0000)]
Another build fix.

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

11 years agoFull build fix for the fail on Tiger.
timothy@apple.com [Fri, 10 Oct 2008 06:12:10 +0000 (06:12 +0000)]
Full build fix for the fail on Tiger.

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

11 years agoWebCore:
hyatt@apple.com [Fri, 10 Oct 2008 05:56:10 +0000 (05:56 +0000)]
WebCore:

2008-10-09  David Hyatt  <hyatt@apple.com>

        Add support for pseudo classes on scrollbar pseudo elements.  As an initial proof of concept only
        :enabled/:disabled are supported.  More pseudo classes will follow quickly now that this works.

        Reviewed by Tim Hatcher

        Added scrollbars/disabled-scrollbar.html

        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::SelectorChecker::checkSelector):
        (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):
        (WebCore::CSSStyleSelector::SelectorChecker::checkScrollbarPseudoClass):
        * css/CSSStyleSelector.h:
        * rendering/RenderObject.cpp:
        (WebCore::RenderObject::getPseudoStyle):
        * rendering/RenderObject.h:
        * rendering/RenderScrollbar.cpp:
        (WebCore::RenderScrollbar::setEnabled):
        (WebCore::RenderScrollbar::scrollbarForStyleResolve):
        (WebCore::RenderScrollbar::getScrollbarPseudoStyle):
        (WebCore::RenderScrollbar::paintPart):
        * rendering/RenderScrollbar.h:

LayoutTests:

2008-10-09  David Hyatt  <hyatt@apple.com>

        Add test case for disabled scrollbars.

        Reviewed by Tim Hatcher

        * platform/mac/scrollbars/disabled-scrollbar-expected.checksum: Added.
        * platform/mac/scrollbars/disabled-scrollbar-expected.png: Added.
        * platform/mac/scrollbars/disabled-scrollbar-expected.txt: Added.
        * scrollbars/disabled-scrollbar.html: Added.

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

11 years agoTiger build fix.
timothy@apple.com [Fri, 10 Oct 2008 04:34:30 +0000 (04:34 +0000)]
Tiger build fix.

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

11 years ago2008-10-09 Eric Carlson <eric.carlson@apple.com>
eric.carlson@apple.com [Fri, 10 Oct 2008 04:04:15 +0000 (04:04 +0000)]
2008-10-09  Eric Carlson  <eric.carlson@apple.com>

        Media controls should not show when element is not visible

        https://bugs.webkit.org/show_bug.cgi?id=21155

        Reviewed by Adele Peterson.

        * html/HTMLMediaElement.cpp:
        (WebCore::HTMLMediaElement::recalcStyle): New, call renderer()->updateFromElement to stay
        in sync with visibility changes
        * html/HTMLMediaElement.h:
        * rendering/RenderMedia.cpp:
        (WebCore::RenderMedia::RenderMedia): initialize m_previousVisible
        (WebCore::RenderMedia::updateControlVisibility): consider style()->visibility() when determining
        if element is visible or not. Don't animate controller visibility when change triggered by
        showing or hiding media element.
        * rendering/RenderMedia.h: declare m_previousVisible

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

11 years agoFix Gtk bustage.
hyatt@apple.com [Fri, 10 Oct 2008 03:42:13 +0000 (03:42 +0000)]
Fix Gtk bustage.

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

11 years agoFix Qt bustage.
hyatt@apple.com [Fri, 10 Oct 2008 03:41:34 +0000 (03:41 +0000)]
Fix Qt bustage.

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

11 years agoFix Windows bustage.
hyatt@apple.com [Fri, 10 Oct 2008 03:37:21 +0000 (03:37 +0000)]
Fix Windows bustage.

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

11 years agoFix typo in my Changelog comment
hyatt@apple.com [Fri, 10 Oct 2008 03:25:37 +0000 (03:25 +0000)]
Fix typo in my Changelog comment

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

11 years agoAnnotate the Objective-C DOM APIs with the JavaScriptCore/WebKitAvailability.h availa...
timothy@apple.com [Fri, 10 Oct 2008 03:23:12 +0000 (03:23 +0000)]
Annotate the Objective-C DOM APIs with the JavaScriptCore/WebKitAvailability.h availability macros and version macros.

WebCore:

2008-10-09  Timothy Hatcher  <timothy@apple.com>

        Annotate the Objective-C DOM APIs with the JavaScriptCore/WebKitAvailability.h
        availability macros and version macros.

        https://bugs.webkit.org/show_bug.cgi?id=21496
        rdar://problem/6259225

        Reviewed by Sam Weinig.

        * bindings/objc/DOMCSS.h: Add a version #if around catgory interface.
        * bindings/objc/DOMEventException.h: Include JavaScriptCore/WebKitAvailability.h
        and add an #if around the enum.
        * bindings/objc/DOMException.h: Ditto. Give the enum a name and remove the
        comment to match the other headers.
        * bindings/objc/DOMExtensions.h: Annotate the methods with version 3.0 or later.
        * bindings/objc/DOMObject.h: Add a version #if around class interface.
        * bindings/objc/DOMRangeException.h: Include JavaScriptCore/WebKitAvailability.h
        and add a version #if around the enum.
        * bindings/objc/DOMSVGException.h: Ditto. Use the latest version since SVG is new.
        * bindings/objc/DOMXPathException.h: Ditto. Use the 3.0 version.
        * bindings/objc/PublicDOMInterfaces.h: Annotate classes and protocols for
        when they where added to WebKit. Also annotate individual methods that are
        deprecated or where added after the class was added to WebKit.
        * bindings/objc/WebScriptObject.h: Include JavaScriptCore/WebKitAvailability.h
        since this header is included by most DOM headers. Annotate the JSObject method.

        * bindings/scripts/CodeGeneratorObjC.pm:
        (ReadPublicInterfaces): Parse out the availability macro for each function/property.
        And parse out the availability version for the class/protocol. Default to
        WEBKIT_VERSION_LATEST if the class is new.
        (GenerateHeader): Include JavaScriptCore/WebKitAvailability.h if needed. Add the
        interface availability version check if the class has a required version. Create
        a public interface key (used to lookup in $publicInterfaces) and make a declaration
        suffix that includes the availability macro (if needed). Use the "available in 1.3
        and later but deprecated in 3.0" macro instead of "deprecated in 10.5 and later" as
        the default availability macro for old style methods. Tweak line breaks in the generated
        headers to look good and not have too many extra lines.

WebKit/mac:

2008-10-09  Timothy Hatcher  <timothy@apple.com>

        Don't convert JavaScriptCore header include paths to WebKit paths. This was needed
        back when NPAPI and WebScriptObject migrated from JavaScriptCore. Also remove
        JavaScriptCore from the VPATH.

        Reviewed by Sam Weinig.

        * MigrateHeaders.make:

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

11 years ago2008-10-09 Simon Fraser <simon.fraser@apple.com>
simon.fraser@apple.com [Fri, 10 Oct 2008 03:15:31 +0000 (03:15 +0000)]
2008-10-09  Simon Fraser  <simon.fraser@apple.com>

        Reviewed by Dave Hyatt

        https://bugs.webkit.org/show_bug.cgi?id=20947

        Refactor setStyle() methods into styleWillChange()
        and styleDidChange(), so most of the setStyle() overrides can
        be removed.

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

11 years ago2008-10-09 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Fri, 10 Oct 2008 03:13:07 +0000 (03:13 +0000)]
2008-10-09  David Hyatt  <hyatt@apple.com>

        Make sure to destroy a custom scrollbar's RenderObject's before the arena goes away.  When our widget
        parent is nulled out is a good time.

        Reviewed by Oliver Hunt

        * rendering/RenderScrollbar.cpp:
        (WebCore::RenderScrollbar::~RenderScrollbar):
        (WebCore::RenderScrollbar::setParent):
        (WebCore::RenderScrollbar::updateScrollbarPart):
        * rendering/RenderScrollbar.h:

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

11 years agoWebCore:
hyatt@apple.com [Fri, 10 Oct 2008 02:33:16 +0000 (02:33 +0000)]
WebCore:

2008-10-09  David Hyatt  <hyatt@apple.com>

        https://bugs.webkit.org/show_bug.cgi?id=21446

        This patch gets CSS scrollbars up and limping.  There's no way to distinguish between states or between
        orientation and back vs. forward on the buttons, but it's a start.

        Reviewed by Oliver Hunt

        Added scrollbars/basic-scrollbar.html

        * WebCore.xcodeproj/project.pbxproj:
        * css/CSSSelector.cpp:
        (WebCore::CSSSelector::extractPseudoType):
        * css/CSSSelector.h:
        (WebCore::CSSSelector::):
        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):
        * platform/ScrollTypes.h:
        (WebCore::):
        * platform/Scrollbar.h:
        (WebCore::Scrollbar::styleChanged):
        * platform/ScrollbarThemeComposite.cpp:
        (WebCore::ScrollbarThemeComposite::paint):
        (WebCore::ScrollbarThemeComposite::splitTrack):
        * platform/ScrollbarThemeComposite.h:
        (WebCore::ScrollbarThemeComposite::paintScrollbarBackground):
        (WebCore::ScrollbarThemeComposite::paintTrackBackground):
        (WebCore::ScrollbarThemeComposite::paintTrackPiece):
        * platform/win/ScrollbarThemeSafari.cpp:
        (WebCore::ScrollbarThemeSafari::paintTrackBackground):
        * platform/win/ScrollbarThemeSafari.h:
        * platform/win/ScrollbarThemeWin.cpp:
        (WebCore::ScrollbarThemeWin::paintTrack):
        * platform/win/ScrollbarThemeWin.h:
        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::createScrollbar):
        (WebCore::RenderLayer::styleChanged):
        * rendering/RenderScrollbar.cpp: Added.
        (WebCore::RenderScrollbar::createCustomScrollbar):
        (WebCore::RenderScrollbar::RenderScrollbar):
        (WebCore::RenderScrollbar::~RenderScrollbar):
        (WebCore::RenderScrollbar::getScrollbarPseudoStyle):
        (WebCore::RenderScrollbar::updateScrollbarParts):
        (WebCore::RenderScrollbar::updateScrollbarPart):
        (WebCore::RenderScrollbar::paintPart):
        (WebCore::RenderScrollbar::buttonRect):
        (WebCore::RenderScrollbar::minimumThumbLength):
        * rendering/RenderScrollbar.h: Added.
        (WebCore::RenderScrollbar::styleChanged):
        (WebCore::RenderScrollbar::owningRenderer):
        * rendering/RenderScrollbarPart.cpp: Added.
        (WebCore::RenderScrollbarPart::RenderScrollbarPart):
        (WebCore::RenderScrollbarPart::~RenderScrollbarPart):
        (WebCore::RenderScrollbarPart::layout):
        (WebCore::RenderScrollbarPart::layoutHorizontalPart):
        (WebCore::RenderScrollbarPart::layoutVerticalPart):
        (WebCore::calcScrollbarThicknessUsing):
        (WebCore::RenderScrollbarPart::computeScrollbarWidth):
        (WebCore::RenderScrollbarPart::computeScrollbarHeight):
        (WebCore::RenderScrollbarPart::calcPrefWidths):
        (WebCore::RenderScrollbarPart::setStyle):
        * rendering/RenderScrollbarPart.h: Added.
        (WebCore::RenderScrollbarPart::renderName):
        (WebCore::RenderScrollbarPart::requiresLayer):
        * rendering/RenderScrollbarTheme.cpp: Added.
        (WebCore::RenderScrollbarTheme::renderScrollbarTheme):
        (WebCore::RenderScrollbarTheme::buttonSizesAlongTrackAxis):
        (WebCore::RenderScrollbarTheme::hasButtons):
        (WebCore::RenderScrollbarTheme::hasThumb):
        (WebCore::RenderScrollbarTheme::minimumThumbLength):
        (WebCore::RenderScrollbarTheme::backButtonRect):
        (WebCore::RenderScrollbarTheme::forwardButtonRect):
        (WebCore::RenderScrollbarTheme::trackRect):
        (WebCore::RenderScrollbarTheme::paintScrollCorner):
        (WebCore::RenderScrollbarTheme::paintScrollbarBackground):
        (WebCore::RenderScrollbarTheme::paintTrackBackground):
        (WebCore::RenderScrollbarTheme::paintTrackPiece):
        (WebCore::RenderScrollbarTheme::paintButton):
        (WebCore::RenderScrollbarTheme::paintThumb):
        * rendering/RenderScrollbarTheme.h: Added.
        (WebCore::RenderScrollbarTheme::~RenderScrollbarTheme):
        (WebCore::RenderScrollbarTheme::scrollbarThickness):
        (WebCore::RenderScrollbarTheme::buttonsPlacement):
        (WebCore::RenderScrollbarTheme::supportsControlTints):
        (WebCore::RenderScrollbarTheme::shouldCenterOnThumb):
        (WebCore::RenderScrollbarTheme::initialAutoscrollTimerDelay):
        (WebCore::RenderScrollbarTheme::autoscrollTimerDelay):
        (WebCore::RenderScrollbarTheme::registerScrollbar):
        (WebCore::RenderScrollbarTheme::unregisterScrollbar):
        * rendering/style/RenderStyle.h:
        (WebCore::RenderStyle::):

LayoutTests:

2008-10-09  David Hyatt  <hyatt@apple.com>

        Land layout test for CSS scrollbars.

        Reviewed by Oliver Hunt

        * platform/mac/scrollbars: Added.
        * platform/mac/scrollbars/basic-scrollbar-expected.checksum: Added.
        * platform/mac/scrollbars/basic-scrollbar-expected.png: Added.
        * platform/mac/scrollbars/basic-scrollbar-expected.txt: Added.
        * scrollbars: Added.
        * scrollbars/basic-scrollbar.html: Added.

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

11 years ago Reviewed by Oliver Hunt.
eric@webkit.org [Fri, 10 Oct 2008 02:27:52 +0000 (02:27 +0000)]
    Reviewed by Oliver Hunt.

        Test cases for https://bugs.webkit.org/show_bug.cgi?id=21498

        * fast/canvas/gradient-with-clip-expected.txt: Added.
        * fast/canvas/gradient-with-clip.html: Added.
        * fast/canvas/pattern-with-transform-expected.txt: Added.
        * fast/canvas/pattern-with-transform.html: Added.
        * fast/canvas/resources/TEMPLATE.html: Copied from LayoutTests/fast/dom/resources/TEMPLATE.html.
        * fast/canvas/resources/gradient-with-clip.js: Added.
        * fast/canvas/resources/pattern-with-transform.js: Added.

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

11 years ago Reviewed by Oliver Hunt.
eric@webkit.org [Fri, 10 Oct 2008 02:26:54 +0000 (02:26 +0000)]
    Reviewed by Oliver Hunt.

        Fix Canvex DOOM Game
        Gradient clips were not getting cleared after filling the
        gradient, this was causing nothing to draw in Canvex
        https://bugs.webkit.org/show_bug.cgi?id=21498

        Tests forthcoming.

        * WebCore.xcodeproj/project.pbxproj:
        * platform/graphics/cg/GraphicsContextCG.cpp:
        (WebCore::GraphicsContext::fillPath):
        (WebCore::GraphicsContext::strokePath):
        (WebCore::GraphicsContext::fillRect):

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

11 years ago2008-10-09 Cameron Zwarich <zwarich@apple.com>
cwzwarich@webkit.org [Fri, 10 Oct 2008 02:17:12 +0000 (02:17 +0000)]
2008-10-09  Cameron Zwarich  <zwarich@apple.com>

        Not reviewed.

        Add StructureID.o to the exclusion list in the global initializers
        script to fix the Debug build.

        * Scripts/check-for-global-initializers:

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

11 years ago2008-10-09 Sam Weinig <sam@webkit.org>
weinig@apple.com [Fri, 10 Oct 2008 01:21:09 +0000 (01:21 +0000)]
2008-10-09  Sam Weinig  <sam@webkit.org>

        Reviewed by Maciej Stachowiak.

        Add leaks checking to StructureID.

        * kjs/StructureID.cpp:
        (JSC::StructureID::StructureID):
        (JSC::StructureID::~StructureID):

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

11 years ago2008-10-09 Alp Toker <alp@nuanti.com>
alp@webkit.org [Fri, 10 Oct 2008 00:50:25 +0000 (00:50 +0000)]
2008-10-09  Alp Toker  <alp@nuanti.com>

        Reviewed by Mark Rowe.

        https://bugs.webkit.org/show_bug.cgi?id=20760
        Implement support for x86 Linux in CTI

        Prepare to enable CTI/WREC on supported architectures.

        Make it possible to use the CTI_ARGUMENT workaround with GCC as well
        as MSVC by fixing some preprocessor conditionals.

        Note that CTI/WREC no longer requires CTI_ARGUMENT on Linux so we
        don't actually enable it except when building with MSVC. GCC on Win32
        remains untested.

        Adapt inline ASM code to use the global symbol underscore prefix only
        on Darwin and to call the properly mangled Machine::cti_vm_throw
        symbol name depending on CTI_ARGUMENT.

        Also avoid global inclusion of the JIT infrastructure headers
        throughout WebCore and WebKit causing recompilation of about ~1500
        source files after modification to X86Assembler.h, CTI.h, WREC.h,
        which are only used deep inside JavaScriptCore.

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

11 years ago Reviewed by Oliver Hunt.
eric@webkit.org [Fri, 10 Oct 2008 00:08:44 +0000 (00:08 +0000)]
    Reviewed by Oliver Hunt.

        Fix transformed patterns
        https://bugs.webkit.org/show_bug.cgi?id=21498

        Test: fast/canvas/patternfill-repeat.html

        * WebCore.xcodeproj/project.pbxproj:
        * platform/graphics/GraphicsContext.cpp:
        (WebCore::GraphicsContext::setStrokePattern):
        (WebCore::GraphicsContext::setFillPattern):
        (WebCore::GraphicsContext::setStrokeGradient):
        (WebCore::GraphicsContext::setFillGradient):
        * platform/graphics/GraphicsContext.h:
        * platform/graphics/cairo/GraphicsContextCairo.cpp:
        * platform/graphics/cg/GraphicsContextCG.cpp:
        (WebCore::applyStrokePattern):
        (WebCore::applyFillPattern):
        (WebCore::GraphicsContext::drawPath):
        (WebCore::GraphicsContext::fillPath):
        (WebCore::GraphicsContext::strokePath):
        (WebCore::GraphicsContext::fillRect):
        * platform/graphics/qt/GraphicsContextQt.cpp:
        * platform/graphics/wx/GraphicsContextWx.cpp:

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

11 years ago2008-10-09 Chris Marrin <cmarrin@apple.com>
cmarrin@apple.com [Fri, 10 Oct 2008 00:04:19 +0000 (00:04 +0000)]
2008-10-09  Chris Marrin  <cmarrin@apple.com>

        Reviewed by Darin Adler.

        More testcases for animation using font metrics
        https://bugs.webkit.org/show_bug.cgi?id=21246

        * animations/font-size-using-ems-expected.txt: Added.
        * animations/font-size-using-ems.html: Added.
        * animations/width-using-ems-expected.txt: Added.
        * animations/width-using-ems.html: Added.

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

11 years ago2008-10-09 Chris Marrin <cmarrin@apple.com>
cmarrin@apple.com [Thu, 9 Oct 2008 23:59:49 +0000 (23:59 +0000)]
2008-10-09  Chris Marrin  <cmarrin@apple.com>

        Reviewed by Darin Adler.

        Fix for https://bugs.webkit.org/show_bug.cgi?id=21217
        Animations assert when navigating from page

        Changed assertions to test the right flag for the paused state

        * page/animation/AnimationBase.cpp:
        (WebCore::AnimationBase::updateStateMachine):

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

11 years agoJavaScriptCore:
barraclough@apple.com [Thu, 9 Oct 2008 23:59:26 +0000 (23:59 +0000)]
JavaScriptCore:

2008-10-09  Gavin Barraclough  <barraclough@apple.com>

        Reviewed by Cameron Zwarich.

        Fix for bug #21160, x=0;1/(x*-1) == -Infinity

        * ChangeLog:
        * VM/CTI.cpp:
        (JSC::CTI::emitFastArithDeTagImmediate):
        (JSC::CTI::emitFastArithDeTagImmediateJumpIfZero):
        (JSC::CTI::compileBinaryArithOp):
        (JSC::CTI::compileBinaryArithOpSlowCase):
        (JSC::CTI::privateCompileMainPass):
        (JSC::CTI::privateCompileSlowCases):
        * VM/CTI.h:
        * masm/X86Assembler.h:
        (JSC::X86Assembler::):
        (JSC::X86Assembler::emitUnlinkedJs):

LayoutTests:

2008-10-09  Gavin Barraclough  <barraclough@apple.com>

        Reviewed by Cameron Zwarich.

        Correct results for -0 cases.

        * fast/js/math-transforms-expected.txt:

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

11 years agowx build fixes.
kevino@webkit.org [Thu, 9 Oct 2008 23:44:40 +0000 (23:44 +0000)]
wx build fixes.

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

11 years ago2008-10-09 Chris Marrin <cmarrin@apple.com>
cmarrin@apple.com [Thu, 9 Oct 2008 23:04:07 +0000 (23:04 +0000)]
2008-10-09  Chris Marrin  <cmarrin@apple.com>

        Reviewed by Dan Bernstein.

        Fixed > 180 degree rotation bug
        Ensure that validateTransformFunctionList() is called after updating keyframe styles

        https://bugs.webkit.org/show_bug.cgi?id=21420

        Test: animations/big-rotation.html

        * page/animation/KeyframeAnimation.cpp:
        (WebCore::KeyframeAnimation::KeyframeAnimation):

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

11 years ago2008-10-09 Cameron Zwarich <zwarich@apple.com>
cwzwarich@webkit.org [Thu, 9 Oct 2008 21:47:31 +0000 (21:47 +0000)]
2008-10-09  Cameron Zwarich  <zwarich@apple.com>

        Reviewed by Oliver Hunt.

        Bug 21459: REGRESSION (r37324): Safari crashes inside JavaScriptCore while browsing hulu.com
        <https://bugs.webkit.org/show_bug.cgi?id=21459>

        After r37324, an Arguments object does not mark an associated activation
        object. This change was made because Arguments no longer directly used
        the activation object in any way. However, if an activation is torn off,
        then the backing store of Arguments becomes the register array of the
        activation object. Arguments directly marks all of the arguments, but
        the activation object is being collected, which causes its register
        array to be freed and new memory to be allocated in its place.

        Unfortunately, it does not seem possible to reproduce this issue in a
        layout test.

        * kjs/Arguments.cpp:
        (JSC::Arguments::mark):
        * kjs/Arguments.h:
        (JSC::Arguments::setActivation):
        (JSC::Arguments::Arguments):
        (JSC::JSActivation::copyRegisters):

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

11 years ago2008-10-09 Brady Eidson <beidson@apple.com>
beidson@apple.com [Thu, 9 Oct 2008 20:43:24 +0000 (20:43 +0000)]
2008-10-09  Brady Eidson  <beidson@apple.com>

        Reviewed by Anders

        <rdar://problem/6250856> - Calling [WebView close] from within a redirection callback can cause bad things

        The API usage to reveal this crash was so particular that a layout test is not possible with our current infrastructure.

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::changeLocation): Protect the Frame from deletion
        (WebCore::FrameLoader::continueLoadAfterNavigationPolicy): stopAllLoaders() might clear the Frame's page,
          so perform a second null check on the page.  Bail if it has been cleared.

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

11 years ago2008-10-09 Alp Toker <alp@nuanti.com>
alp@webkit.org [Thu, 9 Oct 2008 16:21:43 +0000 (16:21 +0000)]
2008-10-09  Alp Toker  <alp@nuanti.com>

        Build fix for recent DOM generation Makefile changes when custom
        CFLAGS/CXXFLAGS are passed in.

        * GNUmakefile.am:

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

11 years ago2008-10-09 Jan Michael Alonzo <jmalonzo@webkit.org>
alp@webkit.org [Thu, 9 Oct 2008 16:13:53 +0000 (16:13 +0000)]
2008-10-09  Jan Michael Alonzo  <jmalonzo@webkit.org>

        Reviewed by Alp Toker.

        https://bugs.webkit.org/show_bug.cgi?id=21390
        [Gtk] Linux/Gtk: GtkLauncher crashes on Acid3 (but after test 80 this time)

        * platform/gtk/ScrollViewGtk.cpp:
        (WebCore::ScrollView::platformRemoveChild):

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

11 years ago2008-10-09 Ariya Hidayat <ariya.hidayat@trolltech.com>
hausmann@webkit.org [Thu, 9 Oct 2008 14:26:27 +0000 (14:26 +0000)]
2008-10-09  Ariya Hidayat  <ariya.hidayat@trolltech.com>

        Reviewed by Simon.

        Build fix for MinGW.

        * wtf/AlwaysInline.h:

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

11 years ago2008-10-09 Darin Adler <darin@apple.com>
darin@apple.com [Thu, 9 Oct 2008 08:37:04 +0000 (08:37 +0000)]
2008-10-09  Darin Adler  <darin@apple.com>

       - try to fix Qt build

        * bridge/qt/qt_runtime.cpp:
        (JSC::Bindings::convertQVariantToValue): Pass JSGlobalData instead of ExecState to RegExp constructor.

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

11 years ago2008-10-09 Darin Adler <darin@apple.com>
darin@apple.com [Thu, 9 Oct 2008 07:25:46 +0000 (07:25 +0000)]
2008-10-09  Darin Adler  <darin@apple.com>

        - checked in a missing expected result

        * fast/dom/css-RGBValue-expected.txt: Added.

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

11 years agoRemove restriction on version number ending in a 4.
mrowe@apple.com [Thu, 9 Oct 2008 05:36:02 +0000 (05:36 +0000)]
Remove restriction on version number ending in a 4.

Rubber-stamped by Jon Honeycutt.

* win/tools/scripts/auto-version.sh:

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