Dreamview: put AINFO within lock (#659)

modules/dreamview/backend/simulation_world/simulation_world_updater.cc

@@ -187,7 +187,8 @@ bool SimulationWorldUpdater::ConstructRoutingRequest(
     map_service_->ConstructLaneWayPoint(end["x"], end["y"], endLane);
   }
 
-  AINFO << "Waypoints: " << routing_request->DebugString();
+  AINFO << "Constructed RoutingRequest to be sent, waypoints: "
+        << routing_request->DebugString();
 
   return true;
 }

modules/dreamview/backend/websocket/websocket.cc

@@ -31,8 +31,8 @@ void WebSocketHandler::handleReadyState(CivetServer *server, Connection *conn) {
   {
     std::unique_lock<std::mutex> lock(mutex_);
     connections_.emplace(conn, std::make_shared<std::mutex>());
+    AINFO << "Accepted connection. Total connections: " << connections_.size();
   }
-  AINFO << "Accepted connection. Total connections: " << connections_.size();
 }
 
 void WebSocketHandler::handleClose(CivetServer *server,
@@ -48,8 +48,8 @@ void WebSocketHandler::handleClose(CivetServer *server,
       std::unique_lock<std::mutex> lock(*connection_lock);
       connections_.erase(connection);
     }
+    AINFO << "Connection closed. Total connections: " << connections_.size();
   }
-  AINFO << "Connection closed. Total connections: " << connections_.size();
 }
 
 bool WebSocketHandler::BroadcastData(const std::string &data) {

modules/dreamview/frontend/src/utils/draw.js

@@ -57,11 +57,7 @@ export function drawCircle(radius, material, segments = 32) {
 
 export function drawThickBandFromPoints(
     points, thickness = 0.5, color = 0xffffff, opacity = 1, zOffset = 0) {
-    // const quality = 5;
-    // const curve = bezier(points.map(p => [p.x, p.y]), quality);
-
     const geometry = Line(points.map(p => [p.x, p.y]));
-
     const material = new THREE.ShaderMaterial(BasicShader({
         side: THREE.DoubleSide,
         diffuse: color,