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

Issue 401913002: Revert of Experimentally disable termination on heap corrruption in order to measure the contribution of this… (Closed)

Created:
6 years, 5 months ago by tonyg
Modified:
6 years, 5 months ago
CC:
chromium-reviews, darin-cc_chromium.org, jam
Project:
chromium
Visibility:
Public.

Description

Revert of Experimentally disable termination on heap corrruption in order to measure the contribution of this… (https://codereview.chromium.org/402723002/) Reason for revert: Crashes chromium.perf bots. ChildEBP RetAddr 0052f9f4 6a47d2ff chrome_69f00000!base::debug::BreakDebugger+0x10 0052fa64 6a47df08 chrome_69f00000!CheckIsChromeSxSProcess+0x26 0052fa68 6a47e664 chrome_69f00000!InstallUtil::IsChromeSxSProcess+0x16 0052fa70 6a47e53b chrome_69f00000!BrowserDistribution::GetSpecificDistribution+0x3b 0052fa78 6a47bb77 chrome_69f00000!BrowserDistribution::GetDistribution+0x7 0052fb64 6a47bb2e chrome_69f00000!`anonymous namespace'::GetChromeChannelInternal+0x2c 0052fb7c 6a2d03f8 chrome_69f00000!GoogleUpdateSettings::GetChromeChannel+0x21 0052fbe8 6a2cd0d8 chrome_69f00000!chrome::VersionInfo::GetChannel+0x60 0052fbec 6a2cd1ab chrome_69f00000!`anonymous namespace'::GetBeaconKeyPath+0x5 0052fc08 6a2cc624 chrome_69f00000!ShouldExperimentallyDisableTerminateOnHeapCorruption+0xe 0052fc50 0020720b chrome_69f00000!ChromeMain+0x42 0052fce0 00206c36 chrome!MainDllLoader::Launch+0x15f 0052fd24 00228c9e chrome!wWinMain+0x5a 0052fd70 766e338a chrome!__tmainCRTStartup+0xfd WARNING: Stack unwind information not available. Following frames may be wrong. 0052fd7c 76f99f72 kernel32!BaseThreadInitThunk+0x12 0052fdbc 76f99f45 ntdll!RtlInitializeExceptionChain+0x63 0052fdd4 00000000 ntdll!RtlInitializeExceptionChain+0x36 Original issue's description: > Experimentally disable termination on heap corruption in order to measure the contribution of this feature to missing crash reports. > > Because this feature is configured very early in the process lifetime it cannot be directly controlled by a field-trial. Rather, we query the status during a given execution, store that status in the registry, and then query the registry during startup. This means the experiment will only take effect the 2nd time it is executed. > > BUG=394842 > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284100 TBR=jochen@chromium.org,brettw@chromium.org,jhawkins@chromium.org,wfh@chromium.org,erikwright@chromium.org NOTREECHECKS=true NOTRY=true BUG=394842 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284197

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -78 lines) Patch
M chrome/app/chrome_main.cc View 2 chunks +0 lines, -4 lines 0 comments Download
M chrome/browser/chrome_browser_main_win.cc View 2 chunks +0 lines, -5 lines 0 comments Download
M chrome/chrome_common.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
D chrome/common/terminate_on_heap_corruption_experiment_win.h View 1 chunk +0 lines, -11 lines 0 comments Download
D chrome/common/terminate_on_heap_corruption_experiment_win.cc View 1 chunk +0 lines, -51 lines 0 comments Download
M content/app/content_main_runner.cc View 1 chunk +1 line, -2 lines 0 comments Download
M content/public/app/content_main.h View 2 chunks +0 lines, -3 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
tonyg
6 years, 5 months ago (2014-07-18 21:11:23 UTC) #1
Created Revert of Experimentally disable termination on heap corrruption in
order to measure the contribution of this…

Powered by Google App Engine
This is Rietveld 408576698