Consolidate MPEG*_AAC_* MimeUtil::Codec enum values

Consolidate MPEG*_AAC_* MimeUtil::Codec enum values

We don't differentiate between the profiles, so we do not need separate values.
Also remove duplicate video codec strings from kFormatCodecMappings.

BUG=591869
TEST=content_browsertests --gtest_filter=MediaCanPlayTypeTest*

Committed: https://crrev.com/f52bd5bf94cf63512e6de19ad25181cabc641357
Cr-Commit-Position: refs/heads/master@{#380045}

[ddorwin, 2016-03-07 18:25:17]

Description was changed from

==========
Consolidate MPEG*_AAC_* Codec enum values.

We don't differentiate between the profiles, so we do not need separate values.

BUG=591869
==========

to

==========
Consolidate MPEG*_AAC_* Codec enum values

We don't differentiate between the profiles, so we do not need separate values.

BUG=591869
TEST=content_browsertests --gtest_filter=MediaCanPlayTypeTest*
==========

[ddorwin, 2016-03-07 18:33:16]

Description was changed from

==========
Consolidate MPEG*_AAC_* Codec enum values

We don't differentiate between the profiles, so we do not need separate values.

BUG=591869
TEST=content_browsertests --gtest_filter=MediaCanPlayTypeTest*
==========

to

==========
Consolidate MPEG*_AAC_* MimeUtil::Codec enum values

We don't differentiate between the profiles, so we do not need separate values.
Also remove duplicate video codec strings from kFormatCodecMappings.

BUG=591869
TEST=content_browsertests --gtest_filter=MediaCanPlayTypeTest*
==========

[ddorwin, 2016-03-07 18:38:54]

ddorwin@chromium.org changed reviewers:
+ chcunningham@chromium.org, servolk@chromium.org

[ddorwin, 2016-03-07 18:38:54]

ddorwin@chromium.org changed required reviewers:
+ chcunningham@chromium.org

[ddorwin, 2016-03-07 18:38:55]

PS1 shows the cut and paste of the comments for the audio comments (and the
deletion of the already irrelevant video comments). The changes are in PS2.

https://codereview.chromium.org/1769833003/diff/20001/media/base/mime_util_in...
File media/base/mime_util_internal.cc (right):

https://codereview.chromium.org/1769833003/diff/20001/media/base/mime_util_in...
media/base/mime_util_internal.cc:46: "avc1.42E00A,"
The H264 profiles were removed in https://codereview.chromium.org/1677563003.

https://codereview.chromium.org/1769833003/diff/20001/media/base/mime_util_in...
media/base/mime_util_internal.cc:65: {"video/webm", COMMON,
"opus,vorbis,vp8,vp9"},
The ".0" variants weree always duplicate here.

[servolk, 2016-03-07 18:44:37]

On 2016/03/07 18:38:55, ddorwin wrote:
> PS1 shows the cut and paste of the comments for the audio comments (and the
> deletion of the already irrelevant video comments). The changes are in PS2.
> 
>
https://codereview.chromium.org/1769833003/diff/20001/media/base/mime_util_in...
> File media/base/mime_util_internal.cc (right):
> 
>
https://codereview.chromium.org/1769833003/diff/20001/media/base/mime_util_in...
> media/base/mime_util_internal.cc:46: "avc1.42E00A,"
> The H264 profiles were removed in https://codereview.chromium.org/1677563003.
> 
>
https://codereview.chromium.org/1769833003/diff/20001/media/base/mime_util_in...
> media/base/mime_util_internal.cc:65: {"video/webm", COMMON,
> "opus,vorbis,vp8,vp9"},
> The ".0" variants weree always duplicate here.

lgtm

[servolk, 2016-03-07 18:44:44]

https://codereview.chromium.org/1769833003/diff/20001/media/base/mime_util_in...
File media/base/mime_util_internal.cc (right):

https://codereview.chromium.org/1769833003/diff/20001/media/base/mime_util_in...
media/base/mime_util_internal.cc:46: "avc1.42E00A,"
On 2016/03/07 18:38:55, ddorwin wrote:
> The H264 profiles were removed in https://codereview.chromium.org/1677563003.

Acknowledged.

https://codereview.chromium.org/1769833003/diff/20001/media/base/mime_util_in...
media/base/mime_util_internal.cc:65: {"video/webm", COMMON,
"opus,vorbis,vp8,vp9"},
On 2016/03/07 18:38:55, ddorwin wrote:
> The ".0" variants weree always duplicate here.

Acknowledged.

[chcunningham, 2016-03-08 02:36:58]

lgtm

https://codereview.chromium.org/1769833003/diff/40001/media/base/mime_util_in...
File media/base/mime_util_internal.cc (right):

https://codereview.chromium.org/1769833003/diff/40001/media/base/mime_util_in...
media/base/mime_util_internal.cc:38: "mp4a.66,mp4a.69,mp4a.40.2";  // MPEG-2
AAC, MP3, and MPEG-4 AAC.
For posterity: It took me a while to follow why you're able to do this
particular reduction. The comment above for ac-3 explained it (they all reduce
to just AAC).

[ddorwin, 2016-03-08 18:40:03]

https://codereview.chromium.org/1769833003/diff/40001/media/base/mime_util_in...
File media/base/mime_util_internal.cc (right):

https://codereview.chromium.org/1769833003/diff/40001/media/base/mime_util_in...
media/base/mime_util_internal.cc:38: "mp4a.66,mp4a.69,mp4a.40.2";  // MPEG-2
AAC, MP3, and MPEG-4 AAC.
On 2016/03/08 02:36:58, chcunningham wrote:
> For posterity: It took me a while to follow why you're able to do this
> particular reduction. The comment above for ac-3 explained it (they all reduce
> to just AAC).

That comment applies to all these strings. I moved it up.

[ddorwin, 2016-03-09 02:09:08]

The CQ bit was checked by ddorwin@chromium.org

[ddorwin, 2016-03-09 02:09:08]

The patchset sent to the CQ was uploaded after l-g-t-m from
servolk@chromium.org, chcunningham@chromium.org
Link to the patchset: https://codereview.chromium.org/1769833003/#ps60001
(title: "Consolidate comments about codec string variants.")

[commit-bot: I haz the power, 2016-03-09 02:09:45]

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

[commit-bot: I haz the power, 2016-03-09 02:51:06]

Description was changed from

==========
Consolidate MPEG*_AAC_* MimeUtil::Codec enum values

We don't differentiate between the profiles, so we do not need separate values.
Also remove duplicate video codec strings from kFormatCodecMappings.

BUG=591869
TEST=content_browsertests --gtest_filter=MediaCanPlayTypeTest*
==========

to

==========
Consolidate MPEG*_AAC_* MimeUtil::Codec enum values

We don't differentiate between the profiles, so we do not need separate values.
Also remove duplicate video codec strings from kFormatCodecMappings.

BUG=591869
TEST=content_browsertests --gtest_filter=MediaCanPlayTypeTest*
==========

[commit-bot: I haz the power, 2016-03-09 02:51:07]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-03-09 02:52:28]

Description was changed from

==========
Consolidate MPEG*_AAC_* MimeUtil::Codec enum values

We don't differentiate between the profiles, so we do not need separate values.
Also remove duplicate video codec strings from kFormatCodecMappings.

BUG=591869
TEST=content_browsertests --gtest_filter=MediaCanPlayTypeTest*
==========

to

==========
Consolidate MPEG*_AAC_* MimeUtil::Codec enum values

We don't differentiate between the profiles, so we do not need separate values.
Also remove duplicate video codec strings from kFormatCodecMappings.

BUG=591869
TEST=content_browsertests --gtest_filter=MediaCanPlayTypeTest*

Committed: https://crrev.com/f52bd5bf94cf63512e6de19ad25181cabc641357
Cr-Commit-Position: refs/heads/master@{#380045}
==========

[commit-bot: I haz the power, 2016-03-09 02:52:29]

Patchset 4 (id:??) landed as
https://crrev.com/f52bd5bf94cf63512e6de19ad25181cabc641357
Cr-Commit-Position: refs/heads/master@{#380045}