Record UMA statistics for file_stream operations.

net/base/file_stream_win.cc

 // Copyright (c) 2011 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/base/file_stream.h"
 
 #include <windows.h>
 
 #include "base/file_path.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/metrics/histogram.h"
 #include "base/threading/thread_restrictions.h"
+#include "net/base/file_stream_metrics.h"
 #include "net/base/net_errors.h"
 
 namespace net {
 
 // Ensure that we can just use our Whence values directly.
 COMPILE_ASSERT(FROM_BEGIN == FILE_BEGIN, bad_whence_begin);
 COMPILE_ASSERT(FROM_CURRENT == FILE_CURRENT, bad_whence_current);
 COMPILE_ASSERT(FROM_END == FILE_END, bad_whence_end);
 
 static void SetOffset(OVERLAPPED* overlapped, const LARGE_INTEGER& offset) {
@@ skipping 22 matching lines @@
       LOG(WARNING) << "Unknown error " << err << " mapped to net::ERR_FAILED";
       return ERR_FAILED;
   }
 }
 
 // FileStream::AsyncContext ----------------------------------------------
 
 class FileStream::AsyncContext : public MessageLoopForIO::IOHandler {
  public:
   AsyncContext(FileStream* owner)
-      : owner_(owner), context_(), callback_(NULL), is_closing_(false) {
+      : owner_(owner), context_(), callback_(NULL), is_closing_(false),
+        record_uma_(false) {
     context_.handler = this;
   }
   ~AsyncContext();
 
   void IOCompletionIsPending(CompletionCallback* callback);
 
   OVERLAPPED* overlapped() { return &context_.overlapped; }
   CompletionCallback* callback() const { return callback_; }
 
+  void EnableRecording() {
+    record_uma_ = true;
+  }
+
  private:
   virtual void OnIOCompleted(MessageLoopForIO::IOContext* context,
                              DWORD bytes_read, DWORD error);
 
   FileStream* owner_;
   MessageLoopForIO::IOContext context_;
   CompletionCallback* callback_;
   bool is_closing_;
+  bool record_uma_;
 };
 
 FileStream::AsyncContext::~AsyncContext() {
   is_closing_ = true;
   bool waited = false;
   base::TimeTicks start = base::TimeTicks::Now();
   while (callback_) {
     waited = true;
     MessageLoopForIO::current()->WaitForIOCompletion(INFINITE, this);
   }
@@ skipping 14 matching lines @@
     MessageLoopForIO::IOContext* context, DWORD bytes_read, DWORD error) {
   DCHECK_EQ(&context_, context);
   DCHECK(callback_);
 
   if (is_closing_) {
     callback_ = NULL;
     return;
   }
 
   int result = static_cast<int>(bytes_read);
-  if (error && error != ERROR_HANDLE_EOF)
+  if (error && error != ERROR_HANDLE_EOF) {
+    if (record_uma_)
+      RecordFileError(error, FILE_ERROR_SOURCE_READ);

[cbentzel, 2011/08/16 13:36:02]

This could be done for both reads and writes. You probably need to record the
source in the AsyncContext. 

[ahendrickson, 2011/08/17 20:12:04]

Done.

     result = MapErrorCode(error);
+  }
 
   if (bytes_read)
     IncrementOffset(&context->overlapped, bytes_read);
 
   CompletionCallback* temp = NULL;
   std::swap(temp, callback_);
   temp->Run(result);
 }
 
 // FileStream ------------------------------------------------------------
 
 FileStream::FileStream()
     : file_(INVALID_HANDLE_VALUE),
       open_flags_(0),
-      auto_closed_(true) {
+      auto_closed_(true),
+      record_uma_(false) {
 }
 
 FileStream::FileStream(base::PlatformFile file, int flags)
     : file_(file),
       open_flags_(flags),
-      auto_closed_(false) {
+      auto_closed_(false),
+      record_uma_(false) {
   // If the file handle is opened with base::PLATFORM_FILE_ASYNC, we need to
   // make sure we will perform asynchronous File IO to it.
   if (flags & base::PLATFORM_FILE_ASYNC) {
     async_context_.reset(new AsyncContext(this));
     MessageLoopForIO::current()->RegisterIOHandler(file_,
                                                    async_context_.get());
   }
 }
 
 FileStream::~FileStream() {
@@ skipping 16 matching lines @@
   if (IsOpen()) {
     DLOG(FATAL) << "File is already open!";
     return ERR_UNEXPECTED;
   }
 
   open_flags_ = open_flags;
   file_ = base::CreatePlatformFile(path, open_flags_, NULL, NULL);
   if (file_ == INVALID_HANDLE_VALUE) {
     DWORD error = GetLastError();
     LOG(WARNING) << "Failed to open file: " << error;
+    if (record_uma_)
+      RecordFileError(error, FILE_ERROR_SOURCE_OPEN);
     return MapErrorCode(error);
   }
 
   if (open_flags_ & base::PLATFORM_FILE_ASYNC) {
     async_context_.reset(new AsyncContext(this));
+    if (record_uma_)
+      async_context_->EnableRecording();
     MessageLoopForIO::current()->RegisterIOHandler(file_,
                                                    async_context_.get());
   }
 
   return OK;
 }
 
 bool FileStream::IsOpen() const {
   return file_ != INVALID_HANDLE_VALUE;
 }
 
 int64 FileStream::Seek(Whence whence, int64 offset) {
-  if (!IsOpen())
+  if (!IsOpen()) {
+    if (record_uma_)

[cbentzel, 2011/08/16 13:36:02]

Again - in this file I would remove the recording of bad IsOpen calls at the top
of the functions.

[ahendrickson, 2011/08/17 20:12:04]

Done.

+      RecordFileError(ERROR_INVALID_HANDLE, FILE_ERROR_SOURCE_IS_NOT_OPEN);
     return ERR_UNEXPECTED;
+  }
+
   DCHECK(!async_context_.get() || !async_context_->callback());
 
   LARGE_INTEGER distance, result;
   distance.QuadPart = offset;
   DWORD move_method = static_cast<DWORD>(whence);
   if (!SetFilePointerEx(file_, distance, &result, move_method)) {
     DWORD error = GetLastError();
     LOG(WARNING) << "SetFilePointerEx failed: " << error;
+    if (record_uma_)
+      RecordFileError(error, FILE_ERROR_SOURCE_SEEK);
     return MapErrorCode(error);
   }
   if (async_context_.get())
     SetOffset(async_context_->overlapped(), result);
   return result.QuadPart;
 }
 
 int64 FileStream::Available() {
   base::ThreadRestrictions::AssertIOAllowed();
 
-  if (!IsOpen())
+  if (!IsOpen()) {
+    if (record_uma_)
+      RecordFileError(ERROR_INVALID_HANDLE, FILE_ERROR_SOURCE_IS_NOT_OPEN);
     return ERR_UNEXPECTED;
+  }
 
   int64 cur_pos = Seek(FROM_CURRENT, 0);
   if (cur_pos < 0)
     return cur_pos;
 
   LARGE_INTEGER file_size;
   if (!GetFileSizeEx(file_, &file_size)) {
     DWORD error = GetLastError();
     LOG(WARNING) << "GetFileSizeEx failed: " << error;
+    if (record_uma_)
+      RecordFileError(error, FILE_ERROR_SOURCE_GET_SIZE);
     return MapErrorCode(error);
   }
 
   return file_size.QuadPart - cur_pos;
 }
 
 int FileStream::Read(
     char* buf, int buf_len, CompletionCallback* callback) {
-  if (!IsOpen())
+  if (!IsOpen()) {
+    if (record_uma_)
+      RecordFileError(ERROR_INVALID_HANDLE, FILE_ERROR_SOURCE_IS_NOT_OPEN);
     return ERR_UNEXPECTED;
+  }
+
   DCHECK(open_flags_ & base::PLATFORM_FILE_READ);
 
   OVERLAPPED* overlapped = NULL;
   if (async_context_.get()) {
     DCHECK(callback);
     DCHECK(!async_context_->callback());
     overlapped = async_context_->overlapped();
   } else {
     DCHECK(!callback);
     base::ThreadRestrictions::AssertIOAllowed();
   }
 
   int rv;
 
   DWORD bytes_read;
   if (!ReadFile(file_, buf, buf_len, &bytes_read, overlapped)) {
     DWORD error = GetLastError();
     if (async_context_.get() && error == ERROR_IO_PENDING) {
       async_context_->IOCompletionIsPending(callback);
       rv = ERR_IO_PENDING;
     } else if (error == ERROR_HANDLE_EOF) {
       rv = 0;  // Report EOF by returning 0 bytes read.
     } else {
       LOG(WARNING) << "ReadFile failed: " << error;
+      if (record_uma_)
+        RecordFileError(error, FILE_ERROR_SOURCE_READ);
       rv = MapErrorCode(error);
     }
   } else if (overlapped) {
     async_context_->IOCompletionIsPending(callback);
     rv = ERR_IO_PENDING;
   } else {
     rv = static_cast<int>(bytes_read);
   }
   return rv;
 }
@@ skipping 14 matching lines @@
     bytes_total += bytes_read;
     buf += bytes_read;
     to_read -= bytes_read;
   } while (bytes_total < buf_len);
 
   return bytes_total;
 }
 
 int FileStream::Write(
     const char* buf, int buf_len, CompletionCallback* callback) {
-  if (!IsOpen())
+  if (!IsOpen()) {
+    if (record_uma_)
+      RecordFileError(ERROR_INVALID_HANDLE, FILE_ERROR_SOURCE_IS_NOT_OPEN);
     return ERR_UNEXPECTED;
+  }
   DCHECK(open_flags_ & base::PLATFORM_FILE_WRITE);
 
   OVERLAPPED* overlapped = NULL;
   if (async_context_.get()) {
     DCHECK(callback);
     DCHECK(!async_context_->callback());
     overlapped = async_context_->overlapped();
   } else {
     DCHECK(!callback);
     base::ThreadRestrictions::AssertIOAllowed();
   }
 
   int rv;
   DWORD bytes_written;
   if (!WriteFile(file_, buf, buf_len, &bytes_written, overlapped)) {
     DWORD error = GetLastError();
     if (async_context_.get() && error == ERROR_IO_PENDING) {

[cbentzel, 2011/08/16 13:36:02]

So, in the Windows case I _don't_ think it will help to have a helper function
to MapAndRecordErrorCode() due to all the special cases. 

[ahendrickson, 2011/08/17 20:12:04]

It's not as useful, but still works.

       async_context_->IOCompletionIsPending(callback);
       rv = ERR_IO_PENDING;
     } else {
       LOG(WARNING) << "WriteFile failed: " << error;
+      if (record_uma_)
+        RecordFileError(error, FILE_ERROR_SOURCE_WRITE);
       rv = MapErrorCode(error);
     }
   } else if (overlapped) {
     async_context_->IOCompletionIsPending(callback);
     rv = ERR_IO_PENDING;
   } else {
     rv = static_cast<int>(bytes_written);
   }
   return rv;
 }
 
 int FileStream::Flush() {
   base::ThreadRestrictions::AssertIOAllowed();
 
-  if (!IsOpen())
+  if (!IsOpen()) {
+    if (record_uma_)
+      RecordFileError(ERROR_INVALID_HANDLE, FILE_ERROR_SOURCE_IS_NOT_OPEN);
     return ERR_UNEXPECTED;
+  }
 
   DCHECK(open_flags_ & base::PLATFORM_FILE_WRITE);
   if (FlushFileBuffers(file_)) {
     return OK;
   }
 
   int rv;
   DWORD error = GetLastError();
+  if (record_uma_)
+    RecordFileError(error, FILE_ERROR_SOURCE_FLUSH);
   rv = MapErrorCode(error);
   return rv;
 }
 
 int64 FileStream::Truncate(int64 bytes) {
   base::ThreadRestrictions::AssertIOAllowed();
 
-  if (!IsOpen())
+  if (!IsOpen()) {
+    if (record_uma_)
+      RecordFileError(ERROR_INVALID_HANDLE, FILE_ERROR_SOURCE_IS_NOT_OPEN);
     return ERR_UNEXPECTED;
+  }
 
   // We better be open for reading.
   DCHECK(open_flags_ & base::PLATFORM_FILE_WRITE);
 
   // Seek to the position to truncate from.
   int64 seek_position = Seek(FROM_BEGIN, bytes);
   if (seek_position != bytes)
     return ERR_UNEXPECTED;
 
   // And truncate the file.
   BOOL result = SetEndOfFile(file_);
   if (!result) {
     DWORD error = GetLastError();
     LOG(WARNING) << "SetEndOfFile failed: " << error;
+    if (record_uma_)
+      RecordFileError(error, FILE_ERROR_SOURCE_SET_EOF);
     return MapErrorCode(error);
   }
 
   // Success.
   return seek_position;
 }
 
+void FileStream::EnableRecording() {
+  record_uma_ = true;
+
+  if (async_context_.get())
+    async_context_->EnableRecording();
+}
+
 }  // namespace net