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

Side by Side Diff: content/common/frame_owner_properties.cc

Issue 2378643002: Propagating csp attribute changes on frameOwnerPropertiesChanged() (Closed)
Patch Set: rebasing on master Created 4 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/common/frame_owner_properties.h" 5 #include "content/common/frame_owner_properties.h"
6 6
7 namespace content { 7 namespace content {
8 8
9 FrameOwnerProperties::FrameOwnerProperties() 9 FrameOwnerProperties::FrameOwnerProperties()
10 : scrolling_mode(blink::WebFrameOwnerProperties::ScrollingMode::Auto), 10 : scrolling_mode(blink::WebFrameOwnerProperties::ScrollingMode::Auto),
11 margin_width(-1), 11 margin_width(-1),
12 margin_height(-1), 12 margin_height(-1),
13 allow_fullscreen(false) {} 13 allow_fullscreen(false) {}
14 14
15 FrameOwnerProperties::FrameOwnerProperties(const FrameOwnerProperties& other) = 15 FrameOwnerProperties::FrameOwnerProperties(const FrameOwnerProperties& other) =
16 default; 16 default;
17 17
18 FrameOwnerProperties::FrameOwnerProperties( 18 FrameOwnerProperties::FrameOwnerProperties(
19 const blink::WebFrameOwnerProperties& web_frame_owner_properties) 19 const blink::WebFrameOwnerProperties& web_frame_owner_properties)
20 : scrolling_mode(web_frame_owner_properties.scrollingMode), 20 : scrolling_mode(web_frame_owner_properties.scrollingMode),
21 margin_width(web_frame_owner_properties.marginWidth), 21 margin_width(web_frame_owner_properties.marginWidth),
22 margin_height(web_frame_owner_properties.marginHeight), 22 margin_height(web_frame_owner_properties.marginHeight),
23 allow_fullscreen(web_frame_owner_properties.allowFullscreen), 23 allow_fullscreen(web_frame_owner_properties.allowFullscreen),
24 required_csp(web_frame_owner_properties.requiredCsp.utf8()),
24 delegated_permissions( 25 delegated_permissions(
25 web_frame_owner_properties.delegatedPermissions.begin(), 26 web_frame_owner_properties.delegatedPermissions.begin(),
26 web_frame_owner_properties.delegatedPermissions.end()) {} 27 web_frame_owner_properties.delegatedPermissions.end()) {}
27 28
28 FrameOwnerProperties::~FrameOwnerProperties() {} 29 FrameOwnerProperties::~FrameOwnerProperties() {}
29 30
30 blink::WebFrameOwnerProperties FrameOwnerProperties::ToWebFrameOwnerProperties() 31 blink::WebFrameOwnerProperties FrameOwnerProperties::ToWebFrameOwnerProperties()
31 const { 32 const {
32 blink::WebFrameOwnerProperties result; 33 blink::WebFrameOwnerProperties result;
33 34
34 result.scrollingMode = scrolling_mode; 35 result.scrollingMode = scrolling_mode;
35 result.marginWidth = margin_width; 36 result.marginWidth = margin_width;
36 result.marginHeight = margin_height; 37 result.marginHeight = margin_height;
37 result.allowFullscreen = allow_fullscreen; 38 result.allowFullscreen = allow_fullscreen;
39 result.requiredCsp = blink::WebString::fromUTF8(required_csp);
38 result.delegatedPermissions = 40 result.delegatedPermissions =
39 blink::WebVector<blink::WebPermissionType>(delegated_permissions); 41 blink::WebVector<blink::WebPermissionType>(delegated_permissions);
40 42
41 return result; 43 return result;
42 } 44 }
43 45
44 bool FrameOwnerProperties::operator==(const FrameOwnerProperties& other) const { 46 bool FrameOwnerProperties::operator==(const FrameOwnerProperties& other) const {
45 return scrolling_mode == other.scrolling_mode && 47 return scrolling_mode == other.scrolling_mode &&
46 margin_width == other.margin_width && 48 margin_width == other.margin_width &&
47 margin_height == other.margin_height && 49 margin_height == other.margin_height &&
48 allow_fullscreen == other.allow_fullscreen && 50 allow_fullscreen == other.allow_fullscreen &&
51 required_csp == other.required_csp &&
49 std::equal(delegated_permissions.begin(), delegated_permissions.end(), 52 std::equal(delegated_permissions.begin(), delegated_permissions.end(),
50 other.delegated_permissions.begin()); 53 other.delegated_permissions.begin());
51 } 54 }
52 55
53 } // namespace content 56 } // namespace content
OLDNEW
« no previous file with comments | « content/common/frame_owner_properties.h ('k') | content/test/data/frame_owner_properties_csp.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698