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

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

Issue 9111032: Change over IgnoreReturn to IgnoreResult. remove IgnoreReturn. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix it for realz Created 8 years, 11 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 | « base/bind_helpers.h ('k') | chrome/browser/browser_process_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/automation/automation_provider_gtk.cc
diff --git a/chrome/browser/automation/automation_provider_gtk.cc b/chrome/browser/automation/automation_provider_gtk.cc
index 60396a3e8b70b3735c832a42db34b1e7aa299c81..46f407ae4985e698ddbde3103210d5d5df6e2eeb 100644
--- a/chrome/browser/automation/automation_provider_gtk.cc
+++ b/chrome/browser/automation/automation_provider_gtk.cc
@@ -53,12 +53,12 @@ void AutomationProvider::WindowSimulateDrag(
// Create a nested stack of tasks to run.
base::Closure drag_response_cb = base::Bind(
&SendWindowDragResponse, make_scoped_refptr(this), reply_message);
- base::Closure move_chain_cb = base::IgnoreReturn<bool>(
- base::Bind(&ui_controls::SendMouseEventsNotifyWhenDone,
- ui_controls::LEFT, ui_controls::UP, drag_response_cb));
- move_chain_cb = base::IgnoreReturn<bool>(
- base::Bind(&ui_controls::SendMouseEventsNotifyWhenDone,
- ui_controls::LEFT, ui_controls::UP, move_chain_cb));
+ base::Closure move_chain_cb = base::Bind(
+ base::IgnoreResult(&ui_controls::SendMouseEventsNotifyWhenDone),
+ ui_controls::LEFT, ui_controls::UP, drag_response_cb);
+ move_chain_cb = base::Bind(
+ base::IgnoreResult(&ui_controls::SendMouseEventsNotifyWhenDone),
+ ui_controls::LEFT, ui_controls::UP, move_chain_cb);
for (size_t i = drag_path.size() - 1; i > 0; --i) {
// Smooth out the mouse movements by adding intermediate points. This
// better simulates a real user drag.
@@ -67,16 +67,16 @@ void AutomationProvider::WindowSimulateDrag(
int half_step_x = (dest_x + drag_path[i - 1].x() + x) / 2;
int half_step_y = (dest_y + drag_path[i - 1].y() + y) / 2;
- move_chain_cb = base::IgnoreReturn<bool>(
- base::Bind(&ui_controls::SendMouseMoveNotifyWhenDone, dest_x, dest_y,
- move_chain_cb));
- move_chain_cb = base::IgnoreReturn<bool>(
- base::Bind(&ui_controls::SendMouseMoveNotifyWhenDone, half_step_x,
- half_step_y, move_chain_cb));
+ move_chain_cb = base::Bind(
+ base::IgnoreResult(&ui_controls::SendMouseMoveNotifyWhenDone),
+ dest_x, dest_y, move_chain_cb);
+ move_chain_cb = base::Bind(
+ base::IgnoreResult(&ui_controls::SendMouseMoveNotifyWhenDone),
+ half_step_x, half_step_y, move_chain_cb);
}
- move_chain_cb = base::IgnoreReturn<bool>(
- base::Bind(&ui_controls::SendMouseEventsNotifyWhenDone,
- ui_controls::LEFT, ui_controls::DOWN, move_chain_cb));
+ move_chain_cb = base::Bind(
+ base::IgnoreResult(&ui_controls::SendMouseEventsNotifyWhenDone),
+ ui_controls::LEFT, ui_controls::DOWN, move_chain_cb);
ui_controls::SendMouseMoveNotifyWhenDone(x + drag_path[0].x(),
y + drag_path[0].y(),
« no previous file with comments | « base/bind_helpers.h ('k') | chrome/browser/browser_process_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698