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

Unified Diff: chrome/renderer/chrome_render_thread_observer.cc

Issue 2048503002: Convert visitedlink to use mojo. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@message-mojom-magic
Patch Set: Created 4 years, 3 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
Index: chrome/renderer/chrome_render_thread_observer.cc
diff --git a/chrome/renderer/chrome_render_thread_observer.cc b/chrome/renderer/chrome_render_thread_observer.cc
index 486f01948734fb1041a0987c425809ccf6a110d8..32e1bceb687647e8cd7b246e22c5e48db2e6b03a 100644
--- a/chrome/renderer/chrome_render_thread_observer.cc
+++ b/chrome/renderer/chrome_render_thread_observer.cc
@@ -37,6 +37,7 @@
#include "chrome/common/url_constants.h"
#include "chrome/renderer/content_settings_observer.h"
#include "chrome/renderer/security_filter_peer.h"
+#include "components/visitedlink/renderer/visitedlink_slave.h"
#include "content/public/child/resource_dispatcher_delegate.h"
#include "content/public/renderer/render_thread.h"
#include "content/public/renderer/render_view.h"
@@ -231,7 +232,9 @@ void CreateResourceUsageReporter(
bool ChromeRenderThreadObserver::is_incognito_process_ = false;
ChromeRenderThreadObserver::ChromeRenderThreadObserver()
- : field_trial_syncer_(this), weak_factory_(this) {
+ : field_trial_syncer_(this),
+ visited_link_slave_(new visitedlink::VisitedLinkSlave),
+ weak_factory_(this) {
const base::CommandLine& command_line =
*base::CommandLine::ForCurrentProcess();
@@ -261,6 +264,10 @@ ChromeRenderThreadObserver::ChromeRenderThreadObserver()
WebSecurityPolicy::registerURLSchemeAsNoAccess(native_scheme);
WebSecurityPolicy::registerURLSchemeAsNotAllowingJavascriptURLs(
native_scheme);
+
+ thread->GetInterfaceRegistry()->AddInterface(
+ base::Bind(&visitedlink::VisitedLinkSlave::Bind,
+ base::Unretained(visited_link_slave_.get())));
}
ChromeRenderThreadObserver::~ChromeRenderThreadObserver() {}
@@ -279,6 +286,10 @@ bool ChromeRenderThreadObserver::OnControlMessageReceived(
return handled;
}
+void ChromeRenderThreadObserver::OnRenderProcessShutdown() {
+ visited_link_slave_.reset();
+}
+
void ChromeRenderThreadObserver::OnFieldTrialGroupFinalized(
const std::string& trial_name,
const std::string& group_name) {

Powered by Google App Engine
This is Rietveld 408576698