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

Side by Side Diff: chrome/browser/feedback/feedback_uploader_delegate.h

Issue 133193004: Revert of Cache feedback reports to disk in case of send failure. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_FEEDBACK_FEEDBACK_UPLOADER_DELEGATE_H_ 5 #ifndef CHROME_BROWSER_FEEDBACK_FEEDBACK_UPLOADER_DELEGATE_H_
6 #define CHROME_BROWSER_FEEDBACK_FEEDBACK_UPLOADER_DELEGATE_H_ 6 #define CHROME_BROWSER_FEEDBACK_FEEDBACK_UPLOADER_DELEGATE_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
11 #include "base/callback.h" 11 #include "base/callback.h"
12 #include "base/memory/scoped_ptr.h"
12 #include "net/url_request/url_fetcher_delegate.h" 13 #include "net/url_request/url_fetcher_delegate.h"
13 14
14 namespace feedback { 15 namespace feedback {
15 16
16 typedef base::Callback<void(const std::string&)> ReportDataCallback; 17 typedef base::Callback<void(scoped_ptr<std::string>)> ReportDataCallback;
17 18
18 // FeedbackUploaderDelegate is a simple http uploader for a feedback report. On 19 // FeedbackUploaderDelegate is a simple http uploader for a feedback report. On
19 // succes or failure, it deletes itself, but on failure it also notifies the 20 // succes or failure, it deletes itself, but on failure it also notifies the
20 // error callback specified when constructing the class instance. 21 // error callback specified when constructing the class instance.
21 class FeedbackUploaderDelegate : public net::URLFetcherDelegate { 22 class FeedbackUploaderDelegate : public net::URLFetcherDelegate {
22 public: 23 public:
23 FeedbackUploaderDelegate(const std::string& post_body, 24 FeedbackUploaderDelegate(scoped_ptr<std::string> post_body,
24 const base::Closure& success_callback, 25 const base::Closure& success_callback,
25 const ReportDataCallback& error_callback); 26 const ReportDataCallback& error_callback);
26 virtual ~FeedbackUploaderDelegate(); 27 virtual ~FeedbackUploaderDelegate();
27 28
28 private: 29 private:
29 // Overridden from net::URLFetcherDelegate. 30 // Overridden from net::URLFetcherDelegate.
30 virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE; 31 virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE;
31 32
32 std::string post_body_; 33 scoped_ptr<std::string> post_body_;
33 base::Closure success_callback_; 34 base::Closure success_callback_;
34 ReportDataCallback error_callback_; 35 ReportDataCallback error_callback_;
35 36
36 DISALLOW_COPY_AND_ASSIGN(FeedbackUploaderDelegate); 37 DISALLOW_COPY_AND_ASSIGN(FeedbackUploaderDelegate);
37 }; 38 };
38 39
39 } // namespace feedback 40 } // namespace feedback
40 41
41 #endif // CHROME_BROWSER_FEEDBACK_FEEDBACK_UPLOADER_DELEGATE_H_ 42 #endif // CHROME_BROWSER_FEEDBACK_FEEDBACK_UPLOADER_DELEGATE_H_
OLDNEW
« no previous file with comments | « chrome/browser/feedback/feedback_uploader.cc ('k') | chrome/browser/feedback/feedback_uploader_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698