Clean up nsContextMenu and make it a bit more window-agnostic

yay!

Comment on attachment 242265 [details] [diff] [review] checkpoint .getComputedStyle(elem, "").getPropertyValue( prop ); nit: unnecessary spaces around parens I think setItemAttr should be put in a global place, that code pattern is repeated a lot (I can file a new bug for that).

Checking in browser/base/jar.mn; /cvsroot/mozilla/browser/base/jar.mn,v <-- jar.mn new revision: 1.109; previous revision: 1.108 done Checking in browser/base/content/browser-context.inc; /cvsroot/mozilla/browser/base/content/browser-context.inc,v <-- browser-contex t.inc new revision: 1.27; previous revision: 1.26 done Checking in browser/base/content/browser-sets.inc; /cvsroot/mozilla/browser/base/content/browser-sets.inc,v <-- browser-sets.inc new revision: 1.83; previous revision: 1.82 done Checking in browser/base/content/browser.js; /cvsroot/mozilla/browser/base/content/browser.js,v <-- browser.js new revision: 1.731; previous revision: 1.730 done Checking in browser/base/content/browser.xul; /cvsroot/mozilla/browser/base/content/browser.xul,v <-- browser.xul new revision: 1.327; previous revision: 1.326 done RCS file: /cvsroot/mozilla/browser/base/content/nsContextMenu.js,v done Checking in browser/base/content/nsContextMenu.js; /cvsroot/mozilla/browser/base/content/nsContextMenu.js,v <-- nsContextMenu.js initial revision: 1.1 done Checking in browser/base/content/pageInfo.js; /cvsroot/mozilla/browser/base/content/pageInfo.js,v <-- pageInfo.js new revision: 1.32; previous revision: 1.31 done Checking in browser/base/content/web-panels.xul; /cvsroot/mozilla/browser/base/content/web-panels.xul,v <-- web-panels.xul new revision: 1.18; previous revision: 1.17 done

Sorry to spam this bug, but could it be related to a display problem of organize bookmarks window (see bug 359361) ?