Type conversion fixes, ui/gfx/ edition.

ui/gfx/display.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/gfx/display.h"
 
 #include <algorithm>
 
 #include "base/command_line.h"
 #include "base/logging.h"
@@ skipping 135 matching lines @@
       gfx::ToFlooredSize(gfx::ScaleSize(bounds_in_pixel.size(),
                                         1.0f / device_scale_factor_)));
   UpdateWorkAreaFromInsets(insets);
 }
 
 void Display::SetSize(const gfx::Size& size_in_pixel) {
   gfx::Point origin = bounds_.origin();
 #if defined(USE_AURA)
   gfx::PointF origin_f = origin;
   origin_f.Scale(device_scale_factor_);
-  origin.SetPoint(origin_f.x(), origin_f.y());
+  // !!! Should this use ToRoundedPoint() instead?

[danakj, 2014/10/18 18:40:06]

It looks like the size gets floored:
https://code.google.com/p/chromium/codesearch#chromium/src/ash/display/displa...

So I'd expect this to match that to avoid a logical gap between monitors.

+oshima we should have unit tests for this?

[Peter Kasting, 2014/10/20 23:38:56]

Acknowledged.

+  origin = gfx::ToFlooredPoint(origin_f);
 #endif
   SetScaleAndBounds(device_scale_factor_, gfx::Rect(origin, size_in_pixel));
 }
 
 void Display::UpdateWorkAreaFromInsets(const gfx::Insets& insets) {
   work_area_ = bounds_;
   work_area_.Inset(insets);
 }
 
 gfx::Size Display::GetSizeInPixel() const {
@@ skipping 16 matching lines @@
 
 int64 Display::InternalDisplayId() {
   return internal_display_id_;
 }
 
 void Display::SetInternalDisplayId(int64 internal_display_id) {
   internal_display_id_ = internal_display_id;
 }
 
 }  // namespace gfx