Why is this an issue?  The state machine guarantees that ParseFieldLength() is called, setting next_field_lenght_, before its value is used.

The preferred style is to initialize everything in the constructor (Even variables that won't be used immediately).  It doesn't look like it's in the current Google style guide (Not sure if it was there at one point or not), but still tends to be preferred style.

Re #5: I understand.  Then let's do this.

Re OP: Please follow steps at https://www.chromium.org/developers/contributing-code, as mentioned at  issue 680052 .  Thank you.

spdy_headers_block_parser.cc doesn't exist in master anymore, so this issue is meaningless now. We can close it now.