ReadDirectory() added in webrtc/test/testsupport/fileutils.h

webrtc/base/fileutils.h

Index: webrtc/base/fileutils.h
diff --git a/webrtc/base/fileutils.h b/webrtc/base/fileutils.h
index cddf2dc367f45a719add0b1a71ea497769e64eaa..4a6a171a7fed79cfaa1fc49adba96088281a2281 100644
--- a/webrtc/base/fileutils.h
+++ b/webrtc/base/fileutils.h
@@ -30,14 +30,9 @@ namespace rtc {
 class FileStream;
 class Pathname;
 
-//////////////////////////
-// Directory Iterator   //
-//////////////////////////
-
 // A DirectoryIterator is created with a given directory. It originally points
 // to the first file in the directory, and can be advanecd with Next(). This
 // allows you to get information about each file.
-
 class DirectoryIterator {
   friend class Filesystem;
  public:
@@ -46,20 +41,19 @@ class DirectoryIterator {
   // Destructor
   virtual ~DirectoryIterator();
 
-  // Starts traversing a directory
-  // dir is the directory to traverse
-  // returns true if the directory exists and is valid
-  // The iterator will point to the first entry in the directory
-  virtual bool Iterate(const Pathname &path);
+  // Starts traversing a directory and returns true if the directory exists and
+  // is valid. |dir| is the directory to traverse. The iterator will point to
+  // the first entry in the directory.
+  virtual bool Iterate(const Pathname &dir);
 
-  // Advances to the next file
-  // returns true if there were more files in the directory.
+  // Advances to the next file and returns true if there were more files in the
+  // directory.
   virtual bool Next();
 
-  // returns true if the file currently pointed to is a directory
+  // Returns true if the file currently pointed to is a directory.
   virtual bool IsDirectory() const;
 
-  // returns the name of the file currently pointed to
+  // Returns the name of the file currently pointed to.
   virtual std::string Name() const;

[kwiberg-webrtc, 2017/05/29 12:11:36]

Please don't clean up code that your CL otherwise doesn't touch (except in CLs
whose sole purpose is such cleanup). It makes reviewing harder, increases the
risk of conflicts, makes reverting harder, etc.

[AleBzk, 2017/05/29 13:04:48]

Yup. This was a left over from a previous PS in which I added a method to
DirectoryIterator.

 
  private: