Removed android scrolling fake mouse moves and device_supports_mouse

third_party/WebKit/Source/core/input/EventHandler.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights
  * reserved.
  * Copyright (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2012 Digia Plc. and/or its subsidiary(-ies)
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
@@ skipping 343 matching lines @@
   updateCursor();
 }
 
 void EventHandler::updateCursor() {
   TRACE_EVENT0("input", "EventHandler::updateCursor");
 
   // We must do a cross-frame hit test because the frame that triggered the
   // cursor update could be occluded by a different frame.
   ASSERT(m_frame == m_frame->localFrameRoot());
 
-  if (m_mouseEventManager->isMousePositionUnknown())

[bokan, 2016/12/02 18:04:05]

This can still get called after layout and style changes and some other places.
I think this should still be left in. Or DCHECK that the mouse position is known
and make sure we can never queue this task if it's unknown.

[amaralp, 2016/12/02 23:00:35]

The next if statement already does this check through
FrameView::shouldSetCursor().

[bokan, 2016/12/07 00:09:53]

Thanks, missed that.

-    return;
-
   FrameView* view = m_frame->view();
   if (!view || !view->shouldSetCursor())
     return;
 
   LayoutViewItem layoutViewItem = view->layoutViewItem();
   if (layoutViewItem.isNull())
     return;
 
   m_frame->document()->updateStyleAndLayout();
 
@@ skipping 1752 matching lines @@
 }
 
 FrameHost* EventHandler::frameHost() const {
   if (!m_frame->page())
     return nullptr;
 
   return &m_frame->page()->frameHost();
 }
 
 }  // namespace blink