New issue
Advanced search Search tips

Issue 841899 link

Starred by 4 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , iOS , Chrome , Mac , Fuchsia
Pri: 3
Type: Bug



Sign in to add a comment

Deprecate base::ResetAndReturn() in favor of std::move().

Project Member Reported by b...@chromium.org, May 10 2018

Issue description

See discussion at
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/PJOXupl9ztA and on https://crrev.com/c/1053831.  This bug is to track cleanup effort, if any.  For convenience, the query https://cs.chromium.org/search/?q=%5CbResetAndReturn%5Cb can be used to track remaining usage of ResetAndReturn.
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 10 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1532745a1335e4cce4677b28c3090ea407dd5aa0

commit 1532745a1335e4cce4677b28c3090ea407dd5aa0
Author: Bence Béky <bnc@chromium.org>
Date: Thu May 10 20:59:07 2018

Discourage base::ResetAndReturn().

See discussion at
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/PJOXupl9ztA

Bug: 841899
Change-Id: I58158270f304c5e6d50a85d47e1d3f599b5870d5
Reviewed-on: https://chromium-review.googlesource.com/1053831
Commit-Queue: Bence Béky <bnc@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557665}
[modify] https://crrev.com/1532745a1335e4cce4677b28c3090ea407dd5aa0/base/callback_helpers.h
[modify] https://crrev.com/1532745a1335e4cce4677b28c3090ea407dd5aa0/docs/callback.md

Sign in to add a comment