[sldev] Tools menu autohiding fix in the 1.21 branch reverted?

[Ambrosia]

Greetings,

According to the 1.21 Nightly Build release notes at
http://wiki.secondlife.com/wiki/Release_Notes/Second_Life_Public_Nightly/1.21
, build 94534 and onward have a fix of

Fixed: VWR-6328: Request for "Tools" menu to not auto-hide.

However, when looking at the 1.21 build 94834 on the svn,
llFloaterTools.cpp clearly has

	gMenuBarView->setItemVisible(std::string("Tools"), FALSE);
	gMenuBarView->arrange();

in onClose, lines 815 and 816, and

	gMenuBarView->setItemVisible(std::string("Tools"), TRUE);
	gMenuBarView->arrange();

in onOpen, lines 781 and 782.

The same goes for llViewerMenu.cpp,

    gMenuBarView->setItemVisible("Tools", FALSE);
	gMenuBarView->arrange();

on lines 705 and 706.


Did the fix get reverted by accident? There are seemingly no if()
conditions that control this behavor, which makes me guess there is no
'do not hide the tools menu' option implemented either.

--Chalice Yao