Skip to content

Commit

Permalink
Mojo: Remove bogus TODO(darin) comment.
Browse files Browse the repository at this point in the history
BUG=384578
TBR=sky@chromium.org

Review URL: https://codereview.chromium.org/334263006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277533 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
darin@chromium.org committed Jun 16, 2014
1 parent afbf7dd commit c784a70
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions mojo/public/cpp/utility/lib/run_loop.cc
Original file line number Diff line number Diff line change
Expand Up @@ -153,11 +153,6 @@ bool RunLoop::NotifyDeadlineExceeded() {

// Make a copy in case someone tries to add/remove new handlers as part of
// notifying.
//
// TODO(darin): This does not protect against removal of handlers! After a
// call to OnHandleError, |cloned_handlers| could contain an invalid pointer
// to a RunLoopHandler! See http://crbug.com/384578.
//
const HandleToHandlerData cloned_handlers(handler_data_);
const MojoTimeTicks now(GetTimeTicksNow());
for (HandleToHandlerData::const_iterator i = cloned_handlers.begin();
Expand Down

0 comments on commit c784a70

Please sign in to comment.