Add granular file tracing.

base/files/file.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 "base/files/file.h"
 #include "base/files/file_path.h"
 #include "base/metrics/histogram.h"
 #include "base/timer/elapsed_timer.h"
+#include "base/trace_event/scoped_file_trace.h"
+#include "base/trace_event/trace_event.h"
 
 #if defined(OS_POSIX)
 #include "base/files/file_posix_hooks_internal.h"
 #endif
 
 namespace base {
 
 File::Info::Info()
     : size(0),
       is_directory(false),
       is_symbolic_link(false) {
 }
 
 File::Info::~Info() {
 }
 
 File::File()
     : error_details_(FILE_ERROR_FAILED),
       created_(false),
       async_(false) {
 }
 
 #if !defined(OS_NACL)
-File::File(const FilePath& name, uint32 flags)
+File::File(const FilePath& path, uint32 flags)
     : error_details_(FILE_OK),
       created_(false),
       async_(false) {
-  Initialize(name, flags);
+  Initialize(path, flags);
 }
 #endif
 
 File::File(PlatformFile platform_file)
     : file_(platform_file),
       error_details_(FILE_OK),
       created_(false),
       async_(false) {
 #if defined(OS_POSIX)
   DCHECK_GE(platform_file, -1);
   if (IsValid())
     ProtectFileDescriptor(platform_file);
 #endif
 }
 
 File::File(Error error_details)
     : error_details_(error_details),
       created_(false),
       async_(false) {
 }
 
 File::File(RValue other)
     : file_(other.object->TakePlatformFile()),
+      path_(other.object->path_),
       error_details_(other.object->error_details()),
       created_(other.object->created()),
       async_(other.object->async_) {
 #if defined(OS_POSIX)
-   if (IsValid())
-     ProtectFileDescriptor(GetPlatformFile());
+  if (IsValid())
+    ProtectFileDescriptor(GetPlatformFile());
 #endif
 }
 
 File::~File() {
   // Go through the AssertIOAllowed logic.
   Close();
+  TRACE_EVENT_ASYNC_END1(ScopedFileTrace::kGroup, "File", this,
+                         "path", path_.AsUTF8Unsafe());
 }
 
 File& File::operator=(RValue other) {
   if (this != other.object) {
     Close();
     SetPlatformFile(other.object->TakePlatformFile());
     error_details_ = other.object->error_details();
     created_ = other.object->created();
     async_ = other.object->async_;
   }
   return *this;
 }
 
 #if !defined(OS_NACL)
-void File::Initialize(const FilePath& name, uint32 flags) {
-  if (name.ReferencesParent()) {
+void File::Initialize(const FilePath& path, uint32 flags) {
+  path_ = path;
+  if (path_.ReferencesParent()) {
     error_details_ = FILE_ERROR_ACCESS_DENIED;
     return;
   }
-  DoInitialize(name, flags);
+  TRACE_EVENT_ASYNC_BEGIN1(ScopedFileTrace::kGroup, "File", this,
+                           "path", path_.AsUTF8Unsafe());

[Dan Beam, 2015/04/23 22:32:19]

this seemed like the sanest place to start this, but open to suggestions...

+  DoInitialize(flags);
 }
 #endif
 
 std::string File::ErrorToString(Error error) {
   switch (error) {
     case FILE_OK:
       return "FILE_OK";
     case FILE_ERROR_FAILED:
       return "FILE_ERROR_FAILED";
     case FILE_ERROR_IN_USE:
@@ skipping 35 matching lines @@
 }
 
 bool File::Flush() {
   ElapsedTimer timer;
   bool return_value = DoFlush();
   UMA_HISTOGRAM_TIMES("PlatformFile.FlushTime", timer.Elapsed());
   return return_value;
 }
 
 }  // namespace base