Set clip children in LayerTreeHostCommonTest.RenderSurfaceWithUnclippedDescendantsClipsSubtree

Set clip children explicitly in
LayerTreeHostCommonTest.RenderSurfaceWithUnclippedDescendantsClipsSubtree.

clip parent is set in the test. We also need to explicitly
set clip_children of clip parent. The test crashes with this
change. The fix is to set unclipped surface of a clip node
to be equal to the negation of ancestor clips subtree.
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/e9d0970c4ff069bc74194ae9a457c54b6938c98a
Cr-Commit-Position: refs/heads/master@{#348805}

[jaydasika, 2015-09-14 16:02:01]

jaydasika@chromium.org changed reviewers:
+ ajuma@chromium.org

[ajuma, 2015-09-14 17:05:27]

lgtm

[ajuma, 2015-09-14 17:07:03]

https://codereview.chromium.org/1339923006/diff/1/cc/trees/layer_tree_host_co...
File cc/trees/layer_tree_host_common_unittest.cc (right):

https://codereview.chromium.org/1339923006/diff/1/cc/trees/layer_tree_host_co...
cc/trees/layer_tree_host_common_unittest.cc:7892: std::set<LayerImpl*>*
clip_children = new std::set<LayerImpl*>;
Please use a scoped_ptr for this (see other unit tests that call
SetClipChildren).

[jaydasika, 2015-09-14 22:19:55]

The CQ bit was checked by jaydasika@chromium.org

[jaydasika, 2015-09-14 22:19:55]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/1339923006/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-09-14 22:20:20]

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

[commit-bot: I haz the power, 2015-09-14 23:46:35]

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

[commit-bot: I haz the power, 2015-09-14 23:46:36]

Try jobs failed on following builders:
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[jaydasika, 2015-09-15 00:53:05]

The CQ bit was checked by jaydasika@chromium.org

[commit-bot: I haz the power, 2015-09-15 00:53:21]

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

[commit-bot: I haz the power, 2015-09-15 01:26:50]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-15 01:27:38]

Patchset 2 (id:??) landed as
https://crrev.com/e9d0970c4ff069bc74194ae9a457c54b6938c98a
Cr-Commit-Position: refs/heads/master@{#348805}

[commit-bot: I haz the power, 2015-09-23 12:40:57]

Patchset 2 (id:??) landed as
https://crrev.com/e9d0970c4ff069bc74194ae9a457c54b6938c98a
Cr-Commit-Position: refs/heads/master@{#348805}