Quit immediately in TestNavigationObserver.

content/public/test/test_navigation_observer.h

Index: content/public/test/test_navigation_observer.h
diff --git a/content/public/test/test_navigation_observer.h b/content/public/test/test_navigation_observer.h
index 93091ca64c7c8aeb893620ce940dfc8accac0672..640948613ba37bd714373e1f46d343dfed541579 100644
--- a/content/public/test/test_navigation_observer.h
+++ b/content/public/test/test_navigation_observer.h
@@ -27,9 +27,13 @@ class TestNavigationObserver {
   // Create and register a new TestNavigationObserver against the
   // |web_contents|.
   TestNavigationObserver(WebContents* web_contents,
-                         int number_of_navigations);
+                         int number_of_navigations,
+                         MessageLoopRunner::QuitMode quit_mode =
+                             MessageLoopRunner::QuitMode::IMMEDIATE);
   // Like above but waits for one navigation.
-  explicit TestNavigationObserver(WebContents* web_contents);
+  explicit TestNavigationObserver(WebContents* web_contents,

[nasko, 2017/01/20 22:54:45]

nit: Do we need for explicit anymore, as there are more than one parameters.

[Alexander Semashko, 2017/01/23 19:45:39]

Constructors should be explicit as long as it is possible to call them with a
single argument. In this case this is possible thanks to the second argument
having a default value. A note w.r.t. default arguments: they are allowed now in
non-virtual functions, and (I hope) this one is temporary and will be removed
when we sort out issues around NavigateToURL in browser_tests.

+                                  MessageLoopRunner::QuitMode quit_mode =
+                                      MessageLoopRunner::QuitMode::IMMEDIATE);
 
   virtual ~TestNavigationObserver();