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

Side by Side Diff: components/translate/core/browser/mock_translate_client.cc

Issue 2906233004: Add translate client mock. (Closed)
Patch Set: fix format Created 3 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include <memory>
6
7 #include "components/translate/core/browser/mock_translate_client.h"
8 #include "components/translate/core/browser/translate_prefs.h"
9
10 namespace translate {
11
12 #if defined(OS_CHROMEOS)
13 const char* preferred_languages_prefs = "settings.language.preferred_languages";
14 #else
15 const char* preferred_languages_prefs = NULL;
napper 2017/05/30 06:24:35 nullptr
Leo 2017/05/31 01:38:26 Acknowledged.
16 #endif
17 const char* accept_languages_prefs = "intl.accept_languages";
18
19 namespace testing {
20 MockTranslateClient::MockTranslateClient(TranslateDriver* driver,
21 PrefService* prefs)
22 : driver_(driver), prefs_(prefs) {}
23
24 MockTranslateClient::~MockTranslateClient() {}
25
26 TranslateDriver* MockTranslateClient::GetTranslateDriver() {
napper 2017/05/30 06:24:35 Can these be inline?
Leo 2017/05/31 01:38:25 I did write them inline. But "git cl format" prefe
27 return driver_;
28 }
29
30 PrefService* MockTranslateClient::GetPrefs() {
31 return prefs_;
32 }
33
34 std::unique_ptr<TranslatePrefs> MockTranslateClient::GetTranslatePrefs() {
35 return base::MakeUnique<TranslatePrefs>(prefs_, accept_languages_prefs,
36 preferred_languages_prefs);
37 }
38
39 #if !defined(USE_AURA)
napper 2017/05/30 06:24:35 remove
Leo 2017/05/31 01:38:26 Thanks for the catch.
40 #endif
41
42 } // namespace testing
43 } // namespace translate
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698