Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(826)

Unified Diff: net/spdy/spdy_framer.cc

Issue 10254021: Merge SettingsMap changes from server code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/spdy/spdy_framer.h ('k') | net/spdy/spdy_framer_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_framer.cc
diff --git a/net/spdy/spdy_framer.cc b/net/spdy/spdy_framer.cc
index ee0ee22318e3efd93a8c2674fb2470ca0035c0b6..e7f33d756a2eaafbe9b248a11d58deaafd40b996 100644
--- a/net/spdy/spdy_framer.cc
+++ b/net/spdy/spdy_framer.cc
@@ -989,7 +989,7 @@ bool SpdyFramer::ParseSettings(const SpdySettingsControlFrame* frame,
SpdySettingsIds id = static_cast<SpdySettingsIds>(flags_and_id.id());
SpdySettingsFlags flags =
static_cast<SpdySettingsFlags>(flags_and_id.flags());
- settings->insert(std::make_pair(id, SettingsFlagsAndValue(flags, value)));
+ (*settings)[id] = SettingsFlagsAndValue(flags, value);
}
return true;
}
@@ -1113,13 +1113,13 @@ SpdySettingsControlFrame* SpdyFramer::CreateSettings(
SpdyFrameBuilder frame(SETTINGS, CONTROL_FLAG_NONE, spdy_version_,
frame_size);
frame.WriteUInt32(values.size());
- SettingsMap::const_iterator it = values.begin();
- while (it != values.end()) {
+ for (SettingsMap::const_iterator it = values.begin();
+ it != values.end();
+ it++) {
SettingsFlagsAndId flags_and_id(it->second.first, it->first);
uint32 id_and_flags_wire = flags_and_id.GetWireFormat(spdy_version_);
frame.WriteBytes(&id_and_flags_wire, 4);
frame.WriteUInt32(it->second.second);
- ++it;
}
DCHECK_EQ(frame.length(), frame_size);
return reinterpret_cast<SpdySettingsControlFrame*>(frame.take());
« no previous file with comments | « net/spdy/spdy_framer.h ('k') | net/spdy/spdy_framer_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698