On Mar 13, 2013, at 10:15 AM, Dana Jansens <[email protected]> wrote:

> On Wed, Mar 13, 2013 at 12:42 PM, Simon Fraser <[email protected]> wrote:
> https://bugs.webkit.org/show_bug.cgi?id=80046 (landed in 
> http://trac.webkit.org/changeset/114081) added internals. 
> setBackgroundBlurOnNode().
> 
> I consider this to be abuse of the Internals object. As shown by the location 
> of the files (Source/WebCore/testing/Internals.*), the Internals interface is 
> intended for testing, not adding API to toggle features in WebCore for 
> non-test code.
> 
> Sorry, but I think this is based on a misunderstanding. That function is used 
> for enabling the feature in layout tests in order to verify it is working. It 
> is not used in non-test code.

So why does the feature exist at all?

Simon


_______________________________________________
webkit-dev mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-dev

Reply via email to