Previews infobar tests

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 007cd9da717135ea9779f9b1bb015445b4a2f6a8..e61eeb0a623b27f26e95160b5f6f4a71f19590ad 100644
--- a/content/public/browser/navigation_handle.h
+++ b/content/public/browser/navigation_handle.h
@@ -7,6 +7,7 @@
 
 #include <memory>
 
+#include "base/memory/ref_counted.h"
 #include "content/common/content_export.h"
 #include "content/public/browser/navigation_throttle.h"
 #include "content/public/common/referrer.h"
@@ -204,7 +205,16 @@ class CONTENT_EXPORT NavigationHandle {
 
   // Simulates the reception of the network response.
   virtual NavigationThrottle::ThrottleCheckResult
-  CallWillProcessResponseForTesting(RenderFrameHost* render_frame_host) = 0;
+  CallWillProcessResponseForTesting(
+      RenderFrameHost* render_frame_host,
+      scoped_refptr<net::HttpResponseHeaders> response_headers) = 0;
+
+  // Simulates the navigation being committed.
+  virtual void CallDidCommitNavigationForTesting(
+      const GURL& url,
+      ui::PageTransition transition,
+      bool same_page,

[clamy, 2016/09/08 13:11:07]

I only see one instance of CallDidCommitNavigationForTesting being used in this
patch, in which |same_page| is set to false. If this is indeed the case, can we
remove this parameter?

[megjablon, 2016/09/08 18:15:25]

Removed.

+      RenderFrameHost* render_frame_host) = 0;

[clamy, 2016/09/08 13:11:07]

It would seem less error prone to not have a RFH there, in particular in light
of this patch https://codereview.chromium.org/2321543002/ since the one that
commits the navigation is supposed to be the one that is inside the
NavigationHandle.

[megjablon, 2016/09/08 18:15:25]

Good call. Done.

 
   // The NavigationData that the embedder returned from
   // ResourceDispatcherHostDelegate::GetNavigationData during commit. This will