diff options
| author | jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-25 15:55:15 +0000 |
|---|---|---|
| committer | jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-25 15:55:15 +0000 |
| commit | f2bc0cb52453bbca0c9b998994de8674f8a1e874 (patch) | |
| tree | 5de676608035f4ace623512767c67d7b6a730a20 /base/histogram.h | |
| parent | 8b97235beb506e6369d87a79cdd90c43e71d51c8 (diff) | |
| download | chromium_src-f2bc0cb52453bbca0c9b998994de8674f8a1e874.zip chromium_src-f2bc0cb52453bbca0c9b998994de8674f8a1e874.tar.gz chromium_src-f2bc0cb52453bbca0c9b998994de8674f8a1e874.tar.bz2 | |
Do some minor cleanup in histogram.*
I was trying to debug some code, and noticed some excess
complexity.
There are no significant semantic changes.
DCHECK on a foo.get() is a waste given we're about to deref
the pointer.
Execution cost should be be minially less, and we get rid
of several lines of code.
BUG=46263
r=mbelshe
Review URL: http://codereview.chromium.org/2805033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50847 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/histogram.h')
| -rw-r--r-- | base/histogram.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/base/histogram.h b/base/histogram.h index a11e293..64ae9cb 100644 --- a/base/histogram.h +++ b/base/histogram.h @@ -1,4 +1,4 @@ -// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. +// Copyright (c) 2006-2010 The Chromium Authors. All rights reserved. // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. @@ -603,7 +603,8 @@ class StatisticsRecorder { static void GetHistograms(Histograms* output); // Find a histogram by name. It matches the exact name. This method is thread - // safe. + // safe. If a matching histogram is not found, then the |histogram| is + // not changed. static bool FindHistogram(const std::string& query, scoped_refptr<Histogram>* histogram); |