Add ClientSocketHandle NULL CHECK in HttpStreamParser constructor

BUG=790776

Change-Id: I445eb3ef0dc957342ef2afc62fed3c51b06f893a
Reviewed-on: https://chromium-review.googlesource.com/804059Reviewed-by: Helen Li <xunjieli@chromium.org>
Commit-Queue: Helen Li <xunjieli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521113}

net/http/http_basic_state.cc

@@ -24,7 +24,10 @@ HttpBasicState::HttpBasicState(std::unique_ptr<ClientSocketHandle> connection,
       connection_(std::move(connection)),
       using_proxy_(using_proxy),
       http_09_on_non_default_ports_enabled_(
-          http_09_on_non_default_ports_enabled) {}
+          http_09_on_non_default_ports_enabled) {
+  CHECK(connection_) << "ClientSocketHandle passed to HttpBasicState must "
+                        "not be NULL. See crbug.com/790776";
+}
 
 HttpBasicState::~HttpBasicState() = default;
 

net/http/http_stream_parser.cc

@@ -207,6 +207,8 @@ HttpStreamParser::HttpStreamParser(ClientSocketHandle* connection,
       sent_last_chunk_(false),
       upload_error_(OK),
       weak_ptr_factory_(this) {
+  CHECK(connection_) << "ClientSocketHandle passed to HttpStreamParser must "
+                        "not be NULL. See crbug.com/790776";
   io_callback_ = base::Bind(&HttpStreamParser::OnIOComplete,
                             weak_ptr_factory_.GetWeakPtr());
 }