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

Sync changes from mozilla-central gfx/wr #3949

Merged
merged 1 commit into from May 13, 2020
Merged

Conversation

@moz-gfx
Copy link

moz-gfx commented May 13, 2020

No description provided.

…ying synthetic-bold, to avoid unreliable subpixel positioning. r=jrmuizel

If we pass an array of glyph IDs (all the same) and positions (with 1-pixel offsets) to CTFontDrawGlyphs
to generate the rasterized image for a synthetic-bold glyph, it's possible that -- depending on the
fractional-pixel x position and the character coverage of the font -- it may misposition the rendering
of one (or more?) of the copies of the glyph bitmap within the context, such that we lose the intended
double-struck/offset effect.

The problem doesn't occur if we render the exact same glyphs and positions to the context using
individual single-glyph calls to CTFontDrawGlyphs.

Differential Revision: https://phabricator.services.mozilla.com/D74918

[ghsync] From https://hg.mozilla.org/mozilla-central/rev/fd7f3a8bba92b52058f3a3d8f3a86632dc993d6b
@moz-gfx
Copy link
Author

moz-gfx commented May 13, 2020

@bors-servo r=auto

@bors-servo
Copy link
Contributor

bors-servo commented May 13, 2020

📌 Commit c784bf3 has been approved by auto

@bors-servo
Copy link
Contributor

bors-servo commented May 13, 2020

Testing commit c784bf3 with merge 205b57a...

@bors-servo
Copy link
Contributor

bors-servo commented May 13, 2020

☀️ Test successful - status-taskcluster
Approved by: auto
Pushing 205b57a to master...

@bors-servo bors-servo merged commit 205b57a into servo:master May 13, 2020
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
Community-TC (pull_request) TaskGroup: success
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.