Fix a bunch of clang warnings/errors.

net/disk_cache/entry_impl.cc

 // Copyright (c) 2006-2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/disk_cache/entry_impl.h"
 
 #include "base/histogram.h"
 #include "base/message_loop.h"
 #include "base/string_util.h"
 #include "net/base/io_buffer.h"
@@ skipping 46 matching lines @@
   entry_->Release();
   delete this;
 }
 
 void SyncCallback::Discard() {
   callback_ = NULL;
   buf_ = NULL;
   OnFileIOComplete(0);
 }
 
-// Clears buffer before offset and after valid_len, knowing that the size of
-// buffer is kMaxBlockSize.
-void ClearInvalidData(char* buffer, int offset, int valid_len) {
-  DCHECK_GE(offset, 0);
-  DCHECK_GE(valid_len, 0);
-  DCHECK(disk_cache::kMaxBlockSize >= offset + valid_len);
-  if (offset)
-    memset(buffer, 0, offset);
-  int end = disk_cache::kMaxBlockSize - offset - valid_len;
-  if (end)
-    memset(buffer + offset + valid_len, 0, end);
-}
-
 const int kMaxBufferSize = 1024 * 1024;  // 1 MB.
 
 }  // namespace
 
 namespace disk_cache {
 
 // This class handles individual memory buffers that store data before it is
 // sent to disk. The buffer can start at any offset, but if we try to write to
 // anywhere in the first 16KB of the file (kMaxBlockSize), we set the offset to
 // zero. The buffer grows up to a size determined by the backend, to keep the
@@ skipping 1182 matching lines @@
   Trace("%s 0x%p 0x%x 0x%x", msg, reinterpret_cast<void*>(this),
         entry_.address().value(), node_.address().value());
 
   Trace("  data: 0x%x 0x%x 0x%x", entry_.Data()->data_addr[0],
         entry_.Data()->data_addr[1], entry_.Data()->long_key);
 
   Trace("  doomed: %d 0x%x", doomed_, dirty);
 }
 
 }  // namespace disk_cache