Skip to content

Commit 317fa6a

Browse files
committed
[Unit Tests] TranslateManage::GetLanguageState now returns pointer.
Chromium change: https://source.chromium.org/chromium/chromium/src/+/83f98320cc0144937d3399b6462f4e3ceaf57816 commit 83f98320cc0144937d3399b6462f4e3ceaf57816 Author: Evan Stade <[email protected]> Date: Mon Sep 28 20:56:16 2020 +0000 Make TranslateManager::GetLanguageState return a pointer instead of ref Also make ChromeTranslateClient return const& instead of &. Non-const refs are forbidden by the style guide. Bug: none
1 parent 9a24636 commit 317fa6a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

chromium_src/components/translate/core/browser/translate_manager_unittest.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ TEST_F(TranslateManagerTest, CanManuallyTranslate_WithoutAPIKey) {
2626
.WillByDefault(Return(true));
2727
network_notifier_.SimulateOnline();
2828

29-
translate_manager_->GetLanguageState().LanguageDetermined("de", true);
29+
translate_manager_->GetLanguageState()->LanguageDetermined("de", true);
3030
EXPECT_TRUE(translate_manager_->CanManuallyTranslate());
3131

3232
::google_apis::SetAPIKeyForTesting(api_key);
@@ -48,7 +48,7 @@ TEST_F(TranslateManagerTest, CanManuallyTranslate_WithAPIKey) {
4848
.WillByDefault(Return(true));
4949
network_notifier_.SimulateOnline();
5050

51-
translate_manager_->GetLanguageState().LanguageDetermined("de", true);
51+
translate_manager_->GetLanguageState()->LanguageDetermined("de", true);
5252
EXPECT_TRUE(translate_manager_->CanManuallyTranslate());
5353

5454
::google_apis::SetAPIKeyForTesting(api_key);

0 commit comments

Comments
 (0)