17118 2008-01-31 09:13:49 -0800 kjs_navigator.cpp should be re-written to be less crazy 2009-01-03 13:44:24 -0800 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Mac OS X 10.5 RESOLVED WONTFIX P2 Normal --- 1 eric webkit-unassigned andersca ddkilzer sam oldest_to_newest 68923 0 eric 2008-01-31 09:13:49 -0800 kjs_navigator.cpp should be re-written to be less crazy kjs_navigator uses a complicated/hacky memory management scheme and thus I bet leaks in some cases. Better would be for PluginBase to hold a reference to some PluginInfoCache static. When the PluginInfoCache ref count drops to 0, it can clear its caches. Those caches can use Vector<OwnPtr> to avoid ever leaking. This could/should also be re-written to use the bindings autogen system. Thus Navigator would become a normal RefCounted class, and Navigator.idl would take care of all the details. 104133 1 ddkilzer 2009-01-02 11:30:10 -0800 Can this bug be closed? I know Navigator.h/cpp/idl were created a while ago, and the plugin code has undergone many refactorings since this was filed.