Fix indentation in spdy_framer_test.cc

Fix indentation in spdy_framer_test.cc

This lands server change 44799980.

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

[akalin, 2013-04-06 00:09:22]

+rch for review

[Ryan Hamilton, 2013-04-08 16:13:00]

lgtm

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

https://codereview.chromium.org/13454035/diff/1/net/spdy/spdy_framer_test.cc#...
net/spdy/spdy_framer_test.cc:103: : version_(version), size_(0),
finished_(false) {}
Huh.  I didn't realize this was the style when then entire method could not fit
on a single line.  learn something every day.

[akalin, 2013-04-08 21:48:42]

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

https://codereview.chromium.org/13454035/diff/1/net/spdy/spdy_framer_test.cc#...
net/spdy/spdy_framer_test.cc:103: : version_(version), size_(0),
finished_(false) {}
On 2013/04/08 16:13:00, Ryan Hamilton wrote:
> Huh.  I didn't realize this was the style when then entire method could not
fit
> on a single line.  learn something every day.

You mean for initializer lists? I don't think this is the style actually; it
should be:

    : version_(version),
      size_(0),
      finished_false() {}

I don't think there's a style rule for {\n} vs. {}. But I'm prioritizing
minimizing the diffs between the server version and the client version over
strictly correct formatting.

[akalin, 2013-04-08 22:33:11]

Committed patchset #1 manually as r192933 (presubmit successful).