No pointer captured when the pointer lock is applied

content/browser/renderer_host/input/synthetic_mouse_driver.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 "content/browser/renderer_host/input/synthetic_mouse_driver.h"
 
 #include "content/browser/renderer_host/input/synthetic_gesture_target.h"
 
 namespace content {
 
 SyntheticMouseDriver::SyntheticMouseDriver() : last_modifiers_(0) {
   mouse_event_.pointer_type = blink::WebPointerProperties::PointerType::kMouse;
 }
 
 SyntheticMouseDriver::~SyntheticMouseDriver() {}
 
 void SyntheticMouseDriver::DispatchEvent(SyntheticGestureTarget* target,
                                          const base::TimeTicks& timestamp) {
   mouse_event_.SetTimeStampSeconds(ConvertTimestampToSeconds(timestamp));
-  target->DispatchInputEventToPlatform(mouse_event_);
+  if (mouse_event_.GetType() != blink::WebInputEvent::kUndefined) {
+    target->DispatchInputEventToPlatform(mouse_event_);
+    mouse_event_.SetType(blink::WebInputEvent::kUndefined);

[dtapuska, 2017/05/29 20:57:40]

Can we avoid the calls to SetType() and just mouse_event_ =
blink::WebMouseEvent();

to clear all state?

[lanwei, 2017/05/30 17:11:36]

We still need other states, like button, click count.

+  }
 }
 
 void SyntheticMouseDriver::Press(float x,
                                  float y,
                                  int index,
                                  SyntheticPointerActionParams::Button button) {
   DCHECK_EQ(index, 0);
   int modifiers =
       SyntheticPointerActionParams::GetWebMouseEventModifier(button);
   mouse_event_ = SyntheticWebMouseEventBuilder::Build(
@@ skipping 53 matching lines @@
   if (params.pointer_action_type() ==
           SyntheticPointerActionParams::PointerActionType::RELEASE &&
       mouse_event_.click_count <= 0) {
     return false;
   }
 
   return true;
 }
 
 }  // namespace content