SkColorCubeFilter: require alpha == 0xFF.

src/opts/SkPMFloat_none.h

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 namespace {  // See SkPMFloat.h
 
 inline SkPMFloat::SkPMFloat(SkPMColor c) {
@@ skipping 16 matching lines @@
     b = b < 0 ? 0 : (b > 1 ? 1 : b);
     SkPMColor c = SkPackARGB32(255*a+0.5f, 255*r+0.5f, 255*g+0.5f, 255*b+0.5f);
     SkPMColorAssert(c);
     return c;
 }
 
 inline Sk4f SkPMFloat::alphas() const {
     return Sk4f(this->a());
 }
 
-inline SkPMFloat SkPMFloat::FromBGRx(SkColor c) {
+inline SkPMFloat SkPMFloat::FromOpaqueColor(SkColor c) {
+  SkASSERT(SkColorGetA(c) == 0xFF);
   float inv255 = 1.0f / 255;
   SkPMFloat pmf = SkPMFloat::FromARGB(1.0f,
                                       SkGetPackedR32(c) * inv255,
                                       SkGetPackedG32(c) * inv255,
                                       SkGetPackedB32(c) * inv255);

[Noel Gordon, 2015/08/20 01:43:29]

Yes, that's a better function name, and the SkGetPackedR32 -> SkColorGetR fix
for lines 41-43 is https://codereview.chromium.org/1296383006 and done /
submitted, so all good here.

   SkASSERT(pmf.isValid());
   return pmf;
 }
 
 }  // namespace