Utilize new mojo::UnwrapPlatformFile function

Utilize new mojo::UnwrapPlatformFile function

Migrate to public api, which should be more stable than embedder api.

BUG=none
TEST=arc video still works

Committed: https://crrev.com/8dbe47eb360b8cdec212900dc46da3e78ce3e018
Cr-Commit-Position: refs/heads/master@{#396782}

[kcwu, 2016-05-30 10:46:21]

Description was changed from

==========
Utilize new mojo::UnwrapPlatformFile function

BUG=none
TEST=arc video still works
==========

to

==========
Utilize new mojo::UnwrapPlatformFile function

Migrate to public api, which should be more than embedder api.

BUG=none
TEST=arc video still works
==========

[kcwu, 2016-05-30 10:46:21]

kcwu@chromium.org changed reviewers:
+ posciak@chromium.org, rockot@chromium.org

[kcwu, 2016-05-30 10:47:33]

Description was changed from

==========
Utilize new mojo::UnwrapPlatformFile function

Migrate to public api, which should be more than embedder api.

BUG=none
TEST=arc video still works
==========

to

==========
Utilize new mojo::UnwrapPlatformFile function

Migrate to public api, which should be more stable than embedder api.

BUG=none
TEST=arc video still works
==========

[Ken Rockot(use gerrit already), 2016-05-30 14:58:51]

lgtm

https://codereview.chromium.org/2026533002/diff/1/chrome/gpu/gpu_arc_video_se...
File chrome/gpu/gpu_arc_video_service.cc (right):

https://codereview.chromium.org/2026533002/diff/1/chrome/gpu/gpu_arc_video_se...
chrome/gpu/gpu_arc_video_service.cc:160: LOG(ERROR) <<
"PassWrappedPlatformHandle failed: " << mojo_result;
nit: update log message

[Pawel Osciak, 2016-05-31 01:10:31]

lgtm

[kcwu, 2016-05-31 03:18:15]

https://codereview.chromium.org/2026533002/diff/1/chrome/gpu/gpu_arc_video_se...
File chrome/gpu/gpu_arc_video_service.cc (right):

https://codereview.chromium.org/2026533002/diff/1/chrome/gpu/gpu_arc_video_se...
chrome/gpu/gpu_arc_video_service.cc:160: LOG(ERROR) <<
"PassWrappedPlatformHandle failed: " << mojo_result;
On 2016/05/30 14:58:51, Ken Rockot wrote:
> nit: update log message

Done.

[kcwu, 2016-05-31 03:18:19]

The CQ bit was checked by kcwu@chromium.org

[kcwu, 2016-05-31 03:18:19]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org,
posciak@chromium.org
Link to the patchset: https://codereview.chromium.org/2026533002/#ps20001
(title: "fix comment")

[commit-bot: I haz the power, 2016-05-31 03:18:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2026533002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2026533002/20001

[commit-bot: I haz the power, 2016-05-31 04:15:01]

Description was changed from

==========
Utilize new mojo::UnwrapPlatformFile function

Migrate to public api, which should be more stable than embedder api.

BUG=none
TEST=arc video still works
==========

to

==========
Utilize new mojo::UnwrapPlatformFile function

Migrate to public api, which should be more stable than embedder api.

BUG=none
TEST=arc video still works
==========

[commit-bot: I haz the power, 2016-05-31 04:15:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-31 04:16:29]

Description was changed from

==========
Utilize new mojo::UnwrapPlatformFile function

Migrate to public api, which should be more stable than embedder api.

BUG=none
TEST=arc video still works
==========

to

==========
Utilize new mojo::UnwrapPlatformFile function

Migrate to public api, which should be more stable than embedder api.

BUG=none
TEST=arc video still works

Committed: https://crrev.com/8dbe47eb360b8cdec212900dc46da3e78ce3e018
Cr-Commit-Position: refs/heads/master@{#396782}
==========

[commit-bot: I haz the power, 2016-05-31 04:16:30]

Patchset 2 (id:??) landed as
https://crrev.com/8dbe47eb360b8cdec212900dc46da3e78ce3e018
Cr-Commit-Position: refs/heads/master@{#396782}