Change DeterministicSocketData to no longer inherit from SocketDataProvider

Change DeterministicSocketData to no longer inherit from SocketDataProvider
and change the DeterministicSockets to not implement AsyncSocket.

This helps make clear using the type system that the Deterministic
classes are distinct from the non-Deterministic classes.

Committed: https://crrev.com/08051efe5a27ee82248a45eee41a99a0943c33cf
Cr-Commit-Position: refs/heads/master@{#330590}

[Ryan Hamilton, 2015-05-19 02:45:13]

rch@chromium.org changed reviewers:
+ rtenneti@chromium.org

[ramant (doing other things), 2015-05-19 17:06:35]

lgtm

https://codereview.chromium.org/1135553003/diff/1/net/socket/socket_test_util.cc
File net/socket/socket_test_util.cc (right):

https://codereview.chromium.org/1135553003/diff/1/net/socket/socket_test_util...
net/socket/socket_test_util.cc:749: // // written. Otherwise return the error
code.
overly nit: " // // written" -> " // written"

[Ryan Hamilton, 2015-05-19 17:33:26]

Thanks!

https://codereview.chromium.org/1135553003/diff/1/net/socket/socket_test_util.cc
File net/socket/socket_test_util.cc (right):

https://codereview.chromium.org/1135553003/diff/1/net/socket/socket_test_util...
net/socket/socket_test_util.cc:749: // // written. Otherwise return the error
code.
On 2015/05/19 17:06:35, ramant wrote:
> overly nit: " // // written" -> " // written"

Done.

[Ryan Hamilton, 2015-05-19 17:33:30]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2015-05-19 17:33:31]

The patchset sent to the CQ was uploaded after l-g-t-m from
rtenneti@chromium.org
Link to the patchset: https://codereview.chromium.org/1135553003/#ps20001
(title: "fix comments")

[commit-bot: I haz the power, 2015-05-19 17:35:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1135553003/20001

[commit-bot: I haz the power, 2015-05-19 17:48:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-05-19 17:48:10]

Try jobs failed on following builders:
  linux_chromium_gn_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Ryan Hamilton, 2015-05-19 18:55:36]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2015-05-19 18:55:37]

The patchset sent to the CQ was uploaded after l-g-t-m from
rtenneti@chromium.org
Link to the patchset: https://codereview.chromium.org/1135553003/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2015-05-19 18:56:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1135553003/40001

[commit-bot: I haz the power, 2015-05-19 20:07:31]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-05-19 20:08:29]

Patchset 3 (id:??) landed as
https://crrev.com/08051efe5a27ee82248a45eee41a99a0943c33cf
Cr-Commit-Position: refs/heads/master@{#330590}