diff options
author | 2016-02-25 10:28:11 -0800 | |
---|---|---|
committer | 2016-02-25 10:28:11 -0800 | |
commit | f6bfccd562f4ac071db9902097ad4bea83294085 (patch) | |
tree | 817d77894f0c3c68b02605addf11d1648187bd5f /include/core/SkRect.h | |
parent | d77061067af664a158058771189d152a30acd59a (diff) |
Cleanups related to SkFixed.
This CL relands https://codereview.chromium.org/1683743005/ with two fixes:
- Removing the test for vertical text. Vertical text doesn't work on Windows and breakText isn't supported for non-trivial text.
- Omit PaintBreakTextTest in Google3 because we don't have the correct font setup yet.
Remove SK_FixedNaN. It does not seem to be supported or used anywhere in Skia, Chromium, Android, or Google3, (except accidentally by TwoPtRadial::kDontDrawT). I think supporting it would erase any benefit of SkFixed over float.
Remove SkBitmapFilter::lookup. It does not appear to be used anywhere in Skia, Chromium, Android, or Google3.
Fix a bug in SkPaint::breakText that limited it to ~5kB of text. Now it can handle more than 1GB.
BUG=skia:4632
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1683743005
Committed: https://skia.googlesource.com/skia/+/7ea5cb18389db11a94175de95c9db2b44972630c
patch from issue 1683743005 at patchset 120001 (http://crrev.com/1683743005#ps120001)
Review URL: https://codereview.chromium.org/1739453002
Diffstat (limited to 'include/core/SkRect.h')
-rw-r--r-- | include/core/SkRect.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/include/core/SkRect.h b/include/core/SkRect.h index 4f649b7c73..3ebe099ae6 100644 --- a/include/core/SkRect.h +++ b/include/core/SkRect.h @@ -463,8 +463,7 @@ struct SK_API SkRect { /** * Returns true iff all values in the rect are finite. If any are - * infinite or NaN (or SK_FixedNaN when SkScalar is fixed) then this - * returns false. + * infinite or NaN then this returns false. */ bool isFinite() const { float accum = 0; |