More net/ header/implementation method reordering.

More net/ header/implementation method reordering.

(Contains some minor de-inlining.)

BUG=68682
TEST=compiles

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

[Elliot Glaysher, 2011-01-21 23:02:33]

More rubberstampable physical code reorganizations.

[willchan no longer on Chromium, 2011-01-21 23:07:32]

LGTM

http://codereview.chromium.org/6263010/diff/1/net/disk_cache/bitmap.h
File net/disk_cache/bitmap.h (right):

http://codereview.chromium.org/6263010/diff/1/net/disk_cache/bitmap.h#newcode16
net/disk_cache/bitmap.h:16: Bitmap() : map_(NULL), num_bits_(0), array_size_(0),
alloc_(false) {}
I'm a bit surprised you didn't define this out of line too. I guess it's small.

[Elliot Glaysher, 2011-01-21 23:11:15]

http://codereview.chromium.org/6263010/diff/1/net/disk_cache/bitmap.h
File net/disk_cache/bitmap.h (right):

http://codereview.chromium.org/6263010/diff/1/net/disk_cache/bitmap.h#newcode16
net/disk_cache/bitmap.h:16: Bitmap() : map_(NULL), num_bits_(0), array_size_(0),
alloc_(false) {}
On 2011/01/21 23:07:33, willchan wrote:
> I'm a bit surprised you didn't define this out of line too. I guess it's
small.

Yeah; the other ctors called multiple methods and required <algorithm> in the
header. This one is comparably small and just does four integral assignments.