Remove the _AlwaysSelectable const constructor since it is always stored as a Selectable<T> and the generic type isn't available as a const.

nshahan · nshahan · commit 177f996e40a9 · 2018-05-18T15:41:32.000-07:00
Fix run time type error exposed by dart2js when compiling with Dart2 semantics. Does not yet appear in DDC because of a known issue. dart-lang/sdk#32918 PiperOrigin-RevId: 197071245
diff --git a/lib/src/material_tree/material_tree_node.dart b/lib/src/material_tree/material_tree_node.dart
@@ -55,7 +55,7 @@ class MaterialTreeNode<T> {
     if (_root.options is Selectable) {
       _selectable = _root.options as Selectable<T>;
     } else {
-      _selectable = const _AlwaysSelectable();
+      _selectable = new _AlwaysSelectable<T>();
     }
   }
 
@@ -284,8 +284,6 @@ class MaterialTreeNode<T> {
 
 // TODO(google): Remove once we switch over Selectable interfaces.
 class _AlwaysSelectable<T> implements Selectable<T> {
-  const _AlwaysSelectable();
-
   @override
   SelectableOption getSelectable(T item) => SelectableOption.Selectable;
 }

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ class MaterialTreeNode<T> {`
`55`	`55`	`if (_root.options is Selectable) {`
`56`	`56`	`_selectable = _root.options as Selectable<T>;`
`57`	`57`	`} else {`
`58`		`- _selectable = const _AlwaysSelectable();`
	`58`	`+ _selectable = new _AlwaysSelectable<T>();`
`59`	`59`	`}`
`60`	`60`	`}`
`61`	`61`
`@@ -284,8 +284,6 @@ class MaterialTreeNode<T> {`
`284`	`284`
`285`	`285`	`// TODO(google): Remove once we switch over Selectable interfaces.`
`286`	`286`	`class _AlwaysSelectable<T> implements Selectable<T> {`
`287`		`- const _AlwaysSelectable();`
`288`		`-`
`289`	`287`	`@override`
`290`	`288`	`SelectableOption getSelectable(T item) => SelectableOption.Selectable;`
`291`	`289`	`}`