DescriptionReland of Update MANIFEST.json to newest version. (patchset #3 id:20001 of https://codereview.chromium.org/2619613002/ )
Reason for revert:
Test expectations were updated in http://crrev.com/2614253003, so layout test baselines and expectations should be up-to-date.
Original issue's description:
> Revert of:
>
> "Import wpt@eeecf3e14368d4ab5221cde688003dedeca30dba"
> This reverts commit cfd8c14d3a0f7fd474248428d230845812848038.
>
> "Update MANIFEST.json to newest version."
> This reverts commit dc2a9e048fd191a43a35c8f6ee1e4cecb7fc919d.
>
> Reason for revert:
> Begins running layout tests without proper expectations.
>
> Original issue's description:
> > Update MANIFEST.json to newest version.
> >
> > Background: Recently in web-platform-tests the format of MANIFEST.json changed, and I filed http://crbug.com/678077 for this. I committed a change http://crrev.com/2610243002 which updated the layout test runner to find web-platform-tests with the new manifest format, but the actual manifest file was too large to upload to Rietveld and was not committed. After that change, no wpt tests are found when running run-webkit-tests imported/wpt.
> >
> > This change would update MANIFEST.json, although the actual file is still too large to upload to Rietveld, so this CL (in patch 1 at least) appears empty. I believe it would have to be landed directly.
> >
> > TBR=dpranke
> >
> > BUG=678077
> >
> > Review-Url: https://codereview.chromium.org/2615903004 .
> > Cr-Commit-Position: refs/heads/master@{#441999}
> > Committed: https://chromium.googlesource.com/chromium/src/+/dc2a9e048fd191a43a35c8f6ee1e4cecb7fc919d
>
> TBR=dpranke@chromium.org,qyearsley@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=678077
>
> Review-Url: https://codereview.chromium.org/2619613002 .
> Cr-Commit-Position: refs/heads/master@{#442054}
> Committed: https://chromium.googlesource.com/chromium/src/+/e0615e1b33112564698760629bef4b56d80572cd
R=dpranke@chromium.org
TBR=dewittj@chromium.org, dpranke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=678077
Review-Url: https://codereview.chromium.org/2625523002 .
Cr-Commit-Position: refs/heads/master@{#442457}
Committed: https://chromium.googlesource.com/chromium/src/+/1297046888300e6524fd1aab198e78bf67481674
Patch Set 1 #Patch Set 2 : Add manifest #
Created: 3 years, 11 months ago
(Patch set is too large to download)
Messages
Total messages: 11 (3 generated)
|