Fetch the surface from GpuSurfaceTracker for mojo AndroidOverlay.

Fetch the surface from GpuSurfaceTracker for mojo AndroidOverlay.

This CL makes the mojo AndroidOverlay client fetch the Android
surface from GpuSurfaceTracker when notified by the service that
the overlay is ready.

This also updates the tests to register a surface, and verify
that the same one is returned by the mojo client.

BUG=719591
Review-Url: https://codereview.chromium.org/2867633002
Cr-Commit-Position: refs/heads/master@{#470106}
Committed: https://chromium.googlesource.com/chromium/src/+/e7c8abfdf3ce705d4070c4d5c457381dbee89a84

[liberato (no reviews please), 2017-05-07 05:59:57]

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

[commit-bot: I haz the power, 2017-05-07 06:00: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, 2017-05-07 06:04:43]

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

[commit-bot: I haz the power, 2017-05-07 06:04:44]

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

[liberato (no reviews please), 2017-05-08 04:51:59]

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

[commit-bot: I haz the power, 2017-05-08 04:52:25]

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, 2017-05-08 06:19:21]

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

[commit-bot: I haz the power, 2017-05-08 06:19:22]

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

[liberato (no reviews please), 2017-05-08 16:52:38]

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

[commit-bot: I haz the power, 2017-05-08 16:53:04]

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, 2017-05-08 16:59:05]

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

[commit-bot: I haz the power, 2017-05-08 16:59:06]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[liberato (no reviews please), 2017-05-08 16:59:54]

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

[commit-bot: I haz the power, 2017-05-08 17:00:17]

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, 2017-05-08 17:05:18]

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

[commit-bot: I haz the power, 2017-05-08 17:05:19]

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

[liberato (no reviews please), 2017-05-08 17:15:11]

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

[commit-bot: I haz the power, 2017-05-08 17:15:37]

Dry run: CQ is trying da patch.

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

[liberato (no reviews please), 2017-05-08 17:27:24]

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

[commit-bot: I haz the power, 2017-05-08 17:27:52]

Dry run: CQ is trying da patch.

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

[liberato (no reviews please), 2017-05-08 17:30:52]

Description was changed from

==========
Fetch the surface from GpuSurfaceTracker for mojo AndroidOverlay.

This CL makes the mojo AndroidOverlay client fetch the Android
surface from GpuSurfaceTracker when notified by the service that
the overlay is ready.

BUG=
==========

to

==========
Fetch the surface from GpuSurfaceTracker for mojo AndroidOverlay.

This CL makes the mojo AndroidOverlay client fetch the Android
surface from GpuSurfaceTracker when notified by the service that
the overlay is ready.

This also updates the tests to register a surface, and verify
that the same one is returned by the mojo client.

BUG=719591
==========

[liberato (no reviews please), 2017-05-08 17:30:52]

liberato@chromium.org changed reviewers:
+ tguilbert@chromium.org

[liberato (no reviews please), 2017-05-08 17:32:25]

without this, the mojo client gets the surface id from the service, then does
nothing.

thanks
-fl

[tguilbert, 2017-05-08 19:52:15]

LGTM

[commit-bot: I haz the power, 2017-05-08 20:11:05]

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

[commit-bot: I haz the power, 2017-05-08 20:11:06]

Dry run: This issue passed the CQ dry run.

[liberato (no reviews please), 2017-05-08 20:14:57]

The CQ bit was checked by liberato@chromium.org

[commit-bot: I haz the power, 2017-05-08 20:16:00]

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, 2017-05-08 20:27:07]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1494274497197130,
"parent_rev": "e9f34ed893dd0ab15ac4834becee26c4886620d7", "commit_rev":
"e7c8abfdf3ce705d4070c4d5c457381dbee89a84"}

[commit-bot: I haz the power, 2017-05-08 20:27:25]

Description was changed from

==========
Fetch the surface from GpuSurfaceTracker for mojo AndroidOverlay.

This CL makes the mojo AndroidOverlay client fetch the Android
surface from GpuSurfaceTracker when notified by the service that
the overlay is ready.

This also updates the tests to register a surface, and verify
that the same one is returned by the mojo client.

BUG=719591
==========

to

==========
Fetch the surface from GpuSurfaceTracker for mojo AndroidOverlay.

This CL makes the mojo AndroidOverlay client fetch the Android
surface from GpuSurfaceTracker when notified by the service that
the overlay is ready.

This also updates the tests to register a surface, and verify
that the same one is returned by the mojo client.

BUG=719591

Review-Url: https://codereview.chromium.org/2867633002
Cr-Commit-Position: refs/heads/master@{#470106}
Committed:
https://chromium.googlesource.com/chromium/src/+/e7c8abfdf3ce705d4070c4d5c457...
==========

[commit-bot: I haz the power, 2017-05-08 20:27:26]

Committed patchset #6 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/e7c8abfdf3ce705d4070c4d5c457...