Closed Bug 1535809 Opened 5 years ago Closed 5 years ago

Warn when implicitly-enabled extensions are used

Categories

(Core :: Graphics: CanvasWebGL, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: jgilbert, Assigned: greyson.gilbert.oss)

References

Details

(Whiteboard: gfx-noted)

Attachments

(1 file, 1 obsolete file)

This will help prevent devs from shipping non-portable code.

Status: NEW → ASSIGNED
Assignee: jgilbert → nobody
Status: ASSIGNED → NEW

Emits a warning when an extension is enabled implicitly.

Thanks for the submission!

I worry that warn-on-implicit-enable will cause false-positives on most existing content.
I think the most helpful we can be here is to watch for usage of implicitly enabled functionality.

For example, FormatUsageInfo::renderable could become an enum or class that tracks if it was enabled implicitly or explicitly. Then, when we perform validation against it, we can generate warnings if it wasn't explicitly enabled.

We also need to be sure that if a user later explicitly enables an extension, we update our implicit/explicit tracking, so that we don't force them to activate extensions in a particular order.

Assignee: nobody → greyson.gilbert.oss
Attachment #9072382 - Attachment is obsolete: true
Pushed by jgilbert@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5d8406bfe056
Add warning on use of implicitly enabled extensions r=jgilbert
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
Attachment #9072382 - Attachment is obsolete: false
Attachment #9072382 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: