[wasm] Use a Managed<WasmModule> to hold metadata about modules.

src/api.cc

Index: src/api.cc
diff --git a/src/api.cc b/src/api.cc
index 6ab515c879cfccff67eddb4bf10c113e17e1efdc..f0cf830cfcae80aadb75efd2df8843d60b5b9cd4 100644
--- a/src/api.cc
+++ b/src/api.cc
@@ -7191,12 +7191,9 @@ WasmCompiledModule::SerializedModule WasmCompiledModule::Serialize() {
   i::Handle<i::wasm::WasmCompiledModule> compiled_part =
       i::handle(i::wasm::WasmCompiledModule::cast(obj->GetInternalField(0)));
 
-  i::Handle<i::SeqOneByteString> wire_bytes = compiled_part->module_bytes();
-  compiled_part->reset_module_bytes();

[Mircea Trofin, 2016/10/15 17:38:39]

Consider the scenario when we try to deserialize from an older version stored in
indexedDB, and need to recompile. To recompile, we need the wasm wire bytes.
Since deserialization won't even get past the serialized header, our only way to
get back to the wire bytes is to explicitly serialize them (as discussed, long
term, we envision they are stored separately as Blobs). 

This means the serialized payload is wire_bytes + serialized_module.

If we leave the wire bytes hanging off the serialized module, the serialized
payload is 2*wire_bytes + serialized_module.

Typically, the serialized module is about 4-5x larger than the wire bytes. So
this means the duplicate wire_bytes constitute a 16-20% increase in serialized
data.

To achieve the goal of this CL, we only need the header bytes. Is the plan to
follow-up with another CL, where that happens? If so, I think we should do that
before we ship, so we should add an issue#; also, are we OK with taking a hit in
user-observable properties in the meantime? (I realize the whole wasm feature is
under a flag, so I don't think this is much of an issue myself, just calling it
out to make sure we understand the trade-offs and are all OK with them).

   std::unique_ptr<i::ScriptData> script_data =
       i::WasmCompiledModuleSerializer::SerializeWasmModule(obj->GetIsolate(),
                                                            compiled_part);
-  compiled_part->set_module_bytes(wire_bytes);
   script_data->ReleaseDataOwnership();
 
   size_t size = static_cast<size_t>(script_data->length());
@@ -7225,7 +7222,7 @@ MaybeLocal<WasmCompiledModule> WasmCompiledModule::Deserialize(
   i::Handle<i::wasm::WasmCompiledModule> compiled_module =
       handle(i::wasm::WasmCompiledModule::cast(*compiled_part));
   return Local<WasmCompiledModule>::Cast(
-      Utils::ToLocal(i::wasm::CreateCompiledModuleObject(
+      Utils::ToLocal(i::wasm::CreateWasmModuleObject(
           i_isolate, compiled_module, i::wasm::ModuleOrigin::kWasmOrigin)));
 }