Transform RefCountedDeleteOnMessageLoop to RefCountedDeleteOnSequence.

Transform RefCountedDeleteOnMessageLoop to RefCountedDeleteOnSequence.

RefCountedDeleteOnSequence behaves the same way as
RefCountedDeleteOnMessageLoop when used with a SingleThreadTaskRunner.
However, it also supports SequencedTaskRunner.

TBR=boliu@chromium.org,mathp@chromium.org,rogerta@chromium.org,pkasting@chromium.org,clamy@chromium.org,rockot@chromium.org
BUG=675631
Committed: https://crrev.com/33d98b6545da7591c58a9f95c401c9bedf826c43
Cr-Commit-Position: refs/heads/master@{#440627}

[fdoray, 2016-12-21 20:13:44]

The CQ bit was checked by fdoray@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-21 20:14:24]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-21 20:22:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-21 20:22:06]

Dry run: Try jobs failed on following builders:
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[fdoray, 2016-12-22 13:20:32]

The CQ bit was checked by fdoray@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-22 13:20:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-22 14:21:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-22 14:21:26]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-12-22 14:25:16]

fdoray@chromium.org changed reviewers:
+ gab@chromium.org

[fdoray, 2016-12-22 14:25:16]

PTAL

[gab, 2016-12-22 20:32:28]

lgtm

https://codereview.chromium.org/2591963004/diff/20001/base/memory/ref_counted...
File base/memory/ref_counted_delete_on_sequence.h (right):

https://codereview.chromium.org/2591963004/diff/20001/base/memory/ref_counted...
base/memory/ref_counted_delete_on_sequence.h:1: // Copyright 2013 The Chromium
Authors. All rights reserved.
Note: preferable to tweak git cl upload --similarity=XX to get a better diff in
such situations.

[fdoray, 2016-12-23 12:49:31]

https://codereview.chromium.org/2591963004/diff/20001/base/memory/ref_counted...
File base/memory/ref_counted_delete_on_sequence.h (right):

https://codereview.chromium.org/2591963004/diff/20001/base/memory/ref_counted...
base/memory/ref_counted_delete_on_sequence.h:1: // Copyright 2013 The Chromium
Authors. All rights reserved.
On 2016/12/22 20:32:28, gab wrote:
> Note: preferable to tweak git cl upload --similarity=XX to get a better diff
in
> such situations.

Done.

[fdoray, 2016-12-23 12:52:23]

Description was changed from

==========
Transform RefCountedDeleteOnMessageLoop to RefCountedDeleteOnSequence.

RefCountedDeleteOnSequence behaves the same way as
RefCountedDeleteOnMessageLoop when used with a SingleThreadTaskRunner.
However, it also supports SequencedTaskRunner.

BUG=675631
==========

to

==========
Transform RefCountedDeleteOnMessageLoop to RefCountedDeleteOnSequence.

RefCountedDeleteOnSequence behaves the same way as
RefCountedDeleteOnMessageLoop when used with a SingleThreadTaskRunner.
However, it also supports SequencedTaskRunner.

TBR=boliu@chromium.org,mathp@chromium.org,rogerta@chromium.org,pkasting@chrom...
BUG=675631
==========

[fdoray, 2016-12-23 12:52:45]

fdoray@chromium.org changed reviewers:
+ boliu@chromium.org, clamy@chromium.org, mathp@chromium.org,
pkasting@chromium.org, rockot@chromium.org, rogerta@chromium.org

[fdoray, 2016-12-23 12:52:46]

TBR owners of affected call sites

[fdoray, 2016-12-23 12:52:58]

The CQ bit was checked by fdoray@chromium.org

[fdoray, 2016-12-23 12:52:58]

The patchset sent to the CQ was uploaded after l-g-t-m from gab@chromium.org
Link to the patchset: https://codereview.chromium.org/2591963004/#ps40001
(title: "similarity")

[commit-bot: I haz the power, 2016-12-23 12:53:06]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-23 14:02:55]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1482497578031320,
"parent_rev": "df402edb58a33046d1169d037d8264fb1edf8cdf", "commit_rev":
"d87b836aae7fcf0e69d7419db985ac340acd47d1"}

[commit-bot: I haz the power, 2016-12-23 14:03:20]

Description was changed from

==========
Transform RefCountedDeleteOnMessageLoop to RefCountedDeleteOnSequence.

RefCountedDeleteOnSequence behaves the same way as
RefCountedDeleteOnMessageLoop when used with a SingleThreadTaskRunner.
However, it also supports SequencedTaskRunner.

TBR=boliu@chromium.org,mathp@chromium.org,rogerta@chromium.org,pkasting@chrom...
BUG=675631
==========

to

==========
Transform RefCountedDeleteOnMessageLoop to RefCountedDeleteOnSequence.

RefCountedDeleteOnSequence behaves the same way as
RefCountedDeleteOnMessageLoop when used with a SingleThreadTaskRunner.
However, it also supports SequencedTaskRunner.

TBR=boliu@chromium.org,mathp@chromium.org,rogerta@chromium.org,pkasting@chrom...
BUG=675631

Review-Url: https://codereview.chromium.org/2591963004
==========

[commit-bot: I haz the power, 2016-12-23 14:03:21]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-23 14:07:16]

Description was changed from

==========
Transform RefCountedDeleteOnMessageLoop to RefCountedDeleteOnSequence.

RefCountedDeleteOnSequence behaves the same way as
RefCountedDeleteOnMessageLoop when used with a SingleThreadTaskRunner.
However, it also supports SequencedTaskRunner.

TBR=boliu@chromium.org,mathp@chromium.org,rogerta@chromium.org,pkasting@chrom...
BUG=675631

Review-Url: https://codereview.chromium.org/2591963004
==========

to

==========
Transform RefCountedDeleteOnMessageLoop to RefCountedDeleteOnSequence.

RefCountedDeleteOnSequence behaves the same way as
RefCountedDeleteOnMessageLoop when used with a SingleThreadTaskRunner.
However, it also supports SequencedTaskRunner.

TBR=boliu@chromium.org,mathp@chromium.org,rogerta@chromium.org,pkasting@chrom...
BUG=675631

Committed: https://crrev.com/33d98b6545da7591c58a9f95c401c9bedf826c43
Cr-Commit-Position: refs/heads/master@{#440627}
==========

[commit-bot: I haz the power, 2016-12-23 14:07:18]

Patchset 3 (id:??) landed as
https://crrev.com/33d98b6545da7591c58a9f95c401c9bedf826c43
Cr-Commit-Position: refs/heads/master@{#440627}