diff options
| author | akalin@chromium.org <akalin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-30 02:29:41 +0000 |
|---|---|---|
| committer | akalin@chromium.org <akalin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-30 02:29:41 +0000 |
| commit | 5a73f90320926dfe831325b8dbc27c791fa250ec (patch) | |
| tree | 9387dcf46b4630a2c8ccf02e98945547b8492e19 /chrome/browser | |
| parent | a16a94090721afacbea455128f9369915106d302 (diff) | |
| download | chromium_src-5a73f90320926dfe831325b8dbc27c791fa250ec.zip chromium_src-5a73f90320926dfe831325b8dbc27c791fa250ec.tar.gz chromium_src-5a73f90320926dfe831325b8dbc27c791fa250ec.tar.bz2 | |
Fixed invalid read in ExtensionsServiceTest.ReloadExtensions.
BUG=47843,47863
TEST=unittests, valgrind
Review URL: http://codereview.chromium.org/2872024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51215 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
| -rw-r--r-- | chrome/browser/extensions/extensions_service_unittest.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/chrome/browser/extensions/extensions_service_unittest.cc b/chrome/browser/extensions/extensions_service_unittest.cc index 90350fa..caeb00b 100644 --- a/chrome/browser/extensions/extensions_service_unittest.cc +++ b/chrome/browser/extensions/extensions_service_unittest.cc @@ -1527,6 +1527,10 @@ TEST_F(ExtensionsServiceTest, ReloadExtensions) { EXPECT_EQ(1u, service_->extensions()->size()); EXPECT_EQ(0u, service_->disabled_extensions()->size()); + // Need to clear |loaded_| manually before reloading as the + // EnableExtension() call above inserted into it and + // UnloadAllExtensions() doesn't send out notifications. + loaded_.clear(); service_->ReloadExtensions(); // Extension counts shouldn't change. |