Redo the SslOption integration into AmqpTcpEndpoint

After looking at the eventual integration into ConnectionFactory,
I've found it better to restructure things a bit differently so
that we create our SslOption very early on in the process and
just pass it down all the way to the SocketFrameHandler.


This commit does the following:

    * Rename SslOption Property to Ssl, so we actually
      create a Namespace, which allows us to do:
      endpoint.Ssl.Enabled, endpoint.Ssl.Foo, etc
    * Remove SslIsEnabled Property, it now resides inside
      the SslOption object itself, and AmqpTcpEndpoint does
      not concern itself with it anymore
    * Remove the UseSsl() family of methods.
    * Implement an overloaded constructor that accepts an SslOption
      object.
    * Reimplement the existing constructor in terms of the new constructor above,
      so that by default, Ssl is disabled

Author: essiene

src/client/api/AmqpTcpEndpoint.cs

@@ -91,47 +91,35 @@ public int Port
             set { m_port = value; }
         }
 
-        private bool m_sslIsEnabled;
 
-        public bool SslIsEnabled
-        {
-            get { return m_sslIsEnabled; }
-            set { m_sslIsEnabled = value; }
-        }
-
-        private SslOption m_sslOption;
+        private SslOption m_ssl;
         ///<summary>Retrieve the SSL options for this AmqpTcpEndpoint.
         ///If not set, null is returned</summary>
-        public SslOption SslOption
+        public SslOption Ssl
         {
-            get { return m_sslOption; }
+            get { return m_ssl; }
+            set { m_ssl = value; }
         }
 
         ///<summary>Construct an AmqpTcpEndpoint with the given
         ///IProtocol, hostname, and port number. If the port number is
         ///-1, the default port number for the IProtocol will be
         ///used.</summary>
-        public AmqpTcpEndpoint(IProtocol protocolVariant, string hostname, int portOrMinusOne)
-        {
-            m_protocol = protocolVariant;
-            m_hostName = hostname;
-            m_port = portOrMinusOne;
-        }
-
-        public void UseSsl(string certPath, string serverName)
-        {
-            m_sslOption = new SslOption(serverName, certPath);
-            m_sslIsEnabled = true;
-        }
-
-        public void UseSsl(string certPath)
+        public AmqpTcpEndpoint(IProtocol protocolVariant, string hostname, int portOrMinusOne):
+            this(protocolVariant, hostname, portOrMinusOne, new SslOption())
         {
-            UseSsl(certPath, m_hostName);
         }
 
-        public void UseSsl()
+        ///<summary>Construct an AmqpTcpEndpoint with the given
+        ///IProtocol, hostname, port number and Ssl option. If the port 
+        ///number is -1, the default port number for the IProtocol 
+        ///will be used.</summary>
+        public AmqpTcpEndpoint(IProtocol protocolVariant, string hostname, int portOrMinusOne, SslOption ssl)
         {
-            UseSsl("", m_hostName);
+            m_protocol = protocolVariant;
+            m_hostName = hostname;
+            m_port = portOrMinusOne;
+            m_ssl = ssl;
         }
 
         ///<summary>Returns a URI-like string of the form