Reland of Add content type information to encoded images and corresponding rtp extension header

Reland of Add content type information to encoded images and corresponding rtp extension header (patchset #1 id:1 of https://codereview.webrtc.org/2809653004/ )

Reason for revert:
Fix failing bots.

BUG=webrtc:7420
Review-Url: https://codereview.webrtc.org/2816493002
Cr-Commit-Position: refs/heads/master@{#17658}
Committed: https://chromium.googlesource.com/external/webrtc/+/00d802b6ee72259b0f8a83865961afc26c0f7e1f

[ilnik, 2017-04-11 13:21:23]

Created Reland of Add content type information to encoded images and
corresponding rtp extension header

[ilnik, 2017-04-11 13:21:45]

The CQ bit was checked by ilnik@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-11 13:21:50]

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

[commit-bot: I haz the power, 2017-04-11 13:21:51]

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

[commit-bot: I haz the power, 2017-04-11 13:21:52]

Dry run: This issue passed the CQ dry run.

[ilnik, 2017-04-11 13:23:03]

Description was changed from

==========
Reland of Add content type information to encoded images and corresponding rtp
extension header (patchset #1 id:1 of https://codereview.webrtc.org/2809653004/
)

Reason for revert:
Fix failing bots.

Original issue's description:
> Revert of Add content type information to encoded images and corresponding rtp
extension header (patchset #4 id:400001 of
https://codereview.webrtc.org/2812913002/ )
> 
> Reason for revert:
> Breaks android buildbots.
> 
> Original issue's description:
> > Reland of Add content type information to encoded images and corresponding
rtp extension header (patchset #1 id:1 of
https://codereview.webrtc.org/2816463002/ )
> > 
> > Reason for revert:
> > Reland with appropriate changes to API to not break depending projects.
> > 
> > Original issue's description:
> > > Revert of Add content type information to encoded images and corresponding
rtp extension header (patchset #31 id:600001 of
https://codereview.webrtc.org/2772033002/ )
> > > 
> > > Reason for revert:
> > > Breaks dependent projects.
> > > 
> > > Original issue's description:
> > > > Add content type information to Encoded Images and add corresponding RTP
extension header.
> > > > Use it to separate UMA e2e delay metric between screenshare from video.
> > > > Content type extension is set based on encoder settings and processed
and decoders.
> > > > 
> > > > Also,
> > > > Fix full-stack-tests to calculate RTT correctly, so new metric could be
tested.
> > > > 
> > > > BUG=webrtc:7420
> > > > 
> > > > Review-Url: https://codereview.webrtc.org/2772033002
> > > > Cr-Commit-Position: refs/heads/master@{#17640}
> > > > Committed:
https://chromium.googlesource.com/external/webrtc/+/64e739aeae5629cbbebf2a19e...
> > > 
> > >
TBR=tommi@webrtc.org,sprang@webrtc.org,stefan@webrtc.org,nisse@webrtc.org,mfl...
> > > # Skipping CQ checks because original CL landed less than 1 days ago.
> > > NOPRESUBMIT=true
> > > NOTREECHECKS=true
> > > NOTRY=true
> > > BUG=webrtc:7420
> > > 
> > > Review-Url: https://codereview.webrtc.org/2816463002
> > > Cr-Commit-Position: refs/heads/master@{#17644}
> > > Committed:
https://chromium.googlesource.com/external/webrtc/+/57218668083fce36a55efb8c8...
> > 
> >
TBR=tommi@webrtc.org,sprang@webrtc.org,stefan@webrtc.org,nisse@webrtc.org,mfl...
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:7420
> > 
> > Review-Url: https://codereview.webrtc.org/2812913002
> > Cr-Commit-Position: refs/heads/master@{#17651}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/774f6b4b968b46d6bd95c42ee...
> 
>
TBR=tommi@webrtc.org,sprang@webrtc.org,stefan@webrtc.org,nisse@webrtc.org,mfl...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7420
> 
> Review-Url: https://codereview.webrtc.org/2809653004
> Cr-Commit-Position: refs/heads/master@{#17653}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/27c46e2872c1da139310bb7ef...

TBR=tommi@webrtc.org,sprang@webrtc.org,stefan@webrtc.org,nisse@webrtc.org,mfl...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7420
==========

to

==========
Reland of Add content type information to encoded images and corresponding rtp
extension header (patchset #1 id:1 of https://codereview.webrtc.org/2809653004/
)

Reason for revert:
Fix failing bots.

Original issue's description:
> Revert of Add content type information to encoded images and corresponding rtp
extension header (patchset #4 id:400001 of
https://codereview.webrtc.org/2812913002/ )
> 
> Reason for revert:
> Breaks android buildbots.
> 
> Original issue's description:
> > Reland of Add content type information to encoded images and corresponding
rtp extension header (patchset #1 id:1 of
https://codereview.webrtc.org/2816463002/ )
> > 
> > Reason for revert:
> > Reland with appropriate changes to API to not break depending projects.
> > 
> > Original issue's description:
> > > Revert of Add content type information to encoded images and corresponding
rtp extension header (patchset #31 id:600001 of
https://codereview.webrtc.org/2772033002/ )
> > > 
> > > Reason for revert:
> > > Breaks dependent projects.
> > > 
> > > Original issue's description:
> > > > Add content type information to Encoded Images and add corresponding RTP
extension header.
> > > > Use it to separate UMA e2e delay metric between screenshare from video.
> > > > Content type extension is set based on encoder settings and processed
and decoders.
> > > > 
> > > > Also,
> > > > Fix full-stack-tests to calculate RTT correctly, so new metric could be
tested.
> > > > 
> > > > BUG=webrtc:7420
> > > > 
> > > > Review-Url: https://codereview.webrtc.org/2772033002
> > > > Cr-Commit-Position: refs/heads/master@{#17640}
> > > > Committed:
https://chromium.googlesource.com/external/webrtc/+/64e739aeae5629cbbebf2a19e...
> > > 
> > >
TBR=tommi@webrtc.org,sprang@webrtc.org,stefan@webrtc.org,nisse@webrtc.org,mfl...
> > > # Skipping CQ checks because original CL landed less than 1 days ago.
> > > NOPRESUBMIT=true
> > > NOTREECHECKS=true
> > > NOTRY=true
> > > BUG=webrtc:7420
> > > 
> > > Review-Url: https://codereview.webrtc.org/2816463002
> > > Cr-Commit-Position: refs/heads/master@{#17644}
> > > Committed:
https://chromium.googlesource.com/external/webrtc/+/57218668083fce36a55efb8c8...
> > 
> >
TBR=tommi@webrtc.org,sprang@webrtc.org,stefan@webrtc.org,nisse@webrtc.org,mfl...
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:7420
> > 
> > Review-Url: https://codereview.webrtc.org/2812913002
> > Cr-Commit-Position: refs/heads/master@{#17651}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/774f6b4b968b46d6bd95c42ee...
> 
>
TBR=tommi@webrtc.org,sprang@webrtc.org,stefan@webrtc.org,nisse@webrtc.org,mfl...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7420
> 
> Review-Url: https://codereview.webrtc.org/2809653004
> Cr-Commit-Position: refs/heads/master@{#17653}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/27c46e2872c1da139310bb7ef...

BUG=webrtc:7420
==========

[ilnik, 2017-04-11 13:23:10]

The CQ bit was checked by ilnik@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-11 13:23:20]

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

[commit-bot: I haz the power, 2017-04-11 13:38:37]

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

[commit-bot: I haz the power, 2017-04-11 13:38:38]

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

[tommi, 2017-04-11 14:00:46]

On 2017/04/11 13:21:23, ilnik wrote:
> Created Reland of Add content type information to encoded images and
> corresponding rtp extension header

Before landing again, can you clean up the description so that it'll be easier
to read in the log and bug reports?  The repeated 'reverted' parts and
indentation, isn't really useful in that sense.

[ilnik, 2017-04-11 14:16:52]

Description was changed from

==========
Reland of Add content type information to encoded images and corresponding rtp
extension header (patchset #1 id:1 of https://codereview.webrtc.org/2809653004/
)

Reason for revert:
Fix failing bots.

Original issue's description:
> Revert of Add content type information to encoded images and corresponding rtp
extension header (patchset #4 id:400001 of
https://codereview.webrtc.org/2812913002/ )
> 
> Reason for revert:
> Breaks android buildbots.
> 
> Original issue's description:
> > Reland of Add content type information to encoded images and corresponding
rtp extension header (patchset #1 id:1 of
https://codereview.webrtc.org/2816463002/ )
> > 
> > Reason for revert:
> > Reland with appropriate changes to API to not break depending projects.
> > 
> > Original issue's description:
> > > Revert of Add content type information to encoded images and corresponding
rtp extension header (patchset #31 id:600001 of
https://codereview.webrtc.org/2772033002/ )
> > > 
> > > Reason for revert:
> > > Breaks dependent projects.
> > > 
> > > Original issue's description:
> > > > Add content type information to Encoded Images and add corresponding RTP
extension header.
> > > > Use it to separate UMA e2e delay metric between screenshare from video.
> > > > Content type extension is set based on encoder settings and processed
and decoders.
> > > > 
> > > > Also,
> > > > Fix full-stack-tests to calculate RTT correctly, so new metric could be
tested.
> > > > 
> > > > BUG=webrtc:7420
> > > > 
> > > > Review-Url: https://codereview.webrtc.org/2772033002
> > > > Cr-Commit-Position: refs/heads/master@{#17640}
> > > > Committed:
https://chromium.googlesource.com/external/webrtc/+/64e739aeae5629cbbebf2a19e...
> > > 
> > >
TBR=tommi@webrtc.org,sprang@webrtc.org,stefan@webrtc.org,nisse@webrtc.org,mfl...
> > > # Skipping CQ checks because original CL landed less than 1 days ago.
> > > NOPRESUBMIT=true
> > > NOTREECHECKS=true
> > > NOTRY=true
> > > BUG=webrtc:7420
> > > 
> > > Review-Url: https://codereview.webrtc.org/2816463002
> > > Cr-Commit-Position: refs/heads/master@{#17644}
> > > Committed:
https://chromium.googlesource.com/external/webrtc/+/57218668083fce36a55efb8c8...
> > 
> >
TBR=tommi@webrtc.org,sprang@webrtc.org,stefan@webrtc.org,nisse@webrtc.org,mfl...
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:7420
> > 
> > Review-Url: https://codereview.webrtc.org/2812913002
> > Cr-Commit-Position: refs/heads/master@{#17651}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/774f6b4b968b46d6bd95c42ee...
> 
>
TBR=tommi@webrtc.org,sprang@webrtc.org,stefan@webrtc.org,nisse@webrtc.org,mfl...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7420
> 
> Review-Url: https://codereview.webrtc.org/2809653004
> Cr-Commit-Position: refs/heads/master@{#17653}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/27c46e2872c1da139310bb7ef...

BUG=webrtc:7420
==========

to

==========
Reland of Add content type information to encoded images and corresponding rtp
extension header (patchset #1 id:1 of https://codereview.webrtc.org/2809653004/
)

Reason for revert:
Fix failing bots.

BUG=webrtc:7420
==========

[ilnik, 2017-04-11 14:17:04]

On 2017/04/11 14:00:46, tommi (wëbrtc) wrote:
> On 2017/04/11 13:21:23, ilnik wrote:
> > Created Reland of Add content type information to encoded images and
> > corresponding rtp extension header
> 
> Before landing again, can you clean up the description so that it'll be easier
> to read in the log and bug reports?  The repeated 'reverted' parts and
> indentation, isn't really useful in that sense.

Done.

[tommi, 2017-04-11 14:36:43]

lgtm

[ilnik, 2017-04-11 15:24:30]

The CQ bit was checked by ilnik@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-11 15:24:41]

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

[commit-bot: I haz the power, 2017-04-11 15:59:15]

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

[commit-bot: I haz the power, 2017-04-11 15:59:18]

Dry run: Try jobs failed on following builders:
  ios32_sim_ios9_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios32_sim_ios9_dbg/buil...)

[ilnik, 2017-04-11 16:04:03]

The CQ bit was checked by ilnik@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-11 16:04:11]

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

[ilnik, 2017-04-11 16:10:21]

The CQ bit was checked by ilnik@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-11 16:10:31]

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

[commit-bot: I haz the power, 2017-04-11 16:39:57]

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

[commit-bot: I haz the power, 2017-04-11 16:39:58]

Dry run: This issue passed the CQ dry run.

[ilnik, 2017-04-11 17:31:24]

The CQ bit was checked by ilnik@webrtc.org

[ilnik, 2017-04-11 17:31:24]

The patchset sent to the CQ was uploaded after l-g-t-m from tommi@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2816493002/#ps340001 (title:
"Rebase")

[commit-bot: I haz the power, 2017-04-11 17:31:32]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-04-11 17:34:32]

CQ is committing da patch.

Bot data: {"patchset_id": 340001, "attempt_start_ts": 1491931884217730,
"parent_rev": "10d095d4f743bc16f8e486e156c48a6d023b32c5", "commit_rev":
"00d802b6ee72259b0f8a83865961afc26c0f7e1f"}

[commit-bot: I haz the power, 2017-04-11 17:34:38]

Description was changed from

==========
Reland of Add content type information to encoded images and corresponding rtp
extension header (patchset #1 id:1 of https://codereview.webrtc.org/2809653004/
)

Reason for revert:
Fix failing bots.

BUG=webrtc:7420
==========

to

==========
Reland of Add content type information to encoded images and corresponding rtp
extension header (patchset #1 id:1 of https://codereview.webrtc.org/2809653004/
)

Reason for revert:
Fix failing bots.

BUG=webrtc:7420

Review-Url: https://codereview.webrtc.org/2816493002
Cr-Commit-Position: refs/heads/master@{#17658}
Committed:
https://chromium.googlesource.com/external/webrtc/+/00d802b6ee72259b0f8a83865...
==========

[commit-bot: I haz the power, 2017-04-11 17:34:39]

Committed patchset #4 (id:340001) as
https://chromium.googlesource.com/external/webrtc/+/00d802b6ee72259b0f8a83865...