Use sequenced workers for the getting the system's font list.

Use sequenced workers for the getting the system's font list.

This is necessary because on Linux we use Pango, which is not threadsafe (only newer versions are).


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

[falken, 2013-04-02 08:18:36]

brettw: Can you please review?
behdad: CC'ing in case you have comments

[brettw, 2013-04-02 18:07:57]

This is being called on the blocking pool which uses different threads. Your
patch will block blocking pool threads on each other which is very bad. The
right solution would be to just make sure that the same blocking thread pool
thread is used for every call to this function (see the blocking pool for how to
do this).

[falken, 2013-04-03 09:24:35]

brettw: Thanks. Please take another look.
raymes: Please review as pepper OWNER

https://codereview.chromium.org/13458002/diff/13001/content/common/font_list.cc
File content/common/font_list.cc (left):

https://codereview.chromium.org/13458002/diff/13001/content/common/font_list....
content/common/font_list.cc:7: });
content/common/font_list.cc is a new file. Rietveld just thinks it's a modified
copy of
chrome/test/data/extensions/platform_apps/web_view/media_access/deny/test.js

[raymes, 2013-04-03 14:26:00]

I think brettw's lg is enough here but it looks ok to me.

[brettw, 2013-04-04 04:50:29]

lgtm

[commit-bot: I haz the power, 2013-04-04 05:08:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/falken@chromium.org/13458002/13001

[commit-bot: I haz the power, 2013-04-04 05:09:57]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...

[commit-bot: I haz the power, 2013-04-04 11:05:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/falken@chromium.org/13458002/13001

[commit-bot: I haz the power, 2013-04-04 12:01:36]

Change committed as 192307