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

Unified Diff: chrome/browser/resources/engagement/site_engagement.js

Issue 2571903003: Mojo JS bindings: switch most usage of "connection"/"router" module to "bindings". (Closed)
Patch Set: Merge branch 'refs/heads/c161_more_bindings_change' into c160_remove_connection_mojo Created 4 years 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/browser/resources/engagement/site_engagement.js
diff --git a/chrome/browser/resources/engagement/site_engagement.js b/chrome/browser/resources/engagement/site_engagement.js
index bd19f119996ea7905d96102ea8eed87ffa498e64..3557fd03845df0763a44ed268928e54671403103 100644
--- a/chrome/browser/resources/engagement/site_engagement.js
+++ b/chrome/browser/resources/engagement/site_engagement.js
@@ -5,15 +5,13 @@
'use strict';
define('main', [
- 'mojo/public/js/connection',
'chrome/browser/ui/webui/engagement/site_engagement.mojom',
'content/public/renderer/frame_interfaces',
-], function(connection, siteEngagementMojom, frameInterfaces) {
+], function(siteEngagementMojom, frameInterfaces) {
return function() {
- var uiHandler = connection.bindHandleToProxy(
+ var uiHandler = new siteEngagementMojom.SiteEngagementUIHandlerPtr(
frameInterfaces.getInterface(
- siteEngagementMojom.SiteEngagementUIHandler.name),
- siteEngagementMojom.SiteEngagementUIHandler);
+ siteEngagementMojom.SiteEngagementUIHandler.name));
var engagementTableBody = $('engagement-table-body');
var updateInterval = null;

Powered by Google App Engine
This is Rietveld 408576698