Update from https://crrev.com/312398

tools/clang/plugins/ChromeClassTester.cpp

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // A general interface for filtering and only acting on classes in Chromium C++
 // code.
 
 #include "ChromeClassTester.h"
 
 #include <sys/param.h>
@@ skipping 44 matching lines @@
 
   return true;  // true means continue parsing.
 }
 
 void ChromeClassTester::CheckTag(TagDecl* tag) {
   // We handle class types here where we have semantic information. We can only
   // check structs/classes/enums here, but we get a bunch of nice semantic
   // information instead of just parsing information.
 
   if (CXXRecordDecl* record = dyn_cast<CXXRecordDecl>(tag)) {
-    // If this is a POD or a class template or a type dependent on a
-    // templated class, assume there's no ctor/dtor/virtual method
-    // optimization that we can do.
-    if (record->isPOD() ||
-        record->getDescribedClassTemplate() ||
-        record->getTemplateSpecializationKind() ||
-        record->isDependentType())
-      return;
-
     if (InBannedNamespace(record))
       return;
 
     SourceLocation record_location = record->getInnerLocStart();
     if (InBannedDirectory(record_location))
       return;
 
     // We sadly need to maintain a blacklist of types that violate these
     // rules, but do so for good reason or due to limitations of this
     // checker (i.e., we don't handle extern templates very well).
@@ skipping 218 matching lines @@
   PresumedLoc ploc = source_manager.getPresumedLoc(spelling_location);
   if (ploc.isInvalid()) {
     // If we're in an invalid location, we're looking at things that aren't
     // actually stated in the source.
     return false;
   }
 
   *filename = ploc.getFilename();
   return true;
 }