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

Side by Side 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: remove the changes to web_ui_mojo.js which seemed to cause flakiness. Created 3 years, 12 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 'use strict'; 5 'use strict';
6 6
7 define('main', [ 7 define('main', [
8 'mojo/public/js/connection',
9 'chrome/browser/ui/webui/engagement/site_engagement.mojom', 8 'chrome/browser/ui/webui/engagement/site_engagement.mojom',
10 'content/public/renderer/frame_interfaces', 9 'content/public/renderer/frame_interfaces',
11 ], function(connection, siteEngagementMojom, frameInterfaces) { 10 ], function(siteEngagementMojom, frameInterfaces) {
12 return function() { 11 return function() {
13 var uiHandler = connection.bindHandleToProxy( 12 var uiHandler = new siteEngagementMojom.SiteEngagementUIHandlerPtr(
14 frameInterfaces.getInterface( 13 frameInterfaces.getInterface(
15 siteEngagementMojom.SiteEngagementUIHandler.name), 14 siteEngagementMojom.SiteEngagementUIHandler.name));
16 siteEngagementMojom.SiteEngagementUIHandler);
17 15
18 var engagementTableBody = $('engagement-table-body'); 16 var engagementTableBody = $('engagement-table-body');
19 var updateInterval = null; 17 var updateInterval = null;
20 var info = null; 18 var info = null;
21 var sortKey = 'score'; 19 var sortKey = 'score';
22 var sortReverse = true; 20 var sortReverse = true;
23 21
24 // Set table header sort handlers. 22 // Set table header sort handlers.
25 var engagementTableHeader = $('engagement-table-header'); 23 var engagementTableHeader = $('engagement-table-header');
26 var headers = engagementTableHeader.children; 24 var headers = engagementTableHeader.children;
(...skipping 140 matching lines...) Expand 10 before | Expand all | Expand 10 after
167 uiHandler.getSiteEngagementInfo().then(function(response) { 165 uiHandler.getSiteEngagementInfo().then(function(response) {
168 info = response.info; 166 info = response.info;
169 renderTable(info); 167 renderTable(info);
170 }); 168 });
171 }; 169 };
172 170
173 updateEngagementTable(); 171 updateEngagementTable();
174 enableAutoupdate(); 172 enableAutoupdate();
175 }; 173 };
176 }); 174 });
OLDNEW
« no previous file with comments | « chrome/browser/resources/bluetooth_internals/interfaces.js ('k') | chrome/browser/resources/omnibox/omnibox.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698