PPAPI: Remove out-dated or unnecessary TODOs from public files.

ppapi/cpp/var.cc

Index: ppapi/cpp/var.cc
diff --git a/ppapi/cpp/var.cc b/ppapi/cpp/var.cc
index 7748ff275ce4f547c768e666131f928b8ed93e98..881ce3288e02e93b0997b8b9d30c631f9417c16a 100644
--- a/ppapi/cpp/var.cc
+++ b/ppapi/cpp/var.cc
@@ -219,9 +219,6 @@ std::string Var::DebugString() const {
   } else if (is_bool()) {
     snprintf(buf, sizeof(buf), AsBool() ? "Var(true)" : "Var(false)");
   } else if (is_int()) {
-    // Note that the following static_cast is necessary because
-    // NativeClient's int32_t is actually "long".
-    // TODO(sehr,polina): remove this after newlib is changed.

[dmichael (off chromium), 2012/03/19 17:41:52]

this static_cast is correct; there's no need to remove it. You should never
assume int32_t is some particular type anyway.

     snprintf(buf, sizeof(buf), "Var(%d)", static_cast<int>(AsInt()));
   } else if (is_double()) {
     snprintf(buf, sizeof(buf), "Var(%f)", AsDouble());
@@ -236,8 +233,6 @@ std::string Var::DebugString() const {
     }
     snprintf(buf, sizeof(buf), format, str.c_str());
   } else if (is_array_buffer()) {
-    // TODO(dmichael): We could make this dump hex. Maybe DebugString should be
-    // virtual?

[dmichael (off chromium), 2012/03/19 17:41:52]

Just a nice-to-have.

     snprintf(buf, sizeof(buf), "Var(ARRAY_BUFFER)");
   } else if (is_object()) {
     snprintf(buf, sizeof(buf), "Var(OBJECT)");