Fetch file metadata of files under Team Drives.

google_apis/drive/drive_api_url_generator.h

 // Copyright (c) 2013 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.
 
 #ifndef GOOGLE_APIS_DRIVE_DRIVE_API_URL_GENERATOR_H_
 #define GOOGLE_APIS_DRIVE_DRIVE_API_URL_GENERATOR_H_
 
 #include <stdint.h>
 
 #include <string>
 
 #include "base/macros.h"
 #include "url/gurl.h"
 
 namespace google_apis {
 
 // This class is used to generate URLs for communicating with drive api
 // servers for production, and a local server for testing.
 class DriveApiUrlGenerator {
  public:
   // |base_url| is the path to the target drive api server.
   // 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.
   static const char kBaseUrlForProduction[];
 
   // The base URL for the thumbnail download server for production.
   static const char kBaseThumbnailUrlForProduction[];
 
   // Returns a URL to invoke "About: get" method.
   GURL GetAboutGetUrl() const;
@@ skipping 84 matching lines @@
                        int height,
                        bool crop) const;
 
   // Generates a URL for batch upload.
   GURL GetBatchUploadUrl() const;
 
  private:
   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.
 };
 
 }  // namespace google_apis
 
 #endif  // GOOGLE_APIS_DRIVE_DRIVE_API_URL_GENERATOR_H_