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

Unified Diff: ash/wm/session_state_animator_impl.cc

Issue 2867673004: Use OnceCallback on Mojo interfaces in //ash (Closed)
Patch Set: count -> container_count Created 3 years, 7 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
Index: ash/wm/session_state_animator_impl.cc
diff --git a/ash/wm/session_state_animator_impl.cc b/ash/wm/session_state_animator_impl.cc
index afe0bbfa31d667ca8e34e3e38784edac4194dca0..1085bc7fc2f4522ba75384016310fb08a7e68e2f 100644
--- a/ash/wm/session_state_animator_impl.cc
+++ b/ash/wm/session_state_animator_impl.cc
@@ -4,6 +4,7 @@
#include "ash/wm/session_state_animator_impl.h"
+#include <utility>
#include <vector>
#include "ash/public/cpp/shell_window_ids.h"
@@ -442,8 +443,8 @@ class SessionStateAnimatorImpl::AnimationSequence
public ui::LayerAnimationObserver {
public:
explicit AnimationSequence(SessionStateAnimatorImpl* animator,
- base::Closure callback)
- : SessionStateAnimator::AnimationSequence(callback),
+ base::OnceClosure callback)
+ : SessionStateAnimator::AnimationSequence(std::move(callback)),
animator_(animator),
sequences_attached_(0),
sequences_completed_(0) {}
@@ -546,11 +547,11 @@ void SessionStateAnimatorImpl::StartAnimationWithCallback(
int container_mask,
AnimationType type,
AnimationSpeed speed,
- base::Closure callback) {
+ base::OnceClosure callback) {
aura::Window::Windows containers;
GetContainers(container_mask, &containers);
base::Closure animation_done_closure =
- base::BarrierClosure(containers.size(), callback);
+ base::BarrierClosure(containers.size(), std::move(callback));
for (aura::Window::Windows::const_iterator it = containers.begin();
it != containers.end(); ++it) {
ui::LayerAnimationObserver* observer =
@@ -560,8 +561,8 @@ void SessionStateAnimatorImpl::StartAnimationWithCallback(
}
SessionStateAnimator::AnimationSequence*
-SessionStateAnimatorImpl::BeginAnimationSequence(base::Closure callback) {
- return new AnimationSequence(this, callback);
+SessionStateAnimatorImpl::BeginAnimationSequence(base::OnceClosure callback) {
+ return new AnimationSequence(this, std::move(callback));
}
bool SessionStateAnimatorImpl::IsWallpaperHidden() const {
« no previous file with comments | « ash/wm/session_state_animator_impl.h ('k') | chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698