diff options
| author | aa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-19 18:27:05 +0000 |
|---|---|---|
| committer | aa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-19 18:27:05 +0000 |
| commit | c6d82c7461f51fdbda2947c6af9219d53447140e (patch) | |
| tree | ce811cc3324772c8cd8f1176ca31c4069dea2c23 /chrome/browser/extensions/extensions_service_unittest.cc | |
| parent | f5289ebcc88f9b7ae5fda8cacc06c47ffeb207bc (diff) | |
| download | chromium_src-c6d82c7461f51fdbda2947c6af9219d53447140e.zip chromium_src-c6d82c7461f51fdbda2947c6af9219d53447140e.tar.gz chromium_src-c6d82c7461f51fdbda2947c6af9219d53447140e.tar.bz2 | |
Fix a bug in unit test where we were not resetting state
properly. At some point, somebody fixed the test to pass, but
it wasn't really doing what was originally intended.
Review URL: http://codereview.chromium.org/2121009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47689 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/extensions_service_unittest.cc')
| -rw-r--r-- | chrome/browser/extensions/extensions_service_unittest.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/chrome/browser/extensions/extensions_service_unittest.cc b/chrome/browser/extensions/extensions_service_unittest.cc index 3f1f8c4..f0ab3cd 100644 --- a/chrome/browser/extensions/extensions_service_unittest.cc +++ b/chrome/browser/extensions/extensions_service_unittest.cc @@ -1607,6 +1607,7 @@ void ExtensionsServiceTest::TestExternalProvider( provider->RemoveExtension(good_crx); loaded_.clear(); + service_->UnloadAllExtensions(); service_->LoadAllExtensions(); loop_.RunAllPending(); ASSERT_EQ(0u, loaded_.size()); @@ -1623,7 +1624,7 @@ void ExtensionsServiceTest::TestExternalProvider( loop_.RunAllPending(); ASSERT_EQ(1u, loaded_.size()); - ASSERT_EQ(1u, GetErrors().size()); + ASSERT_EQ(0u, GetErrors().size()); // User uninstalls. loaded_.clear(); |