Add a SetDoNotFragment() method to DatagramSocket, and call this for

net/udp/udp_socket_posix.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_posix.h"
 
 #include <errno.h>
 #include <fcntl.h>
 #include <net/if.h>
 #include <netdb.h>
@@ skipping 407 matching lines @@
 }
 
 int UDPSocketPosix::SetSendBufferSize(int32_t size) {
   DCHECK_NE(socket_, kInvalidSocket);
   DCHECK(CalledOnValidThread());
   int rv = setsockopt(socket_, SOL_SOCKET, SO_SNDBUF,
                       reinterpret_cast<const char*>(&size), sizeof(size));
   return rv == 0 ? OK : MapSystemError(errno);
 }
 
+int UDPSocketPosix::SetDoNotFragment() {
+  DCHECK_NE(socket_, kInvalidSocket);
+  DCHECK(CalledOnValidThread());
+
+#if !defined(IP_PMTUDISC_DO)
+  return ERR_NOT_IMPLEMENTED;
+#else
+  int v6_only = false;
+  if (addr_family_ == AF_INET6) {
+    socklen_t v6_only_len = sizeof(v6_only);
+    int rv =
+        getsockopt(socket_, IPPROTO_IPV6, IPV6_V6ONLY, &v6_only, &v6_only_len);
+    if (rv != 0)
+      return MapSystemError(errno);
+
+    int val = IPV6_PMTUDISC_DO;
+    rv =
+        setsockopt(socket_, IPPROTO_IPV6, IPV6_MTU_DISCOVER, &val, sizeof(val));
+    if (rv != 0)
+      return MapSystemError(errno);
+    if (v6_only)
+      return rv == 0 ? OK : MapSystemError(errno);

[mmenke, 2016/08/12 21:55:49]

rv is always 0 here (See if statement on above line).  My suggestion is to move
the getsockopt just above the v6_only check, and then do:

if (rv != 0 || v6_only)
  return rv == 0 ? OK : MapSystemError(errno);

Note that you could just merge the two if's you have down here as things are,
but then you'd combining the results of the two platform API calls.  If you
moave the getsocketopt down here, this is would just be handling the result of
one call.  Or could just use two if lines.  Either way, think the getsocketopt
makes more sense second, since it doesn't affect the setsockopt call.

[Ryan Hamilton, 2016/08/12 22:15:26]

Done. 

I realize that we don't actually need rv here since we use errno not rv to
determine the return value so I actually simplified the code ever so slightly. 

+  }
+
+  int val = IP_PMTUDISC_DO;
+  int rv = setsockopt(socket_, IPPROTO_IP, IP_MTU_DISCOVER, &val, sizeof(val));
+  return rv == 0 ? OK : MapSystemError(errno);
+#endif
+}
+
 int UDPSocketPosix::AllowAddressReuse() {
   DCHECK_NE(socket_, kInvalidSocket);
   DCHECK(CalledOnValidThread());
   DCHECK(!is_connected());
   int true_value = 1;
   int rv = setsockopt(
       socket_, SOL_SOCKET, SO_REUSEADDR, &true_value, sizeof(true_value));
   return rv == 0 ? OK : MapSystemError(errno);
 }
 
@@ skipping 403 matching lines @@
     return MapSystemError(errno);
 
   return OK;
 }
 
 void UDPSocketPosix::DetachFromThread() {
   base::NonThreadSafe::DetachFromThread();
 }
 
 }  // namespace net