From d957b76ce4f0fa477a18998747b0db4324dc5f74 Mon Sep 17 00:00:00 2001 From: "jhawkins@chromium.org" Date: Mon, 28 Sep 2009 19:52:41 +0000 Subject: Coverity: Initialize profile_ and process_images_ in the constructor. CID=3971 BUG=none TEST=none Review URL: http://codereview.chromium.org/219014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27396 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/browser_theme_provider.cc | 5 +++-- chrome/browser/browser_theme_provider.h | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/chrome/browser/browser_theme_provider.cc b/chrome/browser/browser_theme_provider.cc index fa6aa89..e59caf8 100644 --- a/chrome/browser/browser_theme_provider.cc +++ b/chrome/browser/browser_theme_provider.cc @@ -6,7 +6,6 @@ #include "app/gfx/skbitmap_operations.h" #include "base/file_util.h" -#include "base/string_util.h" #include "base/gfx/png_decoder.h" #include "base/gfx/png_encoder.h" #include "base/string_util.h" @@ -231,7 +230,9 @@ bool BrowserThemeProvider::WriteImagesToDisk() { } BrowserThemeProvider::BrowserThemeProvider() - : rb_(ResourceBundle::GetSharedInstance()) { + : rb_(ResourceBundle::GetSharedInstance()), + profile_(NULL), + process_images_(false) { static bool initialized = false; if (!initialized) { for (size_t i = 0; i < arraysize(kToolbarButtonIDs); ++i) { diff --git a/chrome/browser/browser_theme_provider.h b/chrome/browser/browser_theme_provider.h index 3c72549e..cec14d1f 100644 --- a/chrome/browser/browser_theme_provider.h +++ b/chrome/browser/browser_theme_provider.h @@ -146,7 +146,7 @@ class BrowserThemeProvider : public NonThreadSafe, } AlignmentMasks; // Background tiling choices. - enum { + typedef enum { NO_REPEAT = 0, REPEAT_X = 1, REPEAT_Y = 2, -- cgit v1.1