Fix bad regex in CheckForSuperfluousStlIncludesInHeaders presubmit.

It is currently matching more than it should.

Bug: 1200833
Change-Id: I99f7e2cb73cabbc8ed6e862b352abfacc647d4e6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2892529
Auto-Submit: Lei Zhang <thestig@chromium.org>
Commit-Queue: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#882158}

PRESUBMIT.py

@@ -4345,7 +4345,7 @@ def CheckBuildConfigMacrosWithoutInclude(input_api, output_api):
 
 def CheckForSuperfluousStlIncludesInHeaders(input_api, output_api):
   stl_include_re = input_api.re.compile(
-      r'^#include\s+<'
+      r'^#include\s+<('
       r'algorithm|'
       r'array|'
       r'limits|'
@@ -4358,7 +4358,7 @@ def CheckForSuperfluousStlIncludesInHeaders(input_api, output_api):
       r'unordered_map|'
       r'unordered_set|'
       r'utility|'
-      r'vector>')
+      r'vector)>')
   std_namespace_re = input_api.re.compile(r'std::')
   errors = []
   for f in input_api.AffectedFiles():

PRESUBMIT_test.py

@@ -95,6 +95,9 @@ def testGoodFiles(self):
       MockFile('other/path/qux.h',
                ['#include "base/stl_util.h"',
                 'foobar']),
+      MockFile('other/path/baz.h',
+               ['#include "set/vector.h"',
+                'bazzab']),
       # The check is only for header files.
       MockFile('other/path/not_checked.cc',
                ['#include <vector>',