Split UserMetrics into API vs. implementation. Move API to content/public.

content/browser/user_metrics_impl.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.

[jam, 2011/12/12 23:06:55]

actually, please leave this as user_metrics.cc. I had spoken to folks working on
the WebKit API. they only add a _impl to the cc if there's a corresponding
_impl.h. If it's just a X.header in the public directory, and one c++ file, then
that file is just named X.cc

 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "content/browser/user_metrics.h"
+#include "content/public/browser/user_metrics.h"
 
 #include "base/bind.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_types.h"
+#include "content/public/browser/user_metrics.h"
+
+namespace {
 
 using content::BrowserThread;
 
-void UserMetrics::RecordAction(const UserMetricsAction& action) {
-  Record(action.str_);
-}
+// Forward declare because of circular dependency.
+void CallRecordOnUI(const std::string& action);
 
-void UserMetrics::RecordComputedAction(const std::string& action) {
-  Record(action.c_str());
-}
-
-void UserMetrics::Record(const char *action) {
+void Record(const char *action) {
   if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
-    BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
-                            base::Bind(&UserMetrics::CallRecordOnUI, action));
+    BrowserThread::PostTask(
+        BrowserThread::UI,
+        FROM_HERE,
+        base::Bind(&CallRecordOnUI, action));
     return;
   }
 
   content::NotificationService::current()->Notify(
       content::NOTIFICATION_USER_ACTION,
       content::NotificationService::AllSources(),
       content::Details<const char*>(&action));
 }
 
-void UserMetrics::CallRecordOnUI(const std::string& action) {
+void CallRecordOnUI(const std::string& action) {
   Record(action.c_str());
 }
+
+}  // namespace
+
+void UserMetrics::RecordAction(const UserMetricsAction& action) {
+  Record(action.str_);
+}
+
+void UserMetrics::RecordComputedAction(const std::string& action) {
+  Record(action.c_str());
+}