Hide handwriting and voice buttons when keyboard is in restricted state

ash/system/ime_menu/ime_menu_tray_unittest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ash/system/ime_menu/ime_menu_tray.h"
 
 #include "ash/accelerators/accelerator_controller.h"
 #include "ash/accessibility_delegate.h"
 #include "ash/ime/ime_controller.h"
 #include "ash/public/interfaces/ime_info.mojom.h"
 #include "ash/shell.h"
 #include "ash/system/ime_menu/ime_list_view.h"
 #include "ash/system/status_area_widget.h"
 #include "ash/test/ash_test_base.h"
 #include "ash/test/status_area_widget_test_helper.h"
 #include "base/run_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "ui/accessibility/ax_node_data.h"
 #include "ui/base/ime/chromeos/input_method_manager.h"
 #include "ui/base/ime/chromeos/mock_input_method_manager.h"
 #include "ui/base/ime/ime_bridge.h"
 #include "ui/base/ime/text_input_flags.h"
 #include "ui/events/event.h"
 #include "ui/views/controls/label.h"
 
 using base::UTF8ToUTF16;
+using chromeos::input_method::InputMethodManager;
+using chromeos::input_method::MockInputMethodManager;
 
 namespace ash {
 namespace {
 
 ImeMenuTray* GetTray() {
   return StatusAreaWidgetTestHelper::GetStatusAreaWidget()->ime_menu_tray();
 }
 
 void SetCurrentIme(const std::string& current_ime_id,
                    const std::vector<mojom::ImeInfo>& available_imes) {
@@ skipping 255 matching lines @@
   // The virtual keyboard should be enabled.
   EXPECT_TRUE(accessibility_delegate->IsVirtualKeyboardEnabled());
 
   // Hides the keyboard.
   GetTray()->OnKeyboardHidden();
   // The keyboard should still be disabled, which is a posted task.
   base::RunLoop().RunUntilIdle();
   EXPECT_FALSE(accessibility_delegate->IsVirtualKeyboardEnabled());
 }
 
-TEST_F(ImeMenuTrayTest, ShowEmojiHandwritingVoiceButtons) {
+TEST_F(ImeMenuTrayTest, ShouldShowBottomButtons) {
   FocusInInputContext(ui::TEXT_INPUT_TYPE_TEXT);
-  EXPECT_FALSE(GetTray()->ShouldShowEmojiHandwritingVoiceButtons());
+  bool show_emoji, show_handwriting, show_voice;
+  EXPECT_FALSE(GetTray()->ShouldShowBottomButtons(show_emoji, show_handwriting,
+                                                  show_voice));
+  EXPECT_FALSE(show_emoji);
+  EXPECT_FALSE(show_handwriting);
+  EXPECT_FALSE(show_voice);
 
-  chromeos::input_method::InputMethodManager* input_method_manager =
+  InputMethodManager* input_method_manager =
       chromeos::input_method::InputMethodManager::Get();

[James Cook, 2017/06/26 17:21:19]

nit: chromeos::input_method:: not needed (you don't use it below)

[Azure Wei, 2017/06/27 14:27:15]

Done.

   EXPECT_FALSE(input_method_manager);
-  chromeos::input_method::InputMethodManager::Initialize(
-      new chromeos::input_method::MockInputMethodManager);
-  input_method_manager = chromeos::input_method::InputMethodManager::Get();
+  InputMethodManager::Initialize(new MockInputMethodManager);
+  input_method_manager = InputMethodManager::Get();
   EXPECT_TRUE(input_method_manager &&
               input_method_manager->IsEmojiHandwritingVoiceOnImeMenuEnabled());
-  EXPECT_TRUE(GetTray()->ShouldShowEmojiHandwritingVoiceButtons());
+  EXPECT_EQ(InputMethodManager::FeaturesRestrictedState::RESTRICTED_NONE,
+            input_method_manager->GetFeaturesRestrictedState());
+  EXPECT_TRUE(GetTray()->ShouldShowBottomButtons(show_emoji, show_handwriting,
+                                                 show_voice));
+  EXPECT_TRUE(show_emoji);
+  EXPECT_TRUE(show_handwriting);
+  EXPECT_TRUE(show_voice);
 
   FocusInInputContext(ui::TEXT_INPUT_TYPE_PASSWORD);
-  EXPECT_FALSE(GetTray()->ShouldShowEmojiHandwritingVoiceButtons());
+  EXPECT_FALSE(GetTray()->ShouldShowBottomButtons(show_emoji, show_handwriting,
+                                                  show_voice));
+  EXPECT_FALSE(show_emoji);
+  EXPECT_FALSE(show_handwriting);
+  EXPECT_FALSE(show_voice);
+}
+
+TEST_F(ImeMenuTrayTest, ShouldShowBottomButtonsSeperate) {
+  FocusInInputContext(ui::TEXT_INPUT_TYPE_TEXT);
+  InputMethodManager* input_method_manager =
+      chromeos::input_method::InputMethodManager::Get();

[James Cook, 2017/06/26 17:21:19]

ditto

[Azure Wei, 2017/06/27 14:27:14]

Done.

+  InputMethodManager::Initialize(new MockInputMethodManager);
+  input_method_manager = InputMethodManager::Get();
+  EXPECT_TRUE(input_method_manager &&
+              input_method_manager->IsEmojiHandwritingVoiceOnImeMenuEnabled());
+  bool show_emoji, show_handwriting, show_voice;
+
+  // Sets emoji restricted.
+  input_method_manager->SetFeaturesRestrictedState(
+      InputMethodManager::FeaturesRestrictedState::RESTRICTED_EMOJI, true);
+  EXPECT_EQ(InputMethodManager::FeaturesRestrictedState::RESTRICTED_EMOJI,
+            input_method_manager->GetFeaturesRestrictedState());
+  EXPECT_TRUE(GetTray()->ShouldShowBottomButtons(show_emoji, show_handwriting,
+                                                 show_voice));
+  EXPECT_FALSE(show_emoji);
+  EXPECT_TRUE(show_handwriting);
+  EXPECT_TRUE(show_voice);
+
+  // Sets emoji not restriected, but voice and handwriting restricted.
+  input_method_manager->SetFeaturesRestrictedState(
+      InputMethodManager::FeaturesRestrictedState::RESTRICTED_EMOJI, false);
+  input_method_manager->SetFeaturesRestrictedState(
+      InputMethodManager::FeaturesRestrictedState::RESTRICTED_VOICE, true);
+  input_method_manager->SetFeaturesRestrictedState(
+      InputMethodManager::FeaturesRestrictedState::RESTRICTED_HANDWRITING,
+      true);
+  uint32_t expected_state =
+      InputMethodManager::FeaturesRestrictedState::RESTRICTED_VOICE |
+      InputMethodManager::FeaturesRestrictedState::RESTRICTED_HANDWRITING;
+  EXPECT_EQ(expected_state, input_method_manager->GetFeaturesRestrictedState());
+  EXPECT_TRUE(GetTray()->ShouldShowBottomButtons(show_emoji, show_handwriting,
+                                                 show_voice));
+  EXPECT_TRUE(show_emoji);
+  EXPECT_FALSE(show_handwriting);
+  EXPECT_FALSE(show_voice);
 }
 
 }  // namespace ash