Index: tools/clang/blink_gc_plugin/Config.h |
diff --git a/tools/clang/blink_gc_plugin/Config.h b/tools/clang/blink_gc_plugin/Config.h |
index 9341306d0ab1d1bcdf9ad20913cfbb1135f83c81..ed76f1a4bad1da2e850f1598e36efc297a67dfd6 100644 |
--- a/tools/clang/blink_gc_plugin/Config.h |
+++ b/tools/clang/blink_gc_plugin/Config.h |
@@ -4,6 +4,11 @@ |
// This file defines the names used by GC infrastructure. |
+// TODO: Restructure the name determination to use fully qualified names (ala, |
+// blink::Foo) so that the plugin can be enabled for all of chromium. Doing so |
+// would allow us to catch errors with structures outside of blink that might |
+// have unsafe pointers to GC allocated blink structures. |
+ |
#ifndef TOOLS_BLINK_GC_PLUGIN_CONFIG_H_ |
#define TOOLS_BLINK_GC_PLUGIN_CONFIG_H_ |
@@ -94,7 +99,7 @@ class Config { |
// Following http://crrev.com/369633033 (Blink r177436), |
// ignore blink::ScriptWrappable's destructor. |
- // FIXME: remove when its non-Oilpan destructor is removed. |
+ // TODO: remove when its non-Oilpan destructor is removed. |
static bool HasIgnorableDestructor(const std::string& ns, |
const std::string& name) { |
return ns == "blink" && name == "ScriptWrappable"; |