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

include/core/SkBitmap.h

 /*
  * Copyright 2006 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkBitmap_DEFINED
 #define SkBitmap_DEFINED
 
@@ skipping 65 matching lines @@
 
     ///////////////////////////////////////////////////////////////////////////
 
     const SkImageInfo& info() const { return fInfo; }
 
     int width() const { return fInfo.width(); }
     int height() const { return fInfo.height(); }
     SkColorType colorType() const { return fInfo.colorType(); }
     SkAlphaType alphaType() const { return fInfo.alphaType(); }
 
+    SkIRect shape() const { return fInfo.shape(); }

[reed1, 2014/10/23 14:39:44]

We already have getBounds(SkIRect*). I'm fine if we want a variant that returns
the rect, but I don't see the value of introducing a new label like "shape".

[hal.canary, 2014/10/24 18:23:31]

Acknowledged.

+    SkISize dimensions() const { return fInfo.dimensions(); }
+
     /**
      *  Return the number of bytes per pixel based on the colortype. If the colortype is
      *  kUnknown_SkColorType, then 0 is returned.
      */
     int bytesPerPixel() const { return fInfo.bytesPerPixel(); }
 
     /**
      *  Return the rowbytes expressed as a number of pixels (like width and height).
      *  If the colortype is kUnknown_SkColorType, then 0 is returned.
      */
@@ skipping 778 matching lines @@
 
 inline SkPMColor SkBitmap::getIndex8Color(int x, int y) const {
     SkASSERT(fPixels);
     SkASSERT(kIndex_8_SkColorType == this->colorType());
     SkASSERT((unsigned)x < (unsigned)this->width() && (unsigned)y < (unsigned)this->height());
     SkASSERT(fColorTable);
     return (*fColorTable)[*((const uint8_t*)fPixels + y * fRowBytes + x)];
 }
 
 #endif