Add ExceptionHandler to ModuleSystem, remove heap allocated v8::TryCatch.

chrome/renderer/extensions/module_system_unittest.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/test/base/module_system_test.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/renderer/extensions/module_system.h"
 
 using extensions::ModuleSystem;
 using extensions::NativeHandler;
@@ skipping 13 matching lines @@
 
   v8::Handle<v8::Value> Increment(const v8::Arguments& args) {
     counter_++;
     return v8::Undefined();
   }
 
  private:
   int counter_;
 };
 
+class TestExceptionHandler : public ModuleSystem::ExceptionHandler {
+ public:
+  TestExceptionHandler()
+      : handled_exception_(false) {
+  }
+
+  virtual void HandleUncaughtException() {
+    handled_exception_ = true;
+  }
+
+  bool handled_exception() const { return handled_exception_; }
+
+ private:
+  bool handled_exception_;
+};
+
+TEST_F(ModuleSystemTest, TestExceptionHandling) {
+  ModuleSystem::NativesEnabledScope natives_enabled_scope(module_system_.get());
+  TestExceptionHandler* handler = new TestExceptionHandler;
+  scoped_ptr<ModuleSystem::ExceptionHandler> scoped_handler(handler);
+  ASSERT_FALSE(handler->handled_exception());
+  module_system_->set_exception_handler(scoped_handler.Pass());
+
+  RegisterModule("test", "throw 'hi';");
+  module_system_->Require("test");
+  ASSERT_TRUE(handler->handled_exception());
+
+  ExpectNoAssertionsMade();
+}
+
 TEST_F(ModuleSystemTest, TestRequire) {
   ModuleSystem::NativesEnabledScope natives_enabled_scope(module_system_.get());
   RegisterModule("add", "exports.Add = function(x, y) { return x + y; };");
   RegisterModule("test",
       "var Add = require('add').Add;"
       "requireNative('assert').AssertTrue(Add(3, 5) == 8);");
   module_system_->Require("test");
 }
 
 TEST_F(ModuleSystemTest, TestNestedRequire) {
@@ skipping 177 matching lines @@
 }
 
 TEST_F(ModuleSystemTest, TestOverrideNonExistentNativeHandler) {
   ModuleSystem::NativesEnabledScope natives_enabled_scope(module_system_.get());
   OverrideNativeHandler("thing", "exports.x = 5;");
   RegisterModule("test",
       "var assert = requireNative('assert');"
       "assert.AssertTrue(requireNative('thing').x == 5);");
   module_system_->Require("test");
 }