Move the thread hop for UMA user actions from content:: to base::.

content/browser/user_metrics.cc

Index: content/browser/user_metrics.cc
diff --git a/content/browser/user_metrics.cc b/content/browser/user_metrics.cc
index 200c851ccb170a20d8e51486b7bcd286fc70ec98..175825c06948a1f9c3ea3d3848a3ec34e424a0f6 100644
--- a/content/browser/user_metrics.cc
+++ b/content/browser/user_metrics.cc
@@ -13,26 +13,10 @@
 namespace content {
 
 void RecordAction(const base::UserMetricsAction& action) {
-  if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
-    BrowserThread::PostTask(
-        BrowserThread::UI,
-        FROM_HERE,
-        base::Bind(&RecordAction, action));
-    return;
-  }
-
   base::RecordAction(action);
 }
 
 void RecordComputedAction(const std::string& action) {
-  if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
-    BrowserThread::PostTask(
-        BrowserThread::UI,
-        FROM_HERE,
-        base::Bind(&RecordComputedAction, action));
-    return;
-  }
-
   base::RecordComputedAction(action);
 }

[Ilya Sherman, 2016/04/06 02:36:29]

These methods seem like they could be removed entirely if they're no longer
providing the thread hop.

[beaudoin, 2016/04/06 15:06:13]

Yes. Will do. Since it affects a lot more files I was thinking of doing it in a
separate clean-up CL though, what do you think?

[Ilya Sherman, 2016/04/06 21:48:48]

Fair enough.  Could you please file a bug and add TODOs for the cleanup?

[beaudoin, 2016/04/07 18:12:34]

Done.

Filed as: crbug.com/601483