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

Side by Side Diff: content/test/image_decoder_test.cc

Issue 102873002: Move GetFileSize, NormalizeFilePath to base namespace (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years 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 | Annotate | Revision Log
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 "content/test/image_decoder_test.h" 5 #include "content/test/image_decoder_test.h"
6 6
7 #include "base/file_util.h" 7 #include "base/file_util.h"
8 #include "base/files/file_enumerator.h" 8 #include "base/files/file_enumerator.h"
9 #include "base/files/file_path.h" 9 #include "base/files/file_path.h"
10 #include "base/md5.h" 10 #include "base/md5.h"
(...skipping 13 matching lines...) Expand all
24 24
25 // Determine if we should test with file specified by |path| based 25 // Determine if we should test with file specified by |path| based
26 // on |file_selection| and the |threshold| for the file size. 26 // on |file_selection| and the |threshold| for the file size.
27 bool ShouldSkipFile(const base::FilePath& path, 27 bool ShouldSkipFile(const base::FilePath& path,
28 ImageDecoderTestFileSelection file_selection, 28 ImageDecoderTestFileSelection file_selection,
29 const int64 threshold) { 29 const int64 threshold) {
30 if (file_selection == TEST_ALL) 30 if (file_selection == TEST_ALL)
31 return false; 31 return false;
32 32
33 int64 image_size = 0; 33 int64 image_size = 0;
34 file_util::GetFileSize(path, &image_size); 34 base::GetFileSize(path, &image_size);
35 return (file_selection == TEST_SMALLER) == (image_size > threshold); 35 return (file_selection == TEST_SMALLER) == (image_size > threshold);
36 } 36 }
37 37
38 } // namespace 38 } // namespace
39 39
40 void ReadFileToVector(const base::FilePath& path, std::vector<char>* contents) { 40 void ReadFileToVector(const base::FilePath& path, std::vector<char>* contents) {
41 std::string raw_image_data; 41 std::string raw_image_data;
42 base::ReadFileToString(path, &raw_image_data); 42 base::ReadFileToString(path, &raw_image_data);
43 contents->resize(raw_image_data.size()); 43 contents->resize(raw_image_data.size());
44 memcpy(&contents->at(0), raw_image_data.data(), raw_image_data.size()); 44 memcpy(&contents->at(0), raw_image_data.data(), raw_image_data.size());
(...skipping 172 matching lines...) Expand 10 before | Expand all | Expand 10 after
217 // Since WebImage does not expose get data by frame, get the size 217 // Since WebImage does not expose get data by frame, get the size
218 // through decoder and pass it to fromData so that the closest 218 // through decoder and pass it to fromData so that the closest
219 // image dats to the size is returned. 219 // image dats to the size is returned.
220 blink::WebSize size(decoder->getImage(desired_frame_index).size()); 220 blink::WebSize size(decoder->getImage(desired_frame_index).size());
221 const blink::WebImage& image = blink::WebImage::fromData(data, size); 221 const blink::WebImage& image = blink::WebImage::fromData(data, size);
222 SaveMD5Sum(md5_sum_path, image); 222 SaveMD5Sum(md5_sum_path, image);
223 #else 223 #else
224 VerifyImage(*decoder, image_path, md5_sum_path, desired_frame_index); 224 VerifyImage(*decoder, image_path, md5_sum_path, desired_frame_index);
225 #endif 225 #endif
226 } 226 }
OLDNEW
« no previous file with comments | « content/renderer/media/webrtc_audio_device_unittest.cc ('k') | content/test/weburl_loader_mock_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698