Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(511)

Unified Diff: chrome/browser/renderer_host/chrome_render_message_filter.cc

Issue 2376403003: Convert RapporRecorder to use mojo. (Closed)
Patch Set: Convert RapporRecorder to use mojo. Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/renderer_host/chrome_render_message_filter.h ('k') | chrome/common/render_messages.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/renderer_host/chrome_render_message_filter.cc
diff --git a/chrome/browser/renderer_host/chrome_render_message_filter.cc b/chrome/browser/renderer_host/chrome_render_message_filter.cc
index f6f74e721ae1a93a9d66c2c06ba3bda25c0cbd51..44b6ad01babc8cbad3bd2467f99b6236fb493160 100644
--- a/chrome/browser/renderer_host/chrome_render_message_filter.cc
+++ b/chrome/browser/renderer_host/chrome_render_message_filter.cc
@@ -25,8 +25,6 @@
#include "components/content_settings/core/browser/cookie_settings.h"
#include "components/network_hints/common/network_hints_common.h"
#include "components/network_hints/common/network_hints_messages.h"
-#include "components/rappor/rappor_service.h"
-#include "components/rappor/rappor_utils.h"
#include "components/web_cache/browser/web_cache_manager.h"
#include "content/public/browser/notification_service.h"
#include "content/public/browser/render_process_host.h"
@@ -89,8 +87,6 @@ bool ChromeRenderMessageFilter::OnMessageReceived(const IPC::Message& message) {
#endif
IPC_MESSAGE_HANDLER(ChromeViewHostMsg_FieldTrialActivated,
OnFieldTrialActivated)
- IPC_MESSAGE_HANDLER(ChromeViewHostMsg_RecordRappor, OnRecordRappor)
- IPC_MESSAGE_HANDLER(ChromeViewHostMsg_RecordRapporURL, OnRecordRapporURL)
IPC_MESSAGE_UNHANDLED(handled = false)
IPC_END_MESSAGE_MAP()
@@ -105,8 +101,6 @@ void ChromeRenderMessageFilter::OverrideThreadForMessage(
case ChromeViewHostMsg_IsCrashReportingEnabled::ID:
#endif
case ChromeViewHostMsg_UpdatedCacheStats::ID:
- case ChromeViewHostMsg_RecordRappor::ID:
- case ChromeViewHostMsg_RecordRapporURL::ID:
*thread = BrowserThread::UI;
break;
default:
@@ -363,17 +357,3 @@ void ChromeRenderMessageFilter::OnFieldTrialActivated(
// and activates the trial.
base::FieldTrialList::FindFullName(trial_name);
}
-
-void ChromeRenderMessageFilter::OnRecordRappor(const std::string& metric,
- const std::string& sample) {
- DCHECK_CURRENTLY_ON(BrowserThread::UI);
- rappor::SampleString(g_browser_process->rappor_service(), metric,
- rappor::ETLD_PLUS_ONE_RAPPOR_TYPE, sample);
-}
-
-void ChromeRenderMessageFilter::OnRecordRapporURL(const std::string& metric,
- const GURL& sample) {
- DCHECK_CURRENTLY_ON(BrowserThread::UI);
- rappor::SampleDomainAndRegistryFromGURL(g_browser_process->rappor_service(),
- metric, sample);
-}
« no previous file with comments | « chrome/browser/renderer_host/chrome_render_message_filter.h ('k') | chrome/common/render_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698