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

Side by Side Diff: Source/devtools/front_end/host/InspectorFrontendHost.js

Issue 1061503002: DevTools: use recordEnumeratedHistogram instead of recordActionTaken (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: fix inspector/user-metrics.html Created 5 years, 8 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 /* 1 /*
2 * Copyright (C) 2009 Google Inc. All rights reserved. 2 * Copyright (C) 2009 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 199 matching lines...) Expand 10 before | Expand all | Expand 10 after
210 * @param {!FileSystem} fileSystem 210 * @param {!FileSystem} fileSystem
211 */ 211 */
212 upgradeDraggedFileSystemPermissions: function(fileSystem) { }, 212 upgradeDraggedFileSystemPermissions: function(fileSystem) { },
213 213
214 /** 214 /**
215 * @return {string} 215 * @return {string}
216 */ 216 */
217 platform: function() { }, 217 platform: function() { },
218 218
219 /** 219 /**
220 * @param {string} actionName
220 * @param {number} actionCode 221 * @param {number} actionCode
222 * @param {number} bucketSize
221 */ 223 */
222 recordActionTaken: function(actionCode) { }, 224 recordEnumeratedHistogram: function(actionName, actionCode, bucketSize) { },
223
224 /**
225 * @param {number} panelCode
226 */
227 recordPanelShown: function(panelCode) { },
228 225
229 /** 226 /**
230 * @param {string} message 227 * @param {string} message
231 */ 228 */
232 sendMessageToBackend: function(message) { }, 229 sendMessageToBackend: function(message) { },
233 230
234 /** 231 /**
235 * @param {boolean} enabled 232 * @param {boolean} enabled
236 */ 233 */
237 setDevicesUpdatesEnabled: function(enabled) { }, 234 setDevicesUpdatesEnabled: function(enabled) { },
(...skipping 205 matching lines...) Expand 10 before | Expand all | Expand 10 after
443 /** 440 /**
444 * @override 441 * @override
445 * @param {string} message 442 * @param {string} message
446 */ 443 */
447 sendMessageToBackend: function(message) 444 sendMessageToBackend: function(message)
448 { 445 {
449 }, 446 },
450 447
451 /** 448 /**
452 * @override 449 * @override
450 * @param {string} actionName
453 * @param {number} actionCode 451 * @param {number} actionCode
452 * @param {number} bucketSize
454 */ 453 */
455 recordActionTaken: function(actionCode) 454 recordEnumeratedHistogram: function(actionName, actionCode, bucketSize)
456 { 455 {
457 }, 456 },
458 457
459 /**
460 * @override
461 * @param {number} panelCode
462 */
463 recordPanelShown: function(panelCode)
464 {
465 },
466
467 /** 458 /**
468 * @override 459 * @override
469 */ 460 */
470 requestFileSystems: function() 461 requestFileSystems: function()
471 { 462 {
472 }, 463 },
473 464
474 /** 465 /**
475 * @override 466 * @override
476 */ 467 */
(...skipping 245 matching lines...) Expand 10 before | Expand all | Expand 10 after
722 // so the host instance should not initialized there. 713 // so the host instance should not initialized there.
723 initializeInspectorFrontendHost(); 714 initializeInspectorFrontendHost();
724 window.InspectorFrontendAPI = new InspectorFrontendAPIImpl(); 715 window.InspectorFrontendAPI = new InspectorFrontendAPIImpl();
725 if (!window.DevToolsHost) { 716 if (!window.DevToolsHost) {
726 WebInspector.setLocalizationPlatform(InspectorFrontendHost.platform()); 717 WebInspector.setLocalizationPlatform(InspectorFrontendHost.platform());
727 } else { 718 } else {
728 WebInspector.setLocalizationPlatform(DevToolsHost.platform()); 719 WebInspector.setLocalizationPlatform(DevToolsHost.platform());
729 } 720 }
730 721
731 })(); 722 })();
OLDNEW
« no previous file with comments | « Source/devtools/front_end/devtools_app/InspectorFrontendHostImpl.js ('k') | Source/devtools/front_end/host/UserMetrics.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698