Use ScopedCrashKey for RendererDidNavigate crash dumps

content/browser/frame_host/navigation_controller_impl.cc

 // Copyright 2013 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.
 
 /*
  * Copyright (C) 2006, 2007, 2008, 2009 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/)
  *
@@ skipping 22 matching lines @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "content/browser/frame_host/navigation_controller_impl.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/debug/alias.h"
+#include "base/debug/crash_logging.h"
 #include "base/debug/dump_without_crashing.h"
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/string_number_conversions.h"  // Temporary
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/time/time.h"
 #include "base/trace_event/trace_event.h"
 #include "build/build_config.h"
@@ skipping 127 matching lines @@
   std::string debug_info =
       navigation_description + ": " +
       std::string(is_in_page ? "Is in page, " : "Is not in page, ") +
       std::string(params.nav_entry_id ? "has nav entry id, "
                                       : "does not have nav entry id, ") +
       std::string(params.did_create_new_entry ? "did create new entry, "
                                               : "did not create new entry, ") +
       std::string(params.should_replace_current_entry
                       ? "should replace current entry, "
                       : "should not replace current entry, ") +
+      std::string(params.intended_as_new_entry
+                      ? "intended as new entry, "
+                      : "not intended as new entry, ") +
+      std::string(params.url_is_unreachable ? "URL is unreachable, " : "") +
       entry->GetURL().spec() + " -> " + params.url.spec();
-  char debug_buf[2000];
-  base::strlcpy(debug_buf, debug_info.c_str(), arraysize(debug_buf));
-  base::debug::Alias(&debug_buf);
+  base::debug::ScopedCrashKey crash_key(
+      "navigation_controller_impl_renderer_did_navigate", debug_info);
   base::debug::DumpWithoutCrashing();
 }
 
 }  // namespace
 
 // NavigationControllerImpl ----------------------------------------------------
 
 const size_t kMaxEntryCountForTestingNotSet = static_cast<size_t>(-1);
 
 // static
@@ skipping 1978 matching lines @@
     }
   }
 }
 
 void NavigationControllerImpl::SetGetTimestampCallbackForTest(
     const base::Callback<base::Time()>& get_timestamp_callback) {
   get_timestamp_callback_ = get_timestamp_callback;
 }
 
 }  // namespace content