diff options
| author | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-21 22:12:58 +0000 |
|---|---|---|
| committer | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-21 22:12:58 +0000 |
| commit | a1923e617066da321821a8ccbdf0fbadd77d80de (patch) | |
| tree | b9d584225fb0ecd77e49493da2d6f80fd4a44186 /chrome/browser/themes | |
| parent | b61c145d0b762e5ef324073091ba6a67f7fc71f0 (diff) | |
| download | chromium_src-a1923e617066da321821a8ccbdf0fbadd77d80de.zip chromium_src-a1923e617066da321821a8ccbdf0fbadd77d80de.tar.gz chromium_src-a1923e617066da321821a8ccbdf0fbadd77d80de.tar.bz2 | |
Coverity: Fix leaks.
CID=2168,8197,8198,100251,100256,100820,101011,101982,101983,101984,101988,
102256
BUG=none
TEST=none
R=dpapad
Review URL: http://codereview.chromium.org/8989036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115422 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/themes')
| -rw-r--r-- | chrome/browser/themes/browser_theme_pack.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/chrome/browser/themes/browser_theme_pack.cc b/chrome/browser/themes/browser_theme_pack.cc index ff1121a..fd55ced 100644 --- a/chrome/browser/themes/browser_theme_pack.cc +++ b/chrome/browser/themes/browser_theme_pack.cc @@ -6,6 +6,7 @@ #include <limits> +#include "base/memory/scoped_ptr.h" #include "base/stl_util.h" #include "base/string_util.h" #include "base/threading/thread_restrictions.h" @@ -338,7 +339,7 @@ BrowserThemePack* BrowserThemePack::BuildFromExtension( DCHECK(extension); DCHECK(extension->is_theme()); - BrowserThemePack* pack = new BrowserThemePack; + scoped_refptr<BrowserThemePack> pack = new BrowserThemePack; pack->BuildHeader(extension); pack->BuildTintsFromJSON(extension->GetThemeTints()); pack->BuildColorsFromJSON(extension->GetThemeColors()); @@ -363,7 +364,7 @@ BrowserThemePack* BrowserThemePack::BuildFromExtension( pack->GenerateTabBackgroundImages(&pack->prepared_images_); // The BrowserThemePack is now in a consistent state. - return pack; + return pack.release(); } // static |