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

Side by Side Diff: webrtc/p2p/base/port.cc

Issue 2099783002: Fixing bug where Connection drops packets when presumed writable. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Merge with master. Created 4 years, 5 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 unified diff | Download patch
« no previous file with comments | « webrtc/p2p/base/port.h ('k') | webrtc/p2p/base/port_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2004 The WebRTC Project Authors. All rights reserved. 2 * Copyright 2004 The WebRTC Project Authors. All rights reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 1507 matching lines...) Expand 10 before | Expand all | Expand 10 after
1518 return waiting > 2 * rtt(); 1518 return waiting > 2 * rtt();
1519 } 1519 }
1520 1520
1521 ProxyConnection::ProxyConnection(Port* port, 1521 ProxyConnection::ProxyConnection(Port* port,
1522 size_t index, 1522 size_t index,
1523 const Candidate& remote_candidate) 1523 const Candidate& remote_candidate)
1524 : Connection(port, index, remote_candidate) {} 1524 : Connection(port, index, remote_candidate) {}
1525 1525
1526 int ProxyConnection::Send(const void* data, size_t size, 1526 int ProxyConnection::Send(const void* data, size_t size,
1527 const rtc::PacketOptions& options) { 1527 const rtc::PacketOptions& options) {
1528 if (!ReadyToSendMedia()) {
1529 error_ = EWOULDBLOCK;
1530 return SOCKET_ERROR;
1531 }
1532 stats_.sent_total_packets++; 1528 stats_.sent_total_packets++;
1533 int sent = port_->SendTo(data, size, remote_candidate_.address(), 1529 int sent = port_->SendTo(data, size, remote_candidate_.address(),
1534 options, true); 1530 options, true);
1535 if (sent <= 0) { 1531 if (sent <= 0) {
1536 ASSERT(sent < 0); 1532 ASSERT(sent < 0);
1537 error_ = port_->GetError(); 1533 error_ = port_->GetError();
1538 stats_.sent_discarded_packets++; 1534 stats_.sent_discarded_packets++;
1539 } else { 1535 } else {
1540 send_rate_tracker_.AddSamples(sent); 1536 send_rate_tracker_.AddSamples(sent);
1541 } 1537 }
1542 return sent; 1538 return sent;
1543 } 1539 }
1544 1540
1545 } // namespace cricket 1541 } // namespace cricket
OLDNEW
« no previous file with comments | « webrtc/p2p/base/port.h ('k') | webrtc/p2p/base/port_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698