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

Unified Diff: remoting/protocol/http_ice_config_request.cc

Issue 2729423003: Network traffic annotation added to chromium_url_request. (Closed)
Patch Set: Annotations 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
Index: remoting/protocol/http_ice_config_request.cc
diff --git a/remoting/protocol/http_ice_config_request.cc b/remoting/protocol/http_ice_config_request.cc
index 7364341169190c08b10fbbf17aebb21d3dd8a41f..07977f888e9235f8db74eb32405a3e0020de59e0 100644
--- a/remoting/protocol/http_ice_config_request.cc
+++ b/remoting/protocol/http_ice_config_request.cc
@@ -11,6 +11,7 @@
#include "base/strings/string_util.h"
#include "base/values.h"
#include "net/base/url_util.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
#include "remoting/protocol/ice_config.h"
namespace remoting {
@@ -97,8 +98,38 @@ HttpIceConfigRequest::HttpIceConfigRequest(
UrlRequestFactory* url_request_factory,
const std::string& url)
: url_(url) {
- url_request_ =
- url_request_factory->CreateUrlRequest(UrlRequest::Type::POST, url_);
+ net::NetworkTrafficAnnotationTag traffic_annotation =
+ net::DefineNetworkTrafficAnnotation("CRD_ice_config_request", R"(
+ semantics {
+ sender: "Chrome Remote Desktop"
+ description:
+ "Request is used by Chrome Remote Desktop to fetch ICE "
+ "configuration which contains list of STUN & TURN servers and TURN "
+ "credentials."
+ trigger: "Chrome Remote Desktop usage."
+ data: "None."
+ destination: GOOGLE_OWNED_SERVICE
+ }
+ policy {
+ cookies_allowed: false/true
+ cookies_store: "..."
+ setting:
+ "This feature cannot be disabled by settings. You can block Chrome "
+ "Remote Desktop as specified here: "
+ "https://support.google.com/chrome/a/answer/2799701?hl=en"
+ chrome_policy {
+ RemoteAccessHostFirewallTraversal {
+ policy_options {mode: MANDATORY}
+ RemoteAccessHostFirewallTraversal: false
+ }
+ }
+ policy_exception_justification:
+ "Above specified policy is only applicable on the host side and "
+ "doesn't have effect in Android and iOS client apps. The product "
+ "is shipped separately from Chromium, except on Chrome OS."
+ })");
+ url_request_ = url_request_factory->CreateUrlRequest(
+ UrlRequest::Type::POST, url_, traffic_annotation);
url_request_->SetPostData("application/json", "");
}

Powered by Google App Engine
This is Rietveld 408576698