2007-07-24 Mark Rowe <mrowe@apple.com>
authorbdash <bdash@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Jul 2007 11:40:22 +0000 (11:40 +0000)
committerbdash <bdash@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Jul 2007 11:40:22 +0000 (11:40 +0000)
        Reviewed by Oliver.

        <rdar://problem/5356666> NSMenuItem's seen leaking on buildbot

        * platform/mac/ContextMenuMac.mm:
        (WebCore::ContextMenu::appendItem): Release the platformItem after we take ownership of it from the ContextMenuItem.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24583 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/platform/mac/ContextMenuMac.mm

index a57935d..11120d2 100644 (file)
@@ -1,3 +1,12 @@
+2007-07-24  Mark Rowe  <mrowe@apple.com>
+
+        Reviewed by Oliver.
+
+        <rdar://problem/5356666> NSMenuItem's seen leaking on buildbot
+
+        * platform/mac/ContextMenuMac.mm:
+        (WebCore::ContextMenu::appendItem): Release the platformItem after we take ownership of it from the ContextMenuItem.
+
 2007-07-24  Mitz Pettel  <mitz@webkit.org>
 
         Reviewed by Darin.
index fed3533..940ae90 100644 (file)
@@ -114,6 +114,7 @@ void ContextMenu::appendItem(ContextMenuItem& item)
         setMenuItemTarget(platformItem);
 
     [m_platformDescription.get() addObject:platformItem];
+    [platformItem release];
 }
 
 void ContextMenu::insertItem(unsigned position, ContextMenuItem& item)