Patches on top of Radu's latest.

src/core/SkPMFloat.h

Index: src/core/SkPMFloat.h
diff --git a/src/core/SkPMFloat.h b/src/core/SkPMFloat.h
index f97f25c9c6edf36198805b04269b6496a540c173..1fc80f50044adc756543d944531db03c695a296c 100644
--- a/src/core/SkPMFloat.h
+++ b/src/core/SkPMFloat.h
@@ -26,6 +26,7 @@ class SkPMFloat : public Sk4f {
 public:
     static SkPMFloat FromPMColor(SkPMColor c) { return SkPMFloat(c); }
     static SkPMFloat FromARGB(float a, float r, float g, float b) { return SkPMFloat(a,r,g,b); }
+    static SkPMFloat FromBGRx(SkColor c);  // Ignores c's alpha, instead forcing it to 1.

[Noel Gordon, 2015/08/19 01:02:59]

The CL lacks a decent description, imho.  We could have mentioned the need for
this new function for example, and added a few comments about perf benefits.

[mtklein, 2015/08/19 01:58:22]

Have we not got all that context over at 1273033005?

 
     Sk4f alphas() const;  // argb -> aaaa, generally faster than the equivalent Sk4f(this->a()).