341881 - Disable "reload" menu item when page is loading

Status: VERIFIED INVALID

Description

[froodian (Ian Leue)]

Currently the toolbar item gets disabled, but the menu item stays enabled.

Comment 1

[froodian (Ian Leue)]

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]);

Comment 2

[froodian (Ian Leue)]

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.