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

Unified Diff: chrome/browser/supervised_user/supervised_user_navigation_observer.cc

Issue 2949023002: [WIP] Fix history.replaceState() not propagating back favicons (Closed)
Patch Set: Focus on history.replaceState() Created 3 years, 5 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
Index: chrome/browser/supervised_user/supervised_user_navigation_observer.cc
diff --git a/chrome/browser/supervised_user/supervised_user_navigation_observer.cc b/chrome/browser/supervised_user/supervised_user_navigation_observer.cc
index 01bad11617023f3e40dccd057c42c29adc193c0e..1ffd00f83a32f70b59f7a28ffd1d8cb1147c8c33 100644
--- a/chrome/browser/supervised_user/supervised_user_navigation_observer.cc
+++ b/chrome/browser/supervised_user/supervised_user_navigation_observer.cc
@@ -93,7 +93,7 @@ void SupervisedUserNavigationObserver::OnRequestBlockedInternal(
history::HistoryAddPageArgs add_page_args(
url, timestamp, history::ContextIDForWebContents(web_contents()), 0, url,
history::RedirectList(), ui::PAGE_TRANSITION_BLOCKED,
- history::SOURCE_BROWSED, false, true);
+ history::SOURCE_BROWSED, base::nullopt, true);
// Add the entry to the history database.
Profile* profile =

Powered by Google App Engine
This is Rietveld 408576698