Re: [8u] Review request: RT-38290: [WebView] HTML canvas clip() functionality broken

2014-08-27 Thread Anton V. Tarasov
Hi Steve, Sure, thanks for the reminder. Anton. On 27.08.2014 1:11, Stephen F Northover wrote: Hi Anton T, The policy for OpenJFX is that code reviews happen in the associated JIRA: https://wiki.openjdk.java.net/display/OpenJFX/Code+Reviews Thanks for reviewing this code, Steve On

Re: [8u] Review request: RT-38290: [WebView] HTML canvas clip() functionality broken

2014-08-26 Thread Anton V. Tarasov
Hi Anton, Does it make sense to add a boolean param to renderLayer() which would tell it if it should dispose the layer after rendering? In order to avoid the splitting. Also, could you please clarify the following. From your comments: 72 // WCBufferedContext might be referenced

Re: [8u] Review request: RT-38290: [WebView] HTML canvas clip() functionality broken

2014-08-26 Thread Stephen F Northover
Hi Anton T, The policy for OpenJFX is that code reviews happen in the associated JIRA: https://wiki.openjdk.java.net/display/OpenJFX/Code+Reviews Thanks for reviewing this code, Steve On 2014-08-26, 3:30 PM, Anton V. Tarasov wrote: Hi Anton, Does it make sense to add a boolean param to