Remove unecessary non fatal error statement that very often is printed in the PSNR or SSIM metric n…

Remove unecessary non fatal error statement that very often is printed in the PSNR or SSIM metric numbered list

BUG=webrtc:7698
NOTRY=TRUE

Review-Url: https://codereview.webrtc.org/2901793002
Cr-Commit-Position: refs/heads/master@{#18234}
Committed: https://chromium.googlesource.com/external/webrtc/+/18d023f9ee6201be20e4bc68e3fc070404a8edbc

[janssonWebRTC, 2017-05-23 12:35:29]

Description was changed from

==========
Remove unecessary non fatal error statement that very often is printed in the
PSNR or SSIM metric numbered list

BUG=7698
NOTRY=TRUE
==========

to

==========
Remove unecessary non fatal error statement that very often is printed in the
PSNR or SSIM metric numbered list

BUG=Webrtc:7698
NOTRY=TRUE
==========

[janssonWebRTC, 2017-05-23 12:35:37]

Description was changed from

==========
Remove unecessary non fatal error statement that very often is printed in the
PSNR or SSIM metric numbered list

BUG=Webrtc:7698
NOTRY=TRUE
==========

to

==========
Remove unecessary non fatal error statement that very often is printed in the
PSNR or SSIM metric numbered list

BUG=webrtc:7698
NOTRY=TRUE
==========

[janssonWebRTC, 2017-05-23 12:37:06]

jansson@webrtc.org changed reviewers:
+ kjellander@webrtc.org, oprypin@webrtc.org

[kjellander_webrtc, 2017-05-23 13:04:25]

lgtm

[kjellander_webrtc, 2017-05-23 13:05:27]

On 2017/05/23 13:04:25, kjellander_webrtc wrote:
> lgtm

Although, if it actually is often printed it seems it would be interesting to
figure out why and for which frames. Then it could make sense to also print the
frame number or similar, or log these messages to a log file instead of stderr
in case it's causing problems.

[oprypin_webrtc, 2017-05-23 13:06:34]

lgtm

[janssonWebRTC, 2017-05-23 14:24:37]

On 2017/05/23 13:05:27, kjellander_webrtc wrote:
> On 2017/05/23 13:04:25, kjellander_webrtc wrote:
> > lgtm
> 
> Although, if it actually is often printed it seems it would be interesting to
> figure out why and for which frames. Then it could make sense to also print
the
> frame number or similar, or log these messages to a log file instead of stderr
> in case it's causing problems.

I did look into the which frame it complained about and it was present hence the
error message is not correct. Not sure why it was added.

[janssonWebRTC, 2017-05-23 14:24:47]

The CQ bit was checked by jansson@webrtc.org

[commit-bot: I haz the power, 2017-05-23 14:24:50]

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-05-23 14:26:41]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495549487550950, "parent_rev":
"868f32f4230140c72a7b7abd6c3a5bb9d7feef67", "commit_rev":
"18d023f9ee6201be20e4bc68e3fc070404a8edbc"}

[commit-bot: I haz the power, 2017-05-23 14:26:45]

Description was changed from

==========
Remove unecessary non fatal error statement that very often is printed in the
PSNR or SSIM metric numbered list

BUG=webrtc:7698
NOTRY=TRUE
==========

to

==========
Remove unecessary non fatal error statement that very often is printed in the
PSNR or SSIM metric numbered list

BUG=webrtc:7698
NOTRY=TRUE

Review-Url: https://codereview.webrtc.org/2901793002
Cr-Commit-Position: refs/heads/master@{#18234}
Committed:
https://chromium.googlesource.com/external/webrtc/+/18d023f9ee6201be20e4bc68e...
==========

[commit-bot: I haz the power, 2017-05-23 14:26:46]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/18d023f9ee6201be20e4bc68e...

[janssonWebRTC, 2017-05-23 15:30:56]

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

The reason for reverting is: Looks like we need the return there after all:
derefence error in:
https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fclient.webrtc.perf%2FAnd....