Add SkColor4f serialization

src/core/SkReadBuffer.cpp

 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkBitmap.h"
 #include "SkDeduper.h"
 #include "SkErrorInternals.h"
@@ skipping 124 matching lines @@
     SkASSERT(fReader.available() > 0);
     return *((uint8_t*) fReader.peek());
 }
 
 void SkReadBuffer::readString(SkString* string) {
     size_t len;
     const char* strContents = fReader.readString(&len);
     string->set(strContents, len);
 }
 
+static_assert(SK_SCALAR_IS_FLOAT, "Color4f serialization needs updating");
+void SkReadBuffer::readColor4f(SkColor4f* color) {

[reed1, 2016/09/13 23:17:03]

Is this any simpler/more-efficient?

memcpy(color, fReader.skip(sizeof(SkColor4f)), sizeof(SkColor4f));

We use this pattern for rects, irects etc.

+    color->fR = fReader.readScalar();
+    color->fG = fReader.readScalar();
+    color->fB = fReader.readScalar();
+    color->fA = fReader.readScalar();
+}
+
 void SkReadBuffer::readPoint(SkPoint* point) {
     point->fX = fReader.readScalar();
     point->fY = fReader.readScalar();
 }
 
 void SkReadBuffer::readMatrix(SkMatrix* matrix) {
     fReader.readMatrix(matrix);
 }
 
 void SkReadBuffer::readIRect(SkIRect* rect) {
@@ skipping 30 matching lines @@
 }
 
 bool SkReadBuffer::readByteArray(void* value, size_t size) {
     return readArray(static_cast<unsigned char*>(value), size, sizeof(unsigned char));
 }
 
 bool SkReadBuffer::readColorArray(SkColor* colors, size_t size) {
     return readArray(colors, size, sizeof(SkColor));
 }
 
+bool SkReadBuffer::readColor4fArray(SkColor4f* colors, size_t size) {
+    return readArray(colors, size, sizeof(SkColor4f));
+}
+
 bool SkReadBuffer::readIntArray(int32_t* values, size_t size) {
     return readArray(values, size, sizeof(int32_t));
 }
 
 bool SkReadBuffer::readPointArray(SkPoint* points, size_t size) {
     return readArray(points, size, sizeof(SkPoint));
 }
 
 bool SkReadBuffer::readScalarArray(SkScalar* values, size_t size) {
     return readArray(values, size, sizeof(SkScalar));
@@ skipping 177 matching lines @@
         if (sizeRecorded != sizeRead) {
             this->validate(false);
             return nullptr;
         }
     } else {
         // we must skip the remaining data
         fReader.skip(sizeRecorded);
     }
     return obj.release();
 }