Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1606)

Unified Diff: base/metrics/histogram.cc

Issue 7215034: Apply CSP to chrome: and about: pages (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | base/tracked_objects.h » ('j') | chrome/browser/browser_about_handler.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/metrics/histogram.cc
===================================================================
--- base/metrics/histogram.cc (revision 90144)
+++ base/metrics/histogram.cc (working copy)
@@ -1064,13 +1064,6 @@
std::string* output) {
if (!IsActive())
return;
- output->append("<html><head><title>About Histograms");
- if (!query.empty())
- output->append(" - " + query);
abarth-chromium 2011/06/24 04:45:18 I can haz XSS?
Evan Martin 2011/06/24 18:05:32 Well, given that you can't link to these pages, yo
abarth-chromium 2011/06/24 18:07:56 We've fixed the pasting attacks we know of (at lea
- output->append("</title>"
- // We'd like the following no-cache... but it doesn't work.
- // "<META HTTP-EQUIV=\"Pragma\" CONTENT=\"no-cache\">"
- "</head><body>");
Histograms snapshot;
GetSnapshot(query, &snapshot);
@@ -1080,7 +1073,6 @@
(*it)->WriteHTMLGraph(output);
output->append("<br><hr><br>");
}
- output->append("</body></html>");
}
// static
« no previous file with comments | « no previous file | base/tracked_objects.h » ('j') | chrome/browser/browser_about_handler.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698