Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(176)

Unified Diff: net/ftp/ftp_network_layer.cc

Issue 2428143002: Clean up FtpTransactionFacory ownership. (Closed)
Patch Set: Response to comments Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/ftp/ftp_network_layer.cc
diff --git a/net/ftp/ftp_network_layer.cc b/net/ftp/ftp_network_layer.cc
index 2c11fc7b5bac901f7cb534938d179dc3577b4e39..3167803c9586385d37c96d78122e3acc342bf8f6 100644
--- a/net/ftp/ftp_network_layer.cc
+++ b/net/ftp/ftp_network_layer.cc
@@ -4,6 +4,7 @@
#include "net/ftp/ftp_network_layer.h"
+#include "base/logging.h"
#include "base/memory/ptr_util.h"
#include "net/ftp/ftp_network_session.h"
#include "net/ftp/ftp_network_transaction.h"
@@ -14,6 +15,7 @@ namespace net {
FtpNetworkLayer::FtpNetworkLayer(HostResolver* host_resolver)
: session_(new FtpNetworkSession(host_resolver)),
suspended_(false) {
+ DCHECK(host_resolver);
}
FtpNetworkLayer::~FtpNetworkLayer() {

Powered by Google App Engine
This is Rietveld 408576698