Commit 8758b479 authored by treib's avatar treib Committed by Commit bot

Local NTP: Allow iframes from *.google.com

if OneGoogleBarOnLocalNtp is enabled.
This achieves two things:
- More robust if things change server-side
- Allows staging instances

BUG=583292

Review-Url: https://codereview.chromium.org/2856133002
Cr-Commit-Position: refs/heads/master@{#468933}
parent ada1e2d7
...@@ -431,11 +431,8 @@ std::string LocalNtpSource::GetContentSecurityPolicyChildSrc() const { ...@@ -431,11 +431,8 @@ std::string LocalNtpSource::GetContentSecurityPolicyChildSrc() const {
if (one_google_bar_service_) { if (one_google_bar_service_) {
// Allow embedding of the most visited iframe, as well as the account // Allow embedding of the most visited iframe, as well as the account
// switcher and the notifications dropdown from the One Google Bar. // switcher and the notifications dropdown from the One Google Bar.
// TODO(treib): Figure out a way to also allow staging instances. return base::StringPrintf("child-src %s https://*.google.com/;",
return base::StringPrintf( chrome::kChromeSearchMostVisitedUrl);
"child-src %s https://accounts.google.com/ https://docs.google.com "
"https://notifications.google.com;",
chrome::kChromeSearchMostVisitedUrl);
} }
// Allow embedding of the most visited iframe. // Allow embedding of the most visited iframe.
return base::StringPrintf("child-src %s;", return base::StringPrintf("child-src %s;",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment