From f299f54987e68ed9704b81aad8706428842e59f5 Mon Sep 17 00:00:00 2001 From: "pinkerton%netscape.com" Date: Wed, 5 Aug 1998 04:07:29 +0000 Subject: [PATCH] Comment out lines that aren't being used. Also fixes build bustage. git-svn-id: svn://10.0.0.236/trunk@7325 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/cmd/macfe/rdfui/CNavCenterContextMenuAtt.cp | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/mozilla/cmd/macfe/rdfui/CNavCenterContextMenuAtt.cp b/mozilla/cmd/macfe/rdfui/CNavCenterContextMenuAtt.cp index 1ccf8c63a54..d0b4f093628 100644 --- a/mozilla/cmd/macfe/rdfui/CNavCenterContextMenuAtt.cp +++ b/mozilla/cmd/macfe/rdfui/CNavCenterContextMenuAtt.cp @@ -97,9 +97,15 @@ CNavCenterContextMenuAttachment :: NewHTContextMenuCursor ( ) if ( table->GetNextSelectedRow(selectedRow) ) clickInBackground = PR_FALSE; +#if 0 +// This will cause a crash if we return NULL because the context menu code assumes a menu will +// be created. Not sure what the right fix is right now, but leaving context menus on is not +// that bad (pinkerton) // only allow context menu if the "useSelection" flag is true for the current view - if ( URDFUtilities::PropertyValueBool(HT_TopNode(table->GetHTView()), gNavCenter->useSelection) ) + if ( URDFUtilities::PropertyValueBool(HT_TopNode(table->GetHTView()), gNavCenter->useSelection, true) == false) return NULL; +#endif + } return HT_NewContextualMenuCursor ( table->GetHTView(), PR_FALSE, clickInBackground );