Bug 469158 removed the option of avoiding the prompt when clearing private data, so there's no reason that we need to set the label to sanitizeWithPromptLabel2 in initializeSanitizer(), since we could instead just update clearRecentHistoryCmd.label accordingly.
Comment on attachment 422037 [details] [diff] [review] First try I'm not a reviewer (I've requested review from one though), however you have to change the entity id when you change the entity for l10n reasons.
Created attachment 422106 [details] [diff] [review] v2 Thanks for the comment, i hope this is the change you meant.
That is, only the |.label| part isn't supposed to be the part that is changed, rather the part before the '.'. Change it to something like 'clearRecentHistory.label'.
Should I make the same change to the name of the accesskey entity too, or only the label?
Created attachment 422111 [details] [diff] [review] v3 Lets start with just the label.
(In reply to comment #5) > Should I make the same change to the name of the accesskey entity too, or only > the label? There are some tools which try to associate the labels and accesskeys, and they usually use try to look for a foorbar.accesskey for each foobar.label. So, in this case, I would actually suggest changing the accesskey entity name as well, in order to keep these two in sync.
Created attachment 422219 [details] [diff] [review] v3 Something like this? I hold on the review request, I don't feel like spamming Gavin any more...
(In reply to comment #8) > Created an attachment (id=422219) [details] > v3 > > Something like this? Yes!
Landed on trunk: http://hg.mozilla.org/mozilla-central/rev/e1ac16b86b80 Thanks for your patch, Cork!