Record UMA statistics for file_stream operations.

net/base/file_stream_metrics.h

Index: net/base/file_stream_metrics.h
diff --git a/net/base/file_stream_metrics.h b/net/base/file_stream_metrics.h
new file mode 100644
index 0000000000000000000000000000000000000000..54eb0aa164b8b640e65dd1752a8e18b1be42a9fc
--- /dev/null
+++ b/net/base/file_stream_metrics.h
@@ -0,0 +1,41 @@
+// 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.
+
+// File error statistics gathering.
+
+#ifndef NET_BASE_FILE_STREAM_METRICS_H_
+#define NET_BASE_FILE_STREAM_METRICS_H_
+#pragma once
+
+namespace net {
+
+enum FileErrorSource {
+  FILE_ERROR_SOURCE_OPEN = 0,
+  FILE_ERROR_SOURCE_WRITE,
+  FILE_ERROR_SOURCE_READ,
+  FILE_ERROR_SOURCE_SEEK,
+  FILE_ERROR_SOURCE_FLUSH,
+  FILE_ERROR_SOURCE_SET_EOF,
+  FILE_ERROR_SOURCE_GET_SIZE,
+  FILE_ERROR_SOURCE_COUNT,
+};
+
+// UMA error statistics gathering.

[cbentzel, 2011/08/31 19:53:23]

Perhaps these should be in a nested file_stream_metrics namespace in net. They
have pretty generic names for top-level functions.

[ahendrickson, 2011/09/01 18:01:20]

I've had objections to using nested namespaces here, so I've renamed the
functions to be less generic.

+// Put the error value into a bucket.
+int GetErrorBucket(int error);
+
+// The largest bucket number, plus 1.
+int MaxBucket();
+
+// The highest error value we want to individually report.
+int MaxError();
+
+// |error| is a platform-specific error (Windows or Posix).
+// |source| indicates the operation that resulted in the error.
+// |record| is a flag indicating that we are interested in this error.
+void RecordFileError(int error, FileErrorSource source, bool record);
+
+}  // namespace net
+
+#endif  // NET_BASE_FILE_STREAM_METRICS_H_