 Chromium Code Reviews
 Chromium Code Reviews Issue 2797813002:
  Replicate feature policy container policies.  (Closed)
    
  
    Issue 2797813002:
  Replicate feature policy container policies.  (Closed) 
  | Index: third_party/WebKit/Source/core/frame/FrameOwner.h | 
| diff --git a/third_party/WebKit/Source/core/frame/FrameOwner.h b/third_party/WebKit/Source/core/frame/FrameOwner.h | 
| index ededeaa03d4436223fba31924bce73e97764cc20..f6f1a5a43bd4264397cdbddc17f3e892a1e4bc68 100644 | 
| --- a/third_party/WebKit/Source/core/frame/FrameOwner.h | 
| +++ b/third_party/WebKit/Source/core/frame/FrameOwner.h | 
| @@ -51,6 +51,7 @@ class CORE_EXPORT FrameOwner : public GarbageCollectedMixin { | 
| virtual bool IsDisplayNone() const = 0; | 
| virtual AtomicString Csp() const = 0; | 
| virtual const WebVector<WebFeaturePolicyFeature>& AllowedFeatures() const = 0; | 
| + virtual const WebParsedFeaturePolicy& ContainerPolicy() const = 0; | 
| }; | 
| // TODO(dcheng): This class is an internal implementation detail of provisional | 
| @@ -88,6 +89,10 @@ class CORE_EXPORT DummyFrameOwner | 
| DEFINE_STATIC_LOCAL(WebVector<WebFeaturePolicyFeature>, features, ()); | 
| return features; | 
| } | 
| + const WebParsedFeaturePolicy& ContainerPolicy() const override { | 
| + DEFINE_STATIC_LOCAL(WebParsedFeaturePolicy, container_policy, ()); | 
| + return container_policy; | 
| 
dcheng
2017/04/18 16:26:37
Nit: it's unclear to me why we need a static local
 
iclelland
2017/04/18 19:19:44
It needs to return a reference (the other non-test
 | 
| + } | 
| private: | 
| // Intentionally private to prevent redundant checks when the type is |