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

Issue 1645443002: [Platform] Add BT.601 and BT.709 video color profile data (Closed)

Created:
4 years, 11 months ago by Noel Gordon
Modified:
4 years, 11 months ago
CC:
chromium-reviews, krit, pdr+graphicswatchlist_chromium.org, drott+blinkwatch_chromium.org, vmpstr+blinkwatch_chromium.org, dshwang, jbroman, Justin Novosad, blink-reviews-platform-graphics_chromium.org, Rik, Stephen Chennney, blink-reviews, f(malita), danakj+watch_chromium.org, kinuko+watch, rwlbuis, enne (OOO)
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[Platform] Add BT.601 and BT.709 video color profile data Add BT.601 and BT.709 video color profiles as static data to Blink Platform. The data was extracted from the golden files in issue 570150 entries #24 and #25 with a script [1]. This data will be used in subsequent changes so no change in behavior, no new tests. [1] #!/bin/bash -v # create an array from static data (echo "const unsigned char data[] = {"; od -txC -v $1 | \ sed -e "s/^[0-9]*//" -e s"/ \([0-9a-f][0-9a-f]\)/0x\1,/g" \ -e "\$d" | sed -e"\$s/,$/};/") > data.c cat data.c BUG=570150 Committed: https://crrev.com/ea1b30c75bf726889619cc9c6d20479ce7f917a8 Cr-Commit-Position: refs/heads/master@{#371978}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+140 lines, -0 lines) Patch
M third_party/WebKit/Source/platform/blink_platform.gypi View 1 chunk +2 lines, -0 lines 0 comments Download
A third_party/WebKit/Source/platform/graphics/ColorSpaceProfileData.h View 1 chunk +18 lines, -0 lines 0 comments Download
A third_party/WebKit/Source/platform/graphics/ColorSpaceProfileData.cpp View 1 chunk +120 lines, -0 lines 0 comments Download

Messages

Total messages: 17 (12 generated)
Noel Gordon
4 years, 11 months ago (2016-01-27 09:28:20 UTC) #3
Ken Russell (switch to Gerrit)
lgtm
4 years, 11 months ago (2016-01-28 01:26:07 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1645443002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1645443002/1
4 years, 11 months ago (2016-01-28 02:48:49 UTC) #9
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 11 months ago (2016-01-28 02:58:02 UTC) #15
commit-bot: I haz the power
4 years, 11 months ago (2016-01-28 02:59:33 UTC) #17
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/ea1b30c75bf726889619cc9c6d20479ce7f917a8
Cr-Commit-Position: refs/heads/master@{#371978}

Powered by Google App Engine
This is Rietveld 408576698