Bug 98034 – Add more tests for calendar picker

• Status: RESOLVED
• Resolution: FIXED
• Component: Forms

[Keishi Hattori]

We need to add more tests for calendar picker

[Keishi Hattori]

Created attachment 166485 [details]
Patch

[Kent Tamura]

Comment on attachment 166485 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=166485&action=review

ok.

> LayoutTests/fast/forms/date/calendar-picker-mouse-operations.html:79
> +function cumulativeOffset(element) {
> +    var x = 0;
> +    var y = 0;
> +    var parentFrame = element.ownerDocument.defaultView.frameElement;
> +    if (parentFrame) {
> +        var parentFrameOffset = cumulativeOffset(parentFrame);
> +        x = parentFrameOffset[0];
> +        y = parentFrameOffset[1];
> +    }
> +    if (element.parentNode) {
> +        do {
> +            x += element.offsetLeft || 0;
> +            y += element.offsetTop  || 0;
> +            element = element.offsetParent;
> +        } while (element);
> +    }
> +    return [x, y];
> +}
> +
> +function hoverOverElement(element) {
> +    var offset = cumulativeOffset(element);
> +    var centerX = offset[0] + element.offsetWidth / 2;
> +    var centerY = offset[1] + element.offsetHeight / 2;
> +    eventSender.mouseMoveTo(centerX, centerY);
> +}
> +
> +function clickElement(element) {

I remember another test has these functions. Don't you share them?

[Keishi Hattori]

Created attachment 166589 [details]
Patch

[Keishi Hattori]

(In reply to comment #2)
> I remember another test has these functions. Don't you share them?

Added these three functions for fast/forms/resources/common.js and used it in date-suggestion-picker-mouse-operations.html

[WebKit Review Bot]

Comment on attachment 166589 [details]
Patch

Clearing flags on attachment: 166589

Committed r130111: <http://trac.webkit.org/changeset/130111>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.