Closed Bug 1149381 Opened 8 years ago Closed 8 years ago

enable the preference for the CSS Font Loading API in release builds

Categories

(Core :: CSS Parsing and Computation, defect)

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla41
Tracking Flags:
Tracking Status
firefox41 --- fixed
relnote-firefox --- 41+

People

(Reporter: heycam, Assigned: heycam)

References

Details

(Keywords: dev-doc-needed)

Attachments

(1 file)

patch
1.07 KB, patch
jtd
: review+
Details | Diff | Splinter Review 
This bug is for enabling the pref for the CSS Font Loading API in release builds.
Depends on: 1161366, 1154150, 1154148, 1152647
Depends on: 1162850
Depends on: 1161413
Attached patch patch — — Splinter Review 
Assignee: nobody → cam
Status: NEW → ASSIGNED

        Attachment #8626791 -
        Flags: review?(jdaggett)

    
    

    
  
Comment on attachment 8626791 [details] [diff] [review]
patch

r+ assuming patches on 1072102 land. I don't think we need to wait for 1161413.

        Attachment #8626791 -
        Flags: review?(jdaggett) → review+

    
  
No longer depends on: 1161413

    
    

    
  
Release Note Request (optional, but appreciated)
[Why is this notable]:
[Suggested wording]:
[Links (documentation, blog post, etc)]: http://dev.w3.org/csswg/css-font-loading/

          relnote-firefox:
          --- → ?

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/bcfa1f801370
Status: ASSIGNED → RESOLVED
Closed: 8 years ago

          status-firefox41:
          --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41

    
    

    
  
Cameron, would this wording be ok for the release notes "Preference for CSS Font Loading API is now enabled"? Thanks!
Flags: needinfo?(cam)

    
    

    
  
John, n-i you also to comment on the suggested wording. Is my proposal in comment #7 ok?
Flags: needinfo?(jdaggett)

    
    

    
  
(In reply to Ritu Kothari (:ritu) from comment #7)
> Cameron, would this wording be ok for the release notes "Preference for CSS
> Font Loading API is now enabled"? Thanks!

How about "CSS Font Loading API (link to spec) is now enabled by default"?

http://dev.w3.org/csswg/css-font-loading/
Flags: needinfo?(jdaggett)

    
    

    
  
John's wording sounds good.
Flags: needinfo?(cam)

    
    

    
  
Thanks John! I will use the wording you suggested with the link.

    
  
Blocks: 1088663
Depends on: 1088437

          You need to log in
          before you can comment on or make changes to this bug.