Move most of FrameLoader::CheckCompleted() to Document

third_party/WebKit/Source/core/loader/NavigationScheduler.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2009 Adam Barth. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
@@ skipping 489 matching lines @@
   if (!frame_->Loader().StateMachine()->CommittedFirstRealDocumentLoad() &&
       frame_->Loader().ProvisionalDocumentLoader() &&
       frame_->Loader().ProvisionalDocumentLoader()->DidStart()) {
     frame_->Loader().StopAllLoaders();
     if (!frame_->GetPage())
       return;
   }
 
   Cancel();
   redirect_ = redirect;
+  if (redirect_->IsLocationChange())
+    frame_->GetDocument()->SuppressLoadEvent();

[Nate Chapin, 2017/04/11 23:25:50]

Instead of the early-exit in Document::ImplicitClose().

   StartTimer();
 }
 
 void NavigationScheduler::StartTimer() {
   if (!redirect_)
     return;
 
   DCHECK(frame_->GetPage());
   if (navigate_task_handle_.IsActive())
     return;
@@ skipping 24 matching lines @@
   navigate_task_handle_.Cancel();
   redirect_.Clear();
 }
 
 DEFINE_TRACE(NavigationScheduler) {
   visitor->Trace(frame_);
   visitor->Trace(redirect_);
 }
 
 }  // namespace blink