mirror of
https://github.com/servo/servo.git
synced 2025-08-06 14:10:11 +01:00
Auto merge of #26002 - servo:regressions, r=jdm
Add Layout 2020 reftests regression report This shows WPT reftests that fail in Servo with Layout 2020 but pass with Layout 2013, together with screenshots of the corresponding runs and HTML source code for the test and its reference. https://community-tc.services.mozilla.com/api/index/v1/task/project.servo.layout-2020-regressions-report/artifacts/public/regressions.html redirects to the latest report for the `master` branch.
This commit is contained in:
bors-servo
GitHub
commit
ca29399bab
6 changed files with 326 additions and 19 deletions
1
.gitignore
vendored
1
.gitignore
vendored
|
|
@ -6,6 +6,7 @@
|
|||
/.vs
|
||||
/android-toolchains
|
||||
/target
|
||||
/etc/layout-2020-regressions/regressions.html
|
||||
/ports/android/bin
|
||||
/ports/android/libs
|
||||
/ports/android/local.properties
|
||||
|
|
|
|||
142
etc/layout-2020-regressions/gen.py
Executable file
142
etc/layout-2020-regressions/gen.py
Executable file
|
|
@ -0,0 +1,142 @@
|
|||
#!/usr/bin/env python
|
||||
|
||||
# This Source Code Form is subject to the terms of the Mozilla Public
|
||||
# License, v. 2.0. If a copy of the MPL was not distributed with this
|
||||
# file, You can obtain one at https://mozilla.org/MPL/2.0/.
|
||||
|
||||
import json
|
||||
import os
|
||||
import re
|
||||
import sys
|
||||
import urllib.request
|
||||
from html import escape as html_escape
|
||||
|
||||
|
||||
TASKCLUSTER_ROOT_URL = "https://community-tc.services.mozilla.com"
|
||||
|
||||
|
||||
def fetch(url):
|
||||
url = TASKCLUSTER_ROOT_URL + "/api/" + url
|
||||
print("Fetching " + url)
|
||||
response = urllib.request.urlopen(url)
|
||||
assert response.getcode() == 200
|
||||
return response
|
||||
|
||||
|
||||
def fetch_json(url):
|
||||
with fetch(url) as response:
|
||||
return json.load(response)
|
||||
|
||||
|
||||
def task(platform, chunk, key):
|
||||
return "index/v1/task/project.servo.%s_wpt_%s.%s" % (platform, chunk, key)
|
||||
|
||||
|
||||
def failing_reftests(platform, key):
|
||||
chunk_1_task_id = fetch_json(task(platform, 1, key))["taskId"]
|
||||
name = fetch_json("queue/v1/task/" + chunk_1_task_id)["metadata"]["name"]
|
||||
match = re.search("WPT chunk (\d+) / (\d+)", name)
|
||||
assert match.group(1) == "1"
|
||||
total_chunks = int(match.group(2))
|
||||
|
||||
for chunk in range(1, total_chunks + 1):
|
||||
with fetch(task(platform, chunk, key) + "/artifacts/public/test-wpt.log") as response:
|
||||
for line in response:
|
||||
message = json.loads(line)
|
||||
if message.get("status") not in {None, "OK", "PASS"}:
|
||||
screenshots = message.get("extra", {}).get("reftest_screenshots")
|
||||
if screenshots:
|
||||
yield message["test"], screenshots
|
||||
|
||||
|
||||
def main(index_key, commit_sha):
|
||||
failures_2013 = {url for url, _ in failing_reftests("linux_x64", index_key)}
|
||||
failures_2020 = Directory()
|
||||
for url, screenshots in failing_reftests("linux_x64_2020", index_key):
|
||||
if url not in failures_2013:
|
||||
assert url.startswith("/")
|
||||
failures_2020.add(url[1:], screenshots)
|
||||
|
||||
here = os.path.dirname(__file__)
|
||||
with open(os.path.join(here, "prism.js")) as f:
|
||||
prism_js = f.read()
|
||||
with open(os.path.join(here, "prism.css")) as f:
|
||||
prism_css = f.read()
|
||||
with open(os.path.join(here, "regressions.html"), "w", encoding="utf-8") as html:
|
||||
os.chdir(os.path.join(here, "../../tests/wpt"))
|
||||
html.write("""
|
||||
<!doctype html>
|
||||
<meta charset=utf-8>
|
||||
<title>Layout 2020 regressions</title>
|
||||
<link rel=stylesheet href=prism.css>
|
||||
<style>
|
||||
ul { padding-left: 1em }
|
||||
li { list-style: "⯈ " }
|
||||
li.expanded { list-style: "⯆ " }
|
||||
li:not(.expanded) > ul, li:not(.expanded) > div { display: none }
|
||||
li > div { display: grid; grid-gap: 1em; grid-template-columns: 1fr 1fr }
|
||||
li > div > p { grid-column: span 2 }
|
||||
li > div > img { grid-row: 2; width: 300px; box-shadow: 0 0 10px }
|
||||
li > div > img:hover { transform: scale(3); transform-origin: 0 0 }
|
||||
li > div > pre { grid-row: 3; font-size: 12px !important }
|
||||
pre code { white-space: pre-wrap !important }
|
||||
%s
|
||||
</style>
|
||||
<h1>Layout 2020 regressions in tree <code>%s</code></h1>
|
||||
""" % (prism_css, commit_sha))
|
||||
failures_2020.write(html)
|
||||
html.write("""
|
||||
<script>
|
||||
for (let li of document.getElementsByTagName("li")) {
|
||||
li.addEventListener('click', event => {
|
||||
li.classList.toggle("expanded")
|
||||
event.stopPropagation()
|
||||
})
|
||||
}
|
||||
%s
|
||||
</script>
|
||||
""" % prism_js)
|
||||
|
||||
|
||||
class Directory:
|
||||
def __init__(self):
|
||||
self.count = 0
|
||||
self.contents = {}
|
||||
|
||||
def add(self, path, screenshots):
|
||||
self.count += 1
|
||||
first, _, rest = path.partition("/")
|
||||
if rest:
|
||||
self.contents.setdefault(first, Directory()).add(rest, screenshots)
|
||||
else:
|
||||
assert path not in self.contents
|
||||
self.contents[path] = screenshots
|
||||
|
||||
def write(self, html):
|
||||
html.write("<ul>\n")
|
||||
for k, v in self.contents.items():
|
||||
html.write("<li><code>%s</code>\n" % k)
|
||||
if isinstance(v, Directory):
|
||||
html.write("<strong>%s</strong>\n" % v.count)
|
||||
v.write(html)
|
||||
else:
|
||||
a, rel, b = v
|
||||
html.write("<div>\n<p><code>%s</code> %s <code>%s</code></p>\n"
|
||||
% (a["url"], rel, b["url"]))
|
||||
for side in [a, b]:
|
||||
html.write("<img src='data:image/png;base64,%s'>\n" % side["screenshot"])
|
||||
url = side["url"]
|
||||
prefix = "/_mozilla/"
|
||||
if url.startswith(prefix):
|
||||
filename = "mozilla/tests/" + url[len(prefix):]
|
||||
else:
|
||||
filename = "web-platform-tests" + url
|
||||
with open(filename, encoding="utf-8") as f:
|
||||
src = html_escape(f.read())
|
||||
html.write("<pre><code class=language-html>%s</code></pre>\n" % src)
|
||||
html.write("</li>\n")
|
||||
html.write("</ul>\n")
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
sys.exit(main(*sys.argv[1:]))
|
||||
141
etc/layout-2020-regressions/prism.css
Normal file
141
etc/layout-2020-regressions/prism.css
Normal file
|
|
@ -0,0 +1,141 @@
|
|||
/* PrismJS 1.19.0
|
||||
https://prismjs.com/download.html#themes=prism&languages=markup+css+clike+javascript */
|
||||
/**
|
||||
* prism.js default theme for JavaScript, CSS and HTML
|
||||
* Based on dabblet (http://dabblet.com)
|
||||
* @author Lea Verou
|
||||
*/
|
||||
|
||||
code[class*="language-"],
|
||||
pre[class*="language-"] {
|
||||
color: black;
|
||||
background: none;
|
||||
text-shadow: 0 1px white;
|
||||
font-family: Consolas, Monaco, 'Andale Mono', 'Ubuntu Mono', monospace;
|
||||
font-size: 1em;
|
||||
text-align: left;
|
||||
white-space: pre;
|
||||
word-spacing: normal;
|
||||
word-break: normal;
|
||||
word-wrap: normal;
|
||||
line-height: 1.5;
|
||||
|
||||
-moz-tab-size: 4;
|
||||
-o-tab-size: 4;
|
||||
tab-size: 4;
|
||||
|
||||
-webkit-hyphens: none;
|
||||
-moz-hyphens: none;
|
||||
-ms-hyphens: none;
|
||||
hyphens: none;
|
||||
}
|
||||
|
||||
pre[class*="language-"]::-moz-selection, pre[class*="language-"] ::-moz-selection,
|
||||
code[class*="language-"]::-moz-selection, code[class*="language-"] ::-moz-selection {
|
||||
text-shadow: none;
|
||||
background: #b3d4fc;
|
||||
}
|
||||
|
||||
pre[class*="language-"]::selection, pre[class*="language-"] ::selection,
|
||||
code[class*="language-"]::selection, code[class*="language-"] ::selection {
|
||||
text-shadow: none;
|
||||
background: #b3d4fc;
|
||||
}
|
||||
|
||||
@media print {
|
||||
code[class*="language-"],
|
||||
pre[class*="language-"] {
|
||||
text-shadow: none;
|
||||
}
|
||||
}
|
||||
|
||||
/* Code blocks */
|
||||
pre[class*="language-"] {
|
||||
padding: 1em;
|
||||
margin: .5em 0;
|
||||
overflow: auto;
|
||||
}
|
||||
|
||||
:not(pre) > code[class*="language-"],
|
||||
pre[class*="language-"] {
|
||||
background: #f5f2f0;
|
||||
}
|
||||
|
||||
/* Inline code */
|
||||
:not(pre) > code[class*="language-"] {
|
||||
padding: .1em;
|
||||
border-radius: .3em;
|
||||
white-space: normal;
|
||||
}
|
||||
|
||||
.token.comment,
|
||||
.token.prolog,
|
||||
.token.doctype,
|
||||
.token.cdata {
|
||||
color: slategray;
|
||||
}
|
||||
|
||||
.token.punctuation {
|
||||
color: #999;
|
||||
}
|
||||
|
||||
.token.namespace {
|
||||
opacity: .7;
|
||||
}
|
||||
|
||||
.token.property,
|
||||
.token.tag,
|
||||
.token.boolean,
|
||||
.token.number,
|
||||
.token.constant,
|
||||
.token.symbol,
|
||||
.token.deleted {
|
||||
color: #905;
|
||||
}
|
||||
|
||||
.token.selector,
|
||||
.token.attr-name,
|
||||
.token.string,
|
||||
.token.char,
|
||||
.token.builtin,
|
||||
.token.inserted {
|
||||
color: #690;
|
||||
}
|
||||
|
||||
.token.operator,
|
||||
.token.entity,
|
||||
.token.url,
|
||||
.language-css .token.string,
|
||||
.style .token.string {
|
||||
color: #9a6e3a;
|
||||
background: hsla(0, 0%, 100%, .5);
|
||||
}
|
||||
|
||||
.token.atrule,
|
||||
.token.attr-value,
|
||||
.token.keyword {
|
||||
color: #07a;
|
||||
}
|
||||
|
||||
.token.function,
|
||||
.token.class-name {
|
||||
color: #DD4A68;
|
||||
}
|
||||
|
||||
.token.regex,
|
||||
.token.important,
|
||||
.token.variable {
|
||||
color: #e90;
|
||||
}
|
||||
|
||||
.token.important,
|
||||
.token.bold {
|
||||
font-weight: bold;
|
||||
}
|
||||
.token.italic {
|
||||
font-style: italic;
|
||||
}
|
||||
|
||||
.token.entity {
|
||||
cursor: help;
|
||||
}
|
||||
7
etc/layout-2020-regressions/prism.js
Normal file
7
etc/layout-2020-regressions/prism.js
Normal file
File diff suppressed because one or more lines are too long
|
|
@ -48,6 +48,7 @@ def tasks(task_for):
|
|||
],
|
||||
"master": [
|
||||
upload_docs,
|
||||
layout_2020_regressions_report,
|
||||
],
|
||||
|
||||
# The "try-*" keys match those in `servo_try_choosers` in Homu’s config:
|
||||
|
|
@ -186,7 +187,7 @@ def linux_tidy_unit():
|
|||
./etc/ci/lockfile_changed.sh
|
||||
./etc/ci/check_no_panic.sh
|
||||
""")
|
||||
.find_or_create("linux_unit." + CONFIG.task_id())
|
||||
.find_or_create("linux_unit." + CONFIG.tree_hash())
|
||||
)
|
||||
|
||||
|
||||
|
|
@ -216,12 +217,12 @@ def linux_docs_check():
|
|||
./mach check
|
||||
""")
|
||||
.with_artifacts("/repo/target/doc/docs.bundle")
|
||||
.find_or_create("docs." + CONFIG.task_id())
|
||||
.find_or_create("docs." + CONFIG.tree_hash())
|
||||
)
|
||||
|
||||
|
||||
def upload_docs():
|
||||
docs_build_task_id = decisionlib.Task.find("docs." + CONFIG.task_id())
|
||||
docs_build_task_id = decisionlib.Task.find("docs." + CONFIG.tree_hash())
|
||||
return (
|
||||
linux_task("Upload docs to GitHub Pages")
|
||||
.with_treeherder("Linux x64", "DocUpload")
|
||||
|
|
@ -247,6 +248,21 @@ def upload_docs():
|
|||
)
|
||||
|
||||
|
||||
def layout_2020_regressions_report():
|
||||
return (
|
||||
linux_task("Layout 2020 regressions report")
|
||||
.with_treeherder("Linux x64", "RegressionsReport")
|
||||
.with_dockerfile(dockerfile_path("base"))
|
||||
.with_repo_bundle()
|
||||
.with_script(
|
||||
"python3 etc/layout-2020-regressions/gen.py %s %s"
|
||||
% (CONFIG.tree_hash(), CONFIG.git_sha)
|
||||
)
|
||||
.with_index_and_artifacts_expire_in(log_artifacts_expire_in)
|
||||
.with_artifacts("/repo/etc/layout-2020-regressions/regressions.html")
|
||||
.find_or_create("layout-2020-regressions-report")
|
||||
)
|
||||
|
||||
def macos_unit():
|
||||
return (
|
||||
macos_build_task("Dev build + unit tests")
|
||||
|
|
@ -257,7 +273,7 @@ def macos_unit():
|
|||
./mach package --dev
|
||||
./etc/ci/lockfile_changed.sh
|
||||
""")
|
||||
.find_or_create("macos_unit." + CONFIG.task_id())
|
||||
.find_or_create("macos_unit." + CONFIG.tree_hash())
|
||||
)
|
||||
|
||||
|
||||
|
|
@ -302,7 +318,7 @@ def windows_arm64():
|
|||
"python mach package --dev --target aarch64-uwp-windows-msvc --uwp=arm64",
|
||||
)
|
||||
.with_artifacts(appx_artifact(debug=True))
|
||||
.find_or_create("build.windows_uwp_arm64_dev." + CONFIG.task_id())
|
||||
.find_or_create("build.windows_uwp_arm64_dev." + CONFIG.tree_hash())
|
||||
)
|
||||
|
||||
|
||||
|
|
@ -318,7 +334,7 @@ def windows_uwp_x64():
|
|||
"python mach test-tidy --force-cpp --no-wpt",
|
||||
)
|
||||
.with_artifacts(appx_artifact(debug=True))
|
||||
.find_or_create("build.windows_uwp_x64_dev." + CONFIG.task_id())
|
||||
.find_or_create("build.windows_uwp_x64_dev." + CONFIG.tree_hash())
|
||||
)
|
||||
|
||||
|
||||
|
|
@ -339,7 +355,7 @@ def uwp_nightly():
|
|||
)
|
||||
.with_artifacts(appx_artifact(debug=False))
|
||||
.with_max_run_time_minutes(3 * 60)
|
||||
.find_or_create("build.windows_uwp_nightlies." + CONFIG.task_id())
|
||||
.find_or_create("build.windows_uwp_nightlies." + CONFIG.tree_hash())
|
||||
)
|
||||
|
||||
|
||||
|
|
@ -368,7 +384,7 @@ def windows_unit(cached=True):
|
|||
"repo/target/debug/msi/Servo.zip")
|
||||
)
|
||||
if cached:
|
||||
return task.find_or_create("build.windows_x64_dev." + CONFIG.task_id())
|
||||
return task.find_or_create("build.windows_x64_dev." + CONFIG.tree_hash())
|
||||
else:
|
||||
return task.create()
|
||||
|
||||
|
|
@ -385,7 +401,7 @@ def windows_nightly():
|
|||
"mach upload-nightly windows-msvc --secret-from-taskcluster")
|
||||
.with_artifacts("repo/target/release/msi/Servo.exe",
|
||||
"repo/target/release/msi/Servo.zip")
|
||||
.find_or_create("build.windows_x64_nightly." + CONFIG.task_id())
|
||||
.find_or_create("build.windows_x64_nightly." + CONFIG.tree_hash())
|
||||
)
|
||||
|
||||
|
||||
|
|
@ -402,7 +418,7 @@ def linux_nightly():
|
|||
"./mach upload-nightly linux --secret-from-taskcluster",
|
||||
)
|
||||
.with_artifacts("/repo/target/release/servo-tech-demo.tar.gz")
|
||||
.find_or_create("build.linux_x64_nightly" + CONFIG.task_id())
|
||||
.find_or_create("build.linux_x64_nightly" + CONFIG.tree_hash())
|
||||
)
|
||||
|
||||
|
||||
|
|
@ -414,7 +430,7 @@ def linux_release():
|
|||
"./mach build --release",
|
||||
"./mach package --release",
|
||||
)
|
||||
.find_or_create("build.linux_x64_release" + CONFIG.task_id())
|
||||
.find_or_create("build.linux_x64_release" + CONFIG.tree_hash())
|
||||
)
|
||||
|
||||
|
||||
|
|
@ -433,7 +449,7 @@ def macos_nightly():
|
|||
"./mach upload-nightly mac --secret-from-taskcluster",
|
||||
)
|
||||
.with_artifacts("repo/target/release/servo-tech-demo.dmg")
|
||||
.find_or_create("build.mac_x64_nightly." + CONFIG.task_id())
|
||||
.find_or_create("build.mac_x64_nightly." + CONFIG.tree_hash())
|
||||
)
|
||||
|
||||
|
||||
|
|
@ -476,7 +492,7 @@ def macos_release_build_with_debug_assertions(priority=None):
|
|||
" target/release/build/osmesa-src-*/out/src/mapi/shared-glapi/.libs",
|
||||
]))
|
||||
.with_artifacts("repo/target.tar.gz")
|
||||
.find_or_create("build.macos_x64_release_w_assertions." + CONFIG.task_id())
|
||||
.find_or_create("build.macos_x64_release_w_assertions." + CONFIG.tree_hash())
|
||||
)
|
||||
|
||||
|
||||
|
|
@ -508,7 +524,7 @@ def linux_release_build_with_debug_assertions(layout_2020):
|
|||
.with_artifacts("/target.tar.gz")
|
||||
.find_or_create("build.linux_x64%s_release_w_assertions.%s" % (
|
||||
index_key_suffix,
|
||||
CONFIG.task_id(),
|
||||
CONFIG.tree_hash(),
|
||||
))
|
||||
)
|
||||
|
||||
|
|
@ -658,7 +674,7 @@ def wpt_chunks(platform, make_chunk_task, build_task, total_chunks, processes,
|
|||
platform.replace(" ", "_").lower(),
|
||||
job_id_prefix.replace("-", "_"),
|
||||
this_chunk,
|
||||
CONFIG.task_id(),
|
||||
CONFIG.tree_hash(),
|
||||
))
|
||||
|
||||
|
||||
|
|
|
|||
|
|
@ -64,14 +64,14 @@ class Config:
|
|||
self.default_provisioner_id = "proj-example"
|
||||
|
||||
|
||||
def task_id(self):
|
||||
if not hasattr(self, "_task_id"):
|
||||
def tree_hash(self):
|
||||
if not hasattr(self, "_tree_hash"):
|
||||
# Use the SHA-1 hash of the git "tree" object rather than the commit.
|
||||
# A `@bors-servo retry` command creates a new merge commit with a different commit hash
|
||||
# but with the same tree hash.
|
||||
output = subprocess.check_output(["git", "show", "-s", "--format=%T", "HEAD"])
|
||||
self._task_id = output.decode("utf-8").strip()
|
||||
return self._task_id
|
||||
self._tree_hash = output.decode("utf-8").strip()
|
||||
return self._tree_hash
|
||||
|
||||
def git_sha_is_current_head(self):
|
||||
output = subprocess.check_output(["git", "rev-parse", "HEAD"])
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue