Back PaintRecord with PaintOpBuffer instead of SkPicture

cc/paint/paint_flags.cc

Index: cc/paint/paint_flags.cc
diff --git a/cc/paint/paint_flags.cc b/cc/paint/paint_flags.cc
new file mode 100644
index 0000000000000000000000000000000000000000..e16a8bb8073b501d5538513b986713c3de7acf9e
--- /dev/null
+++ b/cc/paint/paint_flags.cc
@@ -0,0 +1,42 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "cc/paint/paint_flags.h"
+
+namespace cc {
+
+bool PaintFlags::IsSimpleOpacity() const {

[vmpstr, 2017/03/28 18:27:14]

It might be worth adding a comment somewhere in PaintFlags that if we're adding
new things, this might need to be updated.

[enne (OOO), 2017/03/28 19:28:05]

This function is *on* PaintFlags.  It seems like if you're modifying the class,
you'll need to keep all of its member functions working properly, just by
default?

+  uint32_t color = getColor();
+  if (SK_ColorTRANSPARENT != SkColorSetA(color, SK_AlphaTRANSPARENT))
+    return false;
+  if (!isSrcOver())
+    return false;
+  if (getLooper())
+    return false;
+  if (getPathEffect())
+    return false;
+  if (getShader())
+    return false;
+  if (getMaskFilter())
+    return false;
+  if (getColorFilter())
+    return false;
+  if (getImageFilter())
+    return false;
+  return true;
+}
+
+bool PaintFlags::SupportsFoldingAlpha() const {
+  if (!isSrcOver())
+    return false;
+  if (getColorFilter())
+    return false;
+  if (getImageFilter())
+    return false;
+  if (getLooper())
+    return false;
+  return true;
+}
+
+}  // namespace cc