Revert of Presubmit: Warn about useless forward declarations

PRESUBMIT_test.py

 #!/usr/bin/env python
 # 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.
 
 import re
 import subprocess
 import unittest
 
 import PRESUBMIT
@@ skipping 1096 matching lines @@
     input_api.files = [
       MockFile('content/file.cc',
                ['char* host = "https://www.aol.com"; // google.com'])
     ]
 
     warnings = PRESUBMIT._CheckHardcodedGoogleHostsInLowerLayers(
       input_api, MockOutputApi())
     self.assertEqual(0, len(warnings))
 
 
-class ForwardDeclarationTest(unittest.TestCase):
-  def testCheckHeadersOnly(self):
-    mock_input_api = MockInputApi()
-    mock_input_api.files = [
-      MockAffectedFile('somewhere/file.cc', [
-        'class DummyClass;'
-      ])
-    ]
-    warnings = PRESUBMIT._CheckUselessForwardDeclarations(mock_input_api,
-      MockOutputApi())
-    self.assertEqual(0, len(warnings))
-
-  def testNoNestedDeclaration(self):
-    mock_input_api = MockInputApi()
-    mock_input_api.files = [
-      MockAffectedFile('somewhere/header.h', [
-        'class SomeClass {'
-        ' protected:'
-        '  class NotAMatch;'
-        '};'
-      ])
-    ]
-    warnings = PRESUBMIT._CheckUselessForwardDeclarations(mock_input_api,
-      MockOutputApi())
-    self.assertEqual(0, len(warnings))
-
-  def testSubStrings(self):
-    mock_input_api = MockInputApi()
-    mock_input_api.files = [
-      MockAffectedFile('somewhere/header.h', [
-        'class NotUsefulClass;',
-        'struct SomeStruct;',
-        'UsefulClass *p1;',
-        'SomeStructPtr *p2;'
-      ])
-    ]
-    warnings = PRESUBMIT._CheckUselessForwardDeclarations(mock_input_api,
-      MockOutputApi())
-    self.assertEqual(2, len(warnings))
-
-  def testUselessForwardDeclaration(self):
-    mock_input_api = MockInputApi()
-    mock_input_api.files = [
-      MockAffectedFile('somewhere/header.h', [
-        'class DummyClass;',
-        'struct DummyStruct;',
-        'class UsefulClass;',
-        'std::unique_ptr<UsefulClass> p;'
-      ]),
-    ]
-    warnings = PRESUBMIT._CheckUselessForwardDeclarations(mock_input_api,
-      MockOutputApi())
-    self.assertEqual(2, len(warnings))
-
-
 if __name__ == '__main__':
   unittest.main()