Always enable layer hoisting in Skia

Always enable layer hoisting in Skia

This will prevent bit rot of the layer hoisting code and position us better for turning this on in Chromium.

Committed: https://skia.googlesource.com/skia/+/66cad7669be6c47768d084090a3d498014dfc847

[robertphillips, 2014-10-31 12:45:52]

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

[robertphillips, 2014-10-31 17:35:58]

robertphillips@google.com changed reviewers:
+ bsalomon@google.com

[robertphillips, 2014-11-03 14:04:08]

ping

[bsalomon, 2014-11-03 14:22:49]

lgtm

[robertphillips, 2014-11-03 14:31:25]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2014-11-03 14:31:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/698493002/1

[commit-bot: I haz the power, 2014-11-03 14:31:52]

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

[commit-bot: I haz the power, 2014-11-03 14:31:53]

Failed to apply patch for expectations/gm/ignored-tests.txt:
While running git apply --index -3 -p1;
  error: patch failed: expectations/gm/ignored-tests.txt:56
  Falling back to three-way merge...
  <stdin>:27: new blank line at EOF.
  +
  Applied patch to 'expectations/gm/ignored-tests.txt' with conflicts.
  U expectations/gm/ignored-tests.txt
  warning: 1 line adds whitespace errors.

Patch:       expectations/gm/ignored-tests.txt
Index: expectations/gm/ignored-tests.txt
diff --git a/expectations/gm/ignored-tests.txt
b/expectations/gm/ignored-tests.txt
index
9b49b1af85a325b8b20971e7a6839c4dc8ded5e7..0d316875a3f4e077719d2dc8313aaedad4f1a6bd
100644
--- a/expectations/gm/ignored-tests.txt
+++ b/expectations/gm/ignored-tests.txt
@@ -56,3 +56,21 @@ pathfill
 pathopsinverse
 polygons
 rects
+
+# robertphillips
+# multipicturedraw
+multipicturedraw_biglayer_simple
+multipicturedraw_biglayer_tiled
+multipicturedraw_invpathclip_simple
+multipicturedraw_invpathclip_tiled
+multipicturedraw_noclip_simple
+multipicturedraw_noclip_tiled
+multipicturedraw_pathclip_simple
+multipicturedraw_pathclip_tiled
+multipicturedraw_rectclip_simple
+multipicturedraw_rectclip_tiled
+multipicturedraw_rrectclip_simple
+multipicturedraw_rrectclip_tiled
+multipicturedraw_sierpinski_simple
+multipicturedraw_sierpinski_tiled
+

[robertphillips, 2014-11-03 14:50:26]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2014-11-03 14:51:01]

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

[commit-bot: I haz the power, 2014-11-03 14:52:10]

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

[commit-bot: I haz the power, 2014-11-03 14:52:11]

Try jobs failed on following builders:
  Test-Ubuntu13.10-GCE-NoGPU-x86_64-Debug-Trybot on client.skia
(http://build.chromium.org/p/client.skia/builders/Test-Ubuntu13.10-GCE-NoGPU-x...)
  Build-Win-VS2013-x86-Debug-Trybot on client.skia.compile
(http://build.chromium.org/p/client.skia.compile/builders/Build-Win-VS2013-x86...)

[robertphillips, 2014-11-03 15:10:25]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2014-11-03 15:11:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/698493002/60001

[commit-bot: I haz the power, 2014-11-03 15:19:34]

Committed patchset #4 (id:60001) as 66cad7669be6c47768d084090a3d498014dfc847

[mtklein, 2014-11-03 22:33:46]

On 2014/11/03 15:19:34, I haz the power (commit-bot) wrote:
> Committed patchset #4 (id:60001) as 66cad7669be6c47768d084090a3d498014dfc847

Valgrind sez: memory leak!

http://chromegw.corp.google.com/i/client.skia/builders/Test-Ubuntu12-ShuttleA...