Allow multiple logging::LogMessage{Handler,Listener}s

content/browser/frame_host/render_frame_host_impl.cc

Index: content/browser/frame_host/render_frame_host_impl.cc
diff --git a/content/browser/frame_host/render_frame_host_impl.cc b/content/browser/frame_host/render_frame_host_impl.cc
index 7783a262a27b36201331f3184c5823e7e952f6f8..c598413d5eab49d648ddb7a7ae19fefc31ae165e 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -911,6 +911,10 @@ void RenderFrameHostImpl::OnAddMessageToConsole(
   if (delegate_->AddMessageToConsole(level, message, line_no, source_id))
     return;
 
+  // In order for CSP violation detections in the tests to work, console
+  // messages must be forwarded to logging::LogMessage().
+  // Ref: https://crrev.com/2013573007/

[grt (UTC plus 2), 2016/08/15 07:36:32]

i think it's better to link to a bug rather than a code review. is
https://crbug.com/616181 appropriate here?

[wychen, 2016/08/17 17:08:54]

Neither of the existing bugs in that CL is good.
I've created a new one.

[grt (UTC plus 2), 2016/08/18 08:09:33]

Perfect. Please put this issue number in the "BUG=" line of the CL description
as well. Thanks.

[wychen, 2016/08/18 08:50:30]

Done.

+
   // Pass through log level only on WebUI pages to limit console spew.
   const bool is_web_ui =
       HasWebUIScheme(delegate_->GetMainFrameLastCommittedURL());