aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/ports
Commit message (Collapse)AuthorAge
* Simplify flattening to just write enough to call the ↵Gravatar reed2014-08-21
| | | | | | | | | | | | | | | | | | factory/public-constructor for the class. We want to *not* rely on private constructors, and not rely on calling through the inheritance hierarchy for either flattening or unflattening(CreateProc). Refactoring pattern: 1. guard the existing constructor(readbuffer) with the legacy build-flag 2. If you are a instancable subclass, implement CreateProc(readbuffer) to create a new instances from the buffer params (or return NULL). If you're a shader subclass 1. You must read/write the local matrix if your class accepts that in its factory/constructor, else ignore it. R=robertphillips@google.com, mtklein@google.com, senorblanco@google.com, senorblanco@chromium.org, sugoi@chromium.org Author: reed@google.com Review URL: https://codereview.chromium.org/395603002
* Remove unecessary check from android framework buildGravatar djsollen2014-08-21
| | | | | | | | R=bungeman@google.com, raph@google.com Author: djsollen@google.com Review URL: https://codereview.chromium.org/495063002
* Use medium filtering when downscaling bitmap glyphs in FreeType hostGravatar djsollen2014-08-21
| | | | | | | | | | This enable Android to save space on emoji glyphs R=bungeman@google.com, raph@google.com Author: djsollen@google.com Review URL: https://codereview.chromium.org/492253002
* Directly compute glyphToUnicode array in DirectWrite.Gravatar bungeman2014-08-20
| | | | | | | | | | | | | | | This new code is still rather bad, but has fewer steps. The motivation for looking at this is that on Windows 8 the previous code was occasionally asserting in such a way as to imply that GetGlyphIndices might return different data the second time through the font. This avoids that particular issue by only going through the font once. R=mtklein@google.com, halcanary@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/472333003
* Remove SK_FONT_HOST_DOES_NOT_USE_FONTMGR and guarded code.Gravatar bungeman2014-08-20
| | | | | | | | | | This define is no longer needed or set, so remove it from the code. R=tomhudson@google.com, djsollen@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/488083004
* Revert to old autohinting behavior on Android.Gravatar djsollen2014-08-20
| | | | | | | | | | | | | | | This CL reverts to the NO_AUTOHINT behavior when autohinting is not explicitly requested. This should be removed when Skia supports a better API for selecting the correct variation. This CL has landed in the Android repo and is being upstreamed to avoid fragmenting the codebase. R=bungeman@google.com Author: djsollen@google.com Review URL: https://codereview.chromium.org/490913003
* Add alias mapping for Noto Sans CJK for ja/hansGravatar jshin2014-08-18
| | | | | | | | | | | | | | | | | Chrome/Chromium OS are getting a brand new CJK fonts (Noto Sans CJK). We want them to be used in place of common Japanese and Simplified Chinese sans-serif fonts. BUG=chromium:399080 TEST=With CrOS CL (https://chromium-review.googlesource.com/#/c/212624/), web pages using 'Simhei' and 'MSP Gothic' are rendered with Noto Sans CJK. R=reed@chromium.org, bungeman@google.com Author: jshin@chromium.org Review URL: https://codereview.chromium.org/476203003
* Remove SkPaintOptionsAndroidGravatar djsollen2014-08-14
| | | | | | | | | | Committed: https://skia.googlesource.com/skia/+/f32331ffdb5de0440bb337aa7cbdd6f33e9ff23b R=reed@google.com, mtklein@google.com, tomhudson@google.com Author: djsollen@google.com Review URL: https://codereview.chromium.org/447873003
* Revert of Remove SkPaintOptionsAndroid (patchset #5 of ↵Gravatar bsalomon2014-08-13
| | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/447873003/) Reason for revert: Breaks the Chromium build: http://108.170.220.120:10117/builders/Canary-Chrome-Ubuntu13.10-Ninja-x86_64-DRT/builds/2469/steps/BuildContentShell_1/logs/stdio Original issue's description: > Remove SkPaintOptionsAndroid > > Committed: https://skia.googlesource.com/skia/+/f32331ffdb5de0440bb337aa7cbdd6f33e9ff23b R=reed@google.com, mtklein@google.com, tomhudson@google.com, djsollen@google.com TBR=djsollen@google.com, mtklein@google.com, reed@google.com, tomhudson@google.com NOTREECHECKS=true NOTRY=true Author: bsalomon@google.com Review URL: https://codereview.chromium.org/473543004
* Remove SkPaintOptionsAndroidGravatar djsollen2014-08-13
| | | | | | | | R=reed@google.com, mtklein@google.com, tomhudson@google.com Author: djsollen@google.com Review URL: https://codereview.chromium.org/447873003
* Update path to Android font config fileGravatar tomhudson2014-08-13
| | | | | | | | | | | | | | | | | | | | | In version LMP, Android is expected to change the path to their font configuration file (and condense several files into one), as well as changing the format (qv http://crrev.com/446473003/). This patch tries the new path before falling back to the old path. The new fonts are incompatible with assumptions made by the old SkFontConfigInterfaceAndroid, so this patch MUST NOT land until https://crrev.com/462073002/ or the equivalent has turned on the new SkFontManager. BUG=chromium:400801 R=djsollen@google.com, bungeman@google.com, tomhudson@google.com Committed: https://skia.googlesource.com/skia/+/94fa4b99e2a53e997a90c7808cc5263f1bf40c9f Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/458543002
* Delete the old font management implementation based on ↵Gravatar tomhudson2014-08-13
| | | | | | | | | | | SkFontConfigInterface, now that SkFontMgr_Android is live. BUG=chromium:400801 R=djsollen@google.com, bungeman@google.com Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/445143002
* Mark fallback fonts in new Android font config parserGravatar tomhudson2014-08-13
| | | | | | | | | | | | | | | | | The new Android fonts.xml format doesn't explicitly tag fallback fonts, or separate them from system fonts by putting them in a separate file. We're assuming that any nameless font is a fallback font. Adds a unit test to track that we see as many fallback fonts as expected when parsing a file. BUG=chromium:400801 R=bungeman@google.com, djsollen@google.com, tomhudson@google.com Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/468893002
* Update path to Android font config fileGravatar tomhudson2014-08-12
| | | | | | | | | | | | | | | | | | | In version LMP, Android is expected to change the path to their font configuration file (and condense several files into one), as well as changing the format (qv http://crrev.com/446473003/). This patch tries the new path before falling back to the old path. The new fonts are incompatible with assumptions made by the old SkFontConfigInterfaceAndroid, so this patch MUST NOT land until https://crrev.com/462073002/ or the equivalent has turned on the new SkFontManager. BUG=chromium:400801 R=djsollen@google.com, bungeman@google.com, tomhudson@google.com Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/458543002
* Fix lower-casing error in font config parserGravatar tomhudson2014-08-12
| | | | | | | | | | | | | The Android font config parser was missing some SkAutoAsciiToLC instantiations; this was discovered in https://crrev.com/458543002/ but should land independently. BUG=chromium:400801 R=bungeman@google.com Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/465973002
* remove SkStippleMaskFilter - no external clientsGravatar reed2014-08-12
| | | | | | | | TBR= Author: reed@google.com Review URL: https://codereview.chromium.org/462013002
* Parses sample code provided by Android project. Attempts to keep FontFamily ↵Gravatar tomhudson2014-08-11
| | | | | | | | | | | | | data structures produced consistent with expectations of previous versions of Skia. R=bungeman@google.com, djsollen@google.com, palmer@chromium.org, reed@google.com, hcm@google.com, tomhudson@google.com BUG=400801 Committed: https://skia.googlesource.com/skia/+/07544757c9fcf0f359f1686a3779eb2e75dd5b36 Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/446473003
* Actually use test configuration in SkFontMgr on Android.Gravatar bungeman2014-08-11
| | | | | | | | | BUG=chromium:401954 R=tomhudson@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/461673002
* Blink layout tests all assume it can provide custom font configuration files ↵Gravatar bungeman2014-08-08
| | | | | | | | | | | | | | on every platform. In moving Android to the font manager, we broke those. They're ugly, but this patch reinstates them. R=tomhudson@google.com, djsollen@google.com, reed@google.com TBR=reed@google.com BUG=chromium:401954 Author: bungeman@google.com Review URL: https://codereview.chromium.org/451093002
* Revert "Remove SkPaintOptionsAndroid"Gravatar Derek Sollenberger2014-08-06
| | | | | | This reverts commit 27fb94999b8eec448423884e1d071e563c4c95d9. Review URL: https://codereview.chromium.org/450513002
* Remove SkPaintOptionsAndroidGravatar Derek Sollenberger2014-08-06
| | | | Review URL: https://codereview.chromium.org/447873003
* Fix reference counting on SkStream in SkFontMgr_android.Gravatar bungeman2014-08-06
| | | | | | | | R=tomhudson@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/447003002
* Implement SkFontMgr_Android::onMatchFamilyStyleCharacter.Gravatar bungeman2014-08-06
| | | | | | | | | | | This will allow us to replace SkGetFallbackFamilyNameForChar and also eventually fix issues with it's reliance on names. R=djsollen@google.com, tomhudson@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/427293003
* Remove references to __system_property_get from SkiaGravatar djsollen2014-08-06
| | | | | | | | | BUG=392201 R=tomhudson@chromium.org, bungeman@google.com, tomhudson@google.com Author: djsollen@google.com Review URL: https://codereview.chromium.org/433163002
* Remove ALL font fallback logic from Skia.Gravatar djsollen2014-08-06
| | | | | | | | R=reed@google.com, bungeman@google.com, caryclark@google.com Author: djsollen@google.com Review URL: https://codereview.chromium.org/434623002
* Parses sample code provided by Android project. Attempts to keep FontFamily ↵Gravatar tomhudson2014-08-05
| | | | | | | | | | data structures produced consistent with expectations of previous versions of Skia. R=bungeman@google.com, djsollen@google.com, tomhudson@google.com Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/446473003
* Test and generalize font configuration parserGravatar tomhudson2014-08-05
| | | | | | | | | | | | Add a basic unit test for the Android font configuration parser. Add a check for the new LMP file format; on detection, switch to a new (as-yet unwritten) parser. R=bungeman@google.com, djsollen@google.com, tomhudson@google.com Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/439813002
* Remove dependency on std::stringGravatar mtklein2014-08-04
| | | | | | | | | BUG=skia: R=bungeman@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/437253003
* Correct Android FontMgr use of ttc index and remove unused code.Gravatar bungeman2014-07-30
| | | | | | | | R=djsollen@chromium.org, tomhudson@google.com, djsollen@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/433503003
* Fix debug output compile error and minor style nitsGravatar tomhudson2014-07-29
| | | | | | | | R=djsollen@google.com, tomhudson@google.com Author: tomhudson@chromium.org Review URL: https://codereview.chromium.org/420333002
* Cleanup: Rename SkOSPath functions.Gravatar tfarina2014-07-28
| | | | | | | | | | | | | | | | Mostly for brevity and matches better with Python: Python | Old C++ | New C++ os.path.join | SkOSPath::SkPathJoin | SkOSPath::Join os.path.basename | SkOSPath::SkBasename | SkOSPath::Basename BUG=None TEST=make all R=mtklein@google.com, bsalomon@google.com Author: tfarina@chromium.org Review URL: https://codereview.chromium.org/428443002
* Only lowercase names in Android font config parser.Gravatar bungeman2014-07-28
| | | | | | | | | BUG=chromium:397549 R=tomhudson@chromium.org, tomhudson@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/420223003
* Remove code only used by SkCreateTypefaceForScript which has already been ↵Gravatar djsollen2014-07-28
| | | | | | | | | | deleted R=tomhudson@google.com, bungeman@google.com Author: djsollen@google.com Review URL: https://codereview.chromium.org/425753002
* Remove redundant helper function as all clients pass localeGravatar Derek Sollenberger2014-07-28
| | | | | | R=tomhudson@google.com Review URL: https://codereview.chromium.org/426643003
* Remove unused code now that the android framework has move this logic elsewhere.Gravatar Derek Sollenberger2014-07-28
| | | | | | R=tomhudson@google.com Review URL: https://codereview.chromium.org/424663005
* Fix thread unsafe mutex initialization.Gravatar bungeman2014-07-25
| | | | | | | | | | | | BUG=skia:2779 Committed: https://skia.googlesource.com/skia/+/d6aeb6dc8fe21066f1a2c4813a4256a3acd3edf5 R=robertphillips@google.com, mtklein@google.com, reed@android.com, bsalomon@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/419113002
* Fix thread unsafe mutex initialization.Gravatar bungeman2014-07-25
| | | | | | | | | BUG=skia:2779 R=robertphillips@google.com, mtklein@google.com, reed@android.com, bsalomon@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/419113002
* SkFontMgr for Android.Gravatar bungeman2014-07-24
| | | | | | | | | | Committed: https://skia.googlesource.com/skia/+/4f9a01e03ef3bbe7dd8f9bebdcb3555226087e06 R=djsollen@google.com, tomhudson@google.com, robertphillips@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/414483002
* Get additional DW font metrics when available.Gravatar bungeman2014-07-23
| | | | | | | | | BUG=chromium:395043 R=eae@chromium.org, caryclark@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/412993002
* Revert of SkFontMgr for Android. (https://codereview.chromium.org/414483002/)Gravatar robertphillips2014-07-22
| | | | | | | | | | | | | | | | | | | Reason for revert: Reverting to unblock DEPS roll. Original issue's description: > SkFontMgr for Android. > > Committed: https://skia.googlesource.com/skia/+/4f9a01e03ef3bbe7dd8f9bebdcb3555226087e06 R=djsollen@google.com, tomhudson@google.com, bungeman@google.com TBR=bungeman@google.com, djsollen@google.com, tomhudson@google.com NOTREECHECKS=true NOTRY=true Author: robertphillips@google.com Review URL: https://codereview.chromium.org/408143010
* SkFontMgr for Android.Gravatar bungeman2014-07-22
| | | | | | | | R=djsollen@google.com, tomhudson@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/414483002
* hide public factory for intermediate classGravatar reed2014-07-21
| | | | | | | | | | Use SK_DECLARE_UNFLATTENABLE_OBJECT if the subclass should not be directly instantiated. TBR= Author: reed@google.com Review URL: https://codereview.chromium.org/399383007
* expand workaround to all 32-bit mips devices without the appropriate gcc supportGravatar djsollen2014-07-18
| | | | | | | | R=halcanary@google.com, reed@google.com, mtklein@google.com Author: djsollen@google.com Review URL: https://codereview.chromium.org/400753004
* Remove SkRefPtr.Gravatar bungeman2014-07-17
| | | | | | | | R=mtklein@google.com, reed@google.com, bsalomon@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/393913002
* Remove ambiguous operator= on AutoCFRelease.Gravatar bungeman2014-07-16
| | | | | | | | | | | | The operator= is ambiguous as to the state of the reference count. Until we have rvalue assignment operators, just use 'reset' as the name of the reference stealing assignment to AutoCFRelease. R=reed@google.com, mtklein@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/400453002
* Fix broken android framework builds where 32-bit MIPS compilers lack 64-bit ↵Gravatar Derek Sollenberger2014-07-16
| | | | | | | | __sync operators. R=mtklein@google.com, reed@google.com, scroggo@google.com Review URL: https://codereview.chromium.org/398153002
* Remove yet-unused 64-bit atomic hooks.Gravatar Mike Klein2014-07-16
| | | | | | | | | | | | For now, I think we'll just be using sk_atomic_inc on 64-bit values, so let's cut out the dead code for now. NOTREECHECKS=true BUG=skia: R=bsalomon@google.com Review URL: https://codereview.chromium.org/398003004
* Leaking CGBitmapContext in SkFontHost_macGravatar ljagielski2014-07-15
| | | | | | | | | | | | | third_party/skia/src/ports/SkFontHost_mac.cpp: don't increase retain count of newly assigned CFTypeRef in AutoCFRelease::reset(). Each time CFTypeRef is assigned to AutoCFRelease, AutoCFRelease becomes its sole owner. BUG=skia: R=reed@google.com, bungeman@google.com, rmistry@google.com, sheyang@chromium.org Author: ljagielski@opera.com Review URL: https://codereview.chromium.org/393513003
* Use va_list variable safelyGravatar qiankun.miao2014-07-15
| | | | | | | | | | | | va_list variable will be invalid after the first call. Use va_copy to initialize the va_list for another call. BUG=skia: R=djsollen@google.com, mtklein@google.com Author: qiankun.miao@intel.com Review URL: https://codereview.chromium.org/394763004
* remove SK_SUPPORT_LEGACY_PIXELREF_UNFLATTENABLE codeGravatar reed2014-07-14
| | | | | | | | TBR= Author: reed@google.com Review URL: https://codereview.chromium.org/387313004