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

Side by Side Diff: ui/gfx/icc_profile.h

Issue 2502703002: Add more robust gfx::ColorSpace<->gfx::ICCProfile conversion (Closed)
Patch Set: Created 4 years, 1 month 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 | « no previous file | ui/gfx/icc_profile.cc » ('j') | ui/gfx/mojo/icc_profile_struct_traits.cc » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 UI_GFX_ICC_PROFILE_H_ 5 #ifndef UI_GFX_ICC_PROFILE_H_
6 #define UI_GFX_ICC_PROFILE_H_ 6 #define UI_GFX_ICC_PROFILE_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 #if defined(OS_WIN) 66 #if defined(OS_WIN)
67 // This will read monitor ICC profiles from disk and cache the results for the 67 // This will read monitor ICC profiles from disk and cache the results for the
68 // other functions to read. This should not be called on the UI or IO thread. 68 // other functions to read. This should not be called on the UI or IO thread.
69 static void UpdateCachedProfilesOnBackgroundThread(); 69 static void UpdateCachedProfilesOnBackgroundThread();
70 static bool CachedProfilesNeedUpdate(); 70 static bool CachedProfilesNeedUpdate();
71 #endif 71 #endif
72 72
73 private: 73 private:
74 static bool IsValidProfileLength(size_t length); 74 static bool IsValidProfileLength(size_t length);
75 75
76 bool valid_ = false; 76 enum class Type {
77 // This is not a valid profile.
78 INVALID,
79 // This is from a gfx::ColorSpace. This ensures that GetColorSpace returns
80 // the exact same object as was used to create this.
81 FROM_COLOR_SPACE,
82 // This was created from ICC profile data.
83 FROM_DATA,
84 LAST = FROM_DATA
85 };
86 Type type_ = Type::INVALID;
87 gfx::ColorSpace color_space_;
77 std::vector<char> data_; 88 std::vector<char> data_;
78 89
79 // This globally identifies this ICC profile. It is used to look up this ICC 90 // This globally identifies this ICC profile. It is used to look up this ICC
80 // profile from a ColorSpace object created from it. 91 // profile from a ColorSpace object created from it.
81 uint64_t id_ = 0; 92 uint64_t id_ = 0;
82 93
83 FRIEND_TEST_ALL_PREFIXES(SimpleColorSpace, BT709toSRGBICC); 94 FRIEND_TEST_ALL_PREFIXES(SimpleColorSpace, BT709toSRGBICC);
84 FRIEND_TEST_ALL_PREFIXES(SimpleColorSpace, GetColorSpace); 95 FRIEND_TEST_ALL_PREFIXES(SimpleColorSpace, GetColorSpace);
85 friend int ::LLVMFuzzerTestOneInput(const uint8_t*, size_t); 96 friend int ::LLVMFuzzerTestOneInput(const uint8_t*, size_t);
86 friend class ColorSpace; 97 friend class ColorSpace;
87 friend struct IPC::ParamTraits<gfx::ICCProfile>; 98 friend struct IPC::ParamTraits<gfx::ICCProfile>;
99 friend struct IPC::ParamTraits<gfx::ICCProfile::Type>;
88 friend struct mojo::StructTraits<gfx::mojom::ICCProfileDataView, 100 friend struct mojo::StructTraits<gfx::mojom::ICCProfileDataView,
89 gfx::ICCProfile>; 101 gfx::ICCProfile>;
90 }; 102 };
91 103
92 } // namespace gfx 104 } // namespace gfx
93 105
94 #endif // UI_GFX_ICC_PROFILE_H_ 106 #endif // UI_GFX_ICC_PROFILE_H_
OLDNEW
« no previous file with comments | « no previous file | ui/gfx/icc_profile.cc » ('j') | ui/gfx/mojo/icc_profile_struct_traits.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698