Fix use of history timers in background threads.

src/wasm/module-compiler.cc

Index: src/wasm/module-compiler.cc
diff --git a/src/wasm/module-compiler.cc b/src/wasm/module-compiler.cc
index 9d83480c01d2e9b45498b139c28abea1956bcbb0..0e1821038dd748f172b0dbc339a355b35ebd2819 100644
--- a/src/wasm/module-compiler.cc
+++ b/src/wasm/module-compiler.cc
@@ -322,18 +322,10 @@ MaybeHandle<WasmModuleObject> ModuleCompiler::CompileToModuleObject(
     function_tables->set(i, *temp_instance.function_tables[i]);
     signature_tables->set(i, *temp_instance.signature_tables[i]);
   }
-
-  if (is_sync_) {
-    // TODO(karlschimpf): Make this work when asynchronous.
-    // https://bugs.chromium.org/p/v8/issues/detail?id=6361
-    HistogramTimerScope wasm_compile_module_time_scope(
-        module_->is_wasm()
-            ? isolate_->counters()->wasm_compile_wasm_module_time()
-            : isolate_->counters()->wasm_compile_asm_module_time());
-    return CompileToModuleObjectInternal(
-        thrower, wire_bytes, asm_js_script, asm_js_offset_table_bytes, factory,
-        &temp_instance, &function_tables, &signature_tables);
-  }
+  TimedHistogramScope wasm_compile_module_time_scope(
+      module_->is_wasm()
+          ? isolate_->counters()->wasm_compile_wasm_module_time()

[jochen (gone - plz use gerrit), 2017/06/08 22:15:36]

you can't access isolate_ if !is_sync

[kschimpf, 2017/06/09 20:03:36]

Your right.

+          : isolate_->counters()->wasm_compile_asm_module_time());
   return CompileToModuleObjectInternal(
       thrower, wire_bytes, asm_js_script, asm_js_offset_table_bytes, factory,
       &temp_instance, &function_tables, &signature_tables);