Menu should only be built once

[bbondy]

We currently build a menu from template multiple times.
We should instead only do this only once, the call to Menu.buildFromTemplate returns a menu object and we can use it to find items in the menu and update them as needed.

The problem that happens is when a menu is regenerated and the menu is expanded, then clicks on menu items won't work. This happens especially around when the app starts if you try to quit the menu item click doesn't work.

[bsclifton]

Checking this out now...

[bsclifton]

While not ideal, some great improvements were made with #3055 that I think meet the criteria

The dynamic menus (Bookmarks/History) which are in development presented a challenge. Electron doesn't expose a way to remove MenuItems currently (it's definitely possible from a win32 perspective, I'm not sure about X11/macOS). We'd need to expose that before we can dynamically insert/delete without having to rebuild the menu