Fix unit test leak introduced by fix for crbug.com/70887

It turns out that for this test, we don't actually need a request context 
getter spun up, so this skips doing that and avoids the need to properly run
the IO thread's message loop to let it get cleaned up at shutdown.

BUG=71909
TEST=The leak should no longer show up on memory bots

Review URL: http://codereview.chromium.org/6286152

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74031 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 2 insertions, 15 deletions

diff --git a/chrome/browser/extensions/extension_updater_unittest.cc b/chrome/browser/extensions/extension_updater_unittest.cc
index ebe36a5..0fb7119 100644
--- a/chrome/browser/extensions/extension_updater_unittest.cc
+++ b/chrome/browser/extensions/extension_updater_unittest.cc
@@ -47,12 +47,8 @@ int expected_load_flags =
 // Base class for further specialized test classes.
 class MockService : public ExtensionUpdateService {
  public:
-  MockService() {
-    profile_.CreateRequestContext();
-  }
-  virtual ~MockService() {
-    profile_.ResetRequestContext();
-  }
+  MockService() {}
+  virtual ~MockService() {}
 
   virtual const ExtensionList* extensions() const {
     ADD_FAILURE();
diff --git a/tools/valgrind/memcheck/suppressions.txt b/tools/valgrind/memcheck/suppressions.txt
index a0ac120..6e35331 100644
--- a/tools/valgrind/memcheck/suppressions.txt
+++ b/tools/valgrind/memcheck/suppressions.txt
@@ -3759,15 +3759,6 @@
    fun:*DownloadFileTest5SetUpEv
 }
 {
-   bug_71909
-   Memcheck:Leak
-   fun:_Znw*
-   fun:_ZN14TestingProfile20CreateRequestContextEv
-   fun:_ZN11MockService*
-   ...
-   fun:*ExtensionUpdaterTest*
-}
-{
    bug_72170a
    Memcheck:Addr4
    fun:pthread_mutex_lock