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

Side by Side Diff: base/callback_list_internal.cc

Issue 22877038: Add a CallbackRegistry class to base/ to manage callbacks (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Much clean up and removal of unneeded bits Created 7 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 | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "base/callback_list_internal.h"
6
7 #include <algorithm>
8
9 #include "base/bind.h"
10 #include "base/bind_helpers.h"
11
12 namespace base {
13 namespace internal {
14
15 CallbackListImpl::Iterator::Iterator(
16 const base::WeakPtr<CallbackListImpl>& list)
17 : list_(list),
18 index_(0) {
19 ++list_->active_iterator_count_;
20 }
21
22 CallbackListImpl::Iterator::~Iterator() {
23 if (list_ && --list_->active_iterator_count_ == 0) {
24 list_->Compact();
25 }
26 }
27
28 CallbackBase* CallbackListImpl::Iterator::GetNext() {
29 if (!list_)
30 return NULL;
31 size_t max_index = list_->callbacks_.size();
32 while (index_ < max_index && !list_->callbacks_[index_])
33 ++index_;
34 return index_ < max_index ? list_->callbacks_[index_++] : NULL;
35 }
36
37 CallbackListImpl::CallbackListImpl()
38 : active_iterator_count_(0),
39 weak_factory_(this) {}
40
41 CallbackListImpl::~CallbackListImpl() {
42 Clear();
43 }
44
45 void CallbackListImpl::Clear() {
46 if (active_iterator_count_) {
47 for (size_t i = 0; i < callbacks_.size(); ++i) {
48 CallbackBase* tmp = callbacks_[i];
49 delete tmp;
50 callbacks_[i] = NULL;
51 }
52 } else {
53 for (size_t i = 0; i < callbacks_.size(); ++i)
54 delete callbacks_[i];
55 callbacks_.clear();
56 }
57 }
58
59 void CallbackListImpl::Remove(base::internal::CallbackBase* cb) {
60 for (size_t i = 0; i < callbacks_.size(); i++) {
61 if (callbacks_[i] == cb) {
62 CallbackBase* tmp = callbacks_[i];
63 delete tmp;
64 if (active_iterator_count_) {
65 callbacks_[i] = NULL;
66 } else {
67 callbacks_.erase(callbacks_.begin() + i);
68 }
69 return;
70 }
71 }
72 }
73
74 base::Closure CallbackListImpl::Add(base::internal::CallbackBase* cb) {
75 callbacks_.push_back(cb);
76 return base::Bind(&CallbackListImpl::Remove, Unretained(this), cb);
77 }
78
79 scoped_ptr<CallbackListImpl::Iterator> CallbackListImpl::GetIterator() {
80 return make_scoped_ptr(new Iterator(weak_factory_.GetWeakPtr()));
81 }
82
83 void CallbackListImpl::Compact() {
84 callbacks_.erase(
85 std::remove(callbacks_.begin(), callbacks_.end(),
86 static_cast<base::internal::CallbackBase*>(NULL)),
87 callbacks_.end());
88 }
89
90 } // namespace internal
91 } // namespace base
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698