Fix sending multiple feedback reports within short durations of each other

chrome/browser/extensions/api/feedback_private/feedback_service.h

 // Copyright 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 CHROME_BROWSER_EXTENSIONS_API_FEEDBACK_PRIVATE_FEEDBACK_SERVICE_H_
 #define CHROME_BROWSER_EXTENSIONS_API_FEEDBACK_PRIVATE_FEEDBACK_SERVICE_H_
 
 #include <stdint.h>
 
+#include <map>
 #include <vector>
+
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/memory/linked_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/extensions/blob_reader.h"
 #include "chrome/browser/feedback/system_logs/scrubbed_system_logs_fetcher.h"
 #include "chrome/common/extensions/api/feedback_private.h"
 #include "components/feedback/feedback_data.h"
 
 class Profile;
 
 using extensions::api::feedback_private::SystemInformation;
 
 namespace extensions {
 
 typedef std::vector<linked_ptr<SystemInformation> > SystemInformationList;
 
 // The feedback service provides the ability to gather the various pieces of
 // data needed to send a feedback report and then send the report once all
 // the pieces are available.
 class FeedbackService : public base::SupportsWeakPtr<FeedbackService> {
  public:
+  typedef int64_t RequestId;

[xiyuan, 2016/03/14 23:47:28]

nit: new style is to use "using"

using RequestId = int64;

and update existing typedef since you are here.

[afakhry, 2016/03/15 19:25:38]

Done.

   typedef base::Callback<void(bool)> SendFeedbackCallback;
   typedef base::Callback<void(const SystemInformationList& sys_info)>
       GetSystemInformationCallback;
 
   FeedbackService();
   virtual ~FeedbackService();
 
   // Sends a feedback report.
   void SendFeedback(Profile* profile,
                     scoped_refptr<feedback::FeedbackData> feedback_data,
                     const SendFeedbackCallback& callback);
 
   // Start to gather system information.
   // The |callback| will be invoked once the query is completed.
   void GetSystemInformation(const GetSystemInformationCallback& callback);
 
  private:
+  struct SendRequest;
+
   // Callbacks to receive blob data.
-  void AttachedFileCallback(scoped_ptr<std::string> data,
+  void AttachedFileCallback(RequestId request_id,
+                            scoped_ptr<std::string> data,
                             int64_t total_blob_length);
-  void ScreenshotCallback(scoped_ptr<std::string> data,
+  void ScreenshotCallback(RequestId request_id,
+                          scoped_ptr<std::string> data,
                           int64_t total_blob_length);
 
+  void OnSystemLogsFetchComplete(
+      RequestId request_id,
+      scoped_ptr<system_logs::SystemLogsResponse> sys_info);
+
   // Checks if we have read all the blobs we need to; signals the feedback
   // data object once all the requisite data has been populated.
-  void CompleteSendFeedback();
+  void CompleteSendFeedback(RequestId request_id);
 
-  void OnSystemLogsFetchComplete(
-      scoped_ptr<system_logs::SystemLogsResponse> sys_info);
+  std::map<RequestId, GetSystemInformationCallback>
+      system_information_callbacks_;
 
-  GetSystemInformationCallback system_information_callback_;
-  SendFeedbackCallback send_feedback_callback_;
-
-  scoped_refptr<feedback::FeedbackData> feedback_data_;
+  std::map<RequestId, SendRequest> send_feedback_requests_;
 
   DISALLOW_COPY_AND_ASSIGN(FeedbackService);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_FEEDBACK_PRIVATE_FEEDBACK_SERVICE_H_