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

Unified Diff: chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc

Issue 2323823004: Add action for each button on opt-in IME menu. (Closed)
Patch Set: Created 4 years, 3 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
Index: chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc
diff --git a/chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc b/chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc
index 745551f621b99d687dfaddc1b0f3853885373481..796812a13970822335f4eaf08f20119555a5fb28 100644
--- a/chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc
+++ b/chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc
@@ -36,6 +36,7 @@
#include "ui/chromeos/ime/input_method_menu_item.h"
#include "ui/chromeos/ime/input_method_menu_manager.h"
#include "ui/events/keycodes/keyboard_codes.h"
+#include "ui/keyboard/content/keyboard_content_util.h"
namespace chromeos {
@@ -1409,5 +1410,57 @@ TEST_F(InputMethodManagerImplTest, MigrateInputMethodTest) {
EXPECT_EQ(ImeIdFromEngineId("zh-t-i0-pinyin"), input_method_ids[3]);
}
+TEST_F(InputMethodManagerImplTest, OverrideKeyboardUrlRefWithEmojiOrHwt) {
+ const GURL inputview_url(
+ "chrome-extension://"
+ "inputview.html#id=us.compact.qwerty&language=en-US&passwordLayout=us."
+ "compact.qwerty&name=keyboard_us");
+ keyboard::SetOverrideContentUrl(inputview_url);
+ EXPECT_EQ(inputview_url, keyboard::GetOverrideContentUrl());
+
+ // Override the keyboard url ref with 'emoji'.
+ const GURL overridden_url_emoji(
+ "chrome-extension://"
+ "inputview.html#id=emoji&language=en-US&passwordLayout=us."
+ "compact.qwerty&name=keyboard_us");
+ manager_->OverrideKeyboardUrlRef("emoji");
+ EXPECT_EQ(overridden_url_emoji, keyboard::GetOverrideContentUrl());
+
+ // Override the keyboard url ref with 'hwt'.
+ const GURL overridden_url_hwt(
+ "chrome-extension://"
+ "inputview.html#id=hwt&language=en-US&passwordLayout=us."
+ "compact.qwerty&name=keyboard_us");
+ manager_->OverrideKeyboardUrlRef("hwt");
+ EXPECT_EQ(overridden_url_hwt, keyboard::GetOverrideContentUrl());
+}
+
+TEST_F(InputMethodManagerImplTest, OverrideKeyboardUrlRefWithVoice) {
+ const GURL inputview_url(
+ "chrome-extension://"
+ "inputview.html#id=us.compact.qwerty&language=en-US&passwordLayout=us."
+ "compact.qwerty&name=keyboard_us");
+ keyboard::SetOverrideContentUrl(inputview_url);
+ EXPECT_EQ(inputview_url, keyboard::GetOverrideContentUrl());
+
+ // Override the keyboard url ref with 'voice'.
+ const GURL overridden_url_voice(
+ "chrome-extension://"
+ "inputview.html#id=us.compact.qwerty.voice&language=en-US"
+ "&passwordLayout=us.compact.qwerty&name=keyboard_us");
+ manager_->OverrideKeyboardUrlRef("voice");
+ EXPECT_EQ(overridden_url_voice, keyboard::GetOverrideContentUrl());
+}
+
+TEST_F(InputMethodManagerImplTest, OverrideDefaultKeyboardUrlRef) {
+ const GURL default_url("chrome://inputview.html");
+ keyboard::SetOverrideContentUrl(default_url);
+
+ EXPECT_EQ(default_url, keyboard::GetOverrideContentUrl());
+
+ manager_->OverrideKeyboardUrlRef("emoji");
+ EXPECT_EQ(default_url, keyboard::GetOverrideContentUrl());
+}
+
} // namespace input_method
} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698