refactor(selection): Unify tag name and selector arguments.

Author: jenstroeger

src/content.js

@@ -303,19 +303,17 @@ export function removeFormattingElem (host, range, elem) {
   })
 }
 
-export function removeFormatting (host, range, tagName, selector) {
+export function removeFormatting (host, range, selector) {
   return restoreRange(host, range, () => {
-    nuke(host, range, tagName, selector)
+    nuke(host, range, selector)
   })
 }
 
 // Unwrap all tags this range is affected by.
 // Can also affect content outside of the range.
-export function nuke (host, range, tagName, selector) {
+export function nuke (host, range, selector) {
   getTags(host, range).forEach((elem) => {
-    if (elem.nodeName.toUpperCase() !== 'BR' &&
-      (!tagName || (elem.nodeName.toUpperCase() === tagName.toUpperCase() &&
-        (!selector || elem.matches(selector))))) {
+    if (elem.nodeName.toUpperCase() !== 'BR' && (!selector || elem.matches(selector))) {
       unwrap(elem)
     }
   })

src/selection.js

@@ -203,11 +203,11 @@ export default class Selection extends Cursor {
     }
   }
 
-  // @param {String} tagName. E.g. 'a' to remove all links; if undefined, remove all.
-  // @param {String} selector. E.g. '.foo' or '[data-foo]' to remove all links 'a.foo'
-  //                           or 'a[data-foo]; if undefined, disregard the selector.
-  removeFormatting (tagName, selector) {
-    this.range = content.removeFormatting(this.host, this.range, tagName, selector)
+  // @param {String} selector. An element selector, e.g. 'a' or 'span.some-class'
+  //                           that represents elements to be removed; if undefined,
+  //                           remove all.
+  removeFormatting (selector) {
+    this.range = content.removeFormatting(this.host, this.range, selector)
     this.setSelection()
   }