[go_router] fix Popping state and re-rendering scaffold at the same time doesn't update the URL on web [new] (#8352)

> the old pr #6953

Change the 3 files of lib and add a example, because issue flutter/flutter#150312

The reason for the issue is when call `counterStream.increment();` will notify `notifyListeners()` of `StreamListener`, and finally affect `_handleRouteInformationProviderNotification` of` _RouterState` in `packages/flutter/lib/src/widgets/router.dart`.

When pop should not call `_handleRouteInformationProviderNotification` otherwise will be path wrong.
because the value of `GoRouteInformationProvider` is not the latest.

## Solution:
always make the value of `GoRouteInformationProvider` knew the latest `RouteInformation`[but not to do `notifyListeners()`].

i don't know if adding `pop` to `NavigatingType` is reasonable, if any place needs to be changed please tell me.

## Result:
After changed, the issue been solved and all the others pop not be affect such as drawer/dialog/bottomsheet, See the video below.

> flutter/flutter#150312
> exempt from version changes
> this PR is exempt from CHANGELOG changes.
> this PR is test-exempt.
> the `go_route_test.dart` will error if `testWidgets('throw if redirect to itself')`[I have this problem when I pull the online branch. it's not relate this PR, All of the others test success, So I view this as all test cases successfully passing.]
```
Exception has occurred.
_AssertionError ('package:go_router/src/parser.dart': Failed assertion: line 110 pos 18: '!matchList.last.route.redirectOnly': A redirect-only route must redirect to location different from itself.
 The offending route: GoRoute#0ae4e(name: null, path: "route", Redirect Only))
 ```

Author: ahyangnb

packages/go_router/CHANGELOG.md

@@ -1,6 +1,10 @@
+## 15.2.1
+
+* Fixes Popping state and re-rendering scaffold at the same time doesn't update the URL on web.
+
 ## 15.2.0
 
-- `GoRouteData` now defines `.location`, `.go(context)`, `.push(context)`, `.pushReplacement(context)`, and `replace(context)` to be used for [Type-safe routing](https://pub.dev/documentation/go_router/latest/topics/Type-safe%20routes-topic.html). **Requires go_router_builder >= 3.0.0**.
+* `GoRouteData` now defines `.location`, `.go(context)`, `.push(context)`, `.pushReplacement(context)`, and `replace(context)` to be used for [Type-safe routing](https://pub.dev/documentation/go_router/latest/topics/Type-safe%20routes-topic.html). **Requires go_router_builder >= 3.0.0**.
 
 ## 15.1.3
 

packages/go_router/lib/src/router.dart

@@ -496,12 +496,16 @@ class GoRouter implements RouterConfig<RouteMatchList> {
   ///
   /// If the top-most route is a pop up or dialog, this method pops it instead
   /// of any GoRoute under it.
+  ///
+  /// Ensure that the `value` of `routeInformationProvider` is synced
+  ///  with `routerDelegate.currentConfiguration`.
   void pop<T extends Object?>([T? result]) {
     assert(() {
       log('popping ${routerDelegate.currentConfiguration.uri}');
       return true;
     }());
     routerDelegate.pop<T>(result);
+    restore(routerDelegate.currentConfiguration);
   }
 
   /// Refresh the route.

packages/go_router/pubspec.yaml

@@ -1,7 +1,7 @@
 name: go_router
 description: A declarative router for Flutter based on Navigation 2 supporting
   deep linking, data-driven routes and more
-version: 15.2.0
+version: 15.2.1
 repository: https://github.com/flutter/packages/tree/main/packages/go_router
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+go_router%22
 

packages/go_router/test/delegate_test.dart

@@ -78,6 +78,28 @@ Future<GoRouter> createGoRouterWithStatefulShellRoute(
 
 void main() {
   group('pop', () {
+    testWidgets('restore() update currentConfiguration in pop()',
+        (WidgetTester tester) async {
+      final ValueNotifier<int> valueNotifier = ValueNotifier<int>(0);
+      final GoRouter goRouter = await createGoRouter(tester,
+          refreshListenable: valueNotifier, dispose: false);
+
+      goRouter.push('/a');
+      await tester.pumpAndSettle();
+
+      goRouter.pop();
+      valueNotifier.notifyListeners();
+      await tester.pumpAndSettle();
+      expect(
+        goRouter
+            .routerDelegate.currentConfiguration.matches.last.matchedLocation,
+        '/',
+      );
+
+      addTearDown(valueNotifier.dispose);
+      addTearDown(goRouter.dispose);
+    });
+
     testWidgets('removes the last element', (WidgetTester tester) async {
       final GoRouter goRouter = await createGoRouter(tester)
         ..push('/error');