PlzNavigate: Invoke didFailProvisionalLoad() in the renderer when a navigation request is cancelled…

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

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011 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) 2008 Alp Toker <alp@atoker.com>
  * Copyright (C) Research In Motion Limited 2009. All rights reserved.
  * Copyright (C) 2011 Kris Jordan <krisjordan@gmail.com>
  * Copyright (C) 2011 Google Inc. All rights reserved.
@@ skipping 1213 matching lines @@
       Document::NoDismissal)
     return;
 
   // If this method is called from within this method, infinite recursion can
   // occur (3442218). Avoid this.
   if (m_inStopAllLoaders)
     return;
 
   m_inStopAllLoaders = true;
 
+  if (m_isNavigationHandledByClient && client()) {

[Nate Chapin, 2017/02/09 00:10:07]

I'm ~99% confident client() should be guaranteed non-null at this point.

[ananta, 2017/02/09 00:22:46]

Thanks. Removed the null check

+    client()->dispatchDidFailProvisionalLoad(
+        ResourceError::cancelledError(String()), StandardCommit);
+  }
+
   m_isNavigationHandledByClient = false;
 
   for (Frame* child = m_frame->tree().firstChild(); child;
        child = child->tree().nextSibling()) {
     if (child->isLocalFrame())
       toLocalFrame(child)->loader().stopAllLoaders();
   }
 
   m_frame->document()->suppressLoadEvent();
   if (m_documentLoader)
@@ skipping 753 matching lines @@
       frameLoadRequest.clientRedirect());
 
   loader->setLoadType(loadType);
   loader->setNavigationType(navigationType);
   loader->setReplacesCurrentHistoryItem(loadType ==
                                         FrameLoadTypeReplaceCurrentItem);
   return loader;
 }
 
 }  // namespace blink