EXT-1339 Create onClose method in LLFloater, and remove most bindings to mCloseSignal. In the vast majority of cases, mCloseSignal is being bound to an onClose function. Just make it virtual and be done with it. Renamed a couple of LLPanel close methods to onClosePanel() to distinguish them. Reviewed with Richard.

1 files changed, 3 insertions, 4 deletions

diff --git a/indra/newview/llinspectavatar.cpp b/indra/newview/llinspectavatar.cpp
index fd301505cd..4046c893c1 100644
--- a/indra/newview/llinspectavatar.cpp
+++ b/indra/newview/llinspectavatar.cpp
@@ -83,7 +83,7 @@ public:
 	/*virtual*/ void onOpen(const LLSD& avatar_id);
 
 	// When closing they should close their gear menu 
-	/*virtual*/ void onClose();
+	/*virtual*/ void onClose(bool app_quitting);
 	
 	// Inspectors close themselves when they lose focus
 	/*virtual*/ void onFocusLost();
@@ -228,8 +228,6 @@ BOOL LLInspectAvatar::postBuild(void)
 	getChild<LLUICtrl>("volume_slider")->setCommitCallback(
 		boost::bind(&LLInspectAvatar::onVolumeChange, this, _2));
 
-	mCloseSignal.connect(boost::bind(&LLInspectAvatar::onClose, this));
-
 	return TRUE;
 }
 
@@ -294,7 +292,8 @@ void LLInspectAvatar::onOpen(const LLSD& data)
 	updateVolumeSlider();
 }
 
-void LLInspectAvatar::onClose()
+// virtual
+void LLInspectAvatar::onClose(bool app_quitting)
 {  
   getChild<LLMenuButton>("gear_btn")->hideMenu();
 }