Bug 69356

Summary: Move font-fixup code in WebKitTestRunner to a better place
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: simon.fraser, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch sam: review+

Simon Fraser (smfr)
Reported 2011-10-04 11:25:27 PDT
Move font-fixup code in WebKitTestRunner to a better place
Attachments
Patch (14.55 KB, patch)
2011-10-04 11:26 PDT, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
Patch (13.45 KB, patch)
2011-10-04 11:29 PDT, Simon Fraser (smfr) 		sam: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2011-10-04 11:26:24 PDT
Created attachment 109656 [details] Patch
Sam Weinig
Comment 2 2011-10-04 11:28:47 PDT
Comment on attachment 109656 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=109656&action=review > Tools/WebKitTestRunner/InjectedBundle/ActivateFonts.h:31 > > -void activateFonts(); > +void adjustFonts(); This should stay named activateFonts, because otherwise you will break everyone else.
Simon Fraser (smfr)
Comment 3 2011-10-04 11:29:48 PDT
Created attachment 109657 [details] Patch
Simon Fraser (smfr)
Comment 4 2011-10-04 11:39:53 PDT
http://trac.webkit.org/changeset/96623
Note You need to log in before you can comment on or make changes to this bug.