Make NavigatorConnectProvider finalize its owned callback objects.

Make NavigatorConnectProvider finalize its owned callback objects.

Add missing IDMapOwnPointer annotation to have ConnectCallback objects
be destructed and finalized once done with, as the provider owns these.

Needed with Oilpan enabled to prevent persistent strong references
from going missing, and not being cleared in a timely fashion.

R=mek,mkwst
BUG=445388
Committed: https://crrev.com/419f0760c569b7837b42636a61e0bace316e65a5
Cr-Commit-Position: refs/heads/master@{#309713}

[sof, 2014-12-29 16:47:50]

sigbjornf@opera.com changed reviewers:
+ jochen@chromium.org, mek@chromium.org, mkwst@chromium.org, palmer@chromium.org

[sof, 2014-12-29 16:47:50]

Please take a look.

Fixes immediate cause of some Oilpan untowardness.

[Mike West, 2014-12-29 16:53:52]

LGTM. I'm not an owner here, but I think this is safe to TBR to mek@ to fix the
breakage.

[Marijn Kruisselbrink, 2014-12-29 16:54:29]

On 2014/12/29 16:47:50, sof wrote:
> Please take a look.
> 
> Fixes immediate cause of some Oilpan untowardness.
Oops, good catch. LGTM

[haraken, 2014-12-29 17:08:40]

LGTM

[sof, 2014-12-29 18:46:21]

Thanks all for the quick reviews; best wishes for 2015.

[sof, 2014-12-29 18:46:27]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-12-29 18:47:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/827953002/1

[commit-bot: I haz the power, 2014-12-29 19:34:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-12-29 19:35:49]

Patchset 1 (id:??) landed as
https://crrev.com/419f0760c569b7837b42636a61e0bace316e65a5
Cr-Commit-Position: refs/heads/master@{#309713}