Reland of [wasm][asm.js] Do same work even when not printing asm info....

Reland of [wasm][asm.js] Do same work even when not printing asm info. (patchset #1 id:1 of https://codereview.chromium.org/2627223002/ )

Reason for revert:
Triggers flaky tests.

Original issue's description:
> Revert of [wasm][asm.js] Do same work even when not printing asm info. (patchset #1 id:1 of https://codereview.chromium.org/2629043002/ )
>
> Reason for revert:
> Triggers flaky tests.
>
> Original issue's description:
> > [wasm][asm.js] Do same work even when not printing asm info.
> >
> > Skipping this work seems to perturb a gc-stress issue.
> > More investigation is likely needed.
> >
> > BUG=v8:4203
> > R=danno@chromium.org
> >
> > Review-Url: https://codereview.chromium.org/2629043002
> > Cr-Commit-Position: refs/heads/master@{#42248}
> > Committed: https://chromium.googlesource.com/v8/v8/+/785cedf1ee6671399c2b2b7f6c6a8119f55195b4
>
> TBR=danno@chromium.org,bradnelson@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=v8:4203
>
> Review-Url: https://codereview.chromium.org/2627223002
> Cr-Commit-Position: refs/heads/master@{#42250}
> Committed: https://chromium.googlesource.com/v8/v8/+/636df54873c966eceed4264cabc2360b76221e26

TBR=danno@chromium.org,bradnelson@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:4203

Review-Url: https://codereview.chromium.org/2626953003
Cr-Commit-Position: refs/heads/master@{#42251}

src/asmjs/asm-js.cc

@@ -209,26 +209,22 @@ MaybeHandle<FixedArray> AsmJs::CompileAsmViaWasm(CompilationInfo* info) {
   result->set(kWasmDataScriptPosition,
               Smi::FromInt(info->literal()->position()));
 
-  if (FLAG_trace_asm_time) {
   MessageLocation location(info->script(), info->literal()->position(),
                            info->literal()->position());
   char text[100];
-    int length =
-        base::OS::SNPrintF(text, arraysize(text),
-                           "success, asm->wasm: %0.3f ms, compile: %0.3f ms",
+  int length = base::OS::SNPrintF(
+      text, arraysize(text), "success, asm->wasm: %0.3f ms, compile: %0.3f ms",
       asm_wasm_time, compile_time);
   DCHECK_NE(-1, length);
   USE(length);
-    Handle<String> stext(
-        info->isolate()->factory()->InternalizeUtf8String(text));
+  Handle<String> stext(info->isolate()->factory()->InternalizeUtf8String(text));
   Handle<JSMessageObject> message = MessageHandler::MakeMessageObject(
       info->isolate(), MessageTemplate::kAsmJsCompiled, &location, stext,
       Handle<JSArray>::null());
   message->set_error_level(v8::Isolate::kMessageInfo);
-    if (!FLAG_suppress_asm_messages) {
+  if (FLAG_trace_asm_time && !FLAG_suppress_asm_messages) {
     MessageHandler::ReportMessage(info->isolate(), &location, message);
   }
-  }
 
   return result;
 }
@@ -318,9 +314,7 @@ MaybeHandle<Object> AsmJs::InstantiateAsmWasm(i::Isolate* isolate,
     return single_function;
   }
 
-  if (FLAG_trace_asm_time) {
-    i::Handle<i::Script> script(
-        i::Script::cast(wasm_data->get(kWasmDataScript)));
+  i::Handle<i::Script> script(i::Script::cast(wasm_data->get(kWasmDataScript)));
   int32_t position = 0;
   if (!wasm_data->get(kWasmDataScriptPosition)->ToInt32(&position)) {
     UNREACHABLE();
@@ -337,10 +331,9 @@ MaybeHandle<Object> AsmJs::InstantiateAsmWasm(i::Isolate* isolate,
       isolate, MessageTemplate::kAsmJsInstantiated, &location, stext,
       Handle<JSArray>::null());
   message->set_error_level(v8::Isolate::kMessageInfo);
-    if (!FLAG_suppress_asm_messages) {
+  if (FLAG_trace_asm_time && !FLAG_suppress_asm_messages) {
     MessageHandler::ReportMessage(isolate, &location, message);
   }
-  }
 
   return module_object;
 }