Change NavigationEntry to use virtual URL in error pages for blocked navigations

chrome/browser/extensions/navigation_observer.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/navigation_observer.h"
 
 #include "base/memory/ptr_util.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "content/public/browser/navigation_controller.h"
@@ skipping 51 matching lines @@
 void NavigationObserver::PromptToEnableExtensionIfNecessary(
     NavigationController* nav_controller) {
   // Bail out if we're already running a prompt.
   if (!in_progress_prompt_extension_id_.empty())
     return;
 
   NavigationEntry* nav_entry = nav_controller->GetVisibleEntry();
   if (!nav_entry)
     return;
 
-  const GURL& url = nav_entry->GetURL();
+  // With a disabled extension, the navigation is blocked, which results in
+  // an error page and NavigationEntry with "about:blank" URL. In this case,
+  // the virtual URL is the real URL the user navigated to, so if this is the
+  // case, use it instead.
+  const GURL& url = (nav_entry->GetPageType() == content::PAGE_TYPE_ERROR &&
+                     nav_entry->GetURL() == url::kAboutBlankURL &&
+                     nav_entry->GetVirtualURL().SchemeIs(kExtensionScheme))
+                        ? nav_entry->GetVirtualURL()
+                        : nav_entry->GetURL();
+
   // NOTE: We only consider chrome-extension:// urls, and deliberately don't
   // consider hosted app urls. This is because it's really annoying to visit the
   // site associated with a hosted app (like calendar.google.com or
   // drive.google.com) and have it repeatedly prompt you to re-enable an item.
   // Visiting a chrome-extension:// url is a much stronger signal, and, without
   // the item enabled, we won't show anything.
   // TODO(devlin): While true, I still wonder how useful this is. We should get
   // metrics.
   if (!url.SchemeIs(kExtensionScheme))
     return;
@@ skipping 79 matching lines @@
       in_progress_prompt_extension_id_ != extension->id()) {
     return;
   }
 
   in_progress_prompt_extension_id_ = std::string();
   in_progress_prompt_navigation_controller_ = nullptr;
   extension_install_prompt_.reset();
 }
 
 }  // namespace extensions