| OLD | NEW | 
|---|
| 1 // Copyright 2015 The Chromium Authors. All rights reserved. | 1 // Copyright 2015 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 #include "BlinkGCPluginConsumer.h" | 5 #include "BlinkGCPluginConsumer.h" | 
| 6 | 6 | 
| 7 #include <algorithm> | 7 #include <algorithm> | 
| 8 #include <set> | 8 #include <set> | 
| 9 | 9 | 
| 10 #include "CheckDispatchVisitor.h" | 10 #include "CheckDispatchVisitor.h" | 
| (...skipping 163 matching lines...) Expand 10 before | Expand all | Expand 10 after  Loading... | 
| 174   } | 174   } | 
| 175 | 175 | 
| 176   CheckClass(info); | 176   CheckClass(info); | 
| 177 } | 177 } | 
| 178 | 178 | 
| 179 void BlinkGCPluginConsumer::CheckClass(RecordInfo* info) { | 179 void BlinkGCPluginConsumer::CheckClass(RecordInfo* info) { | 
| 180   if (!info) | 180   if (!info) | 
| 181     return; | 181     return; | 
| 182 | 182 | 
| 183   if (CXXMethodDecl* trace = info->GetTraceMethod()) { | 183   if (CXXMethodDecl* trace = info->GetTraceMethod()) { | 
|  | 184     if (options_.warn_stack_allocated_trace_method && info->IsStackAllocated()) | 
|  | 185       reporter_.TraceMethodForStackAllocatedClass(info, trace); | 
| 184     if (trace->isPure()) | 186     if (trace->isPure()) | 
| 185       reporter_.ClassDeclaresPureVirtualTrace(info, trace); | 187       reporter_.ClassDeclaresPureVirtualTrace(info, trace); | 
| 186   } else if (info->RequiresTraceMethod()) { | 188   } else if (info->RequiresTraceMethod()) { | 
| 187     reporter_.ClassRequiresTraceMethod(info); | 189     reporter_.ClassRequiresTraceMethod(info); | 
| 188   } | 190   } | 
| 189 | 191 | 
| 190   // Check polymorphic classes that are GC-derived or have a trace method. | 192   // Check polymorphic classes that are GC-derived or have a trace method. | 
| 191   if (info->record()->hasDefinition() && info->record()->isPolymorphic()) { | 193   if (info->record()->hasDefinition() && info->record()->isPolymorphic()) { | 
| 192     // TODO: Check classes that inherit a trace method. | 194     // TODO: Check classes that inherit a trace method. | 
| 193     CXXMethodDecl* trace = info->GetTraceMethod(); | 195     CXXMethodDecl* trace = info->GetTraceMethod(); | 
| (...skipping 521 matching lines...) Expand 10 before | Expand all | Expand 10 after  Loading... | 
| 715   SourceLocation spelling_location = source_manager.getSpellingLoc(loc); | 717   SourceLocation spelling_location = source_manager.getSpellingLoc(loc); | 
| 716   PresumedLoc ploc = source_manager.getPresumedLoc(spelling_location); | 718   PresumedLoc ploc = source_manager.getPresumedLoc(spelling_location); | 
| 717   if (ploc.isInvalid()) { | 719   if (ploc.isInvalid()) { | 
| 718     // If we're in an invalid location, we're looking at things that aren't | 720     // If we're in an invalid location, we're looking at things that aren't | 
| 719     // actually stated in the source. | 721     // actually stated in the source. | 
| 720     return false; | 722     return false; | 
| 721   } | 723   } | 
| 722   *filename = ploc.getFilename(); | 724   *filename = ploc.getFilename(); | 
| 723   return true; | 725   return true; | 
| 724 } | 726 } | 
| OLD | NEW | 
|---|