diff options
| author | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-04 19:24:42 +0000 |
|---|---|---|
| committer | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-04 19:24:42 +0000 |
| commit | d22d873921f2f657b483a01a398ddce3001747a1 (patch) | |
| tree | 4190718501c4cae47302ea1a8be8ee7b6f4c41cd /chrome/browser/bookmarks/bookmark_storage.h | |
| parent | 30cf2bf9a44e5f97ddd497113bce74feb6b4b517 (diff) | |
| download | chromium_src-d22d873921f2f657b483a01a398ddce3001747a1.zip chromium_src-d22d873921f2f657b483a01a398ddce3001747a1.tar.gz chromium_src-d22d873921f2f657b483a01a398ddce3001747a1.tar.bz2 | |
Fix various uses of release() that did not check its value
BUG=42904
TEST=compile, unit tests
Review URL: http://codereview.chromium.org/1730024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46376 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/bookmarks/bookmark_storage.h')
| -rw-r--r-- | chrome/browser/bookmarks/bookmark_storage.h | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/chrome/browser/bookmarks/bookmark_storage.h b/chrome/browser/bookmarks/bookmark_storage.h index 033a849..bc55219 100644 --- a/chrome/browser/bookmarks/bookmark_storage.h +++ b/chrome/browser/bookmarks/bookmark_storage.h @@ -40,15 +40,14 @@ class BookmarkLoadDetails { ids_reassigned_(false) { } - void release() { - bb_node_.release(); - other_folder_node_.release(); - index_.release(); - } - BookmarkNode* bb_node() { return bb_node_.get(); } + BookmarkNode* release_bb_node() { return bb_node_.release(); } BookmarkNode* other_folder_node() { return other_folder_node_.get(); } + BookmarkNode* release_other_folder_node() { + return other_folder_node_.release(); + } BookmarkIndex* index() { return index_.get(); } + BookmarkIndex* release_index() { return index_.release(); } // Max id of the nodes. void set_max_id(int64 max_id) { max_id_ = max_id; } |