Fix spelling in histogtrams.xml;comment in header; format in cc file.

net/udp/udp_socket_win.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/udp/udp_socket_win.h"
 
 #include <mstcpip.h>
 
 #include "base/callback.h"
 #include "base/logging.h"
@@ skipping 373 matching lines @@
   socket_ = CreatePlatformSocket(addr_family_, SOCK_DGRAM, IPPROTO_UDP);
   if (socket_ == INVALID_SOCKET)
     return MapSystemError(WSAGetLastError());
   core_ = new Core(this);
   return OK;
 }
 
 bool UDPSocketWin::SetReceiveBufferSize(int32 size) {
   DCHECK(CalledOnValidThread());
   setsockopt(socket_, SOL_SOCKET, SO_RCVBUF,
-              reinterpret_cast<const char*>(&size), sizeof(size));

[wtc, 2014/03/25 17:27:47]

The indentation of this should wait until the other changes I suggested. I think
you should definitely check the return value of setsockopt, and that will
require reformatting this setsockopt call.

+             reinterpret_cast<const char*>(&size), sizeof(size));
   // If the setsockopt fails, but the buffer is big enough, we will return
   // success. It is not worth testing the return value as we still need to check
   // via getsockopt anyway according to Windows documentation.
   int32 actual_size = 0;
   int option_size = sizeof(actual_size);
   int rv = getsockopt(socket_, SOL_SOCKET, SO_RCVBUF,
                       reinterpret_cast<char*>(&actual_size), &option_size);
   if (rv != 0)
     return false;
   if (actual_size < size) {
@@ skipping 427 matching lines @@
 // Note: setsockopt(IP_TOS) does not work on windows XP and later.
 int UDPSocketWin::SetDiffServCodePoint(DiffServCodePoint dscp) {
   return ERR_NOT_IMPLEMENTED;
 }
 
 void UDPSocketWin::DetachFromThread() {
   base::NonThreadSafe::DetachFromThread();
 }
 
 }  // namespace net