RESOLVED FIXED 44068
[Qt] tests whose bug was closed but are still skipped 
https://bugs.webkit.org/show_bug.cgi?id=44068
Summary [Qt] tests whose bug was closed but are still skipped
Ademar Reis
Reported 2010-08-16 12:44:04 PDT
There are a few outdated entries inside platform/qt/Skipped pointing to bugs already fixed. They are: bug #36628 bug #35350 bug #28952 bug #38502 (see discussion at the end) bug #38604 As a result the following tests should be enabled: editing/selection/shrink-selection-after-shift-pagedown.html fast/workers/worker-multi-port.html fast/workers/worker-context-multi-port.html fast/workers/worker-event-listener.html fast/events/show-modal-dialog-onblur-onfocus.html fast/harness/show-modal-dialog.html
Attachments
Enable tests whose bug is fixed (3.45 KB, patch)
2010-08-16 13:00 PDT, Ademar Reis 		no flags
DetailsFormatted DiffDiff
Enable tests whose bug is fixed (3.38 KB, patch)
2010-08-16 13:26 PDT, Ademar Reis 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Ademar Reis
Comment 1 2010-08-16 12:52:08 PDT
My mistake, bug #36628 is not closed (but it should be, IMO). I'll handle the editing/selection/shrink-selection-after-shift-pagedown.html testcase there and close the bug. The others remain valid.
Ademar Reis
Comment 2 2010-08-16 13:00:30 PDT
Created attachment 64511 [details] Enable tests whose bug is fixed
Ademar Reis
Comment 3 2010-08-16 13:26:45 PDT
Created attachment 64514 [details] Enable tests whose bug is fixed patch v2, formating it to avoid ChangeLog dependency from my previous patches. Sorry for spamming, /me new to webkit-devel :)
Ariya Hidayat
Comment 4 2010-08-17 01:38:08 PDT
Comment on attachment 64514 [details] Enable tests whose bug is fixed LGTM, re=me
WebKit Commit Bot
Comment 5 2010-08-17 01:55:41 PDT
Comment on attachment 64514 [details] Enable tests whose bug is fixed Clearing flags on attachment: 64514 Committed r65489: <http://trac.webkit.org/changeset/65489>
WebKit Commit Bot
Comment 6 2010-08-17 01:55:45 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.