The default bug view has changed. See this FAQ.

Rename WEBKIT_lose_context to WEBGL_EXT_lose_context

RESOLVED FIXED in mozilla10

Status

()

Core
Canvas: WebGL
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: bjacob, Assigned: bjacob)

Tracking

unspecified
mozilla10
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

Created attachment 571732 [details] [diff] [review]
rename to MOZ_lose_context

Jeff M mentioned that it's actually a bad idea for us to expose WEBKIT_ prefixed extensions, and it's better for everybody to rename them to MOZ_. The idea is that this way, web devs can rely on WEBKIT_ stuff being actually, bug for bug, WebKit, rather than having to use e.g. user agent strings.
Attachment #571732 - Flags: review?(dsherk)
But as per the khronos list discussion, since we're using the same standard as WebKit, shouldn't we be naming this EXT_?
Comment on attachment 571732 [details] [diff] [review]
rename to MOZ_lose_context

Review of attachment 571732 [details] [diff] [review]:
-----------------------------------------------------------------

Did you try building this? I think WebGLContext.h:529 was missed.
Attachment #571732 - Flags: review?(dsherk) → review-
Created attachment 571739 [details] [diff] [review]
rename to MOZ_lose_context
Attachment #571732 - Attachment is obsolete: true
Attachment #571739 - Flags: review+
(Assignee)

Comment 4

6 years ago
I just had another conversation with jrmuizel and there was a misunderstanding. He thought that was a prefix on a function name. For GL extensions, the norm is for vendors to use each others' extensions without renaming them, so we can stick with that.

-> not landing this patch for now.

However we still want to rename this to WEBGL_ or WEBGL_EXT_ soon.
(Assignee)

Comment 5

6 years ago
(In reply to Jeff Gilbert [:jgilbert] from comment #1)
> But as per the khronos list discussion, since we're using the same standard
> as WebKit, shouldn't we be naming this EXT_?

I would like to bring this up in the next conference call. Unfortunately today's got cancelled, next one is in 2 weeks.
(Assignee)

Updated

6 years ago
Summary: Rename WEBKIT_lose_context to MOZ_lose_context → Rename WEBKIT_lose_context to WEBGL_EXT_lose_context
(Assignee)

Comment 6

6 years ago
Created attachment 571847 [details] [diff] [review]
rename to WEBGL_EXT_lose_context

Per public_webgl list discussion, we're switching to WEBGL_EXT_ now.
Attachment #571739 - Attachment is obsolete: true
Attachment #571847 - Flags: review?(dsherk)
Attachment #571847 - Flags: review?(dsherk) → review+
(Assignee)

Comment 7

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/2964292c2dc7
Assignee: nobody → bjacob
https://hg.mozilla.org/mozilla-central/rev/2964292c2dc7
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in before you can comment on or make changes to this bug.