[blink-dev] Intent to Ship: WebGL canvas color management

2022-05-17 Thread 'Christopher Cameron' via blink-dev
Contact emailsccame...@chromium.org Specification This is part of the WebGL specification. In particular: - 5.14.1 context attributes discusses drawingBufferColorSpace and unpackColorSpace - 5.14.8 texture objects

Re: [blink-dev] Intent to Ship: WebGL canvas color management

2022-05-17 Thread Ken Russell
LGTM not as API owner, but as member of Khronos' WebGL working group (and also the chair). -Ken On Tue, May 17, 2022 at 3:40 PM 'Christopher Cameron' via blink-dev < blink-dev@chromium.org> wrote: > Contact emailsccame...@chromium.org > > Specification > This is part of the WebGL specification

Re: [blink-dev] Intent to Ship: WebGL canvas color management

2022-05-18 Thread Rick Byers
This looks like a pretty straight forward small addition. Just a couple questions: On Tue, May 17, 2022 at 7:35 PM Ken Russell wrote: > LGTM not as API owner, but as member of Khronos' WebGL working group (and > also the chair). > > -Ken > > On Tue, May 17, 2022 at 3:40 PM 'Christopher Cameron'

Re: [blink-dev] Intent to Ship: WebGL canvas color management

2022-05-18 Thread Ken Russell
On Wed, May 18, 2022 at 11:25 AM Rick Byers wrote: > This looks like a pretty straight forward small addition. Just a couple > questions: > > On Tue, May 17, 2022 at 7:35 PM Ken Russell wrote: > >> LGTM not as API owner, but as member of Khronos' WebGL working group (and >> also the chair). >> >

Re: [blink-dev] Intent to Ship: WebGL canvas color management

2022-05-18 Thread Chris Harrelson
Hi, comments below. On Wed, May 18, 2022 at 11:48 AM Ken Russell wrote: > >>> On Tue, May 17, 2022 at 3:40 PM 'Christopher Cameron' via blink-dev < >>> blink-dev@chromium.org> wrote: >>> TAG review status N/A. Canvas 2D color managment API already TAG reviewed at https://gith

Re: [blink-dev] Intent to Ship: WebGL canvas color management

2022-05-18 Thread Ken Russell
On Wed, May 18, 2022 at 3:40 PM Chris Harrelson wrote: > Hi, comments below. > > On Wed, May 18, 2022 at 11:48 AM Ken Russell wrote: > >> On Tue, May 17, 2022 at 3:40 PM 'Christopher Cameron' via blink-dev < blink-dev@chromium.org> wrote: > > TAG review status > N/A. C

Re: [blink-dev] Intent to Ship: WebGL canvas color management

2022-05-18 Thread Chris Harrelson
On Wed, May 18, 2022 at 4:04 PM Ken Russell wrote: > On Wed, May 18, 2022 at 3:40 PM Chris Harrelson > wrote: > >> Hi, comments below. >> >> On Wed, May 18, 2022 at 11:48 AM Ken Russell wrote: >> >>> > On Tue, May 17, 2022 at 3:40 PM 'Christopher Cameron' via blink-dev < > blink-dev@chr

Re: [blink-dev] Intent to Ship: WebGL canvas color management

2022-05-18 Thread Chris Harrelson
On Wed, May 18, 2022 at 4:18 PM Chris Harrelson wrote: > > > On Wed, May 18, 2022 at 4:04 PM Ken Russell wrote: > >> On Wed, May 18, 2022 at 3:40 PM Chris Harrelson >> wrote: >> >>> Hi, comments below. >>> >>> On Wed, May 18, 2022 at 11:48 AM Ken Russell wrote: >>> >> On Tue, May 17,

Re: [blink-dev] Intent to Ship: WebGL canvas color management

2022-05-18 Thread Chris Harrelson
I've updated bit.ly/blink-signals to include an exception for Khronos (see the document for details). Thanks for the developer signal. LGTM1 for this intent! On Wed, May 18, 2022 at 4:19 PM Chris Harrelson wrote: > > > On Wed, May 18, 2022 at 4:18 PM Chris Harrelson > wrote: > >> >> >> On Wed

Re: [blink-dev] Intent to Ship: WebGL canvas color management

2022-05-19 Thread Yoav Weiss
LGTM2 On Thursday, May 19, 2022 at 1:41:32 AM UTC+2 Chris Harrelson wrote: > I've updated bit.ly/blink-signals to include an exception for Khronos > (see the document for details). > > Thanks for the developer signal. > > LGTM1 for this intent! > > On Wed, May 18, 2022 at 4:19 PM Chris Harrelson

Re: [blink-dev] Intent to Ship: WebGL canvas color management

2022-05-19 Thread Mike Taylor
LGTM3 On 5/19/22 10:08 AM, Yoav Weiss wrote: LGTM2 On Thursday, May 19, 2022 at 1:41:32 AM UTC+2 Chris Harrelson wrote: I've updated bit.ly/blink-signals to include an exception for Khronos (see the document for details). Thanks for the developer sign