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

Side by Side Diff: storage/common/fileapi/file_system_util.cc

Issue 657303002: Convert ARRAYSIZE_UNSAFE -> arraysize in storage/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 unified diff | Download patch
« no previous file with comments | « storage/browser/quota/quota_database.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "storage/common/fileapi/file_system_util.h" 5 #include "storage/common/fileapi/file_system_util.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 8
9 #include "base/files/file_path.h" 9 #include "base/files/file_path.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 153 matching lines...) Expand 10 before | Expand all | Expand 10 after
164 { kFileSystemTypePersistent, kPersistentDir }, 164 { kFileSystemTypePersistent, kPersistentDir },
165 { kFileSystemTypeTemporary, kTemporaryDir }, 165 { kFileSystemTypeTemporary, kTemporaryDir },
166 { kFileSystemTypeIsolated, kIsolatedDir }, 166 { kFileSystemTypeIsolated, kIsolatedDir },
167 { kFileSystemTypeExternal, kExternalDir }, 167 { kFileSystemTypeExternal, kExternalDir },
168 { kFileSystemTypeTest, kTestDir }, 168 { kFileSystemTypeTest, kTestDir },
169 }; 169 };
170 170
171 // A path of the inner_url contains only mount type part (e.g. "/temporary"). 171 // A path of the inner_url contains only mount type part (e.g. "/temporary").
172 DCHECK(url.inner_url()); 172 DCHECK(url.inner_url());
173 std::string inner_path = url.inner_url()->path(); 173 std::string inner_path = url.inner_url()->path();
174 for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kValidTypes); ++i) { 174 for (size_t i = 0; i < arraysize(kValidTypes); ++i) {
175 if (inner_path == kValidTypes[i].dir) { 175 if (inner_path == kValidTypes[i].dir) {
176 file_system_type = kValidTypes[i].type; 176 file_system_type = kValidTypes[i].type;
177 break; 177 break;
178 } 178 }
179 } 179 }
180 180
181 if (file_system_type == kFileSystemTypeUnknown) 181 if (file_system_type == kFileSystemTypeUnknown)
182 return false; 182 return false;
183 183
184 std::string path = net::UnescapeURLComponent(url.path(), 184 std::string path = net::UnescapeURLComponent(url.path(),
(...skipping 317 matching lines...) Expand 10 before | Expand all | Expand 10 after
502 return base::File::FILE_ERROR_ABORT; 502 return base::File::FILE_ERROR_ABORT;
503 case net::ERR_ADDRESS_INVALID: 503 case net::ERR_ADDRESS_INVALID:
504 case net::ERR_INVALID_URL: 504 case net::ERR_INVALID_URL:
505 return base::File::FILE_ERROR_INVALID_URL; 505 return base::File::FILE_ERROR_INVALID_URL;
506 default: 506 default:
507 return base::File::FILE_ERROR_FAILED; 507 return base::File::FILE_ERROR_FAILED;
508 } 508 }
509 } 509 }
510 510
511 } // namespace storage 511 } // namespace storage
OLDNEW
« no previous file with comments | « storage/browser/quota/quota_database.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698