Add rpcrt4.dll to the list of allowed imports for chrome_elf.dll in the ELFImportsTest.ChromeElfSan…

chrome_elf/elf_imports_unittest.cc

 // Copyright 2014 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.
 
 #include <stdint.h>
 #include <windows.h>
 
 #include <algorithm>
 #include <vector>
 
@@ skipping 59 matching lines @@
 
   std::vector<std::string> elf_imports;
   GetImports(dll, &elf_imports);
 
   // Check that ELF has imports.
   ASSERT_LT(0u, elf_imports.size()) << "Ensure the chrome_elf_unittests "
     "target was built, instead of chrome_elf_unittests.exe";
 
   static const char* const kValidFilePatterns[] = {
     "KERNEL32.dll",
-#if defined(COMPONENT_BUILD)

[krasin, 2016/07/25 21:10:06]

This change should probably be reflected in the CL description (if intended)

[ananta, 2016/07/25 21:11:42]

Done.

-    "MSVC*.dll",
-    "VCRUNTIME*.dll",
-    "api-ms-win-crt-*.dll",
-#endif
+    "RPCRT4.dll",
 #if defined(SYZYASAN)
     "syzyasan_rtl.dll",
 #endif
 #if defined(ADDRESS_SANITIZER) && defined(COMPONENT_BUILD)
     "clang_rt.asan_dynamic-i386.dll",
 #endif
     "ADVAPI32.dll",
     // On 64 bit the Version API's like VerQueryValue come from VERSION.dll.
     // It depends on kernel32, advapi32 and api-ms-win-crt*.dll. This should
     // be ok.
@@ skipping 42 matching lines @@
   ASSERT_LT(0u, exe_imports.size()) << "Ensure the chrome_elf_unittests "
     "target was built, instead of chrome_elf_unittests.exe";
 
   // Chrome.exe's first import must be ELF.
   EXPECT_EQ("chrome_elf.dll", exe_imports[0]) <<
       "Illegal import order in chrome.exe (ensure the chrome_elf_unittest "
       "target was built, instead of just chrome_elf_unittests.exe)";
 }
 
 }  // namespace