Index: net/spdy/spdy_framer.cc |
diff --git a/net/spdy/spdy_framer.cc b/net/spdy/spdy_framer.cc |
index 32415e1ceead43a110b1c87799be47004696053d..16e298fab06a9d09de7d1443f78e3cad5d615d83 100644 |
--- a/net/spdy/spdy_framer.cc |
+++ b/net/spdy/spdy_framer.cc |
@@ -53,7 +53,7 @@ base::LazyInstance<DictionaryIds>::Leaky g_dictionary_ids; |
FlagsAndLength CreateFlagsAndLength(SpdyControlFlags flags, size_t length) { |
DCHECK_EQ(0u, length & ~static_cast<size_t>(kLengthMask)); |
FlagsAndLength flags_length; |
- flags_length.length_ = htonl(static_cast<uint32>(length)); |
+ flags_length.length_ = base::HostToNet32(static_cast<uint32>(length)); |
DCHECK_EQ(0, flags & ~kControlFlagsMask); |
flags_length.flags_[0] = flags; |
return flags_length; |
@@ -103,7 +103,8 @@ SettingsFlagsAndId SettingsFlagsAndId::FromWireFormat(int version, |
if (version < 3) { |
ConvertFlagsAndIdForSpdy2(&wire); |
} |
- return SettingsFlagsAndId(ntohl(wire) >> 24, ntohl(wire) & 0x00ffffff); |
+ return SettingsFlagsAndId(base::NetToHost32(wire) >> 24, |
+ base::NetToHost32(wire) & 0x00ffffff); |
} |
SettingsFlagsAndId::SettingsFlagsAndId(uint8 flags, uint32 id) |
@@ -112,7 +113,8 @@ SettingsFlagsAndId::SettingsFlagsAndId(uint8 flags, uint32 id) |
} |
uint32 SettingsFlagsAndId::GetWireFormat(int version) const { |
- uint32 wire = htonl(id_ & 0x00ffffff) | htonl(flags_ << 24); |
+ uint32 wire = |
+ base::HostToNet32(id_ & 0x00ffffff) | base::HostToNet32(flags_ << 24); |
if (version < 3) { |
ConvertFlagsAndIdForSpdy2(&wire); |
} |
@@ -790,7 +792,8 @@ bool SpdyFramer::ProcessSetting(const char* data) { |
SettingsFlagsAndId id_and_flags = |
SettingsFlagsAndId::FromWireFormat(spdy_version_, id_and_flags_wire); |
uint8 flags = id_and_flags.flags(); |
- uint32 value = ntohl(*(reinterpret_cast<const uint32*>(data + 4))); |
+ uint32 value = |
+ base::NetToHost32(*(reinterpret_cast<const uint32*>(data + 4))); |
// Validate id. |
switch (id_and_flags.id()) { |
@@ -1005,7 +1008,7 @@ bool SpdyFramer::ParseSettings(const SpdySettingsControlFrame* frame, |
uint32 id_and_flags_wire; |
uint32 value; |
// SettingsFlagsAndId accepts off-the-wire (network byte order) data, so we |
- // use ReadBytes() instead of ReadUInt32() as the latter calls ntohl(). |
+ // use ReadBytes() instead of ReadUInt32(), which calls base::NetToHost32(). |
if (!parser.ReadBytes(&id_and_flags_wire, 4)) { |
return false; |
} |
@@ -1076,7 +1079,7 @@ SpdySynStreamControlFrame* SpdyFramer::CreateSynStream( |
priority = GetLowestPriority(); |
} |
// Priority is 2 bits for <spdy3, 3 bits otherwise. |
- frame.WriteUInt16(ntohs(priority) << (spdy_version_ < 3 ? 6 : 5)); |
+ frame.WriteUInt16(base::NetToHost16(priority) << (spdy_version_ < 3 ? 6 : 5)); |
WriteHeaderBlock(&frame, headers); |
scoped_ptr<SpdySynStreamControlFrame> syn_frame( |
@@ -1295,7 +1298,7 @@ SpdyDataFrame* SpdyFramer::CreateDataFrame(SpdyStreamId stream_id, |
DCHECK_EQ(0u, len & ~static_cast<size_t>(kLengthMask)); |
FlagsAndLength flags_length; |
- flags_length.length_ = htonl(len); |
+ flags_length.length_ = base::HostToNet32(len); |
DCHECK_EQ(0, flags & ~kDataFlagsMask); |
flags_length.flags_[0] = flags; |
frame.WriteBytes(&flags_length, sizeof(flags_length)); |