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

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

Issue 16282003: Translate: make alpha language support enabled by default (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase due to conflict in CQ Created 7 years, 6 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
« no previous file with comments | « chrome/browser/translate/translate_language_list.cc ('k') | chrome/common/chrome_switches.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 1267e2ab92b102d51d2efee016f4735ee9725d59..33471ee9019a384ce9cabdb6160fdb8ca263478d 100644
--- a/chrome/browser/translate/translate_manager_browsertest.cc
+++ b/chrome/browser/translate/translate_manager_browsertest.cc
@@ -275,8 +275,6 @@ class TranslateManagerBrowserTest : public ChromeRenderViewHostTestHarness,
void SimulateSupportedLanguagesURLFetch(
bool success, const std::vector<std::string>& languages) {
- net::TestURLFetcher* fetcher = url_fetcher_factory_.GetFetcherByID(1);
- ASSERT_TRUE(fetcher);
net::URLRequestStatus status;
status.set_status(success ? net::URLRequestStatus::SUCCESS :
net::URLRequestStatus::FAILED);
@@ -296,11 +294,15 @@ class TranslateManagerBrowserTest : public ChromeRenderViewHostTestHarness,
}
data += "}})";
}
- fetcher->set_url(fetcher->GetOriginalURL());
- fetcher->set_status(status);
- fetcher->set_response_code(success ? 200 : 500);
- fetcher->SetResponseString(data);
- fetcher->delegate()->OnURLFetchComplete(fetcher);
+ for (int id = 1; id <= 2; ++id) {
+ net::TestURLFetcher* fetcher = url_fetcher_factory_.GetFetcherByID(id);
+ ASSERT_TRUE(fetcher);
+ fetcher->set_url(fetcher->GetOriginalURL());
+ fetcher->set_status(status);
+ fetcher->set_response_code(success ? 200 : 500);
+ fetcher->SetResponseString(data);
+ fetcher->delegate()->OnURLFetchComplete(fetcher);
+ }
}
void SetPrefObserverExpectation(const char* path) {
« no previous file with comments | « chrome/browser/translate/translate_language_list.cc ('k') | chrome/common/chrome_switches.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698