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

Unified Diff: components/web_resource/web_resource_service.h

Issue 2717473002: Network traffic annotation added to web_resource_service. (Closed)
Patch Set: Annotation updated. 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
« no previous file with comments | « chrome/browser/plugins/plugins_resource_service.cc ('k') | components/web_resource/web_resource_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/web_resource/web_resource_service.h
diff --git a/components/web_resource/web_resource_service.h b/components/web_resource/web_resource_service.h
index d29ab053472070ec7f88b9cc297a8f9915ffabc1..7dab6ab9a75f4555f71de46acf8fdc6cfc555a75 100644
--- a/components/web_resource/web_resource_service.h
+++ b/components/web_resource/web_resource_service.h
@@ -15,6 +15,7 @@
#include "base/memory/ref_counted.h"
#include "base/memory/weak_ptr.h"
#include "components/web_resource/resource_request_allowed_notifier.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
#include "net/url_request/url_fetcher_delegate.h"
#include "url/gurl.h"
@@ -47,15 +48,17 @@ class WebResourceService
// Creates a new WebResourceService.
// If |application_locale| is not empty, it will be appended as a locale
// parameter to the resource URL.
- WebResourceService(PrefService* prefs,
- const GURL& web_resource_server,
- const std::string& application_locale, // May be empty
- const char* last_update_time_pref_name,
- int start_fetch_delay_ms,
- int cache_update_delay_ms,
- net::URLRequestContextGetter* request_context,
- const char* disable_network_switch,
- const ParseJSONCallback& parse_json_callback);
+ WebResourceService(
+ PrefService* prefs,
+ const GURL& web_resource_server,
+ const std::string& application_locale, // May be empty
+ const char* last_update_time_pref_name,
+ int start_fetch_delay_ms,
+ int cache_update_delay_ms,
+ net::URLRequestContextGetter* request_context,
+ const char* disable_network_switch,
+ const ParseJSONCallback& parse_json_callback,
+ const net::NetworkTrafficAnnotationTag& traffic_annotation);
~WebResourceService() override;
@@ -139,6 +142,9 @@ class WebResourceService
// Callback used to parse JSON.
ParseJSONCallback parse_json_callback_;
+ // Network traffic annotation for initialization of URLFetcher.
+ const net::NetworkTrafficAnnotationTag traffic_annotation_;
+
// So that we can delay our start so as not to affect start-up time; also,
// so that we can schedule future cache updates.
base::WeakPtrFactory<WebResourceService> weak_ptr_factory_;
« no previous file with comments | « chrome/browser/plugins/plugins_resource_service.cc ('k') | components/web_resource/web_resource_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698