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

Unified Diff: chrome/browser/automation/automation_provider.cc

Issue 7317: Change all ConvertPointTo* methods to use gfx::Point instead of CPoint.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 12 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/autocomplete/autocomplete_popup.cc ('k') | chrome/browser/automation/ui_controls.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/automation/automation_provider.cc
===================================================================
--- chrome/browser/automation/automation_provider.cc (revision 3347)
+++ chrome/browser/automation/automation_provider.cc (working copy)
@@ -1148,13 +1148,13 @@
ChromeViews::View* view = root_view->GetViewByID(view_id);
if (view) {
succeeded = true;
- CPoint point(0, 0);
+ gfx::Point point;
if (screen_coordinates)
ChromeViews::View::ConvertPointToScreen(view, &point);
else
ChromeViews::View::ConvertPointToView(view, root_view, &point);
view->GetLocalBounds(&bounds, false);
- bounds.MoveToXY(point.x, point.y);
+ bounds.MoveToXY(point.x(), point.y());
}
}
}
@@ -1182,9 +1182,9 @@
// the drag code moved away from using mouse event locations was because
// our conversion to screen location doesn't work well with multiple
// monitors, so this only works reliably in a single monitor setup.
- CPoint screen_location = CPoint(point_.x, point_.y);
+ gfx::Point screen_location(point_.x, point_.y);
view_->ConvertPointToScreen(view_, &screen_location);
- ::SetCursorPos(screen_location.x, screen_location.y);
+ ::SetCursorPos(screen_location.x(), screen_location.y());
switch (type_) {
case ChromeViews::Event::ET_MOUSE_PRESSED:
view_->OnMousePressed(event);
« no previous file with comments | « chrome/browser/autocomplete/autocomplete_popup.cc ('k') | chrome/browser/automation/ui_controls.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698