//base: Make ScopedTempDir::path() a GetPath() with a DCHECK

base/files/file_util_proxy_unittest.cc

 // Copyright (c) 2012 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_util_proxy.h"
 
 #include "base/bind.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 24 matching lines @@
                       const File::Info& file_info) {
     error_ = error;
     file_info_ = file_info;
     MessageLoop::current()->QuitWhenIdle();
   }
 
  protected:
   TaskRunner* file_task_runner() const {
     return file_thread_.task_runner().get();
   }
-  const FilePath& test_dir_path() const { return dir_.path(); }
-  const FilePath test_path() const { return dir_.path().AppendASCII("test"); }
+  const FilePath& test_dir_path() const { return dir_.GetPath(); }

[danakj, 2016/09/07 20:47:18]

dittos

[vabr (Chromium), 2016/09/08 07:45:35]

test_path -- done
test_dir_path -- not used anywhere, so removed

+  const FilePath test_path() const {
+    return dir_.GetPath().AppendASCII("test");
+  }
 
   MessageLoopForIO message_loop_;
   Thread file_thread_;
 
   ScopedTempDir dir_;
   File::Error error_;
   FilePath path_;
   File::Info file_info_;
   std::vector<char> buffer_;
   WeakPtrFactory<FileUtilProxyTest> weak_factory_;
@@ skipping 65 matching lines @@
 
   // The returned values may only have the seconds precision, so we cast
   // the double values to int here.
   EXPECT_EQ(static_cast<int>(last_modified_time.ToDoubleT()),
             static_cast<int>(info.last_modified.ToDoubleT()));
   EXPECT_EQ(static_cast<int>(last_accessed_time.ToDoubleT()),
             static_cast<int>(info.last_accessed.ToDoubleT()));
 }
 
 }  // namespace base