Call dispatcher's releaseRegistration when done with registrations

Call dispatcher's releaseRegistration when done with them

With this series of patches, the BackgroundSyncManager expects clients
to call ReleaseRegistration once no longer referencing them. This CL
has Blink call releaseRegistration when the SyncRegistration and
PeriodicRegistration are deleted.

Patch 1 [Blink]: this
Patch 2 [Browser]: https://codereview.chromium.org/1282013004
Patch 3 [Blink cleanup]: https://codereview.chromium.org/1359653002

BUG=502214
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200504

[jkarlin, 2015-08-10 19:12:26]

jkarlin@chromium.org changed reviewers:
+ iclelland@chromium.org

[jkarlin, 2015-08-10 19:12:41]

iclelland: PTAL, thanks!

[iclelland, 2015-08-12 04:10:16]

lgtm

[jkarlin, 2015-08-12 16:02:10]

I added one more function to WebSyncProvider in PS2. Can you take another look
Ian? Thanks!

[jkarlin, 2015-08-13 12:01:55]

jkarlin@chromium.org changed reviewers:
+ jsbell@chromium.org

[jkarlin, 2015-08-13 12:01:55]

jsbell@: PTAL for committer approval, thanks!

[iclelland, 2015-08-13 12:22:54]

On 2015/08/12 16:02:10, jkarlin wrote:
> I added one more function to WebSyncProvider in PS2. Can you take another look
> Ian? Thanks!

Still lgtm; thanks for the heads-up

[jsbell, 2015-08-13 18:59:28]

lgtm

[jkarlin, 2015-08-13 19:12:08]

The CQ bit was checked by jkarlin@chromium.org

[jkarlin, 2015-08-13 19:12:09]

The patchset sent to the CQ was uploaded after l-g-t-m from
iclelland@chromium.org
Link to the patchset: https://codereview.chromium.org/1279323002/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2015-08-13 19:12:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1279323002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1279323002/60001

[commit-bot: I haz the power, 2015-08-13 21:06:31]

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

[commit-bot: I haz the power, 2015-08-13 21:06:31]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jkarlin, 2015-08-13 21:35:53]

The CQ bit was checked by jkarlin@chromium.org

[commit-bot: I haz the power, 2015-08-13 21:36:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1279323002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1279323002/60001

[commit-bot: I haz the power, 2015-08-14 01:57:23]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200504

[cbiesinger, 2015-08-14 16:05:36]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/1287343004/ by cbiesinger@chromium.org.

The reason for reverting is: Reverting because it causes a crash in
http/tests/background_sync/oneshot.html


https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Leak/b...
https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Lea....

[jkarlin, 2015-08-17 11:20:09]

Reland in https://codereview.chromium.org/1290073003