Fix nwjs#6583: Menu and MenuItem leak

rogerwang · rogerwang · commit 0f1a574c95ae · 2018-05-02T14:27:33.000+08:00
diff --git a/src/api/object_manager.cc b/src/api/object_manager.cc
@@ -137,11 +137,12 @@ void ObjectManager::OnCallObjectMethod(
     const std::string& type,
     const std::string& method,
     const base::ListValue& arguments) {
+#if 0
   DLOG(INFO) << "OnCallObjectMethod: object_id:" << object_id
              << " type:" << type
              << " method:" << method
              << " arguments:" << arguments;
-
+#endif
   Base* object = GetApiObject(object_id);
   if (object)
     object->Call(method, arguments, rvh);
diff --git a/src/resources/api_nw_menu.js b/src/resources/api_nw_menu.js
@@ -24,8 +24,9 @@ function Menu (option) {
   privates(this).items = [];
   privates(this).option = option;
 
+  var items = privates(this).items;
   nw.Obj.create(id, 'Menu', option);
-  messagingNatives.BindToGC(this, nw.Obj.destroy.bind(undefined, id), -1);
+  messagingNatives.BindToGC(this, function() { items.forEach(function(element) { element._destroy(); }); nw.Obj.destroy(id); }, -1);
 };
 
 Menu.prototype.__defineGetter__('items', function() {
diff --git a/src/resources/api_nw_menuitem.js b/src/resources/api_nw_menuitem.js
@@ -97,10 +97,14 @@ function MenuItem(option) {
 
   if (!option.native)
     nw.Obj.create(id, 'MenuItem', option);
-  messagingNatives.BindToGC(this, nw.Obj.destroy.bind(undefined, id), -1);
+  messagingNatives.BindToGC(this, function() { nw.Obj.destroy(id); }, -1);
 
 }
 
+MenuItem.prototype._destroy = function () {
+  menuItems.objs[this.id] = null;
+};
+
 util.inherits(MenuItem, EventEmitter);
 
 MenuItem.prototype.handleGetter = function(name) {
