google_apis: Change protected data members to private

chrome/browser/google_apis/base_operations.h

 // 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.
 //
 // This file provides base classes used to implement operations for Google APIs.
 
 #ifndef CHROME_BROWSER_GOOGLE_APIS_BASE_OPERATIONS_H_
 #define CHROME_BROWSER_GOOGLE_APIS_BASE_OPERATIONS_H_
 
 #include <string>
@@ skipping 119 matching lines @@
   // an user operation. Must be implemented by a derived class.
   virtual void RunCallbackOnPrematureFailure(GDataErrorCode code) = 0;
 
   // Invoked when ProcessURLFetchResults() is completed.
   void OnProcessURLFetchResultsComplete(bool result);
 
   // Returns an appropriate GDataErrorCode based on the HTTP response code and
   // the status of the URLFetcher.
   static GDataErrorCode GetErrorCode(const net::URLFetcher* source);
 
-  // The following members are used by DownloadFileOperation.
-  // TODO(satorux): Make them private.
-  bool save_temp_file_;
-  FilePath output_file_path_;
+  // By default, no temporary file will be saved. Derived classes can set
+  // this to true in their constructors, if they want to save the downloaded
+  // content to a temporary file.
+  void set_save_temp_file(bool save_temp_file) {
+    save_temp_file_ = save_temp_file;
+  }
+
+  // By default, no file will be saved. Derived classes can set an output
+  // file path in their constructors, if they want to save the donwloaded
+  // content to a file at a specific path.
+  void set_output_file_path(const FilePath& output_file_path) {
+    output_file_path_ = output_file_path;
+  }
 
  private:
   // OperationRegistry::Operation overrides.
   virtual void DoCancel() OVERRIDE;
 
   // URLFetcherDelegate overrides.
   virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE;
 
   // AuthenticatedOperationInterface overrides.
   virtual void OnAuthFailed(GDataErrorCode code) OVERRIDE;
 
   net::URLRequestContextGetter* url_request_context_getter_;
   ReAuthenticateCallback re_authenticate_callback_;
   int re_authenticate_count_;
   scoped_ptr<net::URLFetcher> url_fetcher_;
   bool started_;
 
+  bool save_temp_file_;
+  FilePath output_file_path_;
+
   // WeakPtrFactory bound to the UI thread.
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate its weak pointers before any other members are destroyed.
   base::WeakPtrFactory<UrlFetchOperationBase> weak_ptr_factory_;
 };
 
 //============================ EntryActionOperation ============================
 
 // Callback type for Delete/Move DocumentServiceInterface calls.
 typedef base::Callback<void(GDataErrorCode error)> EntryActionCallback;
@@ skipping 62 matching lines @@
 
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate its weak pointers before any other members are destroyed.
   base::WeakPtrFactory<GetDataOperation> weak_ptr_factory_;
   DISALLOW_COPY_AND_ASSIGN(GetDataOperation);
 };
 
 }  // namespace google_apis
 
 #endif  // CHROME_BROWSER_GOOGLE_APIS_BASE_OPERATIONS_H_