 Chromium Code Reviews
 Chromium Code Reviews Issue 129873010:
  Deprecate instead of close SPDY sessions upon network change.  (Closed) 
  Base URL: svn://svn.chromium.org/chrome/trunk/src
    
  
    Issue 129873010:
  Deprecate instead of close SPDY sessions upon network change.  (Closed) 
  Base URL: svn://svn.chromium.org/chrome/trunk/src| OLD | NEW | 
|---|---|
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be | 
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. | 
| 4 | 4 | 
| 5 #ifndef NET_SPDY_SPDY_SESSION_H_ | 5 #ifndef NET_SPDY_SPDY_SESSION_H_ | 
| 6 #define NET_SPDY_SPDY_SESSION_H_ | 6 #define NET_SPDY_SPDY_SESSION_H_ | 
| 7 | 7 | 
| 8 #include <deque> | 8 #include <deque> | 
| 9 #include <map> | 9 #include <map> | 
| 10 #include <set> | 10 #include <set> | 
| (...skipping 335 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 346 // the session as permanently closed. Callers must assume that the | 346 // the session as permanently closed. Callers must assume that the | 
| 347 // session is destroyed after this is called. (However, it may not | 347 // session is destroyed after this is called. (However, it may not | 
| 348 // be destroyed right away, e.g. when a SpdySession function is | 348 // be destroyed right away, e.g. when a SpdySession function is | 
| 349 // present in the call stack.) | 349 // present in the call stack.) | 
| 350 // | 350 // | 
| 351 // |err| should be < ERR_IO_PENDING; this function is intended to be | 351 // |err| should be < ERR_IO_PENDING; this function is intended to be | 
| 352 // called on error. | 352 // called on error. | 
| 353 // |description| indicates the reason for the error. | 353 // |description| indicates the reason for the error. | 
| 354 void CloseSessionOnError(Error err, const std::string& description); | 354 void CloseSessionOnError(Error err, const std::string& description); | 
| 355 | 355 | 
| 356 // Mark this session as going away. It will not be used to service new | |
| 
akalin
2014/01/29 23:26:18
I think the following is better wording:
// Mark
 | |
| 357 // streams. This only changes the session's availability but does not | |
| 358 // actively close streams as StartGoingAway() does. | |
| 359 void MakeUnavailable(); | |
| 360 | |
| 356 // Retrieves information on the current state of the SPDY session as a | 361 // Retrieves information on the current state of the SPDY session as a | 
| 357 // Value. Caller takes possession of the returned value. | 362 // Value. Caller takes possession of the returned value. | 
| 358 base::Value* GetInfoAsValue() const; | 363 base::Value* GetInfoAsValue() const; | 
| 359 | 364 | 
| 360 // Indicates whether the session is being reused after having successfully | 365 // Indicates whether the session is being reused after having successfully | 
| 361 // used to send/receive data in the past. | 366 // used to send/receive data in the past. | 
| 362 bool IsReused() const; | 367 bool IsReused() const; | 
| 363 | 368 | 
| 364 // Returns true if the underlying transport socket ever had any reads or | 369 // Returns true if the underlying transport socket ever had any reads or | 
| 365 // writes. | 370 // writes. | 
| (...skipping 746 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 1112 // This SPDY proxy is allowed to push resources from origins that are | 1117 // This SPDY proxy is allowed to push resources from origins that are | 
| 1113 // different from those of their associated streams. | 1118 // different from those of their associated streams. | 
| 1114 HostPortPair trusted_spdy_proxy_; | 1119 HostPortPair trusted_spdy_proxy_; | 
| 1115 | 1120 | 
| 1116 TimeFunc time_func_; | 1121 TimeFunc time_func_; | 
| 1117 }; | 1122 }; | 
| 1118 | 1123 | 
| 1119 } // namespace net | 1124 } // namespace net | 
| 1120 | 1125 | 
| 1121 #endif // NET_SPDY_SPDY_SESSION_H_ | 1126 #endif // NET_SPDY_SPDY_SESSION_H_ | 
| OLD | NEW |