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

Unified Diff: chrome/browser/search/search.cc

Issue 2560023002: [Merge to M56] Prevent the pre-session-restore log spam (Closed)
Patch Set: 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
« no previous file with comments | « chrome/browser/chromeos/login/signin/merge_session_throttling_utils.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/search/search.cc
diff --git a/chrome/browser/search/search.cc b/chrome/browser/search/search.cc
index 44f7ccdc0ec554e2877ebe8f5ab5cbd7c24f354a..479d095524a926ed10138f895be3decaf15cdd40 100644
--- a/chrome/browser/search/search.cc
+++ b/chrome/browser/search/search.cc
@@ -182,8 +182,8 @@ bool ShouldShowLocalNewTab(const GURL& url, Profile* profile) {
// On Chrome OS, if the session hasn't merged yet, we need to avoid loading
// the remote NTP because that will trigger showing the merge session throttle
// interstitial page, which can show for 5+ seconds. crbug.com/591530.
- if (merge_session_throttling_utils::ShouldDelayRequestForProfile(profile) &&
- merge_session_throttling_utils::ShouldDelayUrl(url)) {
+ if (merge_session_throttling_utils::ShouldDelayUrl(url) &&
+ merge_session_throttling_utils::IsSessionRestorePending(profile)) {
return true;
}
#endif // defined(OS_CHROMEOS)
« no previous file with comments | « chrome/browser/chromeos/login/signin/merge_session_throttling_utils.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698