OLD | NEW |
1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 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 "chrome/browser/chromeos/file_system_provider/fileapi/file_stream_reade
r.h" | 5 #include "chrome/browser/chromeos/file_system_provider/fileapi/file_stream_reade
r.h" |
6 | 6 |
| 7 #include <stddef.h> |
| 8 #include <stdint.h> |
| 9 |
7 #include <string> | 10 #include <string> |
8 #include <vector> | 11 #include <vector> |
9 | 12 |
10 #include "base/files/file.h" | 13 #include "base/files/file.h" |
11 #include "base/files/file_path.h" | 14 #include "base/files/file_path.h" |
12 #include "base/files/scoped_temp_dir.h" | 15 #include "base/files/scoped_temp_dir.h" |
| 16 #include "base/macros.h" |
13 #include "base/memory/scoped_ptr.h" | 17 #include "base/memory/scoped_ptr.h" |
14 #include "base/memory/weak_ptr.h" | 18 #include "base/memory/weak_ptr.h" |
15 #include "base/numerics/safe_math.h" | 19 #include "base/numerics/safe_math.h" |
16 #include "base/run_loop.h" | 20 #include "base/run_loop.h" |
17 #include "chrome/browser/chromeos/file_system_provider/fake_provided_file_system
.h" | 21 #include "chrome/browser/chromeos/file_system_provider/fake_provided_file_system
.h" |
18 #include "chrome/browser/chromeos/file_system_provider/service.h" | 22 #include "chrome/browser/chromeos/file_system_provider/service.h" |
19 #include "chrome/browser/chromeos/file_system_provider/service_factory.h" | 23 #include "chrome/browser/chromeos/file_system_provider/service_factory.h" |
20 #include "chrome/test/base/testing_browser_process.h" | 24 #include "chrome/test/base/testing_browser_process.h" |
21 #include "chrome/test/base/testing_profile.h" | 25 #include "chrome/test/base/testing_profile.h" |
22 #include "chrome/test/base/testing_profile_manager.h" | 26 #include "chrome/test/base/testing_profile_manager.h" |
(...skipping 14 matching lines...) Expand all Loading... |
37 const char kExtensionId[] = "mbflcebpggnecokmikipoihdbecnjfoj"; | 41 const char kExtensionId[] = "mbflcebpggnecokmikipoihdbecnjfoj"; |
38 const char kFileSystemId[] = "testing-file-system"; | 42 const char kFileSystemId[] = "testing-file-system"; |
39 | 43 |
40 // Logs callbacks invocations on the file stream reader. | 44 // Logs callbacks invocations on the file stream reader. |
41 class EventLogger { | 45 class EventLogger { |
42 public: | 46 public: |
43 EventLogger() : weak_ptr_factory_(this) {} | 47 EventLogger() : weak_ptr_factory_(this) {} |
44 virtual ~EventLogger() {} | 48 virtual ~EventLogger() {} |
45 | 49 |
46 void OnRead(int result) { results_.push_back(result); } | 50 void OnRead(int result) { results_.push_back(result); } |
47 void OnGetLength(int64 result) { results_.push_back(result); } | 51 void OnGetLength(int64_t result) { results_.push_back(result); } |
48 | 52 |
49 base::WeakPtr<EventLogger> GetWeakPtr() { | 53 base::WeakPtr<EventLogger> GetWeakPtr() { |
50 return weak_ptr_factory_.GetWeakPtr(); | 54 return weak_ptr_factory_.GetWeakPtr(); |
51 } | 55 } |
52 | 56 |
53 const std::vector<int64>& results() const { return results_; } | 57 const std::vector<int64_t>& results() const { return results_; } |
54 | 58 |
55 private: | 59 private: |
56 std::vector<int64> results_; | 60 std::vector<int64_t> results_; |
57 base::WeakPtrFactory<EventLogger> weak_ptr_factory_; | 61 base::WeakPtrFactory<EventLogger> weak_ptr_factory_; |
58 | 62 |
59 DISALLOW_COPY_AND_ASSIGN(EventLogger); | 63 DISALLOW_COPY_AND_ASSIGN(EventLogger); |
60 }; | 64 }; |
61 | 65 |
62 // Creates a cracked FileSystemURL for tests. | 66 // Creates a cracked FileSystemURL for tests. |
63 storage::FileSystemURL CreateFileSystemURL(const std::string& mount_point_name, | 67 storage::FileSystemURL CreateFileSystemURL(const std::string& mount_point_name, |
64 const base::FilePath& file_path) { | 68 const base::FilePath& file_path) { |
65 const std::string origin = std::string("chrome-extension://") + kExtensionId; | 69 const std::string origin = std::string("chrome-extension://") + kExtensionId; |
66 const storage::ExternalMountPoints* const mount_points = | 70 const storage::ExternalMountPoints* const mount_points = |
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
117 scoped_ptr<TestingProfileManager> profile_manager_; | 121 scoped_ptr<TestingProfileManager> profile_manager_; |
118 TestingProfile* profile_; // Owned by TestingProfileManager. | 122 TestingProfile* profile_; // Owned by TestingProfileManager. |
119 const FakeEntry* fake_file_; // Owned by FakePRovidedFileSystem. | 123 const FakeEntry* fake_file_; // Owned by FakePRovidedFileSystem. |
120 storage::FileSystemURL file_url_; | 124 storage::FileSystemURL file_url_; |
121 storage::FileSystemURL wrong_file_url_; | 125 storage::FileSystemURL wrong_file_url_; |
122 }; | 126 }; |
123 | 127 |
124 TEST_F(FileSystemProviderFileStreamReader, Read_AllAtOnce) { | 128 TEST_F(FileSystemProviderFileStreamReader, Read_AllAtOnce) { |
125 EventLogger logger; | 129 EventLogger logger; |
126 | 130 |
127 const int64 initial_offset = 0; | 131 const int64_t initial_offset = 0; |
128 FileStreamReader reader(NULL, file_url_, initial_offset, | 132 FileStreamReader reader(NULL, file_url_, initial_offset, |
129 *fake_file_->metadata->modification_time); | 133 *fake_file_->metadata->modification_time); |
130 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer( | 134 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer( |
131 base::CheckedNumeric<size_t>(*fake_file_->metadata->size).ValueOrDie())); | 135 base::CheckedNumeric<size_t>(*fake_file_->metadata->size).ValueOrDie())); |
132 | 136 |
133 const int result = | 137 const int result = |
134 reader.Read(io_buffer.get(), *fake_file_->metadata->size, | 138 reader.Read(io_buffer.get(), *fake_file_->metadata->size, |
135 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); | 139 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); |
136 EXPECT_EQ(net::ERR_IO_PENDING, result); | 140 EXPECT_EQ(net::ERR_IO_PENDING, result); |
137 base::RunLoop().RunUntilIdle(); | 141 base::RunLoop().RunUntilIdle(); |
138 | 142 |
139 ASSERT_EQ(1u, logger.results().size()); | 143 ASSERT_EQ(1u, logger.results().size()); |
140 EXPECT_LT(0, logger.results()[0]); | 144 EXPECT_LT(0, logger.results()[0]); |
141 EXPECT_EQ(*fake_file_->metadata->size, logger.results()[0]); | 145 EXPECT_EQ(*fake_file_->metadata->size, logger.results()[0]); |
142 | 146 |
143 std::string buffer_as_string(io_buffer->data(), *fake_file_->metadata->size); | 147 std::string buffer_as_string(io_buffer->data(), *fake_file_->metadata->size); |
144 EXPECT_EQ(fake_file_->contents, buffer_as_string); | 148 EXPECT_EQ(fake_file_->contents, buffer_as_string); |
145 } | 149 } |
146 | 150 |
147 TEST_F(FileSystemProviderFileStreamReader, Read_WrongFile) { | 151 TEST_F(FileSystemProviderFileStreamReader, Read_WrongFile) { |
148 EventLogger logger; | 152 EventLogger logger; |
149 | 153 |
150 const int64 initial_offset = 0; | 154 const int64_t initial_offset = 0; |
151 FileStreamReader reader(NULL, wrong_file_url_, initial_offset, | 155 FileStreamReader reader(NULL, wrong_file_url_, initial_offset, |
152 *fake_file_->metadata->modification_time); | 156 *fake_file_->metadata->modification_time); |
153 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer( | 157 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer( |
154 base::CheckedNumeric<size_t>(*fake_file_->metadata->size).ValueOrDie())); | 158 base::CheckedNumeric<size_t>(*fake_file_->metadata->size).ValueOrDie())); |
155 | 159 |
156 const int result = | 160 const int result = |
157 reader.Read(io_buffer.get(), *fake_file_->metadata->size, | 161 reader.Read(io_buffer.get(), *fake_file_->metadata->size, |
158 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); | 162 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); |
159 EXPECT_EQ(net::ERR_IO_PENDING, result); | 163 EXPECT_EQ(net::ERR_IO_PENDING, result); |
160 base::RunLoop().RunUntilIdle(); | 164 base::RunLoop().RunUntilIdle(); |
161 | 165 |
162 ASSERT_EQ(1u, logger.results().size()); | 166 ASSERT_EQ(1u, logger.results().size()); |
163 EXPECT_EQ(net::ERR_FILE_NOT_FOUND, logger.results()[0]); | 167 EXPECT_EQ(net::ERR_FILE_NOT_FOUND, logger.results()[0]); |
164 } | 168 } |
165 | 169 |
166 TEST_F(FileSystemProviderFileStreamReader, Read_InChunks) { | 170 TEST_F(FileSystemProviderFileStreamReader, Read_InChunks) { |
167 EventLogger logger; | 171 EventLogger logger; |
168 | 172 |
169 const int64 initial_offset = 0; | 173 const int64_t initial_offset = 0; |
170 FileStreamReader reader(NULL, file_url_, initial_offset, | 174 FileStreamReader reader(NULL, file_url_, initial_offset, |
171 *fake_file_->metadata->modification_time); | 175 *fake_file_->metadata->modification_time); |
172 | 176 |
173 for (int64 offset = 0; offset < *fake_file_->metadata->size; ++offset) { | 177 for (int64_t offset = 0; offset < *fake_file_->metadata->size; ++offset) { |
174 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer(1)); | 178 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer(1)); |
175 const int result = | 179 const int result = |
176 reader.Read(io_buffer.get(), | 180 reader.Read(io_buffer.get(), |
177 1, | 181 1, |
178 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); | 182 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); |
179 EXPECT_EQ(net::ERR_IO_PENDING, result); | 183 EXPECT_EQ(net::ERR_IO_PENDING, result); |
180 base::RunLoop().RunUntilIdle(); | 184 base::RunLoop().RunUntilIdle(); |
181 ASSERT_EQ(offset + 1, static_cast<int64>(logger.results().size())); | 185 ASSERT_EQ(offset + 1, static_cast<int64_t>(logger.results().size())); |
182 EXPECT_EQ(1, logger.results()[offset]); | 186 EXPECT_EQ(1, logger.results()[offset]); |
183 EXPECT_EQ(fake_file_->contents[offset], io_buffer->data()[0]); | 187 EXPECT_EQ(fake_file_->contents[offset], io_buffer->data()[0]); |
184 } | 188 } |
185 } | 189 } |
186 | 190 |
187 TEST_F(FileSystemProviderFileStreamReader, Read_Slice) { | 191 TEST_F(FileSystemProviderFileStreamReader, Read_Slice) { |
188 EventLogger logger; | 192 EventLogger logger; |
189 | 193 |
190 // Trim first 3 and last 3 characters. | 194 // Trim first 3 and last 3 characters. |
191 const int64 initial_offset = 3; | 195 const int64_t initial_offset = 3; |
192 const int length = *fake_file_->metadata->size - initial_offset - 3; | 196 const int length = *fake_file_->metadata->size - initial_offset - 3; |
193 ASSERT_GT(*fake_file_->metadata->size, initial_offset); | 197 ASSERT_GT(*fake_file_->metadata->size, initial_offset); |
194 ASSERT_LT(0, length); | 198 ASSERT_LT(0, length); |
195 | 199 |
196 FileStreamReader reader(NULL, file_url_, initial_offset, | 200 FileStreamReader reader(NULL, file_url_, initial_offset, |
197 *fake_file_->metadata->modification_time); | 201 *fake_file_->metadata->modification_time); |
198 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer(length)); | 202 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer(length)); |
199 | 203 |
200 const int result = | 204 const int result = |
201 reader.Read(io_buffer.get(), | 205 reader.Read(io_buffer.get(), |
202 length, | 206 length, |
203 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); | 207 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); |
204 EXPECT_EQ(net::ERR_IO_PENDING, result); | 208 EXPECT_EQ(net::ERR_IO_PENDING, result); |
205 base::RunLoop().RunUntilIdle(); | 209 base::RunLoop().RunUntilIdle(); |
206 | 210 |
207 ASSERT_EQ(1u, logger.results().size()); | 211 ASSERT_EQ(1u, logger.results().size()); |
208 EXPECT_EQ(length, logger.results()[0]); | 212 EXPECT_EQ(length, logger.results()[0]); |
209 | 213 |
210 std::string buffer_as_string(io_buffer->data(), length); | 214 std::string buffer_as_string(io_buffer->data(), length); |
211 std::string expected_buffer(fake_file_->contents.data() + initial_offset, | 215 std::string expected_buffer(fake_file_->contents.data() + initial_offset, |
212 length); | 216 length); |
213 EXPECT_EQ(expected_buffer, buffer_as_string); | 217 EXPECT_EQ(expected_buffer, buffer_as_string); |
214 } | 218 } |
215 | 219 |
216 TEST_F(FileSystemProviderFileStreamReader, Read_Beyond) { | 220 TEST_F(FileSystemProviderFileStreamReader, Read_Beyond) { |
217 EventLogger logger; | 221 EventLogger logger; |
218 | 222 |
219 // Request reading 1KB more than available. | 223 // Request reading 1KB more than available. |
220 const int64 initial_offset = 0; | 224 const int64_t initial_offset = 0; |
221 const int length = *fake_file_->metadata->size + 1024; | 225 const int length = *fake_file_->metadata->size + 1024; |
222 | 226 |
223 FileStreamReader reader(NULL, file_url_, initial_offset, | 227 FileStreamReader reader(NULL, file_url_, initial_offset, |
224 *fake_file_->metadata->modification_time); | 228 *fake_file_->metadata->modification_time); |
225 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer(length)); | 229 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer(length)); |
226 | 230 |
227 const int result = | 231 const int result = |
228 reader.Read(io_buffer.get(), | 232 reader.Read(io_buffer.get(), |
229 length, | 233 length, |
230 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); | 234 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); |
231 EXPECT_EQ(net::ERR_IO_PENDING, result); | 235 EXPECT_EQ(net::ERR_IO_PENDING, result); |
232 base::RunLoop().RunUntilIdle(); | 236 base::RunLoop().RunUntilIdle(); |
233 | 237 |
234 ASSERT_EQ(1u, logger.results().size()); | 238 ASSERT_EQ(1u, logger.results().size()); |
235 EXPECT_LT(0, logger.results()[0]); | 239 EXPECT_LT(0, logger.results()[0]); |
236 EXPECT_EQ(*fake_file_->metadata->size, logger.results()[0]); | 240 EXPECT_EQ(*fake_file_->metadata->size, logger.results()[0]); |
237 | 241 |
238 std::string buffer_as_string(io_buffer->data(), *fake_file_->metadata->size); | 242 std::string buffer_as_string(io_buffer->data(), *fake_file_->metadata->size); |
239 EXPECT_EQ(fake_file_->contents, buffer_as_string); | 243 EXPECT_EQ(fake_file_->contents, buffer_as_string); |
240 } | 244 } |
241 | 245 |
242 TEST_F(FileSystemProviderFileStreamReader, Read_ModifiedFile) { | 246 TEST_F(FileSystemProviderFileStreamReader, Read_ModifiedFile) { |
243 EventLogger logger; | 247 EventLogger logger; |
244 | 248 |
245 const int64 initial_offset = 0; | 249 const int64_t initial_offset = 0; |
246 FileStreamReader reader(NULL, file_url_, initial_offset, base::Time::Max()); | 250 FileStreamReader reader(NULL, file_url_, initial_offset, base::Time::Max()); |
247 | 251 |
248 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer( | 252 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer( |
249 base::CheckedNumeric<size_t>(*fake_file_->metadata->size).ValueOrDie())); | 253 base::CheckedNumeric<size_t>(*fake_file_->metadata->size).ValueOrDie())); |
250 const int result = | 254 const int result = |
251 reader.Read(io_buffer.get(), *fake_file_->metadata->size, | 255 reader.Read(io_buffer.get(), *fake_file_->metadata->size, |
252 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); | 256 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); |
253 | 257 |
254 EXPECT_EQ(net::ERR_IO_PENDING, result); | 258 EXPECT_EQ(net::ERR_IO_PENDING, result); |
255 base::RunLoop().RunUntilIdle(); | 259 base::RunLoop().RunUntilIdle(); |
256 | 260 |
257 ASSERT_EQ(1u, logger.results().size()); | 261 ASSERT_EQ(1u, logger.results().size()); |
258 EXPECT_EQ(net::ERR_UPLOAD_FILE_CHANGED, logger.results()[0]); | 262 EXPECT_EQ(net::ERR_UPLOAD_FILE_CHANGED, logger.results()[0]); |
259 } | 263 } |
260 | 264 |
261 TEST_F(FileSystemProviderFileStreamReader, Read_ExpectedModificationTimeNull) { | 265 TEST_F(FileSystemProviderFileStreamReader, Read_ExpectedModificationTimeNull) { |
262 EventLogger logger; | 266 EventLogger logger; |
263 | 267 |
264 const int64 initial_offset = 0; | 268 const int64_t initial_offset = 0; |
265 FileStreamReader reader(NULL, file_url_, initial_offset, base::Time()); | 269 FileStreamReader reader(NULL, file_url_, initial_offset, base::Time()); |
266 | 270 |
267 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer( | 271 scoped_refptr<net::IOBuffer> io_buffer(new net::IOBuffer( |
268 base::CheckedNumeric<size_t>(*fake_file_->metadata->size).ValueOrDie())); | 272 base::CheckedNumeric<size_t>(*fake_file_->metadata->size).ValueOrDie())); |
269 const int result = | 273 const int result = |
270 reader.Read(io_buffer.get(), *fake_file_->metadata->size, | 274 reader.Read(io_buffer.get(), *fake_file_->metadata->size, |
271 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); | 275 base::Bind(&EventLogger::OnRead, logger.GetWeakPtr())); |
272 | 276 |
273 EXPECT_EQ(net::ERR_IO_PENDING, result); | 277 EXPECT_EQ(net::ERR_IO_PENDING, result); |
274 base::RunLoop().RunUntilIdle(); | 278 base::RunLoop().RunUntilIdle(); |
275 | 279 |
276 ASSERT_EQ(1u, logger.results().size()); | 280 ASSERT_EQ(1u, logger.results().size()); |
277 EXPECT_EQ(*fake_file_->metadata->size, logger.results()[0]); | 281 EXPECT_EQ(*fake_file_->metadata->size, logger.results()[0]); |
278 | 282 |
279 std::string buffer_as_string(io_buffer->data(), *fake_file_->metadata->size); | 283 std::string buffer_as_string(io_buffer->data(), *fake_file_->metadata->size); |
280 EXPECT_EQ(fake_file_->contents, buffer_as_string); | 284 EXPECT_EQ(fake_file_->contents, buffer_as_string); |
281 } | 285 } |
282 | 286 |
283 TEST_F(FileSystemProviderFileStreamReader, GetLength) { | 287 TEST_F(FileSystemProviderFileStreamReader, GetLength) { |
284 EventLogger logger; | 288 EventLogger logger; |
285 | 289 |
286 const int64 initial_offset = 0; | 290 const int64_t initial_offset = 0; |
287 FileStreamReader reader(NULL, file_url_, initial_offset, | 291 FileStreamReader reader(NULL, file_url_, initial_offset, |
288 *fake_file_->metadata->modification_time); | 292 *fake_file_->metadata->modification_time); |
289 | 293 |
290 const int result = reader.GetLength( | 294 const int result = reader.GetLength( |
291 base::Bind(&EventLogger::OnGetLength, logger.GetWeakPtr())); | 295 base::Bind(&EventLogger::OnGetLength, logger.GetWeakPtr())); |
292 EXPECT_EQ(net::ERR_IO_PENDING, result); | 296 EXPECT_EQ(net::ERR_IO_PENDING, result); |
293 base::RunLoop().RunUntilIdle(); | 297 base::RunLoop().RunUntilIdle(); |
294 | 298 |
295 ASSERT_EQ(1u, logger.results().size()); | 299 ASSERT_EQ(1u, logger.results().size()); |
296 EXPECT_LT(0, logger.results()[0]); | 300 EXPECT_LT(0, logger.results()[0]); |
297 EXPECT_EQ(*fake_file_->metadata->size, logger.results()[0]); | 301 EXPECT_EQ(*fake_file_->metadata->size, logger.results()[0]); |
298 } | 302 } |
299 | 303 |
300 TEST_F(FileSystemProviderFileStreamReader, GetLength_WrongFile) { | 304 TEST_F(FileSystemProviderFileStreamReader, GetLength_WrongFile) { |
301 EventLogger logger; | 305 EventLogger logger; |
302 | 306 |
303 const int64 initial_offset = 0; | 307 const int64_t initial_offset = 0; |
304 FileStreamReader reader(NULL, wrong_file_url_, initial_offset, | 308 FileStreamReader reader(NULL, wrong_file_url_, initial_offset, |
305 *fake_file_->metadata->modification_time); | 309 *fake_file_->metadata->modification_time); |
306 | 310 |
307 const int result = reader.GetLength( | 311 const int result = reader.GetLength( |
308 base::Bind(&EventLogger::OnGetLength, logger.GetWeakPtr())); | 312 base::Bind(&EventLogger::OnGetLength, logger.GetWeakPtr())); |
309 EXPECT_EQ(net::ERR_IO_PENDING, result); | 313 EXPECT_EQ(net::ERR_IO_PENDING, result); |
310 base::RunLoop().RunUntilIdle(); | 314 base::RunLoop().RunUntilIdle(); |
311 | 315 |
312 ASSERT_EQ(1u, logger.results().size()); | 316 ASSERT_EQ(1u, logger.results().size()); |
313 EXPECT_EQ(net::ERR_FILE_NOT_FOUND, logger.results()[0]); | 317 EXPECT_EQ(net::ERR_FILE_NOT_FOUND, logger.results()[0]); |
314 } | 318 } |
315 | 319 |
316 TEST_F(FileSystemProviderFileStreamReader, GetLength_ModifiedFile) { | 320 TEST_F(FileSystemProviderFileStreamReader, GetLength_ModifiedFile) { |
317 EventLogger logger; | 321 EventLogger logger; |
318 | 322 |
319 const int64 initial_offset = 0; | 323 const int64_t initial_offset = 0; |
320 FileStreamReader reader(NULL, file_url_, initial_offset, base::Time::Max()); | 324 FileStreamReader reader(NULL, file_url_, initial_offset, base::Time::Max()); |
321 | 325 |
322 const int result = reader.GetLength( | 326 const int result = reader.GetLength( |
323 base::Bind(&EventLogger::OnGetLength, logger.GetWeakPtr())); | 327 base::Bind(&EventLogger::OnGetLength, logger.GetWeakPtr())); |
324 EXPECT_EQ(net::ERR_IO_PENDING, result); | 328 EXPECT_EQ(net::ERR_IO_PENDING, result); |
325 base::RunLoop().RunUntilIdle(); | 329 base::RunLoop().RunUntilIdle(); |
326 | 330 |
327 ASSERT_EQ(1u, logger.results().size()); | 331 ASSERT_EQ(1u, logger.results().size()); |
328 EXPECT_EQ(net::ERR_UPLOAD_FILE_CHANGED, logger.results()[0]); | 332 EXPECT_EQ(net::ERR_UPLOAD_FILE_CHANGED, logger.results()[0]); |
329 } | 333 } |
330 | 334 |
331 TEST_F(FileSystemProviderFileStreamReader, | 335 TEST_F(FileSystemProviderFileStreamReader, |
332 GetLength_ExpectedModificationTimeNull) { | 336 GetLength_ExpectedModificationTimeNull) { |
333 EventLogger logger; | 337 EventLogger logger; |
334 | 338 |
335 const int64 initial_offset = 0; | 339 const int64_t initial_offset = 0; |
336 FileStreamReader reader(NULL, file_url_, initial_offset, base::Time()); | 340 FileStreamReader reader(NULL, file_url_, initial_offset, base::Time()); |
337 | 341 |
338 const int result = reader.GetLength( | 342 const int result = reader.GetLength( |
339 base::Bind(&EventLogger::OnGetLength, logger.GetWeakPtr())); | 343 base::Bind(&EventLogger::OnGetLength, logger.GetWeakPtr())); |
340 EXPECT_EQ(net::ERR_IO_PENDING, result); | 344 EXPECT_EQ(net::ERR_IO_PENDING, result); |
341 base::RunLoop().RunUntilIdle(); | 345 base::RunLoop().RunUntilIdle(); |
342 | 346 |
343 ASSERT_EQ(1u, logger.results().size()); | 347 ASSERT_EQ(1u, logger.results().size()); |
344 EXPECT_LT(0, logger.results()[0]); | 348 EXPECT_LT(0, logger.results()[0]); |
345 EXPECT_EQ(*fake_file_->metadata->size, logger.results()[0]); | 349 EXPECT_EQ(*fake_file_->metadata->size, logger.results()[0]); |
346 } | 350 } |
347 | 351 |
348 } // namespace file_system_provider | 352 } // namespace file_system_provider |
349 } // namespace chromeos | 353 } // namespace chromeos |
OLD | NEW |