Assertion failure fix  * Closing a submenu was hitting NOTREACHED in OnMenuHidden.  * Changed to register OnMenuHidden callback only when we're    about to open the menu, then disconnect it when closed.    This is safer as the same menu can be used as submenu    (although that's probably not great style)
Assertion failure fix
 * Closing a submenu was hitting NOTREACHED in OnMenuHidden.
 * Changed to register OnMenuHidden callback only when we're
   about to open the menu, then disconnect it when closed.
   This is safer as the same menu can be used as submenu
   (although that's probably not great style)

Review URL: http://codereview.chromium.org/328002

git-svn-id: de016e52bd170d2d4f2344f9bf92d50478b649e0@30061 0039d316-1c4b-4281-b951-d872f2087c98

yitian134/chromium,adobe/chromium,gavinp/chromium,Crystalnix/house-of-life-chromium,ropik/chromium,yitian134/chromium,Crystalnix/house-of-life-chromium,yitian134/chromium,Crystalnix/house-of-life-chromium,gavinp/chromium,ropik/chromium,adobe/chromium,adobe/chromium,gavinp/chromium,Crystalnix/house-of-life-chromium,yitian134/chromium,ropik/chromium,gavinp/chromium,ropik/chromium,gavinp/chromium,Crystalnix/house-of-life-chromium,Crystalnix/house-of-life-chromium,adobe/chromium,yitian134/chromium,adobe/chromium,Crystalnix/house-of-life-chromium,Crystalnix/house-of-life-chromium,yitian134/chromium,gavinp/chromium,yitian134/chromium,gavinp/chromium,ropik/chromium,gavinp/chromium,adobe/chromium,adobe/chromium,ropik/chromium,ropik/chromium,Crystalnix/house-of-life-chromium,ropik/chromium,adobe/chromium,ropik/chromium,Crystalnix/house-of-life-chromium,adobe/chromium,Crystalnix/house-of-life-chromium,yitian134/chromium,adobe/chromium,yitian134/chromium,yitian134/chromium,gavinp/chromium,adobe/chromium,gavinp/chromium