I'd drop this and move it into CGUIBaseContainer::OnAction().  That way the `<oninfo>` takes priority over whatever the window would do, which seems the correct way to do things.