linux: Avoid leak in PlatformFontPangoTest.FamilyList.

linux: Avoid leak in PlatformFontPangoTest.FamilyList.

Call FcFini() at the end of the test to prevent FontConfig
from leaking memory.

BUG=114750
TBR=sky@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=213996

[Daniel Erat, 2013-07-24 22:23:45]

free fontconfig resources from pango and cairo

[behdad_google, 2013-07-25 19:15:14]

lgtm

[Daniel Erat, 2013-07-25 19:17:08]

TBR sky for approval

[commit-bot: I haz the power, 2013-07-25 19:29:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/derat@chromium.org/19695014/12001

[commit-bot: I haz the power, 2013-07-26 00:54:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/derat@chromium.org/19695014/12001

[commit-bot: I haz the power, 2013-07-26 03:40:23]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_chromeos_clang.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[Daniel Erat, 2013-07-26 15:59:20]

Hmm. The pango_fc_* functions come from pangoft2. build/linux/system.gyp defines
pangocairo, freetype2, and fontconfig targets. I currently added pangoft2 to the
pangocairo target, but I'm not sure if that makes sense. Do you think that it
should go somewhere else? Should I add a separate pango_freetype2 target?

[behdad_google, 2013-07-26 16:02:25]

On 2013/07/26 15:59:20, Daniel Erat wrote:
> Hmm. The pango_fc_* functions come from pangoft2. build/linux/system.gyp
defines
> pangocairo, freetype2, and fontconfig targets. I currently added pangoft2 to
the
> pangocairo target, but I'm not sure if that makes sense. Do you think that it
> should go somewhere else? Should I add a separate pango_freetype2 target?

On all platforms that we care about pangocairo uses pangoft2, so adding it to
the same target makes sense to me.

[commit-bot: I haz the power, 2013-07-26 16:33:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/derat@chromium.org/19695014/48001

[commit-bot: I haz the power, 2013-07-26 22:54:09]

Change committed as 213996