X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=src%2Fchrome%2Fbrowser%2Fmetrics%2Fvariations%2Fvariations_registry_syncer_win.cc;h=dcfd5e218080f6d22c5279c41db084cc10e27f09;hb=1afa4dd80ef85af7c90efaea6959db1d92330844;hp=a6516fc459138cb83bfb49ad0e81eec62e37cc0c;hpb=90762837333c13ccf56f2ad88e4481fc71e8d281;p=platform%2Fframework%2Fweb%2Fcrosswalk.git

diff --git a/src/chrome/browser/metrics/variations/variations_registry_syncer_win.cc b/src/chrome/browser/metrics/variations/variations_registry_syncer_win.cc
index a6516fc..dcfd5e2 100644
--- a/src/chrome/browser/metrics/variations/variations_registry_syncer_win.cc
+++ b/src/chrome/browser/metrics/variations/variations_registry_syncer_win.cc
@@ -8,37 +8,24 @@
 #include "base/metrics/field_trial.h"
 #include "base/path_service.h"
 #include "base/strings/string16.h"
+#include "base/threading/thread_restrictions.h"
 #include "chrome/common/variations/experiment_labels.h"
 #include "chrome/installer/util/google_update_settings.h"
 #include "chrome/installer/util/install_util.h"
+#include "content/public/browser/browser_thread.h"
+
+namespace chrome_variations {
 
 namespace {
 
 // Delay before performing a registry sync, in seconds.
 const int kRegistrySyncDelaySeconds = 5;
 
-}  // namespace
-
-namespace chrome_variations {
-
-VariationsRegistrySyncer::VariationsRegistrySyncer() {
-}
-
-VariationsRegistrySyncer::~VariationsRegistrySyncer() {
-}
-
-void VariationsRegistrySyncer::RequestRegistrySync() {
-  if (timer_.IsRunning()) {
-    timer_.Reset();
-    return;
-  }
-
-  timer_.Start(FROM_HERE,
-               base::TimeDelta::FromSeconds(kRegistrySyncDelaySeconds),
-               this, &VariationsRegistrySyncer::SyncWithRegistry);
-}
+// Performs the actual synchronization process with the registry, which should
+// be done on a blocking pool thread.
+void SyncWithRegistryOnBlockingPool() {
+  base::ThreadRestrictions::AssertIOAllowed();
 
-void VariationsRegistrySyncer::SyncWithRegistry() {
   // Note that all registry operations are done here on the UI thread as there
   // are no threading restrictions on them.
   base::FilePath chrome_exe;
@@ -81,4 +68,31 @@ void VariationsRegistrySyncer::SyncWithRegistry() {
   }
 }
 
+}  // namespace
+
+VariationsRegistrySyncer::VariationsRegistrySyncer() {
+}
+
+VariationsRegistrySyncer::~VariationsRegistrySyncer() {
+}
+
+void VariationsRegistrySyncer::RequestRegistrySync() {
+  if (timer_.IsRunning()) {
+    timer_.Reset();
+    return;
+  }
+
+  timer_.Start(FROM_HERE,
+               base::TimeDelta::FromSeconds(kRegistrySyncDelaySeconds),
+               this,
+               &VariationsRegistrySyncer::StartRegistrySync);
+}
+
+void VariationsRegistrySyncer::StartRegistrySync() {
+  // Do the work on a blocking pool thread, as chrome://profiler has shown
+  // that it can cause jank if done on the UI thrread.
+  content::BrowserThread::GetBlockingPool()->PostTask(
+      FROM_HERE, base::Bind(&SyncWithRegistryOnBlockingPool));
+}
+
 }  // namespace chrome_variations