Commit f98bf3f9 authored by Mike Wittman's avatar Mike Wittman Committed by Commit Bot

[Sampling profiler] Enable on Mac dev at 90%

Having addressed all known stability issues on Mac, updates Mac enable
rate to be at parity with Windows.

Bug: 531673
Change-Id: I941d2a6d91fcc3272cdd64c98b1af6358301ea8a
Reviewed-on: https://chromium-review.googlesource.com/1101175Reviewed-by: default avatarLeonard Grey <lgrey@chromium.org>
Reviewed-by: default avatarScott Violet <sky@chromium.org>
Commit-Queue: Mike Wittman <wittman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#567418}
parent 43e035ee
......@@ -189,27 +189,16 @@ StackSamplingConfiguration::GenerateConfiguration() {
return PROFILE_DISABLED;
switch (chrome::GetChannel()) {
// Enable the profiler in the ultimate production configuration for
// development/waterfall builds.
// Enable the profiler unconditionally for development/waterfall builds.
case version_info::Channel::UNKNOWN:
return PROFILE_ENABLED;
#if defined(OS_WIN) && defined(ARCH_CPU_X86_64)
#if (defined(OS_WIN) && defined(ARCH_CPU_X86_64)) || defined(OS_MACOSX)
case version_info::Channel::CANARY:
case version_info::Channel::DEV:
return ChooseConfiguration({{PROFILE_ENABLED, 80},
{PROFILE_CONTROL, 10},
{PROFILE_DISABLED, 10}});
#elif defined(OS_MACOSX)
case version_info::Channel::CANARY:
return ChooseConfiguration({{PROFILE_ENABLED, 80},
{PROFILE_CONTROL, 10},
{PROFILE_DISABLED, 10}});
case version_info::Channel::DEV:
return ChooseConfiguration({{PROFILE_ENABLED, 50},
{PROFILE_CONTROL, 0},
{PROFILE_DISABLED, 50}});
#endif
default:
......
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