From baf5783c50128923780ffff9704be4e805218d5e Mon Sep 17 00:00:00 2001 From: "jcivelli@chromium.org" Date: Fri, 16 Apr 2010 06:52:10 +0000 Subject: This CL makes the translate context menu disabled when the page language is the same as the UI language. BUG=39506 TEST=With Chrome configure to be in English, Visit www.google.com. Right click on the page, the context menu "Translate to English" should be disabled. Do the same on www.google.fr, the menu should be enabled. Review URL: http://codereview.chromium.org/1619014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44756 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/translate/translate_manager_unittest.cc | 8 ++++++++ 1 file changed, 8 insertions(+) (limited to 'chrome/browser/translate') diff --git a/chrome/browser/translate/translate_manager_unittest.cc b/chrome/browser/translate/translate_manager_unittest.cc index 33852b6..8c4d242 100644 --- a/chrome/browser/translate/translate_manager_unittest.cc +++ b/chrome/browser/translate/translate_manager_unittest.cc @@ -914,4 +914,12 @@ TEST_F(TranslateManagerTest, ContextMenu) { menu->TestExecuteItemCommand(IDS_CONTENT_CONTEXT_TRANSLATE); // No message expected since the translation should have been ignored. EXPECT_FALSE(GetTranslateMessage(&page_id, &original_lang, &target_lang)); + + // Test that the translate context menu is disabled when the page is in the + // same language as the UI. + SimulateNavigation(url, 0, L"Google", "en"); + menu.reset(TestRenderViewContextMenu::CreateContextMenu(contents())); + menu->Init(); + EXPECT_TRUE(menu->IsItemPresent(IDS_CONTENT_CONTEXT_TRANSLATE)); + EXPECT_FALSE(menu->TestIsItemCommandEnabled(IDS_CONTENT_CONTEXT_TRANSLATE)); } -- cgit v1.1