Call IDirectManipulationViewport::Abandon before releasing viewport.

Call IDirectManipulationViewport::Abandon before releasing viewport.

Otherwise there's a reference cycle and about 4KB is leaked.

BUG=636176
Committed: https://crrev.com/d48b65a4b84e0ca652bf5ab39abaf0809ba4b6cd
Cr-Commit-Position: refs/heads/master@{#410936}

[jbauman, 2016-08-10 01:13:59]

The CQ bit was checked by jbauman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-10 01:14:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jbauman, 2016-08-10 01:14:59]

Description was changed from

==========
Call IDirectManipulationViewport::Abandon before releasing it.

Otherwise there's a reference cycle and about 4KB is leaked.

BUG=636176
==========

to

==========
Call IDirectManipulationViewport::Abandon before releasing viewport.

Otherwise there's a reference cycle and about 4KB is leaked.

BUG=636176
==========

[jbauman, 2016-08-10 01:43:57]

jbauman@chromium.org changed reviewers:
+ ananta@chromium.org, sky@chromium.org

[commit-bot: I haz the power, 2016-08-10 02:06:22]

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

[commit-bot: I haz the power, 2016-08-10 02:06:23]

Dry run: This issue passed the CQ dry run.

[ananta, 2016-08-10 02:41:11]

lgtm

[sky, 2016-08-10 02:45:25]

Rubber stamp LGTM - I'm not familiar with this code at all. I think one of you
should be an OWNER.

[jbauman, 2016-08-10 03:03:37]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2016-08-10 03:04:01]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-10 03:07:34]

Description was changed from

==========
Call IDirectManipulationViewport::Abandon before releasing viewport.

Otherwise there's a reference cycle and about 4KB is leaked.

BUG=636176
==========

to

==========
Call IDirectManipulationViewport::Abandon before releasing viewport.

Otherwise there's a reference cycle and about 4KB is leaked.

BUG=636176
==========

[commit-bot: I haz the power, 2016-08-10 03:07:47]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-10 03:09:45]

Description was changed from

==========
Call IDirectManipulationViewport::Abandon before releasing viewport.

Otherwise there's a reference cycle and about 4KB is leaked.

BUG=636176
==========

to

==========
Call IDirectManipulationViewport::Abandon before releasing viewport.

Otherwise there's a reference cycle and about 4KB is leaked.

BUG=636176

Committed: https://crrev.com/d48b65a4b84e0ca652bf5ab39abaf0809ba4b6cd
Cr-Commit-Position: refs/heads/master@{#410936}
==========

[commit-bot: I haz the power, 2016-08-10 03:09:46]

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