[webkit-dev] Streams API

youenn fablet youennf at gmail.com
Mon Nov 3 01:08:17 PST 2014


Hi Benjamin,

> I have two concerns:
> 1) You have many patches waiting for review. I think you need to get better
> communication channels with reviewers in order to iterate over your patches
> faster. It is unfortunate but getting complex patches reviewed in WebKit
> requires poking reviewers directly. Since you work on the network stack, I
> suggest asking Alexey (ap) and Pratik (psolanki) on IRC, they can help with
> reviews or suggest other reviewers.

Thanks for the suggestion, I will try to do so for my pending patches.
For the particular streams API work, I am teaming up with some Igalia fellows.
I hope that we can make good progress that way.

> 2) Streams is a large spec and it touches platform code. Please be careful
> with stability, make more tests than reasonable and #ifdef everything.
>
> I suggest the flag name "ENABLE_STREAMS_API", I find "ENABLE_STREAMS" a bit
> too generic.

Ah, I was thinking of using ENABLE_STREAMS.
ENABLE_STREAMS_API is fine by me.


More information about the webkit-dev mailing list