c/browser, c/common, components M-N: Change ScopedTempDir::path() to GetPath()

chrome/browser/media_galleries/fileapi/itunes_data_provider_browsertest.cc

Index: chrome/browser/media_galleries/fileapi/itunes_data_provider_browsertest.cc
diff --git a/chrome/browser/media_galleries/fileapi/itunes_data_provider_browsertest.cc b/chrome/browser/media_galleries/fileapi/itunes_data_provider_browsertest.cc
index 63418e96ea031e74e2e2c27c63f934a1ae5cbbe7..e43705f929754ec64a4405230ed74a166cdacd03 100644
--- a/chrome/browser/media_galleries/fileapi/itunes_data_provider_browsertest.cc
+++ b/chrome/browser/media_galleries/fileapi/itunes_data_provider_browsertest.cc
@@ -114,12 +114,10 @@ class ITunesDataProviderTest : public InProcessBrowserTest {
     return ImportedMediaGalleryRegistry::ITunesDataProvider();
   }
 
-  const base::FilePath& library_dir() const {
-    return library_dir_.path();
-  }
+  const base::FilePath& library_dir() const { return library_dir_.GetPath(); }
 
   base::FilePath XmlFile() const {
-    return library_dir_.path().AppendASCII("library.xml");
+    return library_dir_.GetPath().AppendASCII("library.xml");

[Reilly Grant (use Gerrit), 2016/09/14 18:06:19]

I'm assuming this is correct because it is existing code but it would be good if
base::ScopedTempDir::GetPath() said whether or not the path was guaranteed to
have a trailing slash.

[vabr (Chromium), 2016/09/14 20:14:39]

This is guaranteed by FilePath itself ([1]), so dealing with trailing slashes is
transparent to ScopedTempDir.

[1]
https://cs.chromium.org/chromium/src/base/files/file_path.h?sq=package:chromi...

   }
 
   void ExpectTrackLocation(const std::string& artist, const std::string& album,