Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(176)

Side by Side Diff: content/renderer/render_widget.cc

Issue 2279873002: Revert of Use ScaleToEnclosingRect when converting the viewport to window. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/renderer/render_widget.h" 5 #include "content/renderer/render_widget.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/auto_reset.h" 10 #include "base/auto_reset.h"
(...skipping 1497 matching lines...) Expand 10 before | Expand all | Expand 10 after
1508 return; 1508 return;
1509 } 1509 }
1510 composition_character_bounds_ = character_bounds; 1510 composition_character_bounds_ = character_bounds;
1511 composition_range_ = range; 1511 composition_range_ = range;
1512 Send(new InputHostMsg_ImeCompositionRangeChanged( 1512 Send(new InputHostMsg_ImeCompositionRangeChanged(
1513 routing_id(), composition_range_, composition_character_bounds_)); 1513 routing_id(), composition_range_, composition_character_bounds_));
1514 } 1514 }
1515 1515
1516 void RenderWidget::convertViewportToWindow(blink::WebRect* rect) { 1516 void RenderWidget::convertViewportToWindow(blink::WebRect* rect) {
1517 if (IsUseZoomForDSFEnabled()) { 1517 if (IsUseZoomForDSFEnabled()) {
1518 float reverse = 1 / GetOriginalDeviceScaleFactor();
1518 // TODO(oshima): We may need to allow pixel precision here as the the 1519 // TODO(oshima): We may need to allow pixel precision here as the the
1519 // anchor element can be placed at half pixel. 1520 // anchor element can be placed at half pixel.
1520 *rect = 1521 gfx::Rect window_rect =
1521 gfx::ScaleToEnclosingRect(*rect, 1.f / GetOriginalDeviceScaleFactor()); 1522 gfx::ScaleToEnclosedRect(gfx::Rect(*rect), reverse);
1523 rect->x = window_rect.x();
1524 rect->y = window_rect.y();
1525 rect->width = window_rect.width();
1526 rect->height = window_rect.height();
1522 } 1527 }
1523 } 1528 }
1524 1529
1525 void RenderWidget::convertWindowToViewport(blink::WebFloatRect* rect) { 1530 void RenderWidget::convertWindowToViewport(blink::WebFloatRect* rect) {
1526 if (IsUseZoomForDSFEnabled()) { 1531 if (IsUseZoomForDSFEnabled()) {
1527 rect->x *= GetOriginalDeviceScaleFactor(); 1532 rect->x *= GetOriginalDeviceScaleFactor();
1528 rect->y *= GetOriginalDeviceScaleFactor(); 1533 rect->y *= GetOriginalDeviceScaleFactor();
1529 rect->width *= GetOriginalDeviceScaleFactor(); 1534 rect->width *= GetOriginalDeviceScaleFactor();
1530 rect->height *= GetOriginalDeviceScaleFactor(); 1535 rect->height *= GetOriginalDeviceScaleFactor();
1531 } 1536 }
(...skipping 528 matching lines...) Expand 10 before | Expand all | Expand 10 after
2060 void RenderWidget::requestPointerUnlock() { 2065 void RenderWidget::requestPointerUnlock() {
2061 mouse_lock_dispatcher_->UnlockMouse(webwidget_mouse_lock_target_.get()); 2066 mouse_lock_dispatcher_->UnlockMouse(webwidget_mouse_lock_target_.get());
2062 } 2067 }
2063 2068
2064 bool RenderWidget::isPointerLocked() { 2069 bool RenderWidget::isPointerLocked() {
2065 return mouse_lock_dispatcher_->IsMouseLockedTo( 2070 return mouse_lock_dispatcher_->IsMouseLockedTo(
2066 webwidget_mouse_lock_target_.get()); 2071 webwidget_mouse_lock_target_.get());
2067 } 2072 }
2068 2073
2069 } // namespace content 2074 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698