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

Unified Diff: chrome/browser/safe_browsing/two_phase_uploader.h

Issue 2697193003: Network traffic annotation added to safe_browsing (Closed)
Patch Set: Comments addressed. 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/safe_browsing/two_phase_uploader.h
diff --git a/chrome/browser/safe_browsing/two_phase_uploader.h b/chrome/browser/safe_browsing/two_phase_uploader.h
index 123250033367a49b301e1da6adc90895f92ccb65..8c0519060ea7e6e6bbc24b31bc4914973ec73ae7 100644
--- a/chrome/browser/safe_browsing/two_phase_uploader.h
+++ b/chrome/browser/safe_browsing/two_phase_uploader.h
@@ -12,6 +12,7 @@
#include "base/callback.h"
#include "base/files/file_path.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
#include "net/url_request/url_request_context_getter.h"
#include "url/gurl.h"
@@ -69,7 +70,8 @@ class TwoPhaseUploader {
const std::string& metadata,
const base::FilePath& file_path,
const ProgressCallback& progress_callback,
- const FinishCallback& finish_callback);
+ const FinishCallback& finish_callback,
+ const net::NetworkTrafficAnnotationTag& traffic_annotation);
// Makes the passed |factory| the factory used to instantiate
// a TwoPhaseUploader. Useful for tests.
@@ -97,7 +99,8 @@ class TwoPhaseUploaderFactory {
const std::string& metadata,
const base::FilePath& file_path,
const TwoPhaseUploader::ProgressCallback& progress_callback,
- const TwoPhaseUploader::FinishCallback& finish_callback) = 0;
+ const TwoPhaseUploader::FinishCallback& finish_callback,
+ const net::NetworkTrafficAnnotationTag& traffic_annotation) = 0;
};
#endif // CHROME_BROWSER_SAFE_BROWSING_TWO_PHASE_UPLOADER_H_
« no previous file with comments | « chrome/browser/safe_browsing/threat_details_cache.cc ('k') | chrome/browser/safe_browsing/two_phase_uploader.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698