Convert RapporRecorder to use mojo.

chrome/browser/rappor_recorder_impl.cc

+// Copyright 2016 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 "chrome/browser/rappor_recorder_impl.h"
+
+#include "chrome/browser/browser_process.h"
+#include "components/rappor/rappor_utils.h"
+#include "content/public/browser/browser_thread.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
+
+RapporRecorderImpl::RapporRecorderImpl() = default;
+
+RapporRecorderImpl::~RapporRecorderImpl() = default;
+
+// static
+void RapporRecorderImpl::Create(mojom::RapporRecorderRequest request) {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+  mojo::MakeStrongBinding(base::MakeUnique<RapporRecorderImpl>(),
+                          std::move(request));
+}
+
+void RapporRecorderImpl::RecordRappor(const std::string& metric,
+                                      const std::string& sample) {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+  if (g_browser_process != NULL) {

[sky, 2016/10/06 17:28:44]

nullptr, or just if (g_browser_process).
That said, can g_browser_process really be null here? I would think this should
be a DCHECK. Same coment below.

[nigeltao1, 2016/10/07 03:30:51]

As discussed, I've dropped this check.

+    rappor::SampleString(g_browser_process->rappor_service(), metric,
+                         rappor::ETLD_PLUS_ONE_RAPPOR_TYPE, sample);
+  }
+}
+
+void RapporRecorderImpl::RecordRapporURL(const std::string& metric,
+                                         const GURL& sample) {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+  if (g_browser_process != NULL) {
+    rappor::SampleDomainAndRegistryFromGURL(g_browser_process->rappor_service(),
+                                            metric, sample);
+  }
+}