Feature Policy Onion Soup (1)
STEP 1 (Patch 1/2):
- Moved source files content/common/feature_policy/feature_policy.cc, .h
into third_party/WebKit/common/feature_policy/
- Moved third_party/WebKit/public/platform/WebFeaturePolicyFeature.h
into third_party/WebKit/common/feature_policy/feature_policy_feature.h
- Migrated everything into blink namespace
- Fixed BUILD.gn files
- Modified DEPS files temporarily
Feature Policy Onion Soup (2):
- Rename WebFeaturePolicyFeature to FeaturePolicyFeature
https://chromium-review.googlesource.com/c/chromium/src/+/742421
STEP 3:
- Replace WebFeaturePolicy class by FeaturePolicy
- Replace reference to WebFeaturePolicy.h by feature_policy.h
STEP 4:
- Get rid of Web* types (WebVector, WebString, etc)
STEP 5:
- Use url::Origin instead of SecurityOrigin / WebSecurityOrigin
Feature Policy Onion Soup (3)
- Remove unnecessary layer in Platform.h (previously needed for type conversion)
https://chromium-review.googlesource.com/c/chromium/src/+/742601
Feature Policy Onion Soup (4)
- Moved content/common/frame_policy.* to third_party/WebKit/common/
- Moved third_party/public/web/WebSandboxFlags.h to third_party/WebKit/common/sandbox_flags.h
https://chromium-review.googlesource.com/c/chromium/src/+/749783
Bug: 561879
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_dbg_ng;master.tryserver.chromium.linux:linux_site_isolation
Change-Id: Ief3b2edce924e0d74dd3c2ee082469e6fe29d0d8
Reviewed-on: https://chromium-review.googlesource.com/726343
Commit-Queue: Luna Lu <[email protected]>
Reviewed-by: Dimitri Glazkov <[email protected]>
Reviewed-by: Kinuko Yasuda <[email protected]>
Reviewed-by: Daniel Cheng <[email protected]>
Reviewed-by: Ian Clelland <[email protected]>
Reviewed-by: Kentaro Hara <[email protected]>
Cr-Commit-Position: refs/heads/master@{#514344}diff --git a/content/renderer/render_frame_impl.h b/content/renderer/render_frame_impl.h
index cf838ad..2ddb90b1 100644
--- a/content/renderer/render_frame_impl.h
+++ b/content/renderer/render_frame_impl.h
@@ -66,8 +66,8 @@
#include "services/service_manager/public/cpp/interface_provider.h"
#include "services/service_manager/public/interfaces/connector.mojom.h"
#include "services/service_manager/public/interfaces/interface_provider.mojom.h"
+#include "third_party/WebKit/common/feature_policy/feature_policy.h"
#include "third_party/WebKit/public/platform/WebEffectiveConnectionType.h"
-#include "third_party/WebKit/public/platform/WebFeaturePolicy.h"
#include "third_party/WebKit/public/platform/WebFocusType.h"
#include "third_party/WebKit/public/platform/WebLoadingBehaviorFlag.h"
#include "third_party/WebKit/public/platform/WebMediaPlayer.h"
@@ -560,7 +560,7 @@
const blink::WebString& name,
const blink::WebString& fallback_name,
blink::WebSandboxFlags sandbox_flags,
- const blink::WebParsedFeaturePolicy& container_policy,
+ const blink::ParsedFeaturePolicy& container_policy,
const blink::WebFrameOwnerProperties& frame_owner_properties) override;
void DidChangeOpener(blink::WebFrame* frame) override;
void FrameDetached(DetachType type) override;
@@ -574,9 +574,9 @@
void DidChangeFramePolicy(
blink::WebFrame* child_frame,
blink::WebSandboxFlags flags,
- const blink::WebParsedFeaturePolicy& container_policy) override;
+ const blink::ParsedFeaturePolicy& container_policy) override;
void DidSetFeaturePolicyHeader(
- const blink::WebParsedFeaturePolicy& parsed_header) override;
+ const blink::ParsedFeaturePolicy& parsed_header) override;
void DidAddContentSecurityPolicies(
const blink::WebVector<blink::WebContentSecurityPolicy>&) override;
void DidChangeFrameOwnerProperties(
