Pack and align SkStrokeRec to 4-byte boundary.

include/core/SkStrokeRec.h

Index: include/core/SkStrokeRec.h
diff --git a/include/core/SkStrokeRec.h b/include/core/SkStrokeRec.h
index 50810f7324c67904cb47309470ebb0b561633e4d..fa9d90b07eb282fb44a1caa6ff5ca05735ed011b 100644
--- a/include/core/SkStrokeRec.h
+++ b/include/core/SkStrokeRec.h
@@ -35,8 +35,8 @@ public:
     Style getStyle() const;
     SkScalar getWidth() const { return fWidth; }
     SkScalar getMiter() const { return fMiterLimit; }
-    SkPaint::Cap getCap() const { return fCap; }
-    SkPaint::Join getJoin() const { return fJoin; }
+    SkPaint::Cap getCap() const { return (SkPaint::Cap)fCap; }
+    SkPaint::Join getJoin() const { return (SkPaint::Join)fJoin; }
 
     bool isHairlineStyle() const {
         return kHairline_Style == this->getStyle();
@@ -115,9 +115,11 @@ private:
     SkScalar        fResScale;
     SkScalar        fWidth;
     SkScalar        fMiterLimit;
-    SkPaint::Cap    fCap;
-    SkPaint::Join   fJoin;
-    bool            fStrokeAndFill;
+    struct {

[mtklein, 2015/11/20 14:57:25]

So, anonymous structs are not technically part of C++.  (Anonymous unions are
legal in C++ and C, and anonymous structs became legal in C11 but not C++11).
Chrome will yell at you.  :|

It's a little silly because they're supported by all our compilers, and
sometimes make for more readable code.

On the other hand, I don't think you need this here.  You should be fine to just
put the uint32_t ... inline with the rest of the fields.  They'll pack because
they're adjacent.

You might also consider:
    uint8_t  fCap           : 2;  // SkPaint::Cap
    uint8_t  fJoin          : 2;  // SkPaint::Join
    bool     fStrokeAndFill : 1;
    uint32_t fUnusedBits    : 27;

+        uint32_t    fCap : 16;
+        uint32_t    fJoin : 15;
+        uint32_t    fStrokeAndFill : 1;
+    };
 };
 
 #endif