Add stats for rendered pixels (sqrt(w*h)) per second:

Add stats for rendered pixels (sqrt(w*h)) per second:
- "WebRTC.Video.RenderSqrtPixelsPerSecond"

BUG=chromium:512752
Committed: https://crrev.com/f839dcc8709243d1d81d5a23eb15aa43fc0466f6
Cr-Commit-Position: refs/heads/master@{#10208}

[åsapersson, 2015-09-23 12:59:53]

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

[pbos-webrtc, 2015-09-23 13:39:20]

lgtm

https://codereview.webrtc.org/1366583002/diff/20001/webrtc/video/receive_stat...
File webrtc/video/receive_statistics_proxy.cc (right):

https://codereview.webrtc.org/1366583002/diff/20001/webrtc/video/receive_stat...
webrtc/video/receive_statistics_proxy.cc:156: if (width > 0 && height > 0)
DCHECK width>0 and height>0 instead.

[åsapersson, 2015-09-23 14:07:54]

https://codereview.webrtc.org/1366583002/diff/20001/webrtc/video/receive_stat...
File webrtc/video/receive_statistics_proxy.cc (right):

https://codereview.webrtc.org/1366583002/diff/20001/webrtc/video/receive_stat...
webrtc/video/receive_statistics_proxy.cc:156: if (width > 0 && height > 0)
On 2015/09/23 13:39:20, pbos-webrtc wrote:
> DCHECK width>0 and height>0 instead.

Done.

[stefan-webrtc, 2015-09-25 13:02:48]

Please correct the description of this CL as you are not adding a histogram for
fps*sqrt(width*height)

[stefan-webrtc, 2015-09-25 13:02:54]

lgtm

[åsapersson, 2015-09-25 13:50:15]

On 2015/09/25 13:02:48, stefan-webrtc (holmer) wrote:
> Please correct the description of this CL as you are not adding a histogram
for
> fps*sqrt(width*height)

Done.

[mflodman, 2015-09-30 11:28:05]

On 2015/09/25 13:50:15, åsapersson wrote:
> On 2015/09/25 13:02:48, stefan-webrtc (holmer) wrote:
> > Please correct the description of this CL as you are not adding a histogram
> for
> > fps*sqrt(width*height)
> 
> Done.

s/WebRTC.Video.RenderPixelsPerSecond/WebRTC.Video.RenderSqrtPixelsPerSecond ?

Otherwise LGTM.

[åsapersson, 2015-10-05 14:07:37]

On 2015/09/30 11:28:05, mflodman wrote:
> On 2015/09/25 13:50:15, åsapersson wrote:
> > On 2015/09/25 13:02:48, stefan-webrtc (holmer) wrote:
> > > Please correct the description of this CL as you are not adding a
histogram
> > for
> > > fps*sqrt(width*height)
> > 
> > Done.
> 
> s/WebRTC.Video.RenderPixelsPerSecond/WebRTC.Video.RenderSqrtPixelsPerSecond ?
> 
> Otherwise LGTM.

Done.

[åsapersson, 2015-10-08 06:53:41]

The CQ bit was checked by asapersson@webrtc.org

[åsapersson, 2015-10-08 06:53:42]

The patchset sent to the CQ was uploaded after l-g-t-m from pbos@webrtc.org,
stefan@webrtc.org, mflodman@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1366583002/#ps100001 (title:
"renamed histogram")

[commit-bot: I haz the power, 2015-10-08 06:53:48]

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

[commit-bot: I haz the power, 2015-10-08 07:42:04]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-10-08 07:42:18]

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