Revert of add backdrop option to SaveLayerRec

src/core/SkPictureRecord.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkPictureRecord.h"
 #include "SkDevice.h"
 #include "SkImage_Base.h"
@@ skipping 58 matching lines @@
         at this time (and may not be able to afford since during record our
         clip starts out the size of the picture, which is often much larger
         than the size of the actual device we'll use during playback).
      */
     return kNoLayer_SaveLayerStrategy;
 }
 
 void SkPictureRecord::recordSaveLayer(const SaveLayerRec& rec) {
     fContentInfo.onSaveLayer();
 
-    // op + flatflags
+    // op + bool for 'bounds'
     size_t size = 2 * kUInt32Size;
-    uint32_t flatFlags = 0;
+    if (rec.fBounds) {
+        size += sizeof(*rec.fBounds); // + rect
+    }
+    // + paint index + flags
+    size += 2 * kUInt32Size;
 
-    if (rec.fBounds) {
-        flatFlags |= SAVELAYERREC_HAS_BOUNDS;
-        size += sizeof(*rec.fBounds);
-    }
-    if (rec.fPaint) {
-        flatFlags |= SAVELAYERREC_HAS_PAINT;
-        size += sizeof(uint32_t); // index
-    }
-    if (rec.fBackdrop) {
-        flatFlags |= SAVELAYERREC_HAS_BACKDROP;
-        size += sizeof(uint32_t); // (paint) index
-    }
-    if (rec.fSaveLayerFlags) {
-        flatFlags |= SAVELAYERREC_HAS_FLAGS;
-        size += sizeof(uint32_t);
-    }
+    size_t initialOffset = this->addDraw(SAVE_LAYER_SAVELAYERFLAGS, &size);
+    this->addRectPtr(rec.fBounds);
+    this->addPaintPtr(rec.fPaint);
+    this->addInt(rec.fSaveLayerFlags);
 
-    const size_t initialOffset = this->addDraw(SAVE_LAYER_SAVELAYERREC, &size);
-    this->addInt(flatFlags);
-    if (flatFlags & SAVELAYERREC_HAS_BOUNDS) {
-        this->addRect(*rec.fBounds);
-    }
-    if (flatFlags & SAVELAYERREC_HAS_PAINT) {
-        this->addPaintPtr(rec.fPaint);
-    }
-    if (flatFlags & SAVELAYERREC_HAS_BACKDROP) {
-        // overkill, but we didn't already track single flattenables, so using a paint for that
-        SkPaint paint;
-        paint.setImageFilter(const_cast<SkImageFilter*>(rec.fBackdrop));
-        this->addPaint(paint);
-    }
-    if (flatFlags & SAVELAYERREC_HAS_FLAGS) {
-        this->addInt(rec.fSaveLayerFlags);
-    }
     this->validate(initialOffset, size);
 }
 
 #ifdef SK_DEBUG
 /*
  * Read the op code from 'offset' in 'writer' and extract the size too.
  */
 static DrawType peek_op_and_size(SkWriter32* writer, size_t offset, uint32_t* size) {
     uint32_t peek = writer->readTAt<uint32_t>(offset);
 
@@ skipping 115 matching lines @@
         offset = peek;
     }
 
 #ifdef SK_DEBUG
     // offset of 0 has been disabled, so we skip it
     if (offset > 0) {
         // assert that the final offset value points to a save verb
         uint32_t opSize;
         DrawType drawOp = peek_op_and_size(&fWriter, -offset, &opSize);
         SkASSERT(SAVE_LAYER_SAVEFLAGS_DEPRECATED != drawOp);
-        SkASSERT(SAVE_LAYER_SAVELAYERFLAGS_DEPRECATED_JAN_2016 != drawOp);
-        SkASSERT(SAVE == drawOp || SAVE_LAYER_SAVELAYERREC == drawOp);
+        SkASSERT(SAVE == drawOp || SAVE_LAYER_SAVELAYERFLAGS == drawOp);
     }
 #endif
 }
 
 void SkPictureRecord::beginRecording() {
     // we have to call this *after* our constructor, to ensure that it gets
     // recorded. This is balanced by restoreToCount() call from endRecording,
     // which in-turn calls our overridden restore(), so those get recorded too.
     fInitialSaveCount = this->save();
 }
@@ skipping 693 matching lines @@
 void SkPictureRecord::addTextBlob(const SkTextBlob *blob) {
     int index = fTextBlobRefs.count();
     *fTextBlobRefs.append() = blob;
     blob->ref();
     // follow the convention of recording a 1-based index
     this->addInt(index + 1);
 }
 
 ///////////////////////////////////////////////////////////////////////////////