Bug 1127158 - Remove brittle debug only flag math in nsSecureBrowserUIImpl.cpp. draft
authorCykesiopka <cykesiopka.bmo@gmail.com>
Fri, 01 Apr 2016 06:16:58 -0700
changeset 346624 e4975b766aa0d4b477a762038e0b06bf7d3bc881
parent 346622 2ad120057ac1087ee2ac08c348e250c271f5c010
child 517484 96b0eb1ffafd050b571c6fa70f05168489d042e9
push id14420
push usercykesiopka.bmo@gmail.com
push dateFri, 01 Apr 2016 13:25:05 +0000
bugs1127158
milestone48.0a1
Bug 1127158 - Remove brittle debug only flag math in nsSecureBrowserUIImpl.cpp. MozReview-Commit-ID: 3d5mYDjzJwf
security/manager/ssl/nsSecureBrowserUIImpl.cpp
--- a/security/manager/ssl/nsSecureBrowserUIImpl.cpp
+++ b/security/manager/ssl/nsSecureBrowserUIImpl.cpp
@@ -704,124 +704,16 @@ nsSecureBrowserUIImpl::OnStateChange(nsI
                                    nsIProtocolHandler::URI_IS_LOCAL_RESOURCE,
                                    &hasFlag);
     if (NS_SUCCEEDED(rv) && hasFlag) {
       isSubDocumentRelevant = false;
     }
   }
 
 #if defined(DEBUG)
-  nsCString info2;
-  uint32_t testFlags = loadFlags;
-
-  if (testFlags & nsIChannel::LOAD_DOCUMENT_URI)
-  {
-    testFlags -= nsIChannel::LOAD_DOCUMENT_URI;
-    info2.AppendLiteral("LOAD_DOCUMENT_URI ");
-  }
-  if (testFlags & nsIChannel::LOAD_RETARGETED_DOCUMENT_URI)
-  {
-    testFlags -= nsIChannel::LOAD_RETARGETED_DOCUMENT_URI;
-    info2.AppendLiteral("LOAD_RETARGETED_DOCUMENT_URI ");
-  }
-  if (testFlags & nsIChannel::LOAD_REPLACE)
-  {
-    testFlags -= nsIChannel::LOAD_REPLACE;
-    info2.AppendLiteral("LOAD_REPLACE ");
-  }
-
-  const char *_status = NS_SUCCEEDED(aStatus) ? "1" : "0";
-
-  nsCString info;
-  uint32_t f = aProgressStateFlags;
-  if (f & nsIWebProgressListener::STATE_START)
-  {
-    f -= nsIWebProgressListener::STATE_START;
-    info.AppendLiteral("START ");
-  }
-  if (f & nsIWebProgressListener::STATE_REDIRECTING)
-  {
-    f -= nsIWebProgressListener::STATE_REDIRECTING;
-    info.AppendLiteral("REDIRECTING ");
-  }
-  if (f & nsIWebProgressListener::STATE_TRANSFERRING)
-  {
-    f -= nsIWebProgressListener::STATE_TRANSFERRING;
-    info.AppendLiteral("TRANSFERRING ");
-  }
-  if (f & nsIWebProgressListener::STATE_NEGOTIATING)
-  {
-    f -= nsIWebProgressListener::STATE_NEGOTIATING;
-    info.AppendLiteral("NEGOTIATING ");
-  }
-  if (f & nsIWebProgressListener::STATE_STOP)
-  {
-    f -= nsIWebProgressListener::STATE_STOP;
-    info.AppendLiteral("STOP ");
-  }
-  if (f & nsIWebProgressListener::STATE_IS_REQUEST)
-  {
-    f -= nsIWebProgressListener::STATE_IS_REQUEST;
-    info.AppendLiteral("IS_REQUEST ");
-  }
-  if (f & nsIWebProgressListener::STATE_IS_DOCUMENT)
-  {
-    f -= nsIWebProgressListener::STATE_IS_DOCUMENT;
-    info.AppendLiteral("IS_DOCUMENT ");
-  }
-  if (f & nsIWebProgressListener::STATE_IS_NETWORK)
-  {
-    f -= nsIWebProgressListener::STATE_IS_NETWORK;
-    info.AppendLiteral("IS_NETWORK ");
-  }
-  if (f & nsIWebProgressListener::STATE_IS_WINDOW)
-  {
-    f -= nsIWebProgressListener::STATE_IS_WINDOW;
-    info.AppendLiteral("IS_WINDOW ");
-  }
-  if (f & nsIWebProgressListener::STATE_IS_INSECURE)
-  {
-    f -= nsIWebProgressListener::STATE_IS_INSECURE;
-    info.AppendLiteral("IS_INSECURE ");
-  }
-  if (f & nsIWebProgressListener::STATE_IS_BROKEN)
-  {
-    f -= nsIWebProgressListener::STATE_IS_BROKEN;
-    info.AppendLiteral("IS_BROKEN ");
-  }
-  if (f & nsIWebProgressListener::STATE_IS_SECURE)
-  {
-    f -= nsIWebProgressListener::STATE_IS_SECURE;
-    info.AppendLiteral("IS_SECURE ");
-  }
-  if (f & nsIWebProgressListener::STATE_SECURE_HIGH)
-  {
-    f -= nsIWebProgressListener::STATE_SECURE_HIGH;
-    info.AppendLiteral("SECURE_HIGH ");
-  }
-  if (f & nsIWebProgressListener::STATE_CERT_USER_OVERRIDDEN)
-  {
-    f -= nsIWebProgressListener::STATE_CERT_USER_OVERRIDDEN;
-    info.AppendLiteral("STATE_CERT_USER_OVERRIDDEN ");
-  }
-  if (f & nsIWebProgressListener::STATE_RESTORING)
-  {
-    f -= nsIWebProgressListener::STATE_RESTORING;
-    info.AppendLiteral("STATE_RESTORING ");
-  }
-
-  if (f > 0)
-  {
-    info.AppendLiteral("f contains unknown flag!");
-  }
-
-  MOZ_LOG(gSecureDocLog, LogLevel::Debug,
-         ("SecureUI:%p: OnStateChange: %s %s -- %s\n", this, _status, 
-          info.get(), info2.get()));
-
   if (aProgressStateFlags & STATE_STOP
       &&
       channel)
   {
     MOZ_LOG(gSecureDocLog, LogLevel::Debug,
            ("SecureUI:%p: OnStateChange: seeing STOP with security state: %d\n", this,
             GetSecurityStateFromSecurityInfoAndRequest(securityInfo, aRequest)
             ));