Add a new UMA histogram for tracking SpdySessionErrorDetails

Add a new UMA histogram for tracking SpdySessionErrorDetails

BUG=125213
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=134442

[Ryan Hamilton, 2012-04-27 16:22:59]

Hi Will, 

Please take a look.  This CL adds a histogram for SPDY session errors, not just
those that are SPDY_PROTOCOL_ERROR.  I'm not sure if this is better or worse
than what you recommended.  Assuming it's good, I'm working on a similar CL to
add histograms for Stream errors.  Would it make sense to explicitly track
sessions and streams that had no errors, or is that likely to be a performance
issue?

Cheers,

Ryan

[willchan no longer on Chromium, 2012-04-27 21:59:30]

lgtm

http://codereview.chromium.org/10233016/diff/4001/net/spdy/spdy_session.h
File net/spdy/spdy_session.h (right):

http://codereview.chromium.org/10233016/diff/4001/net/spdy/spdy_session.h#new...
net/spdy/spdy_session.h:68: REQUST_FOR_SECURE_CONTENT_OVER_INSECURE_SESSION,
REQUEST

http://codereview.chromium.org/10233016/diff/4001/net/spdy/spdy_session.h#new...
net/spdy/spdy_session.h:74:
COMPILE_ASSERT(static_cast<SpdySessionErrorDetails>(SpdyFramer::LAST_ERROR) ==
include compiler_specific.h for this

[commit-bot: I haz the power, 2012-04-27 23:32:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rch@chromium.org/10233016/13001

[commit-bot: I haz the power, 2012-04-27 23:58:45]

Try job failure for 10233016-13001 (retry) on linux_chromeos for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2012-04-28 03:43:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rch@chromium.org/10233016/13001

[commit-bot: I haz the power, 2012-04-28 05:45:14]

Change committed as 134442