Revert[2] of "add backdrop option to SaveLayerRec"

Revert[2] of "add backdrop option to SaveLayerRec"

Reverted because of picture/serialization failure

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1567063002

TBR=mtklein

Committed: https://skia.googlesource.com/skia/+/bfd5f171e6a3eccd7c4bede652a85fd76bcbce2a

[reed1, 2016-01-07 17:23:37]

Description was changed from

==========
Revert[2] of "add backdrop option to SaveLayerRec"

This reverts commit 0575cb2def2f4dfdad04e0674d0f8eb8e7c24cec.

BUG=skia:
==========

to

==========
Revert[2] of "add backdrop option to SaveLayerRec"

This reverts commit 0575cb2def2f4dfdad04e0674d0f8eb8e7c24cec.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[reed1, 2016-01-07 17:24:11]

Description was changed from

==========
Revert[2] of "add backdrop option to SaveLayerRec"

This reverts commit 0575cb2def2f4dfdad04e0674d0f8eb8e7c24cec.

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Revert[2] of "add backdrop option to SaveLayerRec"

Reverted because of picture/serialization failure

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[reed1, 2016-01-07 17:24:16]

The CQ bit was checked by reed@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-01-07 17:24:25]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1567063002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1567063002/1

[commit-bot: I haz the power, 2016-01-07 17:31:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-07 17:31:45]

Dry run: Try jobs failed on following builders:
  Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-Shared-Trybot on client.skia
(JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2...)

[reed1, 2016-01-07 18:20:56]

The CQ bit was checked by reed@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-01-07 18:21:04]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1567063002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1567063002/20001

[reed1, 2016-01-07 18:24:46]

reed@google.com changed reviewers:
+ mtklein@google.com

[mtklein, 2016-01-07 18:27:42]

https://codereview.chromium.org/1567063002/diff/20001/src/core/SkRecordOpts.cpp
File src/core/SkRecordOpts.cpp (right):

https://codereview.chromium.org/1567063002/diff/20001/src/core/SkRecordOpts.c...
src/core/SkRecordOpts.cpp:255: static bool KillSaveLayerAndRestore(SkRecord*
record, int saveLayerIndex) {
This one too?

[commit-bot: I haz the power, 2016-01-07 18:41:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-07 18:41:34]

Dry run: This issue passed the CQ dry run.

[reed1, 2016-01-07 19:09:18]

ptal

https://codereview.chromium.org/1567063002/diff/20001/src/core/SkRecordOpts.cpp
File src/core/SkRecordOpts.cpp (right):

https://codereview.chromium.org/1567063002/diff/20001/src/core/SkRecordOpts.c...
src/core/SkRecordOpts.cpp:255: static bool KillSaveLayerAndRestore(SkRecord*
record, int saveLayerIndex) {
On 2016/01/07 18:27:42, mtklein wrote:
> This one too?

Done.

[reed1, 2016-01-07 19:09:22]

The CQ bit was checked by reed@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-01-07 19:09:26]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1567063002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1567063002/40001

[reed1, 2016-01-07 19:26:58]

Description was changed from

==========
Revert[2] of "add backdrop option to SaveLayerRec"

Reverted because of picture/serialization failure

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Revert[2] of "add backdrop option to SaveLayerRec"

Reverted because of picture/serialization failure

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

TBR=mtklein
==========

[reed1, 2016-01-07 19:27:07]

The CQ bit was unchecked by reed@google.com

[reed1, 2016-01-07 19:27:15]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2016-01-07 19:27:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1567063002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1567063002/40001

[mtklein, 2016-01-07 19:27:27]

lgtm

[commit-bot: I haz the power, 2016-01-07 19:28:12]

Description was changed from

==========
Revert[2] of "add backdrop option to SaveLayerRec"

Reverted because of picture/serialization failure

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

TBR=mtklein
==========

to

==========
Revert[2] of "add backdrop option to SaveLayerRec"

Reverted because of picture/serialization failure

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

TBR=mtklein

Committed:
https://skia.googlesource.com/skia/+/bfd5f171e6a3eccd7c4bede652a85fd76bcbce2a
==========

[commit-bot: I haz the power, 2016-01-07 19:28:13]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/bfd5f171e6a3eccd7c4bede652a85fd76bcbce2a