Commit 174dcfe7 authored by Etienne Bergeron's avatar Etienne Bergeron Committed by Commit Bot

[Sampling profiler] Disable profiler when Trend Micro DLLs are in process

Sampling profiler is causing jank on the Browser main-thread and
io-thread when Trend Micro DLLs are in the process.

This is a reland of:
  https://chromium-review.googlesource.com/c/chromium/src/+/1881980

Bug: 1113832
Change-Id: Ieabf791b6f1ecce5fc4f3d84de822cc4ca11e52d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2340675
Commit-Queue: Etienne Bergeron <etienneb@chromium.org>
Reviewed-by: default avatarMike Wittman <wittman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#795714}
parent c1aab56c
...@@ -757,6 +757,11 @@ bool StackSamplingProfiler::IsSupported() { ...@@ -757,6 +757,11 @@ bool StackSamplingProfiler::IsSupported() {
// simultaneously can cause crashes and has no known use case. // simultaneously can cause crashes and has no known use case.
if (GetModuleHandleA(base::win::kApplicationVerifierDllName)) if (GetModuleHandleA(base::win::kApplicationVerifierDllName))
return false; return false;
// Checks if Trend Micro DLLs are loaded in process, so we can disable the
// profiler to avoid hitting their performance bug. See
// https://crbug.com/1018291 and https://crbug.com/1113832.
if (GetModuleHandleA("tmmon64.dll") || GetModuleHandleA("tmmonmgr64.dll"))
return false;
#endif #endif
return true; return true;
#else #else
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment