Add a simple compositor for Sky

sky/compositor/layer_client.cc

Index: sky/compositor/layer_client.cc
diff --git a/tools/clang/blink_gc_plugin/tests/own_ptr_to_gc_managed_class.cpp b/sky/compositor/layer_client.cc
similarity index 61%
copy from tools/clang/blink_gc_plugin/tests/own_ptr_to_gc_managed_class.cpp
copy to sky/compositor/layer_client.cc
index 9e27c3d87bc66e8e99bb0144815308d23581d308..229482af622fc36315bb94e14b2ffd49d877772e 100644
--- a/tools/clang/blink_gc_plugin/tests/own_ptr_to_gc_managed_class.cpp
+++ b/sky/compositor/layer_client.cc
@@ -2,10 +2,11 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "own_ptr_to_gc_managed_class.h"
+#include "sky/compositor/layer_client.h"
 
-namespace blink {
-
-void HeapObject::trace(Visitor* visitor) { }
+namespace sky {
 
+LayerClient::~LayerClient() {

[ojan, 2014/11/20 21:25:46]

Does chromium style require we have a cc file here? Seems kinda silly to not
just put the empty destructor in the header?

[eseidel, 2014/11/20 21:35:09]

Destructors are deceptive.  All the member destructors are called from this.  In
this object's case that's like 5 destructors worth.  That ends up compiling into
every .o file and then needing to be stripped by the linker.  Much better to be
in a separate file imo.

[abarth-chromium, 2014/11/20 22:07:20]

It also makes sure layer_client.h compiles without any other headers included
first.  I think the style guide allows us to include {} in the header though.

 }
+
+}  // namespace sky