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);
+  rappor::SampleString(g_browser_process->rappor_service(), metric,

[Sam McNally, 2016/10/05 09:17:25]

I think it's possible g_browser_process could be null.

[nigeltao1, 2016/10/06 04:49:37]

Done.

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