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

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

Issue 374593002: Support ignorance of base class finalizers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 5 months 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
1 // Copyright 2014 The Chromium Authors. All rights reserved. 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 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CLASS_REQUIRES_FINALIZATION_BASE_H_ 5 #ifndef CLASS_REQUIRES_FINALIZATION_BASE_H_
6 #define CLASS_REQUIRES_FINALIZATION_BASE_H_ 6 #define CLASS_REQUIRES_FINALIZATION_BASE_H_
7 7
8 #include "heap/stubs.h" 8 #include "heap/stubs.h"
9 9
10 namespace WebCore { 10 namespace WebCore {
11 11
12 class A : public GarbageCollected<A> { 12 class A : public GarbageCollected<A> {
13 public: 13 public:
14 virtual void trace(Visitor*) {} 14 virtual void trace(Visitor*) {}
15 }; 15 };
16 16
17 class B { 17 class B {
18 public: 18 public:
19 ~B() { /* user-declared, thus, non-trivial */ } 19 ~B() { /* user-declared, thus, non-trivial */ }
20 }; 20 };
21 21
22 class ScriptWrappable {
23 public:
24 ~ScriptWrappable() { /* user-declared, thus, non-trivial */ }
25 };
26
22 // Second base class needs finalization. 27 // Second base class needs finalization.
23 class NeedsFinalizer : public A, public B { 28 class NeedsFinalizer : public A, public B {
24 public: 29 public:
25 void trace(Visitor*); 30 void trace(Visitor*);
26 }; 31 };
27 32
28 // Base does not need finalization. 33 // Base does not need finalization.
29 class DoesNotNeedFinalizer : public A { 34 class DoesNotNeedFinalizer : public A {
30 public: 35 public:
31 void trace(Visitor*); 36 void trace(Visitor*);
32 }; 37 };
33 38
39 class ClassWithIgnoredBasesDoesNotNeedFinalizer
40 : public A, public ScriptWrappable {
41 public:
42 void trace(Visitor*);
43 };
44
45 class ClassWithSomeButNotAllIgnoredBasesDoesNeedFinalizer
46 : public A, public B, public ScriptWrappable {
47 public:
48 void trace(Visitor*);
49 };
50
34 } 51 }
35 52
36 #endif 53 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698