Remove no-op grace area touch calibration code

ui/aura/window_tree_host_x11.cc

 // 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.
 
 #include "ui/aura/window_tree_host_x11.h"
 
 #include <strings.h>
 #include <X11/cursorfont.h>
 #include <X11/extensions/XInput2.h>
 #include <X11/extensions/Xrandr.h>
@@ skipping 131 matching lines @@
   // which need to be expanded when converting to screen coordinates,
   // so that location on bezels will be outside of screen area.
   void Calibrate(ui::TouchEvent* event, const gfx::Rect& bounds) {
     int x = event->x();
     int y = event->y();
 
     if (!left_ && !right_ && !top_ && !bottom_)
       return;
 
     const int resolution_x = bounds.width();
     const int resolution_y = bounds.height();

[pkotwicz, 2015/01/23 19:34:33]

My understanding is that the following is always true:
y < resolution_y

[Mr4D (OOO till 08-26), 2015/01/27 13:35:45]

This has to be enabled by the TP driver. It was meant to be used for the Pixel -
however - due to bending ghost touch events we disabled these things at the
time. 

But they might come back. Please check with our touch gurus if we have decided
to not use bezel anymore.

-    // The "grace area" (10% in this case) is to make it easier for the user to
-    // navigate to the corner.
-    const double kGraceAreaFraction = 0.1;
     if (left_ || right_) {
       // Offset the x position to the real
       x -= left_;
-      // Check if we are in the grace area of the left side.
-      // Note: We might not want to do this when the gesture is locked?
-      if (x < 0 && x > -left_ * kGraceAreaFraction)
-        x = 0;
-      // Check if we are in the grace area of the right side.
-      // Note: We might not want to do this when the gesture is locked?
-      if (x > resolution_x - left_ &&
-          x < resolution_x - left_ + right_ * kGraceAreaFraction)
-        x = resolution_x - left_;
       // Scale the screen area back to the full resolution of the screen.
       x = (x * resolution_x) / (resolution_x - (right_ + left_));
     }
     if (top_ || bottom_) {
       // When there is a top bezel we add our border,
       y -= top_;
-
-      // Check if we are in the grace area of the top side.
-      // Note: We might not want to do this when the gesture is locked?
-      if (y < 0 && y > -top_ * kGraceAreaFraction)
-        y = 0;
-
-      // Check if we are in the grace area of the bottom side.
-      // Note: We might not want to do this when the gesture is locked?
-      if (y > resolution_y - top_ &&
-          y < resolution_y - top_ + bottom_ * kGraceAreaFraction)

[pkotwicz, 2015/01/23 19:34:33]

My understanding is that this if statement is never run. I think that the
ChromeOS devices with touchscreen that we ship either have no bezel or just a
bottom bezel?

[Mr4D (OOO till 08-26), 2015/01/27 13:35:45]

At the moment only the bottom bezel is reliable. Yes. However that might be
subject to change. Do we need to keep it around for it? I guess not.

-        y = resolution_y - top_;
       // Scale the screen area back to the full resolution of the screen.
       y = (y * resolution_y) / (resolution_y - (bottom_ + top_));
     }
 
     // Set the modified coordinate back to the event.
     if (event->root_location() == event->location()) {
       // Usually those will be equal,
       // if not, I am not sure what the correct value should be.
       event->set_root_location(gfx::Point(x, y));
     }
@@ skipping 497 matching lines @@
 }
 
 namespace test {
 
 void SetUseOverrideRedirectWindowByDefault(bool override_redirect) {
   default_override_redirect = override_redirect;
 }
 
 }  // namespace test
 }  // namespace aura