[webkit-dev] Deprecating JS interface

[Dirk Schulze]

On Feb 16, 2013, at 6:54 PM, Adam Barth <abarth at webkit.org> wrote:

> It's much easier to discuss a concrete example.  Which interface are
> you interested in deprecating?

I can understand that it is easier to discuss on a concrete example, even if I would like to discuss this in a general scope. We have multiple interfaces that we may want to deprecate at some point.

A concrete example I thought about is WebKitCSSMatrix[1]. It is not used in WebCore yet but hopefully replaced by a standardized Matrix interface in the future[2]. This new interface will not be fully compatible to WebKitCSSMatrix and I would like to warn authors before they actually start using it.

Again, I think it would be better to discuss this in a wider scope but am happy to discuss it on the concrete example as well. This actually might make it easier to come up with general rules in the future.

Greetings,
Dirk

[1] https://bugs.webkit.org/show_bug.cgi?id=110048
[2] https://bugs.webkit.org/show_bug.cgi?id=110001
> 
> Adam
> 
> 
> On Sat, Feb 16, 2013 at 5:28 PM, Dirk Schulze <dschulze at adobe.com> wrote:
>> Hi,
>> 
>> There are several steps on deprecating features[1]. My question is about deprecating a whole interface and throwing warnings that the feature is deprecated.
>> 
>> If I have the following interface for deprecation:
>> 
>> [Constructor]
>> interface Bla {
>>        attribute bar;
>>        void foo();
>> }
>> 
>> Should I just throw the deprecation warning on calling the constructor (and any method/attribute creating the object), or on calling foo and bar as well?
>> 
>> Greetings,
>> Dirk
>> 
>> [1] http://trac.webkit.org/wiki/DeprecatingFeatures
>> _______________________________________________
>> webkit-dev mailing list
>> webkit-dev at lists.webkit.org
>> https://lists.webkit.org/mailman/listinfo/webkit-dev
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> https://lists.webkit.org/mailman/listinfo/webkit-dev