Bug 86264 – layerX/layerY warning should be removed

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Ojan Vafai]

This wasn't ever discussed on webkit-dev and there doesn't seem to be much agreement that this is worth the compatibility hit. Do we have histograms or anything about the usage of these in the wild?

[Erik Arvidsson]

If I remember correctly, the WebKit implementation of layerX/layerY is broken as is.

[Ojan Vafai]

Relevant webkit-dev discussion: https://lists.webkit.org/pipermail/webkit-dev/2012-May/020620.html. I don't actually care what we do here. I was more annoyed with the other console spew in that discussion.

[Julien Chaffraix]

For the record, warnDeprecatedLayerXYUsage (the layerX / layerY warning) was at some point one of Chromium's top crashers due to Y! mail hitting a NULL console().

Following the talk on IRC last week, it looks like it may have been premature to deprecate it as people seem to be using it and we have no replacement. Also we haven't done a proper investigation before making the move.

I will be moving forward with removing the deprecation warning unless someone objects. We can still add it back once we are ready to follow the proper deprecation protocol.

[James Robinson]

I think we should remove the warning regardless of whether we decide to deprecate, they don't seem to be helping (and we aren't collecting any data to tell us if they would be).

[Simon Fraser (smfr)]

We should do a warnOnce() thing, where it only spews once per session.

[Julien Chaffraix]

(In reply to comment #5)
> We should do a warnOnce() thing, where it only spews once per session.

I have one issue with that. Per our newly minted deprecation policy, we haven't done the proper investigation and we should show the example here. On top of that, maintaining layerX and layerY is virtually zero effort (nobody touched this code in years) and the compatibility breakage relatively high.

All those points make me think that we should just remove the warning for now and come back once we have a better idea of our current state.

[James Robinson]

warnOnce() would be lying to developers once (instead of over and over).  The log message says that layerX/layerY will be removed soon, which isn't true.

[Julien Chaffraix]

Created attachment 142300 [details]
Proposed warning removal.

[James Robinson]

Comment on attachment 142300 [details]
Proposed warning removal.

R=me

[WebKit Review Bot]

Comment on attachment 142300 [details]
Proposed warning removal.

Clearing flags on attachment: 142300

Committed r117335: <http://trac.webkit.org/changeset/117335>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.