DialogSurfaceHolder implementation.

DialogSurfaceHolder implementation.

This CL follows up https://codereview.chromium.org/2178973004/
with the implementation for DialogSurfaceHolder and callbacks.
It also updates DialogSurfaceManager to create them properly.

BUG=618368
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

[liberato (no reviews please), 2016-08-05 18:36:23]

Description was changed from

==========
DialogSurfaceHolder implementation.

This CL follows up https://codereview.chromium.org/2178973004/
with the implementation for DialogSurfaceHolder and callbacks.
It also updates DialogSurfaceManager to create them properly.

BUG=618368
==========

to

==========
DialogSurfaceHolder implementation.

This CL follows up https://codereview.chromium.org/2178973004/
with the implementation for DialogSurfaceHolder and callbacks.
It also updates DialogSurfaceManager to create them properly.

BUG=618368
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[liberato (no reviews please), 2016-08-05 22:02:33]

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

[commit-bot: I haz the power, 2016-08-05 22:03:07]

Dry run: 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-05 23:10:02]

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

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

Dry run: Try jobs failed on following builders:
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)

[liberato (no reviews please), 2016-08-08 16:24:20]

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

[commit-bot: I haz the power, 2016-08-08 16:24:38]

Dry run: 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-08 19:43:04]

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

[commit-bot: I haz the power, 2016-08-08 19:43:05]

Dry run: This issue passed the CQ dry run.

[liberato (no reviews please), 2016-08-08 22:01:16]

Description was changed from

==========
DialogSurfaceHolder implementation.

This CL follows up https://codereview.chromium.org/2178973004/
with the implementation for DialogSurfaceHolder and callbacks.
It also updates DialogSurfaceManager to create them properly.

BUG=618368
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
DialogSurfaceHolder implementation.

This CL follows up https://codereview.chromium.org/2178973004/
with the implementation for DialogSurfaceHolder and callbacks.
It also updates DialogSurfaceManager to create them properly.

BUG=618368
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[liberato (no reviews please), 2016-08-08 22:01:17]

liberato@chromium.org changed reviewers:
+ dalecurtis@chromium.org, sievers@chromium.org, timav@chromium.org,
watk@chromium.org

[liberato (no reviews please), 2016-08-08 22:08:21]

Hi all.

This is 2/2 for the core DialogSurface code.  while
https://codereview.chromium.org/2178973004 is still out for review, it seems
like we're fairly close to converging.

i figured that i'd send this out now, both for context and while the previous
one is still relatively fresh in your minds.  it adds the implementation for
DialogSurfaceHolder.

sievers: it doesn't try to address activity switching yet.

thanks
-fl

[watk, 2017-01-24 00:38:28]

watk@chromium.org changed reviewers:
- watk@chromium.org