fix: Bind this to utils that moved from static to non-static with js api de-globalization (#1795)

I noticed this issue while investigating
deephaven/deephaven-plugins#179 and switching
a param of `(v) => this.chartUtils.unwrapValue(v)` to just
`this.chartUtils.unwrapValue`

Looked through the other de-globalization PRs and added `bindAllMethods`
to any classes where there were some static methods moved to non-static
so that they can be used this way

Author: mattrunyon

packages/chart/src/ChartUtils.ts

@@ -34,7 +34,7 @@ import type {
   OhlcData,
   MarkerSymbol,
 } from 'plotly.js';
-import { assertNotNull, Range } from '@deephaven/utils';
+import { assertNotNull, bindAllMethods, Range } from '@deephaven/utils';
 import { ChartTheme } from './ChartTheme';
 
 export type FilterColumnMap = Map<
@@ -548,6 +548,7 @@ class ChartUtils {
   constructor(dh: DhType) {
     this.dh = dh;
     this.daysOfWeek = Object.freeze(dh.calendar.DayOfWeek.values());
+    bindAllMethods(this);
   }
 
   /**

packages/iris-grid/src/IrisGrid.tsx

@@ -611,6 +611,7 @@ export class IrisGrid extends Component<IrisGridProps, IrisGridState> {
       this.handleGotoValueSelectedFilterChanged.bind(this);
     this.handleGotoValueChanged = this.handleGotoValueChanged.bind(this);
     this.handleGotoValueSubmitted = this.handleGotoValueSubmitted.bind(this);
+    this.makeQuickFilter = this.makeQuickFilter.bind(this);
 
     this.grid = null;
     this.lastLoadedConfig = null;

packages/iris-grid/src/IrisGridUtils.ts

@@ -28,7 +28,12 @@ import {
   FormattingRule,
 } from '@deephaven/jsapi-utils';
 import Log from '@deephaven/log';
-import { assertNotNull, EMPTY_ARRAY, EMPTY_MAP } from '@deephaven/utils';
+import {
+  assertNotNull,
+  bindAllMethods,
+  EMPTY_ARRAY,
+  EMPTY_MAP,
+} from '@deephaven/utils';
 import AggregationUtils from './sidebar/aggregations/AggregationUtils';
 import AggregationOperation from './sidebar/aggregations/AggregationOperation';
 import { FilterData, IrisGridProps, IrisGridState } from './IrisGrid';
@@ -1144,6 +1149,7 @@ class IrisGridUtils {
   constructor(dh: DhType) {
     this.dh = dh;
     this.tableUtils = new TableUtils(dh);
+    bindAllMethods(this);
   }
 
   /**

packages/iris-grid/src/mousehandlers/IrisGridContextMenuHandler.tsx

@@ -159,6 +159,11 @@ class IrisGridContextMenuHandler extends GridMouseHandler {
   constructor(irisGrid: IrisGrid, dh: DhType) {
     super();
 
+    this.getNumberValueEqualsFilter =
+      this.getNumberValueEqualsFilter.bind(this);
+    this.getFilterValueForNumberOrChar =
+      this.getFilterValueForNumberOrChar.bind(this);
+
     this.debouncedUpdateCustomFormat = debounce(
       irisGrid.handleFormatSelection,
       DEBOUNCE_UPDATE_FORMAT

packages/iris-grid/src/sidebar/ChartBuilder.tsx

@@ -16,6 +16,7 @@ import type {
   SeriesPlotStyle,
 } from '@deephaven/jsapi-types';
 import Log from '@deephaven/log';
+import { bindAllMethods } from '@deephaven/utils';
 import shortid from 'shortid';
 import {
   BarIcon,
@@ -105,15 +106,7 @@ class ChartBuilder extends PureComponent<ChartBuilderProps, ChartBuilderState> {
   constructor(props: ChartBuilderProps) {
     super(props);
 
-    this.handleAddSeries = this.handleAddSeries.bind(this);
-    this.handleLinkStateChange = this.handleLinkStateChange.bind(this);
-    this.handleReset = this.handleReset.bind(this);
-    this.handleSeriesChange = this.handleSeriesChange.bind(this);
-    this.handleSeriesDeleteClick = this.handleSeriesDeleteClick.bind(this);
-    this.handleSubmit = this.handleSubmit.bind(this);
-    this.handleTypeClick = this.handleTypeClick.bind(this);
-    this.handleXAxisChange = this.handleXAxisChange.bind(this);
-    this.sendChange = this.sendChange.bind(this);
+    bindAllMethods(this);
 
     const { model } = props;
     const { columns, dh } = model;