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

Unified Diff: net/ftp/ftp_directory_listing_parser_windows.cc

Issue 992733002: Remove //net (except for Android test stuff) and sdch (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 9 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_directory_listing_parser_windows.cc
diff --git a/net/ftp/ftp_directory_listing_parser_windows.cc b/net/ftp/ftp_directory_listing_parser_windows.cc
deleted file mode 100644
index 2a50220587796431c34f0a84af81e06ac18101db..0000000000000000000000000000000000000000
--- a/net/ftp/ftp_directory_listing_parser_windows.cc
+++ /dev/null
@@ -1,72 +0,0 @@
-// 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_directory_listing_parser_windows.h"
-
-#include <vector>
-
-#include "base/strings/string_number_conversions.h"
-#include "base/strings/string_split.h"
-#include "base/strings/string_util.h"
-#include "base/time/time.h"
-#include "net/ftp/ftp_directory_listing_parser.h"
-#include "net/ftp/ftp_util.h"
-
-namespace net {
-
-bool ParseFtpDirectoryListingWindows(
- const std::vector<base::string16>& lines,
- std::vector<FtpDirectoryListingEntry>* entries) {
- for (size_t i = 0; i < lines.size(); i++) {
- if (lines[i].empty())
- continue;
-
- std::vector<base::string16> columns;
- base::SplitString(base::CollapseWhitespace(lines[i], false), ' ', &columns);
-
- // Every line of the listing consists of the following:
- //
- // 1. date
- // 2. time
- // 3. size in bytes (or "<DIR>" for directories)
- // 4. filename (may be empty or contain spaces)
- //
- // For now, make sure we have 1-3, and handle 4 later.
- if (columns.size() < 3)
- return false;
-
- FtpDirectoryListingEntry entry;
- if (EqualsASCII(columns[2], "<DIR>")) {
- entry.type = FtpDirectoryListingEntry::DIRECTORY;
- entry.size = -1;
- } else {
- entry.type = FtpDirectoryListingEntry::FILE;
- if (!base::StringToInt64(columns[2], &entry.size))
- return false;
- if (entry.size < 0)
- return false;
- }
-
- if (!FtpUtil::WindowsDateListingToTime(columns[0],
- columns[1],
- &entry.last_modified)) {
- return false;
- }
-
- entry.name = FtpUtil::GetStringPartAfterColumns(lines[i], 3);
- if (entry.name.empty()) {
- // Some FTP servers send listing entries with empty names.
- // It's not obvious how to display such an entry, so ignore them.
- // We don't want to make the parsing fail at this point though.
- // Other entries can still be useful.
- continue;
- }
-
- entries->push_back(entry);
- }
-
- return true;
-}
-
-} // namespace net
« no previous file with comments | « net/ftp/ftp_directory_listing_parser_windows.h ('k') | net/ftp/ftp_directory_listing_parser_windows_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698