Decrease the visibility of the ENABLE_THREADING_XX guards.

Decrease the visibility of the ENABLE_THREADING_XX guards.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=150286

[lgombos, 2013-04-21 12:34:55]

Cleanup.

I find encapsulation a general good practice. This is also a step forward to
eliminate Platform.h over time.

[abarth-chromium, 2013-04-21 13:54:25]

Can't we just figure out which one of these things we actually use?  Do we use
ParellelJobs.h at all, for example?

[abarth-chromium, 2013-04-21 13:55:19]

I see, we use it graphics/filters.  Which of these flags are actually set?

[lgombos, 2013-04-22 17:05:42]

On 2013/04/21 13:55:19, abarth wrote:
> I see, we use it graphics/filters.  Which of these flags are actually set?

It depends. 

On OS(DARWIN) LIBDISPATCH is used, otherwise (if no OpenMP support present)
GENERIC is the fallback. 

I think the OpenMP support can be removed, but I think the other 2 flags should
stay.

I will post a new patch.

[abarth-chromium, 2013-04-22 17:50:36]

We should like change this code to use Chromium's thread pool, but I think we'll
first need to expose that via the platform API.

[lgombos, 2013-04-22 17:59:30]

Remove ParallelJobsOpenMP.

[lgombos, 2013-04-28 18:15:38]

Any further thoughts on this ?

My immediate motivation on this is to remove things from wtf/Platform.h and
eventually eliminate wtf/Platform.h.

[eseidel, 2013-04-28 19:29:23]

lgtm

This looks like a fine stepping stone.

[commit-bot: I haz the power, 2013-04-28 22:33:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/l.gombos@samsung.com/14390002/7001

[commit-bot: I haz the power, 2013-04-28 22:54:13]

Retried try job too often on win_layout_rel for step(s) webkit_lint,
webkit_tests, webkit_unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_layout...

[commit-bot: I haz the power, 2013-04-29 03:14:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/l.gombos@samsung.com/14390002/7001

[commit-bot: I haz the power, 2013-04-29 03:33:35]

Retried try job too often on win_layout_rel for step(s) webkit_lint,
webkit_tests, webkit_unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_layout...

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

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/l.gombos@samsung.com/14390002/7001

[commit-bot: I haz the power, 2013-04-29 05:33:59]

Retried try job too often on win_layout_rel for step(s) webkit_lint,
webkit_tests, webkit_unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_layout...

[commit-bot: I haz the power, 2013-05-10 19:39:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/l.gombos@samsung.com/14390002/7001

[commit-bot: I haz the power, 2013-05-10 21:21:03]

Retried try job too often on win_layout_rel for step(s) webkit_lint,
webkit_tests, webkit_unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_layout...

[commit-bot: I haz the power, 2013-05-11 23:38:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/l.gombos@samsung.com/14390002/7001

[commit-bot: I haz the power, 2013-05-11 23:54:27]

Retried try job too often on win_layout for step(s) webkit_lint
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_layout...

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

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/l.gombos@samsung.com/14390002/40001

[commit-bot: I haz the power, 2013-05-14 03:19:58]

Change committed as 150286