Make SurfaceFactory lifetime match frame_sink_id registration in OffscreenCanvasCompositorFrameSink

Make SurfaceFactory lifetime match frame_sink_id registration in OffscreenCanvasCompositorFrameSink

This will allow further simplification/refactoring.

BUG=657959
Committed: https://crrev.com/ba306aec9d9b09a427c4f95f095a8c039c56691a
Cr-Commit-Position: refs/heads/master@{#430667}

[piman, 2016-11-07 22:38:40]

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

[commit-bot: I haz the power, 2016-11-07 22:39:10]

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

[piman, 2016-11-07 22:39:20]

piman@chromium.org changed reviewers:
+ xidachen@chromium.org

[commit-bot: I haz the power, 2016-11-07 23:35:22]

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

[commit-bot: I haz the power, 2016-11-07 23:35:23]

Dry run: This issue passed the CQ dry run.

[xidachen, 2016-11-08 01:25:41]

xidachen@chromium.org changed reviewers:
+ xlai@chromium.org

[xidachen, 2016-11-08 01:25:42]

This change looks good.

xlai@: you are more familiar with this part, could you please take a look? Thank
you.

[xlai (Olivia), 2016-11-08 15:51:12]

On 2016/11/08 01:25:42, xidachen wrote:
> This change looks good.
> 
> xlai@: you are more familiar with this part, could you please take a look?
Thank
> you.

I think it lgtm. Although the destruction order of SurfaceFactory and
SurfaceManager is
somewhat unpredictable, but the functions in
OffscreenCanvasCompositorFrameSink's 
destructor seem to handle these well.

[piman, 2016-11-08 18:29:02]

The CQ bit was checked by piman@chromium.org

[commit-bot: I haz the power, 2016-11-08 18:29:36]

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-11-08 18:37:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-08 19:08:32]

Description was changed from

==========
Make SurfaceFactory lifetime match frame_sink_id registration in
OffscreenCanvasCompositorFrameSink

This will allow further simplification/refactoring.

BUG=657959
==========

to

==========
Make SurfaceFactory lifetime match frame_sink_id registration in
OffscreenCanvasCompositorFrameSink

This will allow further simplification/refactoring.

BUG=657959

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

[commit-bot: I haz the power, 2016-11-08 19:08:33]

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