Remove unnecessary synchronization while getting/setting ContainerVar…

… values. Refs issue lift#1629.

web/webkit/src/main/scala/net/liftweb/http/Vars.scala

@@ -181,14 +181,14 @@ private[http] trait HasLogUnreadVal {
  * provides a subset of these.
  */
 abstract class ContainerVar[T](dflt: => T)(implicit containerSerializer: ContainerSerializer[T]) extends AnyVar[T, ContainerVar[T]](dflt) with LazyLoggable {
+
   override protected def findFunc(name: String): Box[T] = S.session match {
     case Full(session) => {
       localGet(session, name) match {
         case Full(array: Array[Byte]) => Full(containerSerializer.deserialize(array))
         case _ => Empty
       }
     }
-
     case _ => {
       if (showWarningWhenAccessedOutOfSessionScope_?)
         logger.warn("Getting a SessionVar " + name + " outside session scope") // added warning per issue 188
@@ -233,7 +233,7 @@ abstract class ContainerVar[T](dflt: => T)(implicit containerSerializer: Contain
    * In the case of ContainerVar, we synchronize on the ContainerVar
    * instance itself.
    */
-  def doSync[F](f: => F): F = this.synchronized(f)
+  def doSync[F](f: => F): F = f
 
   def showWarningWhenAccessedOutOfSessionScope_? = false