Make callers of FromUTC(Local)Exploded in net/ use new time API.

net/ftp/ftp_util.cc

 // Copyright (c) 2011 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_util.h"
 
 #include <map>
 #include <vector>
 
 #include "base/i18n/case_conversion.h"
@@ skipping 269 matching lines @@
         (time_exploded.month == current_exploded.month &&
          time_exploded.day_of_month > current_exploded.day_of_month)) {
       time_exploded.year = current_exploded.year - 1;
     } else {
       time_exploded.year = current_exploded.year;
     }
   }
 
   // We don't know the time zone of the listing, so just use local time.
   *result = base::Time::FromLocalExploded(time_exploded);
   return true;

[maksims (do not use this acc), 2016/11/29 08:41:18]

eroman@, can we leave these parts as they are for now?

[eroman, 2016/11/30 00:21:05]

Sure leave it as is -- you can disable the failing browser_test on Linux and
point to crbug.com/641096 or crbug.com/666535. I need to follow-up on on that.

[maksims (do not use this acc), 2016/12/05 13:15:49]

If I leave it as is here, the test is not failing. I will subscribe to those
issues as well and fix these lines here once the problem is sorted.

[eroman, 2016/12/08 01:39:19]

Sure, sounds good!

 }
 
 // static
 bool FtpUtil::WindowsDateListingToTime(const base::string16& date,
                                        const base::string16& time,
                                        base::Time* result) {
   base::Time::Exploded time_exploded = { 0 };
 
   // Date should be in format MM-DD-YY[YY].
   std::vector<base::StringPiece16> date_parts =
@@ skipping 40 matching lines @@
       if (time_exploded.hour < 12)
         time_exploded.hour += 12;
     } else if (base::EqualsASCII(am_or_pm, "AM")) {
       if (time_exploded.hour == 12)
         time_exploded.hour = 0;
     } else {
       return false;
     }
   }
 
-  // We don't know the time zone of the server, so just use local time.
+  // We don't know the time zone of the listing, so just use local time.
   *result = base::Time::FromLocalExploded(time_exploded);
   return true;
 }
 
 // static
 base::string16 FtpUtil::GetStringPartAfterColumns(const base::string16& text,
                                                   int columns) {
   base::i18n::UTF16CharIterator iter(&text);
 
   for (int i = 0; i < columns; i++) {
     // Skip the leading whitespace.
     while (!iter.end() && u_isspace(iter.get()))
       iter.Advance();
 
     // Skip the actual text of i-th column.
     while (!iter.end() && !u_isspace(iter.get()))
       iter.Advance();
   }
 
   base::string16 result(text.substr(iter.array_pos()));
   base::TrimWhitespace(result, base::TRIM_ALL, &result);
   return result;
 }
 
 }  // namespace net