Make ifdefs consistent in zib.

third_party/zlib/google/zip_reader.cc

Index: third_party/zlib/google/zip_reader.cc
diff --git a/third_party/zlib/google/zip_reader.cc b/third_party/zlib/google/zip_reader.cc
index fdeb1259b3a0f4d9911c2300c98223a3e0c70ca8..a31d1c073fa2cd7c68c7302169b63e242a503f8a 100644
--- a/third_party/zlib/google/zip_reader.cc
+++ b/third_party/zlib/google/zip_reader.cc
@@ -514,7 +514,7 @@ FileWriterDelegate::FileWriterDelegate(base::File* file)
 }
 
 FileWriterDelegate::~FileWriterDelegate() {
-#if !defined(NDEBUG) || defined(DCHECK_ALWAYS_ON)
+#if DCHECK_IS_ON()
   const bool success =

[satorux1, 2016/11/21 01:23:06]

Having a variable guarded by #ifdef looks a bit hard to read (not your fault).
How about:

  if (!file_->SetLength(file_length_)) {
    DPLOG(ERROR) << "Failed updating length of written file";
  }

[Alexander Alekseev, 2016/11/21 10:22:27]

Done.

 #endif
       file_->SetLength(file_length_);