Remove BindCSPImmediately(), it's no longer necessary

Change-Id: I770dded68d2ff6de14897b5a1666f5931fcfc450
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2224830
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Auto-Submit: Nate Chapin <japhet@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#773811}

third_party/blink/renderer/core/execution_context/security_context.cc

@@ -75,8 +75,7 @@ SecurityContext::SecurityContext(const SecurityContextInit& init,
       context_type_for_asserts_(context_type),
       agent_(init.GetAgent()),
       secure_context_mode_(init.GetSecureContextMode()),
-      origin_trial_context_(init.GetOriginTrialContext()),
-      bind_csp_immediately_(init.BindCSPImmediately()) {}
+      origin_trial_context_(init.GetOriginTrialContext()) {}
 
 void SecurityContext::Trace(Visitor* visitor) const {
   visitor->Trace(content_security_policy_);

third_party/blink/renderer/core/execution_context/security_context.h

@@ -191,8 +191,6 @@ class CORE_EXPORT SecurityContext {
     secure_context_mode_ = mode;
   }
 
-  bool BindCSPImmediately() const { return bind_csp_immediately_; }
-
  protected:
   network::mojom::blink::WebSandboxFlags sandbox_flags_;
   scoped_refptr<SecurityOrigin> security_origin_;
@@ -212,7 +210,6 @@ class CORE_EXPORT SecurityContext {
   Member<Agent> agent_;
   SecureContextMode secure_context_mode_;
   Member<OriginTrialContext> origin_trial_context_;
-  bool bind_csp_immediately_ = false;
   DISALLOW_COPY_AND_ASSIGN(SecurityContext);
 };
 

third_party/blink/renderer/core/execution_context/security_context_init.cc

@@ -175,7 +175,6 @@ void SecurityContextInit::InitializeContentSecurityPolicy(
   // not specify a CSP to be used. Ideally they should be forced to explicitly
   // choose one.
   csp_ = MakeGarbageCollected<ContentSecurityPolicy>();
-  bind_csp_immediately_ = true;
 }
 
 void SecurityContextInit::InitializeSandboxFlags(

third_party/blink/renderer/core/execution_context/security_context_init.h

@@ -79,8 +79,6 @@ class CORE_EXPORT SecurityContextInit : public FeaturePolicyParserDelegate {
 
   void ApplyPendingDataToDocument(Document&) const;
 
-  bool BindCSPImmediately() const { return bind_csp_immediately_; }
-
  private:
   void InitializeContentSecurityPolicy(const DocumentInit&);
   void InitializeOrigin(const DocumentInit&);
@@ -109,7 +107,6 @@ class CORE_EXPORT SecurityContextInit : public FeaturePolicyParserDelegate {
   Agent* agent_ = nullptr;
   HashSet<mojom::blink::FeaturePolicyFeature> parsed_feature_policies_;
   HashSet<mojom::WebFeature> feature_count_;
-  bool bind_csp_immediately_ = false;
   base::Optional<SecureContextMode> secure_context_mode_;
 };
 

third_party/blink/renderer/core/frame/local_dom_window.cc

@@ -547,13 +547,6 @@ Document* LocalDOMWindow::InstallNewDocument(const DocumentInit& init) {
   document_ = init.CreateDocument();
   document_->Initialize();
 
-  // The CSP delegate doesn't have access to all of the state it needs until
-  // document_ is set.
-  if (GetSecurityContext().BindCSPImmediately()) {
-    GetSecurityContext().GetContentSecurityPolicy()->BindToDelegate(
-        GetContentSecurityPolicyDelegate());
-  }
-
   if (!GetFrame())
     return document_;