Add thread-per-core setting to SkThreadPool.

src/utils/SkThreadPool.cpp

 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
+#include "SkRunnable.h"
 #include "SkThreadPool.h"
-#include "SkRunnable.h"
 #include "SkThreadUtils.h"
+#include "SkTypes.h"
 
-SkThreadPool::SkThreadPool(const int count)
+#if defined(SK_BUILD_FOR_UNIX) || defined(SK_BUILD_FOR_MAC)
+#include <unistd.h>
+#endif
+
+namespace {
+// Returns the number of cores on this machine.
+int numCores() {

[scroggo, 2013/04/19 18:43:35]

Should this be num_cores(), as if this were a static function?

[mtklein, 2013/04/19 18:52:59]

Yes, done.

+#if defined(SK_BUILD_FOR_WIN32)
+    SYSTEM_INFO sysinfo;
+    GetSystemInfo(&sysinfo);
+    return sysinfo.dwNumberOfProcessors;
+#elif defined(SK_BUILD_FOR_UNIX) || defined(SK_BUILD_FOR_MAC)
+    return sysconf(_SC_NPROCESSORS_ONLN);
+#else
+    return 1;

[caryclark, 2013/04/19 18:37:12]

SK_BUILD_ANDROID ?
could this use the sysconf call?
please add a TODO if not

[mtklein, 2013/04/19 18:52:59]

Yes, that'll work there too.  Done.

+#endif
+}
+}  // namespace
+
+SkThreadPool::SkThreadPool(int count)
 : fDone(false) {
+    if (count < 0) count = numCores();
     // Create count threads, all running SkThreadPool::Loop.
     for (int i = 0; i < count; i++) {
         SkThread* thread = SkNEW_ARGS(SkThread, (&SkThreadPool::Loop, this));
         *fThreads.append() = thread;
         thread->start();
     }
 }
 
 SkThreadPool::~SkThreadPool() {
     fDone = true;
@@ skipping 55 matching lines @@
     }
 
     // We have some threads.  Queue it up!
     fReady.lock();
     LinkedRunnable* linkedRunnable = SkNEW(LinkedRunnable);
     linkedRunnable->fRunnable = r;
     fQueue.addToHead(linkedRunnable);
     fReady.signal();
     fReady.unlock();
 }