Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename QueryOptions.select to QueryOptions.postscriptNames, and update the empty list for postscriptNames match no fonts, instead of matching all fonts. #33406

Merged
merged 1 commit into from
Mar 29, 2022

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Mar 29, 2022

Change-Id: Id361587a0316af3c7512f28c1e60cb47fe3d7611
Bug: 1310341,1310389
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3556637
Reviewed-by: Victor Costan <pwnall@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/main@{#986580}

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

the empty list for postscriptNames match no fonts, instead of matching
all fonts.

Change-Id: Id361587a0316af3c7512f28c1e60cb47fe3d7611
Bug: 1310341,1310389
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3556637
Reviewed-by: Victor Costan <pwnall@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/main@{#986580}
@jonathan-j-lee
Copy link
Contributor

 0:06.86 INFO Downloading ChromeDriver from https://storage.googleapis.com/chromium-browser-snapshots/Linux_x64/986566/chromedriver_linux64.zip
mem avail: 14797 of 15818 MiB (93 %), swap free:    0 of    0 MiB ( 0 %)
mem avail: 14776 of 15818 MiB (93 %), swap free:    0 of    0 MiB ( 0 %)
mem avail: 14857 of 15818 MiB (93 %), swap free:    0 of    0 MiB ( 0 %)
mem avail: 14859 of 15818 MiB (93 %), swap free:    0 of    0 MiB ( 0 %)
...

Looks like a transient failure because I can download chromedriver from that URL now. Retrying.

@jonathan-j-lee jonathan-j-lee merged commit 3836ad1 into master Mar 29, 2022
@jonathan-j-lee jonathan-j-lee deleted the chromium-export-cl-3556637 branch March 29, 2022 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants