Bug 102039 - [Chromium] Add tests for week suggestion picker
Summary: [Chromium] Add tests for week suggestion picker
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Forms (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Keishi Hattori
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-12 20:47 PST by Keishi Hattori
Modified: 2012-11-19 00:03 PST (History)
3 users (show)

See Also:


Attachments
Patch (125.74 KB, patch)
2012-11-16 04:07 PST, Keishi Hattori
no flags Details | Formatted Diff | Diff
Patch (127.51 KB, patch)
2012-11-18 19:28 PST, Keishi Hattori
no flags Details | Formatted Diff | Diff
Patch (127.51 KB, patch)
2012-11-18 23:16 PST, Keishi Hattori
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Keishi Hattori 2012-11-12 20:47:55 PST
Add tests for week suggestion picker
Comment 1 Keishi Hattori 2012-11-16 04:07:39 PST
Created attachment 174642 [details]
Patch
Comment 2 WebKit Review Bot 2012-11-16 06:12:30 PST
Comment on attachment 174642 [details]
Patch

Attachment 174642 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/14860247

New failing tests:
platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-appearance-with-scroll-bar.html
platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-appearance.html
platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-appearance-rtl.html
Comment 3 Kent Tamura 2012-11-18 18:19:08 PST
Comment on attachment 174642 [details]
Patch

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

> LayoutTests/platform/chromium/TestExpectations:2827
> +webkit.org/b/99421 platform/chromium/fast/forms/suggestion-picker/month-suggestion-picker-appearance-rtl.html [ Pass ImageOnlyFailure ]
> +webkit.org/b/99421 platform/chromium/fast/forms/suggestion-picker/month-suggestion-picker-appearance-with-scroll-bar.html [ Pass ImageOnlyFailure ]
> +webkit.org/b/99421 platform/chromium/fast/forms/suggestion-picker/month-suggestion-picker-appearance.html [ Pass ImageOnlyFailure ]

They should have Missing.

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-appearance-with-scroll-bar.html:13
> +<input type=week id=week value="10000-W12" list="suggestions" style="width: 100px;">

The spin button and the picker indicator are drawn outside of the input bounds.  Is it expected?

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-key-operations.html:90
> +description("Tests that key bindings work as expected.");
> +
> +debug('Check that page popup doesn\'t exist at first.');

nit: Quotation mark usage is inconsistent.  " is used for description(), ' is used for debug().

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-key-operations.html:160
> +    debug('Open calendar picker.');
> +    expectingDidHideEvent = true;
> +    popupWindow.addEventListener("didHide", testDidHide, false);
> +    popupWindow.addEventListener("didOpenPicker", test3, false);
> +    eventSender.keyDown('\n');

ditto.

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-key-operations.html:166
> +    popupWindow.removeEventListener("didHide", testDidHide);
> +
> +    shouldBeTrue('expectingDidHideEvent');

ditto.

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-key-operations.html:173
> +    popupWindow.removeEventListener("didOpenPicker", test3);
> +
> +    shouldBeFalse('expectingDidHideEvent');

ditto.

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-key-operations.html:189
> +    sendKey(document.getElementById('week'), "F4");
> +    if (document.getElementById('mock-page-popup'))
> +        testPassed("F4 opened picker.");

ditto.

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-mouse-operations.html:97
> +    debug('Check that hovering over an entry highlights it.');
> +    hoverOverElement(popupWindow.document.querySelector(".suggestion-list-entry:nth-child(2)"));

ditto.

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-mouse-operations.html:105
> +    debug('Check that mouse click closes the popup and sets the value.');
> +    clickElement(popupWindow.document.querySelector(".suggestion-list-entry:nth-child(3)"));

ditto.

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-mouse-operations.html:122
> +    var suggestionList = popupWindow.document.getElementsByClassName("suggestion-list")[0];
> +
> +    debug('Check that you can scroll using the mouse wheel.');

ditto.

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-mouse-operations.html:148
> +    debug('Check that you can scroll using the scroll bar.');
> +
> +    var suggestionList = popupWindow.document.getElementsByClassName("suggestion-list")[0];

ditto.

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-reset-value-after-reload.html:24
> +    shouldBeEqualToString('highlightedEntry()', '2012-W01');
> +    eventSender.keyDown("\n");

ditto.

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-reset-value-after-reload.html:41
> +    shouldBeEqualToString('highlightedEntry()', '2012-W01');
> +    eventSender.keyDown("\n");

ditto.

> LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-reset-value-after-reload.html:49
> +    shouldBeEqualToString('iframe.contentDocument.getElementById("test2").value', '2012-W01');
> +
> +    iframe.addEventListener("load", function () { parent.runOnIFrameLoad(); });

ditto.
Comment 4 Keishi Hattori 2012-11-18 18:28:53 PST
> > LayoutTests/platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-appearance-with-scroll-bar.html:13
> > +<input type=week id=week value="10000-W12" list="suggestions" style="width: 100px;">
> 
> The spin button and the picker indicator are drawn outside of the input bounds.  Is it expected?
Yes. I wanted to check that a horizontal scrollbar won't appear even when we have a small anchor element.
Comment 5 Keishi Hattori 2012-11-18 19:28:37 PST
Created attachment 174877 [details]
Patch
Comment 6 WebKit Review Bot 2012-11-18 21:36:24 PST
Comment on attachment 174877 [details]
Patch

Rejecting attachment 174877 [details] from commit-queue.

New failing tests:
platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-appearance-with-scroll-bar.html
platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-appearance.html
platform/chromium/fast/forms/suggestion-picker/week-suggestion-picker-appearance-rtl.html
Full output: http://queues.webkit.org/results/14906050
Comment 7 Keishi Hattori 2012-11-18 23:16:07 PST
Created attachment 174899 [details]
Patch
Comment 8 WebKit Review Bot 2012-11-19 00:02:57 PST
Comment on attachment 174899 [details]
Patch

Clearing flags on attachment: 174899

Committed r135121: <http://trac.webkit.org/changeset/135121>
Comment 9 WebKit Review Bot 2012-11-19 00:03:02 PST
All reviewed patches have been landed.  Closing bug.