Code to collect issue 97285 debugging info for crash reports.

content/common/content_debug_logging.cc

+// Copyright (c) 2011 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/public/common/content_debug_logging.h"
+
+#include "base/logging.h"
+
+namespace content {
+namespace debug {
+
+namespace {
+
+// These should be initialized by the process in main(), so should be
+// thread-safe.crbug.com

[Bernhard Bauer, 2012/09/05 15:04:55]

Oops! Google Chrome could not find thread-safe.crbug.com

[Scott Hess - ex-Googler, 2012/09/05 17:17:05]

I have no idea where this came from!

+RecordMsgFn* g_record_handler = NULL;
+GetMessagesFn* g_get_handler = NULL;
+
+}  // namespace
+
+void RegisterMessageHandlers(RecordMsgFn* record_handler,
+                             GetMessagesFn* get_handler) {
+  g_record_handler = record_handler;
+  g_get_handler = get_handler;
+}
+
+void RecordMsg(int bug_id, const std::string& msg) {
+  DCHECK(g_record_handler);

[Bernhard Bauer, 2012/09/05 15:04:55]

Are these the ones you want to replace with DLOG_IF? Then you could just put a
DLOG inside of the if statement (or turn it around and early-return if
|g_record_handler| is NULL).

[Scott Hess - ex-Googler, 2012/09/05 17:17:05]

Yeah, pretty much.

Except now I realize that having a DLOG() in here will muddy up test output,
since tests are unlikely to register handlers.  I'll just leave it out entirely,
and for now users (aka me) are on their own to verify that things actually work.

+  if (g_record_handler)
+    (*g_record_handler)(bug_id, msg);
+}
+
+bool GetMessages(int bug_id, std::vector<std::string>* msgs) {
+  DCHECK(g_get_handler);
+  if (g_get_handler)
+    return (*g_get_handler)(bug_id, msgs);
+  return false;
+}
+
+}  // namespace debug
+}  // namespace content