Closed Bug 1278575 Opened 4 years ago Closed 4 years ago

Remove "live" gfxEnv cases

Categories

(Core :: Graphics, defect)

Unspecified
All
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: milan, Assigned: rhunt, Mentored)

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file)

It's a hit and miss, mostly miss, being able to change the environment variable value and have it be picked up by running Gecko, so for sanity and readability, remove all DECL_GFX_ENV_LIVE and DECL_GFX_ENV_ONCE with DECL_GFX_ENV, which is what DECL_GFX_ENV_ONCE used to be.  Make any simplifications to gfxEnv that come out of that.
Jeff, can you pass this to Ryan once he gets a bugzilla account?
Flags: needinfo?(jgilbert)
OS: Unspecified → All
Whiteboard: [gfx-noted]
Yes. Taking until then.
Assignee: nobody → jgilbert
Flags: needinfo?(jgilbert)
Assignee: jgilbert → rhunt
Attached patch gfxenv.patchSplinter Review
Attachment #8761791 - Flags: review?(jgilbert)
Attachment #8761791 - Flags: review?(jgilbert) → review+
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/be7b149e0bf6
Removed refs to GFX_ENV_LIVE. r=jgilbert
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/be7b149e0bf6
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.