Disable console.log() dumping to LogMessages

content/browser/frame_host/render_frame_host_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.
 
 #include "content/browser/frame_host/render_frame_host_impl.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/containers/hash_tables.h"
 #include "base/lazy_instance.h"
@@ skipping 623 matching lines @@
     int32 line_no,
     const base::string16& source_id) {
   if (delegate_->AddMessageToConsole(level, message, line_no, source_id))
     return;
 
   // Pass through log level only on WebUI pages to limit console spew.
   const bool is_web_ui =
       HasWebUIScheme(delegate_->GetMainFrameLastCommittedURL());
   const int32 resolved_level = is_web_ui ? level : ::logging::LOG_INFO;
 
-  // LogMessages can be persisted so this shouldn't be logged in incognito mode.
-  // This rule is not applied to WebUI pages, because source code of WebUI is a
-  // part of Chrome source code, and we want to treat messages from WebUI the
-  // same way as we treat log messages from native code.
-  if (::logging::GetMinLogLevel() <= resolved_level &&
-      (is_web_ui ||
-       !GetSiteInstance()->GetBrowserContext()->IsOffTheRecord())) {
+  // LogMessages should only be created (as they can be persisted) with WebUI

[Charlie Reis, 2015/02/12 01:06:03]

nit: Update the comment to say why they shouldn't be logged in normal tabs
(i.e., for privacy reasons).  Right now it only says why they're need in the
WebUI case.

+  // pages, because source code of WebUI is a part of Chrome source code, and we
+  // want to treat messages from WebUI the same way as we treat log messages
+  // from native code.
+  if (::logging::GetMinLogLevel() <= resolved_level && is_web_ui) {
     logging::LogMessage("CONSOLE", line_no, resolved_level).stream()
         << "\"" << message << "\", source: " << source_id << " (" << line_no
         << ")";
   }
 }
 
 void RenderFrameHostImpl::OnCreateChildFrame(int new_routing_id,
                                              const std::string& frame_name,
                                              SandboxFlags sandbox_flags) {
   // It is possible that while a new RenderFrameHost was committed, the
@@ skipping 1225 matching lines @@
 void RenderFrameHostImpl::DidUseGeolocationPermission() {
   RenderFrameHost* top_frame = frame_tree_node()->frame_tree()->GetMainFrame();
   GetContentClient()->browser()->RegisterPermissionUsage(
       PERMISSION_GEOLOCATION,
       delegate_->GetAsWebContents(),
       GetLastCommittedURL().GetOrigin(),
       top_frame->GetLastCommittedURL().GetOrigin());
 }
 
 }  // namespace content