Skip to content

/ servo

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

Jump to bottom

The Font instance cache should be consulted when building FontGroup #203

Closed
burg opened this issue Nov 13, 2012 · 1 comment
Closed

The Font instance cache should be consulted when building FontGroup #203

burg opened this issue Nov 13, 2012 · 1 comment
Labels
A-platform/fonts E-easy I-perf-bloat I-perf-slow I-refactor

Comments

@burg
Copy link

@burg burg commented Nov 13, 2012

Right now it doesn't which, will really mess up performance.

To fix this, simply make up a dummy FontDescriptor, then ask for an instance from the FontContext.

Realistically, this depends on having a non-monomorphic cache. An LRU cache implementation is tracked by Issue #51.
@sethfowler sethfowler mentioned this issue Jun 18, 2013
Merged
@0b10011
Copy link
Contributor

@0b10011 0b10011 commented Jul 17, 2013

This should be closed now, correct?
@jdm jdm closed this Jul 17, 2013
ChrisParis pushed a commit to ChrisParis/servo that referenced this issue Sep 7, 2014
@odinho
Merge branch 'pr/203', Pull request servo#203 web-platform-tests/wpt#203
82bf073 


TestTWF, kaseijin
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-platform/fonts E-easy I-perf-bloat I-perf-slow I-refactor
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@jdm @burg @0b10011
You can’t perform that action at this time.