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

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

Issue 1311433009: A few updates on connection states (Closed) Base URL: https://chromium.googlesource.com/external/webrtc@master
Patch Set: Created 5 years, 3 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
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
11 #include <sstream> 11 #include <sstream>
12 #include "webrtc/p2p/base/common.h" 12 #include "webrtc/p2p/base/common.h"
13 #include "webrtc/p2p/base/transportchannel.h" 13 #include "webrtc/p2p/base/transportchannel.h"
14 14
15 namespace cricket { 15 namespace cricket {
16 16
17 std::string TransportChannel::ToString() const { 17 std::string TransportChannel::ToString() const {
18 const char READABLE_ABBREV[2] = { '_', 'R' }; 18 const char RECEIVING_ABBREV[2] = {'_', 'R'};
19 const char WRITABLE_ABBREV[2] = { '_', 'W' }; 19 const char WRITABLE_ABBREV[2] = { '_', 'W' };
20 std::stringstream ss; 20 std::stringstream ss;
21 ss << "Channel[" << content_name_ 21 ss << "Channel[" << content_name_ << "|" << component_ << "|"
22 << "|" << component_ 22 << RECEIVING_ABBREV[receiving_] << WRITABLE_ABBREV[writable_] << "]";
23 << "|" << READABLE_ABBREV[readable_] << WRITABLE_ABBREV[writable_] << "]";
24 return ss.str(); 23 return ss.str();
25 } 24 }
26 25
27 void TransportChannel::set_readable(bool readable) {
28 if (readable_ != readable) {
29 readable_ = readable;
30 SignalReadableState(this);
31 }
32 }
33
34 void TransportChannel::set_receiving(bool receiving) { 26 void TransportChannel::set_receiving(bool receiving) {
35 if (receiving_ == receiving) { 27 if (receiving_ == receiving) {
36 return; 28 return;
37 } 29 }
30 LOG(LS_INFO) << "XXX Set receiving " << receiving;
pthatcher1 2015/09/17 05:58:42 XXX?
honghaiz3 2015/09/17 19:47:57 Ah Sorry. It was removed in a later patch.
38 receiving_ = receiving; 31 receiving_ = receiving;
39 SignalReceivingState(this); 32 SignalReceivingState(this);
40 } 33 }
41 34
42 void TransportChannel::set_writable(bool writable) { 35 void TransportChannel::set_writable(bool writable) {
43 if (writable_ != writable) { 36 if (writable_ != writable) {
44 LOG_J(LS_VERBOSE, this) << "set_writable from:" << writable_ << " to " 37 LOG_J(LS_VERBOSE, this) << "set_writable from:" << writable_ << " to "
45 << writable; 38 << writable;
46 writable_ = writable; 39 writable_ = writable;
47 if (writable_) { 40 if (writable_) {
48 SignalReadyToSend(this); 41 SignalReadyToSend(this);
49 } 42 }
50 SignalWritableState(this); 43 SignalWritableState(this);
51 } 44 }
52 } 45 }
53 46
54 } // namespace cricket 47 } // namespace cricket
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698