refactor(multiple): underscore register methods for internal usage only

Author: adolgachev

src/aria/tabs/tabs.ts

@@ -96,7 +96,7 @@ export class Tabs {
     [...this._unorderedPanels()].map(tabpanel => tabpanel._pattern),
   );
 
-  register(child: TabList | TabPanel) {
+  _register(child: TabList | TabPanel) {
     if (child instanceof TabList) {
       this._tablist.set(child);
     }
@@ -107,7 +107,7 @@ export class Tabs {
     }
   }
 
-  deregister(child: TabList | TabPanel) {
+  _unregister(child: TabList | TabPanel) {
     if (child instanceof TabList) {
       this._tablist.set(undefined);
     }
@@ -240,19 +240,19 @@ export class TabList implements OnInit, OnDestroy {
   }
 
   ngOnInit() {
-    this._tabs.register(this);
+    this._tabs._register(this);
   }
 
   ngOnDestroy() {
-    this._tabs.deregister(this);
+    this._tabs._unregister(this);
   }
 
-  register(child: Tab) {
+  _register(child: Tab) {
     this._unorderedTabs().add(child);
     this._unorderedTabs.set(new Set(this._unorderedTabs()));
   }
 
-  deregister(child: Tab) {
+  _unregister(child: Tab) {
     this._unorderedTabs().delete(child);
     this._unorderedTabs.set(new Set(this._unorderedTabs()));
   }
@@ -341,11 +341,11 @@ export class Tab implements HasElement, OnInit, OnDestroy {
   }
 
   ngOnInit() {
-    this._tabList.register(this);
+    this._tabList._register(this);
   }
 
   ngOnDestroy() {
-    this._tabList.deregister(this);
+    this._tabList._unregister(this);
   }
 }
 
@@ -420,11 +420,11 @@ export class TabPanel implements OnInit, OnDestroy {
   }
 
   ngOnInit() {
-    this._Tabs.register(this);
+    this._Tabs._register(this);
   }
 
   ngOnDestroy() {
-    this._Tabs.deregister(this);
+    this._Tabs._unregister(this);
   }
 }
 

src/aria/toolbar/toolbar.ts

@@ -141,15 +141,15 @@ export class Toolbar<V> {
     this._hasBeenFocused.set(true);
   }
 
-  register(widget: ToolbarWidget<V>) {
+  _register(widget: ToolbarWidget<V>) {
     const widgets = this._widgets();
     if (!widgets.has(widget)) {
       widgets.add(widget);
       this._widgets.set(new Set(widgets));
     }
   }
 
-  unregister(widget: ToolbarWidget<V>) {
+  _unregister(widget: ToolbarWidget<V>) {
     const widgets = this._widgets();
     if (widgets.delete(widget)) {
       this._widgets.set(new Set(widgets));
@@ -236,11 +236,11 @@ export class ToolbarWidget<V> implements OnInit, OnDestroy {
   });
 
   ngOnInit() {
-    this._toolbar.register(this);
+    this._toolbar._register(this);
   }
 
   ngOnDestroy() {
-    this._toolbar.unregister(this);
+    this._toolbar._unregister(this);
   }
 }
 

src/aria/tree/tree.ts

@@ -224,12 +224,12 @@ export class Tree<V> {
     this._hasFocused.set(true);
   }
 
-  register(child: TreeItem<V>) {
+  _register(child: TreeItem<V>) {
     this._unorderedItems().add(child);
     this._unorderedItems.set(new Set(this._unorderedItems()));
   }
 
-  unregister(child: TreeItem<V>) {
+  _unregister(child: TreeItem<V>) {
     this._unorderedItems().delete(child);
     this._unorderedItems.set(new Set(this._unorderedItems()));
   }
@@ -349,8 +349,8 @@ export class TreeItem<V> extends DeferredContentAware implements OnInit, OnDestr
   }
 
   ngOnInit() {
-    this.parent().register(this);
-    this.tree().register(this);
+    this.parent()._register(this);
+    this.tree()._register(this);
 
     const treePattern = computed(() => this.tree()._pattern);
     const parentPattern = computed(() => {
@@ -371,15 +371,15 @@ export class TreeItem<V> extends DeferredContentAware implements OnInit, OnDestr
   }
 
   ngOnDestroy() {
-    this.parent().unregister(this);
-    this.tree().unregister(this);
+    this.parent()._unregister(this);
+    this.tree()._unregister(this);
   }
 
-  register(group: TreeItemGroup<V>) {
+  _register(group: TreeItemGroup<V>) {
     this._group.set(group);
   }
 
-  unregister() {
+  _unregister() {
     this._group.set(undefined);
   }
 }
@@ -432,19 +432,19 @@ export class TreeItemGroup<V> implements OnInit, OnDestroy {
 
   ngOnInit() {
     this._deferredContent.deferredContentAware.set(this.ownedBy());
-    this.ownedBy().register(this);
+    this.ownedBy()._register(this);
   }
 
   ngOnDestroy() {
-    this.ownedBy().unregister();
+    this.ownedBy()._unregister();
   }
 
-  register(child: TreeItem<V>) {
+  _register(child: TreeItem<V>) {
     this._unorderedItems().add(child);
     this._unorderedItems.set(new Set(this._unorderedItems()));
   }
 
-  unregister(child: TreeItem<V>) {
+  _unregister(child: TreeItem<V>) {
     this._unorderedItems().delete(child);
     this._unorderedItems.set(new Set(this._unorderedItems()));
   }