Closed Bug 911856 Opened 6 years ago Closed 6 years ago

Remove the custom concept of "stable context" in favor of the standard concept of "lost context"

Categories

(Core :: Canvas: WebGL, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla26

People

(Reporter: bjacob, Assigned: bjacob)

Details

Attachments

(1 file)

IsContextLost immediately evocates the standard concept of a "lost context".

IsContextStable... is not so clear. Stable could mean many different things.

Anyway we should not have two concepts that only differ from each other by being each other's contrary.

Plus, most uses of IsContextStable were preceded by a negation operator (!) so this change removes many negations.

Note that this patch renames the ContextStable enum value by ContextNotLost. Ugly, but very rarely used as such, and means exactly what it means.
Attachment #798643 - Flags: review?(jgilbert)
Attachment #798643 - Flags: review?(jgilbert) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/dc6759deb5e1
Assignee: nobody → bjacob
Target Milestone: --- → mozilla26
You need to log in before you can comment on or make changes to this bug.