Fixes for re-enabling more MSVC level 4 warnings: net/quic/ edition

net/quic/quic_reliable_client_stream.cc

Index: net/quic/quic_reliable_client_stream.cc
diff --git a/net/quic/quic_reliable_client_stream.cc b/net/quic/quic_reliable_client_stream.cc
index 4eb74d6a31d8382e50b1f3678e318786a8684ac7..98b09cba8689aa02bb6b232dc90e6f3f4a7f433e 100644
--- a/net/quic/quic_reliable_client_stream.cc
+++ b/net/quic/quic_reliable_client_stream.cc
@@ -27,8 +27,10 @@ QuicReliableClientStream::~QuicReliableClientStream() {
 uint32 QuicReliableClientStream::ProcessData(const char* data,
                                              uint32 data_len) {
   // TODO(rch): buffer data if we don't have a delegate.
+  // !!! Callers seem to assume this will return 0 (or some value less than
+  // |data_len|) on error!  Returning a very large positive value may be wrong?
   if (!delegate_)
-    return ERR_ABORTED;
+    return static_cast<uint32>(ERR_ABORTED);

[Ryan Hamilton, 2014/07/08 03:37:08]

Yes, this seems problematic. :> It should probably do something very similar to
the rv != OK block below. I'll take care of fixing this in a different CL.

[Peter Kasting, 2014/07/08 06:26:46]

So do you want me to just remove the comment and leave the static cast here for
now?

[Ryan Hamilton, 2014/07/08 18:00:57]

Either way. The comment is a good reminder in any case :>

[Peter Kasting, 2014/07/08 18:16:46]

I think what I'll do, then, is change to something like this, assuming you don't
mind:

  // TODO(rch): buffer data if we don't have a delegate.
  // Also, if we're going to return 0 here, we should probably do something
  // like the "rv != OK" code below.
  if (!delegate_)
    return 0;

[Ryan Hamilton, 2014/07/08 18:24:01]

Sure, that's fine. Or you can simply do:

  if (!delegate_) {
    DLOG(ERROR) << "Missing delegate";
    Reset(QUIC_STREAM_CANCELLED);
    return 0;
  }

Which I was planning to land in https://codereview.chromium.org/372343002/ But
if you want to do it here, that's totally good too.

[Peter Kasting, 2014/07/08 18:44:45]

Ah!  I'd rather let you make the functional change through the normal review
process if you've already got it in the pipeline.  I'll just revert my change to
this file and leave it in your hands.

 
   int rv = delegate_->OnDataReceived(data, data_len);
   if (rv != OK) {