Built on Linux without issue. None of these are scriptable.
Johnny, feel free to bounce the review to me if needed.
Comment on attachment 620189 [details] [diff] [review] patch Nice dead code removal! The one thing that should be changed here before this lands is that we should update the IID of nsITextControlElement. Other than that this looks great! Thanks for the patch!
Created attachment 622527 [details] [diff] [review] patch for checkin now with updated IID for nsITextControlElement (carrying over r=jst)
Created attachment 622542 [details] [diff] [review] patch for checkin oops, last patch should have used a 2 space, not 4 space, indent for the 2nd line of the IID. Here's a fix. (carrying over r+ again)
...I literally was updating the bug with the checkin commit when I mid-aired with you. https://hg.mozilla.org/integration/mozilla-inbound/rev/82853146de86 Please post a follow-up patch rebased on top.
Nevermind, I pushed a followup. https://hg.mozilla.org/integration/mozilla-inbound/rev/b2b38c2f29e7
Sorry about that Ryan!