diff options
| author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-30 19:59:11 +0000 |
|---|---|---|
| committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-30 19:59:11 +0000 |
| commit | 6832cbe85f9d342d84d0ddbb2d017496f7649e5d (patch) | |
| tree | 5df5727a7a04ecf425f349bf4c06c71d0c6c7c57 /base/values.h | |
| parent | e1245358ad4226525d7c1e0da06356ebe5715aa7 (diff) | |
| download | chromium_src-6832cbe85f9d342d84d0ddbb2d017496f7649e5d.zip chromium_src-6832cbe85f9d342d84d0ddbb2d017496f7649e5d.tar.gz chromium_src-6832cbe85f9d342d84d0ddbb2d017496f7649e5d.tar.bz2 | |
Fix a memory leak when calling the one-arg form of ListValue::Remove().
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/434108
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33314 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/values.h')
| -rw-r--r-- | base/values.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/base/values.h b/base/values.h index 49ce287..82a29fd 100644 --- a/base/values.h +++ b/base/values.h @@ -361,8 +361,8 @@ class ListValue : public Value { // it will return false and the ListValue object will be unchanged. bool Remove(size_t index, Value** out_value); - // Removes the first instance of |value| found in the list, if any, returning - // the index that it was located at (-1 for not present). + // Removes the first instance of |value| found in the list, if any, and + // deletes it. Returns the index that it was located at (-1 for not present). int Remove(const Value& value); // Appends a Value to the end of the list. |