Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(95)

Unified Diff: chrome/browser/translate/translate_manager_browsertest.cc

Issue 6883299: Hide translate infobar on programmatic navigation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Update testcase. Created 9 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/translate/translate_manager_browsertest.cc
diff --git a/chrome/browser/translate/translate_manager_browsertest.cc b/chrome/browser/translate/translate_manager_browsertest.cc
index a5103f373ce7a76d77d57f8bee644f9295547504..3ac8d36be55b81a32f937099e022c111f805043e 100644
--- a/chrome/browser/translate/translate_manager_browsertest.cc
+++ b/chrome/browser/translate/translate_manager_browsertest.cc
@@ -721,13 +721,6 @@ TEST_F(TranslateManagerTest, TranslateInPageNavigation) {
infobar = GetTranslateInfoBar();
ASSERT_TRUE(infobar != NULL);
- // Navigate in page, the same infobar should still be shown.
jeremy 2011/05/17 14:52:15 I've removed this part of the test since the navig
- ClearRemovedInfoBars();
- SimulateNavigation(GURL("http://www.google.fr/#ref1"), "fr",
- true);
- EXPECT_FALSE(InfoBarRemoved());
- EXPECT_EQ(infobar, GetTranslateInfoBar());
-
// Navigate out of page, a new infobar should show.
// See note in TranslateCloseInfoBarInPageNavigation test on why it is
// important to navigate to a page in a different language for this test.

Powered by Google App Engine
This is Rietveld 408576698