From Bug 776271: > This was added in bug 365171. Whereas this.focus() in buildContextMenu is > gone by now, this seems to be a leftover. As far as I can tell, no special > treatment is needed. Focus should just go back to where it was.
Created attachment 647629 [details] [diff] [review] patch v0
Comment on attachment 647629 [details] [diff] [review] patch v0 Looks good. No related errors in the error console. r=me
Comment on attachment 647629 [details] [diff] [review] patch v0 [Approval Request Comment] Regression caused by (bug #): bug 365171 FIXED 2006-12-28. This is an identical Sm port of Fx bug 776271. User impact if declined: focus behaviour remains bad after right-click Testing completed (on m-c, etc.): FIXED on Firefox (m-c) since Jul.31, on SeaMonkey (c-c) since Aug.02 Risk to taking this patch (and alternatives if risky): risk quasi null (removes a single focus() statement) String changes made by this patch: none
sorry I forgot to add Cc when asking for approval, see comment #4.