Rename kPMColor_SkColorType to kN32_SkColorType.

include/core/SkImageInfo.h

Index: include/core/SkImageInfo.h
diff --git a/include/core/SkImageInfo.h b/include/core/SkImageInfo.h
index 1c73d959af54d55852516ca369bd933591a22655..7fee31102f820ca75be485b319662160eb0bee77 100644
--- a/include/core/SkImageInfo.h
+++ b/include/core/SkImageInfo.h
@@ -80,12 +80,16 @@ enum SkColorType {
     kLastEnum_SkColorType = kIndex_8_SkColorType,
 
 #if SK_PMCOLOR_BYTE_ORDER(B,G,R,A)

[scroggo, 2014/04/07 21:35:49]

In theory we should change this as well. SK_N32_BYTE_ORDER? (It appears Chrome
does not use this one, so we can just change it in one CL.)

-    kPMColor_SkColorType = kBGRA_8888_SkColorType
+    kN32_SkColorType = kBGRA_8888_SkColorType,

[scroggo, 2014/04/07 21:35:49]

I went with your suggesting here, but I'd be happy with k(N | Native)(32 |
8888)_SkColorType.

8888 follows the convention of the other enums (though 32 follows the convention
of the factories).

Native gives a little more info (again N follows the convention of the
factories).

 #elif SK_PMCOLOR_BYTE_ORDER(R,G,B,A)
-    kPMColor_SkColorType = kRGBA_8888_SkColorType
+    kN32_SkColorType = kRGBA_8888_SkColorType,
 #else
 #error "SK_*32_SHFIT values must correspond to BGRA or RGBA byte order"
 #endif
+
+#ifdef SK_SUPPORT_LEGACY_N32_NAME

[reed1, 2014/04/08 13:04:57]

I suggest we ...

1. don't add a guard until we have removed all of skia's references to the old
name
2. add the guard in chrome's skia/skia_common.gypi or our
gyp/skia_for_chrome.gypi

[scroggo, 2014/04/08 14:46:29]

I 100% agree. I'd like to make the change in another patch to this CL, but I
want to make sure you like the new name before doing the search and replace.
Done.

+    kPMColor_SkColorType = kN32_SkColorType
+#endif
 };
 
 static int SkColorTypeBytesPerPixel(SkColorType ct) {