Don't re-define CompletionCallback, use net::CompletionCallback instead

Socket classes under extensions/browser/api/socket define and use different
types of callbacks, but there's no need to define CompletionCallback as
that one is already defined in //net/base/completion_once_callback.h, so
let's switch to using that one instead, similar to what's already being
done for CompletionOnceCallback.

TBR=rockot@google.com

Bug: 807724
Change-Id: Ibbd7391787f0edbc71eb0e2a87a61f2356b0822a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1520500
Commit-Queue: Mario Sanchez Prada <mario@igalia.com>
Reviewed-by: Bence Béky <bnc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#640766}
5 files changed