Peter Beverloo
2015/08/28 17:18:24
The spec defines this as having a constructor and
The spec defines this as having a constructor and having an interface object
(which means whether it's exposed on the window global).
Perhaps split out the BlobEvent(Init) in a different CL, since that can be
landed stand-alone of the rest?
mcasas
2015/08/29 01:12:57
Ok, following the suggestions I'll leave BlobEvent
On 2015/08/28 17:18:24, Peter Beverloo wrote:
> The spec defines this as having a constructor and having an interface object
> (which means whether it's exposed on the window global).
>
> Perhaps split out the BlobEvent(Init) in a different CL, since that can be
> landed stand-alone of the rest?
Ok, following the suggestions I'll leave BlobEvent out of this CL.
In my defense I'd say that I wanted to land chunks at a time and
was instructed to try and land full functionalities ;)
Peter Beverloo
2015/09/01 17:17:12
I think experience has taught us that small CLs ge
On 2015/08/29 01:12:57, mcasas wrote:
> On 2015/08/28 17:18:24, Peter Beverloo wrote:
> > The spec defines this as having a constructor and having an interface object
> > (which means whether it's exposed on the window global).
> >
> > Perhaps split out the BlobEvent(Init) in a different CL, since that can be
> > landed stand-alone of the rest?
>
> Ok, following the suggestions I'll leave BlobEvent out of this CL.
>
> In my defense I'd say that I wanted to land chunks at a time and
> was instructed to try and land full functionalities ;)
I think experience has taught us that small CLs get reviewed *much* more quickly
as it's easier for reviewers to put them between other tasks :-). I certainly
prefer them.
Issue 1255873002: MediaRecorder Blink part
(Closed)
Created 5 years, 5 months ago by mcasas
Modified 5 years, 3 months ago
Reviewers: esprehn, Guido Urdaneta, mlamouri (slow - plz ping), Peter Beverloo
Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Comments: 208