diff options
| -rw-r--r-- | bg/ExternalLicenses.js | 25 |
1 files changed, 12 insertions, 13 deletions
diff --git a/bg/ExternalLicenses.js b/bg/ExternalLicenses.js index 16ea52f..e37510a 100644 --- a/bg/ExternalLicenses.js +++ b/bg/ExternalLicenses.js @@ -86,34 +86,33 @@ const ExternalLicenses = { * Returns true if the document has been actually modified, false otherwise. */ optimizeDocument(doc, cachePointer) { - let cache = {}; - let { tabId, frameId, documentUrl } = cachePointer; - let frameCache = cachedHrefs.get(tabId); - if (!frameCache) { - cachedHrefs.set(tabId, frameCache = new Map()); - } + const cache = {}; + const { tabId, frameId, documentUrl } = cachePointer; + const frameCache = cachedHrefs.get(tabId) || new Map(); + cachedHrefs.set(tabId, frameCache); frameCache.set(frameId, new Map([[documentUrl, cache]])); - let link = doc.querySelector('link[rel="jslicense"], link[data-jslicense="1"], a[rel="jslicense"], a[data-jslicense="1"]'); + const link = doc.querySelector('link[rel="jslicense"], link[data-jslicense="1"], a[rel="jslicense"], a[data-jslicense="1"]'); if (link) { - let href = link.getAttribute('href'); + const href = link.getAttribute('href'); cache.webLabels = { href }; - let move = () => !!doc.head.insertBefore(link, doc.head.firstChild); + const move = (link) => !!doc.head.insertBefore(link, doc.head.firstChild); if (link.parentNode === doc.head) { + // TODO: eliminate let let node = link.previousElementSibling; for (; node; node = node.previousElementSibling) { if (node.tagName.toUpperCase() === 'SCRIPT') { - return move(); + return move(link); } } } else { // the reference is only in the body if (link.tagName.toUpperCase() === 'A') { - let newLink = doc.createElement('link'); + const newLink = doc.createElement('link'); newLink.rel = 'jslicense'; newLink.setAttribute('href', href); - link = newLink; + return move(newLink); } - return move(); + return move(link); } } |