Add 'NavigationHandle::QueueConsoleMessage'.

content/public/browser/navigation_handle.h

 // Copyright 2015 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_BROWSER_NAVIGATION_HANDLE_H_
 #define CONTENT_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_
 
 #include <memory>
 
 #include "content/common/content_export.h"
 #include "content/public/browser/navigation_throttle.h"
+#include "content/public/common/console_message_level.h"
 #include "content/public/common/referrer.h"
 #include "net/base/net_errors.h"
 #include "ui/base/page_transition_types.h"
 
 class GURL;
 
 namespace content {
 class NavigationData;
 class NavigationThrottle;
 class RenderFrameHost;
@@ skipping 130 matching lines @@
 
   // Resumes a navigation that was previously deferred by a NavigationThrottle.
   virtual void Resume() = 0;
 
   // Cancels a navigation that was previously deferred by a NavigationThrottle.
   // |result| should be equal to NavigationThrottle::CANCEL or
   // NavigationThrottle::CANCEL_AND_IGNORE.
   virtual void CancelDeferredNavigation(
       NavigationThrottle::ThrottleCheckResult result) = 0;
 
+  // Adds a console message to the NavigationHandle's message queue. Messages
+  // queued before the navigation is ready to commit will be delivered to the
+  // final RenderFrameHost upon commit. Messages queued after the navigation has
+  // committed will be delivered immediately.
+  virtual void QueueConsoleMessage(ConsoleMessageLevel level,

[clamy, 2016/06/28 11:30:38]

The policy of content is not to add a method to the public API if it is not used
outside of content. Since this is not happening in this patch, this should be
added to NavigationHandleImpl instead, and then made available to the public API
when there is an external use case.

[Mike West, 2016/06/28 12:22:33]

You're right, sorry. I'd convinced myself that
https://codereview.chromium.org/2025683003 was calling this from //chrome
because that's where the BrowsingDataRemover lives. Fortunately, the throttle is
implemented in //content. I'll drop this addition. :)

+                                   const std::string& message) = 0;
+
   // Testing methods ----------------------------------------------------------
   //
   // The following methods should be used exclusively for writing unit tests.
 
   static std::unique_ptr<NavigationHandle> CreateNavigationHandleForTesting(
       const GURL& url,
       RenderFrameHost* render_frame_host);
 
   // Registers a NavigationThrottle for tests. The throttle can
   // modify the request, pause the request or cancel the request. This will
@@ skipping 22 matching lines @@
 
   // The NavigationData that the embedder returned from
   // ResourceDispatcherHostDelegate::GetNavigationData during commit. This will
   // be a clone of the NavigationData.
   virtual NavigationData* GetNavigationData() = 0;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_