Undisable half of new ftp unit tests in url_request_unittest.cc.

net/ftp/ftp_network_transaction.cc

 // Copyright (c) 2008 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/ftp/ftp_network_transaction.h"
 
 #include "base/compiler_specific.h"
 #include "base/string_util.h"
 #include "net/base/connection_type_histograms.h"
 #include "net/base/net_errors.h"
@@ skipping 895 matching lines @@
   DCHECK(read_data_buf_);
   DCHECK(read_data_buf_len_ > 0);
 
   next_state_ = STATE_DATA_READ_COMPLETE;
   read_data_buf_->data()[0] = 0;
   return data_socket_->Read(read_data_buf_, read_data_buf_len_,
                             &io_callback_);
 }
 
 int FtpNetworkTransaction::DoDataReadComplete(int result) {
-  DLOG(INFO) << read_data_buf_->data();  // The read_data_buf_ is NULL
-                                         // terminated string.
+  DLOG(INFO) << std::string(read_data_buf_->data(), result);

[wtc, 2009/07/30 18:02:32]

This should be protected with if (result > 0).

But in Ibrar's CL http://codereview.chromium.org/146137,
he removed this DLOG(INFO) statement, so I think you
should remove this file from your CL, and let Ibrar's CL
take care of this.

By the way, I'm also not clear on why the data in the
read_data_buf_ is null terminated.  Ibrar says it's
only null terminated if the data is a directory listing.
But I still don't fully understand it.

   return result;
 }
 
 }  // namespace net