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

Unified Diff: components/content_settings/core/browser/cookie_settings.cc

Issue 1569673002: [NOT FOR LANDING] Detailed loading traces Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 10 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 | « chrome/browser/safe_browsing/safe_browsing_database.cc ('k') | components/safe_browsing_db/prefix_set.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/content_settings/core/browser/cookie_settings.cc
diff --git a/components/content_settings/core/browser/cookie_settings.cc b/components/content_settings/core/browser/cookie_settings.cc
index 2339c86cddd95e1b8fb4ddcb39456082a69b8776..09b5a62521569ba1644e7cc53dba527d3fc594f7 100644
--- a/components/content_settings/core/browser/cookie_settings.cc
+++ b/components/content_settings/core/browser/cookie_settings.cc
@@ -4,6 +4,7 @@
#include "components/content_settings/core/browser/cookie_settings.h"
+#include "base/trace_event/trace_event.h"
#include "base/bind.h"
#include "base/logging.h"
#include "components/content_settings/core/browser/content_settings_utils.h"
@@ -57,6 +58,7 @@ ContentSetting CookieSettings::GetDefaultCookieSetting(
bool CookieSettings::IsReadingCookieAllowed(const GURL& url,
const GURL& first_party_url) const {
+ TRACE_EVENT0("toplevel", "CookieSettings::IsReadingCookieAllowed");
ContentSetting setting = GetCookieSetting(url, first_party_url, false, NULL);
return IsAllowed(setting);
}
@@ -132,6 +134,8 @@ ContentSetting CookieSettings::GetCookieSetting(const GURL& url,
const GURL& first_party_url,
bool setting_cookie,
SettingSource* source) const {
+ TRACE_EVENT0("toplevel", "CookieSettings::GetCookieSetting");
+
// Auto-allow in extensions or for WebUI embedded in a secure origin.
if (url.SchemeIsCryptographic() && first_party_url.SchemeIs(kChromeUIScheme))
return CONTENT_SETTING_ALLOW;
« no previous file with comments | « chrome/browser/safe_browsing/safe_browsing_database.cc ('k') | components/safe_browsing_db/prefix_set.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698