process: pid can be a string in process.kill()

Not allowing string was a change from v0.10 behaviour, commented on in
joyent/node#7991. Allow them again, but still check that argument is
numberish. Also, simplify the fragile and non-portable test code
introduced in 832ec1cd507 that required fixups 2a415358ee, and
ef3c4ed3d.

Sorry about the test awfulness I introduced, @misterdjules @tjfontaine.
Mind changing this to `'-1'`? Also the same for other numbers.

`String()` is a funny thing that can bring confusion. `String('happy')` returns a primitive while `new String('happy')` returns a fully instantiated object. I'd like the test to be as explicit as possible.