Closed Bug 653100 Opened 9 years ago Closed 9 years ago

remove redundant storing of OpenType table copies, now that OTS doesn't discard them

Categories

(Core :: Graphics, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: jfkthame, Assigned: jfkthame)

References

Details

Attachments

(1 file, 1 obsolete file)

Since bug 643460 landed, we no longer need to cache the OpenType tables from downloaded fonts, bypassing the sanitizer; the tables will remain in the main sfnt structure (unless they're invalid). So we can trim out the code that was responsible for squirrelling away a copy of the tables prior to sanitization.
Attachment #528577 - Flags: review?(jdaggett)
Updated patch to current m-c tip.
Attachment #528577 - Attachment is obsolete: true
Attachment #528577 - Flags: review?(jdaggett)
Attachment #528794 - Flags: review?(jdaggett)
Comment on attachment 528794 [details] [diff] [review]
patch, remove code to save copies of layout tables prior to OTS sanitization - merged to tip

Excellent!!
Attachment #528794 - Flags: review?(jdaggett) → review+
http://hg.mozilla.org/mozilla-central/rev/52b6489a3140
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Depends on: 653408
Backed out:
  http://hg.mozilla.org/mozilla-central/rev/7253ba2c9d6b
because it turned reftests orange due to bug 653408.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Re-landed, hoping it sticks this time:
http://hg.mozilla.org/mozilla-central/rev/b7d79acc2930
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.