[wasm] Differentiate unnamed and empty names

src/isolate.cc

 // Copyright 2012 the V8 project 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 "src/isolate.h"
 
 #include <stdlib.h>
 
 #include <fstream>  // NOLINT(readability/streams)
 #include <sstream>
@@ skipping 603 matching lines @@
     return stack_frame;
   }
 
   Handle<JSObject> NewStackFrameObject(WasmFrame* frame) {
     Handle<JSObject> stack_frame =
         factory()->NewJSObject(isolate_->object_function());
 
     if (!function_key_.is_null()) {
       Handle<Object> fun_name = handle(frame->function_name(), isolate_);
       if (fun_name->IsUndefined())
         fun_name = isolate_->factory()->InternalizeUtf8String(

[Yang, 2016/05/13 07:41:10]

We should not be using InternalizeUtf8String here. Use NewStringFromStaticChars
instead.

[Clemens Hammacher, 2016/05/17 17:43:24]

Done.

-            Vector<const char>("<WASM>"));
+            Vector<const char>("<WASM UNNAMED>"));
       JSObject::AddProperty(stack_frame, function_key_, fun_name, NONE);
     }
     // Encode the function index as line number.
     if (!line_key_.is_null()) {
       JSObject::AddProperty(
           stack_frame, line_key_,
           isolate_->factory()->NewNumberFromInt(frame->function_index()), NONE);
     }
     // Encode the byte offset as column.
     if (!column_key_.is_null()) {
@@ skipping 2420 matching lines @@
   // Then check whether this scope intercepts.
   if ((flag & intercept_mask_)) {
     intercepted_flags_ |= flag;
     return true;
   }
   return false;
 }
 
 }  // namespace internal
 }  // namespace v8