Bug 1456589 - Part 8 Remove unused telemetry calls and an accidently checked in file from SeaMonkey places. r=IanN

This commit is contained in:
Frank-Rainer Grahl 2018-07-23 22:14:42 +02:00
Родитель 47ba233f4a
Коммит d3d6ff14b0
4 изменённых файлов: 1 добавлений и 36 удалений

Просмотреть файл

@ -1,20 +0,0 @@
Check:
comm-central/mozilla/browser/base/content/nsContextMenu.js
suite/common/nsContextMenu.js
addBookmarkForFrame: function CM_addBookmarkForFrame() {
Not ported yet:
Bug 1401238 - Deprecate Recent Bookmarks, and Pocket List from the Bookmarks Menu.
https://bugzilla.mozilla.org/show_bug.cgi?id=1401238
Non working:
Load in sidebar disabled because of:
Timestamp: 1/29/2018, 9:56:54 PM
Error: TypeError: browserWin.openWebPanel is not a function
Source File: resource:///modules/PlacesUIUtils.jsm
Line: 1053

Просмотреть файл

@ -994,9 +994,6 @@ function PlacesToolbar(aPlace) {
}
PlacesViewBase.call(this, aPlace);
Services.telemetry.getHistogramById("FX_BOOKMARKS_TOOLBAR_INIT_MS")
.add(Date.now() - startTime);
}
PlacesToolbar.prototype = {

Просмотреть файл

@ -3,8 +3,6 @@
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */
ChromeUtils.import("resource://gre/modules/TelemetryStopwatch.jsm");
var gHistoryTree;
var gSearchBox;
var gHistoryGrouping = "";
@ -78,16 +76,10 @@ function searchHistory(aInput) {
options.resultType = resultType;
options.includeHidden = !!aInput;
if (gHistoryGrouping == "lastvisited")
this.TelemetryStopwatch.start("HISTORY_LASTVISITED_TREE_QUERY_TIME_MS");
// call load() on the tree manually
// instead of setting the place attribute in history-panel.xul
// otherwise, we will end up calling load() twice
gHistoryTree.load([query], options);
if (gHistoryGrouping == "lastvisited")
this.TelemetryStopwatch.finish("HISTORY_LASTVISITED_TREE_QUERY_TIME_MS");
}
window.addEventListener("SidebarFocused",

Просмотреть файл

@ -8,7 +8,6 @@
ChromeUtils.import("resource://gre/modules/AppConstants.jsm");
ChromeUtils.import("resource://gre/modules/XPCOMUtils.jsm");
ChromeUtils.import("resource://gre/modules/TelemetryStopwatch.jsm");
ChromeUtils.import("resource://gre/modules/Services.jsm");
ChromeUtils.defineModuleGetter(this, "MigrationUtils",
@ -23,7 +22,6 @@ ChromeUtils.defineModuleGetter(this, "OS",
"resource://gre/modules/osfile.jsm");
const RESTORE_FILEPICKER_FILTER_EXT = "*.json;*.jsonlz4";
const HISTORY_LIBRARY_SEARCH_TELEMETRY = "PLACES_HISTORY_LIBRARY_SEARCH_TIME_MS";
var PlacesOrganizer = {
_places: null,
@ -377,7 +375,7 @@ var PlacesOrganizer = {
* cookies, history, preferences, and bookmarks.
*/
importFromBrowser: function PO_importFromBrowser() {
// We pass in the type of source we're using for use in telemetry:
// We pass in the type of source we're using for future use:
MigrationUtils.showMigrationWizard(window, [MigrationUtils.MIGRATION_ENTRYPOINT_PLACES]);
},
@ -812,9 +810,7 @@ var PlacesSearchBox = {
options.includeHidden = true;
currentView.load([query], options);
} else {
TelemetryStopwatch.start(HISTORY_LIBRARY_SEARCH_TELEMETRY);
currentView.applyFilter(filterString, null, true);
TelemetryStopwatch.finish(HISTORY_LIBRARY_SEARCH_TELEMETRY);
}
break;
}