Optimize GetPathForVectorIcon*, save ~290 KB on disk

ui/gfx/vector_icon_types.h

Index: ui/gfx/vector_icon_types.h
diff --git a/ui/gfx/vector_icon_types.h b/ui/gfx/vector_icon_types.h
index 8def67d2fcc7899a2cb28daa6242e2d8533f4728..8e09523061c3fcc95fb8feb1cdd322c44e01e493 100644
--- a/ui/gfx/vector_icon_types.h
+++ b/ui/gfx/vector_icon_types.h
@@ -64,8 +64,8 @@ enum CommandType {
 
 // A POD that describes either a path command or an argument for it.
 struct PathElement {
-  PathElement(CommandType value) : type(value) {}
-  PathElement(SkScalar value) : arg(value) {}
+  constexpr PathElement(CommandType value) : type(value) {}

[Evan Stade, 2017/01/10 17:30:26]

I believe your tree is slightly out of date :)

[brucedawson, 2017/01/10 18:31:54]

What bizarre timing that I made this change the same day that you landed the
same thing.

constexpr all the things!

+  constexpr PathElement(SkScalar value) : arg(value) {}
 
   union {
     CommandType type;