QUIC - Make |visitor_| nullptr check consistent with other nullptr

net/quic/quic_connection.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/quic/quic_connection.h"
 
 #include <string.h>
 #include <sys/types.h>
 
 #include <algorithm>
@@ skipping 2059 matching lines @@
 void QuicConnection::CloseConnection(QuicErrorCode error, bool from_peer) {
   if (!connected_) {
     DVLOG(1) << "Connection is already closed.";
     return;
   }
   connected_ = false;
   DCHECK(visitor_ != nullptr);
   // TODO(rtenneti): crbug.com/546668. A temporary fix. Added a check for null
   // |visitor_| to fix crash bug. Delete |visitor_| check and histogram after
   // fix is merged.
-  if (visitor_) {
+  if (visitor_ != nullptr) {
     visitor_->OnConnectionClosed(error, from_peer);
   } else {
     UMA_HISTOGRAM_BOOLEAN("Net.QuicCloseConnection.NullVisitor", true);
   }
   if (debug_visitor_ != nullptr) {
     debug_visitor_->OnConnectionClosed(error, from_peer);
   }
   // Cancel the alarms so they don't trigger any action now that the
   // connection is closed.
   ack_alarm_->Cancel();
@@ skipping 444 matching lines @@
 void QuicConnection::OnPathClosed(QuicPathId path_id) {
   // Stop receiving packets on this path.
   framer_.OnPathClosed(path_id);
 }
 
 bool QuicConnection::ack_frame_updated() const {
   return received_packet_manager_.ack_frame_updated();
 }
 
 }  // namespace net