mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-23 10:15:41 +00:00
Bug 1381027 - Remove unnecessary async definitions for functions in editBookmarkOverlay.js. r=mak
MozReview-Commit-ID: HvSotonXYJM --HG-- extra : rebase_source : 990675e01c0da9f4c98f3a1708f308409100fa40
This commit is contained in:
parent
2ee3709256
commit
7564d07c58
@ -172,7 +172,7 @@ var gEditItemOverlay = {
|
||||
}
|
||||
},
|
||||
|
||||
async _initLoadInSidebar() {
|
||||
_initLoadInSidebar() {
|
||||
if (!this._paneInfo.isBookmark)
|
||||
throw new Error("_initLoadInSidebar called unexpectedly");
|
||||
|
||||
@ -262,9 +262,9 @@ var gEditItemOverlay = {
|
||||
|
||||
// Collapse the tag selector if the item does not accept tags.
|
||||
if (showOrCollapse("tagsRow", isURI || bulkTagging, "tags"))
|
||||
this._initTagsField().catch(Components.utils.reportError);
|
||||
this._initTagsField();
|
||||
else if (!this._element("tagsSelectorRow").collapsed)
|
||||
this.toggleTagsSelector().catch(Components.utils.reportError);
|
||||
this.toggleTagsSelector();
|
||||
|
||||
// Load in sidebar.
|
||||
if (showOrCollapse("loadInSidebarCheckbox", isBookmark, "loadInSidebar")) {
|
||||
@ -536,7 +536,7 @@ var gEditItemOverlay = {
|
||||
},
|
||||
|
||||
// Adds and removes tags for one or more uris.
|
||||
async _setTagsFromTagsInputField(aCurrentTags, aURIs) {
|
||||
_setTagsFromTagsInputField(aCurrentTags, aURIs) {
|
||||
let { removedTags, newTags } = this._getTagsChanges(aCurrentTags);
|
||||
if (removedTags.length + newTags.length == 0)
|
||||
return false;
|
||||
@ -582,7 +582,7 @@ var gEditItemOverlay = {
|
||||
let currentTags = this._paneInfo.bulkTagging ?
|
||||
await this._getCommonTags() :
|
||||
PlacesUtils.tagging.getTagsForURI(uris[0]);
|
||||
let anyChanges = await this._setTagsFromTagsInputField(currentTags, uris);
|
||||
let anyChanges = this._setTagsFromTagsInputField(currentTags, uris);
|
||||
if (!anyChanges)
|
||||
return false;
|
||||
|
||||
@ -928,7 +928,7 @@ var gEditItemOverlay = {
|
||||
value: aLastUsed ? new Date().getTime() : null };
|
||||
},
|
||||
|
||||
async _rebuildTagsSelectorList() {
|
||||
_rebuildTagsSelectorList() {
|
||||
let tagsSelector = this._element("tagsSelector");
|
||||
let tagsSelectorRow = this._element("tagsSelectorRow");
|
||||
if (tagsSelectorRow.collapsed)
|
||||
@ -971,7 +971,7 @@ var gEditItemOverlay = {
|
||||
}
|
||||
},
|
||||
|
||||
async toggleTagsSelector() {
|
||||
toggleTagsSelector() {
|
||||
var tagsSelector = this._element("tagsSelector");
|
||||
var tagsSelectorRow = this._element("tagsSelectorRow");
|
||||
var expander = this._element("tagsSelectorExpander");
|
||||
@ -980,7 +980,7 @@ var gEditItemOverlay = {
|
||||
expander.setAttribute("tooltiptext",
|
||||
expander.getAttribute("tooltiptextup"));
|
||||
tagsSelectorRow.collapsed = false;
|
||||
await this._rebuildTagsSelectorList();
|
||||
this._rebuildTagsSelectorList();
|
||||
|
||||
// This is a no-op if we've added the listener.
|
||||
tagsSelector.addEventListener("CheckboxStateChange", this);
|
||||
@ -1060,7 +1060,7 @@ var gEditItemOverlay = {
|
||||
}
|
||||
},
|
||||
|
||||
async _initTagsField() {
|
||||
_initTagsField() {
|
||||
let tags;
|
||||
if (this._paneInfo.isURI)
|
||||
tags = PlacesUtils.tagging.getTagsForURI(this._paneInfo.uri);
|
||||
@ -1099,10 +1099,10 @@ var gEditItemOverlay = {
|
||||
}
|
||||
|
||||
if (updateTagsField) {
|
||||
await this._initTagsField();
|
||||
this._initTagsField();
|
||||
// Any tags change should be reflected in the tags selector.
|
||||
if (this._element("tagsSelector")) {
|
||||
await this._rebuildTagsSelectorList();
|
||||
this._rebuildTagsSelectorList();
|
||||
}
|
||||
}
|
||||
},
|
||||
|
Loading…
Reference in New Issue
Block a user