[Basilisk] Issue MoonchildProductions/UXP#516 - Remove named function syntax from basilisk/components/newtab.

This commit is contained in:
athenian200 2020-03-06 10:41:26 -06:00 • committed by Roy Tam
commit d546845b66

View file

@ -52,18 +52,18 @@ Links.prototype = {
* All history events are emitted from this object. * All history events are emitted from this object.
*/ */
historyObserver: { historyObserver: {
onDeleteURI: function historyObserver_onDeleteURI(aURI) { onDeleteURI: function(aURI) {
// let observers remove sensetive data associated with deleted visit // let observers remove sensetive data associated with deleted visit
gLinks.emit("deleteURI", { gLinks.emit("deleteURI", {
url: aURI.spec, url: aURI.spec,
}); });
}, },
onClearHistory: function historyObserver_onClearHistory() { onClearHistory: function() {
gLinks.emit("clearHistory"); gLinks.emit("clearHistory");
}, },
onFrecencyChanged: function historyObserver_onFrecencyChanged(aURI, onFrecencyChanged: function(aURI,
aNewFrecency, aGUID, aHidden, aLastVisitDate) { // jshint ignore:line aNewFrecency, aGUID, aHidden, aLastVisitDate) { // jshint ignore:line
// The implementation of the query in getLinks excludes hidden and // The implementation of the query in getLinks excludes hidden and
// unvisited pages, so it's important to exclude them here, too. // unvisited pages, so it's important to exclude them here, too.
@ -78,13 +78,13 @@ Links.prototype = {
} }
}, },
onManyFrecenciesChanged: function historyObserver_onManyFrecenciesChanged() { onManyFrecenciesChanged: function() {
// Called when frecencies are invalidated and also when clearHistory is called // Called when frecencies are invalidated and also when clearHistory is called
// See toolkit/components/places/tests/unit/test_frecency_observers.js // See toolkit/components/places/tests/unit/test_frecency_observers.js
gLinks.emit("manyLinksChanged"); gLinks.emit("manyLinksChanged");
}, },
onTitleChanged: function historyObserver_onTitleChanged(aURI, aNewTitle) { onTitleChanged: function(aURI, aNewTitle) {
if (NewTabUtils.linkChecker.checkLoadURI(aURI.spec)) { if (NewTabUtils.linkChecker.checkLoadURI(aURI.spec)) {
gLinks.emit("linkChanged", { gLinks.emit("linkChanged", {
url: aURI.spec, url: aURI.spec,
@ -101,7 +101,7 @@ Links.prototype = {
* Must be called before the provider is used. * Must be called before the provider is used.
* Makes it easy to disable under pref * Makes it easy to disable under pref
*/ */
init: function PlacesProvider_init() { init: function() {
try { try {
PlacesUtils.history.addObserver(this.historyObserver, true); PlacesUtils.history.addObserver(this.historyObserver, true);
} catch (e) { } catch (e) {