Migrate addMessageToConsole API to RenderFrame.

content/browser/frame_host/render_frame_host_delegate.h

 // 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.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_DELEGATE_H_
 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_DELEGATE_H_
 
+#include "base/basictypes.h"
 #include "content/common/content_export.h"
 
+class GURL;
+
 namespace IPC {
 class Message;
 }
 
 namespace content {
 class RenderFrameHost;
 class WebContents;
 struct ContextMenuParams;
 
 // An interface implemented by an object interested in knowing about the state
 // of the RenderFrameHost.
 class CONTENT_EXPORT RenderFrameHostDelegate {
  public:
   // This is used to give the delegate a chance to filter IPC messages.
   virtual bool OnMessageReceived(RenderFrameHost* render_frame_host,
                                  const IPC::Message& message);
 
+  // Gets the URL that is currently being displayed, if there is one.
+  virtual const GURL& GetURL() const;

[nasko, 2014/03/28 23:05:05]

The name of this method should be adjusted to match what it will return. This
will be the URL for the main frame, which is not necessarily the frame the
called the delegate_.

Also, since this is implemented in WebContentsImpl, you should move the method
to the section for RenderFrameHostDelegate.

[mkosiba (inactive), 2014/03/31 09:48:41]

Done.

+
+  // A message was added to to the console.
+  virtual bool AddMessageToConsole(int32 level,
+                                   const base::string16& message,
+                                   int32 line_no,
+                                   const base::string16& source_id);
+
   // Informs the delegate whenever a RenderFrameHost is created.
   virtual void RenderFrameCreated(RenderFrameHost* render_frame_host) {}
 
   // Informs the delegate whenever a RenderFrameHost is deleted.
   virtual void RenderFrameDeleted(RenderFrameHost* render_frame_host) {}
 
   // The top-level RenderFrame began loading a new page. This corresponds to
   // Blink's notion of the throbber starting.
   virtual void DidStartLoading(RenderFrameHost* render_frame_host) {}
 
@@ skipping 13 matching lines @@
   // not a WebContents, returns NULL.
   virtual WebContents* GetAsWebContents();
 
  protected:
   virtual ~RenderFrameHostDelegate() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_DELEGATE_H_