services/ui: make autorepeat work with accelerator.

services/ui/ws/event_matcher.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 "services/ui/ws/event_matcher.h"
 
 namespace ui {
 namespace ws {
 
 EventMatcher::EventMatcher(const mojom::EventMatcher& matcher)
@@ skipping 58 matching lines @@
     fields_to_match_ |= POINTER_LOCATION;
     pointer_region_ = matcher.pointer_location_matcher->region;
   }
 }
 
 EventMatcher::~EventMatcher() {}
 
 bool EventMatcher::MatchesEvent(const ui::Event& event) const {
   if ((fields_to_match_ & TYPE) && event.type() != event_type_)
     return false;
-  int flags = event.flags() & ~ignore_event_flags_;
+  constexpr int kIgnoreFlags = EF_IS_SYNTHESIZED | EF_IS_REPEAT;

[sky, 2017/03/30 23:44:43]

Please add a comment here. Also, I would name this kSyntheticFlags or something
to better indicate these flags should never be matched against.

[Peng, 2017/03/31 14:09:22]

Done.

+  int flags = event.flags() & ~(ignore_event_flags_ | kIgnoreFlags);
   if ((fields_to_match_ & FLAGS) && flags != event_flags_)
     return false;
   if (fields_to_match_ & KEYBOARD_CODE) {
     if (!event.IsKeyEvent())
       return false;
     if (keyboard_code_ != event.AsKeyEvent()->GetConflatedWindowsKeyCode())
       return false;
   }
   if (fields_to_match_ & POINTER_KIND) {
     if (!event.IsPointerEvent() ||
@@ skipping 14 matching lines @@
          event_type_ == other.event_type_ &&
          event_flags_ == other.event_flags_ &&
          ignore_event_flags_ == other.ignore_event_flags_ &&
          keyboard_code_ == other.keyboard_code_ &&
          pointer_type_ == other.pointer_type_ &&
          pointer_region_ == other.pointer_region_;
 }
 
 }  // namespace ws
 }  // namespace ui