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

content/browser/user_metrics.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.
 
 #include "content/public/browser/user_metrics.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/metrics/user_metrics.h"
 #include "content/public/browser/browser_thread.h"
 
 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

 
 }  // namespace content