Bug 129531 - Context menu handler does double delete of handle. r=pinkerton/sr=sfraser/a=asa

This commit is contained in:
ccarlen%netscape.com 2002-03-08 21:57:21 +00:00
parent 653cb03182
commit c0349878fb

View File

@ -103,8 +103,10 @@ void CWebBrowserCMAttachment::ExecuteSelf(MessageT inMessage,
void CWebBrowserCMAttachment::SetCommandList(SInt16 mcmdResID)
{
StResource mcmdRes(ResType_MenuCommands, mcmdResID);
LHandleStream inStream(static_cast<Handle>(mcmdRes));
Handle mcmdRes = ::GetResource(ResType_MenuCommands, mcmdResID);
ThrowIfResFail_(mcmdRes);
::DetachResource(mcmdRes); // LHandleStream takes ownership and uses DisposeHandle()
LHandleStream inStream(mcmdRes);
mCommandList.RemoveItemsAt(mCommandList.GetCount(), LArray::index_First);