Bug 176699 - Older versions should adopt newer version's expectations on mac
Summary: Older versions should adopt newer version's expectations on mac
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jonathan Bedard
URL:
Keywords: InRadar
Depends on:
Blocks: 176463
  Show dependency treegraph
 
Reported: 2017-09-11 09:08 PDT by Jonathan Bedard
Modified: 2017-11-14 16:52 PST (History)
7 users (show)

See Also:


Attachments
Patch (3.31 KB, patch)
2017-09-11 09:16 PDT, Jonathan Bedard
ap: review+
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Bedard 2017-09-11 09:08:13 PDT
In https://bugs.webkit.org/show_bug.cgi?id=174472, test expectation inheritance for Mac was modified so it was more in-line with other platforms.  There was an aspect of this test inheritance, however, which should be retained to make incrementing versions easier.  Consider El Capitan as we transition to High Sierra:

If test expectation inheritance looks like this for El Capitan:

mac-add-elcapitan-wk2 => mac-elcapitan => mac-sierra-wk2 => mac-sierra => mac-wk2 => mac => wk2

Then we can move any Sierra/El Capitan expectations into mac-sierra, and general High Sierra expectations into Mac (since such expectations will likely apply to future operating systems as well).  Before r219507, this was the technique used.  However, general mac overrode version specific expectations before r219507 (which is definitely incorrect).  See the ChangeLog of <https://trac.webkit.org/changeset/219507/webkit> for details.
Comment 1 Radar WebKit Bug Importer 2017-09-11 09:08:37 PDT
<rdar://problem/34367907>
Comment 2 Jonathan Bedard 2017-09-11 09:16:00 PDT
Created attachment 320432 [details]
Patch
Comment 3 WebKit Commit Bot 2017-09-11 14:02:32 PDT
Comment on attachment 320432 [details]
Patch

Rejecting attachment 320432 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 320432, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
    -> origin/master
Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 221872 = 8af2268a70bf54353a9d2d586ef20842d13588ca
r221873 = 52fffebf9340ce45308b469d58249c4e9c41a60d
r221874 = 3e36c33d1a9fd59bc8f66681346174d194164729
Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.

Full output: http://webkit-queues.webkit.org/results/4515570
Comment 4 Jonathan Bedard 2017-09-11 14:13:21 PDT
Committed r221877: <http://trac.webkit.org/changeset/221877>