Revert of Rename OpenH264 frame-type conversion function.

Revert of Rename OpenH264 frame-type conversion function. (patchset #2 id:20001 of https://codereview.webrtc.org/1943193003/ )

Reason for revert:
Speculative revert for perf regression (though unlikely).

Original issue's description:
> Rename OpenH264 frame-type conversion function.
>
> Also removing default case, so if another frame is added to
> EVideoFrameType we have to handle it.
>
> This will now NOTREACHED on videoFrameTypeInvalid, but
> videoFrameTypeInvalid shouldn't happen if encoding succeeds, so it
> should be fine or we should become aware of it.
>
> BUG=
> R=hbos@webrtc.org
>
> Committed: https://crrev.com/39a36705ab734914d500b8a0f214ea630d82ab70
> Cr-Commit-Position: refs/heads/master@{#12636}

TBR=hbos@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:610347
Committed: https://crrev.com/1abf937cecea56ee02ac4a08980ffea9e7ed1054
Cr-Commit-Position: refs/heads/master@{#12677}

[pbos-webrtc, 2016-05-10 17:10:28]

Created Revert of Rename OpenH264 frame-type conversion function.

[commit-bot: I haz the power, 2016-05-10 17:10:31]

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

[pbos-webrtc, 2016-05-10 17:10:40]

Description was changed from

==========
Revert of Rename OpenH264 frame-type conversion function. (patchset #2 id:20001
of https://codereview.webrtc.org/1943193003/ )

Reason for revert:
Speculative revert for perf regression (though not likely).

Original issue's description:
> Rename OpenH264 frame-type conversion function.
> 
> Also removing default case, so if another frame is added to
> EVideoFrameType we have to handle it.
> 
> This will now NOTREACHED on videoFrameTypeInvalid, but
> videoFrameTypeInvalid shouldn't happen if encoding succeeds, so it
> should be fine or we should become aware of it.
> 
> BUG=
> R=hbos@webrtc.org
> 
> Committed: https://crrev.com/39a36705ab734914d500b8a0f214ea630d82ab70
> Cr-Commit-Position: refs/heads/master@{#12636}

TBR=hbos@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=
==========

to

==========
Revert of Rename OpenH264 frame-type conversion function. (patchset #2 id:20001
of https://codereview.webrtc.org/1943193003/ )

Reason for revert:
Speculative revert for perf regression (though not likely).

Original issue's description:
> Rename OpenH264 frame-type conversion function.
> 
> Also removing default case, so if another frame is added to
> EVideoFrameType we have to handle it.
> 
> This will now NOTREACHED on videoFrameTypeInvalid, but
> videoFrameTypeInvalid shouldn't happen if encoding succeeds, so it
> should be fine or we should become aware of it.
> 
> BUG=
> R=hbos@webrtc.org
> 
> Committed: https://crrev.com/39a36705ab734914d500b8a0f214ea630d82ab70
> Cr-Commit-Position: refs/heads/master@{#12636}

TBR=hbos@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:610347
==========

[pbos-webrtc, 2016-05-10 17:10:52]

Description was changed from

==========
Revert of Rename OpenH264 frame-type conversion function. (patchset #2 id:20001
of https://codereview.webrtc.org/1943193003/ )

Reason for revert:
Speculative revert for perf regression (though not likely).

Original issue's description:
> Rename OpenH264 frame-type conversion function.
> 
> Also removing default case, so if another frame is added to
> EVideoFrameType we have to handle it.
> 
> This will now NOTREACHED on videoFrameTypeInvalid, but
> videoFrameTypeInvalid shouldn't happen if encoding succeeds, so it
> should be fine or we should become aware of it.
> 
> BUG=
> R=hbos@webrtc.org
> 
> Committed: https://crrev.com/39a36705ab734914d500b8a0f214ea630d82ab70
> Cr-Commit-Position: refs/heads/master@{#12636}

TBR=hbos@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:610347
==========

to

==========
Revert of Rename OpenH264 frame-type conversion function. (patchset #2 id:20001
of https://codereview.webrtc.org/1943193003/ )

Reason for revert:
Speculative revert for perf regression (though unlikely).

Original issue's description:
> Rename OpenH264 frame-type conversion function.
> 
> Also removing default case, so if another frame is added to
> EVideoFrameType we have to handle it.
> 
> This will now NOTREACHED on videoFrameTypeInvalid, but
> videoFrameTypeInvalid shouldn't happen if encoding succeeds, so it
> should be fine or we should become aware of it.
> 
> BUG=
> R=hbos@webrtc.org
> 
> Committed: https://crrev.com/39a36705ab734914d500b8a0f214ea630d82ab70
> Cr-Commit-Position: refs/heads/master@{#12636}

TBR=hbos@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:610347
==========

[commit-bot: I haz the power, 2016-05-10 17:23:20]

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

[commit-bot: I haz the power, 2016-05-10 17:23:21]

Try jobs failed on following builders:
  android_compile_x86_dbg on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_compile_x86_dbg...)

[pbos-webrtc, 2016-05-10 18:52:19]

Description was changed from

==========
Revert of Rename OpenH264 frame-type conversion function. (patchset #2 id:20001
of https://codereview.webrtc.org/1943193003/ )

Reason for revert:
Speculative revert for perf regression (though unlikely).

Original issue's description:
> Rename OpenH264 frame-type conversion function.
> 
> Also removing default case, so if another frame is added to
> EVideoFrameType we have to handle it.
> 
> This will now NOTREACHED on videoFrameTypeInvalid, but
> videoFrameTypeInvalid shouldn't happen if encoding succeeds, so it
> should be fine or we should become aware of it.
> 
> BUG=
> R=hbos@webrtc.org
> 
> Committed: https://crrev.com/39a36705ab734914d500b8a0f214ea630d82ab70
> Cr-Commit-Position: refs/heads/master@{#12636}

TBR=hbos@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:610347
==========

to

==========
Revert of Rename OpenH264 frame-type conversion function. (patchset #2 id:20001
of https://codereview.webrtc.org/1943193003/ )

Reason for revert:
Speculative revert for perf regression (though unlikely).

Original issue's description:
> Rename OpenH264 frame-type conversion function.
> 
> Also removing default case, so if another frame is added to
> EVideoFrameType we have to handle it.
> 
> This will now NOTREACHED on videoFrameTypeInvalid, but
> videoFrameTypeInvalid shouldn't happen if encoding succeeds, so it
> should be fine or we should become aware of it.
> 
> BUG=
> R=hbos@webrtc.org
> 
> Committed: https://crrev.com/39a36705ab734914d500b8a0f214ea630d82ab70
> Cr-Commit-Position: refs/heads/master@{#12636}

TBR=hbos@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:610347

Committed:
https://chromium.googlesource.com/external/webrtc/+/1abf937cecea56ee02ac4a089...
==========

[pbos-webrtc, 2016-05-10 18:52:20]

Committed patchset #1 (id:1) manually as
1abf937cecea56ee02ac4a08980ffea9e7ed1054 (presubmit successful).

[commit-bot: I haz the power, 2016-05-10 18:52:22]

Description was changed from

==========
Revert of Rename OpenH264 frame-type conversion function. (patchset #2 id:20001
of https://codereview.webrtc.org/1943193003/ )

Reason for revert:
Speculative revert for perf regression (though unlikely).

Original issue's description:
> Rename OpenH264 frame-type conversion function.
> 
> Also removing default case, so if another frame is added to
> EVideoFrameType we have to handle it.
> 
> This will now NOTREACHED on videoFrameTypeInvalid, but
> videoFrameTypeInvalid shouldn't happen if encoding succeeds, so it
> should be fine or we should become aware of it.
> 
> BUG=
> R=hbos@webrtc.org
> 
> Committed: https://crrev.com/39a36705ab734914d500b8a0f214ea630d82ab70
> Cr-Commit-Position: refs/heads/master@{#12636}

TBR=hbos@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:610347

Committed:
https://chromium.googlesource.com/external/webrtc/+/1abf937cecea56ee02ac4a089...
==========

to

==========
Revert of Rename OpenH264 frame-type conversion function. (patchset #2 id:20001
of https://codereview.webrtc.org/1943193003/ )

Reason for revert:
Speculative revert for perf regression (though unlikely).

Original issue's description:
> Rename OpenH264 frame-type conversion function.
>
> Also removing default case, so if another frame is added to
> EVideoFrameType we have to handle it.
>
> This will now NOTREACHED on videoFrameTypeInvalid, but
> videoFrameTypeInvalid shouldn't happen if encoding succeeds, so it
> should be fine or we should become aware of it.
>
> BUG=
> R=hbos@webrtc.org
>
> Committed: https://crrev.com/39a36705ab734914d500b8a0f214ea630d82ab70
> Cr-Commit-Position: refs/heads/master@{#12636}

TBR=hbos@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:610347

Committed: https://crrev.com/1abf937cecea56ee02ac4a08980ffea9e7ed1054
Cr-Commit-Position: refs/heads/master@{#12677}
==========

[pbos-webrtc, 2016-05-11 07:58:29]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/1970513004/ by pbos@webrtc.org.

The reason for reverting is: Not perf-regression culprit..