cc : Delete target id and content target id from transform nodes

cc/trees/draw_property_utils.cc

Index: cc/trees/draw_property_utils.cc
diff --git a/cc/trees/draw_property_utils.cc b/cc/trees/draw_property_utils.cc
index 85d73a5e71de2778a62e119f9a56501d08c80a60..63c65ac811bb1dcae76f3c23dce436e3c708fa69 100644
--- a/cc/trees/draw_property_utils.cc
+++ b/cc/trees/draw_property_utils.cc
@@ -25,8 +25,6 @@ namespace cc {
 
 namespace draw_property_utils {
 
-namespace {

[jaydasika, 2017/03/21 21:25:55]

I don't see any reason to have a separate namespace here. If there is a reason
for it, let me know, I will add it back.

[weiliangc, 2017/03/21 21:34:57]

It is used since these are local helper functions.
https://google.github.io/styleguide/cppguide.html#Unnamed_Namespaces_and_Stat...

[jaydasika, 2017/03/21 22:05:36]

It says either make the functions static OR put them in a unnamed namespace. We
seem to be doing both. But, it does make sense to group them together, so added
back the namespace. Also, there are many more functions in this file that are
local to this file, added them also.

-
 static bool IsRootLayer(const Layer* layer) {
   return !layer->parent();
 }
@@ -404,10 +402,6 @@ static bool LayerNeedsUpdateInternal(LayerType* layer,
   return true;
 }
 
-
-
-}  // namespace
-
 template <typename LayerType>
 static inline bool LayerShouldBeSkippedInternal(
     LayerType* layer,