Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(43)

Side by Side Diff: tools/clang/blink_gc_plugin/tests/base_with_trace_requires_tracing.h

Issue 2536403002: blink_gc_plugin should require tracing for classes that have a trace method
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef BASE_WITH_TRACE_REQUIRES_TRACING_H_
6 #define BASE_WITH_TRACE_REQUIRES_TRACING_H_
7
8 #include "heap/stubs.h"
9
10 namespace blink {
11
12 class A : public GarbageCollected<A> {
13 public:
14 virtual void trace(Visitor*);
15 };
16
17 class B : public A {
18 void trace(Visitor*);
19 private:
20 // Ignoring all fields will mean this does not requiring tracing. But if a
21 // trace method is defined explicitly we should require tracing.
22 GC_PLUGIN_IGNORE("http://crbug.com/12345")
23 Member<A> m_a;
24 };
25
26 class C : public B {
27 public:
28 void trace(Visitor*);
29 private:
30 Member<A> m_a;
31 };
32
33 }
34
35 #endif
OLDNEW
« no previous file with comments | « tools/clang/blink_gc_plugin/RecordInfo.cpp ('k') | tools/clang/blink_gc_plugin/tests/base_with_trace_requires_tracing.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698