Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5449)

Unified Diff: chrome/browser/extensions/api/feedback_private/feedback_service.h

Issue 1794513002: Fix sending multiple feedback reports within short durations of each other (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: self revision Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/extensions/api/feedback_private/feedback_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/api/feedback_private/feedback_service.h
diff --git a/chrome/browser/extensions/api/feedback_private/feedback_service.h b/chrome/browser/extensions/api/feedback_private/feedback_service.h
index 9588c534d61988e998a1891e2c48cd892218f9d0..d806c77baf6ebafc39fb8ca6703fd387a94d3de0 100644
--- a/chrome/browser/extensions/api/feedback_private/feedback_service.h
+++ b/chrome/browser/extensions/api/feedback_private/feedback_service.h
@@ -8,6 +8,7 @@
#include <stdint.h>
#include <vector>
+
#include "base/callback.h"
#include "base/macros.h"
#include "base/memory/linked_ptr.h"
@@ -23,16 +24,16 @@ using extensions::api::feedback_private::SystemInformation;
namespace extensions {
-typedef std::vector<linked_ptr<SystemInformation> > SystemInformationList;
+using SystemInformationList = std::vector<linked_ptr<SystemInformation>>;
// 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 base::Callback<void(bool)> SendFeedbackCallback;
- typedef base::Callback<void(const SystemInformationList& sys_info)>
- GetSystemInformationCallback;
+ using SendFeedbackCallback = base::Callback<void(bool)>;
+ using GetSystemInformationCallback =
+ base::Callback<void(const SystemInformationList&)>;
FeedbackService();
virtual ~FeedbackService();
@@ -48,22 +49,23 @@ class FeedbackService : public base::SupportsWeakPtr<FeedbackService> {
private:
// Callbacks to receive blob data.
- void AttachedFileCallback(scoped_ptr<std::string> data,
+ void AttachedFileCallback(scoped_refptr<feedback::FeedbackData> feedback_data,
+ const SendFeedbackCallback& callback,
+ scoped_ptr<std::string> data,
int64_t total_blob_length);
- void ScreenshotCallback(scoped_ptr<std::string> data,
+ void ScreenshotCallback(scoped_refptr<feedback::FeedbackData> feedback_data,
+ const SendFeedbackCallback& callback,
+ scoped_ptr<std::string> data,
int64_t total_blob_length);
- // 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 OnSystemLogsFetchComplete(
+ const GetSystemInformationCallback& callback,
scoped_ptr<system_logs::SystemLogsResponse> sys_info);
- GetSystemInformationCallback system_information_callback_;
- SendFeedbackCallback send_feedback_callback_;
-
- scoped_refptr<feedback::FeedbackData> feedback_data_;
+ // 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(scoped_refptr<feedback::FeedbackData> feedback_data,
+ const SendFeedbackCallback& callback);
DISALLOW_COPY_AND_ASSIGN(FeedbackService);
};
« no previous file with comments | « no previous file | chrome/browser/extensions/api/feedback_private/feedback_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698