Fix heap-use-after-free in cc::SurfaceManager::Destroy

Fix heap-use-after-free in cc::SurfaceManager::Destroy

There is no need to call DestroyAll() on surface_factory_ inside
OffscreenCanvasSurfaceImpl's destructor, because surface_factory_ is its unique
pointer and SurfaceFactory's own destructor already contains DestroyAll().

TBR=piman@chromium.org

BUG=621849
Committed: https://crrev.com/cfe3e0ebde0e9448cb5964373a6938afe229f397
Cr-Commit-Position: refs/heads/master@{#400994}

[xlai (Olivia), 2016-06-21 14:18:09]

xlai@chromium.org changed reviewers:
+ junov@chromium.org, piman@chromium.org

[xlai (Olivia), 2016-06-21 14:25:52]

Description was changed from

==========
Fix heap-use-after-free in cc::SurfaceManager::Destroy

There is no need to call DestroyAll() on surface_factory_ inside
OffscreenCanvasSurfaceImpl's destructor, because surface_factory_ is its unique
pointer and SurfaceFactory's own destructor already contains DestroyAll().

BUG=621849
==========

to

==========
Fix heap-use-after-free in cc::SurfaceManager::Destroy

There is no need to call DestroyAll() on surface_factory_ inside
OffscreenCanvasSurfaceImpl's destructor, because surface_factory_ is its unique
pointer and SurfaceFactory's own destructor already contains DestroyAll().

TBR=piman@chromium.org

BUG=621849
==========

[xlai (Olivia), 2016-06-21 14:26:07]

The CQ bit was checked by xlai@chromium.org

[commit-bot: I haz the power, 2016-06-21 14:26:19]

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

[piman, 2016-06-21 15:04:43]

lgtm

lgtm

[piman, 2016-06-21 15:04:44]

lgtm

[commit-bot: I haz the power, 2016-06-21 15:05:56]

Description was changed from

==========
Fix heap-use-after-free in cc::SurfaceManager::Destroy

There is no need to call DestroyAll() on surface_factory_ inside
OffscreenCanvasSurfaceImpl's destructor, because surface_factory_ is its unique
pointer and SurfaceFactory's own destructor already contains DestroyAll().

TBR=piman@chromium.org

BUG=621849
==========

to

==========
Fix heap-use-after-free in cc::SurfaceManager::Destroy

There is no need to call DestroyAll() on surface_factory_ inside
OffscreenCanvasSurfaceImpl's destructor, because surface_factory_ is its unique
pointer and SurfaceFactory's own destructor already contains DestroyAll().

TBR=piman@chromium.org

BUG=621849
==========

[commit-bot: I haz the power, 2016-06-21 15:05:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-21 15:07:56]

Description was changed from

==========
Fix heap-use-after-free in cc::SurfaceManager::Destroy

There is no need to call DestroyAll() on surface_factory_ inside
OffscreenCanvasSurfaceImpl's destructor, because surface_factory_ is its unique
pointer and SurfaceFactory's own destructor already contains DestroyAll().

TBR=piman@chromium.org

BUG=621849
==========

to

==========
Fix heap-use-after-free in cc::SurfaceManager::Destroy

There is no need to call DestroyAll() on surface_factory_ inside
OffscreenCanvasSurfaceImpl's destructor, because surface_factory_ is its unique
pointer and SurfaceFactory's own destructor already contains DestroyAll().

TBR=piman@chromium.org

BUG=621849

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

[commit-bot: I haz the power, 2016-06-21 15:08:00]

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