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

Issue 2444573002: Stack sampling profiler: enable at 10% for GPU process on dev (Closed)

Created:
4 years, 2 months ago by Mike Wittman
Modified:
4 years, 1 month ago
Reviewers:
Lei Zhang
CC:
chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Stack sampling profiler: enable at 10% for GPU process on dev We're seeing reasonable stability and performance numbers from the current 10% enable rate on canary, so enable at the same rate on dev. BUG=650869 Committed: https://crrev.com/f65d37b56b3a8498be9b7473854ede13d03e4483 Cr-Commit-Position: refs/heads/master@{#427092}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M chrome/common/stack_sampling_configuration.cc View 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 16 (7 generated)
Mike Wittman
4 years, 2 months ago (2016-10-21 23:31:44 UTC) #2
Lei Zhang
lgtm
4 years, 2 months ago (2016-10-22 00:40:33 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2444573002/1
4 years, 1 month ago (2016-10-24 14:55:09 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_swarming_rel/builds/54149)
4 years, 1 month ago (2016-10-24 15:54:00 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2444573002/1
4 years, 1 month ago (2016-10-24 16:01:33 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_clang_dbg_recipe/builds/151261)
4 years, 1 month ago (2016-10-24 16:54:56 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2444573002/1
4 years, 1 month ago (2016-10-24 16:56:05 UTC) #13
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 1 month ago (2016-10-24 17:35:44 UTC) #14
commit-bot: I haz the power
4 years, 1 month ago (2016-10-24 17:50:42 UTC) #16
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/f65d37b56b3a8498be9b7473854ede13d03e4483
Cr-Commit-Position: refs/heads/master@{#427092}

Powered by Google App Engine
This is Rietveld 408576698