[wasm] Implement correct 2-level namespace for imports.

src/wasm/wasm-module.cc

Index: src/wasm/wasm-module.cc
diff --git a/src/wasm/wasm-module.cc b/src/wasm/wasm-module.cc
index 0d7d7e08502a82c2bacc9ed529a6f340a0f62366..1141585e5ca20e08ea1b0e6cf7bc75fab7895f2d 100644
--- a/src/wasm/wasm-module.cc
+++ b/src/wasm/wasm-module.cc
@@ -1409,21 +1409,16 @@ class WasmInstanceBuilder {
 
     Handle<Object> module = result.ToHandleChecked();
 
-    // TODO(bradnelson): Making this conditional on non-empty names violates the
-    // Wasm spec, but seems to be a hack intended for the asm-to-wasm pipeline.
-    // We need to get rid of it.
-    if (import_name->length() != 0) {
-      // Look up the value in the module.
-      if (!module->IsJSReceiver()) {
-        return ReportTypeError("module is not an object or function", index,
-                               module_name);
-      }
+    // Look up the value in the module.
+    if (!module->IsJSReceiver()) {
+      return ReportTypeError("module is not an object or function", index,
+                             module_name);
+    }
 
-      result = Object::GetPropertyOrElement(module, import_name);
-      if (result.is_null()) {
-        ReportLinkError("import not found", index, module_name, import_name);
-        return MaybeHandle<JSFunction>();
-      }
+    result = Object::GetPropertyOrElement(module, import_name);
+    if (result.is_null()) {
+      ReportLinkError("import not found", index, module_name, import_name);
+      return MaybeHandle<JSFunction>();
     }
 
     return result;