Turn off error resilience for VP9 if no spatial or temporal layers are configured and NACK is enabl…

Turn off error resilience for VP9 if no spatial or temporal layers are configured and NACK is enabled.

Error resilience is currently always enabled for VP9 which reduces quality.

BUG=webrtc:6783
Committed: https://crrev.com/4eb03c76fa2320534d669fda2aabf800e7a6f579
Cr-Commit-Position: refs/heads/master@{#15390}

[åsapersson, 2016-11-28 11:02:21]

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

[commit-bot: I haz the power, 2016-11-28 11:02:26]

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, 2016-11-28 11:05:15]

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

[commit-bot: I haz the power, 2016-11-28 11:05:16]

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

[åsapersson, 2016-11-28 11:09:00]

Description was changed from

==========
Turn off error resilience for VP9 if no spatial or temporal layers are
configured and NACK is enabled.

BUG=
==========

to

==========
Turn off error resilience for VP9 if no spatial or temporal layers are
configured and NACK is enabled.

BUG=webrtc:6783
==========

[åsapersson, 2016-11-28 11:09:01]

asapersson@webrtc.org changed reviewers:
+ marpan@webrtc.org, mflodman@webrtc.org, stefan@webrtc.org

[åsapersson, 2016-11-28 11:17:11]

Patchset #2 (id:20001) has been deleted

[åsapersson, 2016-11-28 11:17:24]

Patchset #1 (id:1) has been deleted

[åsapersson, 2016-11-28 11:19:45]

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

[commit-bot: I haz the power, 2016-11-28 11:19:56]

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, 2016-11-28 11:39:28]

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

[commit-bot: I haz the power, 2016-11-28 11:39:28]

Dry run: This issue passed the CQ dry run.

[åsapersson, 2016-11-28 12:53:24]

Description was changed from

==========
Turn off error resilience for VP9 if no spatial or temporal layers are
configured and NACK is enabled.

BUG=webrtc:6783
==========

to

==========
Turn off error resilience for VP9 if no spatial or temporal layers are
configured and NACK is enabled.

Error resilience is currently always enabled for VP9 which reduces quality.

BUG=webrtc:6783
==========

[marpan, 2016-11-29 21:32:00]

marpan@google.com changed reviewers:
+ marpan@google.com

[marpan, 2016-11-29 21:32:00]

lgtm, for codec setting change

[stefan-webrtc, 2016-11-30 13:04:33]

lgtm

[åsapersson, 2016-12-02 16:39:18]

The CQ bit was checked by asapersson@webrtc.org

[commit-bot: I haz the power, 2016-12-02 16:39:20]

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, 2016-12-02 16:57:56]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1480696758656700,
"parent_rev": "30179ccb10f932ce958a5568df08f0ebcb4efb6b", "commit_rev":
"e2cc316c7aeb9d91ef2244ab878449b12836e14a"}

[commit-bot: I haz the power, 2016-12-02 16:58:02]

Description was changed from

==========
Turn off error resilience for VP9 if no spatial or temporal layers are
configured and NACK is enabled.

Error resilience is currently always enabled for VP9 which reduces quality.

BUG=webrtc:6783
==========

to

==========
Turn off error resilience for VP9 if no spatial or temporal layers are
configured and NACK is enabled.

Error resilience is currently always enabled for VP9 which reduces quality.

BUG=webrtc:6783
==========

[commit-bot: I haz the power, 2016-12-02 16:58:04]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2016-12-02 16:58:08]

Description was changed from

==========
Turn off error resilience for VP9 if no spatial or temporal layers are
configured and NACK is enabled.

Error resilience is currently always enabled for VP9 which reduces quality.

BUG=webrtc:6783
==========

to

==========
Turn off error resilience for VP9 if no spatial or temporal layers are
configured and NACK is enabled.

Error resilience is currently always enabled for VP9 which reduces quality.

BUG=webrtc:6783

Committed: https://crrev.com/4eb03c76fa2320534d669fda2aabf800e7a6f579
Cr-Commit-Position: refs/heads/master@{#15390}
==========

[commit-bot: I haz the power, 2016-12-02 16:58:09]

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

[åsapersson, 2016-12-09 10:14:07]

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

The reason for reverting is: Increase in encode time larger than expected..