Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(88)

Unified Diff: chrome_elf/chrome_elf_util.cc

Issue 1134253002: Revert of Cache IsNonBrowserProcess values, so we only take a loader-lock the first time. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome_elf/chrome_elf_util.h ('k') | chrome_elf/chrome_elf_util_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_elf/chrome_elf_util.cc
diff --git a/chrome_elf/chrome_elf_util.cc b/chrome_elf/chrome_elf_util.cc
index 275271ee3435b639dde737b42d796e200173f3d1..a547d0bda04730b73f6c50dd6bb5ed3118dda50b 100644
--- a/chrome_elf/chrome_elf_util.cc
+++ b/chrome_elf/chrome_elf_util.cc
@@ -4,13 +4,10 @@
#include "chrome_elf/chrome_elf_util.h"
-#include <assert.h>
#include <windows.h>
#include "base/macros.h"
#include "base/strings/string16.h"
-
-ProcessType g_process_type = ProcessType::UNINITIALIZED;
namespace {
@@ -195,27 +192,18 @@
return false;
}
-void InitializeProcessType() {
- assert(g_process_type == UNINITIALIZED);
+bool IsNonBrowserProcess() {
typedef bool (*IsSandboxedProcessFunc)();
IsSandboxedProcessFunc is_sandboxed_process_func =
reinterpret_cast<IsSandboxedProcessFunc>(
GetProcAddress(GetModuleHandle(NULL), "IsSandboxedProcess"));
- if (is_sandboxed_process_func && is_sandboxed_process_func()) {
- g_process_type = NON_BROWSER_PROCESS;
- return;
- }
-
- const wchar_t* command_line = GetCommandLine();
- if (command_line && wcsstr(command_line, L"--type")) {
- g_process_type = NON_BROWSER_PROCESS;
- return;
- }
-
- g_process_type = BROWSER_PROCESS;
-}
-
-bool IsNonBrowserProcess() {
- assert(g_process_type != UNINITIALIZED);
- return g_process_type == NON_BROWSER_PROCESS;
-}
+ bool is_sandboxed_process =
+ is_sandboxed_process_func && is_sandboxed_process_func();
+
+ // TODO(robertshield): Drop the command line check when we drop support for
+ // enabling chrome_elf in unsandboxed processes.
+ wchar_t* command_line = GetCommandLine();
+ bool has_process_type_flag = command_line && wcsstr(command_line, L"--type");
+
+ return (has_process_type_flag || is_sandboxed_process);
+}
« no previous file with comments | « chrome_elf/chrome_elf_util.h ('k') | chrome_elf/chrome_elf_util_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698