Bug 151033

Summary: [Streams API] Shield promises when prototype is replaced from a promise
Product: WebKit Reporter: Xabier Rodríguez Calvar <calvaris>
Component: New BugsAssignee: Xabier Rodríguez Calvar <calvaris>
Severity: Normal CC: benjamin, commit-queue, darin, youennf
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch none

Description Xabier Rodríguez Calvar 2015-11-09 09:24:26 PST
[Streams API] Shield promises when prototype is replaced from a promise
Comment 1 Xabier Rodríguez Calvar 2015-11-09 09:27:30 PST
Created attachment 265062 [details]
Comment 2 Xabier Rodríguez Calvar 2015-11-09 09:51:29 PST
I'll land this tomorrow as I don't have too much time left.
Comment 3 Xabier Rodríguez Calvar 2015-11-09 11:18:34 PST
Created attachment 265076 [details]

Rebased, will cq+ tomorrow
Comment 4 WebKit Commit Bot 2015-11-10 00:51:02 PST
Comment on attachment 265076 [details]

Clearing flags on attachment: 265076

Committed r192207: <http://trac.webkit.org/changeset/192207>
Comment 5 WebKit Commit Bot 2015-11-10 00:51:07 PST
All reviewed patches have been landed.  Closing bug.
Comment 6 youenn fablet 2015-11-10 05:25:36 PST
Nice work :)

The only minor issue I have is that the code is new somehow less readable.
It would have been good to be able to keep blabla.....@then.

One approach for doing so might be to do specific handling like for @call/@apply.
But I prefer your simpler approach. This is not worth it, at least in the current situation where streams API is the only API consuming user-land promises.