PlzNavigate: Do not disclose urls between cross-origin renderers.

content/common/content_security_policy/csp_context.h

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_COMMON_CONTENT_SECURITY_POLICY_CSP_CONTEXT_H_
 #define CONTENT_COMMON_CONTENT_SECURITY_POLICY_CSP_CONTEXT_H_
 
 #include <vector>
 
 #include "content/common/content_export.h"
@@ skipping 37 matching lines @@
 
   bool SelfSchemeShouldBypassCsp();
 
   void ResetContentSecurityPolicies() { policies_.clear(); }
   void AddContentSecurityPolicy(const ContentSecurityPolicy& policy) {
     policies_.push_back(policy);
   }
 
   virtual bool SchemeShouldBypassCSP(const base::StringPiece& scheme);
 
+  // For security reasons, some urls must not be disclosed. It includes the

[alexmos, 2017/05/12 01:37:20]

nit: disclosed -> disclosed cross-origin in violation reports.

[arthursonzogni, 2017/05/15 12:20:46]

Done.

+  // blocked url and the url of the initiator of the navigation. These
+  // informations are potentially transmitted between different renderer

[alexmos, 2017/05/12 01:37:20]

nit: These informations are -> This information is

[arthursonzogni, 2017/05/15 12:20:46]

Done.

+  // processes. When this function returns true, only the url's origin is
+  // displayed instead.
+  // TODO(arthursonzogni): Stop hidding sensitive parts of URLs in console

[alexmos, 2017/05/12 01:37:20]

nit: s/hidding/hiding/

[arthursonzogni, 2017/05/15 12:20:46]

Done.

+  // error message as soon as there is a way to send it to the devtool process

[alexmos, 2017/05/12 01:37:20]

nit: s/message/messages/
s/devtool/devtools/

[arthursonzogni, 2017/05/15 12:20:46]

Done.

+  // without the round trip in the renderer process.
+  // See https://crbug.com/721329
+  virtual bool ShouldProtectDataInCspViolation(const url::Origin& origin) const;
+
  private:
   bool has_self_ = false;
   std::string self_scheme_;
   CSPSource self_source_;
 
   std::vector<ContentSecurityPolicy> policies_;
 
   DISALLOW_COPY_AND_ASSIGN(CSPContext);
 };
 
@@ skipping 38 matching lines @@
 
   // Whether or not the violation happens after a redirect.
   bool after_redirect;
 
   // The source code location that triggered the blocked navigation.
   SourceLocation source_location;
 };
 
 }  // namespace content
 #endif  // CONTENT_COMMON_CONTENT_SECURITY_POLICY_CSP_CONTEXT_H_