PlzNavigate: add DumpWithoutCrashing for invalid main resource request.

content/child/web_url_loader_impl.cc

 // Copyright 2014 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 "content/child/web_url_loader_impl.h"
 
 #include <stdint.h>
 
 #include <algorithm>
 #include <memory>
 #include <string>
 #include <utility>
 #include <vector>
 
 #include "base/bind.h"
+#include "base/debug/dump_without_crashing.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "base/optional.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "base/time/time.h"
 #include "build/build_config.h"
@@ skipping 579 matching lines @@
   // PlzNavigate: during navigation, the renderer should request a stream which
   // contains the body of the response. The network request has already been
   // made by the browser.
   if (stream_override_.get()) {
     CHECK(IsBrowserSideNavigationEnabled());
     DCHECK(!sync_load_response);
     DCHECK_NE(WebURLRequest::FrameTypeNone, request.getFrameType());
     resource_request->resource_body_stream_url = stream_override_->stream_url;
   }
 
+  // PlzNavigate: Invalid renderer main resource requests are rejected by the
+  // browser. This should not happen.
+  // TODO(arthursonzogni): Remove this when the root cause for
+  // https://crbug.com/705508 is found.
+  if (IsBrowserSideNavigationEnabled() &&
+      IsResourceTypeFrame(resource_request->resource_type) &&
+      !resource_request->resource_body_stream_url.SchemeIs(url::kBlobScheme)) {
+    base::debug::DumpWithoutCrashing();
+  }
+
   const RequestExtraData empty_extra_data;
   const RequestExtraData* extra_data;
   if (request.getExtraData())
     extra_data = static_cast<RequestExtraData*>(request.getExtraData());
   else
     extra_data = &empty_extra_data;
   extra_data->CopyToResourceRequest(resource_request.get());
 
   if (sync_load_response) {
     DCHECK(defers_loading_ == NOT_DEFERRING);
@@ skipping 624 matching lines @@
                                          int intra_priority_value) {
   context_->DidChangePriority(new_priority, intra_priority_value);
 }
 
 void WebURLLoaderImpl::setLoadingTaskRunner(
     base::SingleThreadTaskRunner* loading_task_runner) {
   context_->SetTaskRunner(loading_task_runner);
 }
 
 }  // namespace content