[webkit-reviews] review denied: [Bug 56752] [Qt] The minimum size of the select menu list is incorrect for qtwebkit : [Attachment 87900] Updated patch with the updated new test case

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Apr 6 05:52:25 PDT 2011


Benjamin Poulain <benjamin at webkit.org> has denied Doreen Jiang
<doreen.jiang at nokia.com>'s request for review:
Bug 56752: [Qt] The minimum size of the select menu list is incorrect for
qtwebkit
https://bugs.webkit.org/show_bug.cgi?id=56752

Attachment 87900: Updated patch with the updated new test case
https://bugs.webkit.org/attachment.cgi?id=87900&action=review

------- Additional Comments from Benjamin Poulain <benjamin at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=87900&action=review

r- because of the test being Qt specific without justification for that.
If a layout issue is not covered by existing test, you should improve the tests
for everyone, not just for Qt.

> Source/WebCore/ChangeLog:5
> +	   [QT]The minimum size of the select menu list is incorrect for
qtwebkit 

Qt, not QT.

> LayoutTests/ChangeLog:5
> +	   [QT]The minimum size of the select menu list is incorrect for
qtwebkit 

Qt

> LayoutTests/platform/qt/fast/forms/selectlist-minsize.html:5
> +	       [QT]The minimum size of the select menu list is incorrect for
qtwebkit <br>

Qt

> LayoutTests/platform/qt/fast/forms/selectlist-minsize.html:9
> +	   <select id="list3">

Why is there an id?


More information about the webkit-reviews mailing list