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

Unified Diff: chrome/browser/printing/cloud_print/privet_confirm_api_flow.h

Issue 2742763002: Clean up Cloud Print API flow code. (Closed)
Patch Set: Created 3 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
Index: chrome/browser/printing/cloud_print/privet_confirm_api_flow.h
diff --git a/chrome/browser/printing/cloud_print/privet_confirm_api_flow.h b/chrome/browser/printing/cloud_print/privet_confirm_api_flow.h
index dc52e4527038210275bb8dccff9856648d680c19..fc2e8dc1325685d00ae9184da49d543572981dd6 100644
--- a/chrome/browser/printing/cloud_print/privet_confirm_api_flow.h
+++ b/chrome/browser/printing/cloud_print/privet_confirm_api_flow.h
@@ -9,21 +9,20 @@
#include "base/values.h"
#include "chrome/browser/printing/cloud_print/gcd_api_flow.h"
-#include "net/url_request/url_request_context_getter.h"
namespace cloud_print {
// API call flow for server-side communication with CloudPrint for registration.
class PrivetConfirmApiCallFlow : public CloudPrintApiFlowRequest {
public:
- typedef base::Callback<void(GCDApiFlow::Status /*success*/)> ResponseCallback;
+ using ResponseCallback = base::Callback<void(GCDApiFlow::Status)>;
// Create an OAuth2-based confirmation
PrivetConfirmApiCallFlow(const std::string& token,
const ResponseCallback& callback);
-
~PrivetConfirmApiCallFlow() override;
+ // CloudPrintApiFlowRequest implementation:
void OnGCDApiFlowError(GCDApiFlow::Status status) override;
void OnGCDApiFlowComplete(const base::DictionaryValue& value) override;
net::URLFetcher::RequestType GetRequestType() override;
@@ -32,7 +31,7 @@ class PrivetConfirmApiCallFlow : public CloudPrintApiFlowRequest {
private:
ResponseCallback callback_;
- std::string token_;
+ const std::string token_;
};
} // namespace cloud_print

Powered by Google App Engine
This is Rietveld 408576698