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

Issue 5717002: [Mac] Make profile-locking error the same as failing to lock. (Closed)

Created:
10 years ago by Scott Hess - ex-Googler
Modified:
9 years, 6 months ago
Reviewers:
Mark Mentovai
CC:
chromium-reviews, pam+watch_chromium.org, ben+cc_chromium.org, Robert Sesek
Visibility:
Public.

Description

[Mac] Make profile-locking error the same as failing to lock. Previously, the profile-locking code was conservative in the face of errors. Histograms indicate that the error rate is acceptable (no errors seen), so flip locking errors to me failure-to-lock. BUG=58986 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=68871

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -8 lines) Patch
M chrome/browser/process_singleton_mac.cc View 3 chunks +4 lines, -8 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Scott Hess - ex-Googler
Cleaning up an old bug. Zero hits to the histograms seems amazing (I'd expect at ...
10 years ago (2010-12-09 22:42:59 UTC) #1
Mark Mentovai
10 years ago (2010-12-09 22:58:27 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698