On Wed, Mar 13, 2013 at 9:42 AM, Simon Fraser <simon.fra...@apple.com>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, I don't follow. setBackgroundBlurOnNode appears to be only called in
a layout test as far as I read the patch.  Where is this function called
for non-testing purposes?

- R. Niwa
_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev

Reply via email to