commit f622a6c8fd176acf9944de8df00d7f0bb56b67d3
author halcanary <halcanary@google.com> Fri Oct 24 12:54:53 2014 -0700
committer Commit bot <commit-bot@chromium.org> Fri Oct 24 12:54:53 2014 -0700
tree 11a136e816a7dfd26804f199c5d123a28e70c896
parent 87ab51862731ab0cd017e63d828fef31a193be40

Add `SkIRect bounds()` convenience method to SkImageInfo and SkBitmap.

Review URL: https://codereview.chromium.org/646213003

src/lazy/SkCachingPixelRef.cpp

diff --git a/src/lazy/SkCachingPixelRef.cpp b/src/lazy/SkCachingPixelRef.cpp
index 1459567..5ab9656 100644
--- a/src/lazy/SkCachingPixelRef.cpp
+++ b/src/lazy/SkCachingPixelRef.cpp
@@ -45,9 +45,8 @@
     }
 
     const SkImageInfo& info = this->info();
-    if (!SkBitmapCache::Find(this->getGenerationID(),
-                             SkIRect::MakeWH(info.width(), info.height()),
-                             &fLockedBitmap)) {
+    if (!SkBitmapCache::Find(
+                this->getGenerationID(), info.bounds(), &fLockedBitmap)) {
         // Cache has been purged, must re-decode.
         if (!fLockedBitmap.tryAllocPixels(info, fRowBytes)) {
             fErrorInDecoding = true;
@@ -58,9 +57,8 @@
             return false;
         }
         fLockedBitmap.setImmutable();
-        SkBitmapCache::Add(this->getGenerationID(),
-                           SkIRect::MakeWH(info.width(), info.height()),
-                           fLockedBitmap);
+        SkBitmapCache::Add(
+                this->getGenerationID(), info.bounds(), fLockedBitmap);
     }
 
     // Now bitmap should contain a concrete PixelRef of the decoded image.