OLD | NEW |
---|---|
(Empty) | |
1 // Copyright 2017 The Chromium Authors. All rights reserved. | |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #ifndef CONTENT_BROWSER_FRAME_HOST_CSP_CONTEXT_H_ | |
6 #define CONTENT_BROWSER_FRAME_HOST_CSP_CONTEXT_H_ | |
nasko
2017/02/11 00:01:23
Missing _IMPL_ part in the include guard.
arthursonzogni
2017/02/13 16:33:20
Done.
| |
7 | |
8 #include "content/common/content_security_policy/csp_context.h" | |
9 | |
10 namespace content { | |
11 | |
12 class FrameTreeNode; | |
13 | |
14 class CSPContextImpl : public CSPContext { | |
15 public: | |
16 CSPContextImpl(FrameTreeNode* frame_tree_node); | |
17 void LogToConsole(const std::string& message) override; | |
18 | |
19 // Inform the renderer process that a navigation has been blocked by a content | |
20 // security policy. | |
21 void ReportViolation(const CSPViolationParams& violation_params) override; | |
alexmos
2017/02/10 22:59:53
Just a heads-up that lukasza@ worked on some plumb
arthursonzogni
2017/02/13 16:33:20
Thanks! I don't know yet if a race could happens,
| |
22 | |
23 private: | |
24 bool SchemeShouldBypass(const base::StringPiece& scheme) override; | |
alexmos
2017/02/10 22:59:53
Can this be static? Doesn't look like it needs fr
arthursonzogni
2017/02/13 16:33:20
Yes, it doesn't need frame_tree_node_, but it's a
alexmos
2017/02/14 06:57:19
Acknowledged.
| |
25 | |
26 FrameTreeNode* frame_tree_node_; // Never nullptr. | |
nasko
2017/02/11 00:01:23
nit: Comments like that usually go above the membe
arthursonzogni
2017/02/13 16:33:20
Done.
| |
27 }; | |
28 } // namespace content | |
nasko
2017/02/11 00:01:23
nit: Empty line before end of namespace.
arthursonzogni
2017/02/13 16:33:20
Done.
| |
29 | |
30 #endif // CONTENT_BROWSER_FRAME_HOST_CSP_CONTEXT_H_ */ | |
nasko
2017/02/11 00:01:23
Why is there "*/" at the end?
arthursonzogni
2017/02/13 16:33:20
Oops, vim-snipmate + wrong conversion to the googl
| |
OLD | NEW |