[Webkit-unassigned] [Bug 71128] Select multiple options with mouse drag in Select element.

[bugzilla-daemon at webkit.org]

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


Kent Tamura <tkent at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #112892|review?                     |review-
               Flag|                            |




--- Comment #16 from Kent Tamura <tkent at chromium.org>  2011-11-02 01:27:41 PST ---
(From update of attachment 112892)
View in context: https://bugs.webkit.org/attachment.cgi?id=112892&action=review

> Source/WebCore/ChangeLog:8
> +        Selection in select element with an mouse drag.

Please add rationale to ChangeLog.

> LayoutTests/fast/forms/select-multiple-elements-with-mouse-drag.html:5
> +<link rel="stylesheet" href="../js/resources/js-test-style.css">

This line is not needed. js-test-pre.js automatically loads it.

> LayoutTests/fast/forms/select-multiple-elements-with-mouse-drag.html:11
> +    if(window.eventSender) {

Though we don't have an official code style for JavaScript, please follow the C++ style as possible.
Need a space after 'if'

> LayoutTests/fast/forms/select-multiple-elements-with-mouse-drag.html:19
> +        eventSender.mouseMoveTo(x,y);

Need a space after ','

> LayoutTests/fast/forms/select-multiple-elements-with-mouse-drag.html:21
> +        eventSender.mouseMoveTo(x,y + (optionHeight * 3));

Need a space after ','

> LayoutTests/fast/forms/select-multiple-elements-with-mouse-drag.html:33
> +    for(var i=0; i < 4; i++) {

Need a space
 after 'for'
 before and after '='

> LayoutTests/fast/forms/select-multiple-elements-with-mouse-drag.html:34
> +        shouldBeTrue("document.getElementById(\"selectId\").options["+ i + "].selected");

Need a space before the first '+'

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.