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

Unified Diff: Source/core/loader/NavigationScheduler.cpp

Issue 561813003: Prepare blink to unify definitions of load completion (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 3 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: Source/core/loader/NavigationScheduler.cpp
diff --git a/Source/core/loader/NavigationScheduler.cpp b/Source/core/loader/NavigationScheduler.cpp
index 53211a9e063450e28acd0eda902345cbdc043602..22c3f2f986f93235cb264d624abc55442f950948 100644
--- a/Source/core/loader/NavigationScheduler.cpp
+++ b/Source/core/loader/NavigationScheduler.cpp
@@ -153,7 +153,7 @@ public:
class ScheduledLocationChange FINAL : public ScheduledURLNavigation {
public:
ScheduledLocationChange(Document* originDocument, const String& url, const Referrer& referrer, bool lockBackForwardList)
- : ScheduledURLNavigation(0.0, originDocument, url, referrer, lockBackForwardList, true) { }
+ : ScheduledURLNavigation(0.0, originDocument, url, referrer, lockBackForwardList, !protocolIsJavaScript(url)) { }
Nate Chapin 2014/09/15 23:39:35 A pending navigation to a JS url shouldn't stop ht
};
class ScheduledReload FINAL : public ScheduledNavigation {

Powered by Google App Engine
This is Rietveld 408576698