Revert of Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider so that all provider… (patchset #1 id:1 of https://codereview.chromium.org/1127623003/)

Reason for revert:
Still not compiling.  Order-dependent revert fun to ensue.  End result: this CL and any dependent CL will be re-instated.

Original issue's description:
> Re-instating r328154 because it's depended on to compile.
>
> Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider so that all provider subclasses can have their state verified.
>
> Review URL: https://codereview.chromium.org/1114383003
>
> Cr-Commit-Position: refs/heads/master@{#328154}
>
> TBR=rch@chromium.org
> BUG=
>
> Committed: https://chromium.googlesource.com/chromium/src/+/2b2149aa9dfc5fbbb4ef0df2b89b5f392383f41a

TBR=rch@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Review URL: https://codereview.chromium.org/1123643006

Cr-Commit-Position: refs/heads/master@{#328206}

1 files changed, 0 insertions, 8 deletions

diff --git a/remoting/signaling/xmpp_signal_strategy_unittest.cc b/remoting/signaling/xmpp_signal_strategy_unittest.cc
index 2193982..b4c4a53 100644
--- a/remoting/signaling/xmpp_signal_strategy_unittest.cc
+++ b/remoting/signaling/xmpp_signal_strategy_unittest.cc
@@ -29,14 +29,6 @@ class XmppSocketDataProvider: public net::SocketDataProvider {
 
   void Reset() override {}
 
-  bool AllReadDataConsumed() const override {
-    return true;
-  }
-
-  bool AllWriteDataConsumed() const override {
-    return true;
-  }
-
   void ReceiveData(const std::string& text) {
     socket()->OnReadComplete(
         net::MockRead(net::ASYNC, text.data(), text.size()));