Extension Error Piping - Blink: Chrome Side

content/public/renderer/content_renderer_client.h

Index: content/public/renderer/content_renderer_client.h
diff --git a/content/public/renderer/content_renderer_client.h b/content/public/renderer/content_renderer_client.h
index a00e1a9414bb7898cfdc0da90a747b83166cbea5..f3ecf7d908d219c8b6aacd0cf07b50494532e7ee 100644
--- a/content/public/renderer/content_renderer_client.h
+++ b/content/public/renderer/content_renderer_client.h
@@ -243,6 +243,15 @@ class CONTENT_EXPORT ContentRendererClient {
 
   // Returns true if the page at |url| can use Pepper MediaStream APIs.
   virtual bool AllowPepperMediaStreamAPI(const GURL& url);
+
+  // Returns true if we should report a detailed message (including a stack
+  // trace) for console [logs|errors|exceptions]. |source| is the WebKit-
+  // reported source for the error; this can point to a page or a script,
+  // and can be external (e.g., "http://www.google.com"), extension-related
+  // (e.g., "chrome-extension://<extension_id>/background.js"), or internal

[jam, 2013/08/25 07:12:58]

nit: it's a layering violation to mention concepts from embedders, like
extensions, in content. please remove this mention

[Devlin, 2013/08/26 20:32:40]

Done in https://codereview.chromium.org/23252006/

+  // (e.g., "event_bindings" or "schemaUtils").
+  virtual bool ShouldReportDetailedMessageForSource(
+      const base::string16& source) const;
 };
 
 }  // namespace content