WindowManagerClient::AddAccelerator() should take an array

services/ui/ws/event_dispatcher_unittest.cc

 // Copyright 2015 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 "services/ui/ws/event_dispatcher.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <queue>
 
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
+#include "services/ui/common/accelerator_transport_util.h"
 #include "services/ui/common/event_matcher_util.h"
 #include "services/ui/ws/accelerator.h"
 #include "services/ui/ws/event_dispatcher_delegate.h"
 #include "services/ui/ws/server_window.h"
 #include "services/ui/ws/server_window_compositor_frame_sink_manager_test_api.h"
 #include "services/ui/ws/test_server_window_delegate.h"
 #include "services/ui/ws/test_utils.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/events/event.h"
 
@@ skipping 346 matching lines @@
   EXPECT_FALSE(details);
 
   // Delegate was informed that there wasn't a target.
   ui::Event* event_out =
       test_event_dispatcher_delegate()->last_event_target_not_found();
   ASSERT_TRUE(event_out);
   EXPECT_TRUE(event_out->IsKeyEvent());
   EXPECT_EQ(ui::VKEY_A, event_out->AsKeyEvent()->key_code());
 }
 
 TEST_F(EventDispatcherTest, AcceleratorBasic) {

[mfomitchev, 2016/11/29 01:55:46]

We need some coverage for adding more than one accelerator at a time. We could
test this in AcceleratorBasic, or create a separate test case if
AcceleratorBasic gets too big.

[thanhph, 2016/11/29 16:50:17]

Done, I created a new test. Thanks!

   ClearSetup();
   TestEventDispatcherDelegate event_dispatcher_delegate(nullptr);
   EventDispatcher dispatcher(&event_dispatcher_delegate);
 
   uint32_t accelerator_1 = 1;
   mojom::EventMatcherPtr matcher = ui::CreateKeyMatcher(
       ui::mojom::KeyboardCode::W, ui::mojom::kEventFlagControlDown);
-  EXPECT_TRUE(dispatcher.AddAccelerator(accelerator_1, std::move(matcher)));
+
+  std::vector<ui::mojom::AcceleratorTransportPtr> accelerators_1;
+  accelerators_1 = ash::mus::AddAcceleratorHelper(
+      std::move(accelerators_1), accelerator_1, std::move(matcher));
+  EXPECT_TRUE(dispatcher.AddAccelerators(std::move(accelerators_1)));
 
   uint32_t accelerator_2 = 2;
   matcher = ui::CreateKeyMatcher(ui::mojom::KeyboardCode::N,
                                  ui::mojom::kEventFlagNone);
-  EXPECT_TRUE(dispatcher.AddAccelerator(accelerator_2, std::move(matcher)));
+
+  std::vector<ui::mojom::AcceleratorTransportPtr> accelerators_2;
+  accelerators_2 = ash::mus::AddAcceleratorHelper(
+      std::move(accelerators_2), accelerator_2, std::move(matcher));
+  EXPECT_TRUE(dispatcher.AddAccelerators(std::move(accelerators_2)));
 
   // Attempting to add a new accelerator with the same id should fail.
   matcher = ui::CreateKeyMatcher(ui::mojom::KeyboardCode::T,
                                  ui::mojom::kEventFlagNone);
-  EXPECT_FALSE(dispatcher.AddAccelerator(accelerator_2, std::move(matcher)));
+  std::vector<ui::mojom::AcceleratorTransportPtr> accelerators_3;
+  accelerators_3 = ash::mus::AddAcceleratorHelper(
+      std::move(accelerators_3), accelerator_2, std::move(matcher));
+  EXPECT_FALSE(dispatcher.AddAccelerators(std::move(accelerators_3)));
 
   // Adding the accelerator with the same id should succeed once the existing
   // accelerator is removed.
   dispatcher.RemoveAccelerator(accelerator_2);
   matcher = ui::CreateKeyMatcher(ui::mojom::KeyboardCode::T,
                                  ui::mojom::kEventFlagNone);
-  EXPECT_TRUE(dispatcher.AddAccelerator(accelerator_2, std::move(matcher)));
+  std::vector<ui::mojom::AcceleratorTransportPtr> accelerators_4;
+  accelerators_4 = ash::mus::AddAcceleratorHelper(
+      std::move(accelerators_4), accelerator_2, std::move(matcher));
+  EXPECT_TRUE(dispatcher.AddAccelerators(std::move(accelerators_4)));
 
   // Attempting to add an accelerator with the same matcher should fail.
   uint32_t accelerator_3 = 3;
   matcher = ui::CreateKeyMatcher(ui::mojom::KeyboardCode::T,
                                  ui::mojom::kEventFlagNone);
-  EXPECT_FALSE(dispatcher.AddAccelerator(accelerator_3, std::move(matcher)));
+  std::vector<ui::mojom::AcceleratorTransportPtr> accelerators_5;
+  accelerators_5 = ash::mus::AddAcceleratorHelper(
+      std::move(accelerators_5), accelerator_3, std::move(matcher));
+  EXPECT_FALSE(dispatcher.AddAccelerators(std::move(accelerators_5)));
 
   matcher = ui::CreateKeyMatcher(ui::mojom::KeyboardCode::T,
                                  ui::mojom::kEventFlagControlDown);
-  EXPECT_TRUE(dispatcher.AddAccelerator(accelerator_3, std::move(matcher)));
+  std::vector<ui::mojom::AcceleratorTransportPtr> accelerators_6;
+  accelerators_6 = ash::mus::AddAcceleratorHelper(
+      std::move(accelerators_6), accelerator_3, std::move(matcher));
+  EXPECT_TRUE(dispatcher.AddAccelerators(std::move(accelerators_6)));
 }
 
 TEST_F(EventDispatcherTest, EventMatching) {
   TestEventDispatcherDelegate* event_dispatcher_delegate =
       test_event_dispatcher_delegate();
   EventDispatcher* dispatcher = event_dispatcher();
 
   mojom::EventMatcherPtr matcher = ui::CreateKeyMatcher(
       ui::mojom::KeyboardCode::W, ui::mojom::kEventFlagControlDown);
   uint32_t accelerator_1 = 1;
-  dispatcher->AddAccelerator(accelerator_1, std::move(matcher));
+  std::vector<ui::mojom::AcceleratorTransportPtr> accelerators_1;
+  accelerators_1 = ash::mus::AddAcceleratorHelper(
+      std::move(accelerators_1), accelerator_1, std::move(matcher));
+  dispatcher->AddAccelerators(std::move(accelerators_1));
 
   ui::KeyEvent key(ui::ET_KEY_PRESSED, ui::VKEY_W, ui::EF_CONTROL_DOWN);
   dispatcher->ProcessEvent(key, EventDispatcher::AcceleratorMatchPhase::ANY);
   EXPECT_EQ(accelerator_1,
             event_dispatcher_delegate->GetAndClearLastAccelerator());
 
   // EF_NUM_LOCK_ON should be ignored since CreateKeyMatcher defaults to
   // ignoring.
   key = ui::KeyEvent(ui::ET_KEY_PRESSED, ui::VKEY_W,
                      ui::EF_CONTROL_DOWN | ui::EF_NUM_LOCK_ON);
   dispatcher->ProcessEvent(key, EventDispatcher::AcceleratorMatchPhase::ANY);
   EXPECT_EQ(accelerator_1,
             event_dispatcher_delegate->GetAndClearLastAccelerator());
 
   key = ui::KeyEvent(ui::ET_KEY_PRESSED, ui::VKEY_W, ui::EF_NONE);
   dispatcher->ProcessEvent(key, EventDispatcher::AcceleratorMatchPhase::ANY);
   EXPECT_EQ(0u, event_dispatcher_delegate->GetAndClearLastAccelerator());
 
   uint32_t accelerator_2 = 2;
   matcher = ui::CreateKeyMatcher(ui::mojom::KeyboardCode::W,
                                  ui::mojom::kEventFlagNone);
-  dispatcher->AddAccelerator(accelerator_2, std::move(matcher));
+  std::vector<ui::mojom::AcceleratorTransportPtr> accelerators_2;
+  accelerators_2 = ash::mus::AddAcceleratorHelper(
+      std::move(accelerators_2), accelerator_2, std::move(matcher));
+  dispatcher->AddAccelerators(std::move(accelerators_2));
   dispatcher->ProcessEvent(key, EventDispatcher::AcceleratorMatchPhase::ANY);
   EXPECT_EQ(accelerator_2,
             event_dispatcher_delegate->GetAndClearLastAccelerator());
 
   dispatcher->RemoveAccelerator(accelerator_2);
   dispatcher->ProcessEvent(key, EventDispatcher::AcceleratorMatchPhase::ANY);
   EXPECT_EQ(0u, event_dispatcher_delegate->GetAndClearLastAccelerator());
 }
 
 // Tests that a post-target accelerator is not triggered by ProcessEvent.
 TEST_F(EventDispatcherTest, PostTargetAccelerator) {
   TestEventDispatcherDelegate* event_dispatcher_delegate =
       test_event_dispatcher_delegate();
   EventDispatcher* dispatcher = event_dispatcher();
 
   mojom::EventMatcherPtr matcher = ui::CreateKeyMatcher(
       ui::mojom::KeyboardCode::W, ui::mojom::kEventFlagControlDown);
   matcher->accelerator_phase = ui::mojom::AcceleratorPhase::POST_TARGET;
   uint32_t accelerator_1 = 1;
-  dispatcher->AddAccelerator(accelerator_1, std::move(matcher));
+  std::vector<ui::mojom::AcceleratorTransportPtr> accelerators_1;
+  accelerators_1 = ash::mus::AddAcceleratorHelper(
+      std::move(accelerators_1), accelerator_1, std::move(matcher));
+
+  dispatcher->AddAccelerators(std::move(accelerators_1));
 
   ui::KeyEvent key(ui::ET_KEY_PRESSED, ui::VKEY_W, ui::EF_CONTROL_DOWN);
   // The post-target accelerator should be fired if there is no focused window.
   dispatcher->ProcessEvent(key, EventDispatcher::AcceleratorMatchPhase::ANY);
   EXPECT_EQ(accelerator_1,
             event_dispatcher_delegate->GetAndClearLastAccelerator());
   std::unique_ptr<DispatchedEventDetails> details =
       event_dispatcher_delegate->GetAndAdvanceDispatchedEventDetails();
   EXPECT_FALSE(details);
 
@@ skipping 24 matching lines @@
 TEST_F(EventDispatcherTest, ProcessPost) {
   TestEventDispatcherDelegate* event_dispatcher_delegate =
       test_event_dispatcher_delegate();
   EventDispatcher* dispatcher = event_dispatcher();
 
   uint32_t pre_id = 1;
   {
     mojom::EventMatcherPtr matcher = ui::CreateKeyMatcher(
         ui::mojom::KeyboardCode::W, ui::mojom::kEventFlagControlDown);
     matcher->accelerator_phase = ui::mojom::AcceleratorPhase::PRE_TARGET;
-    dispatcher->AddAccelerator(pre_id, std::move(matcher));
+
+    std::vector<ui::mojom::AcceleratorTransportPtr> accelerators_1;
+    accelerators_1 = ash::mus::AddAcceleratorHelper(std::move(accelerators_1),
+                                                    pre_id, std::move(matcher));
+
+    dispatcher->AddAccelerators(std::move(accelerators_1));
   }
 
   uint32_t post_id = 2;
   {
     mojom::EventMatcherPtr matcher = ui::CreateKeyMatcher(
         ui::mojom::KeyboardCode::W, ui::mojom::kEventFlagControlDown);
     matcher->accelerator_phase = ui::mojom::AcceleratorPhase::POST_TARGET;
-    dispatcher->AddAccelerator(post_id, std::move(matcher));
+
+    std::vector<ui::mojom::AcceleratorTransportPtr> accelerators_2;
+    accelerators_2 = ash::mus::AddAcceleratorHelper(
+        std::move(accelerators_2), post_id, std::move(matcher));
+
+    dispatcher->AddAccelerators(std::move(accelerators_2));
   }
 
   // Set focused window for EventDispatcher dispatches key events.
   std::unique_ptr<ServerWindow> child = CreateChildWindow(WindowId(1, 3));
   event_dispatcher_delegate->SetFocusedWindowFromEventDispatcher(child.get());
 
   // Dispatch for ANY, which should trigger PRE and not call
   // DispatchInputEventToWindow().
   ui::KeyEvent key(ui::ET_KEY_PRESSED, ui::VKEY_W, ui::EF_CONTROL_DOWN);
   dispatcher->ProcessEvent(key, EventDispatcher::AcceleratorMatchPhase::ANY);
@@ skipping 1173 matching lines @@
   // The delegate can decide if it really wants to forward the event or not.
   EXPECT_EQ(child.get(),
             test_event_dispatcher_delegate()->lost_capture_window());
   EXPECT_EQ(child.get(), event_dispatcher()->capture_window());
   EXPECT_EQ(kClientAreaId, event_dispatcher()->capture_window_client_id());
 }
 
 }  // namespace test
 }  // namespace ws
 }  // namespace ui