Changeset 109080 in webkit


Ignore:
Timestamp:
Feb 27, 2012 10:49:21 PM (12 years ago)
Author:
commit-queue@webkit.org
Message:

kCTFontTableOptionExcludeSynthetic is unneeded
https://bugs.webkit.org/show_bug.cgi?id=79744

Patch by Ned Holbrook <nholbrook@apple.com> on 2012-02-27
Reviewed by Dan Bernstein.

The aforementioned option is a no-op, so no new tests.

  • platform/graphics/mac/SimpleFontDataMac.mm:

(WebCore::fontHasVerticalGlyphs): Specify no options.

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r109076 r109080  
     12012-02-27  Ned Holbrook  <nholbrook@apple.com>
     2
     3        kCTFontTableOptionExcludeSynthetic is unneeded
     4        https://bugs.webkit.org/show_bug.cgi?id=79744
     5
     6        Reviewed by Dan Bernstein.
     7
     8        The aforementioned option is a no-op, so no new tests.
     9
     10        * platform/graphics/mac/SimpleFontDataMac.mm:
     11        (WebCore::fontHasVerticalGlyphs): Specify no options.
     12
    1132012-02-27  Wei James  <james.wei@intel.com>
    214
  • trunk/Source/WebCore/platform/graphics/mac/SimpleFontDataMac.mm

    r104545 r109080  
    5858{
    5959    // The check doesn't look neat but this is what AppKit does for vertical writing...
    60     RetainPtr<CFArrayRef> tableTags(AdoptCF, CTFontCopyAvailableTables(ctFont, kCTFontTableOptionExcludeSynthetic));
     60    RetainPtr<CFArrayRef> tableTags(AdoptCF, CTFontCopyAvailableTables(ctFont, kCTFontTableOptionNoOptions));
    6161    CFIndex numTables = CFArrayGetCount(tableTags.get());
    6262    for (CFIndex index = 0; index < numTables; ++index) {
     
    7272    if (!fontData->platformData().cgFont())
    7373        return false;
    74 
    7574
    7675    return true;
Note: See TracChangeset for help on using the changeset viewer.