Fetch file metadata of files under Team Drives.

google_apis/drive/drive_api_url_generator.h

Index: google_apis/drive/drive_api_url_generator.h
diff --git a/google_apis/drive/drive_api_url_generator.h b/google_apis/drive/drive_api_url_generator.h
index af07333073254c422f8781f7589603fdd77d2d3a..4f8c456169fbafb33c0f38854d0b4a624bc3ef8d 100644
--- a/google_apis/drive/drive_api_url_generator.h
+++ b/google_apis/drive/drive_api_url_generator.h
@@ -22,6 +22,12 @@ class DriveApiUrlGenerator {
   // Note that this is an injecting point for a testing server.
   DriveApiUrlGenerator(const GURL& base_url,
                        const GURL& base_thumbnail_url);

[hashimoto, 2017/02/22 07:04:51]

You don't need to have this ctor when there is the one below.

[yamaguchi, 2017/02/22 08:41:53]

Removed as well as changing all constructor calls.

+
+  // For unit tests. Inject commandline flag value.
+  DriveApiUrlGenerator(const GURL& base_url,
+                       const GURL& base_thumbnail_url,
+                       bool is_team_drive_enabled);

[hashimoto, 2017/02/22 07:04:51]

Boolean argument is usually difficult to read.

Our style guide says "Consider changing the function signature to replace a bool
argument with an enum argument."
https://google.github.io/styleguide/cppguide.html#Variable_Comments

[yamaguchi, 2017/02/22 08:41:53]

Done.

+  DriveApiUrlGenerator(const DriveApiUrlGenerator& src);
   ~DriveApiUrlGenerator();
 
   // The base URL for communicating with the production drive api server.
@@ -126,6 +132,7 @@ class DriveApiUrlGenerator {
   const GURL base_url_;
   const GURL base_download_url_;
   const GURL base_thumbnail_url_;
+  const bool enable_team_drives_;
 
   // This class is copyable hence no DISALLOW_COPY_AND_ASSIGN here.
 };