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

Unified Diff: chrome/browser/history/history_utils.cc

Issue 2494633004: Remove about:srcdoc url conversion. (Closed)
Patch Set: Rebase Created 4 years 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
Index: chrome/browser/history/history_utils.cc
diff --git a/chrome/browser/history/history_utils.cc b/chrome/browser/history/history_utils.cc
index 1efb0ef893413ed9b4ecb29b4616df3d6263cbdb..8bf6a91e67cd2a866125a0016f098035578df428 100644
--- a/chrome/browser/history/history_utils.cc
+++ b/chrome/browser/history/history_utils.cc
@@ -17,6 +17,7 @@ bool CanAddURLToHistory(const GURL& url) {
// by a shortcut or menu action.
// Right now, URLs like about:version are not registered in the history.
if (url.SchemeIs(url::kJavaScriptScheme) ||
+ url.SchemeIs(url::kAboutScheme) ||
url.SchemeIs(content::kChromeDevToolsScheme) ||
url.SchemeIs(content::kChromeUIScheme) ||
url.SchemeIs(content::kViewSourceScheme) ||
@@ -25,8 +26,5 @@ bool CanAddURLToHistory(const GURL& url) {
url.SchemeIs(dom_distiller::kDomDistillerScheme))
return false;
- if (url == url::kAboutBlankURL)
- return false;
-
return true;
}
« no previous file with comments | « chrome/browser/extensions/api/web_navigation/web_navigation_apitest.cc ('k') | chrome/browser/prerender/prerender_contents.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698