gpu: Introduce separate mojo api for failed initialization.

gpu: Introduce separate mojo api for failed initialization.

Introduce a separate DidFailInitialization() api to notify the host when
gpu initialization fails. Update the DidInitialize() api to take in the
GpuFeatureInfo as well as the GPUInfo.

BUG=643746
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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

Review-Url: https://codereview.chromium.org/2781553002
Cr-Commit-Position: refs/heads/master@{#459909}
Committed: https://chromium.googlesource.com/chromium/src/+/a235d7cf5d45775a50e0b811a7dd3a411494cd64

[sadrul, 2017-03-27 15:31:36]

Description was changed from

==========
gpu: Introduce separate mojo api for failed initialization.

Introduce a separate DidFailInitialization() api to notify the host when
gpu initialization fails. Update the DidInitialize() api to take in the
GpuFeatureInfo as well as the GPUInfo.

BUG=...
==========

to

==========
gpu: Introduce separate mojo api for failed initialization.

Introduce a separate DidFailInitialization() api to notify the host when
gpu initialization fails. Update the DidInitialize() api to take in the
GpuFeatureInfo as well as the GPUInfo.

BUG=...
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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
==========

[sadrul, 2017-03-27 15:31:38]

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

[commit-bot: I haz the power, 2017-03-27 15:32:20]

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

[sadrul, 2017-03-27 16:23:37]

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

[commit-bot: I haz the power, 2017-03-27 16:24:18]

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

[sadrul, 2017-03-27 16:24:25]

Description was changed from

==========
gpu: Introduce separate mojo api for failed initialization.

Introduce a separate DidFailInitialization() api to notify the host when
gpu initialization fails. Update the DidInitialize() api to take in the
GpuFeatureInfo as well as the GPUInfo.

BUG=...
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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

==========
gpu: Introduce separate mojo api for failed initialization.

Introduce a separate DidFailInitialization() api to notify the host when
gpu initialization fails. Update the DidInitialize() api to take in the
GpuFeatureInfo as well as the GPUInfo.

BUG=643746
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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
==========

[sadrul, 2017-03-27 16:25:47]

sadrul@chromium.org changed reviewers:
+ piman@chromium.org, tsepez@chromium.org

[sadrul, 2017-03-27 16:25:48]

tsepez@: Please review the mojom and the struct trait changes.

piman@: Please review the mojom api change (and the rest).

Thank you!

[piman, 2017-03-27 17:59:29]

lgtm

[commit-bot: I haz the power, 2017-03-27 18:19:28]

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

[commit-bot: I haz the power, 2017-03-27 18:19:29]

Dry run: Try jobs failed on following builders:
  mac_optional_gpu_tests_rel on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_optional_gpu_...)

[Tom Sepez, 2017-03-27 18:45:06]

lgtm

[sadrul, 2017-03-27 20:18:40]

The CQ bit was checked by sadrul@chromium.org

[commit-bot: I haz the power, 2017-03-27 20:19:30]

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-03-27 22:32:04]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490645920930760,
"parent_rev": "2372de8b97b72d62e9ece294494a351ff633a2d8", "commit_rev":
"a235d7cf5d45775a50e0b811a7dd3a411494cd64"}

[commit-bot: I haz the power, 2017-03-27 22:34:12]

Description was changed from

==========
gpu: Introduce separate mojo api for failed initialization.

Introduce a separate DidFailInitialization() api to notify the host when
gpu initialization fails. Update the DidInitialize() api to take in the
GpuFeatureInfo as well as the GPUInfo.

BUG=643746
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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

==========
gpu: Introduce separate mojo api for failed initialization.

Introduce a separate DidFailInitialization() api to notify the host when
gpu initialization fails. Update the DidInitialize() api to take in the
GpuFeatureInfo as well as the GPUInfo.

BUG=643746
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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

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

[commit-bot: I haz the power, 2017-03-27 22:34:14]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/a235d7cf5d45775a50e0b811a7dd...