| commit | 09db057da1d70bab29d66ef0c4e564447c4717e0 | |
| author | bnc <bnc@chromium.org> | |
| Wed, 29 Jul 2015 13:15:51 +0000 (29 06:15 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Wed, 29 Jul 2015 13:16:57 +0000 (29 13:16 +0000) | ||
| tree | 3da43c29d8c771cbc2a9714c55d30c808ec9541b | treesnapshot (tar.gz zip) |
| parent | 1ca6eaede71ea9ac9f00a0c3bb55119c8ddf676e | commitdiff |
Remove unnecessary heap allocation of SpdyFrameReader.
The removed comments describe a situation which doesn't exist now, and tests
indicate that having a stack local SpdyFrameReader is just fine. Maybe we'll get
a small perf boost by skipping one heap allocation/free per frame.
This CL lands server change 97233421 by jamessynge.
BUG=488484
Review URL: https://codereview.chromium.org/1256063003
Cr-Commit-Position: refs/heads/master@{#340877}
The removed comments describe a situation which doesn't exist now, and tests
indicate that having a stack local SpdyFrameReader is just fine. Maybe we'll get
a small perf boost by skipping one heap allocation/free per frame.
This CL lands server change 97233421 by jamessynge.
BUG=488484
Review URL: https://codereview.chromium.org/1256063003
Cr-Commit-Position: refs/heads/master@{#340877}
| net/spdy/spdy_framer.cc | diffblobblamehistory |