Fix some leaks and failures under valgrind in JS extensions unit tests.

extensions/renderer/module_system_test.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef EXTENSIONS_RENDERER_MODULE_SYSTEM_TEST_H_
 #define EXTENSIONS_RENDERER_MODULE_SYSTEM_TEST_H_
 
 #include "extensions/renderer/module_system.h"
 #include "extensions/renderer/scoped_persistent.h"
 #include "extensions/renderer/script_context.h"
@@ skipping 63 matching lines @@
 //   module_system_->Require("test");
 // }
 //
 // By default a test will fail if no method in the native module 'assert' is
 // called. This behaviour can be overridden by calling ExpectNoAssertionsMade().
 class ModuleSystemTest : public testing::Test {
  public:
   ModuleSystemTest();
   virtual ~ModuleSystemTest();
 
+  virtual void SetUp() OVERRIDE;
   virtual void TearDown() OVERRIDE;
 
  protected:
   ModuleSystemTestEnvironment* env() { return env_.get(); }
 
   scoped_ptr<ModuleSystemTestEnvironment> CreateEnvironment();
 
   // Make the test fail if any asserts are called. By default a test will fail
   // if no asserts are called.
   void ExpectNoAssertionsMade();
 
   // Runs promises that have been resolved. Resolved promises will not run
   // until this is called.
   void RunResolvedPromises();
 
  private:
   v8::Isolate* isolate_;
   scoped_ptr<ModuleSystemTestEnvironment> env_;
   bool should_assertions_be_made_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ModuleSystemTest);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_RENDERER_MODULE_SYSTEM_TEST_H_