Convert RenderViewHostTest to use the new navigation callbacks.

content/public/browser/navigation_handle.h

Index: content/public/browser/navigation_handle.h
diff --git a/content/public/browser/navigation_handle.h b/content/public/browser/navigation_handle.h
index 153be328985bbcec83b244a096998778cc72d4a5..b329165aa12b0a0794283dbc02db032e4dca9d7d 100644
--- a/content/public/browser/navigation_handle.h
+++ b/content/public/browser/navigation_handle.h
@@ -11,7 +11,8 @@
 #include "content/common/content_export.h"
 #include "content/public/browser/navigation_throttle.h"
 #include "content/public/browser/reload_type.h"
-#include "content/public/common/referrer.h"
+#include "content/public/common/referrer.h"
+#include "net/base/host_port_pair.h"
 #include "net/base/net_errors.h"
 #include "net/http/http_response_info.h"
 #include "ui/base/page_transition_types.h"
@@ -176,6 +177,9 @@ class CONTENT_EXPORT NavigationHandle {
   // GetNetErrorCode will be net::OK.
   virtual bool IsErrorPage() = 0;
 
+  // Returns the remote address of the socket which fetched this resource.
+  virtual net::HostPortPair GetSocketAddress() = 0;

[jam, 2017/01/31 23:59:44]

this is used by safe browsing code (outside of content), which is why I made it
public now.

[nasko, 2017/02/02 16:18:37]

But I don't see changes in chrome/ in this CL. Can you include those changes or
just expose it in a CL that fixes SB code?

[jam, 2017/02/03 05:26:56]

Right, I'm landing each conversion separately to make it easier to track down
regressions if any come up. So I'll land the SB one separately.

I didn't see much benefit for landing this cl with a hacker_style getter in
content, only to remove it the next day and replace it with a public getter.

+
   // Returns the response headers for the request, or nullptr if there aren't
   // any response headers or they have not been received yet. The response
   // headers may change during the navigation (e.g. after encountering a server