Use GetProfileName() for logging VideoCodecProfile strings.

Use GetProfileName() for logging VideoCodecProfile strings.

BUG=676022
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

Review-Url: https://codereview.chromium.org/2626023003
Cr-Commit-Position: refs/heads/master@{#444738}
Committed: https://chromium.googlesource.com/chromium/src/+/9a44ab189f977553b42b9d7706628b959289308a

[jam39, 2017-01-11 16:25:13]

Description was changed from

==========
Use GetProfileName() for logging VideoCodecProfile strings.
Use GetProfileName() for logging VideoCodecProfile strings.

BUG=676022
==========

to

==========
Use GetProfileName() for logging VideoCodecProfile strings.
Use GetProfileName() for logging VideoCodecProfile strings.

BUG=676022
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
==========

[jam39, 2017-01-11 16:27:17]

Description was changed from

==========
Use GetProfileName() for logging VideoCodecProfile strings.
Use GetProfileName() for logging VideoCodecProfile strings.

BUG=676022
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

==========
Use GetProfileName() for logging VideoCodecProfile strings.

BUG=676022
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
==========

[jam39, 2017-01-11 16:27:17]

jem456.vasishta@gmail.com changed reviewers:
+ mcasas@chromium.org, posciak@chromium.org

[jam39, 2017-01-11 16:28:28]

jem456.vasishta@gmail.com changed reviewers:
+ wuchengli@chromium.org

[jam39, 2017-01-12 12:29:15]

On 2017/01/11 16:28:28, jam39 wrote:
> mailto:jem456.vasishta@gmail.com changed reviewers:
> + mailto:wuchengli@chromium.org

PTAL

[mcasas, 2017-01-12 18:31:13]

changes lgtm (you still needs owners though)

[Pawel Osciak, 2017-01-16 02:15:35]

lgtm

[jam39, 2017-01-16 10:51:01]

The CQ bit was checked by jem456.vasishta@gmail.com

[commit-bot: I haz the power, 2017-01-16 10:51:16]

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-01-16 10:58:03]

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

[commit-bot: I haz the power, 2017-01-16 10:58:04]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[jam39, 2017-01-16 13:40:23]

jem456.vasishta@gmail.com changed reviewers:
+ sandersd@chromium.org, wolenetz@chromium.org, xhwang@chromium.org

[jam39, 2017-01-17 15:30:57]

On 2017/01/16 10:58:04, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

PTAL (Path Set #3)

[jbudorick, 2017-01-17 16:41:57]

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

[commit-bot: I haz the power, 2017-01-17 16:42:06]

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-01-17 17:59:21]

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

[commit-bot: I haz the power, 2017-01-17 17:59:23]

Dry run: This issue passed the CQ dry run.

[sandersd (OOO until July 31), 2017-01-17 21:29:00]

https://codereview.chromium.org/2626023003/diff/40001/media/gpu/vaapi_video_d...
File media/gpu/vaapi_video_decode_accelerator.cc (right):

https://codereview.chromium.org/2626023003/diff/40001/media/gpu/vaapi_video_d...
media/gpu/vaapi_video_decode_accelerator.cc:1749: <<
GetProfileName((VideoCodecProfile)frame_hdr->profile);
I've not checked all of the VAAPI decoder for consistency, but we don't use
C-style casts in media code. This should be replaced with a static_cast<>().

That said, these don't look like VideoCodecProfiles; vp9_parser defines
kVp9MaxProfile = 4, which would be an H.264 VideoCodecProfile.

[jam39, 2017-01-18 16:01:40]

PTAL

[jbudorick, 2017-01-18 16:02:19]

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

[sandersd (OOO until July 31), 2017-01-18 19:36:29]

lgtm

[wolenetz, 2017-01-18 20:14:01]

lgtm

[jam39, 2017-01-18 23:50:36]

The CQ bit was checked by jem456.vasishta@gmail.com

[jam39, 2017-01-18 23:50:37]

The patchset sent to the CQ was uploaded after l-g-t-m from mcasas@chromium.org,
posciak@chromium.org
Link to the patchset: https://codereview.chromium.org/2626023003/#ps60001
(title: "Don't use GetProfileName() for H.264 VideoCodecProfile.")

[commit-bot: I haz the power, 2017-01-18 23:51:32]

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-01-19 01:26:53]

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

[commit-bot: I haz the power, 2017-01-19 01:26:55]

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_...)

[jam39, 2017-01-19 14:46:58]

The CQ bit was checked by jem456.vasishta@gmail.com

[commit-bot: I haz the power, 2017-01-19 14:47:18]

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-01-19 15:26:19]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1484837218191240,
"parent_rev": "6ccdcca31bc009460bc1cacedb2ada65588ddf12", "commit_rev":
"9a44ab189f977553b42b9d7706628b959289308a"}

[commit-bot: I haz the power, 2017-01-19 15:26:51]

Description was changed from

==========
Use GetProfileName() for logging VideoCodecProfile strings.

BUG=676022
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

==========
Use GetProfileName() for logging VideoCodecProfile strings.

BUG=676022
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

Review-Url: https://codereview.chromium.org/2626023003
Cr-Commit-Position: refs/heads/master@{#444738}
Committed:
https://chromium.googlesource.com/chromium/src/+/9a44ab189f977553b42b9d770662...
==========

[commit-bot: I haz the power, 2017-01-19 15:26:53]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/9a44ab189f977553b42b9d770662...