CHECK that URLRequestJobs are not orphaned while blocked by extensions.

extensions/browser/api/web_request/web_request_api.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 "extensions/browser/api/web_request/web_request_api.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
+#include "base/debug/alias.h"
 #include "base/json/json_writer.h"
 #include "base/lazy_instance.h"
 #include "base/macros.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/time/time.h"
 #include "base/values.h"
 #include "content/public/browser/browser_message_filter.h"
@@ skipping 1198 matching lines @@
     void* browser_context,
     const net::URLRequest* request) {
   ClearPendingCallbacks(request);
 
   signaled_requests_.erase(request->identifier());
 
   request_time_tracker_->LogRequestEndTime(request->identifier(),
                                            base::Time::Now());
 }
 
+void ExtensionWebRequestEventRouter::OnURLRequestJobOrphaned(
+    void* browser_context,
+    const net::URLRequest* request) {
+  // See https://crbug.com/289715. While a URLRequest is blocking on an
+  // extension, it may not orphan jobs unless OnURLRequestDestroyed is called
+  // first.
+  //
+  // TODO(davidben): Remove this when the crash has been diagnosed.
+  char url_buf[128];
+  base::strlcpy(url_buf, request->url().spec().c_str(), arraysize(url_buf));
+  base::debug::Alias(url_buf);
+  CHECK_EQ(0u, blocked_requests_.count(request->identifier()));
+}
+
 void ExtensionWebRequestEventRouter::ClearPendingCallbacks(
     const net::URLRequest* request) {
   blocked_requests_.erase(request->identifier());
 }
 
 bool ExtensionWebRequestEventRouter::DispatchEvent(
     void* browser_context,
     net::URLRequest* request,
     const std::vector<const EventListener*>& listeners,
     const base::ListValue& args) {
@@ skipping 1243 matching lines @@
   // Continue gracefully.
   RunSync();
 }
 
 bool WebRequestHandlerBehaviorChangedFunction::RunSync() {
   helpers::ClearCacheOnNavigation();
   return true;
 }
 
 }  // namespace extensions