From 6d2b55c44a5da8f52ac1a7cd6314cd1c8eccf2e1 Mon Sep 17 00:00:00 2001 From: "jfrancis%netscape.com" Date: Sat, 15 Aug 1998 10:11:17 +0000 Subject: [PATCH] Adding comment to help prevent recurrences of bug #313498 git-svn-id: svn://10.0.0.236/trunk@8072 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/cmd/macfe/gui/CBrowserWindow.cp | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/mozilla/cmd/macfe/gui/CBrowserWindow.cp b/mozilla/cmd/macfe/gui/CBrowserWindow.cp index 4a5ac9c491e..00c0a39e9ea 100644 --- a/mozilla/cmd/macfe/gui/CBrowserWindow.cp +++ b/mozilla/cmd/macfe/gui/CBrowserWindow.cp @@ -402,6 +402,11 @@ void CBrowserWindow::FindCommandStatus( case cmd_LoadImages: case cmd_Reload: { + // Need to be very careful what gets delegated here, in order to aviod infinite + // recursion a la bug #313498. Handling the command status in CBrowserView is not + // sufficient - it also needs to be handled in CHTMLView, which is the class that + // will receive this if it's a Composer window instead of a Browser window. + // Delegate this to the view. GetHTMLView()->FindCommandStatus(inCommand, outEnabled, outUsesMark, outMark, outName); break;