Remove dummy AddRef/Release from //ipc/ipc_sync_channel_unittest.cc
`Worker` class in ipc_sync_channel_unittest.cc has a dummy AddRef()
and Release() to pretend a ref-counted object. This type of objects
are confusing on base:::Bind. E.g., the object is wrapped with
scoped_refptr if it's passed as a receiver, OTOH, it's unretained if
it's passed as a non-receiver parameter.
This CL removes the dummy AddRef()/Release() and adds explicit
lifetime annotation where needed.
Bug: 737010
Change-Id: I242156184a246f572b98bec156206222396ec40e
Reviewed-on: https://chromium-review.googlesource.com/558310
Reviewed-by: Ken Rockot <rockot@chromium.org>
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#484392}
CrOS-Libchrome-Original-Commit: 9805c860a67407267ff9c3517c29aacf2238a014
1 file changed