LayoutTests/ChangeLog

2010-09-23  James Robinson  <jamesr@chromium.org>

        Reviewed by NOBODY (OOPS!).

        Canvas: radialGradient with negative radius should throw exception

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

        Remove test from skipped lists now that it passes.

        * platform/chromium/test_expectations.txt:

        * platform/gtk/Skipped:

        * platform/mac/Skipped:

        * platform/qt/Skipped:

2010-09-22  Kinuko Yasuda  <kinuko@chromium.org>

        Unreviewed, temporarily disabling the new filesystem test on chromium.

        * fast/js/regexp-bol.html: Added.

        * fast/js/script-tests/regexp-bol.js: Added.

2010-09-22  James Robinson  <jamesr@chromium.org>

        Reviewed by NOBODY (OOPS!).

        Throw INDEX_SIZE_ERR exception if createRadialGradient is called with a negative radius

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

        Unskips canvas/philip/tests/2d.gradient.radial.negative.html since we now pass.

        * platform/chromium/test_expectations.txt:

        * platform/mac/Skipped:

2010-09-21  Kent Tamura  <tkent@chromium.org>

        Unreviewed, text expectation update.

LayoutTests/platform/chromium/test_expectations.txt

@@BUG45991 : canvas/philip/tests/2d.fillStyle.parse.current.changed.html = TEXT

BUG45991 : canvas/philip/tests/2d.fillStyle.parse.current.removed.html = TEXT

BUG45991 : canvas/philip/tests/2d.fillStyle.parse.system.html = TEXT

BUG45991 : canvas/philip/tests/2d.gradient.radial.cone.top.html = TEXT TIMEOUT

2517 BUG45991 : canvas/philip/tests/2d.gradient.radial.negative.html = TEXT

BUG45991 : canvas/philip/tests/2d.imageData.create1.type.html = TEXT

BUG45991 : canvas/philip/tests/2d.imageData.create2.type.html = TEXT

BUG45991 : canvas/philip/tests/2d.imageData.get.source.negative.html = TEXT

LayoutTests/platform/gtk/Skipped

@@canvas/philip/tests/2d.gradient.radial.cone.front.html

canvas/philip/tests/2d.gradient.radial.cone.top.html

canvas/philip/tests/2d.gradient.radial.inside2.html

canvas/philip/tests/2d.gradient.radial.inside3.html

5617 canvas/philip/tests/2d.gradient.radial.negative.html

canvas/philip/tests/2d.gradient.radial.outside1.html

canvas/philip/tests/2d.imageData.create1.type.html

canvas/philip/tests/2d.imageData.create2.type.html

LayoutTests/platform/mac/Skipped

@@canvas/philip/tests/2d.gradient.radial.cone.front.html

canvas/philip/tests/2d.gradient.radial.cone.top.html

canvas/philip/tests/2d.gradient.radial.inside2.html

canvas/philip/tests/2d.gradient.radial.inside3.html

196 canvas/philip/tests/2d.gradient.radial.negative.html

canvas/philip/tests/2d.gradient.radial.outside1.html

canvas/philip/tests/2d.imageData.create1.type.html

canvas/philip/tests/2d.imageData.create2.type.html

LayoutTests/platform/qt/Skipped

@@canvas/philip/tests/2d.gradient.radial.cone.bottom.html

canvas/philip/tests/2d.gradient.radial.cone.cylinder.html

canvas/philip/tests/2d.gradient.radial.cone.shape2.html

canvas/philip/tests/2d.gradient.radial.cone.top.html

5236 canvas/philip/tests/2d.gradient.radial.negative.html

canvas/philip/tests/2d.gradient.radial.touch1.html

canvas/philip/tests/2d.gradient.radial.touch2.html

canvas/philip/tests/2d.gradient.radial.touch3.html

WebCore/ChangeLog

2010-09-22  James Robinson  <jamesr@chromium.org>

        Reviewed by NOBODY (OOPS!).

        Canvas: radialGradient with negative radius should throw exception

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

        This matches the canvas 2d specification, Opera, IE9 beta and our behavior for arc()

        and arcTo().

        Test: canvas/philip/tests/2d.gradient.radial.negative.html

        * html/canvas/CanvasRenderingContext2D.cpp:

        (WebCore::CanvasRenderingContext2D::createRadialGradient):

2010-09-22  Ruben  <chromium@hybridsource.org>

        Reviewed by Tony Chang.

WebCore/html/canvas/CanvasRenderingContext2D.cpp

@@PassRefPtr<CanvasGradient> CanvasRenderingContext2D::createRadialGradient(float

        ec = NOT_SUPPORTED_ERR;

        return 0;

    }

    if (r0 < 0 || r1 < 0) {

        ec = INDEX_SIZE_ERR;

        return 0;

    }

    PassRefPtr<CanvasGradient> gradient =  CanvasGradient::create(FloatPoint(x0, y0), r0, FloatPoint(x1, y1), r1);

    prepareGradientForDashboard(gradient.get());

    return gradient;