Extension Error Piping - Blink: Chrome Side

content/public/renderer/content_renderer_client.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_RENDERER_CONTENT_RENDERER_CLIENT_H_
 #define CONTENT_PUBLIC_RENDERER_CONTENT_RENDERER_CLIENT_H_
 
 #include <string>
 
 #include "base/memory/weak_ptr.h"
@@ skipping 225 matching lines @@
   // Returns true if plugin living in the container can use
   // pp::FileIO::RequestOSFileHandle.
   virtual bool IsPluginAllowedToCallRequestOSFileHandle(
       WebKit::WebPluginContainer* container);
 
   // Returns whether BrowserPlugin should be allowed within the |container|.
   virtual bool AllowBrowserPlugin(WebKit::WebPluginContainer* container);
 
   // 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
 
 #endif  // CONTENT_PUBLIC_RENDERER_CONTENT_RENDERER_CLIENT_H_