Commit e5b4a511 authored by Hwanseung Lee's avatar Hwanseung Lee Committed by Commit Bot

Rename ColorSpaceUtilities namepsace to color_space_utilities

https://google.github.io/styleguide/cppguide.html#Namespace_Names
> Namespace names are all lower-case.

This CL has no behavior changes.

Bug: 889726
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I361dbb5b1bf477623a772f1b0e6d074283bf6c5d
Reviewed-on: https://chromium-review.googlesource.com/c/1258642Reviewed-by: default avatarKent Tamura <tkent@chromium.org>
Commit-Queue: Hwanseung Lee <hs1217.lee@samsung.com>
Cr-Commit-Position: refs/heads/master@{#596126}
parent 0b429a5b
...@@ -172,7 +172,7 @@ DisplayShape MediaValues::CalculateDisplayShape(LocalFrame* frame) { ...@@ -172,7 +172,7 @@ DisplayShape MediaValues::CalculateDisplayShape(LocalFrame* frame) {
ColorSpaceGamut MediaValues::CalculateColorGamut(LocalFrame* frame) { ColorSpaceGamut MediaValues::CalculateColorGamut(LocalFrame* frame) {
DCHECK(frame); DCHECK(frame);
DCHECK(frame->GetPage()); DCHECK(frame->GetPage());
return ColorSpaceUtilities::GetColorSpaceGamut( return color_space_utilities::GetColorSpaceGamut(
frame->GetPage()->GetChromeClient().GetScreenInfo()); frame->GetPage()->GetChromeClient().GetScreenInfo());
} }
......
...@@ -78,8 +78,8 @@ void BitmapImageMetrics::CountImageGammaAndGamut( ...@@ -78,8 +78,8 @@ void BitmapImageMetrics::CountImageGammaAndGamut(
DEFINE_THREAD_SAFE_STATIC_LOCAL( DEFINE_THREAD_SAFE_STATIC_LOCAL(
EnumerationHistogram, gamut_named_histogram, EnumerationHistogram, gamut_named_histogram,
("Blink.ColorGamut.Source", static_cast<int>(ColorSpaceGamut::kEnd))); ("Blink.ColorGamut.Source", static_cast<int>(ColorSpaceGamut::kEnd)));
gamut_named_histogram.Count( gamut_named_histogram.Count(static_cast<int>(
static_cast<int>(ColorSpaceUtilities::GetColorSpaceGamut(color_profile))); color_space_utilities::GetColorSpaceGamut(color_profile)));
} }
void BitmapImageMetrics::CountJpegArea(const IntSize& size) { void BitmapImageMetrics::CountJpegArea(const IntSize& size) {
......
...@@ -9,7 +9,7 @@ ...@@ -9,7 +9,7 @@
namespace blink { namespace blink {
namespace ColorSpaceUtilities { namespace color_space_utilities {
ColorSpaceGamut GetColorSpaceGamut(const WebScreenInfo& screen_info) { ColorSpaceGamut GetColorSpaceGamut(const WebScreenInfo& screen_info) {
const gfx::ColorSpace& color_space = screen_info.color_space; const gfx::ColorSpace& color_space = screen_info.color_space;
...@@ -19,7 +19,7 @@ ColorSpaceGamut GetColorSpaceGamut(const WebScreenInfo& screen_info) { ...@@ -19,7 +19,7 @@ ColorSpaceGamut GetColorSpaceGamut(const WebScreenInfo& screen_info) {
// wide gamut for HDR profiles). // wide gamut for HDR profiles).
skcms_ICCProfile color_profile; skcms_ICCProfile color_profile;
color_space.GetRasterColorSpace().ToSkColorSpace()->toProfile(&color_profile); color_space.GetRasterColorSpace().ToSkColorSpace()->toProfile(&color_profile);
return ColorSpaceUtilities::GetColorSpaceGamut(&color_profile); return color_space_utilities::GetColorSpaceGamut(&color_profile);
} }
ColorSpaceGamut GetColorSpaceGamut(const skcms_ICCProfile* color_profile) { ColorSpaceGamut GetColorSpaceGamut(const skcms_ICCProfile* color_profile) {
...@@ -62,6 +62,6 @@ ColorSpaceGamut GetColorSpaceGamut(const skcms_ICCProfile* color_profile) { ...@@ -62,6 +62,6 @@ ColorSpaceGamut GetColorSpaceGamut(const skcms_ICCProfile* color_profile) {
return ColorSpaceGamut::kUltraWide; return ColorSpaceGamut::kUltraWide;
} }
} // namespace ColorSpaceUtilities } // namespace color_space_utilities
} // namespace blink } // namespace blink
...@@ -29,12 +29,12 @@ enum class ColorSpaceGamut { ...@@ -29,12 +29,12 @@ enum class ColorSpaceGamut {
kEnd kEnd
}; };
namespace ColorSpaceUtilities { namespace color_space_utilities {
PLATFORM_EXPORT ColorSpaceGamut GetColorSpaceGamut(const WebScreenInfo&); PLATFORM_EXPORT ColorSpaceGamut GetColorSpaceGamut(const WebScreenInfo&);
ColorSpaceGamut GetColorSpaceGamut(const skcms_ICCProfile*); ColorSpaceGamut GetColorSpaceGamut(const skcms_ICCProfile*);
} // namespace ColorSpaceUtilities } // namespace color_space_utilities
} // namespace blink } // namespace blink
......
...@@ -290,6 +290,12 @@ _CONFIG = [ ...@@ -290,6 +290,12 @@ _CONFIG = [
'detail::.+', 'detail::.+',
], ],
}, },
{
'paths': ['third_party/blink/renderer/core/css/media_values.cc'],
'allowed': [
'color_space_utilities::GetColorSpaceGamut',
],
},
{ {
'paths': ['third_party/blink/renderer/core/fetch/data_consumer_handle_test_util.cc'], 'paths': ['third_party/blink/renderer/core/fetch/data_consumer_handle_test_util.cc'],
'allowed': [ 'allowed': [
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment