Revert 90625

remoting/base/decompressor_zlib.cc

Index: remoting/base/decompressor_zlib.cc
diff --git a/remoting/base/decompressor_zlib.cc b/remoting/base/decompressor_zlib.cc
index 770abd5a3b1a15b00b93a0e022f94e39ab73172b..6259f2d037ea0c5cf51bef62495062083bc017ac 100644
--- a/remoting/base/decompressor_zlib.cc
+++ b/remoting/base/decompressor_zlib.cc
@@ -59,7 +59,10 @@ void DecompressorZlib::InitStream() {
   stream_->zfree = Z_NULL;
   stream_->opaque = Z_NULL;
 
-  inflateInit(stream_.get());
+  int ret = inflateInit(stream_.get());
+  if (ret != Z_OK) {
+    NOTREACHED() << "zlib init failed";

[Ami GONE FROM CHROMIUM, 2011/06/27 21:54:35]

You're being consistent with the use of NOTREACHED above, but you lose
information relative to 
DCHECK_EQ(ret, Z_OK);
which I'd use if I were you.
(DCHECK version gets you ret in the log, NOTREACHED doesn't; NOTREACHED is
#define'd as DCHECK(false), so all else should be the same, except the DCHECK
version won't do the comparison at all in NDEBUG builds).

[Greg Billock, 2011/06/27 22:03:54]

Makes sense. updated.

+  }
 }
 
 }  // namespace remoting