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

Side by Side Diff: chrome/browser/chromeos/login/users/avatar/user_image_manager_test_util.h

Issue 931993002: Make image_decoder a Leaky LazyInstance (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix a few comments 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 unified diff | Download patch
OLDNEW
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 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_USERS_AVATAR_USER_IMAGE_MANAGER_TEST_UTIL_ H_ 5 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_USERS_AVATAR_USER_IMAGE_MANAGER_TEST_UTIL_ H_
6 #define CHROME_BROWSER_CHROMEOS_LOGIN_USERS_AVATAR_USER_IMAGE_MANAGER_TEST_UTIL_ H_ 6 #define CHROME_BROWSER_CHROMEOS_LOGIN_USERS_AVATAR_USER_IMAGE_MANAGER_TEST_UTIL_ H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "base/files/file_path.h" 10 #include "base/files/file_path.h"
(...skipping 13 matching lines...) Expand all
24 24
25 namespace chromeos { 25 namespace chromeos {
26 namespace test { 26 namespace test {
27 27
28 extern const char kUserAvatarImage1RelativePath[]; 28 extern const char kUserAvatarImage1RelativePath[];
29 extern const char kUserAvatarImage2RelativePath[]; 29 extern const char kUserAvatarImage2RelativePath[];
30 30
31 // Returns |true| if the two given images are pixel-for-pixel identical. 31 // Returns |true| if the two given images are pixel-for-pixel identical.
32 bool AreImagesEqual(const gfx::ImageSkia& first, const gfx::ImageSkia& second); 32 bool AreImagesEqual(const gfx::ImageSkia& first, const gfx::ImageSkia& second);
33 33
34 class ImageLoader : public ImageDecoder::Delegate { 34 class ImageLoader : public ImageDecoder::ImageRequest {
35 public: 35 public:
36 explicit ImageLoader(const base::FilePath& path); 36 explicit ImageLoader(const base::FilePath& path);
37 ~ImageLoader() override; 37 ~ImageLoader() override;
38 38
39 scoped_ptr<gfx::ImageSkia> Load(); 39 scoped_ptr<gfx::ImageSkia> Load();
40 40
41 // ImageDecoder::Delegate: 41 // ImageDecoder::ImageRequest:
42 void OnImageDecoded(const ImageDecoder* decoder, 42 void OnImageDecoded(const SkBitmap& decoded_image) override;
43 const SkBitmap& decoded_image) override; 43 void OnDecodeImageFailed() override;
44 void OnDecodeImageFailed(const ImageDecoder* decoder) override;
45 44
46 private: 45 private:
47 base::FilePath path_; 46 base::FilePath path_;
48 base::RunLoop run_loop_; 47 base::RunLoop run_loop_;
49 48
50 scoped_ptr<gfx::ImageSkia> decoded_image_; 49 scoped_ptr<gfx::ImageSkia> decoded_image_;
51 50
52 DISALLOW_COPY_AND_ASSIGN(ImageLoader); 51 DISALLOW_COPY_AND_ASSIGN(ImageLoader);
53 }; 52 };
54 53
55 } // namespace test 54 } // namespace test
56 } // namespace chromeos 55 } // namespace chromeos
57 56
58 #endif // CHROME_BROWSER_CHROMEOS_LOGIN_USERS_AVATAR_USER_IMAGE_MANAGER_TEST_UT IL_H_ 57 #endif // CHROME_BROWSER_CHROMEOS_LOGIN_USERS_AVATAR_USER_IMAGE_MANAGER_TEST_UT IL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698