Close the find bar on reload.

chrome/browser/ui/find_bar/find_bar_host_browsertest.cc

Index: chrome/browser/ui/find_bar/find_bar_host_browsertest.cc
diff --git a/chrome/browser/ui/find_bar/find_bar_host_browsertest.cc b/chrome/browser/ui/find_bar/find_bar_host_browsertest.cc
index 865e9768f024d3d4a8a28e422e4c6531aee1799a..1917573d788bd15dc4c24fbb54fea0385659ba4d 100644
--- a/chrome/browser/ui/find_bar/find_bar_host_browsertest.cc
+++ b/chrome/browser/ui/find_bar/find_bar_host_browsertest.cc
@@ -839,7 +839,10 @@ IN_PROC_BROWSER_TEST_F(FindInPageControllerTest,
   EXPECT_EQ(1, ordinal);
 }
 
-IN_PROC_BROWSER_TEST_F(FindInPageControllerTest, FindDisappearOnNavigate) {
+// This test will verify that the Find bar should disappear

[msw, 2014/09/08 18:16:55]

nit: consider: "Verify that the find bar is hidden on reload and navigation."

[Deepak, 2014/09/09 03:56:06]

Done.

+// on reload and on navigation.
+IN_PROC_BROWSER_TEST_F(FindInPageControllerTest,
+                       FindDisappearOnNavigateAndReload) {

[msw, 2014/09/08 18:16:55]

nit: HideFindOnNavigateAndReload fits on the line above; otherwise consider
HideFindBarOnNavigateAndReload.

[Deepak, 2014/09/09 03:56:06]

Done.

   // First we navigate to our special focus tracking page.
   GURL url = GetURL(kSimple);
   GURL url2 = GetURL(kFramePage);
@@ -854,7 +857,7 @@ IN_PROC_BROWSER_TEST_F(FindInPageControllerTest, FindDisappearOnNavigate) {
   EXPECT_TRUE(GetFindBarWindowInfo(&position, &fully_visible));
   EXPECT_TRUE(fully_visible);
 
-  // Reload the tab and make sure Find window doesn't go away.
+  // Reload and make sure the Find window goes away.

[msw, 2014/09/08 18:16:55]

nit: s/Find/find

[Deepak, 2014/09/09 03:56:06]

Done.

   content::WindowedNotificationObserver observer(
       content::NOTIFICATION_LOAD_STOP,
       content::Source<NavigationController>(
@@ -862,7 +865,13 @@ IN_PROC_BROWSER_TEST_F(FindInPageControllerTest, FindDisappearOnNavigate) {
               GetController()));
   chrome::Reload(browser(), CURRENT_TAB);
   observer.Wait();
+  EXPECT_TRUE(GetFindBarWindowInfo(&position, &fully_visible));
+  EXPECT_FALSE(fully_visible);
 
+  // Opening Findbar again.

[msw, 2014/09/08 18:16:55]

nit: "Open the find bar again." Or remove this superfluous comment.

[Deepak, 2014/09/09 03:56:06]

Done.

[msw, 2014/09/09 04:25:03]

You missed this nit; the comment should read. "Open the find bar again."

+  chrome::ShowFindBar(browser());
+
+  // Make sure it is open.
   EXPECT_TRUE(GetFindBarWindowInfo(&position, &fully_visible));
   EXPECT_TRUE(fully_visible);