blink_gc_plugin: report illegal on-heap iterators as warnings/errors.

tools/clang/blink_gc_plugin/tests/legacy_naming/fields_illegal_tracing.txt

Index: tools/clang/blink_gc_plugin/tests/legacy_naming/fields_illegal_tracing.txt
diff --git a/tools/clang/blink_gc_plugin/tests/legacy_naming/fields_illegal_tracing.txt b/tools/clang/blink_gc_plugin/tests/legacy_naming/fields_illegal_tracing.txt
index 61dc6a2fbe94af04748c75f0ef8c06fa1a55f3ba..874be1ce8acc1e4c2646fff1062ac0fe6bca49d4 100644
--- a/tools/clang/blink_gc_plugin/tests/legacy_naming/fields_illegal_tracing.txt
+++ b/tools/clang/blink_gc_plugin/tests/legacy_naming/fields_illegal_tracing.txt
@@ -11,10 +11,10 @@ class PartObject {
 ./fields_illegal_tracing.h:40:5: note: [blink-gc] std::unique_ptr field 'm_obj4' to a GC managed class declared here:
     std::unique_ptr<HeapObject> m_obj4;
     ^
-./fields_illegal_tracing.h:42:5: note: [blink-gc] Iterator field 'm_iterator2' to a GC managed collection declared here:
+./fields_illegal_tracing.h:42:5: warning: [blink-gc] Iterator field 'm_iterator2' to a GC managed collection declared here:
     HeapVector<Member<HeapObject>>::iterator m_iterator2;
     ^
-./fields_illegal_tracing.h:43:5: note: [blink-gc] Iterator field 'm_iterator3' to a GC managed collection declared here:
+./fields_illegal_tracing.h:43:5: warning: [blink-gc] Iterator field 'm_iterator3' to a GC managed collection declared here:
     HeapHashSet<PartObject>::const_iterator m_iterator3;
     ^
 ./fields_illegal_tracing.h:46:1: warning: [blink-gc] Class 'HeapObject' contains invalid fields.
@@ -29,13 +29,13 @@ class HeapObject : public GarbageCollectedFinalized<HeapObject> {
 ./fields_illegal_tracing.h:54:5: note: [blink-gc] std::unique_ptr field 'm_obj4' to a GC managed class declared here:
     std::unique_ptr<HeapObject> m_obj4;
     ^
-./fields_illegal_tracing.h:55:5: note: [blink-gc] Iterator field 'm_iterator3' to a GC managed collection declared here:
+./fields_illegal_tracing.h:55:5: warning: [blink-gc] Iterator field 'm_iterator3' to a GC managed collection declared here:
     HeapHashMap<int, Member<HeapObject>>::reverse_iterator m_iterator3;
     ^
-./fields_illegal_tracing.h:56:5: note: [blink-gc] Iterator field 'm_iterator4' to a GC managed collection declared here:
+./fields_illegal_tracing.h:56:5: warning: [blink-gc] Iterator field 'm_iterator4' to a GC managed collection declared here:
     HeapDeque<Member<HeapObject>>::const_reverse_iterator m_iterator4;
     ^
-./fields_illegal_tracing.h:58:5: note: [blink-gc] Iterator field 'm_iterator6' to a GC managed collection declared here:
+./fields_illegal_tracing.h:58:5: warning: [blink-gc] Iterator field 'm_iterator6' to a GC managed collection declared here:
     HeapLinkedHashSet<Member<HeapObject>>::const_iterator m_iterator6;
     ^
 fields_illegal_tracing.cpp:9:1: warning: [blink-gc] Class 'PartObject' has untraced or not traceable fields.
@@ -65,4 +65,4 @@ void HeapObject::trace(Visitor* visitor) {
 ./fields_illegal_tracing.h:57:5: note: [blink-gc] Untraced field 'm_iterator5' declared here:
     HeapListHashSet<Member<HeapObject>>::const_iterator m_iterator5;
     ^
-4 warnings generated.
+9 warnings generated.