Reland Call dispatcher's releaseRegistration when done with registrations

Reland Call dispatcher's releaseRegistration when done with registrations

Reland of https://codereview.chromium.org/1279323002/

OilPan objects can be garbage collected at any time, so it's necessary to check the pointers.

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

[jkarlin, 2015-08-17 11:54:54]

Patchset #1 (id:1) has been deleted

[jkarlin, 2015-08-17 14:51:39]

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

[jkarlin, 2015-08-17 14:51:39]

jsbell: PTAL, thanks!

[jsbell, 2015-08-17 16:34:41]

lgtm, sorry for not checking they were Oilpan objects before.

[jsbell, 2015-08-17 16:34:45]

The CQ bit was unchecked by jsbell@chromium.org

[jsbell, 2015-08-17 16:34:45]

The CQ bit was checked by jsbell@chromium.org

[jsbell, 2015-08-17 16:34:45]

The CQ bit was checked by jsbell@chromium.org

[commit-bot: I haz the power, 2015-08-17 16:34:55]

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

[commit-bot: I haz the power, 2015-08-17 18:41:03]

Committed patchset #2 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200657

[leviw_travelin_and_unemployed, 2015-08-17 20:10:37]

A revert of this CL (patchset #2 id:40001) has been created in
https://codereview.chromium.org/1288893004/ by leviw@chromium.org.

The reason for reverting is: Speculatively reverting to fix crashes in
http/tests/cache/cached-main-resource.html

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpec....

[leviw_travelin_and_unemployed, 2015-08-17 21:02:26]

On 2015/08/17 at 20:10:37, leviw wrote:
> A revert of this CL (patchset #2 id:40001) has been created in
https://codereview.chromium.org/1288893004/ by leviw@chromium.org.
> 
> The reason for reverting is: Speculatively reverting to fix crashes in
http/tests/cache/cached-main-resource.html
> 
>
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpec....

FYI: this also appears to have caused the leak bots to crash on a different
test:
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=htt...

[jkarlin, 2015-08-18 14:53:15]

Fix for crash: https://codereview.chromium.org/1302543002/
Reland: https://codereview.chromium.org/1299903002/