Bug 127428 - [WK2] Unskip now passing tests after r135595
Summary: [WK2] Unskip now passing tests after r135595
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Csaba Osztrogonác
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-22 08:56 PST by Csaba Osztrogonác
Modified: 2015-04-03 11:42 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.53 KB, patch)
2014-01-22 08:59 PST, Csaba Osztrogonác
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-13 for mac-mountainlion-wk2 (472.17 KB, application/zip)
2014-01-22 09:52 PST, Build Bot
no flags Details
Patch (1.50 KB, patch)
2014-01-23 02:32 PST, Csaba Osztrogonác
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2 (472.58 KB, application/zip)
2014-01-23 03:26 PST, Build Bot
no flags Details
Patch (2.43 KB, patch)
2014-01-23 03:29 PST, Csaba Osztrogonác
no flags Details | Formatted Diff | Diff
Patch (2.21 KB, patch)
2015-04-03 04:17 PDT, Csaba Osztrogonác
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews104 for mac-mavericks-wk2 (588.27 KB, application/zip)
2015-04-03 05:11 PDT, Build Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2014-01-22 08:56:56 PST
wk2/TestExpectations:

# WebKitTestRunner needs eventSender.contextClick()
# https://bugs.webkit.org/show_bug.cgi?id=86881
editing/selection/context-menu-text-selection.html
editing/selection/5354455-2.html
fast/events/context-nodrag.html


r135595 implemented the missing featurea, additionally
I tested them on EFL WK2 and they are passing now.
Comment 1 Csaba Osztrogonác 2014-01-22 08:59:21 PST
Created attachment 221868 [details]
Patch

Let's see what the Mac WK2 EWS think about it.
Comment 2 Build Bot 2014-01-22 09:52:47 PST
Comment on attachment 221868 [details]
Patch

Attachment 221868 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/6356663871209472

New failing tests:
editing/selection/context-menu-text-selection.html
Comment 3 Build Bot 2014-01-22 09:52:48 PST
Created attachment 221871 [details]
Archive of layout-test-results from webkit-ews-13 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-13  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 4 Csaba Osztrogonác 2014-01-23 02:32:50 PST
Created attachment 221965 [details]
Patch

maybe it was flakiness
Comment 5 Build Bot 2014-01-23 03:26:29 PST
Comment on attachment 221965 [details]
Patch

Attachment 221965 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5454781204135936

New failing tests:
editing/selection/context-menu-text-selection.html
Comment 6 Build Bot 2014-01-23 03:26:31 PST
Created attachment 221968 [details]
Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-15  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 7 Csaba Osztrogonác 2014-01-23 03:29:11 PST
Created attachment 221969 [details]
Patch

No, it wasn't flakiness, editing/selection/context-menu-text-selection.html really crashes on Mac WK2, so let's skip it on mac-wk2. And I filed a bug report about this crash: https://bugs.webkit.org/show_bug.cgi?id=127473
Comment 8 Csaba Osztrogonác 2015-04-03 04:17:27 PDT
Created attachment 250057 [details]
Patch

patch for EWS
Comment 9 Build Bot 2015-04-03 05:11:35 PDT
Comment on attachment 250057 [details]
Patch

Attachment 250057 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5390905636814848

New failing tests:
fast/fixed-layout/fixed-layout.html
Comment 10 Build Bot 2015-04-03 05:11:38 PDT
Created attachment 250062 [details]
Archive of layout-test-results from ews104 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 11 Csaba Osztrogonác 2015-04-03 05:30:55 PDT
Comment on attachment 250057 [details]
Patch

Clearing flags on attachment: 250057

Committed r182316: <http://trac.webkit.org/changeset/182316>
Comment 12 Csaba Osztrogonác 2015-04-03 05:31:03 PDT
All reviewed patches have been landed.  Closing bug.
Comment 13 Alexey Proskuryakov 2015-04-03 10:27:12 PDT
I re-skipped two tests in r182322.

It doesn't seem worth keeping them enabled with a Timeout expectation - that has a pretty high cost, as one process has to wait 30 seconds.
Comment 14 Csaba Osztrogonác 2015-04-03 11:42:50 PDT
(In reply to comment #13)
> I re-skipped two tests in r182322.
> 
> It doesn't seem worth keeping them enabled with a Timeout expectation - that
> has a pretty high cost, as one process has to wait 30 seconds.

There are 7 tests in mac and 11 in mac-wk2's TestExpectations marked
as timeout, I thought it is common for Mac platforms. (Otherwise I
I always prefered skipping tests instead of marking timeout, fail, etc.,
because small ports didn't care about fixing any regression in my 
6.5 years working on WebKit. So why run always failing tests? :)) )