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

Unified Diff: chrome/browser/ui/cocoa/infobars/translate_infobar_unittest.mm

Issue 292513002: Remove dependency of TranslateInfobarDelegate on chrome/ and content/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review comments Created 6 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
« no previous file with comments | « chrome/browser/translate/translate_tab_helper.cc ('k') | chrome/common/url_constants.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/cocoa/infobars/translate_infobar_unittest.mm
diff --git a/chrome/browser/ui/cocoa/infobars/translate_infobar_unittest.mm b/chrome/browser/ui/cocoa/infobars/translate_infobar_unittest.mm
index b04b2f77d89ddc130841d900bb7eeacfff4ba293..a0463a7b00e62cd2e1e0b9edd08916661fc5302a 100644
--- a/chrome/browser/ui/cocoa/infobars/translate_infobar_unittest.mm
+++ b/chrome/browser/ui/cocoa/infobars/translate_infobar_unittest.mm
@@ -39,8 +39,7 @@ class MockTranslateInfoBarDelegate : public TranslateInfoBarDelegate {
public:
MockTranslateInfoBarDelegate(content::WebContents* web_contents,
translate::TranslateStep step,
- TranslateErrors::Type error,
- PrefService* prefs)
+ TranslateErrors::Type error)
: TranslateInfoBarDelegate(
TranslateTabHelper::GetManagerFromWebContents(
web_contents)->GetWeakPtr(),
@@ -50,7 +49,6 @@ class MockTranslateInfoBarDelegate : public TranslateInfoBarDelegate {
"en",
"es",
error,
- prefs,
false) {}
MOCK_METHOD0(Translate, void());
@@ -96,13 +94,10 @@ class TranslationInfoBarTest : public CocoaProfileTest {
TranslateErrors::Type error = TranslateErrors::NONE;
if (type == translate::TRANSLATE_STEP_TRANSLATE_ERROR)
error = TranslateErrors::NETWORK;
- Profile* profile =
- Profile::FromBrowserContext(web_contents_->GetBrowserContext());
[[infobar_controller_ view] removeFromSuperview];
scoped_ptr<TranslateInfoBarDelegate> delegate(
- new MockTranslateInfoBarDelegate(web_contents_.get(), type, error,
- profile->GetPrefs()));
+ new MockTranslateInfoBarDelegate(web_contents_.get(), type, error));
scoped_ptr<infobars::InfoBar> infobar(
TranslateInfoBarDelegate::CreateInfoBar(delegate.Pass()));
if (infobar_)
« no previous file with comments | « chrome/browser/translate/translate_tab_helper.cc ('k') | chrome/common/url_constants.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698