Currently the toolbar item gets disabled, but the menu item stays enabled.
Created attachment 226020 [details] [diff] [review] Patch Note that 90% of this is cosmetic cleanup. The meat of the patch is if (action == @selector(doReload:)) return (![[browserController getBrowserWrapper] isBusy] && ![browserController bookmarkManagerIsVisible]);
OK. The real problem here is bug 304061 (rather than disable the menu item, enable the toolbar item), and the fact that reloading while a page is in the "Looking up http://foo.bar" stage does Bad Things (don't know if that's a filed bug yet). INVALID.