Add SpdyFramerDebugVisitorInterface for later usage in stats collecting

Add SpdyFramerDebugVisitorInterface for later usage in stats collecting
and more!

Lands server change 32048876.

Also fixes test expectations when USE_SYSTEM_ZLIB is not set.

Also add compiler_specific.h include for OVERRIDE keywords.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=181087

[akalin, 2013-02-06 00:25:07]

+rch for review

https://codereview.chromium.org/12224019/diff/2001/net/spdy/spdy_framer_test.cc
File net/spdy/spdy_framer_test.cc (right):

https://codereview.chromium.org/12224019/diff/2001/net/spdy/spdy_framer_test....
net/spdy/spdy_framer_test.cc:925: 93u,
I do it this way instead of just copying the line for each block because future
CLs change the variable name, and so I want to avoid the possibility of the
names in the two blocks going out of sync

[akalin, 2013-02-06 00:29:01]

https://codereview.chromium.org/12224019/diff/2001/net/spdy/spdy_framer_test.cc
File net/spdy/spdy_framer_test.cc (right):

https://codereview.chromium.org/12224019/diff/2001/net/spdy/spdy_framer_test....
net/spdy/spdy_framer_test.cc:925: 93u,
On 2013/02/06 00:25:07, akalin wrote:
> I do it this way instead of just copying the line for each block because
future
> CLs change the variable name, and so I want to avoid the possibility of the
> names in the two blocks going out of sync

Oh.  looks like only "decompressed" actually changes.  But I still prefer it
this way.

[Ryan Hamilton, 2013-02-06 00:32:29]

lgtm

https://codereview.chromium.org/12224019/diff/2001/net/spdy/spdy_framer_test.cc
File net/spdy/spdy_framer_test.cc (right):

https://codereview.chromium.org/12224019/diff/2001/net/spdy/spdy_framer_test....
net/spdy/spdy_framer_test.cc:925: 93u,
On 2013/02/06 00:29:01, akalin wrote:
> On 2013/02/06 00:25:07, akalin wrote:
> > I do it this way instead of just copying the line for each block because
> future
> > CLs change the variable name, and so I want to avoid the possibility of the
> > names in the two blocks going out of sync
> 
> Oh.  looks like only "decompressed" actually changes.  But I still prefer it
> this way.

Either way is fine with me.

[commit-bot: I haz the power, 2013-02-06 00:38:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/akalin@chromium.org/12224019/4001

[commit-bot: I haz the power, 2013-02-06 01:24:15]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[akalin, 2013-02-06 20:05:24]

On 2013/02/06 00:32:29, Ryan Hamilton wrote:
> Either way is fine with me.

Looks like Visual Studio doesn't like how I did it so I'm putting the whole line
in the blocks

[commit-bot: I haz the power, 2013-02-06 20:16:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/akalin@chromium.org/12224019/11002