Enable CSP on more WebUI pages

content/public/browser/url_data_source.h

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_BROWSER_URL_DATA_SOURCE_H_
 #define CONTENT_PUBLIC_BROWSER_URL_DATA_SOURCE_H_
 
 #include <string>
 
 #include "base/callback.h"
@@ skipping 77 matching lines @@
 
   // If you are overriding this, then you have a bug.
   // It is not acceptable to disable content-security-policy on chrome:// pages
   // to permit functionality excluded by CSP, such as inline script.
   // Instead, you must go back and change your WebUI page so that it is
   // compliant with the policy. This typically involves ensuring that all script
   // is delivered through the data manager backend. Talk to tsepez for more
   // info.
   virtual bool ShouldAddContentSecurityPolicy() const;
 
   // It is OK to override the following two methods to a custom CSP directive

[Avi (use Gerrit), 2016/05/24 23:15:07]

three methods?

[wychen, 2016/05/25 12:55:08]

Reordered.

   // thereby slightly reducing the protection applied to the page.
 
+  // By default, "scrip-src chrome://resources 'self' 'unsafe-eval';" is added

[Avi (use Gerrit), 2016/05/24 23:15:07]

scrip -> script ?

[wychen, 2016/05/25 12:55:08]

*shame cube* Thanks!

+  // to CSP. Override to change this.
+  virtual std::string GetContentSecurityPolicyScriptSrc() const;
   // By default, "object-src 'none';" is added to CSP. Override to change this.
   virtual std::string GetContentSecurityPolicyObjectSrc() const;
   // By default, "frame-src 'none';" is added to CSP. Override to change this.
   virtual std::string GetContentSecurityPolicyFrameSrc() const;
 
   // By default, the "X-Frame-Options: DENY" header is sent. To stop this from
   // happening, return false. It is OK to return false as needed.
   virtual bool ShouldDenyXFrameOptions() const;
 
   // By default, only chrome: and chrome-devtools: requests are allowed.
@@ skipping 23 matching lines @@
   // Gives the source an opportunity to rewrite |path| to incorporate extra
   // information from the URLRequest prior to serving.
   virtual void WillServiceRequest(
       const net::URLRequest* request,
       std::string* path) const {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_URL_DATA_SOURCE_H_