On Wed, Dec 9, 2015 at 3:49 PM, Nicholas Nethercote <n.netherc...@gmail.com> wrote: >> >> One interesting thing I found is that a *lot* of the functions that >> take an nsRenderingContext or gfxContext do so because they end up >> passing it into text run code -- gfxTextRun uses a gfxContext, via >> gfxTextRunFactory::Parameters::mContext. However, only a few things >> from the gfxContext are used by the text run code: > > I was able to find a bunch of functions that only needed > gfxContext::mRefCairo and so I've changed them to take a |cairo_t*| > instead of a |gfxContext*|, which clears things up a bit: > https://bugzilla.mozilla.org/show_bug.cgi?id=1231550
I subsequently did something similar with all the other text/font code, in the same bug. I'm on the fence about the usefulness of this change. Nick _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
