Bug 946116 - Remove unused isResumedSession variable from nsNSSCallbacks.cpp. r=bsmith
authorChris Peterson <cpeterson@mozilla.com>
Tue, 03 Dec 2013 21:07:29 -0800
changeset 174791 77116d8569628db3c5c287bd11b90781230374d8
parent 174790 42ebb2419d34455979a1d8246e1378569af5d925
child 174792 28ef06bb9b8b30a1e2399f48c022b9b04fea47e0
push id445
push userffxbld
push dateMon, 10 Mar 2014 22:05:19 +0000
treeherdermozilla-release@dc38b741b04e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbsmith
bugs946116
milestone28.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 946116 - Remove unused isResumedSession variable from nsNSSCallbacks.cpp. r=bsmith
security/manager/ssl/src/nsNSSCallbacks.cpp
--- a/security/manager/ssl/src/nsNSSCallbacks.cpp
+++ b/security/manager/ssl/src/nsNSSCallbacks.cpp
@@ -1124,21 +1124,16 @@ AccumulateCipherSuite(Telemetry::ID prob
 }
 
 void HandshakeCallback(PRFileDesc* fd, void* client_data) {
   nsNSSShutDownPreventionLock locker;
   SECStatus rv;
 
   nsNSSSocketInfo* infoObject = (nsNSSSocketInfo*) fd->higher->secret;
 
-  // certificate validation sets IsFullHandshake, so if that flag
-  // is absent at handshake time we have a resumed session. Check this before
-  // PreliminaryHandshakeDone() because that function also sets that flag.
-  bool isResumedSession = !infoObject->IsFullHandshake();
-
   // Do the bookkeeping that needs to be done after the
   // server's ServerHello...ServerHelloDone have been processed, but that doesn't
   // need the handshake to be completed.
   PreliminaryHandshakeDone(fd);
 
   nsSSLIOLayerHelpers& ioLayerHelpers
     = infoObject->SharedState().IOLayerHelpers();