Update error-handling logic in ChromiumPacketSocketFactory.

remoting/client/plugin/pepper_packet_socket_factory.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 "remoting/client/plugin/pepper_packet_socket_factory.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "net/base/io_buffer.h"
 #include "ppapi/cpp/net_address.h"
@@ skipping 16 matching lines @@
 const int kMaxSendBufferSize = 256 * 1024;
 
 // Enum for different actions that can be taken after sendto() returns an error.
 enum ErrorAction {
   ERROR_ACTION_FAIL,
   ERROR_ACTION_IGNORE,
   ERROR_ACTION_RETRY,
 };
 
 // Returns ErrorAction to perform if sendto() fails with |error|.
+//
+// Keep this logic in sync with GetErrorAction() in
+// remoting/jingle_glue/chromium_socket_factory.cc .

[Wez, 2014/06/17 03:27:54]

Could we convert the PPAPI error code to a net error code here and then call the
version from jingle_glue here?

[Sergey Ulanov, 2014/06/17 22:32:04]

Done.

 ErrorAction GetErrorAction(int error) {
   switch (error) {
     // UDP is connectionless, so we may receive ICMP unreachable or reset errors
     // for previous sends to different addresses.
     case PP_ERROR_ADDRESS_UNREACHABLE:
     case PP_ERROR_CONNECTION_RESET:
       return ERROR_ACTION_RETRY;
 
     // Target address is invalid. The socket is still usable for different
     // target addresses and the error can be ignored.
@@ skipping 370 matching lines @@
   return NULL;
 }
 
 talk_base::AsyncResolverInterface*
 PepperPacketSocketFactory::CreateAsyncResolver() {
   NOTREACHED();
   return NULL;
 }
 
 }  // namespace remoting