Expand pixelref to return SkImageInfo and rowbytes

tests/BitmapGetColorTest.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "Test.h"
 #include "SkBitmap.h"
 #include "SkRect.h"
 #include "SkRandom.h"
 
 static int nextRand(SkRandom& rand, int min, int max) {
     return min + (int)rand.nextRangeU(0, max - min);
 }
 
 static void rand_irect(SkIRect* rect, int W, int H, SkRandom& rand) {
     const int DX = W / 2;
     const int DY = H / 2;
 
     rect->fLeft   = nextRand(rand, -DX, W + DX);
     rect->fTop    = nextRand(rand, -DY, H + DY);
     rect->fRight  = nextRand(rand, -DX, W + DX);
     rect->fBottom = nextRand(rand, -DY, H + DY);
     rect->sort();
 }
 
 static void test_equal_A1_A8(skiatest::Reporter* reporter,
                        const SkBitmap& bm1, const SkBitmap& bm8) {
+    return;

[scroggo, 2013/11/20 21:04:28]

This doesn't hit any of our warnings?

[reed1, 2013/11/20 21:27:18]

A1 no longer supported

[scroggo, 2013/11/20 21:35:19]

Can we go ahead and remove this whole function?

I was thinking that once this gets submitted it might give us a warning on the
bots for unused code.

     SkASSERT(SkBitmap::kA1_Config == bm1.config());
     SkASSERT(SkBitmap::kA8_Config == bm8.config());
 
     REPORTER_ASSERT(reporter, bm1.width() == bm8.width());
     REPORTER_ASSERT(reporter, bm1.height() == bm8.height());
     for (int y = 0; y < bm1.height(); ++y) {
         for (int x = 0; x < bm1.width(); ++x) {
             int p1 = *bm1.getAddr1(x, y) & (1 << (7 - (x & 7)));
             SkASSERT(SkIsPow2(p1));
             p1 = p1 ? 0xFF : 0;
@@ skipping 64 matching lines @@
 
         SkColor c = bm.getColor(1, 1);
         REPORTER_ASSERT(reporter, c == gRec[i].fOutColor);
     }
 
     test_eraserect_A1(reporter);
 }
 
 #include "TestClassDef.h"
 DEFINE_TESTCLASS("GetColor", TestGetColorClass, TestGetColor)