[extensions] added checks that function is not empty

extensions/renderer/module_system.cc

Index: extensions/renderer/module_system.cc
diff --git a/extensions/renderer/module_system.cc b/extensions/renderer/module_system.cc
index e583efcaa730360068ee32c6bef75aae1d61bfd0..7356d1b30cc255107487bf296e7afdd349b76207 100644
--- a/extensions/renderer/module_system.cc
+++ b/extensions/renderer/module_system.cc
@@ -309,6 +309,9 @@ v8::Local<v8::Value> ModuleSystem::CallModuleMethod(
 
   v8::Local<v8::Function> function =
       GetModuleFunction(module_name, method_name);
+  if (function.IsEmpty()) {

[Devlin, 2016/12/01 22:09:47]

This *shouldn't* ever happen, but I wouldn't be entirely surprised if it were. 
Can we NOTREACHED here (and below)?

[kozy, 2016/12/01 22:19:25]

Done.

+    return handle_scope.Escape(v8::Undefined(GetIsolate()));
+  }
 
   v8::Local<v8::Value> result;
   {
@@ -350,6 +353,9 @@ void ModuleSystem::CallModuleMethodSafe(const std::string& module_name,
 
   v8::Local<v8::Function> function =
       GetModuleFunction(module_name, method_name);
+  if (function.IsEmpty()) {
+    return;
+  }
 
   {
     v8::TryCatch try_catch(GetIsolate());