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"
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+
Assignee: nobody → bjacob
Target Milestone: --- → mozilla26
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.