test: remove &socket from test_inspector_socket

[indutny]

Checklist

• make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
• tests and/or benchmarks are included
• documentation is changed or added
• commit message follows commit guidelines

Affected core subsystem(s)

test

Description of change

What is socket, where did it came from? We don't know.

On a more serious note: socket is function from sys/socket.h. There
is no way casting it to structure could yield any useful results.

R= @nodejs/v8

[indutny]

Please let me know if I missed something obvious here...

[indutny]

CI: https://ci.nodejs.org/job/node-test-pull-request/4211/

[addaleax]

LGTM. I don’t think you’re missing anything, the address of socket(3) does end up in the cctest executable here.

[indutny]

Thanks!

[ofrobots]

/cc @nodejs/v8-inspector

[eugeneo]

This is a pending CL that also fixes this: #8505

(There used to be a variable socket that got removed at some point in the refactorings - that's when the function pointer came into the play)

[indutny]

Oh, cool! Closing this in a favor of #8505 then!