Move the Map view of NamedNodeMap to its companion object.

So that it is always available.

We also make the implementation class private.

Author: sjrd

api-reports/2_12.txt

@@ -13460,6 +13460,7 @@ NamedNodeMap[JC] def removeNamedItemNS(namespaceURI: String, localName: String):
 NamedNodeMap[JC] def setNamedItem(arg: Attr): Attr
 NamedNodeMap[JC] def setNamedItemNS(arg: Attr): Attr
 NamedNodeMap[JC] @scala.scalajs.js.annotation.JSBracketAccess def update(index: Int, v: Attr): Unit
+NamedNodeMap[SO] implicit def namedNodeMapAsMap(namedNodeMap: NamedNodeMap): mutable.Map[String, Attr]
 Navigator[JC] def appName: String
 Navigator[JC] def appVersion: String
 Navigator[JC] def clipboard: Clipboard
@@ -24615,15 +24616,6 @@ ext/KeyValue.Multimedia[SO] final val SpellCheck = "SpellCheck"
 ext/KeyValue.Multimedia[SO] final val VolumeDown = "VolumeDown"
 ext/KeyValue.Multimedia[SO] final val VolumeMute = "VolumeMute"
 ext/KeyValue.Multimedia[SO] final val VolumeUp = "VolumeUp"
-ext/NamedNodeMapMap[SC] override def + = (kv: (String, Attr)): this.type
-ext/NamedNodeMapMap[SC] override def - = (key: String): this.type
-ext/NamedNodeMapMap[SC] override def clear(): Unit
-ext/NamedNodeMapMap[SC] override def get(key: String): Option[Attr]
-ext/NamedNodeMapMap[SC] override def hasNext: Boolean
-ext/NamedNodeMapMap[SC] override def iterator: Iterator[(String, Attr)]
-ext/NamedNodeMapMap[SC] def length: Int
-ext/NamedNodeMapMap[SC] override def next(): (String, Attr)
-ext/package[SO] implicit def pimpNamedNodeMap(namedNodeMap: NamedNodeMap): NamedNodeMapMap
 ext/package.PimpedHtmlCollection[SC] def apply(x: Int) = jsApply(x)
 ext/package.PimpedHtmlCollection[SC] def hasNext: scala.Boolean
 ext/package.PimpedHtmlCollection[SC] def iterator = new Iterator[T] { var index = 0 def hasNext: scala.Boolean = index < jsLength def next() = { val res = jsApply(index) index + = 1 res } }

api-reports/2_13.txt

@@ -13460,6 +13460,7 @@ NamedNodeMap[JC] def removeNamedItemNS(namespaceURI: String, localName: String):
 NamedNodeMap[JC] def setNamedItem(arg: Attr): Attr
 NamedNodeMap[JC] def setNamedItemNS(arg: Attr): Attr
 NamedNodeMap[JC] @scala.scalajs.js.annotation.JSBracketAccess def update(index: Int, v: Attr): Unit
+NamedNodeMap[SO] implicit def namedNodeMapAsMap(namedNodeMap: NamedNodeMap): mutable.Map[String, Attr]
 Navigator[JC] def appName: String
 Navigator[JC] def appVersion: String
 Navigator[JC] def clipboard: Clipboard
@@ -24615,15 +24616,6 @@ ext/KeyValue.Multimedia[SO] final val SpellCheck = "SpellCheck"
 ext/KeyValue.Multimedia[SO] final val VolumeDown = "VolumeDown"
 ext/KeyValue.Multimedia[SO] final val VolumeMute = "VolumeMute"
 ext/KeyValue.Multimedia[SO] final val VolumeUp = "VolumeUp"
-ext/NamedNodeMapMap[SC] override def addOne(kv: (String, Attr)): this.type
-ext/NamedNodeMapMap[SC] override def clear(): Unit
-ext/NamedNodeMapMap[SC] override def get(key: String): Option[Attr]
-ext/NamedNodeMapMap[SC] override def hasNext: Boolean
-ext/NamedNodeMapMap[SC] override def iterator: Iterator[(String, Attr)]
-ext/NamedNodeMapMap[SC] def length: Int
-ext/NamedNodeMapMap[SC] override def next(): (String, Attr)
-ext/NamedNodeMapMap[SC] override def subtractOne(key: String): this.type
-ext/package[SO] implicit def pimpNamedNodeMap(namedNodeMap: NamedNodeMap): NamedNodeMapMap
 ext/package.PimpedHtmlCollection[SC] def apply(x: Int) = jsApply(x)
 ext/package.PimpedHtmlCollection[SC] def hasNext: scala.Boolean
 ext/package.PimpedHtmlCollection[SC] def iterator = new Iterator[T] { var index = 0 def hasNext: scala.Boolean = index < jsLength def next() = { val res = jsApply(index) index + = 1 res } }

src/main/scala-new-collections/org/scalajs/dom/ext/NamedNodeMapMap.scala renamed to src/main/scala-new-collections/org/scalajs/dom/NamedNodeMapMap.scala

@@ -1,9 +1,8 @@
-package org.scalajs.dom.ext
+package org.scalajs.dom
 
-import org.scalajs.dom._
 import scala.collection.mutable
 
-class NamedNodeMapMap private[ext] (namedNodeMap: NamedNodeMap)
+private[dom] class NamedNodeMapMap(namedNodeMap: NamedNodeMap)
     extends mutable.Map[String, Attr] {
   self =>
 

src/main/scala-old-collections/org/scalajs/dom/ext/NamedNodeMapMap.scala renamed to src/main/scala-old-collections/org/scalajs/dom/NamedNodeMapMap.scala

@@ -1,9 +1,8 @@
-package org.scalajs.dom.ext
+package org.scalajs.dom
 
-import org.scalajs.dom._
 import scala.collection.mutable
 
-class NamedNodeMapMap private[ext] (namedNodeMap: NamedNodeMap)
+private[dom] class NamedNodeMapMap(namedNodeMap: NamedNodeMap)
     extends mutable.Map[String, Attr] {
   self =>
 

src/main/scala/org/scalajs/dom/ext/package.scala

@@ -1,7 +1,5 @@
 package org.scalajs.dom
 
-import scala.language.implicitConversions
-
 package object ext {
 
   implicit class PimpedNodeList(nodes: NodeList)
@@ -16,6 +14,4 @@ package object ext {
   implicit class PimpedSVGTransformList(coll: svg.TransformList)
       extends EasySeq[svg.Transform](coll.numberOfItems, coll.getItem)
 
-  implicit def pimpNamedNodeMap(namedNodeMap: NamedNodeMap): NamedNodeMapMap =
-    new NamedNodeMapMap(namedNodeMap)
 }

src/main/scala/org/scalajs/dom/lib.scala

@@ -10,6 +10,8 @@
 package org.scalajs.dom
 
 import org.scalajs.dom.experimental.ReadableStream
+import scala.collection.mutable
+import scala.language.implicitConversions
 import scala.scalajs.js
 import scala.scalajs.js.annotation._
 import scala.scalajs.js.typedarray.ArrayBuffer
@@ -6179,6 +6181,12 @@ class NamedNodeMap extends js.Object {
   def setNamedItemNS(arg: Attr): Attr = js.native
 }
 
+object NamedNodeMap {
+  implicit def namedNodeMapAsMap(
+      namedNodeMap: NamedNodeMap): mutable.Map[String, Attr] =
+    new NamedNodeMapMap(namedNodeMap)
+}
+
 @js.native
 @JSGlobal
 class MediaList extends js.Object {