Bug 91504

Created attachment 152816 [details] Proposed patch Proposed patch (be aware, 3Mb patch!)

Created attachment 152822 [details] Proposed patch (v2) Added missing ChangeLog.

Comment on attachment 152822 [details] Proposed patch (v2) rs=me

Comment on attachment 152822 [details] Proposed patch (v2) Clearing flags on attachment: 152822 Committed r122879: <http://trac.webkit.org/changeset/122879>

All reviewed patches have been landed. Closing bug.

53 new failing tests according to latest build: http://build.webkit.sed.hu/builders/x86-32%20Linux%20Qt%20Release%20WebKit2/builds/26755/steps/layout-test List of failing fast/forms tests: fast/forms/HTMLOptionElement_label06.html = TEXT fast/forms/HTMLOptionElement_label07.html = TEXT fast/forms/basic-inputs.html = TEXT fast/forms/basic-textareas-quirks.html = TEXT fast/forms/box-shadow-override.html = TEXT fast/forms/control-restrict-line-height.html = TEXT fast/forms/encoding-test.html = TEXT fast/forms/fieldset-align.html = TEXT fast/forms/form-element-geometry.html = TEXT fast/forms/input-appearance-bkcolor.html = TEXT fast/forms/input-appearance-default-bkcolor.html = TEXT fast/forms/input-appearance-disabled.html = TEXT fast/forms/input-appearance-focus.html = TEXT fast/forms/input-appearance-preventDefault.html = TEXT fast/forms/input-appearance-readonly.html = TEXT fast/forms/input-appearance-selection.html = TEXT fast/forms/input-appearance-visibility.html = TEXT fast/forms/input-baseline.html = TEXT fast/forms/input-disabled-color.html = TEXT fast/forms/input-double-click-selection-gap-bug.html = TEXT fast/forms/input-placeholder-visibility-1.html = TEXT fast/forms/input-placeholder-visibility-3.html = TEXT fast/forms/input-readonly-dimmed.html = TEXT fast/forms/input-readonly-empty.html = TEXT fast/forms/input-spaces.html = TEXT fast/forms/input-text-click-inside.html = TEXT fast/forms/input-text-double-click.html = TEXT fast/forms/input-text-option-delete.html = TEXT fast/forms/input-text-self-emptying-click.html = TEXT fast/forms/input-text-word-wrap.html = TEXT fast/forms/input-width.html = TEXT fast/forms/menulist-separator-painting.html = TEXT fast/forms/number/input-appearance-number-rtl.html = TEXT fast/forms/number/input-appearance-spinbutton-disabled-readonly.html = TEXT fast/forms/number/input-appearance-spinbutton-layer.html = TEXT fast/forms/placeholder-position.html = TEXT fast/forms/placeholder-pseudo-style.html = TEXT fast/forms/search-cancel-button-style-sharing.html = TEXT fast/forms/search-display-none-cancel-button.html = TEXT fast/forms/search-rtl.html = TEXT fast/forms/search-vertical-alignment.html = TEXT fast/forms/select-background-none.html = TEXT fast/forms/select-baseline.html = TEXT fast/forms/selectlist-minsize.html = TEXT fast/forms/text-style-color.html = TEXT fast/forms/textarea-align.html = TEXT fast/forms/textarea-placeholder-pseudo-style.html = TEXT fast/forms/textarea-placeholder-visibility-1.html = TEXT fast/forms/textarea-placeholder-visibility-2.html = TEXT fast/forms/textarea-setinnerhtml.html = TEXT fast/forms/textfield-focus-ring.html = TEXT fast/forms/textfield-outline.html = TEXT fast/forms/textfield-overflow.html = TEXT I am having a double check at those atm.

Created attachment 153311 [details] Missing/Updated results patch This patch fixes the faulty results for fast/forms directory after r122879.

Comment on attachment 153311 [details] Missing/Updated results patch rs=me

Comment on attachment 153311 [details] Missing/Updated results patch Clearing flags on attachment: 153311 Committed r123156: <http://trac.webkit.org/changeset/123156>

All reviewed patches have been landed. Closing bug.

Created attachment 154633 [details] Re-skip and remove added tests on WK2 platform This patch removes and re-skips the added tests for qt-5.0-wk2 layout test platform directory. As discussed with Caio, these tests should be assigned to proper bugs instead.

Changing status to "reopened".

Created attachment 154635 [details] Re-skip and remove added tests on WK2 platform (w/ ChangeLog) Added missing ChangeLog.

Comment on attachment 154635 [details] Re-skip and remove added tests on WK2 platform (w/ ChangeLog) Rejecting attachment 154635 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: tching file LayoutTests/platform/qt-5.0-wk2/fast/forms/textfield-outline-expected.txt rm 'LayoutTests/platform/qt-5.0-wk2/fast/forms/textfield-outline-expected.txt' patching file LayoutTests/platform/qt-5.0-wk2/fast/forms/textfield-overflow-expected.txt rm 'LayoutTests/platform/qt-5.0-wk2/fast/forms/textfield-overflow-expected.txt' Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Alexis Men..." exit_code: 1 cwd: /mnt/git/webkit-commit-queue/ Full output: http://queues.webkit.org/results/13348766

Comment on attachment 154635 [details] Re-skip and remove added tests on WK2 platform (w/ ChangeLog) Commit-queue complain is valid, this patch is supposed to be applied after patch from bug 92186 gets landed, because they add data to the same file: LayoutTests/platform/qt-5.0-wk2/Skipped Please consider granting cq again.

Comment on attachment 154635 [details] Re-skip and remove added tests on WK2 platform (w/ ChangeLog) Clearing flags on attachment: 154635 Committed r123762: <http://trac.webkit.org/changeset/123762>

All reviewed patches have been landed. Closing bug.