rlz: Remove #include <windows.h>

chrome/browser/rlz/rlz.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // This code glues the RLZ library DLL with Chrome. It allows Chrome to work
 // with or without the DLL being present. If the DLL is not present the
 // functions do nothing and just return false.
 
 #include "chrome/browser/rlz/rlz.h"
 
-#include <windows.h>
-#include <process.h>
-
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/file_path.h"
 #include "base/message_loop.h"
 #include "base/path_service.h"
 #include "base/string_util.h"
 #include "base/synchronization/lock.h"
 #include "base/threading/thread.h"
@@ skipping 200 matching lines @@
     schedule_ping = true;
   }
 
   already_ran_ = true;
 
   if (schedule_ping)
     ScheduleFinancialPing();
 }
 
 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 PostTask() is done,
+  // use that instead of the file thread.
+  BrowserThread::PostTask(
+      BrowserThread::FILE,
+      FROM_HERE,
+      base::Bind(&RLZTracker::PingNowImpl, base::Unretained(this)));
 }
 
 void RLZTracker::PingNowImpl() {
-  // This is the entry point of a background thread, so I/O is allowed.
-  base::ThreadRestrictions::ScopedAllowIO allow_io;
-
   string16 lang;
   GoogleUpdateSettings::GetLanguage(&lang);
   if (lang.empty())
     lang = L"en";
   string16 referral;
   GoogleUpdateSettings::GetReferral(&referral);
 
   std::string brand;
   if (google_util::GetBrand(&brand) && !IsBrandOrganic(brand) &&
       SendFinancialPing(brand, lang, referral)) {
@@ skipping 143 matching lines @@
       base::Bind(base::IgnoreResult(&RLZTracker::GetAccessPointRlz), point,
                  not_used));
   return true;
 }
 
 // static
 void RLZTracker::CleanupRlz() {
   GetInstance()->rlz_cache_.clear();
   GetInstance()->registrar_.RemoveAll();
 }