Reland Resolve breaking change of adding a method to ChangeNotifier. (flutter#134983)

polina-c · web-flow · commit ab66f55728d9 · 2023-09-18T20:31:54.000-07:00
diff --git a/packages/flutter/lib/src/foundation/change_notifier.dart b/packages/flutter/lib/src/foundation/change_notifier.dart
@@ -207,7 +207,7 @@ mixin class ChangeNotifier implements Listenable {
   @protected
   bool get hasListeners => _count > 0;
 
-  /// Dispatches event of object creation to [MemoryAllocations.instance].
+  /// Dispatches event of the [object] creation to [MemoryAllocations.instance].
   ///
   /// If the event was already dispatched or [kFlutterMemoryAllocationsEnabled]
   /// is false, the method is noop.
@@ -227,16 +227,16 @@ mixin class ChangeNotifier implements Listenable {
   /// Make sure to invoke it with condition `if (kFlutterMemoryAllocationsEnabled) ...`
   /// so that the method is tree-shaken away when the flag is false.
   @protected
-  void maybeDispatchObjectCreation() {
+  static void maybeDispatchObjectCreation(ChangeNotifier object) {
     // Tree shaker does not include this method and the class MemoryAllocations
     // if kFlutterMemoryAllocationsEnabled is false.
-    if (kFlutterMemoryAllocationsEnabled && !_creationDispatched) {
+    if (kFlutterMemoryAllocationsEnabled && !object._creationDispatched) {
       MemoryAllocations.instance.dispatchObjectCreated(
         library: _flutterFoundationLibrary,
         className: '$ChangeNotifier',
-        object: this,
+        object: object,
       );
-      _creationDispatched = true;
+      object._creationDispatched = true;
     }
   }
 
@@ -271,7 +271,7 @@ mixin class ChangeNotifier implements Listenable {
     assert(ChangeNotifier.debugAssertNotDisposed(this));
 
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      maybeDispatchObjectCreation(this);
     }
 
     if (_count == _listeners.length) {
@@ -535,7 +535,7 @@ class ValueNotifier<T> extends ChangeNotifier implements ValueListenable<T> {
   /// Creates a [ChangeNotifier] that wraps this value.
   ValueNotifier(this._value) {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/lib/src/rendering/paragraph.dart b/packages/flutter/lib/src/rendering/paragraph.dart
@@ -1321,7 +1321,7 @@ class _SelectableFragment with Selectable, ChangeNotifier implements TextLayoutM
     required this.range,
   }) : assert(range.isValid && !range.isCollapsed && range.isNormalized) {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
     _selectionGeometry = _getSelectionGeometry();
   }
diff --git a/packages/flutter/lib/src/services/restoration.dart b/packages/flutter/lib/src/services/restoration.dart
@@ -155,7 +155,7 @@ class RestorationManager extends ChangeNotifier {
   /// with the engine to get restoration messages (by calling [initChannels]).
   RestorationManager() {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
     initChannels();
   }
diff --git a/packages/flutter/lib/src/widgets/draggable_scrollable_sheet.dart b/packages/flutter/lib/src/widgets/draggable_scrollable_sheet.dart
@@ -1074,7 +1074,7 @@ class _DraggableScrollableActuatorState extends State<DraggableScrollableActuato
 class _ResetNotifier extends ChangeNotifier {
   _ResetNotifier() {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
   /// Whether someone called [sendReset] or not.
diff --git a/packages/flutter/lib/src/widgets/focus_manager.dart b/packages/flutter/lib/src/widgets/focus_manager.dart
@@ -439,7 +439,7 @@ class FocusNode with DiagnosticableTreeMixin, ChangeNotifier {
     this.debugLabel = debugLabel;
 
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
@@ -1468,7 +1468,7 @@ class FocusManager with DiagnosticableTreeMixin, ChangeNotifier {
   /// documentation in that method for caveats to watch out for.
   FocusManager() {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
     rootScope._manager = this;
   }
diff --git a/packages/flutter/lib/src/widgets/focus_traversal.dart b/packages/flutter/lib/src/widgets/focus_traversal.dart
@@ -1788,7 +1788,7 @@ class _FocusTraversalGroupNode extends FocusNode {
     required this.policy,
   }) {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/lib/src/widgets/navigator.dart b/packages/flutter/lib/src/widgets/navigator.dart
@@ -3390,7 +3390,7 @@ class _History extends Iterable<_RouteEntry> with ChangeNotifier {
   /// Creates an instance of [_History].
   _History() {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/lib/src/widgets/restoration.dart b/packages/flutter/lib/src/widgets/restoration.dart
@@ -457,7 +457,7 @@ abstract class RestorableProperty<T> extends ChangeNotifier {
   /// Creates a [RestorableProperty].
   RestorableProperty(){
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/lib/src/widgets/router.dart b/packages/flutter/lib/src/widgets/router.dart
@@ -1467,7 +1467,7 @@ class PlatformRouteInformationProvider extends RouteInformationProvider with Wid
     required RouteInformation initialRouteInformation,
   }) : _value = initialRouteInformation {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/lib/src/widgets/scroll_controller.dart b/packages/flutter/lib/src/widgets/scroll_controller.dart
@@ -65,7 +65,7 @@ class ScrollController extends ChangeNotifier {
     this.onDetach,
   }) : _initialScrollOffset = initialScrollOffset {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/lib/src/widgets/selectable_region.dart b/packages/flutter/lib/src/widgets/selectable_region.dart
@@ -1580,7 +1580,7 @@ abstract class MultiSelectableSelectionContainerDelegate extends SelectionContai
   /// Creates an instance of [MultiSelectableSelectionContainerDelegate].
   MultiSelectableSelectionContainerDelegate() {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/lib/src/widgets/shortcuts.dart b/packages/flutter/lib/src/widgets/shortcuts.dart
@@ -749,7 +749,7 @@ class ShortcutManager with Diagnosticable, ChangeNotifier {
     this.modal = false,
   })  : _shortcuts = shortcuts {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
@@ -1203,7 +1203,7 @@ class ShortcutRegistry with ChangeNotifier {
   /// Creates an instance of [ShortcutRegistry].
   ShortcutRegistry() {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/lib/src/widgets/snapshot_widget.dart b/packages/flutter/lib/src/widgets/snapshot_widget.dart
@@ -482,8 +482,4 @@ class _DefaultSnapshotPainter implements SnapshotPainter {
 
   @override
   bool shouldRepaint(covariant _DefaultSnapshotPainter oldPainter) => false;
-
-  @override
-  @protected
-  void maybeDispatchObjectCreation() { }
 }
diff --git a/packages/flutter/lib/src/widgets/widget_inspector.dart b/packages/flutter/lib/src/widgets/widget_inspector.dart
@@ -2898,7 +2898,7 @@ class InspectorSelection with ChangeNotifier {
   /// Creates an instance of [InspectorSelection].
   InspectorSelection() {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/test/foundation/change_notifier_test.dart b/packages/flutter/test/foundation/change_notifier_test.dart
@@ -29,7 +29,7 @@ class A {
 class B extends A with ChangeNotifier {
   B() {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
@@ -43,7 +43,7 @@ class B extends A with ChangeNotifier {
 class Counter with ChangeNotifier {
   Counter() {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/test/material/app_test.dart b/packages/flutter/test/material/app_test.dart
@@ -1570,7 +1570,7 @@ class SimpleNavigatorRouterDelegate extends RouterDelegate<RouteInformation> wit
     required this.builder,
     required this.onPopPage,
   }) {
-    maybeDispatchObjectCreation();
+    ChangeNotifier.maybeDispatchObjectCreation(this);
   }
 
   @override
diff --git a/packages/flutter/test/widgets/automatic_keep_alive_test.dart b/packages/flutter/test/widgets/automatic_keep_alive_test.dart
@@ -638,7 +638,7 @@ class RenderSliverMultiBoxAdaptorAlt extends RenderSliver with
 class LeakCheckerHandle with ChangeNotifier {
   LeakCheckerHandle() {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/test/widgets/route_notification_messages_test.dart b/packages/flutter/test/widgets/route_notification_messages_test.dart
@@ -340,7 +340,7 @@ class SimpleRouterDelegate extends RouterDelegate<RouteInformation> with ChangeN
     this.reportConfiguration = false,
   }) {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/test/widgets/router_restoration_test.dart b/packages/flutter/test/widgets/router_restoration_test.dart
@@ -92,7 +92,7 @@ class _TestRouteInformationParser extends RouteInformationParser<String> {
 class _TestRouterDelegate extends RouterDelegate<String> with ChangeNotifier {
   _TestRouterDelegate() {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
@@ -136,7 +136,7 @@ class _TestRouterDelegate extends RouterDelegate<String> with ChangeNotifier {
 class _TestRouteInformationProvider extends RouteInformationProvider with ChangeNotifier {
   _TestRouteInformationProvider() {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
diff --git a/packages/flutter/test/widgets/router_test.dart b/packages/flutter/test/widgets/router_test.dart
@@ -1646,7 +1646,7 @@ class SimpleRouterDelegate extends RouterDelegate<RouteInformation> with ChangeN
     this.reportConfiguration = false,
   }) {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
@@ -1734,7 +1734,7 @@ class SimpleRouteInformationProvider extends RouteInformationProvider with Chang
     this.onRouterReport,
   }) {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 
@@ -1794,7 +1794,7 @@ class SimpleAsyncRouterDelegate extends RouterDelegate<RouteInformation> with Ch
     required this.builder,
   }) {
     if (kFlutterMemoryAllocationsEnabled) {
-      maybeDispatchObjectCreation();
+      ChangeNotifier.maybeDispatchObjectCreation(this);
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -1321,7 +1321,7 @@ class _SelectableFragment with Selectable, ChangeNotifier implements TextLayoutM`
`1321`	`1321`	`required this.range,`
`1322`	`1322`	`}) : assert(range.isValid && !range.isCollapsed && range.isNormalized) {`
`1323`	`1323`	`if (kFlutterMemoryAllocationsEnabled) {`
`1324`		`- maybeDispatchObjectCreation();`
	`1324`	`+ ChangeNotifier.maybeDispatchObjectCreation(this);`
`1325`	`1325`	`}`
`1326`	`1326`	`_selectionGeometry = _getSelectionGeometry();`
`1327`	`1327`	`}`
Original file line number	Diff line number	Diff line change
`@@ -155,7 +155,7 @@ class RestorationManager extends ChangeNotifier {`
`155`	`155`	`/// with the engine to get restoration messages (by calling [initChannels]).`
`156`	`156`	`RestorationManager() {`
`157`	`157`	`if (kFlutterMemoryAllocationsEnabled) {`
`158`		`- maybeDispatchObjectCreation();`
	`158`	`+ ChangeNotifier.maybeDispatchObjectCreation(this);`
`159`	`159`	`}`
`160`	`160`	`initChannels();`
`161`	`161`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1074,7 +1074,7 @@ class _DraggableScrollableActuatorState extends State<DraggableScrollableActuato`
`1074`	`1074`	`class _ResetNotifier extends ChangeNotifier {`
`1075`	`1075`	`_ResetNotifier() {`
`1076`	`1076`	`if (kFlutterMemoryAllocationsEnabled) {`
`1077`		`- maybeDispatchObjectCreation();`
	`1077`	`+ ChangeNotifier.maybeDispatchObjectCreation(this);`
`1078`	`1078`	`}`
`1079`	`1079`	`}`
`1080`	`1080`	`/// Whether someone called [sendReset] or not.`
Original file line number	Diff line number	Diff line change
`@@ -439,7 +439,7 @@ class FocusNode with DiagnosticableTreeMixin, ChangeNotifier {`
`439`	`439`	`this.debugLabel = debugLabel;`
`440`	`440`
`441`	`441`	`if (kFlutterMemoryAllocationsEnabled) {`
`442`		`- maybeDispatchObjectCreation();`
	`442`	`+ ChangeNotifier.maybeDispatchObjectCreation(this);`
`443`	`443`	`}`
`444`	`444`	`}`
`445`	`445`
`@@ -1468,7 +1468,7 @@ class FocusManager with DiagnosticableTreeMixin, ChangeNotifier {`
`1468`	`1468`	`/// documentation in that method for caveats to watch out for.`
`1469`	`1469`	`FocusManager() {`
`1470`	`1470`	`if (kFlutterMemoryAllocationsEnabled) {`
`1471`		`- maybeDispatchObjectCreation();`
	`1471`	`+ ChangeNotifier.maybeDispatchObjectCreation(this);`
`1472`	`1472`	`}`
`1473`	`1473`	`rootScope._manager = this;`
`1474`	`1474`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1788,7 +1788,7 @@ class _FocusTraversalGroupNode extends FocusNode {`
`1788`	`1788`	`required this.policy,`
`1789`	`1789`	`}) {`
`1790`	`1790`	`if (kFlutterMemoryAllocationsEnabled) {`
`1791`		`- maybeDispatchObjectCreation();`
	`1791`	`+ ChangeNotifier.maybeDispatchObjectCreation(this);`
`1792`	`1792`	`}`
`1793`	`1793`	`}`
`1794`	`1794`
Original file line number	Diff line number	Diff line change
`@@ -3390,7 +3390,7 @@ class _History extends Iterable<_RouteEntry> with ChangeNotifier {`
`3390`	`3390`	`/// Creates an instance of [_History].`
`3391`	`3391`	`_History() {`
`3392`	`3392`	`if (kFlutterMemoryAllocationsEnabled) {`
`3393`		`- maybeDispatchObjectCreation();`
	`3393`	`+ ChangeNotifier.maybeDispatchObjectCreation(this);`
`3394`	`3394`	`}`
`3395`	`3395`	`}`
`3396`	`3396`
Original file line number	Diff line number	Diff line change
`@@ -457,7 +457,7 @@ abstract class RestorableProperty<T> extends ChangeNotifier {`
`457`	`457`	`/// Creates a [RestorableProperty].`
`458`	`458`	`RestorableProperty(){`
`459`	`459`	`if (kFlutterMemoryAllocationsEnabled) {`
`460`		`- maybeDispatchObjectCreation();`
	`460`	`+ ChangeNotifier.maybeDispatchObjectCreation(this);`
`461`	`461`	`}`
`462`	`462`	`}`
`463`	`463`
Original file line number	Diff line number	Diff line change
`@@ -1467,7 +1467,7 @@ class PlatformRouteInformationProvider extends RouteInformationProvider with Wid`
`1467`	`1467`	`required RouteInformation initialRouteInformation,`
`1468`	`1468`	`}) : _value = initialRouteInformation {`
`1469`	`1469`	`if (kFlutterMemoryAllocationsEnabled) {`
`1470`		`- maybeDispatchObjectCreation();`
	`1470`	`+ ChangeNotifier.maybeDispatchObjectCreation(this);`
`1471`	`1471`	`}`
`1472`	`1472`	`}`
`1473`	`1473`
Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ class ScrollController extends ChangeNotifier {`
`65`	`65`	`this.onDetach,`
`66`	`66`	`}) : _initialScrollOffset = initialScrollOffset {`
`67`	`67`	`if (kFlutterMemoryAllocationsEnabled) {`
`68`		`- maybeDispatchObjectCreation();`
	`68`	`+ ChangeNotifier.maybeDispatchObjectCreation(this);`
`69`	`69`	`}`
`70`	`70`	`}`
`71`	`71`