diff options
author | 2017-10-02 13:44:11 +0000 | |
---|---|---|
committer | 2017-10-02 13:44:18 +0000 | |
commit | 5a2e50edc51006ce91366e177a9d21a16775d7fd (patch) | |
tree | 6301e052ffd179fc8c61681c0f8a0d7b175d4401 /include/core | |
parent | cd284c532376d16fcc4ed75baf3da65c3e4a2e95 (diff) |
Revert "Revert "Revert "guard old apis for querying byte-size of a bitmap/imageinfo/pixmap"""
This reverts commit cd284c532376d16fcc4ed75baf3da65c3e4a2e95.
Reason for revert:
assert fired in SkMallocPixelRef.cpp:61: fatal error: "assert(info.computeByteSize(rowBytes) == info.getSafeSize(rowBytes))"
google3 thinks it was from surface_rowbytes
Original change's description:
> Revert "Revert "guard old apis for querying byte-size of a bitmap/imageinfo/pixmap""
>
> This reverts commit 809cbedd4b252be221b2ac3b4269d312fd8f53a0.
>
> Bug: skia:
> Change-Id: I680d8daeeeeb15526b44c1305d8fb0c6bfa38e1d
> Reviewed-on: https://skia-review.googlesource.com/52665
> Commit-Queue: Mike Reed <reed@google.com>
> Reviewed-by: Florin Malita <fmalita@chromium.org>
TBR=fmalita@chromium.org,reed@google.com
Change-Id: I41e3f7a3f791cc8183291847e783ed8a53bc91d2
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:
Reviewed-on: https://skia-review.googlesource.com/53802
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Reed <reed@google.com>
Diffstat (limited to 'include/core')
-rw-r--r-- | include/core/SkBitmap.h | 6 | ||||
-rw-r--r-- | include/core/SkImageInfo.h | 6 | ||||
-rw-r--r-- | include/core/SkPixmap.h | 2 |
3 files changed, 6 insertions, 8 deletions
diff --git a/include/core/SkBitmap.h b/include/core/SkBitmap.h index cea50b8886..5b4e1a6863 100644 --- a/include/core/SkBitmap.h +++ b/include/core/SkBitmap.h @@ -147,7 +147,10 @@ public: */ size_t computeByteSize() const { return fInfo.computeByteSize(fRowBytes); } -#ifdef SK_SUPPORT_LEGACY_SAFESIZE64 + /** Return the byte size of the pixels, based on the height and rowBytes. + Note this truncates the result to 32bits. Call getSize64() to detect + if the real size exceeds 32bits. + */ size_t getSize() const { return fInfo.height() * fRowBytes; } /** Return the number of bytes from the pointer returned by getPixels() @@ -171,7 +174,6 @@ public: int64_t computeSafeSize64() const { return fInfo.getSafeSize64(fRowBytes); } -#endif /** Returns true if this bitmap is marked as immutable, meaning that the contents of its pixels will not change for the lifetime of the bitmap. diff --git a/include/core/SkImageInfo.h b/include/core/SkImageInfo.h index 50d068a09c..c47dbb28b1 100644 --- a/include/core/SkImageInfo.h +++ b/include/core/SkImageInfo.h @@ -300,7 +300,6 @@ public: void unflatten(SkReadBuffer&); void flatten(SkWriteBuffer&) const; -#ifdef SK_SUPPORT_LEGACY_SAFESIZE64 int64_t getSafeSize64(size_t rowBytes) const { if (0 == fHeight) { return 0; @@ -315,7 +314,6 @@ public: } return sk_64_asS32(size); } -#endif /** * Returns the size (in bytes) of the image buffer that this info needs, given the specified @@ -333,8 +331,8 @@ public: } bool validRowBytes(size_t rowBytes) const { - uint64_t minRB = sk_64_mul(fWidth, this->bytesPerPixel()); - return rowBytes >= minRB; + uint64_t rb = sk_64_mul(fWidth, this->bytesPerPixel()); + return rowBytes >= rb; } void reset() { diff --git a/include/core/SkPixmap.h b/include/core/SkPixmap.h index f304bde933..8115756b91 100644 --- a/include/core/SkPixmap.h +++ b/include/core/SkPixmap.h @@ -206,7 +206,6 @@ public: */ int shiftPerPixel() const { return fInfo.shiftPerPixel(); } -#ifdef SK_SUPPORT_LEGACY_SAFESIZE64 /** Returns conservative memory required for pixel storage. Includes unused memory on last row when rowBytesAsPixels() exceeds width(). @@ -229,7 +228,6 @@ public: @return exact pixel storage size if size fits in signed 32 bits */ size_t getSafeSize() const { return fInfo.getSafeSize(fRowBytes); } -#endif /** * Returns the size (in bytes) of the pixmap's image buffer. |