rlz: Remove #include <windows.h>

chrome/browser/rlz/rlz.cc

Index: chrome/browser/rlz/rlz.cc
diff --git a/chrome/browser/rlz/rlz.cc b/chrome/browser/rlz/rlz.cc
index 91f447c16279b6f11b3bc10ece858b50671c9601..c31ecfb18ebcbdd279dace2fcc7b0be5c082c04a 100644
--- a/chrome/browser/rlz/rlz.cc
+++ b/chrome/browser/rlz/rlz.cc
@@ -8,9 +8,6 @@
 
 #include "chrome/browser/rlz/rlz.h"
 
-#include <windows.h>
-#include <process.h>
-
 #include <algorithm>
 
 #include "base/bind.h"
@@ -231,15 +228,12 @@ void RLZTracker::DelayedInit() {
 }
 
 void RLZTracker::ScheduleFinancialPing() {
-  // Investigate why _beginthread() is used here, and not chrome's threading
-  // API.  Tracked in bug http://crbug.com/106213
-  _beginthread(PingNow, 0, this);
-}
-
-// static
-void _cdecl RLZTracker::PingNow(void* arg) {
-  RLZTracker* tracker = reinterpret_cast<RLZTracker*>(arg);
-  tracker->PingNowImpl();
+  // TODO(thakis): Once akalin's TaskRunner around PostDelayedTask() is done,
+  // use that instead of the file thread.
+  BrowserThread::PostTask(
+      BrowserThread::FILE,
+      FROM_HERE,
+      base::Bind(&RLZTracker::PingNowImpl, base::Unretained(this)));

[akalin, 2012/02/16 01:03:36]

does PingNowImpl actually depend on instance variables?  If so, you probably
want a weak ptr here.  If not, maybe PingNowImpl should be static

[Nico, 2012/02/16 01:11:47]

Yes, it seems to look at cache_lock_. The instance lives in a singleton, so it 
shouldn't go away (and the class isn't SupportsWeakPtr<>).

 }
 
 void RLZTracker::PingNowImpl() {