Fix leak in ServiceWorkerVersionTest.RepeatedlyObserveStatusChanges

Fix leak in ServiceWorkerVersionTest.RepeatedlyObserveStatusChanges

In r262636 I deleted the Shutdown() methods from SWRegistration and
SWVersion, but it caused a leak in RepeatedlyObserveStatusChanges.

In the previous code the status change callback (ObserveStatusChanges)
holds a ref to ServiceWorkerVersion, which was ok when we had Shutdown()
(as we could explicitly clear the status change callback there), but
now we don't, so we have cyclic reference there.

I just changed make_scoped_refptr(version) to base::Unretained(version)
when we pass the version to the callback, so that the callback no longer
holds a ref to the Version.

BUG=361504
TEST=ServiceWorkerVersionTest.RepeatedlyObserveStatusChanges with asan/lsan

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=262933

[kinuko, 2014-04-09 10:04:21]

PTL

[falken, 2014-04-09 20:09:01]

lgtm

[falken, 2014-04-09 20:11:49]

+michaeln for OWNER review

[kinuko, 2014-04-10 00:53:39]

The CQ bit was checked by kinuko@chromium.org

[commit-bot: I haz the power, 2014-04-10 00:53:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinuko@chromium.org/230373003/1

[commit-bot: I haz the power, 2014-04-10 01:16:19]

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

[commit-bot: I haz the power, 2014-04-10 01:16:20]

Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_chromeos_rel

[kinuko, 2014-04-10 01:31:00]

The CQ bit was checked by kinuko@chromium.org

[commit-bot: I haz the power, 2014-04-10 01:31:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinuko@chromium.org/230373003/1

[commit-bot: I haz the power, 2014-04-10 05:29:43]

Change committed as 262933