Handle floating point coordinates from ozone to exosphere

ui/events/event_utils.h

 // Copyright (c) 2012 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.
 
 #ifndef UI_EVENTS_EVENT_UTILS_H_
 #define UI_EVENTS_EVENT_UTILS_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 49 matching lines @@
 
 // Ensures that the event timestamp values are coming from the same underlying
 // monotonic clock as base::TimeTicks::Now() and if it is not then falls
 // back to using the current ticks for event timestamp.
 EVENTS_EXPORT void ValidateEventTimeClock(base::TimeTicks* timestamp);
 
 // Get the location from a native event.  The coordinate system of the resultant
 // |Point| has the origin at top-left of the "root window".  The nature of
 // this "root window" and how it maps to platform-specific drawing surfaces is
 // defined in ui/aura/root_window.* and ui/aura/window_tree_host*.
 // TODO(tdresser): Return gfx::PointF here. See crbug.com/337827.

[oshima, 2017/01/23 16:34:32]

We should be able to remove old one (we may want to do it in separate CL if this
has to be merged). Can you add this one to BUG too?

[sky, 2017/01/23 17:04:06]

If the only caller is from the constructor that was just changed, it should be
easy to remove it now.

 EVENTS_EXPORT gfx::Point EventLocationFromNative(

[sky, 2017/01/23 16:08:00]

Do we still call the non-floating point version?

[denniskempin, 2017/01/23 20:36:41]

The method is called in a couple of locations besides that Constructor and
unittests. See:
https://cs.chromium.org/search/?q=EventLocationFromNative&sq=package:chromium...

They are not many, but since we need to merge this into M56 (now stable or about
to be stable) I would like to keep the impact as low as possible. What do you
think?

     const base::NativeEvent& native_event);
+EVENTS_EXPORT gfx::PointF EventLocationFromNativeF(
+    const base::NativeEvent& native_event);
 
 // Gets the location in native system coordinate space.
 EVENTS_EXPORT gfx::Point EventSystemLocationFromNative(
     const base::NativeEvent& native_event);
 
 #if defined(USE_X11)
 // Returns the 'real' button for an event. The button reported in slave events
 // does not take into account any remapping (e.g. using xmodmap), while the
 // button reported in master events do. This is a utility function to always
 // return the mapped button.
@@ skipping 94 matching lines @@
 // Update the native X11 event to correspond to the new button flags.
 EVENTS_EXPORT void UpdateX11EventForChangedButtonFlags(MouseEvent* event);
 #endif
 
 // Registers a custom event type.
 EVENTS_EXPORT int RegisterCustomEventType();
 
 }  // namespace ui
 
 #endif  // UI_EVENTS_EVENT_UTILS_H_