diff options
author | asvitkine@chromium.org <asvitkine@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-13 18:55:25 +0000 |
---|---|---|
committer | asvitkine@chromium.org <asvitkine@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-13 18:55:25 +0000 |
commit | 79ba16881712dc74d764cc8dbbe4e59c73cad754 (patch) | |
tree | 6e9620ea1e0f9661ae6f2a677020448fe0f6b649 /chrome_elf | |
parent | b882b2ee87dc7ba66f10393a9dd92406aee85247 (diff) | |
download | chromium_src-79ba16881712dc74d764cc8dbbe4e59c73cad754.zip chromium_src-79ba16881712dc74d764cc8dbbe4e59c73cad754.tar.gz chromium_src-79ba16881712dc74d764cc8dbbe4e59c73cad754.tar.bz2 |
Revert 250828 "Add a UMA stat to track if the Browser blacklist ..."
Speculative revert for failures here:
http://build.chromium.org/p/chromium.win/builders/Win7%20Sync%20x64/builds/11201
> Add a UMA stat to track if the Browser blacklist is Set on the Renderer
>
> This shouldn't be happening, but we got some crash reports suggesting it
> does. Unable to repo locally so this stat will verify it does occur and
> then can be used to verify our fixes actually fix it.
>
> BUG=329023
>
> Review URL: https://codereview.chromium.org/140763008
TBR=csharp@chromium.org
Review URL: https://codereview.chromium.org/163633005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@251095 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome_elf')
-rw-r--r-- | chrome_elf/blacklist/blacklist.cc | 11 | ||||
-rw-r--r-- | chrome_elf/blacklist/blacklist.h | 3 | ||||
-rw-r--r-- | chrome_elf/blacklist/test/blacklist_test.cc | 4 | ||||
-rw-r--r-- | chrome_elf/blacklist/test/blacklist_test_main_dll.def | 3 | ||||
-rw-r--r-- | chrome_elf/chrome_elf.def | 1 |
5 files changed, 1 insertions, 21 deletions
diff --git a/chrome_elf/blacklist/blacklist.cc b/chrome_elf/blacklist/blacklist.cc index 32d3cb2..ea140c4 100644 --- a/chrome_elf/blacklist/blacklist.cc +++ b/chrome_elf/blacklist/blacklist.cc @@ -60,10 +60,6 @@ enum WOW64Status { WOW64_UNKNOWN, }; -// Record if the blacklist was successfully initialized so processes can easily -// determine if the blacklist is enabled for them. -bool g_blacklist_initialized = false; - WOW64Status GetWOW64StatusForCurrentProcess() { typedef BOOL (WINAPI* IsWow64ProcessFunc)(HANDLE, PBOOL); IsWow64ProcessFunc is_wow64_process = reinterpret_cast<IsWow64ProcessFunc>( @@ -276,10 +272,6 @@ int BlacklistSize() { return size; } -bool IsBlacklistInitialized() { - return g_blacklist_initialized; -} - bool AddDllToBlacklist(const wchar_t* dll_name) { int blacklist_size = BlacklistSize(); // We need to leave one space at the end for the null pointer. @@ -381,9 +373,6 @@ bool Initialize(bool force) { } #endif - // Record that we have initialized the blacklist. - g_blacklist_initialized = true; - BYTE* thunk_storage = reinterpret_cast<BYTE*>(&g_thunk_storage); // Mark the thunk storage as readable and writeable, since we diff --git a/chrome_elf/blacklist/blacklist.h b/chrome_elf/blacklist/blacklist.h index 2e21f20..5237a5c 100644 --- a/chrome_elf/blacklist/blacklist.h +++ b/chrome_elf/blacklist/blacklist.h @@ -61,9 +61,6 @@ bool ResetBeacon(); // Return the size of the current blacklist. int BlacklistSize(); -// Returns if true if the blacklist has been initialized. -extern "C" bool IsBlacklistInitialized(); - // Adds the given dll name to the blacklist. Returns true if the dll name is in // the blacklist when this returns, false on error. Note that this will copy // |dll_name| and will leak it on exit if the string is not subsequently removed diff --git a/chrome_elf/blacklist/test/blacklist_test.cc b/chrome_elf/blacklist/test/blacklist_test.cc index 39db737..3a881ad 100644 --- a/chrome_elf/blacklist/test/blacklist_test.cc +++ b/chrome_elf/blacklist/test/blacklist_test.cc @@ -32,7 +32,6 @@ extern "C" { // functions on the test blacklist dll, not the ones linked into the test // executable itself. __declspec(dllimport) bool TestDll_AddDllToBlacklist(const wchar_t* dll_name); -__declspec(dllimport) bool TestDLL_IsBlacklistInitialized(); __declspec(dllimport) bool TestDll_RemoveDllFromBlacklist( const wchar_t* dll_name); } @@ -121,9 +120,6 @@ TEST_F(BlacklistTest, LoadBlacklistedLibrary) { base::FilePath current_dir; ASSERT_TRUE(PathService::Get(base::DIR_EXE, ¤t_dir)); - // Ensure that the blacklist is loaded. - ASSERT_TRUE(TestDLL_IsBlacklistInitialized()); - // Test that an un-blacklisted DLL can load correctly. base::ScopedNativeLibrary dll1(current_dir.Append(kTestDllName1)); EXPECT_TRUE(dll1.is_valid()); diff --git a/chrome_elf/blacklist/test/blacklist_test_main_dll.def b/chrome_elf/blacklist/test/blacklist_test_main_dll.def index 82e0f0e..63522a0 100644 --- a/chrome_elf/blacklist/test/blacklist_test_main_dll.def +++ b/chrome_elf/blacklist/test/blacklist_test_main_dll.def @@ -6,6 +6,5 @@ LIBRARY "blacklist_test_main_dll.dll" EXPORTS TestDll_AddDllToBlacklist=AddDllToBlacklist - TestDLL_IsBlacklistInitialized=IsBlacklistInitialized TestDll_RemoveDllFromBlacklist=RemoveDllFromBlacklist - InitBlacklistTestDll
\ No newline at end of file + InitBlacklistTestDll diff --git a/chrome_elf/chrome_elf.def b/chrome_elf/chrome_elf.def index ee9808f..3e88cfa 100644 --- a/chrome_elf/chrome_elf.def +++ b/chrome_elf/chrome_elf.def @@ -6,5 +6,4 @@ LIBRARY "chrome_elf.dll" EXPORTS CreateFileW=CreateFileWRedirect - IsBlacklistInitialized SignalChromeElf |